Skip to content

Stabilize setup and status JSON output#20

Merged
joerucci merged 1 commit into
mainfrom
fix/stable-json-output
Jul 2, 2026
Merged

Stabilize setup and status JSON output#20
joerucci merged 1 commit into
mainfrom
fix/stable-json-output

Conversation

@joerucci

@joerucci joerucci commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds explicit JSON response DTOs for setup --json and status --json instead of exposing raw domain structs.
  • Uses stable lower-camel project fields like schema, id, packageManager, scanIgnores, and environments.
  • Sanitizes status device and access request payloads so compatibility/signature fields such as device_id and client_sig are not exposed through status JSON.
  • Adds tests that reject the old raw ProjectManifest field shape and device signature aliases.

Why

setup --json and status --json are developer-facing machine-readable surfaces. Exposing internal struct names and signed-record compatibility fields made the output look unstable and implementation-driven.

Validation

  • go test ./internal/app
  • make check
  • Rebuilt CLI and smoke-tested setup --json and status --json output shape locally.

Replaces closed PR #17 after renaming the branch to remove the previous branch prefix.

@joerucci
joerucci marked this pull request as ready for review July 2, 2026 16:21
@joerucci
joerucci merged commit cef3581 into main Jul 2, 2026
2 checks passed
@joerucci
joerucci deleted the fix/stable-json-output branch July 2, 2026 16:21
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