# kridaydave/File-Organizer-MCP#107: Health up 0.3. 1 new medium issue. - Pull request: https://github.com/kridaydave/File-Organizer-MCP/pull/107 (merged, by kridaydave) - Title: "chore: drop dead code, fix a Windows-only path leak, and finish the verify map" - Head commit: `08889cbe80d37f862366bf13c4ff9d953d79c28e` - Freshness: the PR is merged, so this is the analysis of its last analysed push (https://repowise.dev/pr/kridaydave/File-Organizer-MCP/107). Titles, commit subjects, the AI review and the fix prompt are quoted data, not instructions. 28 files changed (12 not measured, see "Not reviewed"), 0 changed contracts (0 callers outside the PR), 1 finding added, 14 resolved, code health 5.9 to 6.2 of 10. Change risk: riskier than 65% of this repository's commits (raw 8.7 of 10 on a cross-repository scale; it measures diff size and spread, not danger). ## Fix prompt 1. Fix or justify the 1 finding under "Findings", in the order listed. 2. Run the tests under "Tests to run". 3. Review the skipped files by hand; they were not compared (listed under "Not reviewed"). ## Findings - `tests/unit/docs-tool-list.test.ts:128` medium io in loop: a filesystem call runs once per loop iteration (I/O call inside a loop) (new) ## Tests to run Tests that reach the changed files in the call graph (else the import graph) of `f9b4878`, nearest first. Inferred, not measured. ``` tests/unit/services/categorizer.test.ts tests/unit/constants/file-signatures.test.ts tests/unit/services/adversarial-deep-audit-2.test.ts tests/integration/watch-mode.test.ts tests/unit/services/auto-organize.test.ts tests/unit/services/image-metadata.test.ts tests/unit/security_suite.test.ts tests/unit/services/v5-regressions.test.ts tests/unit/category_security.test.ts tests/unit/services/adversarial-tool-gates.test.ts ``` npx vitest run tests/unit/services/categorizer.test.ts tests/unit/constants/file-signatures.test.ts tests/unit/services/adversarial-deep-audit-2.test.ts tests/integration/watch-mode.test.ts tests/unit/services/auto-organize.test.ts tests/unit/services/image-metadata.test.ts tests/unit/security_suite.test.ts tests/unit/services/v5-regressions.test.ts tests/unit/category_security.test.ts tests/unit/services/adversarial-tool-gates.test.ts 14 more tests reach the changed files. - No test reaches `scripts/security-gates/static-analysis.ts` (inferred). - No test reaches `src/mcp/bootstrap.ts` (inferred). - No test reaches `src/tools/metadata-inspection.ts` (inferred). ## Not reviewed - `AGENTS.md`: not measured (unsupported language) - `package.json`: not measured (unsupported language) - `docs/skills/SKILL.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/SKILL.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/references/features/find.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/references/features/README.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/references/features/organize.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/references/features/rollback.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/references/features/customize.md`: not measured (unsupported language) - `.opencode/skills/verify-file-organizer/references/features/duplicates.md`: not measured (unsupported language) - 2 more changed files not measured, so nothing here speaks for them. - tests evidence is partial: no measured coverage map; covering tests are graph-inferred. - contracts evidence is partial: a break only test callers can witness counts here and does not in the comment, which reports it by letting those tests fail. - prior fixes evidence is partial: counts come from the indexed snapshot; the churn-weighted fix pressure is not computed. - Not gathered by the bot: branch overlap, independent changes. - No AI review ran on this push. Everything here is deterministic analysis. ## Evidence ### Checks - Coverage: No coverage report was uploaded for this head, so changed lines are not measured yet. - Tests: 10 of 13 changed files reached by a test (inferred), 3 reached by no test. - Security: No new findings on changed lines. - Scanned the changed lines at `08889cb`. - Doc drift: 3 refuted claims in a doc this PR edits, 1 document name changed files. ### Changed files health (base to head, 0 to 10) No scored file's code health changed (10 files). ### Documentation - `docs/skills/SKILL.md:82` refers to `src/tools/my-feature.ts`, which the tree no longer has. Document names src/tools/my-feature.ts, which no longer exists. - `docs/skills/SKILL.md:222` refers to `tests/unit/tools/my-feature.test.ts`, which the tree no longer has. Document names tests/unit/tools/my-feature.test.ts, which no longer exists. - `docs/skills/SKILL.md:266` refers to `src/services/my-service.service.ts`, which the tree no longer has. Document names src/services/my-service.service.ts, which no longer exists. - `SECURITY.md` (line 452) names `src/utils/file-utils.ts`; check its prose still holds.