Skip to content

Remove unfinished rollback command#33

Merged
joerucci merged 1 commit into
mainfrom
release-readiness
Jul 15, 2026
Merged

Remove unfinished rollback command#33
joerucci merged 1 commit into
mainfrom
release-readiness

Conversation

@joerucci

Copy link
Copy Markdown
Contributor

Summary

  • Remove the hidden ghostable var rollback stub that only returned a not-implemented error.
  • Add regression coverage confirming rollback is outside the supported variable command surface.
  • Keep repository-state recovery delegated to Git instead of implying a partially implemented CLI feature.

Why

The rollback branch was the only production-facing placeholder found during the release-readiness audit. It was absent from help and documentation but still accepted as a special command, which made the CLI appear partially implemented.

Impact

There is no change to the documented command surface. Unsupported rollback requests now receive the same unknown-command behavior as other unsupported variable operations. Repository-backed state can be recovered through Git; externally deployed values still require redeploying the reverted state.

Validation

  • make check
  • make security-check
  • go test -race ./...
  • Clean local build and diff checks

@joerucci
joerucci marked this pull request as ready for review July 15, 2026 18:23
@joerucci
joerucci merged commit 5a8c491 into main Jul 15, 2026
9 checks passed
@joerucci
joerucci deleted the release-readiness branch July 15, 2026 18:23
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