-
由 Will Chen 提交于
## Summary - Adds a new step to run `npm test` in the `/dyad:pr-push` skill after lint checks - Ensures tests pass before pushing changes to prevent broken code from being pushed - Updates step numbering and summary section accordingly ## Test plan - Run `/dyad:pr-push` and verify it now runs `npm test` as step 4 - Confirm the skill fails if tests don't pass #skip-bugbot
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2364"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds an npm test step to the /dyad:pr-push skill after linting, blocking pushes when tests fail. Also updates step ordering and the final summary to confirm tests passed. - New Features - Run npm test after lint and before push (required). - Fail the skill if tests fail. - Renumber follow-up steps and update instructions. - Add “tests passed” to the result summary. <sup>Written for commit ba2ef4fb11989841170a9f1e56732d60ae205c29. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com>