docvet: suppress the heuristic scan's false positives - #70
Open
inth3shadows wants to merge 1 commit into
Open
Conversation
testgraph reads OTHER repositories, so its docs are full of filenames and symbols belonging to the repos under analysis rather than to this tree. All 38 candidates `docvet --triage` reported are false positives, checked by hand: - package.json, route.ts, page.tsx, urls.py, schemas.py, available-locales.ts and friends are files in target repos, named as examples. testgraph is a Python tool and never claimed to have them. - path:NN entries are line references; testgraph/record.py, testgraph/hook.py and tests/test_propose.py are all tracked, the ":NN" just isn't part of the name. - codegraph_impact is a CodeGraph MCP tool; imp.localName is an ESTree/Babel AST property; clean_db is a fixture in a target repo's conftest. - /verify is a gate, /autorun a slash-command, /path/to/repo a shell placeholder -- none is an HTTP route. testgraph now reports 0 candidates and 0 broken anchors.
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
1 participant
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.
testgraph reads other repositories, so its docs are full of filenames and symbols belonging to the repos under analysis rather than to this tree. All 38 candidates
docvet --triagereported are false positives, checked by hand:package.json,route.ts,page.tsx,urls.py,schemas.py,available-locales.tsand friends — files in target repos, named as examples. testgraph is a Python tool and never claimed to have them.path:NNentries — line references.testgraph/record.py,testgraph/hook.pyandtests/test_propose.pyare all tracked; the:NNjust isn't part of the name.codegraph_impactis a CodeGraph MCP tool,imp.localNamean ESTree/Babel AST property,clean_dba fixture in a target repo'sconftest.py./verifyis a gate,/autoruna slash-command,/path/to/repoa shell placeholder — none is an HTTP route.Suppression, not anchoring. An anchor is a verified
token -> pathclaim and the only docvet tier allowed to block a push; none of these qualifies.docvet --triagenow reports 0 candidates, 0 broken anchors. No code changes.