Skip to content

fix(web): deny open coordinator approval when sending a chat message - #77

Closed
TheGreatAxios wants to merge 1 commit into
mainfrom
cl-11954-deny-pending-coordinator-approval-when-the-person-sends-a
Closed

TheGreatAxios wants to merge 1 commit into
mainfrom
cl-11954-deny-pending-coordinator-approval-when-the-person-sends-a

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11954

Sending a chat message while a coordinator approval card (openApproval, such as design_app, approve_document, or request_approval) is open now denies that card (choice: "no") before delivering the message, so the parked coordinator wakes to read the person's message instead of remaining held on SignalAwaited. typedYes continues to hold plain affirmations for approval after the countdown, undoing the countdown denies the open approval before sending the words, and chooseApproach approves a matching approach card without double-deciding it.

Verified:

  • bun --conditions=intx-src test apps/web/src/pages/workspace/typed-yes.test.ts (5 pass, 0 fail)
  • bun --conditions=intx-src test apps/web/src (24 pass, 0 fail)
  • bun run lint (exit 0)
  • bun run format:check (exit 0)
  • bun run typecheck (exit 0)
  • bun run vendor:verify (exit 0)
  • bun run pins:verify (exit 0)

@linear-code

linear-code Bot commented Nov 2, 2026

Copy link
Copy Markdown

CL-11954

@TheGreatAxios
TheGreatAxios force-pushed the cl-11954-deny-pending-coordinator-approval-when-the-person-sends-a branch from 24f29b8 to 0f9c114 Compare November 2, 2026 03:54
@TheGreatAxios TheGreatAxios reopened this Nov 2, 2026
@TheGreatAxios
TheGreatAxios deleted the cl-11954-deny-pending-coordinator-approval-when-the-person-sends-a branch November 2, 2026 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant