Skip to content

Conversation

@coyotte508
Copy link
Member

@coyotte508 coyotte508 commented Aug 25, 2025

Helpful to provide an image when you want to run the codebase (including installling deps & extensions) inside a dev container, to avoid the codebase + extensions to impact the rest of your system.

When opening the project, VSCode & Cursor will now prompt you to reopen the project in a dev container.

There is also no need anymore to "trust" the codebase through VSCode's dialog

Copy link
Contributor

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 adds dev-container support to the project by introducing a devcontainer configuration that allows developers to run the codebase in an isolated containerized environment. This helps avoid system-wide impacts from dependencies and provides a consistent development environment.

  • Adds a devcontainer configuration using a TypeScript/Node.js base image
  • Includes pnpm store mounting feature for efficient package management
  • Provides commented configuration options for ports, post-create commands, and customizations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


// Features to add to the dev container. More info: https://containers.dev/features.
"features": {
"ghcr.io/joshuanianji/devcontainer-features/mount-pnpm-store:1": {}
Copy link
Member Author

Choose a reason for hiding this comment

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

allows sharing pnpm deps between multiple dev containers

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

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

Worked for me - VS Code opened the project inside a container, I could pnpm stuff in a terminal, and got auto-linting on save (with the Prettier extension I suppose, which was installed inside the container).

@coyotte508
Copy link
Member Author

yea we can edit also extensions added by the container if we want to

@coyotte508 coyotte508 merged commit 127767f into main Aug 29, 2025
5 checks passed
@coyotte508 coyotte508 deleted the dev-container branch August 29, 2025 10:08
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.

3 participants