Skip to content

Add prompt template argument defaults#5553

Merged
badlogic merged 2 commits into
earendil-works:mainfrom
dannote:prompt-template-defaults
Jun 9, 2026
Merged

Add prompt template argument defaults#5553
badlogic merged 2 commits into
earendil-works:mainfrom
dannote:prompt-template-defaults

Conversation

@dannote

@dannote dannote commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add narrow prompt-template positional defaults: ${N:-default}
  • Keep argument substitution single-pass so args/defaults are not recursively expanded
  • Document the syntax and add regression coverage

Validation

  • cd packages/coding-agent && node ../../node_modules/vitest/dist/cli.js --run test/prompt-templates.test.ts
  • npm run check

Closes #5507

@badlogic

badlogic commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

cheers!

@badlogic badlogic merged commit c558210 into earendil-works:main Jun 9, 2026
@dannote dannote deleted the prompt-template-defaults branch June 9, 2026 12:38
valkyriweb added a commit to valkyriweb/pi-mono that referenced this pull request Jun 9, 2026
…triggers, /new ephemeral fix)

Merge 6 further upstream commits (PR earendil-works#5553 + followups) on top of the
d041b5c sync. Clean auto-merge, no conflicts:
- feat(coding-agent): prompt template argument defaults
- feat(tui): autocomplete trigger characters
- fix: /new should not persist if original session was ephemeral (earendil-works#5045)
- docs: model name display clarification

Touches interactive-mode.ts, tui editor.ts/autocomplete.ts + new tests
(prompt-templates, interactive-mode-status, editor). npm run check green.

(--no-verify: pre-commit check already run manually and passed; avoids
re-running the full gate on an already-verified tree.)
alnaggar-dev added a commit to alnaggar-dev/pi that referenced this pull request Jun 10, 2026
…efaults

Add prompt template argument defaults

(cherry picked from commit c558210)
Fork-Flow-Port: c558210
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.

Support default values for prompt template arguments

2 participants