[t3dotgg]

Lint fails on main right now, and that fails the required Check job on every open PR. The four-line comment cap merged after the deduped-bodies change, which added three longer comment blocks.

This trims those three blocks to four lines without changing what they say. Comment-only, no code changes. Repo-wide vp lint passes locally.

Created with Claude Opus 5.5 in Claude Code, running in T3 Code.

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at 23363fa

Macroscope's review found this PR approvable — This PR only reformats existing comments to comply with the current four-line lint limit. No executable code, product defaults, schemas, or lint suppressions are changed, so there is no runtime impact.

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

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: fix/main-comment-cap
• File: .coderabbit.config.ts
• Commit: 23363fa
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Team
• Run ID: f07e9740-c6da-4a17-8458-ef76213b6d15

Commits

Reviewing files that changed from the base of the PR and between 8f26ed7 and 23363fa.

Files selected for processing: 3

• apps/server/src/orchestration-v2/WireProjection.ts
• packages/contracts/src/orchestrationV2.ts
• packages/shared/src/orchestrationV2DedupedBodies.ts

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

Walkthrough

The documentation clarifies when per-subscription body deduplication occurs, which received-record ordering allows omission markers, and how clients restore marked fields. The summaries report no schema behavior changes.

Changes: Body Deduplication Documentation

Layer / File(s)	Summary
Deduplication and omission-marker documentation apps/server/src/orchestration-v2/WireProjection.ts, packages/contracts/src/orchestrationV2.ts, packages/shared/src/orchestrationV2DedupedBodies.ts	The comments clarify deduplication timing and record ordering, the conditions for omission markers, and client handling of marked fields. No schema behavior changes are described.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Possibly related PRs

• pingdotgg/t3code#24647: Adds the body-deduplication implementation and wire-contract fields documented here.

Suggested reviewers: maria-rcks

Merge Risk: Minimal · up to 23363

The documentation-only changes are mergeable.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly identifies the main change: trimming three comments to satisfy the four-line lint limit.
Description check		The description explains the lint problem, the comment-only change, verification results, and the agent used. It does not include explicit section headings or scope and approval details, but it provid…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description explains the lint problem, the comment-only change, verification results, and the agent used. It does not include explicit section headings or scope and approval details, but it provides the main required information for this focused change.

Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

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

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	5.0 KiB	5.0 KiB	−7 B (−0.1%)	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	−7 B (−0.6%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.9 KiB	20.9 KiB	0 B (0.0%)	29.3 KiB	✅
Codex	Live turn messages	2	2	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	+7 B (+0.1%)	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	+7 B (+0.6%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	0 B (0.0%)	29.3 KiB	✅
Claude	Live turn messages	2	2	0 (0.0%)	8	✅

Baseline: e19fe08 · PR result: 23363fa · 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.