fix: dead token-budget tracking after working memory section in BuildPrompt

[claude]

Problem

In internal/template/render.go, BuildPrompt tracks a token remaining budget and decrements it as sections are appended. At lines 305-307, the budget is decremented after writing the working memory section, then immediately discarded:

wmSection := buildWorkingMemorySection(wm, remaining)
if wmSection != "" {
    sb.WriteString(wmSection)
    remaining -= CountTokens(wmSection)
    _ = remaining   // value is never read after this
}

The _ = remaining confirms the value is dead. The footer on line 311 is always appended unconditionally without a budget check. This means:

1. The remaining -= line is dead code.
2. The footer appends even when the prompt is already at or over the token budget.
3. Any section added after working memory in the future would use the wrong (pre-working-memory) remaining value.

Fix

Remove the dead remaining -= and _ = remaining lines. If the footer should respect the token budget, add a guard before appending it.

Location

• internal/template/render.go, lines 303-311 (BuildPrompt)

@claude please implement this

[claude]

Claude finished @claude[bot]'s task in 1m 42s —— View job • claude/issue-264-20260228-1511 • [Create PR ➔](https://github.com/supermodeltools/Uncompact/compare/main...claude/issue-264-20260228-1511?quick_pull=1&title=Issue%20%23264%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%23264%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

---

Tasks

• Gather context and understand request
• Read internal/template/render.go to analyze the code
• Remove dead _ = remaining suppressor (keep remaining -= since it now feeds footer guard)
• Add footer budget guard
• Build to verify (go build ./...)
• Commit and push changes
• Create PR

Summary

Fixed dead token-budget tracking in BuildPrompt (internal/template/render.go):

• Removed _ = remaining — the dead-code suppressor is gone
• Kept remaining -= CountTokens(wmSection) — now meaningful because remaining is consumed by the footer guard below
• Added a budget guard before appending the footer, making token-budget tracking consistent end-to-end

PR: #265

[coderabbitai]

🔗 Related PRs

#13 - fix: correct context bomb zeros for lines and language counts [merged]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!