Skip to content

phase3/integration-modify-chain: chain 9 Modify verbs end-to-end - #107

Open
URaux wants to merge 3 commits into
phase2/w2from
phase3/integration-modify-chain
Open

URaux wants to merge 3 commits into
phase2/w2from
phase3/integration-modify-chain

Conversation

@URaux

@URaux URaux commented Apr 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • Integration test in tests/integration/modify-chain.test.ts chains 9 Modify verbs sequentially on a single TS file
  • Vendors all Modify v0.4 implementations (add-import, sort-imports, remove-unused-imports, replace-in-file, insert-method, add-export) onto branch
  • Verb sequence: planExtract → planAddImport → planReplaceInFile → planAddExport → planInsertMethod → planRemoveUnusedImports → planSortImports → planAddExport (idempotency)
  • Final step: runs tsc --noEmit in a temp dir to confirm output type-checks

Test plan

  • 3 tests: full 9-verb chain + tsc validation, zero-conflict isolation for each verb, replace+export cooperation
  • 60s timeout for the full chain (tsc included)

URaux and others added 3 commits April 27, 2026 00:30
Adds noise-suppression lists to policy.drift. Block/container/edge IDs in
the ignore lists get filtered out of every bucket before drift is reported.
Pure function; never mutates input. clean flag is recomputed after filtering.

Use case: auto-generated blocks (cache layers, codegen output, archived
containers) that legitimately drift but shouldn't trigger PR comments.

5 tests covering: pass-through when no lists set, multi-bucket filtering,
container+edge filters, clean recomputation, no-mutation invariant.
Adds scripts/drift-stats.mjs that reads a folder of drift snapshot JSON
files and prints per-week avg/median/max drift counts and top-N most-drifted
blocks. Supports --last N, --top N, --format md|json. 10 tests covering
empty dir, populated multi-week, --last capping, and top-N block ranking.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Vendors all Modify v0.4 verb implementations onto branch (add-import,
sort-imports, remove-unused-imports, replace-in-file, insert-method,
add-export, types). Integration test chains: extract → add-import →
replace-in-file → add-export → insert-method → remove-unused-imports →
sort-imports → idempotent add-export, then asserts tsc --noEmit passes.
3 tests; 60s timeout for the full chain.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant