Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 19, 2026

Fixes the CI pipeline failure caused by a mismatch between pnpm-lock.yaml and packages/types/package.json.

Problem

The CI was failing with:

ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with packages/types/package.json

specifiers in the lockfile don't match specifiers in package.json:
* @objectstack/spec (lockfile: workspace:*, manifest: ^0.1.1)

The lockfile expected @objectstack/spec to be a workspace package, but package.json specified it as an external npm dependency (^0.1.1). Additionally, the npm package @objectstack/spec@0.1.1 doesn't export the UIComponent interface that was being imported.

Changes Made

  • Removed dependency: Removed @objectstack/spec from packages/types/package.json
  • Reverted BaseSchema: Restored packages/types/src/base.ts to its original definition where BaseSchema defines type and id properties directly instead of extending UIComponent
  • Updated lockfile: Synchronized pnpm-lock.yaml to reflect the removed dependency

Testing

  • ✅ Frozen lockfile installation works correctly
  • ✅ TypeScript compilation passes
  • @object-ui/types package builds successfully
  • ✅ All dependencies resolve without errors

The changes restore the types package to its original state, removing the incorrect dependency on an external package that was causing the CI failure.

Original prompt

引用: https://github.com/objectstack-ai/objectui/actions/runs/21141019742/job/60794817921#step:7:1


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@vercel
Copy link

vercel bot commented Jan 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
objectui-docs Error Error Jan 19, 2026 2:57pm

@hotlong
Copy link
Contributor

hotlong commented Jan 19, 2026

@copilot 继续

The pnpm-lock.yaml was out of sync with packages/types/package.json.
The @objectstack/spec npm package (v0.1.1) doesn't export UIComponent interface,
so reverting to the original BaseSchema definition without external dependencies.

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 19, 2026

@copilot 继续

Fixed the CI error in commit e423a5f. The issue was that pnpm-lock.yaml referenced @objectstack/spec as a workspace dependency, but the npm package doesn't export the UIComponent interface. I removed the dependency and reverted BaseSchema to its original definition. The frozen lockfile installation now works correctly.

Copilot AI changed the title [WIP] 引用: https://github.com/objectstack-ai/objectui/actions/runs/21141019742/job/60794817921#step:7:1 Fix CI lockfile error by removing @objectstack/spec dependency Jan 19, 2026
Copilot AI requested a review from hotlong January 19, 2026 14:58
@huangyiirene huangyiirene marked this pull request as ready for review January 19, 2026 15:01
@github-actions
Copy link

✅ All checks passed!

  • ✅ Type check passed
  • ✅ Tests passed
  • ✅ Lint check completed

@huangyiirene huangyiirene merged commit 21ca482 into copilot/update-types-data-display Jan 19, 2026
3 of 4 checks passed
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.

3 participants