Skip to content

fix(core): deprecate camelCase targets aliases#935

Merged
christso merged 4 commits intomainfrom
fix/907-targets-camelcase
Apr 4, 2026
Merged

fix(core): deprecate camelCase targets aliases#935
christso merged 4 commits intomainfrom
fix/907-targets-camelcase

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Apr 4, 2026

Closes #907

Summary

  • warn on deprecated camelCase aliases in targets.yaml validation and target resolution
  • keep existing aliases working while guiding users to snake_case keys
  • avoid duplicate CLI eval warnings by suppressing resolver-side warning emission after validation

Verification

  • bun test packages/core/test/evaluation/providers/targets.test.ts packages/core/test/evaluation/validation/targets-validator.test.ts
  • bunx biome check packages/core/src/evaluation/providers/targets.ts packages/core/src/evaluation/providers/types.ts packages/core/src/evaluation/validation/targets-validator.ts apps/cli/src/commands/eval/targets.ts packages/core/test/evaluation/providers/targets.test.ts packages/core/test/evaluation/validation/targets-validator.test.ts

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 4, 2026

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: dd1d86b
Status: ✅  Deploy successful!
Preview URL: https://03074b57.agentv.pages.dev
Branch Preview URL: https://fix-907-targets-camelcase.agentv.pages.dev

View logs

@christso christso merged commit 4ed7e20 into main Apr 4, 2026
4 checks passed
@christso christso deleted the fix/907-targets-camelcase branch April 4, 2026 22:06
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.

refactor: deprecate camelCase aliases in targets.yaml schema

1 participant