Skip to content

docs(ghost-drift): surface Node 18+ requirement in package README#43

Merged
nahiyankhan merged 1 commit intomainfrom
docs/ghost-drift-readme-requirements
Apr 21, 2026
Merged

docs(ghost-drift): surface Node 18+ requirement in package README#43
nahiyankhan merged 1 commit intomainfrom
docs/ghost-drift-readme-requirements

Conversation

@nahiyankhan
Copy link
Copy Markdown
Collaborator

Summary

  • Adds a small Requirements section to packages/ghost-drift/README.md surfacing the Node 18+ minimum (already enforced by engines.node, just not previously discoverable on the npm page).
  • Ships a patch changeset for ghost-drift that will bump 0.1.00.1.1 on the next Version Packages PR.

This PR also functions as a CI publish smoke test. Path:

  1. Bootstrap ghost-drift@0.1.0 to npm locally first (one-time, from off-VPN machine: npm login && pnpm --filter ghost-drift publish --access public --provenance=false).
  2. Merge this PR.
  3. Changesets workflow opens a new Version Packages PR bumping to 0.1.1.
  4. Merge that PR.
  5. CI publish runs with the now-scopeable NPM_PUBLISH_TOKEN — if the token permissions are sorted, 0.1.1 lands on npm cleanly. If not, we see the failure mode with no cost to shipping real features.

Test plan

  • Local ghost-drift@0.1.0 bootstrap publish completed off-VPN
  • Merge this PR
  • Merge the resulting Version Packages PR
  • Confirm npm view ghost-drift version reports 0.1.1
  • Confirm a GitHub Release appears for ghost-drift@0.1.1

🤖 Generated with Claude Code

The engines.node constraint is only discoverable by opening package.json. Adds a small Requirements section near the top of the README so the npm listing page communicates the minimum Node version at a glance.

Also serves as the first post-bootstrap CI publish smoke test: doubles as a patch-level changeset, so merging → auto-opened Version Packages PR → merging that triggers ghost-drift@0.1.1 publish from CI. If the NPM token scope fix worked, this is the run that proves it.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nahiyankhan nahiyankhan merged commit 5f8038b into main Apr 21, 2026
6 checks passed
@github-actions github-actions Bot mentioned this pull request Apr 21, 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