Conversation
Scaffolded repos do not have project-kit checked out, so a local path would dangle; the public URL keeps SETUP-CHECKLIST.md the single source of truth. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe scaffold script now directs users to the public setup checklist as the final Next steps item after their first push, helping them complete manual repository configuration such as branch protection and reviewer setup. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="scripts/scaffold.sh" line_range="210" />
<code_context>
- Fill in CLAUDE.md (Domain Context, Architecture, Key Files)
- bd ready # start tracking work
- git add -A && git commit # first commit
+ - After the first push: https://github.com/R0SEWT/project-kit/blob/main/SETUP-CHECKLIST.md
EOF
</code_context>
<issue_to_address>
**issue:** The scaffolded Next steps link returns 404 because this repository does not currently contain `SETUP-CHECKLIST.md` on `main`, so users cannot access the checklist after following the new instruction.
**Triggers:** Until the checklist is merged to the repository's `main` branch.
**Suggested fix:** Merge the checklist change before this pointer, or land both changes together so the public URL resolves when scaffolding is released.
</issue_to_address>Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: scripts/scaffold.sh:210
| - Fill in CLAUDE.md (Domain Context, Architecture, Key Files) | ||
| - bd ready # start tracking work | ||
| - git add -A && git commit # first commit | ||
| - After the first push: https://github.com/R0SEWT/project-kit/blob/main/SETUP-CHECKLIST.md |
There was a problem hiding this comment.
issue: The scaffolded Next steps link returns 404 because this repository does not currently contain SETUP-CHECKLIST.md on main, so users cannot access the checklist after following the new instruction.
Triggers: Until the checklist is merged to the repository's main branch.
Suggested fix: Merge the checklist change before this pointer, or land both changes together so the public URL resolves when scaffolding is released.
Summary
The scaffold's "Next steps" now ends with a link to
SETUP-CHECKLIST.md, the post-push manual steps (branch protection, reviewers).Changes
Next stepsheredoc ofscripts/scaffold.sh.Testing
bash -n scripts/scaffold.shshellcheck scripts/*.sh(0.11.0) — cleanscripts/scaffold.sh <tmp> --name demo --dry-run— the new line prints last in Next stepsRelated
bd: project-kit-5q6.8
🤖 Generated with Claude Code
Summary by Sourcery
Enhancements: