chore: remove all development phase markers from code and tests - #18
Merged
Conversation
Strips P0-01..P4-15, Phase references, and DP contract numbers from test group names, inline comments, and doc strings across all three packages. No functional changes.
- Remove all spec/phase/requirement-ID references from code and docs - Delete dev-phase files from specs/ (PHASE plans, DEVELOPMENT, etc.) - Delete root test/ and tool/ (CI infrastructure) - Add AGENTS.md, .opencode/, opencode.json to .gitignore - Fix gaps: SSG CLI, SSG runner (asset copy, dynamic segments, zero-overhead) - Fix bot-aware delivery middleware with UA branching - Add DocumentBuilder UA-blind contract test - Add Island widget registration + entry-point tests - Add JS budget checks (dispatcher, custom element, service worker) - Coverage: hydraline 100%, hydraline_server 100%, hydraline_flutter 99%
- Delete .gitkeep placeholders (api/ and integration_test/) - Delete empty example/README.md skeletons - Fix CI yaml: remove broken boundaries step + stale docs/adr ref - Clean stale refs to deleted files in specs/README, ARCHITECTURE, REQUIREMENTS_TRACEABILITY, EXAMPLES, PR template, .gitattributes - Clean melos run boundaries reference in hydraline_server barrel
- Delete entire specs/ directory (development-phase artifacts) - Move L4 API contracts to packages/*/api/ (English, clean) - Move hydraline.routes.schema.json to packages/hydraline/ - Translate all Russian comments to English in API contracts - Clean spec/dev references from API contracts, issue templates, pubspec - All useful spec info already exists in docs/ (English)
Actual public API is in lib/ with dartdoc. Throwaway stubs from spec phase are no longer needed post-implementation.
… cross-link everything - Add root LICENSE, CONTRIBUTING.md, CODE_OF_CONDUCT.md, SECURITY.md - Add root CHANGELOG.md - Rewrite all 3 package READMEs with full content, code examples, doc links - Fix stale Flutter version (3.22 -> 3.35) in CHANGELOG and README - Fix quick-start version (^1.0.0 -> ^0.0.1) in docs/README.md - Add Community section to root README - Cross-link everything: READMEs <-> docs <-> packages
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.
Strips P0-01..P4-15, Phase references, and DP contract numbers from test group names, inline comments, and doc strings across all three packages. No functional changes. All 244 tests pass, precommit green.