Skip to content

chore(scripts): align npm scripts and docs with Vue stack#3138

Merged
PierreBrisorgueil merged 1 commit into
masterfrom
chore/npm-scripts-align
Feb 21, 2026
Merged

chore(scripts): align npm scripts and docs with Vue stack#3138
PierreBrisorgueil merged 1 commit into
masterfrom
chore/npm-scripts-align

Conversation

@PierreBrisorgueil

Copy link
Copy Markdown
Contributor

Summary

Aligns Node stack npm scripts and documentation with the Vue stack for cross-stack consistency.

Script changes (package.json)

Script Before After
dev missing npm run start (alias)
test:unit missing alias of npm test (one-shot)
format missing prettier --write . (prettier was already installed)

Documentation changes

  • CLAUDE.md: added dev, test:unit, format, prod, explicit release strategy (release = standard-version, release:auto = semantic-release)
  • README.md: added npm run dev alias, test:unit, npm run format in Code Quality section
  • copilot-instructions.md: added dev, test:unit, format, release:auto

Scope

  • Modules impacted: none (scripts/docs only)
  • Cross-module impact: none
  • Risk level: low

Validation

  • npm run lint passes
  • npm test passes

- Add `dev` alias for `start` (cross-stack consistency)
- Add `test:unit` alias for `test` (one-shot, mirrors Vue naming)
- Add `format` script (prettier already installed, script was missing)
- Document `prod`, `format`, `test:unit`, `release` strategy in CLAUDE.md
- Update README and copilot-instructions to reflect all scripts
Copilot AI review requested due to automatic review settings February 21, 2026 11:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aligns the Node stack's npm scripts and documentation with the Vue stack to establish cross-stack consistency for developer experience. The changes add standard script aliases (dev, test:unit, format) and update all documentation files to reflect these additions.

Changes:

  • Added three npm script aliases in package.json: dev (alias for start), test:unit (alias for test), and format (runs Prettier)
  • Updated CLAUDE.md, README.md, and copilot-instructions.md to document the new scripts and clarify release strategies

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
package.json Added dev, test:unit, and format script aliases for cross-stack consistency
README.md Updated documentation to include new script aliases in development, testing, and code quality sections
CLAUDE.md Enhanced canonical commands table with new aliases and clarified release strategy (manual vs CI)
.github/copilot-instructions.md Updated canonical commands list to include new aliases and release:auto for CI

@codecov

codecov Bot commented Feb 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.49%. Comparing base (cf3a66d) to head (931ecf5).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3138   +/-   ##
=======================================
  Coverage   89.49%   89.49%           
=======================================
  Files          52       52           
  Lines        1133     1133           
  Branches      217      217           
=======================================
  Hits         1014     1014           
  Misses        107      107           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@PierreBrisorgueil PierreBrisorgueil merged commit 14f653d into master Feb 21, 2026
8 checks 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.

2 participants