docs: fix onboarding path bootstrap chicken-and-egg#327
Merged
serenakeyitan merged 1 commit intomainfrom Apr 23, 2026
Merged
Conversation
The previous end-to-end prompt (merged in #326) told a cold-start agent to read `.agents/skills/first-tree/references/onboarding.md` right after CLI install — but that path only exists after `first-tree skill install` runs, which is part of setup itself. A literal paste into a fresh Claude Code session would fail at the first step. Point the prompt at `first-tree tree help onboarding` instead, which is available immediately after the CLI is installed (verified in src/products/tree/cli.ts). Mention the skill-path reference as the post-install artifact it actually is, not the starting point. Follow-up to yuezengwu's CHANGES_REQUESTED review on #326 (arrived 6s after admin-merge).
yuezengwu
approved these changes
Apr 23, 2026
yuezengwu
left a comment
yuezengwu
left a comment
There was a problem hiding this comment.
Approving. This correctly fixes the chicken-and-egg I flagged on #326.
Verified:
first-tree tree help onboardingis a registered subcommand (src/products/tree/cli.ts:37→src/products/tree/engine/commands/help.ts:23→runOnboarding), and the text is bundled into the build (import ONBOARDING_TEXT from "#skill/references/onboarding.md"), so it works immediately afternpm install -g first-treewith no priorskill installstep.- The skill-path reference is now positioned as a post-
first-tree skill installartifact, which matches reality. - Literal paste of the new prompt no longer references any path that doesn't exist at cold start.
This reply was drafted by breeze, an autonomous agent running on behalf of the account owner.
yuezengwu
added
breeze:done
3 tasks
This was referenced Apr 23, 2026
serenakeyitan
added a commit
that referenced
this pull request
Apr 23, 2026
## Summary Closes #328. The README's end-to-end pitch promises "source PR → tree issue → breeze pickup → draft-node PR." Step 6.4 of the onboarding narrative already demonstrated the tree side (manual `gardener sync`), but never scripted the actual automatic chain from a source-repo PR. Cold-start E2E on #327 confirmed a fresh agent would have to invent that leg. Splits 6.4 into: - **6.4.a** — existing manual `gardener sync` trigger (fast credential check) - **6.4.b** — scripts the real source-PR chain: open a trivial PR, `breeze watch` / `status` for pickup, confirm gardener verdict, clean up ## Verified against real CLI surfaces - `first-tree breeze watch` is the live TUI (per `src/products/breeze/cli.ts:36,88`) - `first-tree breeze status` is one-shot (per `cli.ts:113`) - `first-tree breeze doctor` reports interval + auth state (per `cli.ts:121`) - Not using `breeze status --watch` (does not exist) ## Test plan - [x] `pnpm typecheck` — clean - [x] `pnpm test` — 1192 passing, 51 skipped, no regressions - [x] `pnpm build && node dist/cli.js tree help onboarding` — narrative renders the new 6.4.b section from the bundled import
serenakeyitan
added a commit
that referenced
this pull request
Apr 23, 2026
## Summary Docs-only patch release so npm users get the updated onboarding narrative bundled into `first-tree tree help onboarding`. Unreleased since v0.2.14: - #325 — make end-to-end onboarding prompt source/tree-agnostic - #326 — default to breeze-first setup - #327 — fix onboarding path bootstrap chicken-and-egg - #329 — add 6.4.b real source-PR trigger ## Test plan - [x] `pnpm release:check` green locally (version:check, validate:skill, typecheck, test, build, test:dist, test:release)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to yuezengwu's CHANGES_REQUESTED review on #326 (review arrived 6 seconds after admin-merge — addressing it here).
The prompt merged in #326 told a cold-start agent to read
.agents/skills/first-tree/references/onboarding.mdright after CLI install. But that path only exists afterfirst-tree skill installruns, which is part of the setup the onboarding skill itself walks the user through — classic chicken-and-egg.Points the prompt at
first-tree tree help onboardinginstead (verified insrc/products/tree/cli.ts:12,37, which already advertises this command as the "new to first-tree" entry). The skill-path reference is now mentioned as a post-install artifact, not the starting point.Test plan
first-tree tree help onboardingis a real registered subcommand insrc/products/tree/cli.tsnpm install -g first-tree/npx first-treeRefs #326.