golem15/msd-core
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

no-mistakes(review): Fix onboard skip handoffs

Browse Source
This commit is contained in:
Jeremy McSpadden
2026-07-03 15:12:07 -05:00
committed by Codesmith
parent 2a38ea5331
commit cd5971fd3f
2 changed files with 38 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
gsd-core/workflows/onboard.md
View File

@@ -72,7 +72,17 @@ Run from worktree root {ONBOARDING_ROOT}:
Then rerun /gsd:onboard from the same worktree root.
```
Exit. If the user skips mapping, continue with a warning.
Exit. If the user skips mapping, print:
```text
Skipping codebase mapping may give /gsd:new-project weaker context.
Run from worktree root {ONBOARDING_ROOT}:
/gsd:new-project
```
Exit.
### `ingest-docs`
@@ -107,7 +117,17 @@ Run from worktree root {ONBOARDING_ROOT}:
Then rerun /gsd:onboard from the same worktree root.
```
Exit. If the user skips docs ingest, continue to the new-project handoff below.
Exit. If the user skips docs ingest, print:
```text
Skipping docs ingest may omit existing ADR/PRD/SPEC/RFC context from /gsd:new-project.
Run from worktree root {ONBOARDING_ROOT}:
/gsd:new-project
```
Exit.
### `complete-map-before-new-project`

16
tests/onboard-command.test.cjs
View File

@@ -362,6 +362,22 @@ describe('/gsd:onboard command contract', () => {
assert.ok(content.includes('AskUserQuestion'), 'workflow must still support interactive choices');
assert.ok(content.includes('--text'), 'workflow must document text-mode fallback');
assert.ok(
content.includes('Exit. If the user skips mapping, print:'),
'skip mapping must hand off explicitly instead of falling through to summary creation',
);
assert.ok(
content.includes('Skipping codebase mapping may give /gsd:new-project weaker context.'),
'skip mapping must warn about weaker context',
);
assert.ok(
content.includes('Exit. If the user skips docs ingest, print:'),
'skip docs ingest must hand off explicitly instead of falling through to summary creation',
);
assert.ok(
content.includes('Skipping docs ingest may omit existing ADR/PRD/SPEC/RFC context from /gsd:new-project.'),
'skip docs ingest must warn about omitted docs context',
);
assert.match(content, /do not overwrite/i, 'workflow must protect existing summary/planning');
assert.match(content, /query commit "docs: create onboarding summary" --files \.planning\/onboarding\/SUMMARY\.md/);
assert.ok(!content.includes('execute-phase'), 'onboarding must not execute implementation phases');