Skip to content

optimize: stop counting new-file writes in the read/edit ratio - #1761

Open
iamtoruk wants to merge 1 commit into
mainfrom
fix/read-edit-ratio-ignore-new-files
Open

iamtoruk wants to merge 1 commit into
mainfrom
fix/read-edit-ratio-ignore-new-files

Conversation

@iamtoruk

Copy link
Copy Markdown
Member

Writing a new file has nothing to read first, so setups where subagents write report and evidence files got a false "Claude edits more than it reads" finding (reported by a multi-agent team: 1.2:1, almost all of it new files).

  • detectLowReadEditRatio now counts only Edit, FileEditTool and NotebookEdit as edits.
  • EDIT_TOOL_NAMES is unchanged on purpose: act report baselines for already-applied fixes were recorded with it, so changing it would skew their verdicts.
  • Test flipped: 30 Writes no longer change the ratio.

tsc clean, optimize, optimize-fs and act-report tests pass (218).

Writing a new file has nothing to read first, so teams whose subagents
write report and evidence files got a false 'edits more than it reads'
finding. The detector now counts only Edit-type calls. EDIT_TOOL_NAMES
stays as is because act report baselines were recorded with it.

This branch has not been deployed

No deployments
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