Skip to content

feat(init): prompt for optional projects during interactive setup (#757) - #910

Merged
jeff-r2026 merged 2 commits into
Tencent:mainfrom
ydflow:contrib/init-optional-project-prompt
Oct 1, 2026
Merged

jeff-r2026 merged 2 commits into
Tencent:mainfrom
ydflow:contrib/init-optional-project-prompt

Conversation

@ydflow

@ydflow ydflow commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Prompt for declared projects during interactive init when --project is omitted.
  • Support selecting multiple projects; an empty selection remains valid.
  • Skip the optional picker in non-interactive runs and print a projects set hint.
  • Preserve explicit --project behavior and update English/Chinese usage and design docs.

Closes #757

Verification

  • npm run typecheck
  • npm run lint
  • npm run build
  • Targeted init project-selection unit tests: 2 passed
  • Targeted non-interactive CLI E2E test: 1 passed

Note: the full init unit test file has 5 Windows path assertion failures that also reproduce on the unmodified upstream base.

@jeff-r2026 jeff-r2026 self-assigned this Sep 29, 2026
@github-actions

Copy link
Copy Markdown
  • [P1 blocking] Update the deployed setup skill for the new project prompt. src/init.ts:384 introduces a user decision, but skill-data/setup/references/join-member.md:70 and skill-data/setup/references/setup-admin.md:164 still direct agents to run init without asking which projects to select. In an agent-led interactive setup, this can stall at the unexpected prompt or accept blank and incorrectly save projects: []. This also violates the repository rule requiring behavior changes to update affected skill-data/.

The PR description’s testing record is otherwise sufficient: it includes a representative real-CLI E2E run for the runtime change.

@github-actions

Copy link
Copy Markdown

No findings.

  • The earlier [P1 blocking] skill documentation issue is resolved in skill-data/setup/references/join-member.md:70 and skill-data/setup/references/setup-admin.md:166.
  • The PR description includes a representative non-interactive real-CLI E2E run, so the runtime-change testing requirement is satisfied.

@jeff-r2026
jeff-r2026 merged commit 7bb0652 into Tencent:main Oct 1, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should interactive init optionally prompt for a project?

2 participants