Skip to content

Keep dist/cli.mjs a plain file - #5

Merged
silveltman merged 1 commit into
mainfrom
plain-dist-file
Oct 6, 2026
Merged

silveltman merged 1 commit into
mainfrom
plain-dist-file

Conversation

@silveltman

Copy link
Copy Markdown
Contributor

The release workflow commits through the GitHub API, which refuses executable files ("Unexpected executable file at dist/cli.mjs"). npm makes a package's bin executable when it installs it, so the build now leaves the file plain.

🤖 Generated with Claude Code

The release workflow commits through the GitHub API, which refuses
executable files. npm marks the bin executable when it installs, so the
bit is not needed in Git.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown

Warning

Review limit reached

The included review limit has been reached and this organization has disabled usage-based review continuation. Wait for reviews to reset or ask a billing admin to change After included review limits.

  • Ask an admin to enable usage-based reviews

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Next included review available in 6 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 5 included reviews currently available. Your 19 included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 3e46cecf-ea9d-420b-9b6f-8a68001a45a3
📥 Commits

Reviewing files that changed from the base of the PR and between 7b3ae62 and 72990c6.

⛔ Files ignored due to path filters (1)
  • dist/cli.mjs is excluded by !**/dist/**
📒 Files selected for processing (1)
  • package.json
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@silveltman
silveltman merged commit a6f726f into main Oct 6, 2026
3 checks passed
@silveltman
silveltman deleted the plain-dist-file branch October 6, 2026 18:30
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