Skip to content

revert: restore route event contracts - #668

Closed
ScriptedAlchemy wants to merge 5 commits into
mainfrom
revert/664-red-main
Closed

revert: restore route event contracts#668
ScriptedAlchemy wants to merge 5 commits into
mainfrom
revert/664-red-main

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

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:

  • PASS — pnpm build && pnpm typecheck && pnpm lint && pnpm test:unit
  • PASS — pnpm 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.

@changeset-bot

changeset-bot Bot commented Sep 6, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6d9542a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
agent-bundle Patch

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

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@pkg-pr-new

pkg-pr-new Bot commented Sep 6, 2026

Copy link
Copy Markdown
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle@668
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/create-agent-bundle@668
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/rsc-markdown-stream@668
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/@agent-bundle/runtime@668

commit: b3e0043

@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

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.

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