feat: change the scope to @webship-js/diffy-steps (#3) - #4
Merged
Conversation
"name": "@webship/diffy-steps", -> "name": "@webship-js/diffy-steps", @webship is the GitHub organisation covering the whole Webship product family - the Drupal distribution, the web* modules, the theme, the patch metapackages - almost none of which is JavaScript or belongs on npm. This package is a plugin for the webship-js harness and is useful with nothing else, so the scope names what it plugs into. It also reads better at the point of use: a cucumber.js require: list shows webship-js core steps on one line and @webship-js/diffy-steps on the next, and the relationship is self-evident. version stays 2.0.0; publishConfig.access stays public. Every reference carrying the scope follows - install commands, the node_modules/@webship-js/diffy-steps/... require paths, the copy and uninstall snippets, the doc headings, the step-file header - and package-lock.json is regenerated. Nothing is published yet, so this costs no deprecation, redirect, or consumer migration. The window closes at first publish. Verified: npm pack --dry-run reports @webship-js/diffy-steps@2.0.0, and the suite still passes 6 scenarios / 37 steps against the mock Diffy API. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
Closes #3
@webshipis the GitHub organisation for the whole Webship product family — theDrupal distribution, the 14
web*modules, the theme, the patch metapackages,the project templates. Almost none of that is JavaScript and none of it is on
npm. This package is a plugin for the webship-js harness and is useful with
nothing else, so the scope now names what it plugs into.
It also reads better where it matters — a consumer's
cucumber.js:versionstays2.0.0,publishConfig.accessstayspublic.Changed
package.jsonnameREADME.mdrequire:pathdocs/README.mdtests/step-definitions/README.mdrequire:path,cp,npm uninstalltests/step-definitions/diffy.steps.jspackage-lock.jsonTiming
Nothing is published, so this is free — no deprecation, no redirect, no consumer
migration. That window closes at first publish, which is the argument for
settling the scope now rather than after.
Namespace caveat — worth knowing before creating the org
An npm package named
webship-jsalready exists (the harness). As of thisPR there is no org or reserved scope of that name —
registry.npmjs.org/-/org/webship-jsreturns 404 and
@webship-js/diffy-stepsis unclaimed — but whether npm permitsan org whose name collides with an existing package name is only settled when
you try to create it. I could not verify that ahead of time.
If npm rejects
webship-jsas an org name, the fallbacks are to revert to@webshipor pick a third scope; either way #3 should record the outcome.Neither publish blocker from before has moved:
npm whoamistill reportsENEEDAUTH, and no org exists yet.Test notes
npm pack --dry-run→@webship-js/diffy-steps@2.0.0, filenamewebship-js-diffy-steps-2.0.0.tgz, same 5-file payload.@webship/references remain anywhere in the tree.Knock-on outside this repo: webship-js
CLAUDE.md§2.6 on2.0.xnames the@webship/...require path and needs the same correction — separate issue andPR so this one stays single-purpose.
AI-Generated: Yes (applied the rescope and updated every reference; reviewed by
Rajab Natshah.)
Checkpoints
Automated unit/functional testing coverage