Repository navigation
Triage Financial-Modeling's tests/test_audit.py as a Path Collision, and Consider Keeping Hub tests/ Out of the Hub-Only Detector #3528
Description
Observed
spec/audit.py Financial-Modeling reports an untriaged hub-only hit:
audit run 2026-10-27T21:04:28Z | hub f60f57d9
== Financial-Modeling (python, source-only; release) @ main@fb5f003 ==
DRIFT hub-only: tests/test_audit.py is undeclared in spec/files.json and this repo carries it - read the file and triage it in spec/divergences.json 'gaps' (this repo's own content at a shared path, a carry to declare, or a hub copy to delete)
Triage, from reading the file
It is a path collision, not a carry. Financial-Modeling's tests/test_audit.py tests that repository's own finmodel.audit package (its rule registry, judged-file window and report renderer), and it shares no content with the hub's tests/test_audit.py, which drives spec/audit.py's audit_repo against a faked gh. The repository's file dates from 2026-07-31, and the hub's copy at that path arrived on 2026-10-15 with #3111. Deleting it per a hub-only: remedy would destroy the repository's own test suite for that package.
Expected
An accepted entry in spec/divergences.json gaps for tests/test_audit.py, shaped like the existing scripts/README.md collision entry, so the finding closes permanently.
Worth deciding alongside
tests/ is as generic a path as scripts/, and every Python carrier keeps its own tests there. Each hub test module whose name a carrier happens to share becomes another untriaged hit, one entry at a time. A rule that keeps hub tests/ paths out of the hub-only detector (the hub's tests are never carried, so a carrier's tests/ file can only be a collision) would close the class rather than this one instance.
Affected
- Repository:
ptr727/Financial-Modeling, atmain@fb5f003and atdevelop(both report it). - Hub:
mainatf60f57d9.
Activity
ptr727 commentedon Oct 27, 2026
Backlog burndown round 1 claim. Branch:
feature/3528-audit-hub-tests-collision.Maintainer decision: close the class, so
spec/audit.pystops reporting a hubtests/path ashub-only:in a carrier.Predicted file set:
spec/audit.py,tests/test_audit.py,AUDIT.md(thehub-only:bullets only, if the rule is stated there),spec/divergences.jsonandreports/divergences.md(only if an existing entry becomes obsolete).ptr727 commentedon Oct 27, 2026
Claim update (round 1, branch
feature/3528-audit-hub-tests-collision, PR #3540): the orchestrator addsspec/project-types.json(thecarried-scopecarried.hubonly.absentassertion only) andspec/validate.py(a shape check for the newrepoOwnedTreesarray beside the existingtreeschecks) to this group, after Copilot threads on #3540. No other live group holds either file.ptr727 commentedon Oct 27, 2026
Claim update (round 1, PR #3540): the group also holds
spec/files.json(newrepoOwnedTreeslist) andspec/files.schema.json(its schema), the data mechanism the brief allowed.spec/divergences.jsonandreports/divergences.mdare released, since no entry there became obsolete.ptr727 commentedon Oct 27, 2026
Claim update (round 1, PR #3540): the orchestrator adds
RESYNC.md(section 4, "Deleting Is a Remedy, and It Is the One That Can Destroy Work", the hub-only definition and its mermaid node only) andtests/test_spec_validate.py(therepoOwnedTreesgate case). No other live group holds either file.Keep Hub tests/ Paths Out of the Hub-Only Detector (#3540)
ptr727 commentedon Oct 28, 2026
Promotion fix claim (round 1, during the #3544 freeze). Branch:
feature/3528-repo-owned-overlap. Files:spec/validate.py(therepoOwnedTreescheck only) andtests/test_spec_validate.py. It fixes Copilot threadPRRT_kwDOQ5caqM6w8BW3on #3544: reject arepoOwnedTreessource that overlaps a carriedtrees[].sourceor contains a baseline path.Refuse a Repo-Owned Tree Overlapping Carried Content (#3545)