release: v0.0.12#36
Conversation
Ships: optional project-create name (paste-runnable one-liner, #34), next-action Next: hints (#32), insta compute start/stop/suspend/status lifecycle commands, agents.instacloud.com canonical docs (#31). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GMbAe5K1RfwaAcge5inX7P
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
jwfing
left a comment
There was a problem hiding this comment.
Summary
Release PR bumping package.json version 0.0.11 → 0.0.12; the bump itself is correct, but v0.0.12 has already been merged to main and already tagged on the remote, making this PR a stale duplicate that should not be merged.
Requirements context
No /docs/superpowers/ (or docs/specs/) directory exists in this repo, so there is no matching spec/plan — assessed against the PR description and the version-release process conventions. This PR matches the established release-PR pattern exactly: prior releases (e.g. v0.0.11 32d5166) are also single-line package.json version bumps with no CHANGELOG, so the change shape is correct.
Findings
Critical
Functionality — v0.0.12 is already released; this PR is a redundant/stale duplicate (package.json:3)
- The remote already carries
refs/tags/v0.0.12, pointing at commitd445b2f(release: v0.0.12 (#36)), which is the currentmainHEAD and whosepackage.jsonis already at0.0.12. - This PR's branch head
4880bdashares parent7507d32withd445b2fbut is not an ancestor ofmain. Its diff (0.0.11 → 0.0.12) is byte-for-byte identical to what already landed. Merging it against amainthat is already at 0.0.12 is a no-op (or a same-line conflict) — it adds nothing. - More importantly, re-running the release for v0.0.12 is a hard STOP under the project's release rules: an existing
v<version>tag means the version must be bumped before releasing again (npm publishes and git tags are not safely re-writable). Proceeding here would either fail the idempotency check or, worse, attempt to overwrite a shipped release. - Recommended action: close this PR rather than merge it. If a new release is intended, open a fresh PR that bumps to the next unused version (0.0.13) from current
main.
Suggestion
(none)
Information
- Software engineering — no test changes required. A version-only bump needs no tests; the release is exercised by
.github/workflows/release.yml(binaries via Bun +npm publish --provenance). Nothing to add here. - Lockfile version is decoupled (pre-existing, not introduced by this PR).
package-lock.jsonrecords"version": "0.0.0"at the root whilepackage.jsonis0.0.12. This is harmless for the pipeline —npm cionly enforces dependency sync (not the root version), andnpm publishreads the version frompackage.json— and every prior release shipped with the same0.0.0lock. Worth aligning to the real version at some point for hygiene, but out of scope for a release PR. - Security / Performance — no relevant changes. The diff is a single version string; no user input, secrets, dependencies, queries, or hot-path code are touched. npm publish uses OIDC trusted publishing (no token secret in the diff).
Verdict
request_changes — the version bump is correct in isolation, but v0.0.12 is already on main and already tagged v0.0.12 on the remote. This PR is a stale duplicate; it should be closed rather than merged, and any further release should target the next unused version.
Ships to npm + binaries via the OIDC pipeline: optional project-create name (#34), next-action hints (#32), insta compute lifecycle commands, agents.instacloud.com docs (#31).
🤖 Generated with Claude Code
https://claude.ai/code/session_01GMbAe5K1RfwaAcge5inX7P
Summary by cubic
Publishes v0.0.12 of the
instaCLI to npm and releases binaries via the OIDC pipeline. Adds next-step guidance, compute lifecycle commands, and smoother project creation with updated docs.project-createname with a pasteable one-liner (fix: insta project create name optional (recommended one-liner was unrunnable) #34)insta compute start|stop|suspend|statusWritten for commit 4880bda. Summary will update on new commits.