docs(releasing): record the Open VSX publish, and the half of it that is still open - #7
Merged
Merged
Conversation
… is still open RELEASING.md ends with "Whichever way this is resolved, record it here", and it was already stale in exactly the way it warns about. It said Open VSX served 0.1.0 as of 2026-08-23; the registry had 0.3.1 as `latest`, published 2026-08-23T21:48Z — after that check, by someone who did not come back to the document. 0.3.2 is now published and `latest` resolves to it, verified against the API rather than from the CLI's own output. Three things that cost time and are written down so they cost it once: - `ovsx publish` prints "Published" BEFORE the version exists. The version endpoint 404'd for ~100 seconds while the registry indexed. Poll the API; the success line is not the observation. - The published bytes are ours byte-for-byte. Downloaded back and compared: sha256 94bb2036… matches the v0.3.2 tag message AND promptster-cli's embedded artifact. The registry does not repackage, so reproducibility holds all the way to what a registry install pulls — which is the claim the whole tag-and- checksum scheme exists to make and which nobody had ever checked end to end. - `ovsx` writes the PAT clear-text to ~/.ovsx when it cannot open the OS credential store. Deleted. And `VAR=x cmd -p "$VAR"` silently does not work — the expansion happens before the assignment, so ovsx falls through to an interactive prompt. What did NOT get resolved is stated as such: 0.1.0 is still installable by pin, and I re-verified it against the live registry today rather than repeating the earlier note — it hooks onDidChangeWindowState and emits editor_focus, and dist/utils/ holds no command redactor. Publishing over it does not retract it; removal needs the registry's admins and that request has not been made. Also removes the `lint` script. eslint is not a devDependency, there is no eslint config in the repo, and ci.yml never invoked it, so `pnpm run lint` could only ever fail with "command not found". A script naming a tool the repo does not install is worse than no script: it reads as "we lint" and does not. Adding a linter for real is its own change with its own diff. Gates: tsc -p ./ clean, vitest 107/107 across 7 files. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Greptile SummaryThe PR updates the release record with the verified Open VSX 0.3.2 publication while preserving the outstanding removal task for privacy-incompatible 0.1.0.
Confidence Score: 5/5The PR appears safe to merge with no actionable defects identified in the documentation or package-script change. The release instructions align with the pinned Open VSX dependency and documented artifact flow, while removing the lint script does not break any repository workflow or automation.
|
| Filename | Overview |
|---|---|
| RELEASING.md | Accurately records the Open VSX publication outcome, verification steps, credential caveat, and unresolved legacy-version removal. |
| package.json | Removes an unused lint script whose executable and configuration are not provided and which repository CI does not invoke. |
Reviews (1): Last reviewed commit: "docs(releasing): record the Open VSX pub..." | Re-trigger Greptile
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.
RELEASING.mdends with "Whichever way this is resolved, record it here" — and it was already stale in exactly the way it warns about.The doc was wrong before I touched anything
It said Open VSX served 0.1.0 as of 2026-08-23. The registry actually had 0.3.1 as
latest, published 2026-08-23T21:48Z — after that check, by someone who didn't come back to the document. So the privacy liability onlatesthad already been closed and nobody knew.Done: 0.3.2 published
latestnow resolves to 0.3.2, verified against the API rather than from the CLI's own output. Three things that cost time, written down so they cost it once:ovsx publishprintsPublishedbefore the version exists. The version endpoint 404'd for ~100 seconds while the registry indexed. Poll the API; the success line is not the observation.sha256 94bb2036…matches thev0.3.2tag message and promptster-cli's embedded artifact. The registry does not repackage — so the reproducibility chain holds all the way to what a registry install pulls, which is the claim the whole tag-and-checksum scheme exists to make and which nobody had ever checked end to end.ovsxwrites the PAT clear-text to~/.ovsxwhen it cannot open the OS credential store, in one easy-to-miss line. Deleted. Also:VAR=x cmd -p "$VAR"silently doesn't work — the expansion happens before the assignment, soovsxfalls through to an interactive prompt.Not done, and stated as such
0.1.0is still installable by pin. Re-verified against the live registry today rather than carried over from the earlier note:onDidChangeWindowStateand emitseditor_focusfromdist/collectors/focus.jsdist/utils/holds onlyeditorDetector,logger,pathSanitizer, anddist/collectors/terminal.jshas zero occurrences ofredactPublishing over it did not retract it and could not. Removal goes through the registry's admins; that request has not been made. Recorded as open.
What publishing did buy: anything resolving
latest— which is what adevcontainer.jsoncustomizations.vscode.extensionsentry does — now gets a build that honours the promise. That matters the moment a hosted lane ships, since it installs by registry id, not from a local file.Also: the
lintscript is removedeslintis not a devDependency, there is no eslint config in the repo, andci.ymlnever invoked it —pnpm run lintcould only ever fail withcommand not found. A script naming a tool the repo doesn't install is worse than no script: it reads as "we lint" and doesn't. Adding a linter for real is its own change.Gates
tsc -p ./✅ ·vitest107/107 across 7 files ✅🤖 Generated with Claude Code