client: refactor executor to use Spec + Invocation (PRINFRA-121)

[somanshreddy]

Description

Migrates the HTTP executor from the old RequestSpec (one struct mixing static metadata and runtime values) to the new two-type design: command.Spec for what the command is, command.Invocation for what the user provided.

The executor now receives both and combines them: it reads the endpoint template and HTTP method from Spec, substitutes path params and encodes query params from Invocation, and marshals the body only if Invocation.Body is non-nil (preventing accidental {} on bodyless endpoints).

Deletes the old RequestSpec, QueryParam, and FieldSpec types entirely. The hand-written video list command is updated to use the new types as a migration proof — it will be replaced by the generated version in Stack 2.

Testing

All existing executor tests updated to new signature. Added tests for nil body (no content sent) and missing Method (returns error). All M0 command tests pass. TestVideoList_LimitOutOfRange expectations updated — limit validation is now server-side since the OpenAPI spec defines min/max on the generated version.

Files

• internal/client/executor.go — refactored to Execute(*Spec, *Invocation)
• internal/client/executor_test.go — updated to new signature + nil body + method-required tests
• internal/client/request_spec.go — deleted (types moved to internal/command)
• cmd/heygen/video_list.go — migrated to new types
• cmd/heygen/video_list_test.go — limit validation test updated to expect API error

[somanshreddy]

• client: add multipart form-data support for file uploads (PRINFRA-121) #6 : 2 dependent PRs (#7 , #11 )
• cmd: add generic command builder (PRINFRA-121) #10
• client: refactor executor to use Spec + Invocation (PRINFRA-121) #9 👈 (View in Graphite)
• add command.Spec and command.Invocation types (PRINFRA-121) #5
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[linear]

PRINFRA-121 CLI M1: OpenAPI codegen pipeline

Build the code generation pipeline that reads openapi/external-api.json and produces CommandSpec structs.

Scope

• Go program in codegen/ using text/template
• Parse OpenAPI spec, filter to v3 endpoints only
• Generate CommandSpec data structs in gen/ (zero logic)
• overrides.yaml for:
  • Positional arg promotion (prompt, text, file)
  • Command group naming (video-translations → translate, webhooks/endpoints → webhook)
  • Help examples per command
• make generate SPEC=... target
• Nested path collapsing (e.g., /webhooks/endpoints/{id}/rotate-secret → webhook rotate-secret)
• Golden file tests for codegen output stability
• Generate all ~30 commands, wire into Cobra at startup

Mapping Rules

• OpenAPI tag → command group
• POST → create, GET (list) → list, GET (by id) → get, DELETE → delete, PATCH → update
• Path params → positional args, query params → flags, body fields → flags
• overrides.yaml exceptions for positional promotion and group naming

Acceptance Criteria

• make generate produces compilable CommandSpec files for all v3 endpoints
• Adding a new endpoint to spec + re-running codegen produces new command
• Generated help text includes descriptions and examples
• Golden file tests pass
• Override validation: make generate fails if overrides reference non-existent endpoints

[somanshreddy]

Merge activity

• Mar 31, 10:38 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Mar 31, 10:39 PM UTC: Graphite rebased this pull request as part of a merge.
• Mar 31, 10:41 PM UTC: @somanshreddy merged this pull request with Graphite.