Skip to content

feat(channels)!: typed delivery hooks and canonical channel identity - #5139

Closed
OwenKephart wants to merge 2 commits into
v27/lab-fixesfrom
stack/v27-api-channel-delivery
Closed

OwenKephart wants to merge 2 commits into
v27/lab-fixesfrom
stack/v27-api-channel-delivery

Conversation

@OwenKephart

@OwenKephart OwenKephart commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Superseded by #4855. This layer is included in consolidated signed commit 473ac5999bc359a116855ed394413b9cd3000d24. Its source is unchanged from the green final stack tree. Review history below is retained; continue review on #4855. The original branch and backup ref are preserved. This PR was closed, not merged.

Summary

A channel delivery hook previously returned harness StepInput, so adapter authors had to know execution internals to replace an inbound message. It now receives a typed ChannelDelivery<TState> and returns a channel-owned replacement without a state field. The framework performs the harness conversion and preserves transport-owned context.

What changes

  • Return a replacement delivery to change message, answers, context, or output schema; mutate state through the channel context. Return nothing to keep the original; return {} to remove this delivery's input.
  • Run a delivery-only channel's hook even when it has no state or event handlers. Preserve hidden client context and original input text across replacement.
  • Derive authored channel identity from its path as channel:<name>; remove the separate kindHint choice.
  • Classify audience through the explicit audience callback, not metadata().audience. Missing classification stays unknown, rather than implying visibility.

Before / after

Inside defineChannel, replace the harness-shaped return with channel-owned input:

// Before: deliver() returns StepInput
deliver(payload) {
  return {
    message: normalize(payload.message),
    answers: payload.answers,
    context: payload.context,
    outputSchema: payload.outputSchema,
  };
}

// After: return channel-owned input; no manual harness conversion
// Return nothing to keep the original payload.
deliver(payload) {
  return { ...payload, message: normalize(payload.message) };
}

The replacement affects this inbound delivery, not other queued messages or authorization.

Diff size

Production source: approximately +94 / −146 lines (net −52).

Relative to this PR's actual single-commit parent; excludes tests/test support, docs, fixtures, generated reports, and compatibility-table data. Includes types, comments, and code moves.

Validation

The latest assembled tree passed all 11 workspace unit tasks (7,941 core tests), focused approval and child-session regressions, package and affected eval-fixture types, docs, formatting, lint, and invariants. The prior assembled baseline also passed the workspace build, 54-script typecheck matrix with dependencies, and real crash/promotion coverage. E2E runs only in CI; no local e2e success is claimed.

Fresh exact-head CI/e2e: green on 4b953039089e0484e4b0cb0f9e26fccb5156d14b. CI · E2E Tests (Local) · E2E Tests (Postgres) · E2E Tests (Vercel)

Intermediate heads are not independently gated. The tested integration base is 7a908f57c; later main changes are deferred to the planned restack. No merge or auto-merge is authorized.

Checklist

  • 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 Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
eve-docs Error Error v0 Oct 22, 2026 1:54am UTC
1 Skipped Deployment
Project Deployment Actions Updated
eve-pkg Skipped Skipped v0 Oct 22, 2026 1:54am UTC

@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

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

Key takeaways

  • No notable deltas vs v27/lab-fixes (4a13bcf).

Delta vs v27/lab-fixes (4a13bcf)

Area Metric Baseline Current Delta
Package Packed tarball 7.72 MB 7.72 MB +197 B ⚠️
Package Unpacked publish size 27.15 MB 27.15 MB +2.5 kB ⚠️
Package Installed footprint 75.15 MB 75.15 MB +2.5 kB ⚠️
Package Published files 4281 4283 +2
Package Installed files 8262 8264 +2
Package Installed package instances 34 34 0
Package Distinct installed package names 32 32 0
Package Installed dependency edges 51 51 0
Package Installed optional peer edges 9 9 0
Runtime Unique function payloads 2 2 0
Runtime Total function bytes 21.08 MB 21.08 MB -836 B ✅
Runtime Public routes 17 17 0
Changed function payloads vs v27/lab-fixes (4a13bcf) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 10.54 MB 10.54 MB -418 B ✅ none
functions/.well-known/workflow/v1/flow.func changed 10.54 MB 10.54 MB -418 B ✅ none

eve init install

Metric Baseline Current Delta
Installed footprint 113.53 MB 113.53 MB +2.5 kB ⚠️
Installed packages 97 97 0
dependencies 4 4 0
devDependencies 2 2 0
Dependency package bytes 45.45 MB 45.45 MB +2.5 kB ⚠️
devDependency package bytes 5.11 MB 5.11 MB 0 B ➖
Build Metadata
  • Preset: vercel
  • Nitro: nitro@3.0.260903-beta
  • Output directory: apps/fixtures/weather-agent/.vercel/output
  • Build metadata timestamp: 2026-10-21T06:43:51.427Z
  • 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.81.1
  • Package directory: packages/eve
  • Tarball: 7.72 MB (eve-0.81.1.tgz)
  • Unpacked payload: 27.15 MB across 4283 published files
  • Installed footprint: 75.15 MB across 8264 installed files
  • Installed root package: 27.15 MB
  • Installed dependencies: 48.00 MB
  • Installed package instances: 34
  • Distinct installed package names: 32
  • Installed dependency edges: 51
  • Installed optional peer edges: 9
  • Runtime dependencies: 2
  • Peer dependencies: 7 (6 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: 27.15 MB (36.1%)
  • @rolldown/binding-linux-x64-gnu: 19.81 MB (26.4%)
  • ai: 8.27 MB (11.0%)
  • zod: 6.15 MB (8.2%)
  • undici: 3.58 MB (4.8%)
Publish payload breakdown
Published file size
🟠 dist/src/compiled/shadcn-registry/index.js       [#####...................] 3.85 MB 14.2%
🟠 dist/src/compiled/@photon-ai/chat-adapter-ime... [###.....................] 2.29 MB 8.4%
🟠 dist/src/compiled/@ai-sdk/code-mode/index.js     [#.......................] 1.03 MB 3.8%
🟡 dist/src/compiled/@vercel/blob/index.js          [#.......................] 604.9 kB 2.2%
🟡 dist/src/compiled/_chunks/workflow/undici-D-I... [#.......................] 521.9 kB 1.9%
🔴 Other published files                            [########################] 18.86 MB 69.5%
Installed footprint breakdown
Installed package size
🔴 eve                             [########################] 27.15 MB 36.1%
🔴 @rolldown/binding-linux-x64-gnu [##################......] 19.81 MB 26.4%
🔴 ai                              [#######.................] 8.27 MB 11.0%
🔴 zod                             [#####...................] 6.15 MB 8.2%
🟠 undici                          [###.....................] 3.58 MB 4.8%
🟠 nitro                           [##......................] 1.89 MB 2.5%
🔴 Other installed packages        [#######.................] 8.30 MB 11.0%
Runtime dependencies (2)
Package Range Notes
nitro 3.0.260903-beta
undici 8.10.2
Peer dependencies (7)
Package Range Notes
@opentelemetry/api ^1.0.0 optional peer
ai catalog:
braintrust ^3.0.0 optional peer
chat ^4.42.0 optional peer
dd-trace ^6.13.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: 113.53 MB across 10181 installed files
  • Installed packages: 97 total (91 transitive-only)
  • dependencies: 4 direct packages totaling 45.45 MB
  • devDependencies: 2 direct packages totaling 5.11 MB
  • Other transitive package files: 62.97 MB

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

Heavy installed dependencies

  • @typescript/typescript-linux-x64: 27.95 MB (24.6%)
  • eve: 27.15 MB (23.9%)
  • @rolldown/binding-linux-x64-gnu: 19.81 MB (17.5%)
  • zod: 9.76 MB (8.6%)
  • ai: 8.27 MB (7.3%)
Installed footprint breakdown
Installed package size
🔴 @typescript/typescript-linux-x64 [########################] 27.95 MB 24.6%
🔴 eve                              [#######################.] 27.15 MB 23.9%
🔴 @rolldown/binding-linux-x64-gnu  [#################.......] 19.81 MB 17.5%
🔴 zod                              [########................] 9.76 MB 8.6%
🔴 ai                               [#######.................] 8.27 MB 7.3%
🟠 undici                           [###.....................] 3.58 MB 3.1%
🔴 Other installed packages         [###############.........] 17.02 MB 15.0%
dependencies (4)
Package Range Installed size Share
@vercel/connect 2.5.2 276.3 kB 0.2%
ai ^7.0.137 8.27 MB 7.3%
eve file:eve-0.81.1.tgz 27.15 MB 23.9%
zod 4.5.4 9.76 MB 8.6%
devDependencies (2)
Package Range Installed size Share
@types/node 24.x 2.61 MB 2.3%
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     [########################] 10.54 MB 50.0%
🔴 functions/__server.func                         [########################] 10.54 MB 50.0%

Top Function Payloads

🟠 functions/.well-known/workflow/v1/flow.func • 1 public route • 10.54 MB
Metric Value
Public routes /.well-known/workflow/v1/flow
Runtime nodejs24.x
Handler index.mjs
Payload 10.54 MB
Function files 10.54 MB across 125 files
Traced dependencies 0 B
Signal 🟠 Bundled file _chunks/vercel.web.mjs is 2.55 MB (24.2%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 _chunks/vercel.web.mjs                         [#############...........] 2.55 MB 24.2%
🟡 _libs/undici.mjs                               [#####...................] 994.5 kB 9.4%
🟡 _chunks/sandbox3.mjs                           [####....................] 815.0 kB 7.7%
🟡 _chunks/compiled-artifacts-instrumentation.mjs [####....................] 730.4 kB 6.9%
🟡 _chunks/token-util-Bla5Z0G9.mjs                [####....................] 704.5 kB 6.7%
🔴 Other bundled files                            [########################] 4.75 MB 45.1%

🧾 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 • 10.54 MB
Metric Value
Public routes /
/.well-known/workflow/v1/webhook/[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/session/[sessionId]/stubs
Internal aliases /__server
Runtime nodejs24.x
Handler index.mjs
Payload 10.54 MB
Function files 10.54 MB across 125 files
Traced dependencies 0 B
Signal 🟠 Bundled file _chunks/vercel.web.mjs is 2.55 MB (24.2%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 _chunks/vercel.web.mjs                         [#############...........] 2.55 MB 24.2%
🟡 _libs/undici.mjs                               [#####...................] 994.5 kB 9.4%
🟡 _chunks/sandbox3.mjs                           [####....................] 815.0 kB 7.7%
🟡 _chunks/compiled-artifacts-instrumentation.mjs [####....................] 730.4 kB 6.9%
🟡 _chunks/token-util-Bla5Z0G9.mjs                [####....................] 704.5 kB 6.7%
🔴 Other bundled files                            [########################] 4.75 MB 45.1%

🧾 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: 3.42 s -> 3.42 s (+2.4 ms) vs v27/lab-fixes (4a13bcf).
  • Timing is informational: shared GitHub runners are too variable for a hard timing budget.
Detailed phase timings vs `v27/lab-fixes (4a13bcf)`
Phase Baseline Current Delta
extension.check 20.2 ms 19.7 ms -0.5 ms
project.resolve 0.4 ms 0.4 ms 0.0 ms
workspace.create 0.5 ms 0.5 ms 0.0 ms
host.prepare 533.9 ms 495.9 ms -38.0 ms
vercel.service-prefix.resolve 1.6 ms 1.6 ms 0.0 ms
nitro.create 678.2 ms 696.5 ms +18.3 ms
sandbox.prewarm 322.3 ms 311.5 ms -10.8 ms
nitro.cache.prepare 0.2 ms 0.3 ms +0.1 ms
nitro.prepare 0.6 ms 0.7 ms +0.1 ms
nitro.public-assets 0.8 ms 0.8 ms 0.0 ms
nitro.prerender 0.5 ms 0.5 ms 0.0 ms
nitro.bundle 1.72 s 1.76 s +40.0 ms
nitro.cache.write 0.4 ms 0.3 ms -0.1 ms
vercel.workflow-function.materialize 52.3 ms 48.5 ms -3.8 ms
agent-summary.emit 0.7 ms 0.7 ms 0.0 ms
connect-manifest.emit 0.3 ms 0.3 ms 0.0 ms
nitro.close 0.1 ms 0.1 ms 0.0 ms
output.publish 3.4 ms 3.3 ms -0.1 ms
workspace.remove 2.4 ms 2.4 ms 0.0 ms

@OwenKephart
OwenKephart removed this pull request from stack #5148 October 21, 2026 16:09
@OwenKephart
OwenKephart added this pull request to stack #5168 October 21, 2026 16:09

@AndrewBarba AndrewBarba left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Found while reviewing the v27 stack from #4616 down, and checked by a second reviewer with fresh context. Still present at the top of the stack (cb570bea59, #5180).

@OwenKephart
OwenKephart force-pushed the stack/v27-api-channel-delivery branch from a13c8b1 to 296239d Compare October 22, 2026 00:55
Reconcile v27 committed facts with main's authored session usage policies.

Co-authored-by: Rui <1834568+ruiconti@users.noreply.github.com>
Co-authored-by: Andrew Barba <barba@hey.com>

Signed-off-by: OwenKephart <owen.kephart@vercel.com>
Address verified review findings in this layer while keeping the public API stack independently reviewable.

Signed-off-by: OwenKephart <owen.kephart@vercel.com>
@OwenKephart
OwenKephart force-pushed the stack/v27-api-channel-delivery branch from 296239d to 5b3d9c4 Compare October 22, 2026 01:54
@OwenKephart
OwenKephart force-pushed the stack/v27-api-channel-delivery branch from 296239d to 5b3d9c4 Compare October 22, 2026 01:54
@vercel
vercel Bot temporarily deployed to Preview – eve-pkg October 22, 2026 01:54 Inactive
OwenKephart added a commit that referenced this pull request Oct 22, 2026
Consolidate the reviewed 20-layer pre-1.0 stack into #4855 without changing its tested source tree.

Includes v27 committed session facts, delivery/read contracts, ordinary function-based dynamic capabilities and JSON memos, approval provenance, public API ownership and naming, cancellation and interrupted-step recovery, client following, and consolidated capability epochs.

Review history: #5139, #5140, #5141, #5142, #5143, #5144, #5145, #5146, #5147, #4616, #5053, #5149, #5150, #5151, #5153, #5154, #5155, #5156, #5165.

Integration base remains 7a908f5; later-main integration is deferred.

Co-authored-by: Rui <1834568+ruiconti@users.noreply.github.com>
Co-authored-by: Andrew Barba <barba@hey.com>
Signed-off-by: OwenKephart <owen.kephart@vercel.com>
@OwenKephart
OwenKephart removed this pull request from stack #5168 October 22, 2026 19:09

This branch had an error being deployed

1 failed and 1 inactive deployments
Preview – eve-pkg — 5b3d9c40 Deployed Oct 22, 2026 by vercel[bot]
Preview – eve-docs — 5b3d9c40 Deployed Oct 22, 2026 by vercel[bot]
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