lsm / neokai / 27893201234

82%

Build Type

push

github

Committed by web-flow

Commit Message

feat(space): event-driven gate evaluation for blocked runs (#2141)

* feat(space): event-driven gate evaluation for blocked runs

Auto-subscribe workflow runs to GitHub PR events when a gate blocks with a
resolvable PR URL, and re-evaluate gates immediately when a matching event
arrives instead of waiting for the next 5-minute poll cycle.

Pieces:
- parsePrUrl() util shared between pr-ready-validator and SpaceRuntime
- SpaceRuntime.registerPrEventSubscriptionForRun / clearPrEventSubscriptionsForRun
  wrap registerSubscription with the github/<owner>/<repo>/pull_request/<n>.*
  topic pattern; cleared automatically on run termination via clearRunInterests
- SpaceRuntimeConfig.onBlockedRunExternalEvent hook fires once per (run, event)
  when handleExternalEvent delivers to a workflow_run currently in 'blocked'
  status
- SpaceRuntimeService.notifyGateDataChanged now syncs the auto-subscription:
  subscribes when the gate stays blocked and prUrl is resolvable, clears when
  any gate for the run opens
- SpaceRuntimeService.handleBlockedRunExternalEvent re-evaluates every gate of
  the run's workflow on external-event arrival; when a gate opens it clears
  the subscription and injects a courtesy notification into the active agent
  session via taskAgentManager.injectRuntimeRecoveryMessage

Tests:
- parse-pr-url unit tests (canonical, GHE, suffix, malformed)
- runtime-level integration tests for register/clear/idempotency/blocked-hook
- service-level integration tests covering auto-subscribe, event-driven gate
  open, session notification, and non-blocked / no-prUrl no-op paths
- space-test-db helper gains the gate_open_state table (migration 130) so
  service-level tests can exercise the real ChannelRouter path

* fix(space): address #2141 review — auto-kind, write_gate sync, blocked→in_progress, 120s poll

Review feedback (4 P2/P3 comments) + 1 follow-up:

- Auto-subscriptions now use subscriptionKind='auto' instead of 'dynamic' so
  clearPrEventSubscripti... (continued)

Coverage Stats

10913 of 15827 branches covered (68.95%)

Branch coverage included in aggregate %.

534 of 616 new or added lines in 12 files covered. (86.69%)

4 existing lines in 4 files now uncovered.

93943 of 112140 relevant lines covered (83.77%)

296.89 hits per line