Skip to content

Fix demo setup#113

Merged
gilbarbara merged 1 commit into
mainfrom
fixes
Feb 22, 2026
Merged

Fix demo setup#113
gilbarbara merged 1 commit into
mainfrom
fixes

Conversation

@gilbarbara
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings February 22, 2026 05:58
@codesandbox
Copy link
Copy Markdown

codesandbox Bot commented Feb 22, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link
Copy Markdown

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 fixes the demo setup by making the development server port configurable and updating dependencies. The changes improve flexibility for different deployment environments where the default port 3000 might not be available.

Changes:

  • Made the Vite dev server port configurable via PORT environment variable
  • Updated pnpm package manager from version 10.29.3 to 10.30.1
  • Removed CodeSandbox-specific configuration file

Reviewed changes

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

File Description
demo/vite.config.ts Added support for PORT environment variable to configure dev server port
demo/package.json Updated packageManager field to pnpm@10.30.1
demo/.codesandbox/tasks.json Removed CodeSandbox-specific tasks configuration
Comments suppressed due to low confidence (1)

demo/vite.config.ts:17

  • The current implementation doesn't handle the case where process.env.PORT contains an invalid value. If PORT is set to a non-numeric string, parseInt will return NaN, which could cause the dev server to fail to start. Consider using a fallback pattern like: const port = parseInt(process.env.PORT || '3000', 10) or add validation to handle NaN: const port = parseInt(process.env.PORT, 10); port: isNaN(port) ? 3000 : port.
    port: process.env.PORT ? parseInt(process.env.PORT, 10) : 3000,

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

@sonarqubecloud
Copy link
Copy Markdown

@gilbarbara gilbarbara merged commit cf9c796 into main Feb 22, 2026
5 checks passed
@gilbarbara gilbarbara deleted the fixes branch February 22, 2026 06:04
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