Skip to content

feat(composer): sync drafts across web and desktop - #39

Merged
incognitojam merged 3 commits into
mainfrom
feat/composer-draft-sync
Aug 8, 2026
Merged

feat(composer): sync drafts across web and desktop#39
incognitojam merged 3 commits into
mainfrom
feat/composer-draft-sync

Conversation

@incognitojam

@incognitojam incognitojam commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Problem

Composer drafts were browser-local, so moving between web and desktop clients connected to the same environment could leave unfinished work behind or expose stale text after sending elsewhere.

Solution

  • Persist revisioned, existing-thread composer drafts on the environment server.
  • Stream transferable text and model/mode selections between web and desktop clients.
  • Keep images and richer context client-local, tombstoning the transferable copy while they are attached.
  • Use compare-and-swap revisions so delayed sends and concurrent editors cannot delete newer work.
  • Advertise the feature through an optional capability so older clients and servers continue using local drafts.
  • Share the draft broadcaster at server lifetime so separate WebSocket sessions receive live changes.
  • Keep the wire API subscription-only and remove drafts as part of thread deletion projection.
  • Leave the mobile client and its existing device-local draft/outbox behavior unchanged.

Verification

  • Preview: fresh-tab hydration, bidirectional live updates, and cross-tab clearing across three independent WebSocket sessions.
  • Server/shared runtime: 10 focused tests passed, including cross-WebSocket propagation, revision-aware turn-start clearing, and thread-deletion cleanup.
  • Web: 81 composer-store tests passed.
  • Mobile regression: 28 pre-existing composer/outbox tests passed with zero mobile diff against main.
  • Typechecks passed for contracts, client runtime, web, and server.
  • Targeted lint and formatting checks passed.

Model: GPT-5.6 Codex; harness: T3 Code.

@github-actions github-actions Bot added size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Aug 8, 2026
@incognitojam
incognitojam marked this pull request as draft August 8, 2026 11:50
@incognitojam incognitojam changed the title feat(composer): sync drafts across devices feat(composer): sync drafts across web and desktop Aug 8, 2026
@github-actions github-actions Bot added size:XL and removed size:XXL labels Aug 8, 2026
@incognitojam
incognitojam marked this pull request as ready for review August 8, 2026 12:20
@github-actions github-actions Bot added size:XXL and removed size:XL labels Aug 8, 2026
@incognitojam
incognitojam force-pushed the feat/composer-draft-sync branch from 4ec175c to 9580bcf Compare August 8, 2026 12:22
@github-actions github-actions Bot added size:XL and removed size:XXL labels Aug 8, 2026
@incognitojam
incognitojam force-pushed the feat/composer-draft-sync branch from 9580bcf to 7baf7b6 Compare August 8, 2026 14:56
@incognitojam
incognitojam merged commit 977e42e into main Aug 8, 2026
16 checks passed
@incognitojam
incognitojam deleted the feat/composer-draft-sync branch August 8, 2026 15:16
incognitojam added a commit that referenced this pull request Aug 9, 2026
* feat(composer): sync drafts across devices

* fix(composer): address draft sync review findings

* refactor(composer): scope draft sync to web and desktop

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
incognitojam added a commit that referenced this pull request Aug 9, 2026
The desktop update tooltip flattens the release body into plain bullet
lines, so the structured nightly highlights (`## Nightly highlights`,
`### Added`, …) rendered their headings as list items and wasted 3 of
the tooltip's 8 item slots. This PR makes the generated changelog and
the tooltip work together, and leans harder into the LLM pipeline while
keeping attribution.

## Changes

- **Nightly highlights are a flat list** of plain items with compact PR
references that survive the tooltip's markup stripping (`#61` for fork
PRs, `t3code#123` for upstream, short SHAs for direct commits). Removals
are phrased as "Remove …" via prompt guidance instead of a section
label.
- **Attribution is structural:** summary items are now `{ text,
evidenceIds }` objects. The model only returns evidence IDs; URLs are
rendered deterministically and IDs are validated against the collected
evidence, so references can't be fabricated. The rolling fork-features
issue gets the same per-item references.
- **Extraction sees diffs:** each commit contributes a truncated patch
(4 KB cap, lockfiles excluded) so descriptions are grounded in what
actually changed rather than paraphrasing PR titles. Over the 160k-char
prompt budget, the largest diffs are dropped first — titles and PR
bodies never are.
- **`## What's Changed` stays** as the exhaustive record and now credits
contributors once per PR (`by @username`, fetched via `gh api` with
graceful fallback).
- **The tooltip parser stops at `## What's Changed`** when curated
highlights precede it, so raw commit lines no longer mix with highlight
lines. Bodies that open with the commit list (upstream-style, or a
nightly with no highlights) still show it.

## Pipeline example

Evidence in (abridged; one upstream commit from the delta below):

```jsonc
{
  "id": "pingdotgg#5756",
  "title": "fix(web): usage page loses the cost quality panel, gains a back button",
  "description": "<sanitized PR body>",
  "files": ["apps/web/src/components/usage/UsagePage.tsx", "…"],
  "diff": "--- a/apps/web/src/components/usage/UsagePage.tsx\n+import { useCanGoBack, useNavigate, useRouter } from \"@tanstack/react-router\";\n…"
}
```

Summary items out (actual output — note one PR correctly yielding both
an improvement and a removal):

```json
{ "text": "Return from Usage through history or to home", "evidenceIds": ["pingdotgg#5756"] }
{ "text": "Remove Cost quality panel from the Usage page", "evidenceIds": ["pingdotgg#5756"] }
```

Rendered in the release body:

```markdown
- Return from Usage through history or to home ([t3code#5756](pingdotgg#5756))
- Remove Cost quality panel from the Usage page ([t3code#5756](pingdotgg#5756))
```

## Regenerated release notes for `v0.0.33-nightly.20260809.121`

Rerunning the new generator against the existing release's refs produced
13 items (the published release had 5, plus 3 heading lines). What the
tooltip shows, via the actual parser:

```
• Identify sidebar threads with unsent drafts (#23)
• Show subagent activity counts in right-panel controls (t3code#5745)
• Return from Usage through history or to home (t3code#5756)
• Give agents access to pasted image files (t3code#5757)
• Manage projects from the dedicated Settings page (t3code#5768)
• Preserve selected diff view across panel closures and reloads (t3code#5731)
• Show pointer cursors for clickable controls and actions (t3code#5716)
• Resume Claude sessions without falsely completing skipped turns (t3code#5710)
```

<details>
<summary>Full regenerated nightly highlights</summary>

- Identify sidebar threads with unsent drafts
([#23](#23))
- Show subagent activity counts in right-panel controls
([t3code#5745](pingdotgg#5745))
- Return from Usage through history or to home
([t3code#5756](pingdotgg#5756))
- Give agents access to pasted image files
([t3code#5757](pingdotgg#5757))
- Manage projects from the dedicated Settings page
([t3code#5768](pingdotgg#5768))
- Preserve selected diff view across panel closures and reloads
([t3code#5731](pingdotgg#5731))
- Show pointer cursors for clickable controls and actions
([t3code#5716](pingdotgg#5716))
- Resume Claude sessions without falsely completing skipped turns
([t3code#5710](pingdotgg#5710))
- Stop the active Codex turn before queued follow-ups
([t3code#5762](pingdotgg#5762))
- Keep pinned thread ordering stable while saving changes
([t3code#5767](pingdotgg#5767))
- Show multi-device usage progress while totals load
([t3code#5772](pingdotgg#5772))
- Remove Cost quality panel from the Usage page
([t3code#5756](pingdotgg#5756))
- Remove the limited project settings modal
([t3code#5768](pingdotgg#5768))

</details>

<details>
<summary>Regenerated rolling fork-features summary (excerpt, for issue
#43)</summary>

## Added

- Identify yngatech builds through names, icons, palettes, and About
details ([#3](#3),
[#27](#27),
[#29](#29),
[#30](#30),
[#56](#56))
- Choose and preview sounds for completed agent turns
([#5](#5),
[#7](#7),
[#19](#19))
- Detect GitHub outages and show status in the sidebar
([#14](#14))
- Show unsent thread drafts with sidebar pencil indicators
([#23](#23))
- Start new threads with GitHub issues as context
([#31](#31))
- Open thread pull or merge requests from command palette
([#46](#46))
- Synchronize existing-thread text and composer settings across
connected devices ([#39](#39),
[#44](#44))

## Improved

- Inspect full commands, labeled streams, numeric exit codes, and
failures ([#11](#11),
[#4](#4),
[#36](#36))
- Close terminals directly from the grouped terminal list
([#22](#22))
- Show clearer snooze countdowns near hour and day boundaries
([#45](#45))
- Show setup script outcomes in the thread timeline
([#40](#40))
- Install yngatech alongside upstream T3 Code on Windows
([#49](#49))

</details>

## Rollout

The generator and workflow changes take effect on the next nightly. The
tooltip parser change ships inside the app, so currently installed
builds still show the mixed list until users update onto a build
containing it.

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant