Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Coalesce Vue starter template and related tooling/docs to use pnpm instead of npm, including CI pipeline changes and minor template/runtime adjustments to avoid phantom dependencies that pnpm would expose.
Changes:
- Switch template CI/build scripts from
npmtopnpm(Azure Pipelines + GitHub Actions) and addpnpmconfiguration in the template app startup. - Add
pnpm-lock.yamland update template/package dependencies (including explicitdate-fns/date-fns-tz) to work withpnpm’s stricter dependency model. - Update documentation and some runtime code paths (e.g., AppInsights trace ID generation,
coalesce-vuere-export usage) to align with the new dependency structure.
Reviewed changes
Copilot reviewed 18 out of 20 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| templates/Coalesce.Vue.Template/content/azure-pipelines.yml | Use pnpm for install/lint/build in Azure Pipelines. |
| templates/Coalesce.Vue.Template/content/.github/workflows/build-test-and-deploy.yml | Use pnpm for install/lint/build in GitHub Actions. |
| templates/Coalesce.Vue.Template/content/Coalesce.Starter.Vue.Web/package.json | Declare pnpm package manager, adjust scripts/deps, and move overrides under pnpm. |
| templates/Coalesce.Vue.Template/content/Coalesce.Starter.Vue.Web/pnpm-lock.yaml | Add lockfile for pnpm installs. |
| templates/Coalesce.Vue.Template/content/Coalesce.Starter.Vue.Web/Program.cs | Configure Vite dev middleware to use pnpm. |
| src/IntelliTect.Coalesce.Vue/DevMiddleware/ViteServerOptions.cs | Add PackageManagerCommand option (default npm). |
| src/IntelliTect.Coalesce.Vue/DevMiddleware/ViteDevelopmentServerMiddleware.cs | Improve stderr-based error reporting for missing package manager. |
| src/coalesce-vue/src/build.ts | Skip npm-specific package version checks when not running under npm. |
| templates/Coalesce.Vue.Template/content/Coalesce.Starter.Vue.Web/src/composables/useAppInsights.ts | Replace AppInsights trace ID generator implementation. |
| templates/Coalesce.Vue.Template/content/.template.config/template.json | Update template exclude patterns (now includes pnpm-lock.yaml). |
| docs/topics/vite-integration.md | Document Vite integration (includes new pnpm-related wording). |
| docs/stacks/vue/getting-started.md | Update getting-started commands to pnpm. |
| templates/Coalesce.Vue.Template/content/AGENTS.md | Update template agent guidance from npm to pnpm. |
| templates/Coalesce.Vue.Template/content/.vscode/settings.json | Update tool auto-approval command to pnpm test. |
| CHANGELOG.md | Document pnpm migration steps and isAxiosError re-export note. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.