Skip to content

docs(devlog): doc 110 new-wave triage rows (#2248-#2254) #2256

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lidge-jun merged 1 commit into from
Aug 21, 2026
Merged

docs(devlog): doc 110 new-wave triage rows (#2248-#2254) #2256

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions devlog/_plan/260820_sidecar_selection_unification/110_global_merge_order.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,5 +46,17 @@ itself (audit R1-B4).
| #2213 | louis-tepe | OUT (draft, overlaps doc-130 design) | revisit post-switch |
| #2072 | olddonkey | DEFERRED (audit R2-B2) | Fast policy composes later; must re-verify against both wires |

## New wave (user expansion 260821, lands in step 5-6 order after the chat-default unit)

| PR | author | disposition | rationale |
|---|---|---|---|
| #2248 | olddonkey | MERGE after #2227 unit | cross-backend reasoning strip on model switch; wire-agnostic hygiene |

@chatgpt-codex-connector chatgpt-codex-connector Bot Aug 21, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Anchor these dependencies to the landed #2255 unit

At this commit's parent, the chat-default integration has already landed as #2255 (63d387ca), and the commit summary itself says this wave follows #2255. Naming the still-separate #2227 in the #2248 and #2251 dispositions leaves the execution matrix pointing at the wrong PR and can make the merge operator wait for or attempt to integrate #2227 again; update both rows to reference #2255 (or the post-chat-default dev anchor).

Useful? React with 👍 / 👎.

| #2249 | olddonkey | MERGE after #2228 line | compaction blob serving-identity comparison; composes with provenance fix |
| #2251 | olddonkey | MERGE after #2227 unit | recovery when upstream rejects foreign opaque state |
| #2252 | olddonkey | MERGE | strip output-only reasoning status unconditionally; small + wire-agnostic |
| #2254 | olddonkey | RECONCILE with doc 130 | restores Grok Responses on the native passthrough route — must compose with the chat default + opt-in switch, not fight them; adjudicate against the doc-100 matrix |
| #2250 | lilinxiong | MERGE (bug) | integrations honor OFF for Claude Desktop drift + Grok ensure |
| #2253 | lilinxiong | MERGE (bug, gui) | dropdown opacity; needs screenshot per PR gate |

Rebase anchors are named at execution time in each phase's B (exact dev head SHA),
with cascade verification (typecheck + focused suites) after every land.
Loading