Skip to content

chore: add the missing changeset for the whoami fix - #54

Closed
spencer-zqian wants to merge 1 commit into
mainfrom
chore/changeset-whoami
Closed

chore: add the missing changeset for the whoami fix#54
spencer-zqian wants to merge 1 commit into
mainfrom
chore/changeset-whoami

Conversation

@spencer-zqian

Copy link
Copy Markdown
Contributor

#53 landed without a changeset, so changeset status saw nothing to bump and the fix would have sat on main indefinitely. Every published version still has sonilo whoami exiting 0 while signed out.

$ npx changeset status
🦋  info Packages to be bumped at minor:
🦋  - sonilo-cli

Why minor and not patch

The exit code changes from 0 to 1 when there is no credential. Anything currently scripted around the old behaviour — sonilo whoami && do-something — will start taking the other branch. That is a breaking change in substance, and at 0.x minor is the slot for it.

The blank-account: half is a plain bug fix and would have been a patch on its own; the two ship together.

Context

This is the release half of sonilo-ai/skills#10, which stopped using sonilo whoami as the transport probe. That change deliberately does not depend on this release — sonilo account is a correct probe on every published version — so nothing is blocked on shipping this. It is worth shipping anyway: a status command that cannot be branched on is a trap for the next person who reaches for it.

🤖 Generated with Claude Code

#53 landed without one, so the release tooling saw nothing to bump and the
fix would have sat on main indefinitely — `sonilo whoami` still exits 0 while
signed out on every published version.

minor, not patch: the exit code goes from 0 to 1 when there is no credential,
which breaks anything currently scripted around the old behaviour. At 0.x
that is the slot for it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@sapient-app

sapient-app Bot commented Aug 14, 2026

Copy link
Copy Markdown

Lightsage docs evals

Waiting for the staging docs URL before running evals.

Lightsage will start the selected PR evals automatically when GitHub reports a successful docs deployment for this PR. This usually happens within 15 minutes.

Commit: adedfbf
Status: waiting for staging docs URL

spencer-zqian added a commit that referenced this pull request Aug 17, 2026
…ed it (#57)

#53 landed on main without a changeset, so `sonilo whoami`'s exit-code
change rode out inside 0.14.0 with nothing in that release's notes. The
version number is right — it went out in a minor bump, which is the
correct slot for a behaviour change at 0.x — only the changelog was
silent.

Written against the existing 0.14.0 heading rather than as a new
changeset, so no phantom 0.15.0 gets published for a change that is
already in users' hands. Anyone whose script broke on `whoami` returning
1 can now find the version that did it.

Supersedes #54, which proposed the changeset and would have dated this
to the next release instead.
@spencer-zqian

Copy link
Copy Markdown
Contributor Author

Closing in favour of #57.

The fix this changeset describes already shippedsonilo-cli@0.14.0 contains it. #53 landed on main without a changeset, but changesets versions whatever code is on main regardless of which changesets exist, so the whoami behaviour rode out with the video-analysis release. Verified two ways: git merge-base --is-ancestor 8ada33d sonilo-cli@0.14.0 passes, and the published 0.14.0 tarball's runWhoami already returns false (exit 1) when there is no credential.

The version number was right by accident — it went out in a minor bump, the correct slot for a behaviour change at 0.x. Only the changelog was silent.

Merging this now would bump sonilo-cli to 0.15.0 and date the note to that release, telling anyone whose script broke on whoami returning 1 to look at a version they don't have. #57 instead records it under the existing ## 0.14.0 heading — the version that actually contains it — and publishes no phantom release. The GitHub Release notes for sonilo-cli@0.14.0 have been updated to match.

@spencer-zqian
spencer-zqian deleted the chore/changeset-whoami branch August 17, 2026 04:52
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