Skip to content

test: reject missing declared bin targets - #36

Merged
rogerchappel merged 1 commit into
mainfrom
agent/oss-ca5e33-missing-bin-target
Nov 13, 2026
Merged

rogerchappel merged 1 commit into
mainfrom
agent/oss-ca5e33-missing-bin-target

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • Add regression coverage ensuring the package-content checker rejects a declared bin target absent from the npm archive.
  • The check verifies the exact missing path and failing exit status.

Verification

  • node --test test/package-content.test.js
  • npm test (27 tests)
  • npm run build
  • npm run smoke
  • npm run package:smoke
  • git diff --check
  • Manual review completed

Risk Level

  • Low

Notes:

One commit: test: reject missing declared bin targets. Only test/package-content.test.js changed. CI workflow triggers only validation on pull requests; no release, deploy, or package publishing workflow is triggered.

Rollback Plan

  • Revert this test-only commit.

Human Decision Needed

  • Maintainer review

@rogerchappel
rogerchappel marked this pull request as ready for review November 13, 2026 02:35
@rogerchappel
rogerchappel merged commit 23e50b5 into main Nov 13, 2026
1 check passed
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