Repository navigation
Rename npm package to @intelip/writset - #64
Conversation
Entire-Checkpoint: ed379b0f87aa
GitHub Actions
Failure
GitHub Actions
Active
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f9831ff065
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| function checkVersions(text, name, version, errors) { | ||
| for (const line of text.split(/\r?\n/)) { | ||
| const installation = /\bnpm\s+(?:install|i|add)\b/.test(line) ? line.match(/@intelip\/tabellio(?:@([^\s`"']+))?/) : null; | ||
| const installation = /\bnpm\s+(?:install|i|add)\b/.test(line) ? line.match(/@intelip\/(?:writset|tabellio)(?:@([^\s`"']+))?/) : null; |
There was a problem hiding this comment.
When an onboarding file contains npm install @intelip/tabellio@0.7.1, this regex accepts it because it validates only the shared version suffix, even though 0.7.1 is being released exclusively as @intelip/writset and the old package remains at 0.7.0. Consequently npm run docs:check can certify an installation command that cannot install the intended release; distinguish the current WritSet package from historical Tabellio references rather than treating both names interchangeably.
AGENTS.md reference: AGENTS.md:L3-L6
Useful? React with 👍 / 👎.
Summary
Prepare
@intelip/writset@0.7.1so users can install the product under its WritSet name.writset-*commands while retaining everytabellio-*command as an alias to the same implementation.The existing
@intelip/tabellio@0.7.0publication remains available. The new package has not been published, and no old package is deprecated or removed.Verification
Candidate:
f9831ff065943e095651def2fedeeedbdf0e8d35Base:
c916d4ea1e02474b00182513d14e0a7e92277b1aPassed locally:
npm run check.npm run quality:changed-code,npm pack --dry-run, syntax checks, andgit diff --check.writset-versionandtabellio-versionreport the new identity without attributing the consumer repository's package.writset-provenance-demoreturnsstatus: passedandcleanup: passed. External-service observations remain synthetic.GitHub Tests, Fallow changed-code, and Package dry-run passed on this candidate. Hosted product validation, attempt 2 also passed, including exact-candidate evidence loading, validation, safe summary publication, and runner-file cleanup.
After the run, both temporary environment secrets and the task-owned branch policy were removed, the prior environment settings were restored, and the five local transport files were deleted. Readback confirmed zero environment secrets. Native checkpoints remain local.
Review notes
Review the package-identity compatibility boundary and migration instructions. No review threads or requested changes were open at the final review check.
Merge is user-approved. The landed main commit requires its own approved checkpoint handoff. Release tagging and npm publication remain separate actions.