Skip to content

refactor(pipeline): simplify invoke.json to two paths#807

Merged
christso merged 1 commit intomainfrom
fix/797-simplify-invoke-json
Mar 28, 2026
Merged

refactor(pipeline): simplify invoke.json to two paths#807
christso merged 1 commit intomainfrom
fix/797-simplify-invoke-json

Conversation

@christso
Copy link
Copy Markdown
Collaborator

Summary

  • Remove dead third invoke.json path that wrote kind: "<provider>" with subagent_mode_allowed: false — no consumer read this format
  • manifest.json already carries target.subagent_mode_allowed so the bench skill decides execution mode from there

Follow-up to #804 based on code review feedback.

Test plan

  • All 1610 tests pass
  • Typecheck + lint clean

🤖 Generated with Claude Code

Remove the third invoke.json path that wrote kind: "<provider>" with
subagent_mode_allowed: false. No consumer read this format, making it
dead code. The manifest already carries subagent_mode_allowed so the
bench skill can decide whether to use executor subagents or agentv eval.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: a428929
Status: ✅  Deploy successful!
Preview URL: https://00ff1e76.agentv.pages.dev
Branch Preview URL: https://fix-797-simplify-invoke-json.agentv.pages.dev

View logs

@christso christso merged commit 61c9318 into main Mar 28, 2026
2 checks passed
@christso christso deleted the fix/797-simplify-invoke-json branch March 28, 2026 08:23
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