Skip to content

fix(tools): BrightVision patches for v0.100.7#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.7#572
JessicaMulein wants to merge 5 commits into
cecli-dev:mainfrom
Digital-Defiance:brightvision/v0.100.6-patches

Conversation

@JessicaMulein

@JessicaMulein JessicaMulein commented Jun 13, 2026

Copy link
Copy Markdown

Summary

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

  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 2 commits June 15, 2026 09:23
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>
@JessicaMulein JessicaMulein changed the title fix(tools): BrightVision patches for v0.100.6 fix(tools): BrightVision patches for v0.100.7 Jun 15, 2026
JessicaMulein and others added 3 commits June 15, 2026 09:25
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)
@JessicaMulein JessicaMulein force-pushed the brightvision/v0.100.6-patches branch from f975771 to d815e24 Compare June 15, 2026 16:26
@JessicaMulein

Copy link
Copy Markdown
Author

updated to v0.100.7

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