Skip to content

refactor(tools): remove partial read support from read-file#60

Merged
cniska merged 8 commits intomainfrom
remove-partial-reads
Mar 25, 2026
Merged

refactor(tools): remove partial read support from read-file#60
cniska merged 8 commits intomainfrom
remove-partial-reads

Conversation

@cniska
Copy link
Owner

@cniska cniska commented Mar 25, 2026

Summary

  • remove start/end line range params from read-file tool, always read full files
  • rename readSnippet/readSnippets to readFileContent/readFileContents
  • replace silent output truncation with 2000-line hard limit (80k char budget)
  • fix tool cache populateSubEntries to handle structured result objects
  • simplify file-churn guard to track paths only, no range signatures
  • allow re-reading edited files once for follow-up edits (fixes guard deadlock)
  • block lifecycle-managed commands (lint/format/verify) via run-command in work mode
  • add preamble instruction to run only changed test files, not full suite
  • fix /clear and /new duplicate header key warning in TUI
  • remove dead code: toInt, formatReadDetail, normalizeReadEntries, range types

@cniska cniska changed the title Remove partial read support and fix tool guards refactor(tools): remove partial read support from read-file Mar 25, 2026
@cniska cniska force-pushed the remove-partial-reads branch 5 times, most recently from cb1af96 to 0552dd9 Compare March 25, 2026 12:56
@cniska cniska force-pushed the remove-partial-reads branch from 0552dd9 to d027b1e Compare March 25, 2026 14:16
@cniska cniska merged commit a4c469b into main Mar 25, 2026
2 checks passed
@cniska cniska deleted the remove-partial-reads branch March 25, 2026 14:42
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