Skip to content

refactor(eve): construct instrumentation at channel delivery - #2448

Draft
chadhietala wants to merge 2 commits into
mainfrom
chad/channel-audience-construction-minimal
Draft

refactor(eve): construct instrumentation at channel delivery#2448
chadhietala wants to merge 2 commits into
mainfrom
chad/channel-audience-construction-minimal

Conversation

@chadhietala

Copy link
Copy Markdown
Contributor

Summary

  • Resolve channel audience at delivery into one serializable drop/record decision.
  • Construct the harness hooks, telemetry settings, integrations, and trace callbacks from that decision.
  • Remove channelAudience from harness lifecycle events, attempt scopes, trace state, and span export context.
  • Persist only the decision for turn continuations; the harness receives neither audience nor the decision.

This intentionally limits the change to the audience-to-harness boundary. It does not add a general execution-scope framework, wrap unrelated durable operations, change dynamic resolver channel metadata, or bump the channel extension epoch.

tracePolicy now returns { action: "drop" } or { action: "record", recordInputs, recordOutputs }. Production defaults still record public conversations and drop private/unknown conversations; local tracing continues to admit public/unknown conversations.

Follow-up to #2335. Replaces #2438.

Validation

  • ./node_modules/.bin/tsc -p packages/eve/tsconfig.json --noEmit
  • Full unit suite: 667 files, 7,021 passed, 1 skipped
  • Focused integration suite: 2 files, 2 passed
  • node ./scripts/guard-invariants.mjs
  • oxlint on changed TypeScript files
  • Docs validation, snippets, and MDX compilation
  • git diff --check

Stack

This draft is stacked on #2400 to isolate the diff from the branch that was active when the work began. Retarget to main after #2400 merges.

Signed-off-by: Chad Hietala <chad.hietala@vercel.com>
Signed-off-by: Chad Hietala <chad.hietala@vercel.com>
@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
eve-docs Ready Ready Preview Aug 24, 2026 10:47am
eve-docs-4759 Ready Ready Preview, v0 Aug 24, 2026 10:47am
eve-pkg Ready Ready Preview Aug 24, 2026 10:47am

@github-actions

Copy link
Copy Markdown
Contributor

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

Key takeaways

  • No notable deltas vs chad/vercel-session-id-trace-attribute-v2 (c4f5592).

Delta vs chad/vercel-session-id-trace-attribute-v2 (c4f5592)

Area Metric Baseline Current Delta
Package Packed tarball 8.18 MB 8.19 MB +1.7 kB ⚠️
Package Unpacked publish size 30.81 MB 30.81 MB +3.7 kB ⚠️
Package Installed footprint 75.00 MB 75.00 MB +3.7 kB ⚠️
Package Published files 3404 3408 +4
Package Installed files 7118 7122 +4
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 18.20 MB 18.20 MB +1.9 kB ⚠️
Runtime Public routes 16 16 0
Changed function payloads vs chad/vercel-session-id-trace-attribute-v2 (c4f5592) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 9.10 MB 9.10 MB +974 B ⚠️ none
functions/.well-known/workflow/v1/flow.func changed 9.10 MB 9.10 MB +974 B ⚠️ none

eve init install

Metric Baseline Current Delta
Installed footprint 113.41 MB 113.41 MB +3.7 kB ⚠️
Installed packages 121 121 0
dependencies 4 4 0
devDependencies 2 2 0
Dependency package bytes 45.37 MB 45.38 MB +3.7 kB ⚠️
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-08-24T10:48:56.644Z
  • Route aliases: 16 public, 1 internal (17 total aliases)
  • Vercel routes in config: 19
  • Severity legend: 🔴 dominant/large, 🟠 notable, 🟡 watch, ⚪ small
Package Drill-Down

Package Details

  • Package: eve@0.44.0
  • Package directory: packages/eve
  • Tarball: 8.19 MB (eve-0.44.0.tgz)
  • Unpacked payload: 30.81 MB across 3408 published files
  • Installed footprint: 75.00 MB across 7122 installed files
  • Installed root package: 29.46 MB
  • Installed dependencies: 45.55 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: 29.46 MB (39.3%)
  • @rolldown/binding-linux-x64-gnu: 19.91 MB (26.5%)
  • ai: 6.75 MB (9.0%)
  • zod: 5.07 MB (6.8%)
  • undici: 3.50 MB (4.7%)
Publish payload breakdown
Published file size
🔴 dist/src/compiled/shadcn-registry/index.js       [###############.........] 9.76 MB 31.7%
🟠 dist/src/compiled/@photon-ai/chat-adapter-ime... [####....................] 2.44 MB 7.9%
🟠 dist/src/compiled/experimental-ai-sdk-code-mo... [##......................] 1.51 MB 4.9%
🟡 dist/src/compiled/_chunks/workflow/undici-Dzn... [#.......................] 512.7 kB 1.7%
🟡 dist/src/compiled/_chunks/node/undici-CUIybhQ... [#.......................] 508.8 kB 1.7%
🔴 Other published files                            [########################] 16.08 MB 52.2%
Installed footprint breakdown
Installed package size
🔴 eve                             [########################] 29.46 MB 39.3%
🔴 @rolldown/binding-linux-x64-gnu [################........] 19.91 MB 26.5%
🔴 ai                              [######..................] 6.75 MB 9.0%
🔴 zod                             [####....................] 5.07 MB 6.8%
🟠 undici                          [###.....................] 3.50 MB 4.7%
🟠 nitro                           [##......................] 2.41 MB 3.2%
🔴 Other installed packages        [######..................] 7.90 MB 10.5%
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.0.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: 113.41 MB across 8998 installed files
  • Installed packages: 121 total (115 transitive-only)
  • dependencies: 4 direct packages totaling 45.38 MB
  • devDependencies: 2 direct packages totaling 5.04 MB
  • Other transitive package files: 63.00 MB

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

Heavy installed dependencies

  • eve: 29.46 MB (26.0%)
  • @typescript/typescript-linux-x64: 27.95 MB (24.6%)
  • @rolldown/binding-linux-x64-gnu: 19.91 MB (17.6%)
  • zod: 9.02 MB (8.0%)
  • ai: 6.75 MB (6.0%)
Installed footprint breakdown
Installed package size
🔴 eve                              [########################] 29.46 MB 26.0%
🔴 @typescript/typescript-linux-x64 [#######################.] 27.95 MB 24.6%
🔴 @rolldown/binding-linux-x64-gnu  [################........] 19.91 MB 17.6%
🔴 zod                              [#######.................] 9.02 MB 8.0%
🔴 ai                               [######..................] 6.75 MB 6.0%
🟠 undici                           [###.....................] 3.50 MB 3.1%
🔴 Other installed packages         [##############..........] 16.82 MB 14.8%
dependencies (4)
Package Range Installed size Share
@vercel/connect 0.4.3 141.0 kB 0.1%
ai ^7.0.58 6.75 MB 6.0%
eve file:eve-0.44.0.tgz 29.46 MB 26.0%
zod 4.4.3 9.02 MB 8.0%
devDependencies (2)
Package Range Installed size Share
@types/node 24.x 2.54 MB 2.2%
typescript 7.0.2 2.50 MB 2.2%
Function Drill-Down

Payload Size Graph

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

Top Function Payloads

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

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                     [##################......] 2.06 MB 22.7%
🟠 _chunks/runtime-artifacts.mjs [###############.........] 1.80 MB 19.8%
🟡 _libs/undici.mjs              [########................] 980.5 kB 10.8%
🟡 _chunks/sandbox.mjs           [#######.................] 781.9 kB 8.6%
🟡 _chunks/frames-Dl2Hcma4.mjs   [######..................] 678.7 kB 7.5%
🟠 Other bundled files           [########################] 2.80 MB 30.8%

🧾 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 • 15 public routes, 1 internal alias • 9.10 MB
Metric Value
Public routes /
/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 9.10 MB
Function files 9.10 MB across 50 files
Traced dependencies 0 B
Signal 🟠 Bundled file index.mjs is 2.06 MB (22.7%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                     [##################......] 2.06 MB 22.7%
🟠 _chunks/runtime-artifacts.mjs [###############.........] 1.80 MB 19.8%
🟡 _libs/undici.mjs              [########................] 980.5 kB 10.8%
🟡 _chunks/sandbox.mjs           [#######.................] 781.9 kB 8.6%
🟡 _chunks/frames-Dl2Hcma4.mjs   [######..................] 678.7 kB 7.5%
🟠 Other bundled files           [########################] 2.80 MB 30.8%

🧾 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.33 s -> 2.06 s (-272.4 ms) vs chad/vercel-session-id-trace-attribute-v2 (c4f5592).
  • Timing is informational: shared GitHub runners are too variable for a hard timing budget.
Detailed phase timings vs `chad/vercel-session-id-trace-attribute-v2 (c4f5592)`
Phase Baseline Current Delta
extension.check 9.5 ms 10.5 ms +1.0 ms
project.resolve 0.3 ms 0.4 ms +0.1 ms
workspace.create 0.6 ms 0.6 ms 0.0 ms
host.prepare 294.1 ms 278.8 ms -15.3 ms
vercel.service-prefix.resolve 1.5 ms 1.9 ms +0.4 ms
nitro.create 194.3 ms 200.6 ms +6.3 ms
sandbox.prewarm 238.0 ms 242.3 ms +4.3 ms
nitro.cache.prepare 0.2 ms 0.2 ms 0.0 ms
nitro.prepare 0.7 ms 0.8 ms +0.1 ms
nitro.public-assets 0.6 ms 0.5 ms -0.1 ms
nitro.prerender 0.4 ms 0.3 ms -0.1 ms
nitro.bundle 1.20 s 1.05 s -142.8 ms
nitro.cache.write 0.2 ms 0.2 ms 0.0 ms
vercel.workflow-function.materialize 89.9 ms 258.4 ms +168.5 ms
agent-summary.emit 0.5 ms 0.6 ms +0.1 ms
nitro.close 0.1 ms 0.1 ms 0.0 ms
output.publish 294.5 ms 3.0 ms -291.5 ms
workspace.remove 5.3 ms 1.7 ms -3.6 ms

@vercel vercel Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Additional Suggestion:

On a cancelled first-delivery turn, InstrumentationDecisionKey is dropped from the preserved serialized context, so the cancellation epilogue reconstructs UNINSTRUMENTED and never closes provider operations opened by the discarded step nor emits turn.cancelled instrumentation.

Fix on Vercel

@chadhietala
chadhietala force-pushed the chad/vercel-session-id-trace-attribute-v2 branch from c4f5592 to 659ba80 Compare August 24, 2026 12:28
Base automatically changed from chad/vercel-session-id-trace-attribute-v2 to main August 24, 2026 12:38
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