Skip to content

fix(gateway): restore cooperative suspension validation#103925

Merged
vincentkoc merged 1 commit into
mainfrom
fix/main-ci-103618
Jul 10, 2026
Merged

fix(gateway): restore cooperative suspension validation#103925
vincentkoc merged 1 commit into
mainfrom
fix/main-ci-103618

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

Related: #103618

What Problem This Solves

Resolves a problem where the cooperative Gateway suspension change left main
with a 557-file Madge import cycle and five core lint failures, blocking hosted
CI and downstream PR preparation.

Why This Change Was Made

The Gateway-only cron suspension contract now lives in a leaf type module, so
shared request context types no longer depend on the full cron runtime graph.
The accompanying test cleanup removes the exact lint violations without
changing the covered behavior.

User Impact

No runtime behavior changes. Maintainers regain clean architecture and lint
validation for the cooperative suspension feature.

Evidence

  • Failing main CI: run 29117924498, architecture job 86446192773, lint job
    86446192688.
  • Blacksmith Testbox tbx_01kx6srz7emmr8nhqenmfcjser:
    • pnpm check:madge-import-cycles: 0 cycles.
    • pnpm check:import-cycles: 0 runtime value cycles.
    • Focused core lint: 0 warnings, 0 errors.
    • Focused tests: 149/149 passed across cron lazy loading, chat/queue retained
      work, suspension admission, and active-work accounting.
    • pnpm check:changed: passed core, coreTests, including both tsgo lanes,
      changed-file lint, and guards.
  • Structured autoreview: no findings; patch correct, confidence 0.86.

@vincentkoc vincentkoc self-assigned this Jul 10, 2026
@vincentkoc
vincentkoc marked this pull request as ready for review July 10, 2026 20:12
@openclaw-barnacle openclaw-barnacle Bot added gateway Gateway runtime size: XS maintainer Maintainer-authored PR labels Jul 10, 2026
@vincentkoc

Copy link
Copy Markdown
Member Author

Land-ready proof for exact head 9a7f055f3250015763015e8d035f3815dbaac83c:

  • Hosted CI run 29120615634: zero pending or failing checks. The previously
    failing check-lint and check-additional-runtime-topology-architecture
    jobs both succeeded.
  • Blacksmith Testbox tbx_01kx6srz7emmr8nhqenmfcjser: Madge and runtime
    import-cycle checks passed, focused lint passed, 149 focused tests passed,
    and pnpm check:changed passed both core and coreTests lanes.
  • Structured autoreview returned no findings and judged the patch correct
    (confidence 0.86).
  • scripts/pr review-validate-artifacts 103925 and
    OPENCLAW_TESTBOX=1 scripts/pr prepare-run 103925 completed successfully.

Known proof gaps: none for this type-ownership and lint-only repair.

@vincentkoc
vincentkoc merged commit 99e1a60 into main Jul 10, 2026
164 of 167 checks passed
@vincentkoc
vincentkoc deleted the fix/main-ci-103618 branch July 10, 2026 20:20
@vincentkoc

Copy link
Copy Markdown
Member Author

Merged via squash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gateway Gateway runtime maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant