Skip to content

fix(eve): flush and shut down the registered meter provider - #2818

Open
chadhietala wants to merge 1 commit into
mainfrom
chad/otel-meter-lifecycle
Open

fix(eve): flush and shut down the registered meter provider#2818
chadhietala wants to merge 1 commit into
mainfrom
chad/otel-meter-lifecycle

Conversation

@chadhietala

@chadhietala chadhietala commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

registerOTel creates a global meter provider and enables declared instrumentations, but eve previously retained lifecycle control only for the tracer provider. Buffered metrics could therefore be lost at teardown, while instrumentations could continue recording into providers that had already shut down. Registration now captures the global meter provider so forceFlush includes metrics, and shutdown disables declared instrumentations before shutting down both providers. Without metric readers the meter lifecycle remains a no-op; shutdown still leaves the global context manager untouched, and failed-registration rollback is unchanged.

Validation

Scenario coverage registers real metric and instrumentation pipelines and verifies metric flush and shutdown plus instrumentation enable and disable behavior. Both regressions fail without the fix; the full otel-registration scenario and unit suites pass.

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)

Diff size

Docs — 1 file · +5 / -0

Adds the patch changeset for complete OpenTelemetry lifecycle handling.

Implementation — 2 files · +60 / -4

Captures the feature-detected meter lifecycle, sequences instrumentation shutdown, and adds the matching vendored API declaration.

Tests — 1 file · +41 / -0

Adds focused scenarios for metric-reader lifecycle and instrumentation cleanup.

registerOTel registers a global meter provider when metric readers are
declared and installs auto-instrumentations, but returns no handle to
either, and eve retained lifecycle control over only the tracer
provider. Metrics buffered near teardown were silently dropped and
instrumentations kept recording into shut-down providers. The
registration now captures the global meter provider so forceFlush and
shutdown cover it alongside the tracer provider, and shutdown disables
declared instrumentations first. The vendored API declaration gains
metrics.getMeterProvider, which the runtime already exposed.

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

vercel Bot commented Sep 1, 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 1, 2026 1:29am UTC
eve-docs-4759 Ready Ready Preview, v0 Sep 1, 2026 1:29am UTC
eve-pkg Ready Ready Preview, v0 Sep 1, 2026 1:29am UTC
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

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

Key takeaways

  • No notable deltas vs main (bc2a1f6).

Delta vs main (bc2a1f6)

Area Metric Baseline Current Delta
Package Packed tarball 8.54 MB 8.54 MB +979 B ⚠️
Package Unpacked publish size 32.18 MB 32.18 MB +744 B ⚠️
Package Installed footprint 77.21 MB 77.21 MB +744 B ⚠️
Package Published files 3668 3668 0
Package Installed files 7503 7503 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 19.86 MB 19.86 MB -80 B ✅
Runtime Public routes 17 17 0
Changed function payloads vs main (bc2a1f6) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 9.93 MB 9.93 MB -40 B ✅ none
functions/.well-known/workflow/v1/flow.func changed 9.93 MB 9.93 MB -40 B ✅ none

eve init install

Metric Baseline Current Delta
Installed footprint 115.64 MB 115.64 MB +744 B ⚠️
Installed packages 120 120 0
dependencies 4 4 0
devDependencies 2 2 0
Dependency package bytes 48.15 MB 48.15 MB +744 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-01T01:31:02.292Z
  • Route aliases: 17 public, 1 internal (18 total aliases)
  • Vercel routes in config: 20
  • Severity legend: 🔴 dominant/large, 🟠 notable, 🟡 watch, ⚪ small
Package Drill-Down

Package Details

  • Package: eve@0.47.6
  • Package directory: packages/eve
  • Tarball: 8.54 MB (eve-0.47.6.tgz)
  • Unpacked payload: 32.18 MB across 3668 published files
  • Installed footprint: 77.21 MB across 7503 installed files
  • Installed root package: 30.72 MB
  • Installed dependencies: 46.48 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: 30.72 MB (39.8%)
  • @rolldown/binding-linux-x64-gnu: 19.33 MB (25.0%)
  • ai: 6.89 MB (8.9%)
  • zod: 6.41 MB (8.3%)
  • undici: 3.50 MB (4.5%)
Publish payload breakdown
Published file size
🔴 dist/src/compiled/shadcn-registry/index.js       [#############...........] 9.76 MB 30.3%
🟠 dist/src/compiled/@photon-ai/chat-adapter-ime... [###.....................] 2.27 MB 7.1%
🟠 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-Dzn... [#.......................] 512.7 kB 1.6%
🔴 Other published files                            [########################] 17.71 MB 55.0%
Installed footprint breakdown
Installed package size
🔴 eve                             [########################] 30.72 MB 39.8%
🔴 @rolldown/binding-linux-x64-gnu [###############.........] 19.33 MB 25.0%
🔴 ai                              [#####...................] 6.89 MB 8.9%
🔴 zod                             [#####...................] 6.41 MB 8.3%
🟠 undici                          [###.....................] 3.50 MB 4.5%
🟠 nitro                           [##......................] 2.41 MB 3.1%
🔴 Other installed packages        [######..................] 7.94 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: 115.64 MB across 9399 installed files
  • Installed packages: 120 total (114 transitive-only)
  • dependencies: 4 direct packages totaling 48.15 MB
  • devDependencies: 2 direct packages totaling 5.04 MB
  • Other transitive package files: 62.45 MB

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

Heavy installed dependencies

  • eve: 30.72 MB (26.6%)
  • @typescript/typescript-linux-x64: 27.95 MB (24.2%)
  • @rolldown/binding-linux-x64-gnu: 19.33 MB (16.7%)
  • zod: 10.37 MB (9.0%)
  • ai: 6.89 MB (6.0%)
Installed footprint breakdown
Installed package size
🔴 eve                              [########################] 30.72 MB 26.6%
🔴 @typescript/typescript-linux-x64 [######################..] 27.95 MB 24.2%
🔴 @rolldown/binding-linux-x64-gnu  [###############.........] 19.33 MB 16.7%
🔴 zod                              [########................] 10.37 MB 9.0%
🔴 ai                               [#####...................] 6.89 MB 6.0%
🟠 undici                           [###.....................] 3.50 MB 3.0%
🔴 Other installed packages         [#############...........] 16.88 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.89 MB 6.0%
eve file:eve-0.47.6.tgz 30.72 MB 26.6%
zod 4.5.4 10.37 MB 9.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.93 MB 50.0%
🔴 functions/__server.func                         [########################] 9.93 MB 50.0%

Top Function Payloads

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

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                       [##########..............] 1.97 MB 19.9%
🟡 _libs/undici.mjs                [#####...................] 980.5 kB 9.9%
🟡 _chunks/decode_jwt-Col8JOnf.mjs [###.....................] 702.6 kB 7.1%
🟡 _chunks/frames-Dl2Hcma4.mjs     [###.....................] 678.7 kB 6.8%
🟡 _chunks/chatgpt-model.mjs       [###.....................] 667.7 kB 6.7%
🔴 Other bundled files             [########################] 4.93 MB 49.6%

🧾 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 • 16 public routes, 1 internal alias • 9.93 MB
Metric Value
Public routes /
/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 9.93 MB
Function files 9.93 MB across 110 files
Traced dependencies 0 B
Signal 🟠 Bundled file index.mjs is 1.97 MB (19.9%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                       [##########..............] 1.97 MB 19.9%
🟡 _libs/undici.mjs                [#####...................] 980.5 kB 9.9%
🟡 _chunks/decode_jwt-Col8JOnf.mjs [###.....................] 702.6 kB 7.1%
🟡 _chunks/frames-Dl2Hcma4.mjs     [###.....................] 678.7 kB 6.8%
🟡 _chunks/chatgpt-model.mjs       [###.....................] 667.7 kB 6.7%
🔴 Other bundled files             [########################] 4.93 MB 49.6%

🧾 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.39 s -> 2.37 s (-17.0 ms) vs main (bc2a1f6).
  • Timing is informational: shared GitHub runners are too variable for a hard timing budget.
Detailed phase timings vs `main (bc2a1f6)`
Phase Baseline Current Delta
extension.check 11.5 ms 11.7 ms +0.2 ms
project.resolve 0.4 ms 0.4 ms 0.0 ms
workspace.create 0.7 ms 0.8 ms +0.1 ms
host.prepare 410.3 ms 413.2 ms +2.9 ms
vercel.service-prefix.resolve 2.0 ms 1.6 ms -0.4 ms
nitro.create 277.5 ms 277.7 ms +0.2 ms
sandbox.prewarm 259.2 ms 247.5 ms -11.7 ms
nitro.cache.prepare 0.2 ms 0.6 ms +0.4 ms
nitro.prepare 0.7 ms 2.8 ms +2.1 ms
nitro.public-assets 0.7 ms 0.9 ms +0.2 ms
nitro.prerender 0.5 ms 0.6 ms +0.1 ms
nitro.bundle 1.37 s 1.36 s -10.4 ms
nitro.cache.write 0.3 ms 0.3 ms 0.0 ms
vercel.workflow-function.materialize 42.5 ms 41.4 ms -1.1 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 3.0 ms 3.2 ms +0.2 ms
workspace.remove 1.9 ms 1.9 ms 0.0 ms
@chadhietala
chadhietala enabled auto-merge (squash) September 1, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants