fix(tooling): revert corepack setup — pnpm is pre-installed in devcontainer image#122
Conversation
…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>
…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>
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s GitHub Copilot instructions, aiming to document the project setup and (per the PR title/description) remove unnecessary Corepack-related bootstrap steps.
Changes:
- Expanded
.github/copilot-instructions.mdwith project overview, environment details, common commands, CI expectations, repo layout, and architecture notes. - Removed the older “Project Context” block from the MCP tool usage section (replaced by the new overview content).
| ## Project Overview | ||
|
|
||
| This is **TechWatching.dev**, Alexandre Nédélec's personal blog at https://techwatching.dev. It is a statically generated site built with: | ||
|
|
There was a problem hiding this comment.
The PR description says this file change only removes a corepack enable step from the bootstrap section, but this diff also adds substantial new repository documentation (project overview, commands, layout, architecture notes). Please either (1) update the PR description/title to reflect the broader scope, or (2) split these documentation additions into a separate PR so the tooling revert can be reviewed/merged independently.
| - **Nuxt 4** — Vue.js meta-framework (compatibility date `2024-07-11`) | ||
| - **Nuxt UI v3** (`@nuxt/ui`) — UI component library based on TailwindCSS v4 | ||
| - **Nuxt Content v3** (`@nuxt/content`) — Markdown/YAML content management with SQLite | ||
| - **TailwindCSS v4** — Utility-first CSS |
There was a problem hiding this comment.
This document says the project uses “Nuxt UI v3”, but package.json currently depends on @nuxt/ui ^4.4.0. Please update the Nuxt UI version references (including the “Nuxt UI v3 Styling Guidelines” section title) to match the actual dependency to avoid confusing contributors.
The previous session introduced
corepack enable && corepack installas the devcontaineronCreateCommand, butmcr.microsoft.com/devcontainers/typescript-nodealready ships pnpm via the Node.js devcontainer feature — corepack is unnecessary.Changes
.devcontainer/devcontainer.json— restoresonCreateCommandto the originalpnpm self-update.github/copilot-instructions.md— removes thecorepack enablestep from the bootstrap section; correct bootstrap is justpnpm installOriginal prompt
🔒 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.