Repository navigation
Publish as @project516/java-web - #7
Conversation
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Package name and usage references package.json, README.md, docs/browser.md, docs/node.md |
The package metadata and browser and Node examples now use @project516/java-web. |
Estimated code review effort: 2 (Simple) | ~5 minutes
Change: Other
Possibly related PRs
- Project516/java-web#4: Adds the Node package entry and root export that this PR renames in usage references.
- Project516/java-web#99: Adds npm publication support, which uses the package identity changed here.
Suggested reviewers: project516-review-bot
Merge Risk: 
d5a8c
-
No actionable merge-blocking risk is established by this package rename.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly and concisely describes the package rename to @project516/java-web, which is the main change.Description check The description directly explains the package rename, documentation updates, npm availability reason, and unchanged command name. Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)
-
Finishing Touches
-
Simplify code
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
A rabbit checks the package name,
The scoped path now leads the same.
Browser files and Node calls align,
Imports point to the updated sign.
Then hops away, content and fine.
There was a problem hiding this comment.
This PR renames the npm package from java-web to @project516/java-web because the unscoped name is unavailable. It updates package.json and all user-facing documentation (README, docs/browser.md, docs/node.md) with the new scoped name for install commands, static file paths, npx invocations, and import statements. The java-web binary command name is preserved per the package's bin configuration (not shown in the diff).
review-bot, model nvidia/nemotron-3-ultra-550b-a55b:free, verdict approve
The unscoped name
java-webis not available on npm, so the package is now@project516/java-web. This renames it inpackage.jsonand updates the install,npxand import lines in the README and docs.The
java-webcommand name stays the same.npx @project516/java-webruns it because it is the package's only bin.