Skip to content

fix(tools): steer ReadRange away from empty-file read loops#560

Merged
dwash96 merged 1 commit into
cecli-dev:v0.100.5from
Digital-Defiance:pr/read-range-empty-hint
Jun 6, 2026
Merged

fix(tools): steer ReadRange away from empty-file read loops#560
dwash96 merged 1 commit into
cecli-dev:v0.100.5from
Digital-Defiance:pr/read-range-empty-hint

Conversation

@JessicaMulein
Copy link
Copy Markdown

@JessicaMulein JessicaMulein commented Jun 6, 2026

Summary

  • When ReadRange hits an empty file, return an explicit next-step hint (EditText @000 or ContextManager) instead of a silent no-op.
  • Route the message through new_context_details so it appears in tool output (previously written to all_outputs, which was never returned).

Base branch

Targets v0.100.5 (includes #557 validation pipeline) — not main. Merge after #557 lands, or stack into v0.100.5 before release.

Motivation

Local models on greenfield repos loop ReadRange on empty pubspec.yaml / STEERING.md until repetition detection fires. BrightVision spec-focus implement sessions hit this in production dogfood.

Test plan

  • python -m pytest cecli/tests/tools/test_get_lines.py -q

Rebased onto v0.100.5 validation pipeline (cecli-dev#557). Route empty-file guidance
through new_context_details so the model sees it. Hint EditText @000 or
ContextManager instead of re-reading empty files.

Co-authored-by: Cursor <cursoragent@cursor.com>
@JessicaMulein JessicaMulein force-pushed the pr/read-range-empty-hint branch from 227b343 to ef26a64 Compare June 6, 2026 16:55
@JessicaMulein JessicaMulein changed the base branch from main to v0.100.5 June 6, 2026 16:55
@dwash96 dwash96 merged commit 6cec132 into cecli-dev:v0.100.5 Jun 6, 2026
8 of 12 checks passed
@dwash96 dwash96 mentioned this pull request Jun 6, 2026
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.

2 participants