Remove @azphalt/registry-store-vercel; fix changesets naming a non-workspace package - #238
Conversation
…rkspace package The Neon + Vercel Blob store served the retired Vercel storefront and nothing imports it. Its lockfile entries, docs rows, registry doc comments and pending changeset lines go with it. changeset status also failed on main: two changesets named @azphalt/storefront-cmp, which is outside the pnpm workspace, so changeset version could not run. Dropped those lines. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QwYWVPse8spRrfMMcuTJPv
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Reviewer's guide (collapsed on small PRs)Reviewer's GuideRetires and removes the unused @azphalt/registry-store-vercel workspace package, cleans documentation and registry comments, regenerates the lockfile, and fixes pending changesets so release commands no longer reference deleted or non-workspace packages. Flow diagram for changeset validation after package retirementflowchart LR
Changesets[Pending changesets] --> Status[changeset status]
Status --> Workspace[Workspace package resolution]
Workspace -->|all package names valid| Version[changeset version succeeds]
Workspace -->|deleted or non-workspace package referenced| Failure[Release pipeline fails]
Retired[Remove registry-store-vercel entries] --> Changesets
NonWorkspace[Remove storefront-cmp entries] --> Changesets
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
🚀 Deploying Preview to Cloudflare 🚀Preview Deployments by commit
|
|
Two checks here are red for reasons outside this PR:
The checks that actually test this PR are the Generated by Claude Code |
There was a problem hiding this comment.
Sorry @HereLiesAz, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 22 hours and 47 minutes by commenting @sourcery-ai review. Upgrade to get a review now.
OpenCode security review |
Remove
@azphalt/registry-store-vercelThis was the Neon + Vercel Blob
RegistryStorefrom the retired Vercel storefront. Nothing imports it.packages/registry-store-verceland its lockfile entries (the lockfile was regenerated bypnpm install, not edited by hand).README.mdanddocs/ARCHITECTURE.md.apache-2andlatest-stable-dependencieschangesets. Otherwisechangeset versionwould fail on a package that no longer exists.packages/registrythat pointed at its Postgres stores as the example of a durable store.This doesn't stop the Vercel PR builds. Those come from the Vercel project still being connected to the repo, which is fixed in Vercel's settings, not here.
Fix the release pipeline
changeset statusfails onmaintoday withFound changeset cmp-llm-disclosure for package @azphalt/storefront-cmp which is not in the workspace.storefront-cmpis deliberately outside the pnpm workspace, so any changeset naming it blockschangeset version.cmp-llm-disclosure.mdis deleted. I added it in Store app: disclose llm listings before install #233 and it named only that package.storefront-cmpline is removed fromplay-application-id.md; its worker line stays.Verification
pnpm install --frozen-lockfilepasses.pnpm -r typecheckis clean.@azphalt/registrypasses 126/126 tests.npx changeset statusnow passes.After merging (user-side)
The already published npm versions stay up. To steer people away from them:
🤖 Generated with Claude Code
https://claude.ai/code/session_01QwYWVPse8spRrfMMcuTJPv
Generated by Claude Code
Summary by Sourcery
Remove the retired Vercel registry store and clean up invalid changeset references so the workspace release pipeline remains valid.
Bug Fixes:
Enhancements:
Build:
Documentation: