You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Non-blocking follow-ups from the final readiness review of #4440 (default unrelated-messaging consent for user-created workspaces):
Pending mark leak. If create() takes its removing/aborted branch, or throws between marking a deferred checkout pending and retaining the settlement, the pendingDefaultUnrelatedConsent entry is never cleared. This is harmless (only materializeDeferredCheckout reads it, and IDs are random), but it should be cleared there for tidiness.
Failed deferred checkout stays off. A deferred checkout that fails without a sanitization error leaves a usable workspace with consent off, while an immediate checkout is granted. This fails closed. Either grant on that path once sanitization has succeeded, or mention the caveat in the tool and docs text.
Multi-project has no plugin-override sanitization at all. This predates 🤖 feat: opt new user-created workspaces in to unrelated messaging by default #4440, but default consent now makes multi-project workspaces reachable by other task trees as soon as they are created. Decide whether multi-project needs registration-time sanitization.
Test gap. Add a create()-level deferred-checkout test that asserts the mark is set before the announcement and cleared after settlement. Existing tests call materializeDeferredCheckout directly.
Non-blocking follow-ups from the final readiness review of #4440 (default unrelated-messaging consent for user-created workspaces):
create()takes its removing/aborted branch, or throws between marking a deferred checkout pending and retaining the settlement, thependingDefaultUnrelatedConsententry is never cleared. This is harmless (onlymaterializeDeferredCheckoutreads it, and IDs are random), but it should be cleared there for tidiness.create()-level deferred-checkout test that asserts the mark is set before the announcement and cleared after settlement. Existing tests callmaterializeDeferredCheckoutdirectly.Related: #4444, #4446, #4453.
Generated with
xum• Model:anthropic:claude-opus-5-5• Thinking:high