docs(architecture): establish canonical PRD TRD and UML contract - #361
Draft
seonghobae wants to merge 8 commits into
Draft
docs(architecture): establish canonical PRD TRD and UML contract#361seonghobae wants to merge 8 commits into
seonghobae wants to merge 8 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 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 |
Contributor
Author
|
@coderabbitai review |
|
This was referenced Sep 12, 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
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.
Finding
Protected
main@f8260f1e03836039ff9463dd99fa982e4e270c4bhas code-current architecture/specification material and ADRs, but no canonical PRD, TRD, or UML artifact. That leaves product scope, technical acceptance, and owner boundaries distributed across prose and makes documentation drift hard to detect mechanically.This lane is intentionally disjoint from #130 (
docs/product-technical-gap-baseline.md) and #111 (accepted ADR consolidation). It does not edit either owner path and does not change runtime behavior.Hostile RED
Exact structural RED
01d6640304eab7913d508b342243e222fda1c689addedtests/documentation_contract.rs. It references the canonical PRD/TRD/UML paths and explicit owner-boundary/performance/release markers. That first head advanced before its hosted run completed, so it is not claimed as hosted failure evidence.The first hosted documentation-contract RED was observed on exact head
a533001a8d42fe90ec2310143b71c5cea191e3e6: repository Rust CI reached the new documentation test after 142 core tests passed and failed on the case-sensitive PRD authority marker. Minimum causal repair5334a6039d368d59f455c5040f03e5dbc5c7d8fealigned that marker with the canonical heading without weakening the contract.A second buyer-facing contract defect was then reproduced on exact head
5668ba29dd7ee1b4488c601597bfc49c4944dabb. The Material UI requirement allowed design-system or Storybook/Figma evidence only "where applicable" and the TRD did not mechanically bind Figma/Storybook evidence. CI34690078651, job103543672297, reached the new documentation contract after all 142 core tests passed and failed exactly atmaterial_ui_and_slow_work_boundaries_remain_explicit:PRD must retain the code-current marker "Figma/Storybook evidence".The minimum causal repair was split into ordinary fast-forward commits because the contents API writes one file at a time:
bec85782c8e17865beec2267b0c406e5d16649c9makes PRD Figma/Storybook evidence mandatory, andc6d3fd45eba1aceef074cc8b9934937f67b1d41abinds the same requirement in the TRD. The hostile test itself remains intact.GREEN acceptance
Add the minimum code-current canonical artifacts without duplicating foreign-owner implementation logic:
The artifacts must trace protected-main truth and authoritative standards, remain compatible with existing ADR/architecture/security/ops/release documentation, and make
tests/documentation_contract.rsGREEN on one unchanged exact head.Exact-head evidence
Current exact head:
c6d3fd45eba1aceef074cc8b9934937f67b1d41a.34690323747: SUCCESS on the unchanged exact head; rust job103544312321passed format, locked workspace tests including the hostile documentation contract, and strict Clippy.34690323751: SUCCESS on the unchanged exact head.34690323719: SUCCESS on the unchanged exact head.34690323728: FAILURE only at central delegated terminal settlement. Detect job103545057667checked out the exact head and succeeded; compatibility job103545768079successfully read the current-head verdict and failed atRelease runner or enforce current-head CodeQL verdict; downstream dispatch job103547018215then succeeded. Fresh consumer evidence and unchanged-head RED/GREEN acceptance are handed to canonical owner.github#1929; no Wardnet workflow copy, source churn, synthetic status, or bypass is authorized.Foreign-owner revalidation confirms
context-graph-contractsprotecteddevelop@99cb5468ba3c15c5e79688f53dee74724fae2d13andenterprise-architecture-coreprotecteddevelop@dd71e40a86385fb7861b0f1be19891a3f3e29ecestill have no immutable GitHub Release; both remain read-only to this lane.contextual-orchestratorprotectedmain@012beaacd0631f8cd3391c77744eeb626269b5deincludes its typed versioned admission work but still has no immutable GitHub Release, so Wardnet does not add a mutable CO dependency.EgressWeave,quarantine-sandbox-runtime, andappguardrailalso have no immutable GitHub Release, so their canonical logic remains foreign and is not copied into Wardnet.Keep Draft until exact-head required checks/review/thread evidence is terminal and current. No force push, self/model approval, routine bypass, source-copy, cross-service SQL, mutable foreign dependency, or edits to CGC/EA.