Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

The vscode-objectql package contained a package-lock.json while being part of the pnpm workspace (packages/tools/* in pnpm-workspace.yaml), causing dependency review failures due to conflicting package manager metadata.

Changes

  • Removed packages/tools/vscode-objectql/package-lock.json
  • Package now uses root pnpm-lock.yaml exclusively

The dependency review action was scanning both lock files with different resolution algorithms, triggering false positives. All workspace packages must use pnpm for consistent dependency management.

Original prompt

引用: https://github.com/objectstack-ai/objectql/actions/runs/21031854488/job/60469618410#step:3:1

帮我修正有问题的包


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

The vscode-objectql package had its own package-lock.json which conflicted with the pnpm workspace setup. This caused the dependency review workflow to fail.

Fix: Removed packages/tools/vscode-objectql/package-lock.json since the package is part of the pnpm workspace (defined in pnpm-workspace.yaml under packages/tools/*) and should use the root pnpm-lock.yaml instead.

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue with problematic package Remove npm package-lock.json from pnpm workspace package Jan 15, 2026
Copilot AI requested a review from hotlong January 15, 2026 13:12
@hotlong hotlong marked this pull request as ready for review January 15, 2026 13:17
@hotlong hotlong merged commit aea12cd into copilot/design-vscode-plugin Jan 15, 2026
1 check passed
@hotlong hotlong deleted the copilot/fix-problematic-package branch January 15, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants