Skip to content

🔥 Remove production smoke harness#313

Merged
Robdel12 merged 1 commit into
mainfrom
rd/remove-prod-smoke
Jul 22, 2026
Merged

🔥 Remove production smoke harness#313
Robdel12 merged 1 commit into
mainfrom
rd/remove-prod-smoke

Conversation

@Robdel12

Copy link
Copy Markdown
Contributor

Why

The CLI included a release smoke script that copied a real user login, linked a live production project, created a scoped token, and queried production builds. Even with cleanup logic, release verification should not mutate customer-facing infrastructure or depend on personal production credentials.

What changed

  • Removed the production smoke harness entirely.
  • Removed its package script entry.

Impact

Local builds, tests, type checks, and package verification continue to cover the publishable CLI without contacting production. There is no replacement live-system workflow.

Confidence

The complete CLI test suite and local release checks pass, the package still builds correctly, and no production-smoke references remain in the repository.

Delete the release script that copied real user authentication, created a production project token, and queried live builds. Release verification should remain local and non-mutating.
@Robdel12
Robdel12 marked this pull request as ready for review July 22, 2026 02:27
@Robdel12
Robdel12 merged commit 99213c2 into main Jul 22, 2026
27 of 30 checks passed
@Robdel12
Robdel12 deleted the rd/remove-prod-smoke branch July 22, 2026 02:27
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