Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: agentclientprotocol/codex-acp
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.2.1
Choose a base ref
...
head repository: agentclientprotocol/codex-acp
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.2.2
Choose a head ref
  • 2 commits
  • 10 files changed
  • 3 contributors

Commits on Oct 10, 2026

  1. fix: title listed sessions by name, title, summary, then preview (#606) 

    ## Summary

The title of a `session/list` row and of a `_session/list/changes` row
now follows AIR's native Codex chain
(`CodexAppServerProtocol.resolveThreadTitle`): the first non-blank of
`Thread.name`, `title`, `summary` and `preview`, collapsed to one line.

- Before: `name ?? preview`, cut to 256 characters. A blank (not null)
`name` hid the preview.
- Current Codex threads carry only `name` and `preview`; `title` and
`summary` are read when Codex sends them, as AIR reads them.
- Both list paths (with and without `sessionIndex`) and subscription
rows share `listedSessionTitle`.

## Decisions

- **No cut for listed titles.** The native list does not cut, ACP
`SessionInfo.title` has no length limit, and the stdio transport has
none either. Codex already sends the whole `preview` to the adapter, so
the adapter adds no payload of its own.
- **Rename limit unchanged.** `_session/rename` still cuts to 256, so
the write queue and validation are unchanged. `session_info_update`
titles of loaded sessions keep their 256 cut as well (out of scope).
- **`null` fallback, not `Thread <id>`.** ACP `SessionInfo.title` is
optional and nullable ("Human-readable title for the session"), and
AIR's ACP list (`SessionListRow`) shows the session id for a `null`
title. A synthetic `Thread <id8>` would replace that and would count as
a real title.

## Test plan

- `SessionTitle.test.ts`: chain order, blank fields fall through, `null`
when all blank, no cut.
- `session-index.test.ts`: `session/list` rows through the chain without
a cut; a subscription change carries the uncut name.
- `npm run typecheck`, `npm test`, `npm run bundle:all`.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 5.5 <[email protected]>
    @nikita-ashihmin @claude
    nikita-ashihmin and claude authored Oct 10, 2026
    Configuration menu
    Copy the full SHA
    90fea43 View commit details
    Browse the repository at this point in the history

  2. chore(main): release 2.2.2 (#607) 

    🤖 I have created a release *beep* *boop*
---


##
[2.2.2](v2.2.1...v2.2.2)
(2026-10-10)


### Bug Fixes

* title listed sessions by name, title, summary, then preview
([#606](#606))
([90fea43](90fea43))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: acp-release-bot[bot] <246668977+acp-release-bot[bot]@users.noreply.github.com>
    @acp-release-bot
    acp-release-bot[bot] authored Oct 10, 2026
    Configuration menu
    Copy the full SHA
    202e66e View commit details
    Browse the repository at this point in the history
Loading