Skip to content

fix: keep the deprecated alias publishing, and drop the leaked tool id - #157

Merged
viztor merged 1 commit into
mainfrom
fix/keep-alias
Nov 4, 2026
Merged

viztor merged 1 commit into
mainfrom
fix/keep-alias

Conversation

@viztor

@viztor viztor commented Nov 4, 2026

Copy link
Copy Markdown
Owner

Two corrections to P2, both about what a name is for.

@example-org/dsh-opencode keeps publishing

P2 retired it (D9). That is the wrong call for a compatibility name: an install still on it stops resolving the day the name stops publishing, which is the one thing an alias must not do.

It stays deprecated — the registry message points at the canonical name, the wrapper ships no client half, and nothing in the source tree registers or advertises it — but it publishes, so the installs that still use it keep working.

Restored:

  • scripts/wrapper-forwarder.ts and its test — the forwarder that links, because the canonical package has no default export (the defect fix(publish): the retired alias must LINK, not fail on every load #135 fixed);
  • the wrapper branch of publish-scoped.ts;
  • the alias in SCOPED_TARGETS;
  • the alias in release.yml's verify list — a name that publishes without being verified is how a release reports success while one of its three packages sits behind.

test/package-checks.test.ts now asserts the opposite of what P2 made it assert: the alias is in both lists, and the publish script names it on purpose.

.freebuff/project-id is actually gone

P1b meant to delete it and the patch emptied the file instead, so main carried a zero-byte tracked file — a tool's local project id, still in the tree. Removed, with .freebuff/ ignored.

Verification

--- @example-org/dsh-opencode-patch@1.9.0 (dry run) ---
--- @example-org/dsh-opencode@1.9.0 (dry run) ---
  "deprecated": "Renamed to @example-org/dsh-opencode-patch. Please install that instead: …"
gate result
unit suite 1377 passed / 89 files
vp check 0 errors
scripts/check.ts passes (the publish list and the verify list agree)

Two corrections to P2, both about what a name is FOR.

**`@viztor/dsh-opencode` keeps publishing.** P2 retired it (D9), and that is the
wrong call for a compatibility name: an install still on it stops resolving the
day the name stops publishing, which is the one thing an alias must not do. It
is deprecated — the registry message points at the canonical name, the wrapper
ships no client half, and nothing in the source tree registers or advertises it —
but it publishes, so the installs that still use it keep working.

Restored: `scripts/wrapper-forwarder.ts` and its test (the forwarder that LINKS,
because the canonical package has no `default` export), the wrapper branch of
`publish-scoped.ts`, the alias in `SCOPED_TARGETS`, and the alias in
`release.yml`'s verify list — a name that publishes without being verified is how
a release reports success while one of its three packages sits behind.

`test/package-checks.test.ts` now asserts the opposite of what P2 made it assert:
the alias is in both lists, and the publish script names it on purpose.

**`.freebuff/project-id` is actually gone.** P1b meant to delete it and the patch
emptied the file instead, so `main` carried a zero-byte tracked file — a tool's
local project id, still in the tree. Removed, with `.freebuff/` ignored.

The READMEs say what is true now: the old name is deprecated and still published,
and moving off it is a rename rather than a fix.
@viztor
viztor merged commit cee95a7 into main Nov 4, 2026
4 checks passed
@viztor
viztor deleted the fix/keep-alias branch November 4, 2026 20:57
@viztor viztor mentioned this pull request Nov 4, 2026
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