Skip to content

Sync server lockfile with nested esbuild deps required by deterministic CI - #64

Merged
jamalfrnk merged 1 commit into
mainfrom
copilot/fix-with-copilot
Aug 3, 2026
Merged

Sync server lockfile with nested esbuild deps required by deterministic CI#64
jamalfrnk merged 1 commit into
mainfrom
copilot/fix-with-copilot

Conversation

Copilot AI commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Quality Gate failed in server because npm ci detected server/package-lock.json out of sync with resolved dependencies, specifically missing esbuild@0.28.1 platform package entries required by the vitest -> vite dependency graph.

  • Lockfile consistency fix (server/package-lock.json)

    • Added missing nested vitest/node_modules/esbuild resolution and its @esbuild/*@0.28.1 platform packages.
    • Keeps dependency installation deterministic under CI (npm ci) without changing application/runtime code.
  • Scope

    • Lockfile-only update in server; no source, config, or behavior changes.
{
  "packages": {
    "node_modules/vitest/node_modules/esbuild": {
      "version": "0.28.1",
      "optionalDependencies": {
        "@esbuild/linux-x64": "0.28.1",
        "@esbuild/darwin-arm64": "0.28.1"
      }
    }
  }
}

Copilot AI changed the title [WIP] Fix issues identified by Copilot in workflow Sync server lockfile with nested esbuild deps required by deterministic CI Aug 3, 2026
Copilot AI requested a review from jamalfrnk August 3, 2026 08:11
@jamalfrnk
jamalfrnk marked this pull request as ready for review August 3, 2026 08:16
Base automatically changed from feat/paper-realism to main August 3, 2026 17:40
@jamalfrnk
jamalfrnk force-pushed the copilot/fix-with-copilot branch from 6eac510 to 48a0b00 Compare August 3, 2026 17:40
@jamalfrnk
jamalfrnk merged commit 69ae8fc into main Aug 3, 2026
3 checks passed
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.

2 participants