Skip to content

fix: resume automatic onboarding kickoff - #2749

Merged
superdav42 merged 1 commit into
mainfrom
feature/auto-20260921-112111
Sep 21, 2026
Merged

superdav42 merged 1 commit into
mainfrom
feature/auto-20260921-112111

Conversation

@superdav42

@superdav42 superdav42 commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • recover a persisted empty Setup Assistant session and submit its automatic kickoff
  • avoid duplicate kickoff messages when the bootstrap session already contains conversation history
  • suppress generic empty-state suggestion cards while automatic onboarding owns the first turn
  • document the automatic onboarding interaction contract

Verification

  • pnpm run verify
  • pnpm run test:js -- src/floating-widget/__tests__/frontend-onboarding.test.js --runInBand
  • pnpm run lint:js
  • pnpm run build
  • pnpm run check:bundle
  • focused OnboardingManagerTest PHPUnit coverage
  • git diff --check

Implementation context

The server derives kickoff_required from the persisted bootstrap-session messages. The frontend always asks the onboarding endpoint to open or recover that session, sends the neutral kickoff only when required, and keeps generic suggestion cards hidden during automatic startup.


aidevops.sh v3.32.317 plugin for OpenCode v1.18.31 with gpt-5.6-sol spent 5d 19h and 15,052,079 tokens on this with the user in an interactive session.

Summary by CodeRabbit

  • New Features

    • Automatic onboarding now starts after providers and sessions are ready.
    • New onboarding sessions receive a kickoff message automatically.
    • Resumed sessions only receive a kickoff when they contain no messages.
  • Bug Fixes

    • Prevented generic greetings and suggestion cards from briefly appearing during onboarding.
    • Avoided duplicate kickoff messages when reopening an already initialized session.
    • Improved recovery behavior when onboarding resumes an existing session.

@superdav42 superdav42 added the origin:interactive Created by interactive user session label Sep 21, 2026
@coderabbitai

coderabbitai Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: dd528f5f-3513-4664-97d3-c422ca038beb

📥 Commits

Reviewing files that changed from the base of the PR and between b5a50c8 and 33f4a8e.

📒 Files selected for processing (7)
  • DESIGN.md
  • includes/Core/OnboardingManager.php
  • src/components/chat-widget/widget-panel.js
  • src/floating-widget/__tests__/frontend-onboarding.test.js
  • src/floating-widget/frontend-onboarding.js
  • src/floating-widget/index.js
  • tests/SdAiAgent/Core/OnboardingManagerTest.php
 ____________________________
< The OG: Original Grepster. >
 ----------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the bug Auto-created from TODO.md tag label Sep 21, 2026
@superdav42
superdav42 marked this pull request as ready for review September 21, 2026 18:22
@superdav42
superdav42 merged commit 77a7152 into main Sep 21, 2026
25 of 26 checks passed
@superdav42 superdav42 added the review-feedback-scanned Merged PR already scanned for quality feedback label Sep 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Auto-created from TODO.md tag origin:interactive Created by interactive user session review-feedback-scanned Merged PR already scanned for quality feedback

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant