Skip to content

fix(eve): hook ownership - skip metadata hydration - #3043

Merged
AndrewBarba merged 1 commit into
mainfrom
barba/hook-owner-reads
Sep 4, 2026
Merged

fix(eve): hook ownership - skip metadata hydration#3043
AndrewBarba merged 1 commit into
mainfrom
barba/hook-owner-reads

Conversation

@AndrewBarba

@AndrewBarba AndrewBarba commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Channel ownership checks and inbox polling only need a hook's run ID, but getHookByToken() also decrypts metadata and can fetch the target deployment's encryption key. Add getRawHookByToken() to the runtime wrapper and use it at those five sites, preserving ownership checks, retry behavior, and legacy inbox classification. The helper documents that it skips metadata hydration and excludes metadata from its return type. This is the first of three stacked PRs reducing hook-read latency; consumer wire negotiation remains unchanged here. Related to #1765.

Validation

  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/execution/workflow-runtime.test.ts src/execution/wire/session-inbox-resume.test.ts src/execution/continuation-conflict-step.test.ts — 66 tests passed.

  • pnpm fmt, pnpm lint, pnpm guard:invariants, and pnpm typecheck — passed. Unrelated formatting output was excluded.

  • pnpm test:unit — 8,025 eve tests passed, one skipped, and two untouched telemetry-preference tests failed on macOS: they expect Linux XDG paths but receive Library/Preferences/eve. Other workspace unit suites passed.

  • In-memory probes against the installed Workflow SDK confirmed that raw ownership reads do not resolve encryption keys, while hydrated hook reads do.

  • After extracting the helper, the 66 focused tests and pnpm --filter eve exec tsc --noEmit passed again. The restacked tip passed all 235 focused tests across 10 relevant files.

  • Prior revision's CI: the hosted task fixture passed 18 of 19 evals; task.parent.wake.emitted-ready.conditional-delivery failed because its session stream closed before a turn boundary. The cause is not established. See the failed job.

Stack: #3043#3044#3045 (merge in that order).

Checklist

  • This change was requested or approved by a maintainer
  • I ran the relevant checks from CONTRIBUTING.md
  • I added tests and documentation where relevant
  • I added a changeset if this touches the published eve package
  • DCO sign-off passes for every commit (git commit --signoff)

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
eve-docs Ready Ready Preview, v0 Sep 4, 2026 5:13pm UTC
eve-docs-4759 Ready Ready Preview, v0 Sep 4, 2026 5:13pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
eve-pkg Skipped Skipped v0 Sep 4, 2026 5:13pm UTC

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Bundle + Package Summary: apps/fixtures/weather-agent

Key takeaways

  • No notable deltas vs main (16c7f24).

Delta vs main (16c7f24)

Area Metric Baseline Current Delta
Package Packed tarball 8.65 MB 8.65 MB -434 B ✅
Package Unpacked publish size 32.56 MB 32.56 MB +478 B ⚠️
Package Installed footprint 77.73 MB 77.74 MB +478 B ⚠️
Package Published files 3836 3836 0
Package Installed files 7682 7682 0
Package Installed package instances 35 35 0
Package Distinct installed package names 34 34 0
Package Installed dependency edges 53 53 0
Package Installed optional peer edges 47 47 0
Runtime Unique function payloads 2 2 0
Runtime Total function bytes 20.56 MB 20.56 MB +546 B ⚠️
Runtime Public routes 18 18 0
Changed function payloads vs main (16c7f24) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 10.28 MB 10.28 MB +273 B ⚠️ none
functions/.well-known/workflow/v1/flow.func changed 10.28 MB 10.28 MB +273 B ⚠️ none

eve init install

Metric Baseline Current Delta
Installed footprint 116.17 MB 116.17 MB +478 B ⚠️
Installed packages 115 115 0
dependencies 4 4 0
devDependencies 2 2 0
Dependency package bytes 48.65 MB 48.65 MB +478 B ⚠️
devDependency package bytes 5.04 MB 5.04 MB 0 B ➖
Build Metadata
  • Preset: vercel
  • Nitro: nitro@3.0.260610-beta
  • Output directory: apps/fixtures/weather-agent/.vercel/output
  • Build metadata timestamp: 2026-09-04T17:14:39.438Z
  • Route aliases: 18 public, 1 internal (19 total aliases)
  • Vercel routes in config: 21
  • Severity legend: 🔴 dominant/large, 🟠 notable, 🟡 watch, ⚪ small
Package Drill-Down

Package Details

  • Package: eve@0.51.1
  • Package directory: packages/eve
  • Tarball: 8.65 MB (eve-0.51.1.tgz)
  • Unpacked payload: 32.56 MB across 3836 published files
  • Installed footprint: 77.74 MB across 7682 installed files
  • Installed root package: 31.13 MB
  • Installed dependencies: 46.61 MB
  • Installed package instances: 35
  • Distinct installed package names: 34
  • Installed dependency edges: 53
  • Installed optional peer edges: 47
  • Runtime dependencies: 2
  • Peer dependencies: 5 (4 optional)

Installed footprint is measured from an isolated temporary npm install of the packed tarball.
Graph metrics read only package.json files in package directories directly beneath a node_modules boundary, including nested boundaries. Each directory is one package instance; distinct names come from those manifests. Dependency edges count each unique name in dependencies or optionalDependencies per instance; optional peer edges count peerDependencies marked optional.

Heavy installed dependencies

  • eve: 31.13 MB (40.0%)
  • @rolldown/binding-linux-x64-gnu: 19.31 MB (24.8%)
  • ai: 6.99 MB (9.0%)
  • zod: 6.41 MB (8.3%)
  • undici: 3.51 MB (4.5%)
Publish payload breakdown
Published file size
🔴 dist/src/compiled/shadcn-registry/index.js       [#############...........] 9.76 MB 30.0%
🟠 dist/src/compiled/@photon-ai/chat-adapter-ime... [###.....................] 2.27 MB 7.0%
🟠 dist/src/compiled/@ai-sdk/code-mode/index.js     [#.......................] 1.03 MB 3.2%
🟡 dist/src/compiled/@vercel/blob/index.js          [#.......................] 901.8 kB 2.8%
🟡 dist/src/compiled/_chunks/workflow/undici-Dfx... [#.......................] 512.5 kB 1.6%
🔴 Other published files                            [########################] 18.09 MB 55.6%
Installed footprint breakdown
Installed package size
🔴 eve                             [########################] 31.13 MB 40.0%
🔴 @rolldown/binding-linux-x64-gnu [###############.........] 19.31 MB 24.8%
🔴 ai                              [#####...................] 6.99 MB 9.0%
🔴 zod                             [#####...................] 6.41 MB 8.3%
🟠 undici                          [###.....................] 3.51 MB 4.5%
🟠 nitro                           [##......................] 2.41 MB 3.1%
🔴 Other installed packages        [######..................] 7.97 MB 10.3%
Runtime dependencies (2)
Package Range Notes
nitro 3.0.260610-beta
undici 8.9.0
Peer dependencies (5)
Package Range Notes
@opentelemetry/api ^1.0.0 optional peer
ai catalog:
braintrust ^3.0.0 optional peer
just-bash ^3.1.0 optional peer
microsandbox ^0.5.0 optional peer
eve init install drill-down

eve init install details

  • Command: eve init my-agent
  • Package manager: npm
  • Installed footprint: 116.17 MB across 9578 installed files
  • Installed packages: 115 total (109 transitive-only)
  • dependencies: 4 direct packages totaling 48.65 MB
  • devDependencies: 2 direct packages totaling 5.04 MB
  • Other transitive package files: 62.48 MB

Installed footprint is measured from an isolated temporary eve init my-agent using the current packed eve tarball.

Heavy installed dependencies

  • eve: 31.13 MB (26.8%)
  • @typescript/typescript-linux-x64: 27.95 MB (24.1%)
  • @rolldown/binding-linux-x64-gnu: 19.31 MB (16.6%)
  • zod: 10.37 MB (8.9%)
  • ai: 6.99 MB (6.0%)
Installed footprint breakdown
Installed package size
🔴 eve                              [########################] 31.13 MB 26.8%
🔴 @typescript/typescript-linux-x64 [######################..] 27.95 MB 24.1%
🔴 @rolldown/binding-linux-x64-gnu  [###############.........] 19.31 MB 16.6%
🔴 zod                              [########................] 10.37 MB 8.9%
🔴 ai                               [#####...................] 6.99 MB 6.0%
🟠 undici                           [###.....................] 3.51 MB 3.0%
🔴 Other installed packages         [#############...........] 16.91 MB 14.6%
dependencies (4)
Package Range Installed size Share
@vercel/connect 1.0.0 167.9 kB 0.1%
ai ^7.0.82 6.99 MB 6.0%
eve file:eve-0.51.1.tgz 31.13 MB 26.8%
zod 4.5.4 10.37 MB 8.9%
devDependencies (2)
Package Range Installed size Share
@types/node 24.x 2.54 MB 2.2%
typescript 7.0.2 2.50 MB 2.1%
Function Drill-Down

Payload Size Graph

Unique function payload size and share of total
🔴 functions/.well-known/workflow/v1/flow.func     [########################] 10.28 MB 50.0%
🔴 functions/__server.func                         [########################] 10.28 MB 50.0%

Top Function Payloads

🟠 functions/.well-known/workflow/v1/flow.func • 1 public route • 10.28 MB
Metric Value
Public routes /.well-known/workflow/v1/flow
Runtime nodejs24.x
Handler index.mjs
Payload 10.28 MB
Function files 10.28 MB across 112 files
Traced dependencies 0 B
Signal 🟠 Bundled file index.mjs is 2.24 MB (21.8%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                   [###########.............] 2.24 MB 21.8%
🟡 _libs/undici.mjs            [#####...................] 980.5 kB 9.5%
🟡 _chunks/esm-CE0tAvRh.mjs    [####....................] 723.5 kB 7.0%
🟡 _chunks/chatgpt-model.mjs   [###.....................] 698.0 kB 6.8%
🟡 _chunks/frames-Boesh7kU.mjs [###.....................] 680.2 kB 6.6%
🔴 Other bundled files         [########################] 4.96 MB 48.2%

🧾 Vercel Config

{
  "handler": "index.mjs",
  "launcherType": "Nodejs",
  "shouldAddHelpers": false,
  "supportsResponseStreaming": true,
  "runtime": "nodejs24.x",
  "maxDuration": "max",
  "experimentalTriggers": [
    {
      "type": "queue/v2beta",
      "topic": "__eve776561746865722d6167656e74_wkf_workflow_*",
      "consumer": "default",
      "retryAfterSeconds": 5,
      "initialDelaySeconds": 0
    }
  ],
  "environment": {
    "WORKFLOW_PRECONDITION_GUARD": "1"
  }
}

🟠 functions/__server.func • 17 public routes, 1 internal alias • 10.28 MB
Metric Value
Public routes /
/.well-known/workflow/v1/webhook/[token]
/eve/v1/activity/[token]
/eve/v1/callback/[token]
/eve/v1/connections/[name]/callback/[attemptId]/[token]
/eve/v1/connections/[name]/callback/[token]
/eve/v1/health
/eve/v1/info
/eve/v1/session
/eve/v1/session/[parentSessionId]/subagents/[callId]/[childSessionId]/stream
/eve/v1/session/[sessionId]
/eve/v1/session/[sessionId]/cancel
/eve/v1/session/[sessionId]/clear
/eve/v1/session/[sessionId]/compact
/eve/v1/session/[sessionId]/reset
/eve/v1/session/[sessionId]/stream
/eve/v1/task-input/[token]
Internal aliases /__server
Runtime nodejs24.x
Handler index.mjs
Payload 10.28 MB
Function files 10.28 MB across 112 files
Traced dependencies 0 B
Signal 🟠 Bundled file index.mjs is 2.24 MB (21.8%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                   [###########.............] 2.24 MB 21.8%
🟡 _libs/undici.mjs            [#####...................] 980.5 kB 9.5%
🟡 _chunks/esm-CE0tAvRh.mjs    [####....................] 723.5 kB 7.0%
🟡 _chunks/chatgpt-model.mjs   [###.....................] 698.0 kB 6.8%
🟡 _chunks/frames-Boesh7kU.mjs [###.....................] 680.2 kB 6.6%
🔴 Other bundled files         [########################] 4.96 MB 48.2%

🧾 Vercel Config

{
  "handler": "index.mjs",
  "launcherType": "Nodejs",
  "shouldAddHelpers": false,
  "supportsResponseStreaming": true,
  "runtime": "nodejs24.x"
}

Build Timing: e2e/fixtures/agent-tools-sandbox

This is an informational timing measurement inside eve build, from preflight through publication. Output-size measurement and profile writing are excluded.

Build mode: deployable Vercel build with sandbox template prewarm included.

  • Build pipeline: 2.83 s -> 2.82 s (-16.8 ms) vs main (16c7f24).
  • Timing is informational: shared GitHub runners are too variable for a hard timing budget.
Detailed phase timings vs `main (16c7f24)`
Phase Baseline Current Delta
extension.check 12.9 ms 0.9 ms -12.0 ms
project.resolve 0.3 ms 0.3 ms 0.0 ms
workspace.create 0.5 ms 0.4 ms -0.1 ms
host.prepare 540.1 ms 492.3 ms -47.8 ms
vercel.service-prefix.resolve 1.4 ms 1.4 ms 0.0 ms
nitro.create 462.4 ms 476.3 ms +13.9 ms
sandbox.prewarm 403.4 ms 387.6 ms -15.8 ms
nitro.cache.prepare 0.2 ms 0.2 ms 0.0 ms
nitro.prepare 0.9 ms 1.0 ms +0.1 ms
nitro.public-assets 0.7 ms 0.7 ms 0.0 ms
nitro.prerender 0.4 ms 0.4 ms 0.0 ms
nitro.bundle 1.33 s 1.40 s +61.7 ms
nitro.cache.write 0.3 ms 0.3 ms 0.0 ms
vercel.workflow-function.materialize 57.6 ms 53.4 ms -4.2 ms
agent-summary.emit 0.7 ms 0.7 ms 0.0 ms
nitro.close 0.1 ms 0.1 ms 0.0 ms
output.publish 15.4 ms 2.8 ms -12.6 ms
workspace.remove 1.6 ms 1.7 ms +0.1 ms

Signed-off-by: Andrew Barba <barba@hey.com>
@AndrewBarba
AndrewBarba merged commit b3e4b73 into main Sep 4, 2026
117 checks passed
@AndrewBarba
AndrewBarba deleted the barba/hook-owner-reads branch September 4, 2026 17:23
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.

2 participants