Skip to content

fix(tooling): revert corepack setup — pnpm is pre-installed in devcontainer image#122

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/set-up-copilot-instructions
Draft

fix(tooling): revert corepack setup — pnpm is pre-installed in devcontainer image#122
Copilot wants to merge 3 commits intomainfrom
copilot/set-up-copilot-instructions

Conversation

Copy link
Contributor

Copilot AI commented Mar 18, 2026

The previous session introduced corepack enable && corepack install as the devcontainer onCreateCommand, but mcr.microsoft.com/devcontainers/typescript-node already ships pnpm via the Node.js devcontainer feature — corepack is unnecessary.

Changes

  • .devcontainer/devcontainer.json — restores onCreateCommand to the original pnpm self-update
  • .github/copilot-instructions.md — removes the corepack enable step from the bootstrap section; correct bootstrap is just pnpm install
Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

<agent_instructions>Adapt it to my environment : I am using pnpm as my package manager. The project is using Nuxt 4</agent_instructions>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…uxt 4

Add development environment bootstrap, pnpm commands table, CI
validation steps, and project layout to copilot-instructions.md.
Remove redundant Project Context section (covered by Project Overview).
Update devcontainer to use corepack for reliable pnpm setup.

Co-authored-by: TechWatching <15186176+TechWatching@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Copilot setup instructions for repository docs(tooling): configure Copilot coding agent instructions for pnpm/Nuxt 4 Mar 18, 2026
Copilot AI requested a review from TechWatching March 18, 2026 10:47
…image

The mcr.microsoft.com/devcontainers/typescript-node image ships pnpm
via the Node.js devcontainer feature. No corepack setup is needed.
Restore the original onCreateCommand to pnpm self-update and remove
the corepack step from copilot-instructions.md bootstrap section.

Co-authored-by: TechWatching <15186176+TechWatching@users.noreply.github.com>
Copilot AI changed the title docs(tooling): configure Copilot coding agent instructions for pnpm/Nuxt 4 fix(tooling): revert corepack setup — pnpm is pre-installed in devcontainer image Mar 18, 2026
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.

✨ Set up Copilot instructions

2 participants