Skip to content

Remove legacy -p request parameter fallback - #3

Merged
jasonHzq merged 1 commit into
mainfrom
codex/fix-unified-cli-codegen
Aug 14, 2026
Merged

jasonHzq merged 1 commit into
mainfrom
codex/fix-unified-cli-codegen

Conversation

@jasonHzq

Copy link
Copy Markdown
Contributor

What changed

  • remove the legacy -p/--param key=value compatibility option from the standalone Hub CLI
  • parse named Endpoint options correctly for both controller and flat API collections
  • reject -p with a migration message
  • update the public README and regression tests

Why

Named OpenAPI parameters are the canonical unified CLI contract. The fallback was no longer needed and also masked the flat-command parser bug.

Validation

  • 12 tests pass
  • TypeScript typecheck and build pass
  • npm package contents verified
  • live Frankfurter v1/v2 read calls returned HTTP 200
  • Dida365 and Massive preview-only paths passed

@jasonHzq
jasonHzq marked this pull request as ready for review August 14, 2026 07:34
@jasonHzq
jasonHzq merged commit e19b6ca into main Aug 14, 2026
1 check passed
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