Skip to content

fix(tools): BrightVision patches for v0.100.6#572

Open
JessicaMulein wants to merge 5 commits into
cecli-dev:mainfrom
Digital-Defiance:brightvision/v0.100.6-patches
Open

fix(tools): BrightVision patches for v0.100.6#572
JessicaMulein wants to merge 5 commits into
cecli-dev:mainfrom
Digital-Defiance:brightvision/v0.100.6-patches

Conversation

@JessicaMulein

Copy link
Copy Markdown

Summary

Five patches from BrightVision integration work, rebased onto v0.100.6.

  1. fix(add): do not create files under host attachment staging paths
  2. fix(repomap): resolve absolute paths before existence checks
  3. fix(tools): coerce ReadRange start_text/end_text to str
  4. fix(tools): ReadRange resilience for weak model inputs (supersedes fix(tools): ReadRange resilience for weak model inputs #567)
  5. feat(tools): emit full tool output for frontend display (ls + Command)

Test plan

  • Tested against BrightVision headless agent (Ollama qwen3.6:35b + gemma4:26b)
  • No cecli test regressions

Closes #567

JessicaMulein and others added 5 commits June 13, 2026 11:30
Headless/desktop hosts stage chat uploads under .<app>/attachments/.
When /add targets a missing file there, error instead of prompting to
create path segments (e.g. after a failed image attach).

Co-authored-by: Cursor <cursoragent@cursor.com>
Cherry-pick repomap-only hunk from 50e9b27; omit grep hint (superseded by v0.100.5 validation pipeline / cecli-dev#557).

Co-authored-by: Cursor <cursoragent@cursor.com>
Models sometimes pass line numbers as int; str() before strip/count ops
avoids AttributeError in execute, format_output, and format_error.

Co-authored-by: Cursor <cursoragent@cursor.com>
- is_provided() now rejects common junk values ('N/A', 'null', 'none')
  that small models pass instead of real text markers

- ReadRange auto-fallback: when start_text/end_text are missing or junk
  but the file is already in editable or read-only context, show the
  whole file (@000 → 000@) instead of erroring. Files not in context
  still get the strict error.

Reduces wasted turns on quantized local models (qwen3.6:27b-q4_K_M)
that struggle with ReadRange's start_text/end_text contract.
- ls: emit full sorted file listing via tool_output (UI shows in collapsible section)
- Command: always emit stdout/stderr via tool_output (was TUI-only)
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