Repository navigation
Tooling: derive find_overlap task terms from research package roots - #2154
Merged
Mister-Meeseeks merged 1 commit intoOct 15, 2026
Merged
Conversation
find_overlap.py --task added each research starting point's immediate parent folder as a search term. For nested files such as research/joint-query/evidence/QQ16-SELECTED-WITNESS.md that term was the generic word "evidence" (or "proofs"), which matched unrelated records, could push genuine leads below the default limit, and never searched for the package itself. Take the package component of research/PACKAGE/... paths instead. Explicit and catalogue terms, snapshot coverage, scoring and path matching are unchanged. Four new CLI regression groups fail on the old helper and pass with the fix. Changes scripts/find_overlap.py; adds tests/test_research_overlap_terms.py. Submitted by @commdata2338 with substantial AI assistance (OpenAI Codex research and implementation; packaged with Claude Code). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X86sCSXdoDd61SDFfC1PKG
This was referenced Oct 15, 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.
Change and mathematical scope
A tooling fix; no mathematical result, supplier or κ change.
The defect:
scripts/find_overlap.py --taskadds each research starting point's immediate parent folder as an OR search term. For nested files that folder is a generic word. The revision-4joint-query-controllercard searches forevidence(twice) and never for its package,joint-query;centre-path-sharingaddsproofs. Generic hits can push a genuine lead below the default 20-result limit.The fix: take the package component of repository-relative
research/PACKAGE/...paths, so a nested file contributes its package; top-level research documents contribute no term. Explicit and catalogue overlap terms (generic words included), snapshot collection, scoring,--pathmatching and coverage reporting are unchanged. The other revision-4 cards derive the same term sets as before.Effect on one fixed snapshot (403 open PRs and 810 Discussion records, plus this base's local records):
--task joint-query-controllergoes from 1,487 possible matches to 33, and--task centre-path-sharingfrom 386 to 261. The removed matches came only from the generic terms; every match to a retained term remains. These counts illustrate the defect; they are not test assertions or a novelty guarantee.Remaining: literal substring search is still heuristic, so this does not make overlap detection complete. Windows execution and hosted CI were not run.
Task and overlap
overlap-receipt(lookup key5ae821d1b52092164092), changed tofind_overlap/task-derived-terms/nested-research-package-roots. Chooser nonce062c8d19991bc9ca650f6c96f1b1d3b6, track bit-mechanisms, priority revision 4; its suggestions (bit-helper-contract,bit-new-register-model) overlap this account's completed work, and the central handoffs considered were already claimed. The defect showed up while running overlap checks for those handoffs. No reroll.0fdd7d82de03e43c9e5a1000dfbcd380d1b687d2, rechecked unchanged at 21:12 UTC; contribution5984d9999bceaeb7d1ce9a3f4edfbfcd0b70c118. No pending dependency. Nick Petalas's earlier pagination repair (9c2fdd89e553751e40ef8011716a249e8385d284) is retained unchanged.scripts/find_overlap.py.Validation
Linux x86_64, CPython 3.12.3. The four new CLI regression groups (a lead displaced below the default limit, exclusively nested packages, explicit generic terms and
--all-termspreserved, catalogue terms not treated as stopwords) use isolated offline snapshots and check that the cache bytes and coverage metadata are unchanged.Credit
Douglas Colkitt and the contributors to the overlap helper and its tests, including Nick Petalas's pagination repair. Apache-2.0.
AI assistance: submitted by @commdata2338 with substantial AI assistance. OpenAI Codex (GPT-6 Astra) found the defect and wrote the fix, the tests and the evidence; it was packaged and submitted with Claude Code. No independent human review is claimed.
Open next questions
None justified beyond this fix. A broader change to ranking would need its own relevance fixtures and overlap check.
Workflow feedback (optional)
--outputpath as suggested in PR Scoped proof: PR1918's selected centre-source minor lifts to native rank 8192 (no new kappa) #2048, would keep the first draw durable.🤖 Generated with Claude Code
https://claude.ai/code/session_01X86sCSXdoDd61SDFfC1PKG