fix(mcp): preserve nullable checkout responses in event logging

Co-Authored-By: Paperclip <noreply@paperclip.ing>
This commit is contained in:
DottaandPaperclip committed 2026-10-01 12:41:54 -05:00
1 parent 4802426f63
commit a140d533a6
2 files changed
+6 -1

No files matched your search

@@ -184,6 +184,11 @@ Updated supporting verification:
regressions cover quota reuse and unchanged-status suppression.
- All 28 adjacent issue-tree/stale-lock route tests pass after checkout/release
activity gained previous/current status for event delivery.
- CI subsequently found that checkout logging dereferenced a null result in an
existing concurrent workspace-reopen path. The log now omits status when no
task is returned, preserving that path's response and avoiding a false event.
The existing 12-test closed-workspace suite reproduces and verifies the fix.
This narrow null-result correction follows the paid regression above.
- Repository-wide typecheck and production build pass after merging master.
- The Core → Cloud → Core authority round trip passes real OAuth, MCP 2.0
subscription, delivery, current Cloud membership loss, unsubscribe, legacy MCP
+1 -1
View File
@@ -15083,7 +15083,7 @@ export function issueRoutes(
action: "issue.checked_out",
entityType: "issue",
entityId: issue.id,
details: { agentId: req.body.agentId, status: updated.status, _previous: { status: issue.status } },
details: { agentId: req.body.agentId, status: updated?.status, _previous: { status: issue.status } },
});
if (