fix(server): Pi thread titles use linked PR context

[juliusmarminge]

Problem

Each provider in apps/server/src/textGeneration/ had its own copy of the four operations (commit message, PR content, branch name, thread title): it built the prompt, called the provider, and sanitized the reply. The copies drifted. Pi never passed linkedContext into the thread title prompt, so Pi titles ignored a linked PR or issue that the other providers used. Pi also dropped the needsRefinement signal for an unresolved subject.

OpenCode 2 was already correct here. It and OpenCode 1.x share makeOpenCodeOperations, which passes linkedContext.

Fix

TextGenerationOperations.fromRunner(name, run) now owns the four operations in one place: prompt building, linkedContext and attachments in the thread title prompt, reply sanitizing, and the needsRefinement passthrough. Each provider supplies only a runner, which sends one prompt and decodes the reply as the request's schema. decodeJsonReply covers the six providers whose reply is text (Pi, Cursor, Grok, Antigravity, OpenCode 1.x and 2.x). makeOpenCodeOperations / OpenCodeJsonRunner are replaced by the shared module.

What each runner still owns, all unchanged:

• Codex passes the JSON schema through --output-schema and reads the output file. Image attachments are now resolved inside the runner from the request, rather than in the branch-name and title operations.
• Claude decodes structured_output from the CLI's JSON envelope. Titles still run in a temp directory instead of the checkout.
• Cursor and Antigravity ignore cwd and run in an empty temp directory. Antigravity still adds its "use only the input below" preamble to the prompt.
• OpenCode 1.x keeps its typed session and prompt errors. OpenCode 2 keeps its temporary-session flow.
• Each provider keeps its own timeouts, error details ("Pi returned empty output.", "Codex CLI command failed: ..."), and "invalid structured output" wording.

Two side effects, both kept on purpose:

• Image attachments now reach the Pi, Grok, Cursor, Claude, and Antigravity runners on the request too. Those runners ignore them, as before. Only Codex and OpenCode send images, as before.
• Span names keep the <Provider>TextGeneration.generate* shape. OpenCode 2 spans are now OpenCode2TextGeneration.* instead of reusing OpenCodeTextGeneration.*.

No contract or TextGeneration service interface changes.

Verification

From apps/server:

• vp test run src/textGeneration: 12 files, 151 tests passed (was 11 / 150 on main; the new file is PiTextGeneration.test.ts).
• The new Pi test drives makePiTextGeneration over an in-process fake pi --mode rpc and asserts that the prompt Pi receives carries the linked context. Against main's PiTextGeneration.ts it fails with expected 'Generate a title that will help the u…' to include 'Linked source control context (refere…'.
• vp test run src/provider/Drivers src/orchestration-v2/ThreadTitleRegenerationService.test.ts src/orchestration-v2/ThreadLaunchService.test.ts src/git/GitManager.test.ts: 17 files, 295 tests passed.
• vp exec tsc --noEmit -p .: exit 0, no errors.

From the repo root:

• vp lint --report-unused-disable-directives <changed files>: clean.
• vp exec knip --workspace apps/server --exports --preprocessor ./scripts/knip-schemas.ts --no-config-hints: exit 0.

Net change: git diff --stat origin/main reports 10 files changed, 317 insertions(+), 1049 deletions(-).

Model/harness: Claude Opus 5.5 (1M context) via Claude Code in T3 Code.

🤖 Generated with Claude Code

---

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Team
• Run ID: 17dc0ca5-6267-4ddd-815f-1f907ac6712d

📥 Commits

Reviewing files that changed from the base of the PR and between a216dc7 and ec87b8b.

📒 Files selected for processing (1)

• apps/server/src/textGeneration/CodexTextGeneration.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 1 remain after this review.

---

📝 Walkthrough

Walkthrough

The PR adds shared text-generation operation contracts, prompt construction, result processing, and JSON reply decoding. Provider runners adopt the shared interfaces. Codex updates image attachment handling, and a Pi test checks thread-title generation with linked source-control context.

Changes

Shared text-generation operations

Layer / File(s)	Summary
Shared operation contracts and service methods apps/server/src/textGeneration/TextGenerationOperations.ts	Adds shared request and runner contracts, JSON reply decoding, and service methods for commit messages, PR content, branch names, and thread titles.
Provider runner integration apps/server/src/textGeneration/*TextGeneration.ts, apps/server/src/textGeneration/PiTextGeneration.test.ts	Provider runners adopt the shared contracts and decoder. Their factories use the shared service adapter. The Pi test checks generated thread-title output and prompt context.
Codex image attachment handling apps/server/src/textGeneration/CodexTextGeneration.ts	Codex derives image paths from shared request attachments and limits cleanup to schema and output temporary-file directories.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to ec87b

No actionable defect is supported by the supplied review context; the suspected attachment-file leak is not introduced by this change. The PR appears ready for normal checks.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the problem, implementation, side effects, and verification. It does not include the required scope and approval information: a triaged issue or discussion with explicit maint…	Add a link to the triaged issue or discussion and the explicit maintainer approval comment for the scope and direction. If no prior approval exists, obtain it and include the link. Do not use the small-fix exemption unless maintainers agree…

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the Pi thread-title linked-context fix, which is the primary reported behavior change.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description explains the problem, implementation, side effects, and verification. It does not include the required scope and approval information: a triaged issue or discussion with explicit maintainer approval. The reported cross-provider refactor is not described as a very small, focused fix that qualifies for the exemption.

Resolution

Add a link to the triaged issue or discussion and the explicit maintainer approval comment for the scope and direction. If no prior approval exists, obtain it and include the link. Do not use the small-fix exemption unless maintainers agree this broad cross-provider refactor qualifies as a very small, focused fix of an obvious bug.

• Fix all pre-merge checks with AI

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR fixes Pi thread-title context handling but also replaces duplicated operation implementations across all text-generation providers with a new shared runtime abstraction. Because the cross-provider refactor can affect multiple production integrations, it merits human review.

No code changes detected at ec87b8b. Prior analysis still applies.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	4.9 KiB	5.0 KiB	+23 B (+0.5%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	+23 B (+1.9%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.8 KiB	20.9 KiB	+41 B (+0.2%)	29.3 KiB	✅
Codex	Live turn messages	1	2	+1 (+100.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	−24 B (−0.5%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	−24 B (−2.0%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	−41 B (−0.2%)	29.3 KiB	✅
Claude	Live turn messages	2	1	−1 (−50.0%)	8	✅

Baseline: eef36a6 · PR result: ec87b8b · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.