agent/pi - pi - Penguin

Release v0.34.0

Mario Zechner · 2026-01-04 20:29:50 +01:00

dc2f5d43cc

revert: remove unnecessary themeOverride params from theme functions

The optional theme parameter was added as a workaround for tsx dev mode,
but that's a dev-only issue. Users running the built package don't need it.

Mario Zechner · 2026-01-04 20:27:46 +01:00

00fa8b71c8

docs: remove dev-only changelog entry

Mario Zechner · 2026-01-04 20:26:29 +01:00

7302fe5063

docs: update CHANGELOG and hooks.md with missing items

- tools.ts example hook
- Multiple messages from before_agent_start
- getSettingsListTheme/getSelectListTheme optional theme param
- Hook error stack traces

Mario Zechner · 2026-01-04 19:01:35 +01:00

5318b42bd2

fix(tools.ts): sync with active tools when no saved state

Hook was showing all tools as enabled even though only 4 were active.
Now initializes from pi.getActiveTools() to match actual state.

Mario Zechner · 2026-01-04 18:50:45 +01:00

b06bd1d24c

fix(tools): wrap ALL registry tools with hooks, not just active ones

wrappedToolRegistry was only containing activeToolsArray (4 tools).
Now wraps all tools from the registry so hooks can enable any tool.

Mario Zechner · 2026-01-04 18:47:57 +01:00

31438fdf2a

fix(tools): tool registry now contains ALL built-in tools

- createAllTools() populates registry with all 7 built-in tools
- --tools flag only sets initially active tools (default: read/bash/edit/write)
- Hooks can enable any tool from registry via setActiveTools()
- System prompt rebuilds with correct tool guidelines when tools change
- Document tsx module resolution workaround in README

Mario Zechner · 2026-01-04 18:44:41 +01:00

2849623afc

fix(theme): add optional themeOverride param to getSettingsListTheme/getSelectListTheme

When hooks are loaded via jiti, they get a separate module instance from
the main app. This means the global 'theme' variable in the hook's module
is never initialized. Adding an optional theme parameter allows hooks to
pass the theme from ctx.ui.custom() callback.

Usage in hooks:
  getSettingsListTheme(theme)  // theme from ctx.ui.custom callback

Mario Zechner · 2026-01-04 18:39:00 +01:00

c447e62662

fix(hooks): add stack traces to hook errors, fix tools.ts theme bug

- HookError now includes optional stack field
- Hook error display shows stack trace in dim color below error message
- tools.ts: create SettingsListTheme using the theme passed to ctx.ui.custom()
  instead of using getSettingsListTheme() which depends on global theme

Mario Zechner · 2026-01-04 18:33:28 +01:00

6390ff87ef

feat(hooks): add systemPromptAppend to before_agent_start, full tool registry

- before_agent_start handlers can return systemPromptAppend to dynamically
  append text to the system prompt for that turn
- Multiple hooks' systemPromptAppend strings are concatenated
- Multiple hooks' messages are now all injected (not just first)
- Tool registry now contains ALL built-in tools (read, bash, edit, write,
  grep, find, ls) regardless of --tools flag
- --tools only sets initially active tools, hooks can enable any via
  setActiveTools()
- System prompt automatically rebuilds when tools change, updating tool
  descriptions and guidelines
- Add pirate.ts example hook demonstrating systemPromptAppend
- Update hooks.md with systemPromptAppend documentation

Mario Zechner · 2026-01-04 18:21:26 +01:00

e4dd21a3b2

feat(hooks): add tools.ts example hook for interactive tool enable/disable

- /tools command opens SettingsList-based selector for all loaded tools
- Space/Enter toggles individual tools between enabled/disabled
- Changes apply immediately on toggle (like /settings)
- Tool selection persisted to session via appendEntry()
- State restored from current branch on session_start, session_tree, session_branch
- Uses getBranch() to respect branch-specific tool configuration
- Export getSettingsListTheme and getSelectListTheme for hooks to use

Mario Zechner · 2026-01-04 18:13:30 +01:00

892acedb6b

fix(hooks): deep copy messages in context event before passing to hooks

The context event handler documentation promised a deep copy but the
implementation passed the original array reference. This could cause
hooks to accidentally mutate session messages.

Uses structuredClone() for fast native deep copying.

Mario Zechner · 2026-01-04 18:13:30 +01:00

ddf8bfceee

refactor: address PR feedback - merge setWidget, use KeyId for shortcuts

1. Merge setWidget and setWidgetComponent into single overloaded method
   - Accepts either string[] or component factory function
   - Uses single Map<string, Component> internally
   - String arrays wrapped in Container with Text components

2. Use KeyId type for registerShortcut instead of plain string
   - Import Key from @mariozechner/pi-tui
   - Update plan-mode example to use Key.shift('p')
   - Type-safe shortcut registration

3. Fix tool API docs
   - Both built-in and custom tools can be enabled/disabled
   - Removed incorrect 'custom tools always active' statement

4. Use matchesKey instead of matchShortcut (already done in rebase)

Helmut Januschka · 2026-01-04 18:13:30 +01:00

8ecb1d6c0b

fix: use robust matchShortcut from TUI library

- Add matchShortcut() function to @mariozechner/pi-tui
- Handles Kitty protocol, legacy terminal sequences, and lock keys
- Supports special keys (enter, tab, space, backspace, escape)
- Replace custom implementation in interactive-mode.ts
- Remove unused imports