Skip to content

fix: resolve npm bin symlinks - #6

Merged
fqjony merged 2 commits into
mainfrom
release/v0.1.4
Aug 10, 2026
Merged

fix: resolve npm bin symlinks#6
fqjony merged 2 commits into
mainfrom
release/v0.1.4

Conversation

@fqjony

@fqjony fqjony commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • resolve the npm global-bin symlink before starting the Node implementation
  • release the fix as @udx/rabbit-repo 0.1.4

Validation

  • make test
  • npm pack --dry-run --json

Copilot AI lite review requested due to automatic review settings August 10, 2026 09:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes execution when rabbit.ci is invoked through an npm-installed symlinked global bin entry by resolving the symlink chain to compute the correct script directory before launching the Node implementation. It also publishes the fix as @udx/rabbit-repo@0.1.4.

Changes:

  • Resolve bin/rabbit.ci’s real location by following symlinks before computing SCRIPT_DIR.
  • Extend tests/suite.sh to validate running the CLI via a symlinked “global bin” path.
  • Bump package version to 0.1.4 (and lockfile version fields).

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
bin/rabbit.ci Adds symlink-resolution loop to ensure rabbit.ci.js is located correctly when invoked via npm global-bin symlinks.
tests/suite.sh Adds coverage that runs the CLI through a symlink to emulate npm global-bin behavior.
package.json Bumps package version to 0.1.4.
package-lock.json Updates lockfile version fields to match 0.1.4.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@fqjony
fqjony enabled auto-merge August 10, 2026 09:20
@fqjony
fqjony requested review from a team and udx-github August 10, 2026 09:21
@fqjony fqjony assigned fqjony and unassigned fqjony Aug 10, 2026
@fqjony
fqjony merged commit afaafa4 into main Aug 10, 2026
6 checks passed
@fqjony
fqjony deleted the release/v0.1.4 branch August 10, 2026 09:21
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.

3 participants