Skip to content

Conversation

@IvanGoncharov
Copy link
Member

No description provided.

@github-actions
Copy link

github-actions bot commented Jun 11, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

  • example/express-server/package.json
  • package-lock.json
  • package.json

Copy link

Copilot AI left a comment

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 removes the ts-node dependency and replaces all script invocations of ts-node and npm run with direct node commands.

  • Swapped ts-node for node in project scripts and CI workflows
  • Removed ts-node entries from dependencies
  • Updated example READMEs to reflect new invocation style

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
playwright.config.ts Updated webServer command to use node --run serve
package.json Replaced ts-node calls in changelog and serve
example/webpack/README.md Updated build command to node --run build:release
example/express-server/package.json Swapped ts-node start script for node index.ts
.github/workflows/ci.yml Changed all npm run steps to node --run commands
.github/workflows/changelog.yml Updated changelog generation to use node --run
Comments suppressed due to low confidence (3)

.github/workflows/ci.yml:25

  • The --run flag is not recognized by Node.js, and testonly is an npm script. Use npm run testonly or execute a JS file directly.
run: node --run testonly

.github/workflows/changelog.yml:29

  • Node does not support a --run flag for npm scripts. Change to npm run changelog > changelog.txt or invoke a compiled changelog script.
run: node --run changelog > changelog.txt

playwright.config.ts:61

  • The --run flag is not a valid Node.js option. If you intend to invoke the serve script, use npm run serve, or if running a TS file directly, compile it to JS or add a proper loader (e.g., --loader ts-node/esm).
command: 'node --run serve',

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants