Skip to content

Conversation

@dkattan
Copy link
Owner

@dkattan dkattan commented Jan 17, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 17, 2026 14:36
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 makes npm scripts Windows-compatible by replacing inline shell commands with cross-platform alternatives using the cross-env package. It also includes test stability improvements for CI environments.

Changes:

  • Added cross-env package to enable cross-platform environment variable setting in npm scripts
  • Extracted inline cleanup logic to a dedicated CJS script for Windows compatibility
  • Increased test timeouts and added error handling for diagnostic tests that may be flaky under load

Reviewed changes

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

File Description
package.json Updated npm scripts to use cross-env for setting NODE_OPTIONS; added cross-env to devDependencies
package-lock.json Added cross-env package; downgraded @playwright/test version; marked vscode-test-playwright as extraneous
scripts/clean-vscode-test-user-data.cjs New cross-platform script to clean VS Code test user data directory before running tests
src/test/debugUtils.test.ts Increased test timeouts; added timeoutSeconds parameter; added error handling for diagnostic test

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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