Skip to content

fix: include kdm binary wrapper#20

Merged
utkarsh232005 merged 1 commit into
KDM-cli:mainfrom
fizyxbt:fix-global-kdm-bin
May 14, 2026
Merged

fix: include kdm binary wrapper#20
utkarsh232005 merged 1 commit into
KDM-cli:mainfrom
fizyxbt:fix-global-kdm-bin

Conversation

@fizyxbt
Copy link
Copy Markdown
Contributor

@fizyxbt fizyxbt commented May 14, 2026

Summary

  • add the missing bin/kdm.js executable wrapper for the existing package.json bin entry
  • keep the wrapper in the published package by allowing the bin directory to be committed

Fixes #19.

Checks

  • npm ci
  • npm run build
  • npm test
  • packed and installed the tarball with a temporary npm prefix
  • verified kdm --version runs from the installed package

@fizyxbt fizyxbt requested a review from utkarsh232005 as a code owner May 14, 2026 10:53
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 14, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • .gitignore is excluded by none and included by none
  • bin/kdm.js is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 39e43b9d-638a-48c6-b59a-46ac833a3dcc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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

Copy link
Copy Markdown

@Rishiraj-Pathak-27 Rishiraj-Pathak-27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall! Clean and focused fix for including the CLI wrapper. Just make sure the bin mapping and executable permissions are properly set before merge. LGTM 👍

@utkarsh232005 utkarsh232005 merged commit dd8c742 into KDM-cli:main May 14, 2026
2 checks passed
@utkarsh232005 utkarsh232005 added this to the v1.2.0 milestone May 14, 2026
@utkarsh232005
Copy link
Copy Markdown
Member

@fizyxbt Thank you for your contribution and support to the project. Your efforts are truly appreciated. Looking forward to more valuable contributions from your side in the future 🚀

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.

Bug: kdm Command Not Found After Global Installation

3 participants