Analyzed and merged GitHub PR with new keybindingsPublic

Agent

Model

Claude Opus 4.5 (latest)

Started

Jan 31, 9:57 PM

Messages71

Downloads0

Summary

Closed, Completed

Understand
Reviewed linked issue and source files.
Understand
Verify
Confirmed PR merge and changelog update.
Verify

You are given one or more GitHub PR URLs: https://github.com/badlogic/pi-mono/pull/1114

For each PR URL, do the following in order:

Read the PR page in full. Include description, all comments, all commits, and all changed files.
Identify any linked issues referenced in the PR body, comments, commit messages, or cross links. Read each issue in full, including all comments.
Analyze the PR diff. Read all relevant code files in full with no truncation from the current main branch and compare against the diff. Do not fetch PR file blobs unless a file is missing on main or the diff context is insufficient. Include related code paths that are not in the diff but are required to validate behavior.
Check for a changelog entry in the relevant packages/*/CHANGELOG.md files. Report whether an entry exists. If missing, state that a changelog entry is required before merge and that you will add it if the user decides to merge. Follow the changelog format rules in AGENTS.md. Verify:
- Entry uses correct section (### Breaking Changes, ### Added, ### Fixed, etc.)
- External contributions include PR link and author: Fixed foo ([#123](https://github.com/badlogic/pi-mono/pull/123) by [@user](https://github.com/user))
- Breaking changes are in ### Breaking Changes, not just ### Fixed
Check if packages/coding-agent/README.md, packages/coding-agent/docs/.md, packages/coding-agent/examples/**/.md require modification. This is usually the case when existing features have been changed, or new features have been added.
Provide a structured review with these sections:
- Good: solid choices or improvements
- Bad: concrete issues, regressions, missing tests, or risks
- Ugly: subtle or high impact problems
Add Questions or Assumptions if anything is unclear.
Add Change summary and Tests.

Output format per PR: PR: Changelog:

... Good:
... Bad:
... Ugly:
... Questions or Assumptions:
... Change summary:
... Tests:
...

If no issues are found, say so under Bad and Ugly.

Let me read the linked issue #1088 and the relevant source files:

Let me continue reading the interactive-mode.ts file to see the methods being called:

Now let me check the changelog and the docs that were modified:

Let me check the specific methods being called and verify they exist:

Let me check if the README needs updates by looking at the relevant keybinding section:

PR: https://github.com/badlogic/pi-mono/pull/1114

Changelog:

Missing: No entry in packages/coding-agent/CHANGELOG.md under ## [Unreleased]. A changelog entry is required before merge. I will add it if you decide to merge.

Good:

Clean implementation that reuses existing methods (handleClearCommand(), showTreeSelector(), showUserMessageSelector()) instead of duplicating code
All three actions are unbound by default (empty arrays), allowing users to opt-in via keybindings.json
Follows existing patterns for keybinding registration: adds to AppAction type, DEFAULT_APP_KEYBINDINGS, and APP_ACTIONS array consistently
Documentation update in keybindings.md adds a new "Session" section with clear descriptions
PR author correctly notes the design choice that keybinding handlers don't clear the editor (unlike / commands), which lets users draft a message before navigating
Closes the linked issue #1088 which was explicitly approved by repo owner

Bad:

None

Ugly:

None

Questions or Assumptions:

None. The implementation is straightforward and matches the issue request.

Change summary:

packages/coding-agent/src/core/keybindings.ts: Adds newSession, tree, and fork to AppAction type union, adds them to DEFAULT_APP_KEYBINDINGS with empty arrays (unbound), and adds them to APP_ACTIONS array for type checking
packages/coding-agent/src/modes/interactive/interactive-mode.ts: Registers handlers for the three new actions via onAction() that delegate to existing methods
packages/coding-agent/docs/keybindings.md: Adds new "Session" section documenting the three keybinding actions

Tests:

No automated tests added. This is acceptable given the nature of the change (keybinding wiring to existing functionality). The existing functionality (handleClearCommand, showTreeSelector, showUserMessageSelector) is already tested through the /new, /tree, and /fork commands.

good to merge then?

Yes, code is solid. I'll add the changelog entry and merge.

continue

Done. Merged to main with changelog entry added:

### Added- Added `newSession`, `tree`, and `fork` keybinding actions for `/new`, `/tree`, and `/fork` commands. All unbound by default. ([#1114](https://github.com/badlogic/pi-mono/pull/1114) by [@juanibiapina](https://github.com/juanibiapina))

Good:

Clean implementation that reuses existing methods (handleClearCommand(), showTreeSelector(), showUserMessageSelector()) instead of duplicating code

All three actions are unbound by default (empty arrays), allowing users to opt-in via keybindings.json

Follows existing patterns for keybinding registration: adds to AppAction type, DEFAULT_APP_KEYBINDINGS, and APP_ACTIONS array consistently

Documentation update in keybindings.md adds a new "Session" section with clear descriptions

PR author correctly notes the design choice that keybinding handlers don't clear the editor (unlike / commands), which lets users draft a message before navigating

Closes the linked issue #1088 which was explicitly approved by repo owner

Change summary:

packages/coding-agent/src/core/keybindings.ts: Adds newSession, tree, and fork to AppAction type union, adds them to DEFAULT_APP_KEYBINDINGS with empty arrays (unbound), and adds them to APP_ACTIONS array for type checking

packages/coding-agent/src/modes/interactive/interactive-mode.ts: Registers handlers for the three new actions via onAction() that delegate to existing methods

packages/coding-agent/docs/keybindings.md: Adds new "Session" section documenting the three keybinding actions