-
Notifications
You must be signed in to change notification settings - Fork 129
Commit 74b5790
fix: remove unused dependencies from
The `globby` package was listed as a direct dependency in
`devDependencies` but is not actually used anywhere in the codebase.
This dependency is only needed as a transitive dependency through
existing packages.
## Analysis
A comprehensive search of the codebase revealed:
- No direct imports or usage of `globby` in any source files
- `globby` is available as a transitive dependency through:
- `@docusaurus/core` → `@docusaurus/bundler` → `copy-webpack-plugin` →
`[email protected]`
- `@docusaurus/utils` → `[email protected]`
## Changes
- Removed `"globby": "^14.0.0"` from `devDependencies` in `package.json`
- Updated `package-lock.json` to reflect the removal of the extraneous
package
## Testing
All functionality remains intact after the removal:
- ✅ ESLint and code linting work correctly
- ✅ Markdown linting works correctly
- ✅ API documentation build (`redoc:test`) succeeds
- ✅ Docusaurus core functionality is unaffected
- ✅ Transitive dependencies continue to provide needed functionality
This change follows best practices by only listing packages as direct
dependencies when they are actually imported or used directly in the
project code.
> [!WARNING]
>
> <details>
> <summary>Firewall rules blocked me from connecting to one or more
addresses (expand for details)</summary>
>
> #### I tried to connect to the following addresses, but was blocked by
firewall rules:
>
> - `otel.cloud.redocly.com`
> - Triggering command: `node
/home/REDACTED/work/apify-docs/apify-docs/node_modules/.bin/redocly
lint` (dns block)
> - Triggering command: `node
/home/REDACTED/work/apify-docs/apify-docs/node_modules/.bin/redocly
bundle apify-api/openapi/openapi.yaml -o apify-api` (dns block)
>
> If you need me to access, download, or install something from one of
these locations, you can either:
>
> - Configure [Actions setup
steps](https://gh.io/copilot/actions-setup-steps) to set up my
environment, which run before the firewall is enabled
> - Add the appropriate URLs or hosts to the custom allowlist in this
repository's [Copilot coding agent
settings](https://github.com/apify/apify-docs/settings/copilot/coding_agent)
(admins only)
>
> </details>
<!-- START COPILOT CODING AGENT SUFFIX -->
<details>
<summary>Original prompt</summary>
> remove dependency on globby from package.json, it is not used
directly, only as part of our dependencies
</details>
<!-- START COPILOT CODING AGENT TIPS -->
---
💡 You can make Copilot smarter by setting up custom instructions,
customizing its development environment and configuring Model Context
Protocol (MCP) servers. Learn more [Copilot coding agent
tips](https://gh.io/copilot-coding-agent-tips) in the docs.
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> Removes several unused dependencies/devDependencies from
`package.json` and updates `package-lock.json` accordingly.
>
> - **Dependencies cleanup**:
> - Remove unused `@docusaurus/utils`, `@mdx-js/react`, `ajv`,
`form-data`, `proxy-from-env`, `search-insights`.
> - **DevDependencies cleanup**:
> - Remove unused `@rsbuild/plugin-styled-components`,
`eslint-plugin-json`, `eslint-plugin-markdown`,
`eslint-plugin-react-hooks`, `fs-extra`, `globby`.
> - **Lockfile**:
> - Prunes corresponding entries from `package-lock.json`.
>
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
68bd577. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
---------
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: B4nan <[email protected]>
Co-authored-by: Jindřich Bär <[email protected]>package.json
(#1953)1 parent 5ee5a14 commit 74b5790Copy full SHA for 74b5790
File tree
Expand file treeCollapse file tree
2 files changed
+0
-357
lines changedFilter options
Expand file treeCollapse file tree
2 files changed
+0
-357
lines changed
0 commit comments