Repository navigation
chore(evi): upgrade eve to 0.74.0 - #791
Open
evlogai[bot] wants to merge 1 commit into
Open
evlogai[bot] wants to merge 1 commit into
evlogai[bot] wants to merge 1 commit into
Conversation
eve 0.71.0 -> 0.74.0, with the satellite versions the new tool contract forces: @github-tools/eve-extension 0.8.0 -> 0.8.1 (built against eve 0.72.1, tool contract 76), the vendored @agent-browser/eve tarball rebuilt from the same upstream source against eve 0.74.0 (tool contract 78), and the workspace `ai` override to ^7.0.128 so `experimental_decide` resolves. The photon channel gets the explicit `input.resolved` / `approval.settled` no-ops the 0.73.0 prompt queue requires for a channel that renders prompts itself.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
Thank you for following the naming conventions! 🙏 |
@evlog/cli
evlog
@evlog/nuxthub
@evlog/signals
@evlog/telemetry
commit: |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
apps/evimovesevefrom0.71.0to0.74.0, picking up 0.72.0, 0.72.1, 0.73.0 and 0.74.0, plus the satellite versions the new tool contract forces:@github-tools/eve-extension0.8.0 → 0.8.1 (rebuilt upstream against eve 0.72.1).@agent-browser/evetarball rebuilt against eve 0.74.0: upstream still publishes only a build against eve 0.57 (tool contract 44; npm0.38.2), and eve 0.74.0 drops every contract below 76, so vendoring stays and the tarball is rebuilt from the same upstream commit with its build pinned to eve 0.74.0.pnpm-workspace.yamloverrideai: ^7.0.127→^7.0.128: eve 0.72.0 follows the AI SDK evaluation→decisions rename, and the extension'sexperimental_decideimport does not resolve against 7.0.127. The override is what decides the installed copy tree-wide, soapps/evi's own range move alone was not enough. Lockfile resolvesai@7.0.133.Nothing in the app used the other breaking surfaces: no
evaluate/eve/aiimports, nodefineInstrumentationaction.*handlers (the evlog integration andagent/instrumentation.tsusestep.started, untouched), noeve/self-modificationmount, and nothing reads Nitro's droppedassets:serverentry.Adaptation: photon prompt handlers
eve 0.73.0 (17d5b11) spans the built-in prompt queue across
input.requested,input.resolvedandapproval.settledon Chat SDK channels, and says a channel that renders prompts itself must make the other two no-ops.agent/channels/photon.tsrenders prompts itself, so it gains both:await channel.thread.post(body) }, + // eve 0.73 spans the built-in prompt queue across `input.requested`, + // `input.resolved` and `approval.settled` on Chat SDK channels. Photon + // renders prompts itself from `input.requested`, so the other two are + // explicit no-ops; otherwise the built-in handlers post prompts twice + // and stop posting later ones. + async 'input.resolved'() {}, + async 'approval.settled'() {},Behavior notes checked, no action:
mcpChannelhere uses none of the opt-in 0.73 surfaces (notools/skills/trustedForwarders), and schedules with no explicitprovidernow default to Vercel Schedules in production, which is the backend the notes already document (docs/notes.md, "Vercel evaluates schedule cron in UTC").Why the vendored extensions mount on 0.74.0
eve buildfails its template initialization on this host (no/dev/kvm, no Docker), so the compiler-time manifest check was run directly againstEXTENSION_CAPABILITY_CONTRACTSineve@0.74.0(dist/src/compiler/extension-compatibility.js), the table the build consults:agent-browser-eve-0.38.2-eve0.74.0.tgz: extension 1, tool 78, instructions 2, config 1@github-tools/eve-extension@0.8.1: extension 1, tool 76, dynamicTool 72, config 1The tarball was rebuilt in the sandbox from upstream commit
15dd3b4e6ee21c26b7b5f8048bbe73f31182803cwith the repo's patch (updated for the eve 0.74.0 pin andai@7.0.128):eve extension buildemits tool contract 78,tscpasses, and the package's own 42 tests pass. Its provenance and reproduce steps are updated inapps/evi/vendor/README.md.Checks
From
/workspace/repoon5fd4a49, with the remote cache connected:pnpm run lint: 25/25 tasks successful.pnpm run typecheck: 30/30 tasks successful.apps/evipnpm test: 43 files, 305 tests passed; the rebuilt extension's ownnode --testsuite: 42/42.apps/evipnpm build(eve build): the authored module graph evaluates and the compiler accepts both extension manifests, then stops at sandbox template initialization, exit 1: microsandbox requires/dev/kvm, unavailable in this sandbox (same limitation as the 0.71.0 upgrade, chore(evi): upgrade eve to 0.71.0 #777). The manifest table above is the substitute; the deployed build on Vercel uses the Vercel sandbox provider and runs this for real.No changeset: the change is confined to
apps/evi(plus the workspaceaioverride and lockfile it requires).ELI5
Evi runs on eve, a framework that moves independently upstream. This pull request moves it three releases forward. The framework changed the "contract" its extensions must speak, so the two extensions get new builds that speak the new contract (one comes from upstream, the other had to be rebuilt locally because upstream has not caught up yet). One small behavior fix: iMessage prompts are rendered by the app itself, and the new version needs two extra empty handlers so prompts are not posted twice. All tests pass; the one check that needs special hardware (a sandbox with virtualization) cannot run here and was verified by hand against the contract table instead.