Skip to content

fix(prompts): fix debug.eta and small partial issues for consistent agent prompts#988

Merged
aaight merged 1 commit intodevfrom
fix/debug-eta-and-partial-fixes
Mar 23, 2026
Merged

fix(prompts): fix debug.eta and small partial issues for consistent agent prompts#988
aaight merged 1 commit intodevfrom
fix/debug-eta-and-partial-fixes

Conversation

@aaight
Copy link
Copy Markdown
Collaborator

@aaight aaight commented Mar 23, 2026

Summary

Fixes several small issues in agent prompt templates to improve consistency and completeness.

  • debug.eta: Fixed listDirectoryListDirectory (capitalized), added RipGrep and Tmux to Available Gadgets section, and added partials/environment and partials/tmux partial includes so the debug agent has full shell access instructions
  • environment.eta: Removed vague "Refactoring: Use the gadgets available..." bullet that was redundant with the detailed tool selection guidance in rules-efficiency.eta
  • pattern-discovery.eta: Added Squint-aware conditional — when squintEnabled is true, recommends squint symbols show --file <path> --json for understanding similar files before reading source; existing instructions remain as the fallback
  • splitting.eta: Moved the "## Context Variables" section (BACKLOG_LIST_ID, PROCESSED_LABEL_ID) from its standalone position after CRITICAL rules to inline, right before "## Your Task" where the variables are actually referenced

Test plan

  • All existing unit tests pass (npm test — 332 files, 6292 tests)
  • Lint passes (npm run lint — no issues)
  • debug.eta uses ListDirectory (capitalized)
  • debug.eta Available Gadgets lists RipGrep and Tmux
  • debug.eta includes partials/environment and partials/tmux
  • environment.eta no longer has vague Refactoring bullet
  • pattern-discovery.eta includes squint-aware conditional for squint symbols show
  • splitting.eta Context Variables section positioned near "Your Task"

Closes: https://trello.com/c/adpRz0RK/512-as-a-developer-i-want-debugeta-and-small-partial-issues-fixed-so-that-all-agent-prompts-are-consistent-and-complete

🤖 Generated with Claude Code

🕵️ claude-code · claude-sonnet-4-6 · run details

Copy link
Copy Markdown
Collaborator

@nhopeatall nhopeatall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary

LGTM — Clean set of prompt template fixes that improve consistency across agent prompts. All changes are straightforward and correct.

Verified:

  • debug.eta: listDirectoryListDirectory capitalization fix matches the actual gadget name. Added RipGrep and Tmux to Available Gadgets, which aligns with the newly included partials. The partials/environment and partials/tmux includes are consistent with how other agents (implementation, review, respond-to-ci, etc.) include these partials.
  • environment.eta: The removed "Refactoring" bullet was genuinely redundant — rules-efficiency.eta already covers AstGrep usage with a detailed decision table and syntax guide.
  • pattern-discovery.eta: The squint conditional is properly guarded by it.squintEnabled (which is always present in the prompt context via buildPromptContext). The template syntax is correct and the existing non-squint path is preserved as fallback.
  • splitting.eta: Context Variables section moved from line ~11 to line ~89, immediately before "Your Task" at line ~94. The variables BACKLOG_LIST_ID and PROCESSED_LABEL_ID are referenced later in the template (line 106, 216), so the new position is more logical — closer to where they're first used.

All CI checks pass (332 files, 6292 tests, lint clean).

🕵️ claude-code · claude-opus-4-6 · run details

@aaight aaight merged commit 47560dc into dev Mar 23, 2026
9 checks passed
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