Skip to content

docs: bring the docs in line with the Schema epic (#120) - #165

Merged
MatheusCastro99 merged 2 commits into
masterfrom
docs/120-epic-136-audit
Nov 14, 2026
Merged

MatheusCastro99 merged 2 commits into
masterfrom
docs/120-epic-136-audit

Conversation

@MatheusCastro99

Copy link
Copy Markdown
Owner

Refs #120. Plan: #120 (comment)

The docs pass from Epic #136's knowledge audit (A1-A9, two adjacent doc items, three checkboxes in #120). #120 stays open.

🤖 Generated with Claude Code

Refs #120

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…#120)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MatheusCastro99
MatheusCastro99 marked this pull request as ready for review November 14, 2026 14:00
@MatheusCastro99

Copy link
Copy Markdown
Owner Author

Ready to merge at 4db145f. Reviewed SHA = PR head = the SHA CI passed on.

G2 (2026-11-14): both findings were fixed in 4db145f as the owner approved: the ADR-004 $0-subtotal exception (#153), and the ADR-005 note now names the remaining literal messages. The narrow re-review approved with no findings.

Note, no change: ADR-004's first bullet ("never computes either itself") and the Critical row in guardrails-review could still be read literally against the $0 case. The re-review scored that at 3, under the bar, because the exception sits in the next bullet and the reviewer's checklist names it. Recommendation: leave it.

#120 stays open (Refs, not Closes). Next: the owner merges, then /close 120 and /close 136 epic.

Copilot AI balanced review requested due to automatic review settings November 14, 2026 14:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@MatheusCastro99
MatheusCastro99 merged commit 75fd5c2 into master Nov 14, 2026
8 checks passed
@MatheusCastro99
MatheusCastro99 deleted the docs/120-epic-136-audit branch November 14, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants