Skip to content

fix(cli): reject partial numeric flag values#54

Open
EfeDurmaz16 wants to merge 1 commit into
vercel-labs:mainfrom
EfeDurmaz16:fix/strict-numeric-flag-parsing
Open

fix(cli): reject partial numeric flag values#54
EfeDurmaz16 wants to merge 1 commit into
vercel-labs:mainfrom
EfeDurmaz16:fix/strict-numeric-flag-parsing

Conversation

@EfeDurmaz16
Copy link
Copy Markdown

Summary

  • validate decimal flags against the full input before converting
  • reject partial numeric values like 1abc or 1.5s for temperature/duration-style flags
  • add parser coverage for trailing non-numeric characters

Tests

  • bun test packages/ai-cli/src/lib/parse.test.ts
  • bun test
  • bun run typecheck
  • bun run format:check
  • bun run lint (passes with existing warnings in unrelated files)
  • bun run build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 15, 2026

@EfeDurmaz16 is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

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