revert: restore route event contracts - #668
Conversation
🦋 Changeset detectedLatest commit: 6d9542a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
commit: |
|
Closing without merge. Current main now contains #667 (npm-root prepack paths), #669 (ignore package compile scratch so dev does not self-rebuild), and #670 (the #664 event input type/docs follow-up). Those changes directly address every failure from run 34006833369 while preserving #664, so reverting it would now regress the intended feature and its updated contract. |
Reverts #664 after merged-main CI failed deterministically across Node 22/24/26.
The #664 event contract change broke generated route type assertions and production operator-env expectations. This reverts that feature and stabilizes the pre-existing dev-route acceptance race by binding direct-process comparisons to the epoch actually invoked and writing operator env to each active epoch immediately before invocation.
Local gate:
pnpm build && pnpm typecheck && pnpm lint && pnpm test:unitpnpm exec rstest --config rstest.integration.config.ts packages/agent-bundle/tests/route-register-typegen.test.ts packages/agent-bundle/tests/route-invocation-dev-server.test.ts packages/agent-bundle/tests/prepack.test.ts(29 passed)Deslop: GPT-5.6 Sol, 2 edits (reverted the broken feature wholesale; replaced stale epoch assumptions without adding a helper).
Self-review: pending Claude Fable 5.1 Thinking High.