Skip to content

feat(eve): support structured output for GitHub turns#221

Closed
marcoshernanz wants to merge 2 commits into
mainfrom
feat/github-turn-output-schema
Closed

feat(eve): support structured output for GitHub turns#221
marcoshernanz wants to merge 2 commits into
mainfrom
feat/github-turn-output-schema

Conversation

@marcoshernanz

Copy link
Copy Markdown

Description

Adds turn-scoped structured output to GitHub channel deliveries.

  • Lets GitHubInboundResult return outputSchema for webhook issue, pull-request, and comment turns.
  • Lets cross-channel receive(channel, ...) pass the same schema for scheduled or proactive retries.
  • Accepts Standard Schema or raw JSON Schema and normalizes it into durable JSON data.
  • Scopes schemas to the current conversation turn while preserving runtime continuations and task-mode HITL contracts.
  • Documents consuming validated results from the existing result.completed agent hook with trusted session auth.

Sessions remain durable and in conversation mode. This does not add a channel-specific result callback.

Closes #214.

Relationship to #207

#207 modifies cross-channel receive for empty deliveries. This PR touches the same receive path; whichever lands second should rebase and resolve that small overlap without changing either behavior.

How did you test your changes?

  • pnpm lint
  • pnpm fmt
  • pnpm typecheck
  • pnpm build
  • pnpm test — 3,895 unit tests and 369 integration tests passed
  • pnpm test:scenario — 260 passed, 15 platform-specific skips
  • pnpm docs:check
  • pnpm guard:invariants
  • pnpm guard:fixtures
  • pnpm check:deps
  • pnpm changeset status
  • NODE_ENV=test pnpm --filter agent-channels test:e2e — 2 evals and 5 gates passed

Credentialed workspace-wide E2E, model-driven TUI, and Vercel deployment checks could not complete because the required AI Gateway and Vercel credentials are unavailable locally. The relevant channel E2E passed in deterministic mock mode.

PR Checklist

  • I linked an issue with prior discussion confirming this change is wanted
  • I ran the relevant checks from CONTRIBUTING.md
  • I added tests and documentation where relevant
  • I added a changeset because this touches the published eve package
  • DCO sign-off passes for every commit (git commit --signoff)

Signed-off-by: Marcos Hernanz <96699542+marcoshernanz@users.noreply.github.com>
Signed-off-by: Marcos Hernanz <96699542+marcoshernanz@users.noreply.github.com>
@marcoshernanz marcoshernanz force-pushed the feat/github-turn-output-schema branch from c213a48 to 57d1916 Compare June 23, 2026 21:43
@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
eve-docs Ready Ready Preview, Comment, Open in v0 Jun 23, 2026 9:44pm

@github-actions

Copy link
Copy Markdown
Contributor

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

Key takeaways

  • No notable deltas vs main (d602309).

Delta vs main (d602309)

Area Metric Baseline Current Delta
Package Packed tarball 3.39 MB 3.39 MB +831 B ⚠️
Package Unpacked publish size 12.35 MB 12.36 MB +4.9 kB ⚠️
Package Installed footprint 52.56 MB 52.56 MB +4.9 kB ⚠️
Package Published files 2280 2280 0
Package Installed files 5485 5485 0
Runtime Unique function payloads 2 2 0
Runtime Total function bytes 9.35 MB 9.35 MB +716 B ⚠️
Runtime Public routes 9 9 0
Changed function payloads vs main (d602309) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 3.87 MB 3.87 MB +566 B ⚠️ none
functions/.well-known/workflow/v1/flow.func changed 5.48 MB 5.48 MB +150 B ⚠️ none
Build Metadata
  • Preset: vercel
  • Nitro: nitro@3.0.260610-beta
  • Output directory: apps/fixtures/weather-agent/.vercel/output
  • Build metadata timestamp: 2026-06-23T21:44:53.933Z
  • Route aliases: 9 public, 1 internal (10 total aliases)
  • Vercel routes in config: 10
  • Severity legend: 🔴 dominant/large, 🟠 notable, 🟡 watch, ⚪ small
Package Drill-Down

Package Details

  • Package: eve@0.13.3
  • Package directory: packages/eve
  • Tarball: 3.39 MB (eve-0.13.3.tgz)
  • Unpacked payload: 12.36 MB across 2280 published files
  • Installed footprint: 52.56 MB across 5485 installed files
  • Installed root package: 11.08 MB
  • Installed dependencies: 41.48 MB
  • Runtime dependencies: 1
  • Peer dependencies: 12 (11 optional)

Installed footprint is measured from an isolated temporary npm install of the packed tarball.

Heavy installed dependencies

  • @rolldown/binding-linux-x64-gnu: 20.61 MB (39.2%)
  • eve: 11.08 MB (21.1%)
  • ai: 6.20 MB (11.8%)
  • zod: 5.04 MB (9.6%)
  • nitro: 2.41 MB (4.6%)
Publish payload breakdown
Published file size
🟠 dist/src/compiled/experimental-ai-sdk-code-mo... [####....................] 1.51 MB 12.2%
🟡 dist/src/compiled/@workflow/core/runtime.js      [##......................] 775.4 kB 6.3%
🟡 dist/src/compiled/@vercel/sandbox/index.js       [##......................] 632.0 kB 5.1%
🟡 dist/src/compiled/@chat-adapter/slack/index.js   [#.......................] 438.4 kB 3.5%
🟡 dist/src/compiled/_chunks/workflow/attribute-... [#.......................] 370.9 kB 3.0%
🔴 Other published files                            [########################] 8.63 MB 69.9%
Installed footprint breakdown
Installed package size
🔴 @rolldown/binding-linux-x64-gnu [########################] 20.61 MB 39.2%
🔴 eve                             [#############...........] 11.08 MB 21.1%
🔴 ai                              [#######.................] 6.20 MB 11.8%
🔴 zod                             [######..................] 5.04 MB 9.6%
🟠 nitro                           [###.....................] 2.41 MB 4.6%
🟡 rolldown                        [#.......................] 771.0 kB 1.5%
🔴 Other installed packages        [########................] 6.46 MB 12.3%
Runtime dependencies (1)
Package Range Notes
nitro 3.0.260610-beta
Peer dependencies (12)
Package Range Notes
@opentelemetry/api ^1.0.0 optional peer
@sveltejs/kit ^2.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
next ^16.0.0 optional peer
nuxt ^4.0.0 optional peer
react ^19.0.0 optional peer
svelte ^5.0.0 optional peer
vite ^8.0.0 optional peer
vue ^3.5.0 optional peer
Function Drill-Down

Payload Size Graph

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

Top Function Payloads

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

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 __eve_nitro_handler__.mjs              [########################] 1.51 MB 27.6%
🟠 _chunks/runtime.mjs                    [###############.........] 958.4 kB 17.5%
🟡 _chunks/sandbox.mjs                    [############............] 766.0 kB 14.0%
🟡 _chunks/attribute-changes-Bi5DLT8S.mjs [########................] 472.2 kB 8.6%
🟡 _chunks/dist-DTchiX0N.mjs              [#######.................] 460.6 kB 8.4%
🟠 Other bundled files                    [#####################...] 1.31 MB 24.0%

🧾 Vercel Config

{
  "handler": "index.mjs",
  "launcherType": "Nodejs",
  "shouldAddHelpers": false,
  "supportsResponseStreaming": true,
  "runtime": "nodejs24.x",
  "environment": {
    "NODE_OPTIONS": "--experimental-require-module",
    "WORKFLOW_QUEUE_NAMESPACE": "eve"
  },
  "maxDuration": "max",
  "experimentalTriggers": [
    {
      "type": "queue/v2beta",
      "topic": "__eve_wkf_workflow_*",
      "consumer": "default",
      "retryAfterSeconds": 5,
      "initialDelaySeconds": 0
    }
  ]
}

🟠 functions/__server.func • 8 public routes, 1 internal alias • 3.87 MB
Metric Value
Public routes /
/eve/v1/callback/[token]
/eve/v1/connections/[name]/callback/[token]
/eve/v1/health
/eve/v1/info
/eve/v1/session
/eve/v1/session/[sessionId]
/eve/v1/session/[sessionId]/stream
Internal aliases /__server
Runtime nodejs24.x
Handler index.mjs
Payload 3.87 MB
Function files 3.87 MB across 21 files
Traced dependencies 0 B
Signal 🟠 Bundled file index.mjs is 1.40 MB (36.3%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                              [########################] 1.40 MB 36.3%
🟠 _chunks/runtime.mjs                    [###############.........] 875.8 kB 22.6%
🟠 _chunks/sandbox.mjs                    [#############...........] 766.0 kB 19.8%
🟡 _chunks/attribute-changes-Bi5DLT8S.mjs [########................] 448.5 kB 11.6%
⚪ _libs/zod.mjs                          [##......................] 114.2 kB 3.0%
🟡 Other bundled files                    [####....................] 258.8 kB 6.7%

🧾 Vercel Config

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

@marcoshernanz marcoshernanz marked this pull request as ready for review June 23, 2026 21:49
@marcoshernanz marcoshernanz marked this pull request as draft June 23, 2026 22:00
@marcoshernanz

Copy link
Copy Markdown
Author

Closing this draft in favor of #227, which contains only the generic receive(...) + outputSchema change requested in #224.

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.

Allow GitHub webhook and receive(github, ...) turns to request structured output

1 participant