Skip to content

Fix CI: Sync pnpm lockfile and package versions#287

Merged
hotlong merged 3 commits into
mainfrom
copilot/fix-ci-build-and-test
Feb 1, 2026
Merged

Fix CI: Sync pnpm lockfile and package versions#287
hotlong merged 3 commits into
mainfrom
copilot/fix-ci-build-and-test

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 31, 2026

CI was failing at pnpm install --frozen-lockfile due to lockfile drift. The @objectql/driver-tck package added typescript@^5.3.0 as a devDependency but the lockfile was not updated.

Changes

  • pnpm-lock.yaml: Regenerated to include typescript@^5.3.0 dependency
  • @objectql/driver-redis: Version corrected from 4.1.04.0.2 (monorepo sync)
  • GraphQL protocol test: Fixed version assertion 0.1.00.2.0 (actual plugin version)

All packages now synchronized at version 4.0.2 per monorepo versioning scheme.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • fonts.googleapis.com
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/objectql/objectql/apps/site/node_modules/.bin/../next/dist/bin/next build (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/objectql/objectql/apps/site/node_modules/.bin/../next/dist/bin/next build sh -c tsc (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

引用: https://github.com/objectstack-ai/objectql/actions/runs/21548910376/job/62094588983#step:7:1

fix ci build and test


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

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 31, 2026

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

Project Deployment Actions Updated (UTC)
objectql Ready Ready Preview, Comment Jan 31, 2026 6:59pm

Request Review

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI build and test issues Fix CI: Sync pnpm lockfile and package versions Jan 31, 2026
Copilot AI requested a review from hotlong January 31, 2026 18:53
@hotlong hotlong marked this pull request as ready for review February 1, 2026 02:32
Copilot AI review requested due to automatic review settings February 1, 2026 02:32
@hotlong hotlong merged commit 872f5f9 into main Feb 1, 2026
3 checks passed
@hotlong hotlong deleted the copilot/fix-ci-build-and-test branch February 1, 2026 02:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes CI build failures caused by lockfile drift and version inconsistencies in the monorepo. The primary issue was that pnpm install --frozen-lockfile was failing because the lockfile didn't include a TypeScript dependency that was added to @objectql/driver-tck.

Changes:

  • Regenerated pnpm-lock.yaml to include the missing typescript@^5.3.0 dependency for @objectql/driver-tck
  • Corrected @objectql/driver-redis package version from 4.1.0 to 4.0.2 to align with monorepo versioning
  • Fixed GraphQL protocol test assertion to expect version 0.2.0 instead of 0.1.0, matching the actual plugin version

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
pnpm-lock.yaml Added missing TypeScript dependency entry for @objectql/driver-tck package
packages/drivers/redis/package.json Corrected version from 4.1.0 to 4.0.2 for monorepo synchronization
packages/protocols/graphql/src/index.test.ts Updated version assertion from 0.1.0 to 0.2.0 to match actual plugin version
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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.

3 participants