Fix prepublishOnly hook by adding prettier formatting check #291
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
prepublishOnly
hook was incomplete compared to the CI quality checks, potentially allowing packages to be published with formatting issues that would later fail CI.Problem
The current
prepublishOnly
script only ran:However, the CI pipeline (
test-ci
) also includes prettier formatting checks:This mismatch could lead to a scenario where:
prepublishOnly
hook passes (tests + linting)Solution
Updated the
prepublishOnly
script to include prettier formatting validation:Now the prepublishOnly hook enforces the same quality standards as CI:
tsd
)jest unit
)eslint
)prettier
)This ensures that only properly formatted, tested, and linted code can be published to npm, maintaining consistency with the project's quality standards.
Fixes #279.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.