Skip to content

fix(cli): accept kebab-case workspace flags#529

Draft
luokerenx4 wants to merge 1 commit into
devfrom
fix/workspace-cli-kebab-flags
Draft

fix(cli): accept kebab-case workspace flags#529
luokerenx4 wants to merge 1 commit into
devfrom
fix/workspace-cli-kebab-flags

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

What

  • render manifest-derived CLI help in conventional kebab-case
  • map kebab-case flags back to their exact camelCase JSON-schema properties before invoking a tool
  • keep existing camelCase flags compatible
  • preserve strict unknown-flag validation by normalizing only names that map to a declared property

Why

The real cross-session workflow exposed a sharp edge: --resume-id, --task-id, and --workspace-id are the natural shell spellings, but the shared shim previously sent those names literally and the gateway rejected them. Fixing this once in the shared shim covers every current and future CLI command.

Verification

  • pnpm vitest run src/workspaces/cli/shim.spec.ts (6 passed)
  • pnpm test (2530 passed, 6 skipped)
  • npx tsc --noEmit
  • pnpm build
  • live manifest help rendered existing camelCase fields as --bar-id, --entry-date, --exit-rule, and related shell-style flags
  • git diff --check

Draft by design: parallel contribution mode, awaiting review; do not merge automatically.

@vercel

vercel Bot commented Jul 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview, Comment Jul 11, 2026 10:36am

Request Review

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