Feat/incremental data model - #4
Merged
Merged
Conversation
/architect still designs the whole coherent data model per feature, but the build plan no longer front-loads it as "migration = task 1". The migration is now sized to the feature: one for a normal feature, sliced across build slices for a large feature or a thin thread / Facade approach, and omitted for a slice that touches no schema. /develop creates schema with the slice that needs it; real model changes mid build still route back through /architect. Recorded as spec 0004. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
docs/conventions.md captures the house rules for keeping skill files lean (they load in full on their path every run): a line must change behavior, reasoning lives in specs, name concepts instead of explaining them, steps not narrative, state a rule once, split out only rare and long content. Linked from CLAUDE.md, and applied once (ui/implementation.md Phase 0 compressed, clearing its size-budget warning). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
… first commit setup
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (27)
📝 WalkthroughWalkthroughDocumentation and skill workflows were revised to remove superseded spec references, add authoring conventions, size migrations by feature scope, introduce conditional Git handling, and tighten review, verification, documentation, testing, and scope-closure reporting. ChangesWorkflow guidance
Estimated code review effort: 3 (Moderate) | ~25 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Merged
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 by CodeRabbit
New Features
Documentation