-
Notifications
You must be signed in to change notification settings - Fork 8
Add --apply flag to write fixes in place #62
Copy link
Copy link
Open
Labels
area:cliiac-scan CLI (click) / entry pointsiac-scan CLI (click) / entry pointsarea:orchestrationAnalysis + fix pipeline, LangChain tasks, hybrid runnerAnalysis + fix pipeline, LangChain tasks, hybrid runnerhelp wantedMaintainers welcome a PR hereMaintainers welcome a PR herepriority:highBlocks a supported use caseBlocks a supported use casetype:featureNew user-facing capabilityNew user-facing capability
Milestone
Description
Metadata
Metadata
Assignees
Labels
area:cliiac-scan CLI (click) / entry pointsiac-scan CLI (click) / entry pointsarea:orchestrationAnalysis + fix pipeline, LangChain tasks, hybrid runnerAnalysis + fix pipeline, LangChain tasks, hybrid runnerhelp wantedMaintainers welcome a PR hereMaintainers welcome a PR herepriority:highBlocks a supported use caseBlocks a supported use casetype:featureNew user-facing capabilityNew user-facing capability
Summary
Add an
--applyflag that writes the LLM-fixed code back into the working tree (in place), instead of only intofixed/. Today the tool never mutates the user's files — you always have to copy fromfixed/manually.Why
Several downstream features want this (the hybrid post-fix re-verification in the related issue notes that a temp-dir dance is "cleaner with an
--applyflow"). It's also the single most-requested ergonomic improvement for the fix workflow.Design notes / guardrails
--applymust be opt-in and loud: print exactly which files will be overwritten, and honor the AI-generated banner policy.--apply --force), so users can alwaysgit diffthe result.fixed/only) unchanged.--apply --dry-runthat lists the diff without writing.Acceptance criteria
--applywrites fixes in place with a clear pre-write summary.--force).--apply) behavior unchanged.Notes
Design-sensitive (it's the one path that mutates user files). Please open a short design comment before coding.
area:cli,area:orchestration. Not a first issue.