Repository navigation
fix: build before running tests - #9517
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Other documented test entry points still fail on clean checkouts when dist/ is absent.
Review effort: Balanced
Findings: 1
Open (1)
What changed in this PR
Builds TypeScript output before the default Jest suite to satisfy tests requiring dist/.
Changes:
- Updates
npm testto run the build first.
| File | Description |
|---|---|
package.json |
Adds the build prerequisite to the default test script. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
|
✅ Copilot review passed with no inline comments. @lpcox Add the |
|
@copilot address review feedback |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Updated the Jest entry points so each path builds |

Summary
npm test.dist/.Validation
npm test— passed (435 suites, 7,405 tests; 22 skipped).