feat(skill): webster-onboarding v2 with verify scripts#12
Merged
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
richsak
added a commit
that referenced
this pull request
Apr 26, 2026
- README: add a "The `wbs` alias" section under Prerequisites so judges who clone the repo can replicate the dispatcher launcher (or run the equivalent `claude --settings ...` directly without aliasing). - Untrack `deploy/webster-dispatcher.plist`. The launchd plist hardcodes Richie's macOS user paths (`/Users/richiesakhon/...`) so it cannot be shipped publicly. Preserved in `~/Vault/Projects/webster/internal-tracking/deploy/`. - Run `bun run format` to fix prettier table-padding in `context/ONBOARDING-CASE-STUDY.md` (PR #12 CI format-check failure).
Section A polish — pre-judging cleanup. Moved internal-only docs to ~/Vault/Projects/webster/internal-tracking/ (preserved locally, removed from public tree). Untracked from context/: EXPANSION-TASKS.md, E2E-IMPLEMENTATION-TRACKER.md, SITE-FORK-CHECKLIST.md, ROADMAP.md, VIDEO-PLAN.md, VIDEO-PLAN-90s.md, v2-design.md Untracked from prompts/ (only first/second-wbs and sim-council stay public, matching what README documents): third-wbs-session.md, fourth-wbs-session.md, sim-audit-fix-session.md, composition-session.md, e2e-demo-run-session.md, sim-runner.md Untracked from history/: AGENTS.md (+ CLAUDE.md symlink) Untracked .forge/ralph/ — already gitignored for new dispatches; old PRDs were still in the index. Forge workflow YAMLs and config remain tracked. AGENTS.md updated to redirect EXPANSION-TASKS references to the vault path and drop stale doc references.
Implements the P0–P5 phase model locked in context/ONBOARDING-CASE-STUDY.md
(Q1–Q15). Skill is a thin shell over scripts/onboarding/* and the runtime
registration patterns in prompts/first-wbs-session.md, with status-file
resume and machine-checked phase exit gates.
- skills/webster-onboarding/SKILL.md — orchestration, P0 overview through
P5 first council, ! pre-load checks, hard rules on key handling
- references/{qa-bank,business-yaml-schema,key-handling,remediation,
empire-fixture}.md — detail split per skill-authoring conventions
- scripts/onboarding/verify-env.ts — reads .env.local, hits Anthropic /
GitHub / Cloudflare verify endpoints without echoing key values
- scripts/onboarding/verify-all.ts — rollup with --phase {p3,p4}, agent
count derived dynamically from agents/*.json non-sim specs
- scripts/onboarding/scaffold-repo.ts — creates GitHub repo + Astro
starter using brand identity from context/business.yaml
- package.json — wire onboarding:verify-env, :verify-all, :scaffold-repo
- context/ONBOARDING-CASE-STUDY.md — fix 9-vs-10 production agent drift
Smoke-tested verify-env (exit 1 on missing .env.local with clean hint)
and verify-all --phase p3 (per-check ✓/✗, exit 1 on real env gaps).
End-to-end skill invocation and scaffold-repo (side-effecting) untested
— follow up via Empire dry-run before recording.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- README: add a "The `wbs` alias" section under Prerequisites so judges who clone the repo can replicate the dispatcher launcher (or run the equivalent `claude --settings ...` directly without aliasing). - Untrack `deploy/webster-dispatcher.plist`. The launchd plist hardcodes Richie's macOS user paths (`/Users/richiesakhon/...`) so it cannot be shipped publicly. Preserved in `~/Vault/Projects/webster/internal-tracking/deploy/`. - Run `bun run format` to fix prettier table-padding in `context/ONBOARDING-CASE-STUDY.md` (PR #12 CI format-check failure).
richsak
force-pushed
the
feat/repo-polish
branch
from
c16f4cc to
632198d
Compare
3 tasks
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
1 participant
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
Implements the P0–P5 onboarding phase model locked in
context/ONBOARDING-CASE-STUDY.md(Q1–Q15). The skill is a thin shell overscripts/onboarding/*and the runtime registration patterns inprompts/first-wbs-session.md, with status-file resume and machine-checked phase exit gates.skills/webster-onboarding/SKILL.md— P0 overview → P5 first council,!pre-load checks, hard security rulesskills/webster-onboarding/references/— qa-bank, business-yaml-schema, key-handling, remediation, empire-fixturescripts/onboarding/verify-env.ts— reads.env.local, hits Anthropic/GitHub/Cloudflare verify endpoints without echoing key valuesscripts/onboarding/verify-all.ts— rollup with--phase {p3,p4}, agent count derived dynamically fromagents/*.jsonnon-sim specsscripts/onboarding/scaffold-repo.ts— creates GitHub repo + Astro starter fromcontext/business.yamlpackage.json— wiresonboarding:verify-env,:verify-all,:scaffold-repocontext/ONBOARDING-CASE-STUDY.md— fixes 9-vs-10 production agent count drift (3 spots)Test plan
bun run validate— type-check, lint (0 warnings), format, agent specs (29 valid), findings (7 valid), markdownlint (0 errors), 181/181 tests passverify-envwith no.env.local→ exit 1, clean error + hintverify-all --phase p3→ per-check ✓/✗, exit 1 on real env gapsscaffold-repoend-to-end (side-effecting — needs intentional test repo target)Notes
skills/webster-onboarding/(matches deleted v1 path + EXPANSION-TASKS T12). The stalewebster-decision-log.mdreference to.claude/skills/onboard-smb/is out of date.bun runwrapper sometimes printserror: script ... exited with code 1while the wrapper itself returns 0. Directbun scripts/onboarding/*.tsreturns the correct exit code; the skill's gate logic checks the script output, not just exit code.context/brand-corpus/population, dad consent artifact.🤖 Generated with Claude Code