Skip to content

fix(eve): drop contradictory OpenAPI schema annotations - #2370

Draft
vercel-gh-bot-2[bot] wants to merge 1 commit into
mainfrom
e0/af49ea7c-e712-4e30-9666-107d23ce4e02
Draft

fix(eve): drop contradictory OpenAPI schema annotations#2370
vercel-gh-bot-2[bot] wants to merge 1 commit into
mainfrom
e0/af49ea7c-e712-4e30-9666-107d23ce4e02

Conversation

@vercel-gh-bot-2

Copy link
Copy Markdown
Contributor

OpenAPI connections now remove default and example values that disagree with the schema's effective JSON type. This stops model-facing tool schemas from recommending input that eve's local validator rejects.

Motivation

Published OpenAPI specs can contradict themselves. For example, Xero declares boolean request fields with default: "false" and example: "false". On main, the generated tool schema preserves both strings, so a model can submit a string where the generated validator requires a boolean.

A focused regression reproduced this before the fix:

Expected: { type: "boolean" }
Received: { type: "boolean", default: "false", example: "false" }

Fixes #2369

Hypothesis

Because derefSchema already normalizes JSON Schema types and OpenAPI nullability, checking annotations after those passes should remove only contradictory values. Matching values, nullable values, and annotations on untyped schemas should remain unchanged.

The dereferencer now treats annotation values as opaque data rather than nested schemas. This preserves object-valued defaults and examples while validating their outer JSON type.

Verification

  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/runtime/connections/openapi-client.test.ts passed all 36 tests
  • pnpm --filter eve typecheck passed
  • oxfmt --check and oxlint passed on changed files
  • The full eve unit suite passed 6,976 of 6,981 tests. Four unrelated tests failed under full-suite load, and all 98 tests in those three files passed when run together in isolation
  • Correctness re-review passed after adding coverage for opaque object-valued annotations

Requested by: Josh-moreton

Co-authored-by: Josh-moreton <144067649+Josh-moreton@users.noreply.github.com>
@vercel

vercel Bot commented Aug 21, 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 21, 2026 8:07am
eve-docs-4759 Ready Ready Preview, v0 Aug 21, 2026 8:07am
eve-pkg Ready Ready Preview Aug 21, 2026 8:07am

@github-actions

Copy link
Copy Markdown
Contributor

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

Key takeaways

  • No notable deltas vs main (f3f4f4a).

Delta vs main (f3f4f4a)

Area Metric Baseline Current Delta
Package Packed tarball 8.17 MB 8.17 MB +212 B ⚠️
Package Unpacked publish size 30.74 MB 30.74 MB +626 B ⚠️
Package Installed footprint 74.82 MB 74.82 MB +626 B ⚠️
Package Published files 3386 3386 0
Package Installed files 7097 7097 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 18.20 MB 18.20 MB +1.5 kB ⚠️
Runtime Public routes 16 16 0
Changed function payloads vs main (f3f4f4a) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 9.10 MB 9.10 MB +733 B ⚠️ none
functions/.well-known/workflow/v1/flow.func changed 9.10 MB 9.10 MB +733 B ⚠️ none

eve init install

Metric Baseline Current Delta
Installed footprint 113.23 MB 113.23 MB +626 B ⚠️
Installed packages 121 121 0
dependencies 4 4 0
devDependencies 2 2 0
Dependency package bytes 45.26 MB 45.26 MB +626 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-08-21T08:08:33.972Z
  • 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.42.0
  • Package directory: packages/eve
  • Tarball: 8.17 MB (eve-0.42.0.tgz)
  • Unpacked payload: 30.74 MB across 3386 published files
  • Installed footprint: 74.82 MB across 7097 installed files
  • Installed root package: 29.38 MB
  • Installed dependencies: 45.44 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.38 MB (39.3%)
  • @rolldown/binding-linux-x64-gnu: 19.91 MB (26.6%)
  • ai: 6.71 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.8%
🟠 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.01 MB 52.1%
Installed footprint breakdown
Installed package size
🔴 eve                             [########################] 29.38 MB 39.3%
🔴 @rolldown/binding-linux-x64-gnu [################........] 19.91 MB 26.6%
🔴 ai                              [#####...................] 6.71 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.84 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.23 MB across 8973 installed files
  • Installed packages: 121 total (115 transitive-only)
  • dependencies: 4 direct packages totaling 45.26 MB
  • devDependencies: 2 direct packages totaling 5.04 MB
  • Other transitive package files: 62.93 MB

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

Heavy installed dependencies

  • eve: 29.38 MB (25.9%)
  • @typescript/typescript-linux-x64: 27.95 MB (24.7%)
  • @rolldown/binding-linux-x64-gnu: 19.91 MB (17.6%)
  • zod: 9.02 MB (8.0%)
  • ai: 6.71 MB (5.9%)
Installed footprint breakdown
Installed package size
🔴 eve                              [########################] 29.38 MB 25.9%
🔴 @typescript/typescript-linux-x64 [#######################.] 27.95 MB 24.7%
🔴 @rolldown/binding-linux-x64-gnu  [################........] 19.91 MB 17.6%
🔴 zod                              [#######.................] 9.02 MB 8.0%
🔴 ai                               [#####...................] 6.71 MB 5.9%
🟠 undici                           [###.....................] 3.50 MB 3.1%
🔴 Other installed packages         [##############..........] 16.75 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.71 MB 5.9%
eve file:eve-0.42.0.tgz 29.38 MB 25.9%
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.30 s -> 2.20 s (-100.1 ms) vs main (f3f4f4a).
  • Timing is informational: shared GitHub runners are too variable for a hard timing budget.
Detailed phase timings vs `main (f3f4f4a)`
Phase Baseline Current Delta
extension.check 11.0 ms 11.0 ms 0.0 ms
project.resolve 0.4 ms 0.4 ms 0.0 ms
workspace.create 0.8 ms 0.7 ms -0.1 ms
host.prepare 357.5 ms 277.7 ms -79.8 ms
vercel.service-prefix.resolve 1.9 ms 2.0 ms +0.1 ms
nitro.create 240.7 ms 230.3 ms -10.4 ms
sandbox.prewarm 286.1 ms 279.5 ms -6.6 ms
nitro.cache.prepare 0.3 ms 0.2 ms -0.1 ms
nitro.prepare 0.8 ms 0.7 ms -0.1 ms
nitro.public-assets 0.7 ms 0.7 ms 0.0 ms
nitro.prerender 0.5 ms 0.5 ms 0.0 ms
nitro.bundle 1.37 s 1.36 s -2.6 ms
nitro.cache.write 0.3 ms 0.3 ms 0.0 ms
vercel.workflow-function.materialize 23.0 ms 22.3 ms -0.7 ms
agent-summary.emit 0.5 ms 0.5 ms 0.0 ms
nitro.close 0.2 ms 0.1 ms -0.1 ms
output.publish 3.1 ms 3.2 ms +0.1 ms
workspace.remove 1.9 ms 1.9 ms 0.0 ms

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.

OpenAPI connections pass through default/example values that contradict the schema's own type, causing models to emit invalid tool input

0 participants