CI's empty corpus is a named skip, and the documents catch up to the … - #1
Merged
Conversation
…build tests/test_api.py probed /funds at import time. CI applies the migrations to an empty Postgres - the corpus is gitignored - so periods[0] raised IndexError during collection and 946 tests never ran: eight consecutive red pushes while the local gate stayed green against the loaded demo schema. The probe is now a session-scoped fixture, and the six ledger-dependent tests skip with a reason SANCTIONED_SKIPS names, exactly as the four other corpus-dependent suites already do. Seeding CI a ledger rich enough to satisfy 0 < unsupported < total means maintaining a second corpus shaped to produce specific verdicts - the fixture-written-to-fit failure this project has already had once. Verified: 20/20 against demo, 14 passed + 6 sanctioned skips against a freshly migrated schema, mypy at ceiling, 966 collect. The README, Blueprint and roadmap described the system as designed rather than as built. Model extraction has shipped - Lucra's CEO email fired the stated trigger and the citation binding accepted three of five figures - retrieval is built to layer 2 and measured both ways (40/40 scoped, 24/40 blind), settlement gates R1, and the ledger holds 253 cited facts. The scope note now leads with what the gates cannot see, and each new cut carries its trigger - including a relationship graph projected over the ledger once multi-hop questions on the firm's real data stop being writable as SQL. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
…build
tests/test_api.py probed /funds at import time. CI applies the migrations to an empty Postgres - the corpus is gitignored - so periods[0] raised IndexError during collection and 946 tests never ran: eight consecutive red pushes while the local gate stayed green against the loaded demo schema. The probe is now a session-scoped fixture, and the six ledger-dependent tests skip with a reason SANCTIONED_SKIPS names, exactly as the four other corpus-dependent suites already do. Seeding CI a ledger rich enough to satisfy 0 < unsupported < total means maintaining a second corpus shaped to produce specific verdicts - the fixture-written-to-fit failure this project has already had once. Verified: 20/20 against demo, 14 passed + 6 sanctioned skips against a freshly migrated schema, mypy at ceiling, 966 collect.
The README, Blueprint and roadmap described the system as designed rather than as built. Model extraction has shipped - Lucra's CEO email fired the stated trigger and the citation binding accepted three of five figures - retrieval is built to layer 2 and measured both ways (40/40 scoped, 24/40 blind), settlement gates R1, and the ledger holds 253 cited facts. The scope note now leads with what the gates cannot see, and each new cut carries its trigger - including a relationship graph projected over the ledger once multi-hop questions on the firm's real data stop being writable as SQL.
Summary
Risk tier
routinesemantictrust-criticalPath-derived minimum (if known): ___________
Oracle / evidence
Review topology (from review-policy.yaml)
check_all/ CI)Fresh-eyes check (the solo substitute for code-owner review)
Implementer model family
Unresolved assumptions
Test plan