Conversation
Money has been out of the report since #681 until the team is "confident" in how it is added back, and nothing said what confident means. ADR 0025 (Proposed) defines it: every figure typed on its own, matched on meaning against official sources, zero tolerance for errors that change a total, uncertainty carried per figure in the data, and money returning in stages ordered by claim strength. ADR 0001 is rewritten to say its money table is the goal and is gated by 0025. The financial-semantics research README gains a research question and exit criteria. Refs #147 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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.
Related issue
Refs #147 (financial semantics epic). Not a
Closes: the epic stays open, and this PR proposes how it is rescoped.Draft for discussion, not for merge yet. @mattzamora @ruggbk, I'd like to talk this over with you before anything is committed. It sets the bar your work on #736 and #724 would be measured against, so your read on whether it is the right bar matters most.
What does this change?
Dollar amounts have been out of the report since #681 (#671), and the rule has been that they stay out until we're confident in how they come back. Nothing said what "confident" means. This PR proposes a definition as ADR 0025 (Proposed):
diff.json(ADR 0006).It also:
docs/research/financial-semantics/README.md.The ADR is numbered 0025 because #734, #736 and #739 already claim 0022 to 0024.
Where the criteria come from
The five errors found in review of #736 and #724 are the evidence base. Each is traced to the bill text in the ADR's Context: the $71B appropriation shown as a rescission, the $31M cap picked over $10.55B, the missed $920M advance appropriation, the mislabelled amended-law flags, and the $1.91B Title I shortfall. Case 1 and the Title I shortfall come directly from the research classifier typing clauses instead of figures. Case 3 is a ceiling-pattern miss made worse by the same choice. That is why per-figure typing is a requirement, not a suggestion.
Questions for discussion
Proposed rewrite of epic #147 (not applied; after we agree)
How to test
Docs only. Read the ADR as the proposal, since there is no behaviour to exercise. All five CI gates were run locally:
ruff check .ruff format --check .-m "not slow and not browser")-m browser --run-browser)-m slow --deselect tests/test_govinfo_corpus_parity.py)tests/test_adr_index.pychecks the new record's heading, status and index row.Checklist
Refs #147, deliberately notCloses)AI assistance
Drafted with Claude Code (Claude Opus 5.5) from a discussion with the maintainer, who set the criteria. The model checked the evidence against the PR reviews, the classifier source and the existing ADRs.
🤖 Generated with Claude Code