Skip to content

chore(deps): Remove unused dependency proxyquire#1387

Merged
Tobbe merged 2 commits intomainfrom
tobbe-chore-structure-remove-dep
Mar 17, 2026
Merged

chore(deps): Remove unused dependency proxyquire#1387
Tobbe merged 2 commits intomainfrom
tobbe-chore-structure-remove-dep

Conversation

@Tobbe
Copy link
Member

@Tobbe Tobbe commented Mar 17, 2026

proxyquire was listed as a @cedarjs/structure dependency, but it wasn't actually used

@netlify
Copy link

netlify bot commented Mar 17, 2026

Deploy Preview for cedarjs canceled.

Name Link
🔨 Latest commit 62591f7
🔍 Latest deploy log https://app.netlify.com/projects/cedarjs/deploys/69b96b400592e100072f07fc

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 17, 2026

Greptile Summary

This PR removes proxyquire (v2.1.3) from @cedarjs/structure's production dependencies, as it was not actually used anywhere in the package source. The lock files are regenerated accordingly, with some incidental version bumps to @ast-grep/napi-* packages (0.41.00.41.1) picked up during the lock file refresh.

  • proxyquire is confirmed absent from all source files in packages/structure — the removal is safe and correct.
  • Lock file changes are expected artifacts of running yarn install after updating package.json.
  • The @ast-grep/napi-* patch-version bumps in test-project/yarn.lock are unrelated but harmless minor updates.

Confidence Score: 5/5

  • This PR is safe to merge — it removes a confirmed unused dependency with no functional impact.
  • The only substantive change is deleting a single unused entry from package.json. Grep confirms proxyquire is not imported anywhere in the package. The lock file updates are purely mechanical regeneration artifacts.
  • No files require special attention.

Important Files Changed

Filename Overview
packages/structure/package.json Removes unused proxyquire dependency from the production dependencies list. Confirmed via grep that proxyquire is not imported anywhere in the package source.
test-project/yarn.lock Lock file regenerated after removing proxyquire; also includes incidental version bumps for @ast-grep/napi-* packages from 0.41.0 to 0.41.1.
yarn.lock Root-level lock file updated to reflect removal of proxyquire and associated transitive dependencies.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["packages/structure/package.json"] -->|"remove proxyquire 2.1.3"| B["yarn install"]
    B --> C["yarn.lock updated\n(proxyquire entries removed)"]
    B --> D["test-project/yarn.lock updated\n(proxyquire removed +\n@ast-grep/napi-* 0.41.0→0.41.1)"]
Loading

Last reviewed commit: d624622

@github-actions github-actions bot added this to the chore milestone Mar 17, 2026
@nx-cloud
Copy link

nx-cloud bot commented Mar 17, 2026

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci


View your CI Pipeline Execution ↗ for commit 62591f7

Command Status Duration Result
nx run-many -t build:pack --exclude create-ceda... ✅ Succeeded 2s View ↗
nx run-many -t build ✅ Succeeded 4s View ↗
nx run-many -t test --minWorkers=1 --maxWorkers=4 ✅ Succeeded 2m 47s View ↗
nx run-many -t test:types ✅ Succeeded 9s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-17 15:15:26 UTC

@Tobbe Tobbe merged commit cffa7bc into main Mar 17, 2026
42 checks passed
@Tobbe Tobbe deleted the tobbe-chore-structure-remove-dep branch March 17, 2026 16:16
@github-actions
Copy link

The changes in this PR are now available on npm.

Try them out by running yarn cedar upgrade -t 3.0.0-canary.13596

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