Skip to content

Let advisors recover truncated review context - #262

Merged
fcakyon merged 2 commits into
mainfrom
fix/advisor-truncated-context
Jul 29, 2026
Merged

Let advisors recover truncated review context#262
fcakyon merged 2 commits into
mainfrom
fix/advisor-truncated-context

Conversation

@fcakyon

@fcakyon fcakyon commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Both advisors refused to review instead of ruling: the hooks shredded tool output, then the prompts banned the file reads that could have recovered it.

before after
tool_use cap 280 8000
tool_result cap 700 8000
context used of 160k budget 22,365 115,845
  • reviewers get read-only access, so they grep what the window drops
  • the 80-record cutoff now announces itself instead of passing silently
  • fable blocked reads in two more places: tools: [] and --tools ""
-return `  ↳ result: ${clip(text, 700)}`;
+return `  ↳ result: ${clip(text, 8000)}`;
+if (i >= 0) recent = `…[this is only the newest ${records.length} turns of a longer session]\n` + recent;

Both advisor plugins clipped tool output to 280 chars inside a 160k budget and forbade file reads, so reviewers refused instead of ruling.
@fcakyon fcakyon self-assigned this Jul 29, 2026
@fcakyon
fcakyon merged commit c6636b9 into main Jul 29, 2026
2 checks passed
@fcakyon
fcakyon deleted the fix/advisor-truncated-context branch July 29, 2026 01:00
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