fix(eve): make extension tools durable and pass context - #61
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
4 tasks
Contributor
|
Thank you for following the naming conventions! 🙏 |
![vercel[bot]](https://cdn.statically.io/img/avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
Register eve-extension tools with authored inline execute that only closes over serializable tool names (Closes #51), forward context through the extension config, and document that connect/eve is not durable. Defer eve-managed auth until ctx.getToken('github') lands.
Export shared eve helpers on @github-tools/sdk/eve-runtime so the eve-extension does not import the deprecated consumer registration path. Keep createGithubTools and per-tool factories deprecated on /eve.
HugoRCD
force-pushed
the
feat/agent-batch-pr2-eve
branch
from
August 6, 2026 15:15
5c12d4c to
8da3854
Compare
Declare GithubExtensionConfig as an interface and cast the Zod schema so IDE hovers show property docs instead of bare Zod-inferred types.
Restore overrides[name].outputSchema when registering tools via the authored defineTool loop, and narrow the override type to Zod schemas that eve defineTool accepts.
Vercel turbo builds the example but never produced .output because the package had no build script. Run eve build, and add @vercel/connect for the connector used in the mount.
The @vercel/connect/eve barrel pulls eve/connections; NFT drops #public subpath files when Connect is externalized, crashing the deploy. Keep the official connectGitHubCredentials channel and stop externalizing Connect. Also add @nuxt/fonts to chat so workspace installs succeed for docs deploy.
eve's GitHub channel defaults omit input.requested (unlike Slack/Linear), so write-tool approval parked turns with only the eyes reaction. Also restrict @mention dispatch to HugoRCD.
Allow HugoRCD approval replies without @mention, shorten approval comments, and patch eve so github_context is turn context instead of being prepended onto the message (which broke Yes/No matching).
After HITL approval, eve was still feeding PR/github context into a follow-up turn with no user ask, so the model asked what to do next.
This was referenced Aug 6, 2026
Merged
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
1 participant
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.
Summary
@github-tools/eve-extensiontools with authored inlineexecutethat only closes over serializable tool names so multi-turn durable Workflow replay works (Closes connect/eve dynamic tools skipped on durable replay ("step function … not registered") #51)contextto the eve-extension config schema and forward it to tool execution@github-tools/sdk/eve-runtimefor shared eve primitives (descriptors, execution, approval) — the extension imports this path, not the deprecated consumer registration APIscreateGithubTools/ per-tool factories on@github-tools/sdk/eveandconnectGithubToolson/connect/evestay deprecated for directagent/tools/useTODO(eve-auth)) untilctx.getToken('github')landsStacked on #60 (hygiene).
Test plan
pnpm --filter @github-tools/eve-extension buildsucceeds@github-tools/sdk/eve-runtimeeve buildshows hoisted__closureVars: { name })contextfills default owner/repo on tool inputs