Skip to content

chore: remove deprecated @ngaf/partial-json workspace#326

Merged
blove merged 1 commit into
mainfrom
codex/remove-ngaf-partial-json
May 15, 2026
Merged

chore: remove deprecated @ngaf/partial-json workspace#326
blove merged 1 commit into
mainfrom
codex/remove-ngaf-partial-json

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 15, 2026

Summary

  • Remove the deprecated @ngaf/partial-json workspace package and Nx release/build references.
  • Point remaining live parser imports and docs to @cacheplane/partial-json.
  • Refresh the lockfile so npm no longer links node_modules/@ngaf/partial-json.

Validation

  • PATH="/Users/blove/.nvm/versions/node/v22.14.0/bin:$PATH" npm ci --ignore-scripts
  • PATH="/Users/blove/.nvm/versions/node/v22.14.0/bin:$PATH" NX_DAEMON=false npx nx run-many -t build --projects=chat,render,website,cockpit-render-computed-functions-angular,cockpit-render-element-rendering-angular,cockpit-render-state-management-angular,cockpit-render-spec-rendering-angular,cockpit-render-repeat-loops-angular,cockpit-render-registry-angular
  • PATH="/Users/blove/.nvm/versions/node/v22.14.0/bin:$PATH" NX_DAEMON=false npx nx test chat
  • rg -n "@ngaf/partial-json|libs/partial-json" --glob '!docs/superpowers/**' --glob '!node_modules/**'
  • git diff --check
  • node -e "const fs=require('fs'); JSON.parse(fs.readFileSync('package-lock.json','utf8')); console.log('package-lock json ok')"

Notes: build output still reports pre-existing warnings for cockpit bundle budgets, Next metadataBase, and NO_COLOR/FORCE_COLOR; npm still warns that @copilotkit/aimock@1.23.0 asks for Node >=24 while the local runner uses Node 22.14.0.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 15, 2026

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

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 15, 2026 6:18pm

Request Review

@blove blove marked this pull request as ready for review May 15, 2026 18:25
@blove blove merged commit 45bbf7d into main May 15, 2026
16 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.

1 participant