Skip to content

fix: npm pkg warning#49

Merged
weaponsforge merged 3 commits intodevfrom
fix/npm-pkg-bin
Feb 25, 2026
Merged

fix: npm pkg warning#49
weaponsforge merged 3 commits intodevfrom
fix/npm-pkg-bin

Conversation

@weaponsforge
Copy link
Copy Markdown
Owner

@weaponsforge weaponsforge commented Feb 25, 2026

  • Fix: npm pkg warning in the bin field
  • Docs: note CLI usage via npx - no installation required
  • Chore: add an NPM start script

Summary by CodeRabbit

  • Documentation

    • Added instructions for running via npx without installation (Node.js LTS v24.11.0+ required) and how to view CLI help.
    • Minor wording tweak in the quickstart guide.
  • New Features

    • Added an npm start alias to run the CLI as a convenient shorthand.
  • Chores

    • Bumped package version to 1.2.3.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bc722e0 and 4dd5b11.

📒 Files selected for processing (2)
  • README.md
  • app/package.json
🚧 Files skipped from review as they are similar to previous changes (2)
  • README.md
  • app/package.json

📝 Walkthrough

Walkthrough

Documentation and package.json updates to add npx-based CLI usage and a new npm start script, plus a version bump from 1.2.2 to 1.2.3 and a minor bin path normalization in app/package.json. No runtime code changes.

Changes

Cohort / File(s) Summary
Documentation & CLI Guidance
README.md, docs/README_NPM.md
Added instructions for running via npx @weaponsforge/sendemail`` (Node.js LTS v24.11.0+), added --help example, and minor quickstart wording tweak.
Package Configuration
app/package.json
Bumped version 1.2.2 → 1.2.3, removed leading ./ from bin.sendemail path (dist/scripts/cli/send.js), and added start script: node ./dist/scripts/cli/send.js.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 I hopped in docs and bumped a patch,
npx ready now — no install to catch.
A tiny path tweak, version set free,
Run --help and nibble a CLI tea. 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: npm pkg warning' directly addresses the main change—fixing an npm package warning by correcting the bin field path format in package.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/npm-pkg-bin

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Test Results

29 tests  ±0   29 ✅ ±0   19s ⏱️ -3s
 6 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 4dd5b11. ± Comparison against base commit e27d177.

@weaponsforge weaponsforge merged commit 3a28732 into dev Feb 25, 2026
4 checks passed
@weaponsforge weaponsforge deleted the fix/npm-pkg-bin branch February 25, 2026 15:59
@weaponsforge weaponsforge mentioned this pull request Feb 25, 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