chore: final-polish — remove video footprint + polish context docs#16
Merged
chore: final-polish — remove video footprint + polish context docs#16
Conversation
Remove submission-tooling that does not belong in the public repo: skills/webster-video/, video/, context/webster-video/, the video composition prompt, the onboarding case-study, and the webster-onboarding empire fixture. The HyperFrames render pipeline that produced the timelapse was always tooling-side, not product. Polish ARCHITECTURE / FEATURES / DOMAIN-MODEL / VISION / QUALITY-GATES to the post-eeda2bc shipped state: 9 production Managed Agents mirrored 1:1 by 9 simulation specs. Reframe Layer 6 (video) as external submission tooling, not a blocked product layer. Move visual-design-critic to its true provenance — a W4 genealogy spawn, not a permanent L2 base agent. Drop deferred / hackathon-crunch language from the canon docs; that posture is over. Strip unverifiable projections from README (cost-per-month, cost-per-run, agency-pricing comparisons). Standardize the test-count phrasing to "29 test files green via bun run validate". Tidy dangling references in skills/webster-onboarding and qa-bank. Validate is green: 176 tests pass, 0 lint warnings, 0 type errors, 0 markdown errors, 19 JSON specs valid, 7 findings files valid. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
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 |
Forge dispatch watcher with no live consumer post-hackathon. Only references were in checkpoints/compactions, no callers in scripts/, prompts/, package.json, or workflows. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Add a "5-minute judge tour" section to README naming the exact click-path for evaluation: pitch → INDEX → one week's visual review → critic-genealogy.ts → optional live --dry-run. Add demo-output/landing-page/INDEX.md narrating the 11-week LP timelapse, one beat per week with classification + links to that week's screenshots and visual review. Turns scattered week directories into a scannable evidence path. Drop the "rendered MP4 hosted externally — link in submission form" references from README, ARCHITECTURE, and FEATURES. The video link is owned by the submission form, not the repo, and pointing at a form judges may not have open is dead weight. The per-week assets plus INDEX.md carry the visual evidence on their own. Validate green: 176 tests pass, 0 lint warnings, 0 type errors, 0 markdown errors, 19 specs valid, 7 findings files valid. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
Final polish pass before the dev → main rollup (PR #15).
skills/webster-video/,video/,context/webster-video/, the video composition prompt, the onboarding case-study, and the webster-onboarding empire fixture (~7000 line deletion).eeda2bcshipped state — 9 production Managed Agents mirrored 1:1 by 9 simulation specs. Layer 6 (video) reframed as external submission tooling, not a blocked product layer.visual-design-criticcorrectly framed as a W4 genealogy spawn (its true provenance), not a permanent L2 base agent. Drop deferred / hackathon-crunch framing from the canon docs.skills/webster-onboardingand qa-bank.Test plan
bun run validategreen: 176 tests pass, 0 lint warnings, 0 type errors, 0 markdown errors, 19 JSON specs valid, 7 findings files valid.dev. Then PR feat: hackathon expansion — sim council, skills, video, repo polish #15 (dev → main) becomes a clean rollup.🤖 Generated with Claude Code