fix(serverless): drop activity and errorRate from apps list --sort - #129
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Those orderings stay in the API enum, but the server returns 422 until traffic metrics exist. The CLI only offers createdAt and name.
The allowlist and help now share one constant, so a rename in the OpenAPI enum fails to compile instead of drifting in the flag text.
1ec1157 to
a9c12c1
Compare
Summary
apps list --sortnow accepts onlycreatedAtandname.activityanderrorRatewere cut from the MVP: the API enum still lists them, but the server returns 422 because those metrics are not collected yet.Test plan
go test ./internal/cmd/serverless/ -run TestParseAppSortrunware serverless apps list --helplistscreatedAtandnameonlyrunware serverless apps list --sort activityfails locally and does not call the API