Skip to content

fix(eve): tolerate shared skill metadata - #1798

Open
OwenKephart wants to merge 2 commits into
mainfrom
fix/skill-frontmatter-compatibility
Open

fix(eve): tolerate shared skill metadata#1798
OwenKephart wants to merge 2 commits into
mainfrom
fix/skill-frontmatter-compatibility

Conversation

@OwenKephart

@OwenKephart OwenKephart commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Currently, eve supports the full spec for skills frontmatter as defined by: https://agentskills.io/specification, outside of the experimental allowed-tools.

I did a search through the top N most popular skills on skills.sh to see if there are other popular keys that we should consider, and the short answer is no (lest we have eve's standard be an amalgamation of multiple other coding harness extensions).

The most popular non-standard conventions:

  • version
    • just a skill version identifier
    • irrelevant for eve usage
  • disable-model-invocation
    • means that the user has to invoke the skill directly (rather than the model directly invoking it)
    • this is claude code convention
    • we don't currently have a concept of "explicitly invoking a skill" in the same way that in other TUIs you can use skills as slash-commands. debatable if this would be useful
  • hidden
    • same as above
  • argument-hint
    • when invoking as a slash command, gives you a hint of what arguments you should put into the skill
    • claude code convention
    • again, we don't currently have a concept like this
  • allowed-tools
    • experimental extension to the existing spec
    • doesn't really jive with our existing notion of what a skill is -- there's not a single skill invocation lifecycle for which that tool usage would be relevant for. could maybe apply to the rest of the turn but not enough evidence right now that this would be particularly useful

The one thing that did turn up though was that, even though the official spec indicates that metadata should be of type Record<string, string>, many existing skills do not abide by this and have arbitrary structured data underneath certain metadata keys (lists, dicts, etc.).

This PR avoids erroring in those cases by stripping out the non-conforming fields.

Testing

  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/internal/helpers/markdown.test.ts
  • pnpm --filter eve exec vitest run --config vitest.integration.config.ts src/discover/skills.integration.test.ts
  • pnpm --filter eve typecheck
  • pnpm docs:check
  • targeted oxfmt and oxlint checks

Signed-off-by: owenkephart <owen.kephart@vercel.com>
@vercel

vercel Bot commented Aug 7, 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 7, 2026 5:01pm
eve-docs-4759 Ready Ready Preview, v0 Aug 7, 2026 5:01pm

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

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

Key takeaways

  • No notable deltas vs main (4719e11).

Delta vs main (4719e11)

Area Metric Baseline Current Delta
Package Packed tarball 7.69 MB 7.69 MB -152 B ✅
Package Unpacked publish size 28.80 MB 28.80 MB -50 B ✅
Package Installed footprint 72.23 MB 72.23 MB -50 B ✅
Package Published files 3055 3055 0
Package Installed files 6763 6763 0
Runtime Unique function payloads 2 2 0
Runtime Total function bytes 17.06 MB 17.06 MB -40 B ✅
Runtime Public routes 13 13 0
Changed function payloads vs main (4719e11) (2)
Function Status Baseline Current Delta Route changes
functions/__server.func changed 8.53 MB 8.53 MB -20 B ✅ none
functions/.well-known/workflow/v1/flow.func changed 8.53 MB 8.53 MB -20 B ✅ none

eve init install

Metric Baseline Current Delta
Installed footprint 110.64 MB 110.64 MB -50 B ✅
Installed packages 121 121 0
dependencies 4 4 0
devDependencies 2 2 0
Dependency package bytes 43.28 MB 43.28 MB -50 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-07T17:02:12.338Z
  • Route aliases: 13 public, 1 internal (14 total aliases)
  • Vercel routes in config: 16
  • Severity legend: 🔴 dominant/large, 🟠 notable, 🟡 watch, ⚪ small
Package Drill-Down

Package Details

  • Package: eve@0.31.2
  • Package directory: packages/eve
  • Tarball: 7.69 MB (eve-0.31.2.tgz)
  • Unpacked payload: 28.80 MB across 3055 published files
  • Installed footprint: 72.23 MB across 6763 installed files
  • Installed root package: 27.45 MB
  • Installed dependencies: 44.78 MB
  • Runtime dependencies: 2
  • Peer dependencies: 5 (4 optional)

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

Heavy installed dependencies

  • eve: 27.45 MB (38.0%)
  • @rolldown/binding-linux-x64-gnu: 19.35 MB (26.8%)
  • ai: 6.67 MB (9.2%)
  • zod: 5.07 MB (7.0%)
  • undici: 3.50 MB (4.9%)
Publish payload breakdown
Published file size
🔴 dist/src/compiled/shadcn-registry/index.js       [################........] 9.68 MB 33.6%
🟠 dist/src/compiled/@photon-ai/chat-adapter-ime... [####....................] 2.42 MB 8.4%
🟠 dist/src/compiled/experimental-ai-sdk-code-mo... [###.....................] 1.51 MB 5.2%
🟡 dist/src/compiled/_chunks/workflow/undici-CUI... [#.......................] 508.8 kB 1.8%
🟡 dist/src/compiled/_chunks/node/undici-DWL_MYm... [#.......................] 502.4 kB 1.7%
🔴 Other published files                            [########################] 14.18 MB 49.2%
Installed footprint breakdown
Installed package size
🔴 eve                             [########################] 27.45 MB 38.0%
🔴 @rolldown/binding-linux-x64-gnu [#################.......] 19.35 MB 26.8%
🔴 ai                              [######..................] 6.67 MB 9.2%
🔴 zod                             [####....................] 5.07 MB 7.0%
🟠 undici                          [###.....................] 3.50 MB 4.9%
🟠 nitro                           [##......................] 2.41 MB 3.3%
🔴 Other installed packages        [#######.................] 7.79 MB 10.8%
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: 110.64 MB across 8639 installed files
  • Installed packages: 121 total (115 transitive-only)
  • dependencies: 4 direct packages totaling 43.28 MB
  • devDependencies: 2 direct packages totaling 5.04 MB
  • Other transitive package files: 62.32 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 (25.3%)
  • eve: 27.45 MB (24.8%)
  • @rolldown/binding-linux-x64-gnu: 19.35 MB (17.5%)
  • zod: 9.02 MB (8.2%)
  • ai: 6.67 MB (6.0%)
Installed footprint breakdown
Installed package size
🔴 @typescript/typescript-linux-x64 [########################] 27.95 MB 25.3%
🔴 eve                              [########################] 27.45 MB 24.8%
🔴 @rolldown/binding-linux-x64-gnu  [#################.......] 19.35 MB 17.5%
🔴 zod                              [########................] 9.02 MB 8.2%
🔴 ai                               [######..................] 6.67 MB 6.0%
🟠 undici                           [###.....................] 3.50 MB 3.2%
🔴 Other installed packages         [##############..........] 16.70 MB 15.1%
dependencies (4)
Package Range Installed size Share
@vercel/connect 0.4.3 141.0 kB 0.1%
ai ^7.0.38 6.67 MB 6.0%
eve file:eve-0.31.2.tgz 27.45 MB 24.8%
zod 4.4.3 9.02 MB 8.2%
devDependencies (2)
Package Range Installed size Share
@types/node 24.x 2.54 MB 2.3%
typescript 7.0.2 2.50 MB 2.3%
Function Drill-Down

Payload Size Graph

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

Top Function Payloads

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

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                       [#######################.] 2.33 MB 27.3%
🟠 _chunks/runtime-artifacts.mjs   [################........] 1.63 MB 19.1%
🟡 _libs/undici.mjs                [##########..............] 980.5 kB 11.5%
🟡 _chunks/sandbox.mjs             [########................] 769.0 kB 9.0%
🟡 _libs/@ai-sdk/gateway+[...].mjs [####....................] 432.8 kB 5.1%
🟠 Other bundled files             [########################] 2.38 MB 28.0%

🧾 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 • 12 public routes, 1 internal alias • 8.53 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]/cancel
/eve/v1/session/[sessionId]/clear
/eve/v1/session/[sessionId]/compact
/eve/v1/session/[sessionId]/reset
/eve/v1/session/[sessionId]/stream
Internal aliases /__server
Runtime nodejs24.x
Handler index.mjs
Payload 8.53 MB
Function files 8.53 MB across 44 files
Traced dependencies 0 B
Signal 🟠 Bundled file index.mjs is 2.33 MB (27.3%)

🟠 🔎 Dependency Analysis

📦 Bundled files:

Bundled file size
🟠 index.mjs                       [#######################.] 2.33 MB 27.3%
🟠 _chunks/runtime-artifacts.mjs   [################........] 1.63 MB 19.1%
🟡 _libs/undici.mjs                [##########..............] 980.5 kB 11.5%
🟡 _chunks/sandbox.mjs             [########................] 769.0 kB 9.0%
🟡 _libs/@ai-sdk/gateway+[...].mjs [####....................] 432.8 kB 5.1%
🟠 Other bundled files             [########################] 2.38 MB 28.0%

🧾 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.07 s (-264.5 ms) vs main (4719e11).
  • Timing is informational: shared GitHub runners are too variable for a hard timing budget.
Detailed phase timings vs `main (4719e11)`
Phase Baseline Current Delta
extension.check 9.4 ms 1.1 ms -8.3 ms
project.resolve 2.8 ms 0.7 ms -2.1 ms
workspace.create 1.2 ms 0.7 ms -0.5 ms
host.prepare 191.0 ms 151.3 ms -39.7 ms
vercel.service-prefix.resolve 1.9 ms 2.6 ms +0.7 ms
nitro.create 213.3 ms 244.0 ms +30.7 ms
sandbox.prewarm 524.1 ms 259.5 ms -264.6 ms
nitro.cache.prepare 0.2 ms 0.3 ms +0.1 ms
nitro.prepare 0.8 ms 0.9 ms +0.1 ms
nitro.public-assets 0.7 ms 0.8 ms +0.1 ms
nitro.prerender 0.6 ms 0.6 ms 0.0 ms
nitro.bundle 1.35 s 1.37 s +18.8 ms
nitro.cache.write 0.4 ms 0.4 ms 0.0 ms
vercel.workflow-function.materialize 23.4 ms 23.7 ms +0.3 ms
agent-summary.emit 0.5 ms 0.5 ms 0.0 ms
nitro.close 0.1 ms 0.1 ms 0.0 ms
output.publish 3.5 ms 3.3 ms -0.2 ms
workspace.remove 2.2 ms 2.3 ms +0.1 ms

Signed-off-by: owenkephart <owen.kephart@vercel.com>
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