chore(release): @commonlyai/cli 0.1.10 — seven source commits shipped to nobody - #1017
Merged
Merged
Conversation
npm 0.1.9 and main 0.1.9 are DIFFERENT CODE. Seven source commits have landed since the last bump, four of them real fixes: #995 a quota stall was classified as a runtime blip and probed every 5s #973 a directly-addressed seat can answer past the cascade cap #1002 a seat log recorded four failures and not one time #1006 stamp the line that dates the restart #961 pin an agent's model in the environment spec Anyone who installs the CLI today gets none of them. Our own seats have them only because their CLI is npm-linked into a worktree that was pulled by hand. Same defect as #979 one package over: a version that maps to two different artifacts defeats the only check available from outside the repo.
The published version is the only check available from OUTSIDE this repo. Ship source without bumping it and that check silently passes while the artifact and the repo disagree. Twice now: #979 @commonlyai/mcp — npm 0.3.0 and main 0.3.0 were different code; the PR-tool removal reached the repo and reached zero seats. Found months late, by unpacking the tarball. #1017 @commonlyai/cli — npm 0.1.9 and main 0.1.9 were different code, seven source commits deep, including #995 (a quota stall classified as a runtime blip and probed every 5s). Found the same day, by hand, only because someone thought to check content rather than version. Guards cli/src and commonly-mcp/src. Docs and tests are exempt — they do not require a release. Verified before landing: YAML parses, bash -n clean, and the comparison was dry-run against real history — 7 commits touched cli/src since its last bump, every one of which this would have failed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
npm 0.1.9 and main 0.1.9 are different code
Same defect as #979, one package over. Seven source commits have landed since the last version bump, four of them real fixes:
Anyone who installs the CLI today gets none of them. Our own seats have them only because their CLI is npm-linked into a worktree that was pulled by hand — which also means the fleet's behaviour and the published artifact have silently diverged.
#995 is the one I'd care about most: without it,
"You've hit your session limit"classifies as a generic runtime blip and the seat retries every 5s against a provider that will not answer for hours, instead of opening the circuit at the 15-minute ceiling. That ran all of last night.How this keeps happening
A published version is the only check available from outside the repo. Ship source without bumping it and that check silently passes while the artifact and the repo disagree — and there is no way to tell from the outside, which is exactly how #979 survived for months.
Verifying by version is what fails. Verifying by content is what works:
The guard, and why it is not in this PR
I wrote
.github/workflows/package-version-guard.yml— fails a PR whencli/src/**orcommonly-mcp/src/**changes without a version bump, exempting docs and tests. Validated before proposing: YAML parses,bash -nclean, and dry-run against real history shows it would have caught all 7 of the commits above.I could not push it. Neither account carries the
workflowscope (lilyshen0722:repo, gist, read:org, admin:public_key;samxu01:repo, gist, read:org), and the attempt fails as a bareHTTP 404that reads like a missing file. The global instructions claimed samxu01 held a workflow-scoped token; that has been corrected.To land it:
gh auth refresh -h github.com -s workflow, then it is a one-call push. Alternatively I can convert it toscripts/verify-package-versions.js+ averify:package-versionsnpm script — matching the existingverify:moltbot-toolspattern — leaving only a two-linerun:step intests.ymlfor someone with the scope.After merge
If those greps return
0, the publish went out of a stale tree and the version is lying again.