Summary
desloppify show <file> can return stale findings even after a fresh desloppify scan --path . updates the detector counts and rewrites query.json.
Version
Minimal repro
- Run
desloppify scan --path . in a TypeScript repo.
- Remove an actually-unused import and a state-sync anti-pattern from a file. In my repro file,
apps/web/src/components/plan/AddDestinationModal.tsx, I removed:
showErrorToast from the imports
- the
useEffect(() => setIsClient(true), []) pattern
- Re-run
desloppify scan --path ..
- Observe the scan summary reflects the change:
Unused (tsc)... 133 -> 132
react: 5 state sync anti-patterns -> 4
- Run
desloppify show apps/web/src/components/plan/AddDestinationModal.tsx.
Actual
show still reports the deleted findings for that file:
Unused imports: showErrorToast
State sync anti-pattern: useEffect only calls setIsClient
It also still points at the old line/issue ids even though the current file content no longer contains either symbol.
Expected
After a successful fresh scan, desloppify show <file> should reflect the current scan state for that file and should not continue returning findings that were removed from the source tree.
Notes
desloppify scan --path . printed ? query.json updated on the successful rerun.
- The stale results were reproducible immediately after the rerun, not after a long-lived shell session.
Summary
desloppify show <file>can return stale findings even after a freshdesloppify scan --path .updates the detector counts and rewritesquery.json.Version
desloppify 0.9.14Minimal repro
desloppify scan --path .in a TypeScript repo.apps/web/src/components/plan/AddDestinationModal.tsx, I removed:showErrorToastfrom the importsuseEffect(() => setIsClient(true), [])patterndesloppify scan --path ..Unused (tsc)... 133 -> 132react: 5 state sync anti-patterns -> 4desloppify show apps/web/src/components/plan/AddDestinationModal.tsx.Actual
showstill reports the deleted findings for that file:Unused imports: showErrorToastState sync anti-pattern: useEffect only calls setIsClientIt also still points at the old line/issue ids even though the current file content no longer contains either symbol.
Expected
After a successful fresh scan,
desloppify show <file>should reflect the current scan state for that file and should not continue returning findings that were removed from the source tree.Notes
desloppify scan --path .printed? query.json updatedon the successful rerun.