chore: collapse quality vocabulary, delete sweep system, ground self-quality framing#124
Open
abdout wants to merge 6 commits into
Open
chore: collapse quality vocabulary, delete sweep system, ground self-quality framing#124abdout wants to merge 6 commits into
abdout wants to merge 6 commits into
Conversation
/ship promotes a /check-passed build to Vercel production. Distinct from /deploy (staging/preview) — names telegraph the target. Wraps `npx vercel --prod --yes` with the same 5-attempt auto-fix loop the deploy skill uses. Refuses dirty working tree; runs /check inline if not already passed in the session; suggests /watch on success. Implements the orphan referenced in the /feature pipeline (stage 7). Refs #123 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Real implementation of the pre-demo quality pass referenced in docs/WORKFLOWS.md. Five passes per route in a feature block: 1. Bug-free — console + network clean 2. Flow — primary user journey completes 3. Responsive — 375 / 768 / 1440 layouts 4. RTL + i18n — Arabic mirrors, LTR exemptions hold 5. Translation — no hardcoded strings, all keys resolve Uses browser-headed MCP for visible runs. Outputs a report with screenshots to .claude/handover-reports/<block>-<timestamp>/. Supports --fix for translation + RTL automated fixes; flow and responsive failures stay manual. This is the "self-quality" verb the engine was missing: user- triggered, deeper than /check, scoped to a block. Refs #123 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Delete 15 sweep command files + the .claude/coverage/ directory.
Every ledger has been empty (12 bytes) since the system was designed
in April. No sweep has ever run end-to-end on any product. The
protocol and keyword definitions were thorough but unused — Tier 2b
existed as documentation, not as a live capability.
Removed:
- .claude/coverage/{keywords.json,sweep-protocol.md}
- .claude/commands/{nextjs,react,typescript,tailwind,shadcn,
accessibility,barrel,prisma,authjs,translate,skeleton,structure,
guard,waterfall,coverage}.md
Subsequent commits collapse the Tier 2b framing in CLAUDE.md and
strip the sweep orchestration from the quality-engineer agent.
If full-coverage sweeps are wanted later, schedule /handover via
the schedule skill against staging URLs — that path needs zero new
infrastructure.
Refs #123
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…p orchestration Strip the 5 sweep references (translate, skeleton, structure, guard, coverage as full-coverage sweeps) and the per-product ledger/health- monitoring table — none of that ran in practice and the underlying files were deleted in the previous commit. What remains is the truthful surface: - 17 niche keywords across browser (6), code (6), deep (3), compare (2) - 2 orchestrators: qa (per-URL, 12 checks) and /handover (per-block, 5-pass Playwright via .claude/commands/handover.md) The agent is now a router — keyword → MCP/specialist — not a sweep orchestrator pretending to maintain a ledger. Refs #123 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…esponsive Project-level CLAUDE.md rewrite: - Drop the "Tier 1 / Tier 2 / Tier 2b / Tier 3" mental model. Replace with three flat sections — Pipeline, Tools, Vocabulary. - Remove "Mode: Full Autopilot (100-turn cycles)" — aspirational, no autonomous loop exists in the codebase. /handover is the user-triggered materialization of self-quality. - Add /handover to the pipeline narrative as the pre-demo gate. - Drop the Tier 2b table (sweeps were deleted in a prior commit). - Rename the niche browser keyword `check` to `responsive` to remove the collision with the /check pre-ship gate command. The 17-keyword surface in quality-engineer.md is updated to match. The global ~/.claude/CLAUDE.md keyword vocabulary table also gets a new Lifecycle row (check, handover, ship, watch, report, quick) — see that file for the change; it lives outside the repo. Refs #123 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Spellbook (content/docs): - Replace the /validate orphan with a real /check spell (the pre-ship gate). Update the Nox quote accordingly. - Add /handover spell in Defense Against the Dark Arts, alongside test / e2e / review. - Fix the build spell's connects: validate → check. - Sweep keywords (nextjs, react, tailwind, etc.) stay in the spellbook — those entries describe framework patterns, not the deleted sweep commands. docs/CONFIGURATION.md: - Strip the "Full Autopilot (100-turn cycles)" mode row in the Layer 1 settings table — engine is interactive, not autonomous. docs/WORKFLOWS.md: - Point section 1.7 (Handover QA) at the new command file and the report output directory. Aligns the documented passes with the implemented spec. Verified: pnpm tsc --noEmit introduces zero new errors from these edits (existing layout + mdx-components type issues predate this PR). Closes #123 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced May 28, 2026
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
/check,/handover,/ship,/watch,/report,/quick/ship(production deploy) and/handover(Playwright multi-pass QA)/handoveris the user-triggered self-quality verbChanges
Added
.claude/commands/ship.md— production deploy via Vercel --prod, distinct from/deploy(staging).claude/commands/handover.md— five-pass headed Playwright QA scoped to a feature blockRemoved
.claude/coverage/— the entire sweep protocol + keyword definitions.claude/commands/{nextjs,react,typescript,tailwind,shadcn,accessibility,barrel,prisma,authjs,translate,skeleton,structure,guard,waterfall,coverage}.md(15 files)Refactored
.claude/agents/quality-engineer.md— agent is now a router for 17 niche keywords + 2 orchestrators (qa, /handover). Dropped the ledger / sweep orchestration framing..claude/CLAUDE.md— three flat sections (Pipeline / Tools / Vocabulary); dropped Tier 2b and "Full Autopilot"check→responsiveto remove collision with the/checkpre-ship gate commandSynced docs
docs/CONFIGURATION.md— removed Full Autopilot rowdocs/WORKFLOWS.md— section 1.7 points at.claude/commands/handover.mdsrc/components/docs/spellbook-data.ts—/validateorphan replaced with real/checkspell;/handoverspell added to Defense Against the Dark ArtsCanonical vocabulary
/check/handover/ship/watch/report/quickTest plan
grep -rn "Tier 2b\|/verify\|/validate\|Full Autopilot" .claude/ docs/returns zero active matchesls .claude/commands/contains ship.md + handover.md, no sweep filesls .claude/coverage/returns "no such directory"/shiptriggers the new command; typing/handover admissionruns five passespnpm tsc --noEmitshows no new errors from spellbook-data.tskun.databayt.org/docs/keywordsshow the canonical vocabulary after deployCloses #123
🤖 Generated with Claude Code