From 474b748a51947a44b83bb0aa07bc7a0f309aaf42 Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Wed, 15 Jan 2025 13:16:59 +0100 Subject: [PATCH 1/8] Improve knip reports by using a single config with workspaces Also move the dependency from the workspace into repo-tools. The run with a configured workspace is slower, so I had to bump the timeout. Signed-off-by: Axel Hecht --- .changeset/cool-pants-boil.md | 5 ++ package.json | 1 - packages/repo-tools/package.json | 1 + .../commands/knip-reports/knip-extractor.ts | 69 ++++++++++--------- packages/repo-tools/src/commands/util.ts | 3 +- yarn.lock | 10 +-- 6 files changed, 51 insertions(+), 38 deletions(-) create mode 100644 .changeset/cool-pants-boil.md diff --git a/.changeset/cool-pants-boil.md b/.changeset/cool-pants-boil.md new file mode 100644 index 0000000000..792fb57195 --- /dev/null +++ b/.changeset/cool-pants-boil.md @@ -0,0 +1,5 @@ +--- +'@backstage/repo-tools': patch +--- + +Improve knip reports by using a single workspace-based config diff --git a/package.json b/package.json index 41b8529c70..0595e00b68 100644 --- a/package.json +++ b/package.json @@ -126,7 +126,6 @@ "eslint-plugin-testing-library": "^6.0.0", "fs-extra": "^11.2.0", "husky": "^9.0.0", - "knip": "^5.0.0", "lint-staged": "^15.0.0", "minimist": "^1.2.5", "node-gyp": "^10.0.0", diff --git a/packages/repo-tools/package.json b/packages/repo-tools/package.json index f5c6a0c060..4ccad33712 100644 --- a/packages/repo-tools/package.json +++ b/packages/repo-tools/package.json @@ -74,6 +74,7 @@ "just-diff": "^6.0.2", "knex": "^3.0.0", "knex-pglite": "^0.11.0", + "knip": "^5.42.0", "lodash": "^4.17.21", "minimatch": "^9.0.0", "p-limit": "^3.0.2", diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index 50c429f986..645d084017 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -18,6 +18,7 @@ import pLimit from 'p-limit'; import os from 'os'; import { relative as relativePath, resolve as resolvePath } from 'path'; import fs from 'fs-extra'; +import type { KnipConfig } from 'knip'; import { createBinRunner } from '../util'; // Ignore this due to Knip error: Error: ENAMETOOLONG: name too long, scandir @@ -29,7 +30,7 @@ interface KnipExtractionOptions { } interface KnipConfigOptions { - packageDir: string; + knipConfigPath: string; } interface KnipPackageOptions { @@ -55,15 +56,20 @@ function logKnipReportInstructions() { console.log(''); } -async function generateKnipConfig({ packageDir }: KnipConfigOptions) { - const knipConfig = { - entry: [ - 'dev/index.{ts,tsx}', - 'src/index.{ts,tsx}', - 'src/alpha.{ts,tsx}', - 'src/routes.ts', - 'src/run.ts', - ], +async function generateKnipConfig({ knipConfigPath }: KnipConfigOptions) { + const knipConfig: KnipConfig = { + workspaces: { + '.': {}, + '{packages,plugins}/*': { + entry: [ + 'dev/index.{ts,tsx}', + 'src/index.{ts,tsx}', + 'src/alpha.{ts,tsx}', + 'src/routes.ts', + 'src/run.ts', + ], + }, + }, jest: { entry: ['src/setupTests.ts', '**/*.test.{ts,tsx}'], }, @@ -81,15 +87,12 @@ async function generateKnipConfig({ packageDir }: KnipConfigOptions) { '@backstage/theme', // this uses `declare module` in .d.ts so is implicitly used whenever extensions are needed ], }; - await fs.writeFile( - `${packageDir}/knip.json`, - JSON.stringify(knipConfig, null, 2), - ); + await fs.writeFile(knipConfigPath, JSON.stringify(knipConfig, null, 2)); } -function cleanKnipConfig({ packageDir }: KnipConfigOptions) { - if (fs.existsSync(`${packageDir}/knip.json`)) { - fs.rmSync(`${packageDir}/knip.json`); +function cleanKnipConfig({ knipConfigPath }: KnipConfigOptions) { + if (fs.existsSync(knipConfigPath)) { + fs.rmSync(knipConfigPath); } } @@ -105,13 +108,11 @@ async function handlePackage({ } const fullDir = cliPaths.resolveTargetRoot(packageDir); const reportPath = resolvePath(fullDir, 'knip-report.md'); - const run = createBinRunner(fullDir, ''); + const run = createBinRunner(cliPaths.targetRoot, ''); - await generateKnipConfig({ packageDir: fullDir }); - - const report = await run( + let report = await run( `${knipDir}/knip.js`, - `--directory ${fullDir}`, // Run in the package directory + `-W ${packageDir}`, // Run the desired workspace '--config knip.json', '--no-exit-code', // Removing this will end the process in case there are findings by knip '--no-progress', // Remove unnecessary debugging from output @@ -120,7 +121,17 @@ async function handlePackage({ '--reporter markdown', ); - cleanKnipConfig({ packageDir: fullDir }); + // Adjust report paths to be relative to workspace + report = report.replaceAll(`| ${packageDir}/`, '| '); + // Adjust table separators + report = report.replaceAll( + new RegExp(`(\\| :-+ \\| :)-{${packageDir.length + 1}}`, 'g'), + (_, p1) => p1, + ); + report = report.replaceAll( + new RegExp(` \\| Location {1,${packageDir.length + 2}}`, 'g'), + ' | Location ', + ); const existingReport = await fs.readFile(reportPath, 'utf8').catch(error => { if (error.code === 'ENOENT') { @@ -160,8 +171,10 @@ export async function runKnipReports({ isLocalBuild, }: KnipExtractionOptions) { const knipDir = cliPaths.resolveTargetRoot('./node_modules/knip/bin/'); + const knipConfigPath = cliPaths.resolveTargetRoot('./knip.json'); const limiter = pLimit(os.cpus().length); + await generateKnipConfig({ knipConfigPath }); try { await Promise.all( packageDirs.map(packageDir => @@ -170,15 +183,9 @@ export async function runKnipReports({ ), ), ); + await cleanKnipConfig({ knipConfigPath }); } catch (e) { - console.log( - `Error occurred during knip reporting: ${e}, cleaning knip configs`, - ); - packageDirs.map(packageDir => { - const fullDir = cliPaths.resolveTargetRoot(packageDir); - cleanKnipConfig({ packageDir: fullDir }); - }); - + console.log(`Error occurred during knip reporting: ${e}`); throw e; } } diff --git a/packages/repo-tools/src/commands/util.ts b/packages/repo-tools/src/commands/util.ts index e28aae6267..ff4cdad1b8 100644 --- a/packages/repo-tools/src/commands/util.ts +++ b/packages/repo-tools/src/commands/util.ts @@ -35,11 +35,12 @@ export function createBinRunner(cwd: string, path: string) { { cwd, shell: true, - timeout: 60000, + timeout: 3 * 60 * 1000, maxBuffer: 1024 * 1024, }, (err, stdout, stderr) => { if (err) { + console.log('err', err); reject(new Error(`${err.message}\n${stderr}`)); } else if (stderr) { reject(new Error(`Command printed error output: ${stderr}`)); diff --git a/yarn.lock b/yarn.lock index cf388516b3..3d19a0021b 100644 --- a/yarn.lock +++ b/yarn.lock @@ -8329,6 +8329,7 @@ __metadata: just-diff: ^6.0.2 knex: ^3.0.0 knex-pglite: ^0.11.0 + knip: ^5.42.0 lodash: ^4.17.21 minimatch: ^9.0.0 p-limit: ^3.0.2 @@ -34104,7 +34105,7 @@ __metadata: languageName: node linkType: hard -"knip@npm:^5.0.0": +"knip@npm:^5.42.0": version: 5.42.2 resolution: "knip@npm:5.42.2" dependencies: @@ -39801,11 +39802,11 @@ __metadata: linkType: hard "pretty-ms@npm:^9.0.0": - version: 9.0.0 - resolution: "pretty-ms@npm:9.0.0" + version: 9.2.0 + resolution: "pretty-ms@npm:9.2.0" dependencies: parse-ms: ^4.0.0 - checksum: 072b17547e09cb232e8e4c7be0281e256b6d8acd18dfb2fdd715d50330d1689fdaa877f53cf90c62ed419ef842f0f5fb94a2cd8ed1aa6d7608ad48834219435d + checksum: d3a5a5b1c8a3417f64a877dba5ee2bacee404b59bc12083466e5e6dce2745e4bd716e1f9860624c7dceb1b4a532e808e4f2a7a03903a132344b3818951e2d125 languageName: node linkType: hard @@ -42217,7 +42218,6 @@ __metadata: eslint-plugin-testing-library: ^6.0.0 fs-extra: ^11.2.0 husky: ^9.0.0 - knip: ^5.0.0 lint-staged: ^15.0.0 minimist: ^1.2.5 node-gyp: ^10.0.0 From a904b605aa5b0465db4819cbf0e713d23d62b453 Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Wed, 15 Jan 2025 14:37:24 +0100 Subject: [PATCH 2/8] Update existing knip-report files Also remove a stray one Signed-off-by: Axel Hecht --- packages/app-defaults/knip-report.md | 6 + .../app-next-example-plugin/knip-report.md | 6 + packages/app-next/knip-report.md | 56 ++------ packages/app/knip-report.md | 32 ++--- packages/backend-app-api/knip-report.md | 49 ++++++- packages/backend-defaults/knip-report.md | 23 +++- .../knip-report.md | 3 +- packages/backend-legacy/knip-report.md | 9 +- packages/backend-openapi-utils/knip-report.md | 3 +- packages/backend-plugin-api/knip-report.md | 6 + packages/backend-test-utils/knip-report.md | 16 ++- packages/backend/knip-report.md | 7 +- packages/cli/knip-report.md | 123 +++++++++--------- packages/core-app-api/knip-report.md | 10 +- packages/core-compat-api/knip-report.md | 21 +++ packages/core-components/knip-report.md | 19 +-- packages/core-plugin-api/knip-report.md | 9 +- packages/create-app/knip-report.md | 62 ++++++++- packages/dev-utils/knip-report.md | 8 +- packages/e2e-test/knip-report.md | 9 +- packages/eslint-plugin/knip-report.md | 7 - packages/frontend-app-api/knip-report.md | 22 ++++ packages/frontend-plugin-api/knip-report.md | 22 ++++ packages/frontend-test-utils/knip-report.md | 19 +++ packages/integration-react/knip-report.md | 6 + packages/repo-tools/knip-report.md | 2 +- packages/techdocs-cli/knip-report.md | 8 +- packages/test-utils/knip-report.md | 9 +- packages/theme/knip-report.md | 15 ++- packages/version-bridge/knip-report.md | 12 ++ .../knip-report.md | 13 ++ plugins/api-docs/knip-report.md | 15 ++- plugins/app-backend/knip-report.md | 9 +- plugins/app-visualizer/knip-report.md | 6 + .../knip-report.md | 3 +- .../knip-report.md | 12 -- .../knip-report.md | 9 +- .../knip-report.md | 6 - .../knip-report.md | 9 +- .../knip-report.md | 20 +-- .../knip-report.md | 12 ++ plugins/auth-backend/knip-report.md | 7 +- plugins/auth-node/knip-report.md | 8 +- .../catalog-backend-module-aws/knip-report.md | 9 +- .../knip-report.md | 6 + .../knip-report.md | 6 + .../knip-report.md | 6 + .../knip-report.md | 16 --- .../knip-report.md | 6 + .../knip-report.md | 6 + .../knip-report.md | 9 +- .../knip-report.md | 6 + .../knip-report.md | 6 + .../knip-report.md | 6 + .../knip-report.md | 6 + .../knip-report.md | 7 + plugins/catalog-backend/knip-report.md | 7 - plugins/catalog-graph/knip-report.md | 8 +- plugins/catalog-import/knip-report.md | 8 +- plugins/catalog-react/knip-report.md | 15 ++- .../knip-report.md | 6 + plugins/catalog/knip-report.md | 8 +- plugins/config-schema/knip-report.md | 8 +- plugins/devtools-backend/knip-report.md | 7 + plugins/devtools/knip-report.md | 6 + plugins/events-backend/knip-report.md | 6 + plugins/events-node/knip-report.md | 6 + .../example-todo-list-backend/knip-report.md | 8 ++ plugins/example-todo-list/knip-report.md | 6 + plugins/home-react/knip-report.md | 6 + plugins/home/knip-report.md | 8 +- plugins/kubernetes-backend/knip-report.md | 24 ++-- plugins/kubernetes-cluster/knip-report.md | 15 ++- plugins/kubernetes-node/knip-report.md | 6 + plugins/kubernetes-react/knip-report.md | 6 + plugins/kubernetes/knip-report.md | 8 +- plugins/notifications-backend/knip-report.md | 11 +- plugins/notifications-node/knip-report.md | 3 +- plugins/notifications/knip-report.md | 9 +- plugins/org-react/knip-report.md | 14 +- plugins/org/knip-report.md | 12 +- .../knip-report.md | 6 + plugins/permission-react/knip-report.md | 6 + plugins/proxy-backend/knip-report.md | 11 +- .../knip-report.md | 6 - .../knip-report.md | 6 - .../knip-report.md | 11 +- .../knip-report.md | 13 ++ .../knip-report.md | 6 - .../knip-report.md | 6 +- .../knip-report.md | 6 + .../knip-report.md | 6 - .../knip-report.md | 6 - plugins/scaffolder-backend/knip-report.md | 17 ++- .../scaffolder-node-test-utils/knip-report.md | 11 +- plugins/scaffolder-react/knip-report.md | 21 ++- plugins/scaffolder/knip-report.md | 14 +- .../knip-report.md | 7 + .../search-backend-module-pg/knip-report.md | 6 + plugins/search-backend-node/knip-report.md | 6 + plugins/search-backend/knip-report.md | 7 + plugins/search-react/knip-report.md | 9 +- plugins/search/knip-report.md | 8 +- plugins/signals-backend/knip-report.md | 13 +- plugins/signals-node/knip-report.md | 3 +- plugins/signals-react/knip-report.md | 6 + plugins/signals/knip-report.md | 10 +- .../techdocs-addons-test-utils/knip-report.md | 6 + .../knip-report.md | 8 +- plugins/techdocs-react/knip-report.md | 11 +- plugins/techdocs/knip-report.md | 9 +- plugins/user-settings-backend/knip-report.md | 7 + plugins/user-settings/knip-report.md | 8 +- 113 files changed, 894 insertions(+), 426 deletions(-) delete mode 100644 plugins/catalog-backend-module-bitbucket/knip-report.md diff --git a/packages/app-defaults/knip-report.md b/packages/app-defaults/knip-report.md index 2661c35327..27a145e95c 100644 --- a/packages/app-defaults/knip-report.md +++ b/packages/app-defaults/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/packages/app-next-example-plugin/knip-report.md b/packages/app-next-example-plugin/knip-report.md index e01cd18c93..54dc66d59e 100644 --- a/packages/app-next-example-plugin/knip-report.md +++ b/packages/app-next-example-plugin/knip-report.md @@ -7,3 +7,9 @@ | cross-fetch | package.json | error | | msw | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/packages/app-next/knip-report.md b/packages/app-next/knip-report.md index 78d11bdebd..c30e426318 100644 --- a/packages/app-next/knip-report.md +++ b/packages/app-next/knip-report.md @@ -1,79 +1,45 @@ # Knip report -## Unused dependencies (63) +## Unused dependencies (30) | Name | Location | Severity | | :----------------------------------------------- | :----------- | :------- | | @backstage/plugin-techdocs-module-addons-contrib | package.json | error | -| @roadiehq/backstage-plugin-github-pull-requests | package.json | error | | @backstage/plugin-catalog-unprocessed-entities | package.json | error | -| @roadiehq/backstage-plugin-github-insights | package.json | error | -| @oriflame/backstage-plugin-score-card | package.json | error | -| @backstage/plugin-newrelic-dashboard | package.json | error | -| @roadiehq/backstage-plugin-buildkite | package.json | error | -| @roadiehq/backstage-plugin-travis-ci | package.json | error | +| @backstage/plugin-kubernetes-cluster | package.json | error | | @backstage/plugin-permission-react | package.json | error | | @backstage/plugin-scaffolder-react | package.json | error | -| @backstage/plugin-entity-feedback | package.json | error | -| @backstage/plugin-linguist-common | package.json | error | -| @backstage/plugin-apache-airflow | package.json | error | | @backstage/plugin-catalog-common | package.json | error | -| @backstage/plugin-github-actions | package.json | error | -| @backstage/plugin-octopus-deploy | package.json | error | | @backstage/plugin-techdocs-react | package.json | error | | @backstage/plugin-catalog-graph | package.json | error | -| @backstage/plugin-catalog-react | package.json | error | -| @backstage/plugin-code-coverage | package.json | error | -| @backstage/plugin-cost-insights | package.json | error | +| @backstage/plugin-notifications | package.json | error | | @backstage/plugin-search-common | package.json | error | -| @backstage/plugin-tech-insights | package.json | error | -| @backstage/plugin-azure-devops | package.json | error | -| @backstage/plugin-gcp-projects | package.json | error | | @backstage/plugin-search-react | package.json | error | -| @backstage/plugin-azure-sites | package.json | error | -| @backstage/plugin-cloudbuild | package.json | error | -| @backstage/plugin-kubernetes | package.json | error | -| @backstage/plugin-lighthouse | package.json | error | +| @backstage/integration-react | package.json | error | +| @backstage/plugin-auth-react | package.json | error | | @backstage/plugin-scaffolder | package.json | error | -| @backstage/plugin-stackstorm | package.json | error | -| @backstage/plugin-dynatrace | package.json | error | -| @backstage/plugin-shortcuts | package.json | error | -| @backstage/plugin-airbrake | package.json | error | +| @backstage/frontend-app-api | package.json | error | +| @backstage/core-plugin-api | package.json | error | | @backstage/plugin-api-docs | package.json | error | -| @backstage/plugin-devtools | package.json | error | -| @backstage/plugin-linguist | package.json | error | -| @backstage/plugin-newrelic | package.json | error | -| @backstage/plugin-playlist | package.json | error | -| @backstage/plugin-puppetdb | package.json | error | -| @circleci/backstage-plugin | package.json | error | | @backstage/plugin-catalog | package.json | error | -| @backstage/plugin-explore | package.json | error | -| @backstage/plugin-jenkins | package.json | error | -| @backstage/plugin-rollbar | package.json | error | +| @backstage/plugin-signals | package.json | error | | @backstage/catalog-model | package.json | error | -| @backstage/plugin-badges | package.json | error | | @backstage/plugin-search | package.json | error | -| @backstage/plugin-sentry | package.json | error | | @backstage/app-defaults | package.json | error | -| @backstage/plugin-kafka | package.json | error | -| app-next-example-plugin | package.json | error | -| @backstage/plugin-gocd | package.json | error | -| @backstage/plugin-todo | package.json | error | -| @backstage/plugin-adr | package.json | error | +| @backstage/plugin-app | package.json | error | | @backstage/plugin-org | package.json | error | +| @backstage/config | package.json | error | | @material-ui/lab | package.json | error | | zen-observable | package.json | error | | @octokit/rest | package.json | error | | react-use | package.json | error | | history | package.json | error | -| lodash | package.json | error | -## Unused devDependencies (6) +## Unused devDependencies (5) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @testing-library/user-event | package.json | error | -| @testing-library/react | package.json | error | | @types/zen-observable | package.json | error | | @testing-library/dom | package.json | error | | @types/jquery | package.json | error | diff --git a/packages/app/knip-report.md b/packages/app/knip-report.md index 118f6f14e0..b05d232915 100644 --- a/packages/app/knip-report.md +++ b/packages/app/knip-report.md @@ -1,33 +1,25 @@ # Knip report -## Unused dependencies (14) +## Unused dependencies (8) -| Name | Location | Severity | -| :-------------------------------- | :----------- | :------- | -| @backstage/plugin-linguist-common | package.json | error | -| @backstage/plugin-search-common | package.json | error | -| @backstage/plugin-azure-sites | package.json | error | -| @backstage/frontend-app-api | package.json | error | -| @backstage/plugin-airbrake | package.json | error | -| vite-plugin-node-polyfills | package.json | error | -| @vitejs/plugin-react | package.json | error | -| @material-ui/lab | package.json | error | -| vite-plugin-html | package.json | error | -| zen-observable | package.json | error | -| @octokit/rest | package.json | error | -| react-router | package.json | error | -| history | package.json | error | -| vite | package.json | error | +| Name | Location | Severity | +| :------------------------------ | :----------- | :------- | +| @backstage/plugin-search-common | package.json | error | +| @backstage/plugin-auth-react | package.json | error | +| @backstage/frontend-app-api | package.json | error | +| @material-ui/lab | package.json | error | +| zen-observable | package.json | error | +| @octokit/rest | package.json | error | +| react-router | package.json | error | +| history | package.json | error | -## Unused devDependencies (5) +## Unused devDependencies (3) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @testing-library/user-event | package.json | error | | @types/zen-observable | package.json | error | -| @testing-library/dom | package.json | error | | @types/jquery | package.json | error | -| cross-env | package.json | error | ## Unlisted dependencies (1) diff --git a/packages/backend-app-api/knip-report.md b/packages/backend-app-api/knip-report.md index 8bf5c4c458..5180cb33c8 100644 --- a/packages/backend-app-api/knip-report.md +++ b/packages/backend-app-api/knip-report.md @@ -1,8 +1,49 @@ # Knip report -## Unused dependencies (1) +## Unused dependencies (27) -| Name | Location | Severity | -| :---------------- | :----------- | :------- | -| winston-transport | package.json | error | +| Name | Location | Severity | +| :-------------------------------- | :----------- | :------- | +| @backstage/plugin-permission-node | package.json | error | +| @backstage/plugin-auth-node | package.json | error | +| @backstage/config-loader | package.json | error | +| @backstage/cli-common | package.json | error | +| @manypkg/get-packages | package.json | error | +| @backstage/config | package.json | error | +| winston-transport | package.json | error | +| @backstage/types | package.json | error | +| path-to-regexp | package.json | error | +| compression | package.json | error | +| triple-beam | package.json | error | +| node-forge | package.json | error | +| selfsigned | package.json | error | +| minimatch | package.json | error | +| stoppable | package.json | error | +| minimist | package.json | error | +| logform | package.json | error | +| winston | package.json | error | +| cookie | package.json | error | +| helmet | package.json | error | +| lodash | package.json | error | +| morgan | package.json | error | +| luxon | package.json | error | +| cors | package.json | error | +| jose | package.json | error | +| knex | package.json | error | +| uuid | package.json | error | + +## Unused devDependencies (10) + +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/backend-defaults | package.json | error | +| @types/compression | package.json | error | +| @types/http-errors | package.json | error | +| @types/node-forge | package.json | error | +| @types/stoppable | package.json | error | +| @types/minimist | package.json | error | +| @types/morgan | package.json | error | +| http-errors | package.json | error | +| supertest | package.json | error | +| msw | package.json | error | diff --git a/packages/backend-defaults/knip-report.md b/packages/backend-defaults/knip-report.md index 14b2b4cc88..22105d3219 100644 --- a/packages/backend-defaults/knip-report.md +++ b/packages/backend-defaults/knip-report.md @@ -1,8 +1,25 @@ # Knip report +## Unused dependencies (6) + +| Name | Location | Severity | +| :-------------------- | :----------- | :------- | +| @backstage/cli-common | package.json | error | +| @azure/identity | package.json | error | +| better-sqlite3 | package.json | error | +| isomorphic-git | package.json | error | +| minimist | package.json | error | +| mysql2 | package.json | error | + ## Unused devDependencies (1) -| Name | Location | Severity | -| :---------------------------- | :----------- | :------- | -| @backstage/backend-test-utils | package.json | error | +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| @types/stoppable | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :-------------------------------- | :----------- | :------- | +| @google-cloud/cloud-sql-connector | package.json | error | diff --git a/packages/backend-dynamic-feature-service/knip-report.md b/packages/backend-dynamic-feature-service/knip-report.md index e738c4d931..79f2435433 100644 --- a/packages/backend-dynamic-feature-service/knip-report.md +++ b/packages/backend-dynamic-feature-service/knip-report.md @@ -1,9 +1,8 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (1) | Name | Location | Severity | | :------------------------------ | :----------- | :------- | | @backstage/plugin-search-common | package.json | error | -| @backstage/errors | package.json | error | diff --git a/packages/backend-legacy/knip-report.md b/packages/backend-legacy/knip-report.md index f57bda79b7..8ec6203e20 100644 --- a/packages/backend-legacy/knip-report.md +++ b/packages/backend-legacy/knip-report.md @@ -6,22 +6,23 @@ | :------------------------------------------------- | :----------- | :------- | | @backstage/plugin-scaffolder-backend-module-gitlab | package.json | error | | @backstage/plugin-scaffolder-backend-module-rails | package.json | error | -| @backstage/plugin-azure-sites-common | package.json | error | -| @backstage/plugin-tech-insights-node | package.json | error | +| @backstage/plugin-search-backend-module-catalog | package.json | error | +| @backstage/plugin-signals-backend | package.json | error | | azure-devops-node-api | package.json | error | -| pg-connection-string | package.json | error | | @gitbeaker/node | package.json | error | | better-sqlite3 | package.json | error | | @octokit/rest | package.json | error | | example-app | package.json | error | +| dockerode | package.json | error | | mysql2 | package.json | error | | luxon | package.json | error | | pg | package.json | error | -## Unused devDependencies (2) +## Unused devDependencies (3) | Name | Location | Severity | | :------------------------------- | :----------- | :------- | | @types/express-serve-static-core | package.json | error | +| @types/dockerode | package.json | error | | @types/luxon | package.json | error | diff --git a/packages/backend-openapi-utils/knip-report.md b/packages/backend-openapi-utils/knip-report.md index 5d1536b3c1..cf540a8d1d 100644 --- a/packages/backend-openapi-utils/knip-report.md +++ b/packages/backend-openapi-utils/knip-report.md @@ -1,9 +1,8 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (1) | Name | Location | Severity | | :---------------------------- | :----------- | :------- | | @backstage/backend-plugin-api | package.json | error | -| lodash | package.json | error | diff --git a/packages/backend-plugin-api/knip-report.md b/packages/backend-plugin-api/knip-report.md index 2661c35327..89988530e6 100644 --- a/packages/backend-plugin-api/knip-report.md +++ b/packages/backend-plugin-api/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/plugin-auth-node | package.json | error | + diff --git a/packages/backend-test-utils/knip-report.md b/packages/backend-test-utils/knip-report.md index 18741dce81..6ba3e32cd6 100644 --- a/packages/backend-test-utils/knip-report.md +++ b/packages/backend-test-utils/knip-report.md @@ -1,11 +1,13 @@ # Knip report -## Unused dependencies (4) +## Unused dependencies (6) -| Name | Location | Severity | -| :------------- | :----------- | :------- | -| better-sqlite3 | package.json | error | -| mysql2 | package.json | error | -| msw | package.json | error | -| pg | package.json | error | +| Name | Location | Severity | +| :------------------------------- | :----------- | :------- | +| @types/express-serve-static-core | package.json | error | +| @backstage/plugin-auth-node | package.json | error | +| better-sqlite3 | package.json | error | +| @types/qs | package.json | error | +| mysql2 | package.json | error | +| pg | package.json | error | diff --git a/packages/backend/knip-report.md b/packages/backend/knip-report.md index a26b412ee9..d24322fd2b 100644 --- a/packages/backend/knip-report.md +++ b/packages/backend/knip-report.md @@ -1,12 +1,15 @@ # Knip report -## Unused dependencies (5) +## Unused dependencies (8) | Name | Location | Severity | | :----------------------------------------------- | :----------- | :------- | | @backstage/plugin-catalog-backend-module-openapi | package.json | error | +| @opentelemetry/auto-instrumentations-node | package.json | error | | @backstage/plugin-search-backend-node | package.json | error | | @backstage/plugin-permission-common | package.json | error | +| @opentelemetry/exporter-prometheus | package.json | error | | @backstage/plugin-permission-node | package.json | error | -| @backstage/backend-tasks | package.json | error | +| @opentelemetry/sdk-node | package.json | error | +| example-app | package.json | error | diff --git a/packages/cli/knip-report.md b/packages/cli/knip-report.md index 16926bb87d..60ccba435c 100644 --- a/packages/cli/knip-report.md +++ b/packages/cli/knip-report.md @@ -1,68 +1,75 @@ # Knip report -## Unused dependencies (28) +## Unused dependencies (26) -| Name | Location | Severity | -| :------------------------------- | :----------- | :------- | -| @typescript-eslint/eslint-plugin | package.json | error | -| @spotify/eslint-config-react | package.json | error | -| eslint-formatter-friendly | package.json | error | -| eslint-plugin-deprecation | package.json | error | -| eslint-plugin-react-hooks | package.json | error | -| @sucrase/webpack-loader | package.json | error | -| eslint-plugin-jsx-a11y | package.json | error | -| jest-environment-jsdom | package.json | error | -| terser-webpack-plugin | package.json | error | -| eslint-plugin-react | package.json | error | -| @types/webpack-env | package.json | error | -| @svgr/plugin-svgo | package.json | error | -| @svgr/plugin-jsx | package.json | error | -| jest-css-modules | package.json | error | -| @swc/helpers | package.json | error | -| jest-runtime | package.json | error | -| cross-fetch | package.json | error | -| @svgr/core | package.json | error | -| @swc/core | package.json | error | -| @swc/jest | package.json | error | -| esbuild | package.json | error | -| pirates | package.json | error | -| postcss | package.json | error | -| process | package.json | error | -| sucrase | package.json | error | -| buffer | package.json | error | -| glob | package.json | error | -| util | package.json | error | +| Name | Location | Severity | +| :--------------------------- | :----------- | :------- | +| @spotify/eslint-config-react | package.json | error | +| eslint-formatter-friendly | package.json | error | +| eslint-plugin-react-hooks | package.json | error | +| @backstage/catalog-model | package.json | error | +| @octokit/graphql-schema | package.json | error | +| @sucrase/webpack-loader | package.json | error | +| @backstage/integration | package.json | error | +| eslint-plugin-jsx-a11y | package.json | error | +| jest-environment-jsdom | package.json | error | +| terser-webpack-plugin | package.json | error | +| eslint-plugin-react | package.json | error | +| @octokit/oauth-app | package.json | error | +| @svgr/plugin-svgo | package.json | error | +| @octokit/graphql | package.json | error | +| @svgr/plugin-jsx | package.json | error | +| jest-css-modules | package.json | error | +| git-url-parse | package.json | error | +| jest-runtime | package.json | error | +| cross-fetch | package.json | error | +| @svgr/core | package.json | error | +| @swc/jest | package.json | error | +| process | package.json | error | +| sucrase | package.json | error | +| buffer | package.json | error | +| glob | package.json | error | +| util | package.json | error | -## Unused devDependencies (11) +## Unused devDependencies (14) -| Name | Location | Severity | -| :-------------------------------------- | :----------- | :------- | -| @types/rollup-plugin-peer-deps-external | package.json | error | -| @backstage/backend-plugin-api | package.json | error | -| @types/terser-webpack-plugin | package.json | error | -| @backstage/core-components | package.json | error | -| @backstage/core-app-api | package.json | error | -| @backstage/dev-utils | package.json | error | -| @types/http-proxy | package.json | error | -| @types/svgo | package.json | error | -| @types/ejs | package.json | error | -| nodemon | package.json | error | -| del | package.json | error | +| Name | Location | Severity | +| :--------------------------------------------------- | :----------- | :------- | +| @backstage/plugin-auth-backend-module-guest-provider | package.json | error | +| @types/rollup-plugin-peer-deps-external | package.json | error | +| @backstage/plugin-auth-backend | package.json | error | +| @backstage/plugin-catalog-node | package.json | error | +| @types/terser-webpack-plugin | package.json | error | +| @backstage/core-components | package.json | error | +| @backstage/catalog-client | package.json | error | +| @backstage/core-app-api | package.json | error | +| @types/webpack-sources | package.json | error | +| @backstage/dev-utils | package.json | error | +| @types/http-proxy | package.json | error | +| @types/svgo | package.json | error | +| @types/ejs | package.json | error | +| del | package.json | error | -## Referenced optional peerDependencies (4) +## Referenced optional peerDependencies (3) -| Name | Location | Severity | -| :------------------------- | :----------- | :------- | -| vite-plugin-node-polyfills | package.json | error | -| @vitejs/plugin-react | package.json | error | -| vite-plugin-html | package.json | error | -| vite | package.json | error | +| Name | Location | Severity | +| :--------------------------- | :----------- | :------- | +| @rspack/plugin-react-refresh | package.json | error | +| @rspack/dev-server | package.json | error | +| @rspack/core | package.json | error | -## Unlisted dependencies (3) +## Unlisted dependencies (10) -| Name | Location | Severity | -| :----------------- | :----------------------------------------- | :------- | -| prettier | src/commands/migrate/packageLintConfigs.ts | error | -| react-dom/client | src/lib/bundler/hasReactDomClient.ts | error | -| react/package.json | src/lib/bundler/server.ts | error | +| Name | Location | Severity | +| :--------------------- | :----------------------------------------------------------------------------------------------- | :------- | +| @testing-library/react | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.test.tsx | error | +| react | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.test.tsx | error | +| @material-ui/core | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.tsx | error | +| react | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.tsx | error | +| supertest | templates/default-backend-plugin/src/router.test.ts | error | +| express-promise-router | templates/default-backend-plugin/src/router.ts | error | +| react-dom | src/lib/bundler/hasReactDomClient.ts | error | +| react-dom | src/lib/bundler/config.ts | error | +| react | src/lib/bundler/config.ts | error | +| react | src/lib/bundler/server.ts | error | diff --git a/packages/core-app-api/knip-report.md b/packages/core-app-api/knip-report.md index 4d61a32f32..010afdd8b4 100644 --- a/packages/core-app-api/knip-report.md +++ b/packages/core-app-api/knip-report.md @@ -6,15 +6,19 @@ | :------ | :----------- | :------- | | history | package.json | error | -## Unused devDependencies (7) +## Unused devDependencies (5) | Name | Location | Severity | | :--------------------------- | :----------- | :------- | | @testing-library/react-hooks | package.json | error | -| @testing-library/user-event | package.json | error | | react-router-dom-stable | package.json | error | | react-router-dom-beta | package.json | error | -| @testing-library/dom | package.json | error | | react-router-stable | package.json | error | | react-router-beta | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/packages/core-compat-api/knip-report.md b/packages/core-compat-api/knip-report.md index 2661c35327..d6bf611772 100644 --- a/packages/core-compat-api/knip-report.md +++ b/packages/core-compat-api/knip-report.md @@ -1,2 +1,23 @@ # Knip report +## Unused devDependencies (3) + +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/frontend-app-api | package.json | error | +| @backstage/types | package.json | error | +| zod | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + +## Unlisted dependencies (2) + +| Name | Location | Severity | +| :----------------- | :------------------------------- | :------- | +| @internal/frontend | src/collectLegacyRoutes.test.tsx | error | +| @internal/frontend | src/convertLegacyPlugin.test.tsx | error | + diff --git a/packages/core-components/knip-report.md b/packages/core-components/knip-report.md index 655c324084..6d8a954385 100644 --- a/packages/core-components/knip-report.md +++ b/packages/core-components/knip-report.md @@ -1,22 +1,25 @@ # Knip report -## Unused dependencies (3) +## Unused dependencies (1) -| Name | Location | Severity | -| :------------ | :----------- | :------- | -| @date-io/core | package.json | error | -| linkifyjs | package.json | error | -| pluralize | package.json | error | +| Name | Location | Severity | +| :-------- | :----------- | :------- | +| pluralize | package.json | error | -## Unused devDependencies (4) +## Unused devDependencies (3) | Name | Location | Severity | | :---------------------- | :----------- | :------- | | @backstage/app-defaults | package.json | error | | @types/google-protobuf | package.json | error | -| @testing-library/dom | package.json | error | | cross-fetch | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + ## Unlisted dependencies (3) | Name | Location | Severity | diff --git a/packages/core-plugin-api/knip-report.md b/packages/core-plugin-api/knip-report.md index d0b97e0e68..20e35cfd7d 100644 --- a/packages/core-plugin-api/knip-report.md +++ b/packages/core-plugin-api/knip-report.md @@ -1,9 +1,14 @@ # Knip report -## Unused devDependencies (2) +## Unused devDependencies (1) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @testing-library/user-event | package.json | error | -| @testing-library/dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/packages/create-app/knip-report.md b/packages/create-app/knip-report.md index 65ef020fc3..0afa527ab0 100644 --- a/packages/create-app/knip-report.md +++ b/packages/create-app/knip-report.md @@ -1,9 +1,67 @@ # Knip report -## Unused devDependencies (2) +## Unused devDependencies (1) | Name | Location | Severity | | :-------------------- | :----------- | :------- | | @types/command-exists | package.json | error | -| nodemon | package.json | error | + +## Unlisted dependencies (54) + +| Name | Location | Severity | +| :----------------------------------------------- | :----------------------------------------------------------------------- | :------- | +| @backstage/plugin-techdocs-module-addons-contrib | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-techdocs-react | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-catalog-graph | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-kubernetes | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-api-docs | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-techdocs | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/core-components | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-catalog | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/catalog-model | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-org | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @material-ui/core | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| react | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | +| @backstage/plugin-catalog-react | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @backstage/plugin-search-react | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @backstage/plugin-techdocs | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @backstage/core-components | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @backstage/core-plugin-api | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @backstage/plugin-catalog | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @backstage/plugin-search | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @material-ui/core | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| react | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | +| @material-ui/core | templates/default-app/packages/app/src/components/Root/LogoFull.tsx | error | +| @material-ui/core | templates/default-app/packages/app/src/components/Root/LogoIcon.tsx | error | +| react | templates/default-app/packages/app/src/components/Root/LogoFull.tsx | error | +| react | templates/default-app/packages/app/src/components/Root/LogoIcon.tsx | error | +| @backstage/plugin-user-settings | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| @backstage/core-components | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| @backstage/plugin-search | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| @backstage/plugin-org | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| @material-ui/icons | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| @material-ui/core | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| react | templates/default-app/packages/app/src/components/Root/Root.tsx | error | +| @testing-library/react | templates/default-app/packages/app/src/App.test.tsx | error | +| react | templates/default-app/packages/app/src/App.test.tsx | error | +| @backstage/plugin-techdocs-module-addons-contrib | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-permission-react | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-catalog-import | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-techdocs-react | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-catalog-common | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-user-settings | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-catalog-graph | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-scaffolder | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/integration-react | templates/default-app/packages/app/src/apis.ts | error | +| @backstage/plugin-api-docs | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-techdocs | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/core-components | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/core-plugin-api | templates/default-app/packages/app/src/apis.ts | error | +| @backstage/plugin-catalog | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-search | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/app-defaults | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/core-app-api | templates/default-app/packages/app/src/App.tsx | error | +| @backstage/plugin-org | templates/default-app/packages/app/src/App.tsx | error | +| react-router-dom | templates/default-app/packages/app/src/App.tsx | error | +| react | templates/default-app/packages/app/src/App.tsx | error | diff --git a/packages/dev-utils/knip-report.md b/packages/dev-utils/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/packages/dev-utils/knip-report.md +++ b/packages/dev-utils/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/packages/e2e-test/knip-report.md b/packages/e2e-test/knip-report.md index baf9661c0a..33d38c8508 100644 --- a/packages/e2e-test/knip-report.md +++ b/packages/e2e-test/knip-report.md @@ -1,9 +1,8 @@ # Knip report -## Unused devDependencies (2) +## Unused dependencies (1) -| Name | Location | Severity | -| :--------------- | :----------- | :------- | -| @types/puppeteer | package.json | error | -| nodemon | package.json | error | +| Name | Location | Severity | +| :-------------------- | :----------- | :------- | +| @backstage/create-app | package.json | error | diff --git a/packages/eslint-plugin/knip-report.md b/packages/eslint-plugin/knip-report.md index d4c4be0549..2661c35327 100644 --- a/packages/eslint-plugin/knip-report.md +++ b/packages/eslint-plugin/knip-report.md @@ -1,9 +1,2 @@ # Knip report -## Unlisted dependencies (2) - -| Name | Location | Severity | -| :----- | :----------------------------- | :------- | -| estree | rules/no-undeclared-imports.js | error | -| estree | lib/visitImports.js | error | - diff --git a/packages/frontend-app-api/knip-report.md b/packages/frontend-app-api/knip-report.md index 2661c35327..517e88ce8a 100644 --- a/packages/frontend-app-api/knip-report.md +++ b/packages/frontend-app-api/knip-report.md @@ -1,2 +1,24 @@ # Knip report +## Unused dependencies (3) + +| Name | Location | Severity | +| :--------------------------- | :----------- | :------- | +| @backstage/frontend-defaults | package.json | error | +| @backstage/errors | package.json | error | +| zod | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + +## Unlisted dependencies (3) + +| Name | Location | Severity | +| :----------------- | :---------------------------------- | :------- | +| @internal/frontend | src/wiring/createSpecializedApp.tsx | error | +| @internal/frontend | src/tree/resolveAppNodeSpecs.ts | error | +| @internal/frontend | src/routing/collectRouteIds.ts | error | + diff --git a/packages/frontend-plugin-api/knip-report.md b/packages/frontend-plugin-api/knip-report.md index e571899442..43b78effd5 100644 --- a/packages/frontend-plugin-api/knip-report.md +++ b/packages/frontend-plugin-api/knip-report.md @@ -8,3 +8,25 @@ | @material-ui/core | package.json | error | | lodash | package.json | error | +## Unused devDependencies (1) + +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/frontend-app-api | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + +## Unlisted dependencies (5) + +| Name | Location | Severity | +| :----------------- | :------------------------------------------- | :------- | +| @internal/frontend | src/wiring/createExtensionBlueprint.test.tsx | error | +| @internal/frontend | src/wiring/resolveExtensionDefinition.ts | error | +| @internal/frontend | src/wiring/createFrontendModule.ts | error | +| @internal/frontend | src/wiring/createFrontendPlugin.ts | error | +| @internal/frontend | src/wiring/createExtension.ts | error | + diff --git a/packages/frontend-test-utils/knip-report.md b/packages/frontend-test-utils/knip-report.md index 2661c35327..5b35805816 100644 --- a/packages/frontend-test-utils/knip-report.md +++ b/packages/frontend-test-utils/knip-report.md @@ -1,2 +1,21 @@ # Knip report +## Unused dependencies (2) + +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/version-bridge | package.json | error | +| zod | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + +## Unlisted dependencies (1) + +| Name | Location | Severity | +| :----------------- | :-------------------------------- | :------- | +| @internal/frontend | src/app/createExtensionTester.tsx | error | + diff --git a/packages/integration-react/knip-report.md b/packages/integration-react/knip-report.md index 551f19ab2f..9ccde91b0c 100644 --- a/packages/integration-react/knip-report.md +++ b/packages/integration-react/knip-report.md @@ -7,3 +7,9 @@ | @testing-library/dom | package.json | error | | msw | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/packages/repo-tools/knip-report.md b/packages/repo-tools/knip-report.md index 32502b699c..4d8084d9da 100644 --- a/packages/repo-tools/knip-report.md +++ b/packages/repo-tools/knip-report.md @@ -5,8 +5,8 @@ | Name | Location | Severity | | :---------------------------------- | :----------- | :------- | | @openapitools/openapi-generator-cli | package.json | error | -| @stoplight/spectral-functions | package.json | error | | @stoplight/spectral-runtime | package.json | error | +| @electric-sql/pglite | package.json | error | | is-glob | package.json | error | | glob | package.json | error | diff --git a/packages/techdocs-cli/knip-report.md b/packages/techdocs-cli/knip-report.md index 7329783857..5948844a00 100644 --- a/packages/techdocs-cli/knip-report.md +++ b/packages/techdocs-cli/knip-report.md @@ -8,8 +8,8 @@ ## Unused devDependencies (2) -| Name | Location | Severity | -| :----------------- | :----------- | :------- | -| @types/webpack-env | package.json | error | -| nodemon | package.json | error | +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| techdocs-cli-embedded-app | package.json | error | +| @types/webpack-env | package.json | error | diff --git a/packages/test-utils/knip-report.md b/packages/test-utils/knip-report.md index 125cdef7fd..1b48f703a1 100644 --- a/packages/test-utils/knip-report.md +++ b/packages/test-utils/knip-report.md @@ -1,8 +1,9 @@ # Knip report -## Unused dependencies (1) +## Referenced optional peerDependencies (2) -| Name | Location | Severity | -| :---------------- | :----------- | :------- | -| @material-ui/core | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | +| @types/jest | package.json | error | diff --git a/packages/theme/knip-report.md b/packages/theme/knip-report.md index e19ce53e6f..c5f148fbd3 100644 --- a/packages/theme/knip-report.md +++ b/packages/theme/knip-report.md @@ -1,9 +1,14 @@ # Knip report -## Unused dependencies (2) +## Unused devDependencies (1) -| Name | Location | Severity | -| :-------------- | :----------- | :------- | -| @emotion/styled | package.json | error | -| @emotion/react | package.json | error | +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| react-router-dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/packages/version-bridge/knip-report.md b/packages/version-bridge/knip-report.md index 2661c35327..c5f148fbd3 100644 --- a/packages/version-bridge/knip-report.md +++ b/packages/version-bridge/knip-report.md @@ -1,2 +1,14 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| react-router-dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/api-docs-module-protoc-gen-doc/knip-report.md b/plugins/api-docs-module-protoc-gen-doc/knip-report.md index 2661c35327..9743763ca9 100644 --- a/plugins/api-docs-module-protoc-gen-doc/knip-report.md +++ b/plugins/api-docs-module-protoc-gen-doc/knip-report.md @@ -1,2 +1,15 @@ # Knip report +## Unused devDependencies (2) + +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| react-router-dom | package.json | error | +| react-dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/api-docs/knip-report.md b/plugins/api-docs/knip-report.md index 4bf9058b8d..27e141bb8c 100644 --- a/plugins/api-docs/knip-report.md +++ b/plugins/api-docs/knip-report.md @@ -8,9 +8,16 @@ | graphql-config | package.json | error | | graphql-ws | package.json | error | -## Unused devDependencies (1) +## Unused devDependencies (2) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :---------------------- | :----------- | :------- | +| @backstage/core-app-api | package.json | error | +| @types/highlightjs | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/app-backend/knip-report.md b/plugins/app-backend/knip-report.md index 06609425e8..588eca0213 100644 --- a/plugins/app-backend/knip-report.md +++ b/plugins/app-backend/knip-report.md @@ -1,10 +1,11 @@ # Knip report -## Unused dependencies (1) +## Unused dependencies (2) -| Name | Location | Severity | -| :- | :----------- | :------- | -| yn | package.json | error | +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/plugin-auth-node | package.json | error | +| yn | package.json | error | ## Unused devDependencies (1) diff --git a/plugins/app-visualizer/knip-report.md b/plugins/app-visualizer/knip-report.md index 2661c35327..27a145e95c 100644 --- a/plugins/app-visualizer/knip-report.md +++ b/plugins/app-visualizer/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/auth-backend-module-aws-alb-provider/knip-report.md b/plugins/auth-backend-module-aws-alb-provider/knip-report.md index 56eb028c3f..14abcfe083 100644 --- a/plugins/auth-backend-module-aws-alb-provider/knip-report.md +++ b/plugins/auth-backend-module-aws-alb-provider/knip-report.md @@ -1,9 +1,8 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (1) | Name | Location | Severity | | :----------------------------- | :----------- | :------- | | @backstage/plugin-auth-backend | package.json | error | -| @backstage/backend-common | package.json | error | diff --git a/plugins/auth-backend-module-cloudflare-access-provider/knip-report.md b/plugins/auth-backend-module-cloudflare-access-provider/knip-report.md index 1768e133a8..2661c35327 100644 --- a/plugins/auth-backend-module-cloudflare-access-provider/knip-report.md +++ b/plugins/auth-backend-module-cloudflare-access-provider/knip-report.md @@ -1,14 +1,2 @@ # Knip report -## Unused dependencies (1) - -| Name | Location | Severity | -| :--------- | :----------- | :------- | -| node-fetch | package.json | error | - -## Unused devDependencies (1) - -| Name | Location | Severity | -| :--------------- | :----------- | :------- | -| @backstage/types | package.json | error | - diff --git a/plugins/auth-backend-module-guest-provider/knip-report.md b/plugins/auth-backend-module-guest-provider/knip-report.md index 4a3213a86c..d2b8db6f32 100644 --- a/plugins/auth-backend-module-guest-provider/knip-report.md +++ b/plugins/auth-backend-module-guest-provider/knip-report.md @@ -1,11 +1,10 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (1) -| Name | Location | Severity | -| :------------------------ | :----------- | :------- | -| @backstage/backend-common | package.json | error | -| passport-oauth2 | package.json | error | +| Name | Location | Severity | +| :-------------- | :----------- | :------- | +| passport-oauth2 | package.json | error | ## Unused devDependencies (2) diff --git a/plugins/auth-backend-module-microsoft-provider/knip-report.md b/plugins/auth-backend-module-microsoft-provider/knip-report.md index 4386ad9c4a..2661c35327 100644 --- a/plugins/auth-backend-module-microsoft-provider/knip-report.md +++ b/plugins/auth-backend-module-microsoft-provider/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Unused dependencies (1) - -| Name | Location | Severity | -| :------- | :----------- | :------- | -| passport | package.json | error | - diff --git a/plugins/auth-backend-module-oidc-provider/knip-report.md b/plugins/auth-backend-module-oidc-provider/knip-report.md index 448be9818e..0d36158781 100644 --- a/plugins/auth-backend-module-oidc-provider/knip-report.md +++ b/plugins/auth-backend-module-oidc-provider/knip-report.md @@ -1,11 +1,10 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (1) -| Name | Location | Severity | -| :------------------------ | :----------- | :------- | -| @backstage/backend-common | package.json | error | -| passport | package.json | error | +| Name | Location | Severity | +| :------- | :----------- | :------- | +| passport | package.json | error | ## Unused devDependencies (3) diff --git a/plugins/auth-backend-module-onelogin-provider/knip-report.md b/plugins/auth-backend-module-onelogin-provider/knip-report.md index a9ad7e156a..6624c86073 100644 --- a/plugins/auth-backend-module-onelogin-provider/knip-report.md +++ b/plugins/auth-backend-module-onelogin-provider/knip-report.md @@ -1,19 +1,9 @@ # Knip report -## Unused dependencies (4) +## Unused dependencies (2) -| Name | Location | Severity | -| :---------------------------- | :----------- | :------- | -| @backstage/backend-plugin-api | package.json | error | -| @backstage/plugin-auth-node | package.json | error | -| passport | package.json | error | -| express | package.json | error | - -## Unused devDependencies (3) - -| Name | Location | Severity | -| :----------------------------- | :----------- | :------- | -| @backstage/plugin-auth-backend | package.json | error | -| @backstage/backend-test-utils | package.json | error | -| @backstage/backend-defaults | package.json | error | +| Name | Location | Severity | +| :------- | :----------- | :------- | +| passport | package.json | error | +| express | package.json | error | diff --git a/plugins/auth-backend-module-vmware-cloud-provider/knip-report.md b/plugins/auth-backend-module-vmware-cloud-provider/knip-report.md index 2661c35327..9f46420ca0 100644 --- a/plugins/auth-backend-module-vmware-cloud-provider/knip-report.md +++ b/plugins/auth-backend-module-vmware-cloud-provider/knip-report.md @@ -1,2 +1,14 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :----------------------- | :----------- | :------- | +| @backstage/catalog-model | package.json | error | + +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---------------- | :----------- | :------- | +| @backstage/errors | package.json | error | + diff --git a/plugins/auth-backend/knip-report.md b/plugins/auth-backend/knip-report.md index 37d0866820..fd6926afb9 100644 --- a/plugins/auth-backend/knip-report.md +++ b/plugins/auth-backend/knip-report.md @@ -1,13 +1,15 @@ # Knip report -## Unused dependencies (12) +## Unused dependencies (14) | Name | Location | Severity | | :---------------------- | :----------- | :------- | | passport-google-oauth20 | package.json | error | +| passport-onelogin-oauth | package.json | error | | google-auth-library | package.json | error | | passport-microsoft | package.json | error | | passport-github2 | package.json | error | +| passport-auth0 | package.json | error | | openid-client | package.json | error | | compression | package.json | error | | node-cache | package.json | error | @@ -17,7 +19,7 @@ | cors | package.json | error | | yn | package.json | error | -## Unused devDependencies (7) +## Unused devDependencies (8) | Name | Location | Severity | | :----------------------------- | :----------- | :------- | @@ -25,6 +27,7 @@ | @types/passport-microsoft | package.json | error | | @types/passport-strategy | package.json | error | | @types/passport-github2 | package.json | error | +| @types/passport-auth0 | package.json | error | | @types/passport-saml | package.json | error | | @types/body-parser | package.json | error | | @types/xml2js | package.json | error | diff --git a/plugins/auth-node/knip-report.md b/plugins/auth-node/knip-report.md index 2338f48dd7..51c7faf1d8 100644 --- a/plugins/auth-node/knip-report.md +++ b/plugins/auth-node/knip-report.md @@ -2,8 +2,8 @@ ## Unused dependencies (2) -| Name | Location | Severity | -| :--------- | :----------- | :------- | -| node-fetch | package.json | error | -| winston | package.json | error | +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/backend-common | package.json | error | +| winston | package.json | error | diff --git a/plugins/catalog-backend-module-aws/knip-report.md b/plugins/catalog-backend-module-aws/knip-report.md index d42770729d..c637c25d2f 100644 --- a/plugins/catalog-backend-module-aws/knip-report.md +++ b/plugins/catalog-backend-module-aws/knip-report.md @@ -1,8 +1,15 @@ # Knip report -## Unused dependencies (1) +## Unused dependencies (2) | Name | Location | Severity | | :---------------------------- | :----------- | :------- | | @aws-sdk/credential-providers | package.json | error | +| winston | package.json | error | + +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | diff --git a/plugins/catalog-backend-module-azure/knip-report.md b/plugins/catalog-backend-module-azure/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-azure/knip-report.md +++ b/plugins/catalog-backend-module-azure/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-bitbucket-cloud/knip-report.md b/plugins/catalog-backend-module-bitbucket-cloud/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-bitbucket-cloud/knip-report.md +++ b/plugins/catalog-backend-module-bitbucket-cloud/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-bitbucket-server/knip-report.md b/plugins/catalog-backend-module-bitbucket-server/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-bitbucket-server/knip-report.md +++ b/plugins/catalog-backend-module-bitbucket-server/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-bitbucket/knip-report.md b/plugins/catalog-backend-module-bitbucket/knip-report.md deleted file mode 100644 index 7f1069ffcc..0000000000 --- a/plugins/catalog-backend-module-bitbucket/knip-report.md +++ /dev/null @@ -1,16 +0,0 @@ -# Knip report - -## Unused dependencies (1) - -| Name | Location | -|:--------------------------|:-------------| -| @backstage/backend-common | package.json | - -## Unused devDependencies (3) - -| Name | Location | -|:------------------------------|:-------------| -| msw | package.json | -| @backstage/cli | package.json | -| @backstage/backend-test-utils | package.json | - diff --git a/plugins/catalog-backend-module-gerrit/knip-report.md b/plugins/catalog-backend-module-gerrit/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-gerrit/knip-report.md +++ b/plugins/catalog-backend-module-gerrit/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-github-org/knip-report.md b/plugins/catalog-backend-module-github-org/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-github-org/knip-report.md +++ b/plugins/catalog-backend-module-github-org/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-github/knip-report.md b/plugins/catalog-backend-module-github/knip-report.md index 922f5fbf1d..1768d98608 100644 --- a/plugins/catalog-backend-module-github/knip-report.md +++ b/plugins/catalog-backend-module-github/knip-report.md @@ -6,11 +6,18 @@ | :-------------------------------- | :----------- | :------- | | @backstage/plugin-catalog-backend | package.json | error | -## Unlisted dependencies (3) +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + +## Unlisted dependencies (4) | Name | Location | Severity | | :---------------------- | :-------------------------------------------- | :------- | | @octokit/webhooks-types | src/providers/GithubMultiOrgEntityProvider.ts | error | +| @octokit/webhooks-types | src/providers/GithubEntityProvider.test.ts | error | | @octokit/webhooks-types | src/providers/GithubOrgEntityProvider.ts | error | | @octokit/webhooks-types | src/providers/GithubEntityProvider.ts | error | diff --git a/plugins/catalog-backend-module-gitlab/knip-report.md b/plugins/catalog-backend-module-gitlab/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-gitlab/knip-report.md +++ b/plugins/catalog-backend-module-gitlab/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-incremental-ingestion/knip-report.md b/plugins/catalog-backend-module-incremental-ingestion/knip-report.md index 2661c35327..21e59cdcea 100644 --- a/plugins/catalog-backend-module-incremental-ingestion/knip-report.md +++ b/plugins/catalog-backend-module-incremental-ingestion/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :---------------------------------- | :----------- | :------- | +| @backstage/plugin-permission-common | package.json | error | + diff --git a/plugins/catalog-backend-module-msgraph/knip-report.md b/plugins/catalog-backend-module-msgraph/knip-report.md index 2661c35327..cb5315967a 100644 --- a/plugins/catalog-backend-module-msgraph/knip-report.md +++ b/plugins/catalog-backend-module-msgraph/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-puppetdb/knip-report.md b/plugins/catalog-backend-module-puppetdb/knip-report.md index 2661c35327..acef584675 100644 --- a/plugins/catalog-backend-module-puppetdb/knip-report.md +++ b/plugins/catalog-backend-module-puppetdb/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-unprocessed/knip-report.md b/plugins/catalog-backend-module-unprocessed/knip-report.md index 2661c35327..fe81af1cb9 100644 --- a/plugins/catalog-backend-module-unprocessed/knip-report.md +++ b/plugins/catalog-backend-module-unprocessed/knip-report.md @@ -1,2 +1,9 @@ # Knip report +## Unused dependencies (2) + +| Name | Location | Severity | +| :----------------------------- | :----------- | :------- | +| @backstage/plugin-catalog-node | package.json | error | +| @backstage/plugin-auth-node | package.json | error | + diff --git a/plugins/catalog-backend/knip-report.md b/plugins/catalog-backend/knip-report.md index c7fec36b28..d547deea65 100644 --- a/plugins/catalog-backend/knip-report.md +++ b/plugins/catalog-backend/knip-report.md @@ -1,12 +1,5 @@ # Knip report -## Unused dependencies (2) - -| Name | Location | Severity | -| :-------------------- | :----------- | :------- | -| @backstage/repo-tools | package.json | error | -| node-fetch | package.json | error | - ## Unused devDependencies (1) | Name | Location | Severity | diff --git a/plugins/catalog-graph/knip-report.md b/plugins/catalog-graph/knip-report.md index bae463e7f2..118e62e2f9 100644 --- a/plugins/catalog-graph/knip-report.md +++ b/plugins/catalog-graph/knip-report.md @@ -6,9 +6,9 @@ | :------ | :----------- | :------- | | p-limit | package.json | error | -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/catalog-import/knip-report.md b/plugins/catalog-import/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/plugins/catalog-import/knip-report.md +++ b/plugins/catalog-import/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/catalog-react/knip-report.md b/plugins/catalog-react/knip-report.md index 2cfb7066e5..74674f0a9f 100644 --- a/plugins/catalog-react/knip-report.md +++ b/plugins/catalog-react/knip-report.md @@ -1,9 +1,14 @@ # Knip report -## Unused devDependencies (2) +## Unused devDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | -| react-test-renderer | package.json | error | +| Name | Location | Severity | +| :------------------ | :----------- | :------- | +| react-test-renderer | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/catalog-unprocessed-entities/knip-report.md b/plugins/catalog-unprocessed-entities/knip-report.md index ccda99cbfb..5fb4eceb90 100644 --- a/plugins/catalog-unprocessed-entities/knip-report.md +++ b/plugins/catalog-unprocessed-entities/knip-report.md @@ -6,3 +6,9 @@ | :--------------------- | :----------- | :------- | | @testing-library/react | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/catalog/knip-report.md b/plugins/catalog/knip-report.md index 6ca80a6843..466d6bf8bf 100644 --- a/plugins/catalog/knip-report.md +++ b/plugins/catalog/knip-report.md @@ -6,9 +6,9 @@ | :------ | :----------- | :------- | | history | package.json | error | -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/config-schema/knip-report.md b/plugins/config-schema/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/plugins/config-schema/knip-report.md +++ b/plugins/config-schema/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/devtools-backend/knip-report.md b/plugins/devtools-backend/knip-report.md index 2661c35327..0923e3006f 100644 --- a/plugins/devtools-backend/knip-report.md +++ b/plugins/devtools-backend/knip-report.md @@ -1,2 +1,9 @@ # Knip report +## Unused dependencies (2) + +| Name | Location | Severity | +| :----- | :----------- | :------- | +| semver | package.json | error | +| yn | package.json | error | + diff --git a/plugins/devtools/knip-report.md b/plugins/devtools/knip-report.md index 2661c35327..27a145e95c 100644 --- a/plugins/devtools/knip-report.md +++ b/plugins/devtools/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/events-backend/knip-report.md b/plugins/events-backend/knip-report.md index 2661c35327..7df8949921 100644 --- a/plugins/events-backend/knip-report.md +++ b/plugins/events-backend/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :------------------------- | :----------- | :------- | +| @backstage/backend-app-api | package.json | error | + diff --git a/plugins/events-node/knip-report.md b/plugins/events-node/knip-report.md index 2661c35327..3f80c71260 100644 --- a/plugins/events-node/knip-report.md +++ b/plugins/events-node/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| @backstage/types | package.json | error | + diff --git a/plugins/example-todo-list-backend/knip-report.md b/plugins/example-todo-list-backend/knip-report.md index 2661c35327..c84ed61781 100644 --- a/plugins/example-todo-list-backend/knip-report.md +++ b/plugins/example-todo-list-backend/knip-report.md @@ -1,2 +1,10 @@ # Knip report +## Unused dependencies (3) + +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/plugin-auth-node | package.json | error | +| @backstage/backend-common | package.json | error | +| yn | package.json | error | + diff --git a/plugins/example-todo-list/knip-report.md b/plugins/example-todo-list/knip-report.md index ba9c1dddc5..8265f7cba3 100644 --- a/plugins/example-todo-list/knip-report.md +++ b/plugins/example-todo-list/knip-report.md @@ -7,3 +7,9 @@ | @testing-library/react | package.json | error | | @testing-library/dom | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/home-react/knip-report.md b/plugins/home-react/knip-report.md index c9f34521a3..5eba7ba786 100644 --- a/plugins/home-react/knip-report.md +++ b/plugins/home-react/knip-report.md @@ -6,3 +6,9 @@ | :----------------------- | :----------- | :------- | | @types/react-grid-layout | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/home/knip-report.md b/plugins/home/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/plugins/home/knip-report.md +++ b/plugins/home/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/kubernetes-backend/knip-report.md b/plugins/kubernetes-backend/knip-report.md index 346cb90217..7804e0ecb8 100644 --- a/plugins/kubernetes-backend/knip-report.md +++ b/plugins/kubernetes-backend/knip-report.md @@ -1,15 +1,21 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (7) -| Name | Location | Severity | -| :------------- | :----------- | :------- | -| stream-buffers | package.json | error | -| morgan | package.json | error | +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/plugin-auth-node | package.json | error | +| stream-buffers | package.json | error | +| compression | package.json | error | +| helmet | package.json | error | +| morgan | package.json | error | +| cors | package.json | error | +| yn | package.json | error | -## Unused devDependencies (1) +## Unused devDependencies (2) -| Name | Location | Severity | -| :---------- | :----------- | :------- | -| @types/aws4 | package.json | error | +| Name | Location | Severity | +| :------------------------- | :----------- | :------- | +| @backstage/backend-app-api | package.json | error | +| @types/aws4 | package.json | error | diff --git a/plugins/kubernetes-cluster/knip-report.md b/plugins/kubernetes-cluster/knip-report.md index 1e57f8e9cb..4dfb36f550 100644 --- a/plugins/kubernetes-cluster/knip-report.md +++ b/plugins/kubernetes-cluster/knip-report.md @@ -10,10 +10,15 @@ | lodash | package.json | error | | luxon | package.json | error | -## Unused devDependencies (2) +## Unused devDependencies (1) -| Name | Location | Severity | -| :--------------------- | :----------- | :------- | -| @testing-library/react | package.json | error | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :------------------- | :----------- | :------- | +| @testing-library/dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/kubernetes-node/knip-report.md b/plugins/kubernetes-node/knip-report.md index 2661c35327..7df8949921 100644 --- a/plugins/kubernetes-node/knip-report.md +++ b/plugins/kubernetes-node/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :------------------------- | :----------- | :------- | +| @backstage/backend-app-api | package.json | error | + diff --git a/plugins/kubernetes-react/knip-report.md b/plugins/kubernetes-react/knip-report.md index 2661c35327..27a145e95c 100644 --- a/plugins/kubernetes-react/knip-report.md +++ b/plugins/kubernetes-react/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/kubernetes/knip-report.md b/plugins/kubernetes/knip-report.md index 1dcb32d0b6..3bbc096084 100644 --- a/plugins/kubernetes/knip-report.md +++ b/plugins/kubernetes/knip-report.md @@ -16,9 +16,9 @@ | luxon | package.json | error | | xterm | package.json | error | -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/notifications-backend/knip-report.md b/plugins/notifications-backend/knip-report.md index b1e6023b22..858e4d279e 100644 --- a/plugins/notifications-backend/knip-report.md +++ b/plugins/notifications-backend/knip-report.md @@ -1,10 +1,13 @@ # Knip report -## Unused dependencies (1) +## Unused dependencies (4) -| Name | Location | Severity | -| :--------- | :----------- | :------- | -| node-fetch | package.json | error | +| Name | Location | Severity | +| :---------------------------- | :----------- | :------- | +| @backstage/plugin-events-node | package.json | error | +| @backstage/plugin-auth-node | package.json | error | +| winston | package.json | error | +| yn | package.json | error | ## Unused devDependencies (1) diff --git a/plugins/notifications-node/knip-report.md b/plugins/notifications-node/knip-report.md index ab82a40e51..1206fc9c3b 100644 --- a/plugins/notifications-node/knip-report.md +++ b/plugins/notifications-node/knip-report.md @@ -1,11 +1,10 @@ # Knip report -## Unused dependencies (6) +## Unused dependencies (5) | Name | Location | Severity | | :----------------------------- | :----------- | :------- | | @backstage/plugin-signals-node | package.json | error | -| @backstage/backend-common | package.json | error | | @backstage/catalog-client | package.json | error | | @backstage/catalog-model | package.json | error | | knex | package.json | error | diff --git a/plugins/notifications/knip-report.md b/plugins/notifications/knip-report.md index 0dd3947f6a..07352d7523 100644 --- a/plugins/notifications/knip-report.md +++ b/plugins/notifications/knip-report.md @@ -7,11 +7,16 @@ | @backstage/types | package.json | error | | @material-ui/lab | package.json | error | -## Unused devDependencies (3) +## Unused devDependencies (2) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @testing-library/user-event | package.json | error | | @backstage/core-app-api | package.json | error | -| @testing-library/react | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/org-react/knip-report.md b/plugins/org-react/knip-report.md index 8d850953c7..61b4972034 100644 --- a/plugins/org-react/knip-report.md +++ b/plugins/org-react/knip-report.md @@ -1,8 +1,14 @@ # Knip report -## Unused devDependencies (1) +## Unused dependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/catalog-client | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/org/knip-report.md b/plugins/org/knip-report.md index 8d850953c7..7a38c994a2 100644 --- a/plugins/org/knip-report.md +++ b/plugins/org/knip-report.md @@ -2,7 +2,13 @@ ## Unused devDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :---------------------------------- | :----------- | :------- | +| @backstage/plugin-permission-common | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/permission-backend-module-policy-allow-all/knip-report.md b/plugins/permission-backend-module-policy-allow-all/knip-report.md index 14b2b4cc88..02ee898bec 100644 --- a/plugins/permission-backend-module-policy-allow-all/knip-report.md +++ b/plugins/permission-backend-module-policy-allow-all/knip-report.md @@ -1,5 +1,11 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/plugin-auth-node | package.json | error | + ## Unused devDependencies (1) | Name | Location | Severity | diff --git a/plugins/permission-react/knip-report.md b/plugins/permission-react/knip-report.md index 2661c35327..27a145e95c 100644 --- a/plugins/permission-react/knip-report.md +++ b/plugins/permission-react/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/proxy-backend/knip-report.md b/plugins/proxy-backend/knip-report.md index 90f5e90277..6f8dd76631 100644 --- a/plugins/proxy-backend/knip-report.md +++ b/plugins/proxy-backend/knip-report.md @@ -1,6 +1,6 @@ # Knip report -## Unused dependencies (4) +## Unused dependencies (5) | Name | Location | Severity | | :----- | :----------- | :------- | @@ -8,11 +8,12 @@ | uuid | package.json | error | | yaml | package.json | error | | yup | package.json | error | +| yn | package.json | error | ## Unused devDependencies (2) -| Name | Location | Severity | -| :---------- | :----------- | :------- | -| @types/uuid | package.json | error | -| @types/yup | package.json | error | +| Name | Location | Severity | +| :------------------------- | :----------- | :------- | +| @backstage/backend-app-api | package.json | error | +| @types/yup | package.json | error | diff --git a/plugins/scaffolder-backend-module-bitbucket-cloud/knip-report.md b/plugins/scaffolder-backend-module-bitbucket-cloud/knip-report.md index 8c142397d7..2661c35327 100644 --- a/plugins/scaffolder-backend-module-bitbucket-cloud/knip-report.md +++ b/plugins/scaffolder-backend-module-bitbucket-cloud/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Unused dependencies (1) - -| Name | Location | Severity | -| :------- | :----------- | :------- | -| fs-extra | package.json | error | - diff --git a/plugins/scaffolder-backend-module-bitbucket-server/knip-report.md b/plugins/scaffolder-backend-module-bitbucket-server/knip-report.md index a99d743140..2661c35327 100644 --- a/plugins/scaffolder-backend-module-bitbucket-server/knip-report.md +++ b/plugins/scaffolder-backend-module-bitbucket-server/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Unused dependencies (1) - -| Name | Location | Severity | -| :--- | :----------- | :------- | -| yaml | package.json | error | - diff --git a/plugins/scaffolder-backend-module-cookiecutter/knip-report.md b/plugins/scaffolder-backend-module-cookiecutter/knip-report.md index 30b061a5ef..0e8b43df56 100644 --- a/plugins/scaffolder-backend-module-cookiecutter/knip-report.md +++ b/plugins/scaffolder-backend-module-cookiecutter/knip-report.md @@ -1,9 +1,10 @@ # Knip report -## Unused dependencies (2) +## Unused dependencies (3) -| Name | Location | Severity | -| :------ | :----------- | :------- | -| winston | package.json | error | -| yn | package.json | error | +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/backend-defaults | package.json | error | +| winston | package.json | error | +| yn | package.json | error | diff --git a/plugins/scaffolder-backend-module-gcp/knip-report.md b/plugins/scaffolder-backend-module-gcp/knip-report.md index 2661c35327..5eace4cfc6 100644 --- a/plugins/scaffolder-backend-module-gcp/knip-report.md +++ b/plugins/scaffolder-backend-module-gcp/knip-report.md @@ -1,2 +1,15 @@ # Knip report +## Unused dependencies (2) + +| Name | Location | Severity | +| :--------------------- | :----------- | :------- | +| @backstage/integration | package.json | error | +| @backstage/errors | package.json | error | + +## Unused devDependencies (1) + +| Name | Location | Severity | +| :------------------------------------------- | :----------- | :------- | +| @backstage/plugin-scaffolder-node-test-utils | package.json | error | + diff --git a/plugins/scaffolder-backend-module-gitea/knip-report.md b/plugins/scaffolder-backend-module-gitea/knip-report.md index a5a6c0f115..2661c35327 100644 --- a/plugins/scaffolder-backend-module-gitea/knip-report.md +++ b/plugins/scaffolder-backend-module-gitea/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Unused dependencies (1) - -| Name | Location | Severity | -| :--------- | :----------- | :------- | -| node-fetch | package.json | error | - diff --git a/plugins/scaffolder-backend-module-github/knip-report.md b/plugins/scaffolder-backend-module-github/knip-report.md index f6ffc37c84..d153d4dd55 100644 --- a/plugins/scaffolder-backend-module-github/knip-report.md +++ b/plugins/scaffolder-backend-module-github/knip-report.md @@ -2,7 +2,7 @@ ## Unlisted dependencies (1) -| Name | Location | Severity | -| :----------------------------- | :--------------------- | :------- | -| @octokit/core/dist-types/types | src/actions/helpers.ts | error | +| Name | Location | Severity | +| :------------ | :--------------------- | :------- | +| @octokit/core | src/actions/helpers.ts | error | diff --git a/plugins/scaffolder-backend-module-gitlab/knip-report.md b/plugins/scaffolder-backend-module-gitlab/knip-report.md index 2661c35327..c955ca23f0 100644 --- a/plugins/scaffolder-backend-module-gitlab/knip-report.md +++ b/plugins/scaffolder-backend-module-gitlab/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :------ | :----------- | :------- | +| winston | package.json | error | + diff --git a/plugins/scaffolder-backend-module-sentry/knip-report.md b/plugins/scaffolder-backend-module-sentry/knip-report.md index a99d743140..2661c35327 100644 --- a/plugins/scaffolder-backend-module-sentry/knip-report.md +++ b/plugins/scaffolder-backend-module-sentry/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Unused dependencies (1) - -| Name | Location | Severity | -| :--- | :----------- | :------- | -| yaml | package.json | error | - diff --git a/plugins/scaffolder-backend-module-yeoman/knip-report.md b/plugins/scaffolder-backend-module-yeoman/knip-report.md index 7b2c9043ee..c955ca23f0 100644 --- a/plugins/scaffolder-backend-module-yeoman/knip-report.md +++ b/plugins/scaffolder-backend-module-yeoman/knip-report.md @@ -6,9 +6,3 @@ | :------ | :----------- | :------- | | winston | package.json | error | -## Unused devDependencies (1) - -| Name | Location | Severity | -| :------------------------ | :----------- | :------- | -| @backstage/backend-common | package.json | error | - diff --git a/plugins/scaffolder-backend/knip-report.md b/plugins/scaffolder-backend/knip-report.md index 6b1996899f..51ea7441f5 100644 --- a/plugins/scaffolder-backend/knip-report.md +++ b/plugins/scaffolder-backend/knip-report.md @@ -1,8 +1,17 @@ # Knip report -## Unused dependencies (1) +## Unused dependencies (4) -| Name | Location | Severity | -| :------ | :----------- | :------- | -| p-limit | package.json | error | +| Name | Location | Severity | +| :--------------------------------------- | :----------- | :------- | +| @backstage/plugin-bitbucket-cloud-common | package.json | error | +| concat-stream | package.json | error | +| p-limit | package.json | error | +| tar | package.json | error | + +## Unused devDependencies (1) + +| Name | Location | Severity | +| :------------------------- | :----------- | :------- | +| @backstage/backend-app-api | package.json | error | diff --git a/plugins/scaffolder-node-test-utils/knip-report.md b/plugins/scaffolder-node-test-utils/knip-report.md index 11aea47050..0459ceb140 100644 --- a/plugins/scaffolder-node-test-utils/knip-report.md +++ b/plugins/scaffolder-node-test-utils/knip-report.md @@ -1,8 +1,11 @@ # Knip report -## Unused devDependencies (1) +## Unused devDependencies (4) -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| react-router-dom | package.json | error | +| @types/react | package.json | error | +| react-dom | package.json | error | +| react | package.json | error | diff --git a/plugins/scaffolder-react/knip-report.md b/plugins/scaffolder-react/knip-report.md index 3c2e8f3c3c..7893bde707 100644 --- a/plugins/scaffolder-react/knip-report.md +++ b/plugins/scaffolder-react/knip-report.md @@ -1,17 +1,28 @@ # Knip report -## Unused dependencies (3) +## Unused dependencies (1) | Name | Location | Severity | | :------------------------ | :----------- | :------- | | @backstage/catalog-client | package.json | error | -| zod-to-json-schema | package.json | error | -| zod | package.json | error | -## Unused devDependencies (2) +## Unused devDependencies (1) | Name | Location | Severity | | :------------------------------- | :----------- | :------- | | @backstage/plugin-catalog-common | package.json | error | -| @testing-library/dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + +## Unlisted dependencies (3) + +| Name | Location | Severity | +| :------------------- | :--------------------------------------------------- | :------- | +| @internal/scaffolder | src/next/extensions/createScaffolderFormDecorator.ts | error | +| @internal/scaffolder | src/next/blueprints/FormFieldBlueprint.tsx | error | +| @internal/scaffolder | src/next/api/FormFieldsApi.ts | error | diff --git a/plugins/scaffolder/knip-report.md b/plugins/scaffolder/knip-report.md index 6004665f89..7eff004077 100644 --- a/plugins/scaffolder/knip-report.md +++ b/plugins/scaffolder/knip-report.md @@ -8,9 +8,15 @@ | @rjsf/material-ui | package.json | error | | git-url-parse | package.json | error | -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + +## Unlisted dependencies (1) + +| Name | Location | Severity | +| :------------------- | :----------------------------------- | :------- | +| @internal/scaffolder | src/alpha/hooks/useFormDecorators.ts | error | diff --git a/plugins/search-backend-module-catalog/knip-report.md b/plugins/search-backend-module-catalog/knip-report.md index 2661c35327..5a09f2cded 100644 --- a/plugins/search-backend-module-catalog/knip-report.md +++ b/plugins/search-backend-module-catalog/knip-report.md @@ -1,2 +1,9 @@ # Knip report +## Unused devDependencies (2) + +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/backend-common | package.json | error | +| msw | package.json | error | + diff --git a/plugins/search-backend-module-pg/knip-report.md b/plugins/search-backend-module-pg/knip-report.md index 2661c35327..2fbb979fec 100644 --- a/plugins/search-backend-module-pg/knip-report.md +++ b/plugins/search-backend-module-pg/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused dependencies (1) + +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/backend-common | package.json | error | + diff --git a/plugins/search-backend-node/knip-report.md b/plugins/search-backend-node/knip-report.md index 2661c35327..ec020eab9a 100644 --- a/plugins/search-backend-node/knip-report.md +++ b/plugins/search-backend-node/knip-report.md @@ -1,2 +1,8 @@ # Knip report +## Unused devDependencies (1) + +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/backend-common | package.json | error | + diff --git a/plugins/search-backend/knip-report.md b/plugins/search-backend/knip-report.md index 2661c35327..90e0425294 100644 --- a/plugins/search-backend/knip-report.md +++ b/plugins/search-backend/knip-report.md @@ -1,2 +1,9 @@ # Knip report +## Unused dependencies (2) + +| Name | Location | Severity | +| :-------------------------------- | :----------- | :------- | +| @backstage/plugin-permission-node | package.json | error | +| yn | package.json | error | + diff --git a/plugins/search-react/knip-report.md b/plugins/search-react/knip-report.md index e4c2d5b55c..95b2395ba4 100644 --- a/plugins/search-react/knip-report.md +++ b/plugins/search-react/knip-report.md @@ -1,9 +1,14 @@ # Knip report -## Unused devDependencies (2) +## Unused devDependencies (1) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @backstage/frontend-app-api | package.json | error | -| @testing-library/dom | package.json | error | + +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/search/knip-report.md b/plugins/search/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/plugins/search/knip-report.md +++ b/plugins/search/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/signals-backend/knip-report.md b/plugins/signals-backend/knip-report.md index b809a13e13..8f8d01a437 100644 --- a/plugins/signals-backend/knip-report.md +++ b/plugins/signals-backend/knip-report.md @@ -1,12 +1,13 @@ # Knip report -## Unused dependencies (3) +## Unused dependencies (4) -| Name | Location | Severity | -| :-------------------- | :----------- | :------- | -| http-proxy-middleware | package.json | error | -| @backstage/config | package.json | error | -| node-fetch | package.json | error | +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @backstage/plugin-auth-node | package.json | error | +| http-proxy-middleware | package.json | error | +| winston | package.json | error | +| yn | package.json | error | ## Unused devDependencies (1) diff --git a/plugins/signals-node/knip-report.md b/plugins/signals-node/knip-report.md index 24da0710d8..34f37bfdb1 100644 --- a/plugins/signals-node/knip-report.md +++ b/plugins/signals-node/knip-report.md @@ -1,11 +1,10 @@ # Knip report -## Unused dependencies (6) +## Unused dependencies (5) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @backstage/plugin-auth-node | package.json | error | -| @backstage/backend-common | package.json | error | | @backstage/config | package.json | error | | express | package.json | error | | uuid | package.json | error | diff --git a/plugins/signals-react/knip-report.md b/plugins/signals-react/knip-report.md index 0d1d20d45b..55c2adfde5 100644 --- a/plugins/signals-react/knip-report.md +++ b/plugins/signals-react/knip-report.md @@ -13,3 +13,9 @@ | @testing-library/react | package.json | error | | @backstage/test-utils | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/signals/knip-report.md b/plugins/signals/knip-report.md index 31b0ba28ea..785af5a7c6 100644 --- a/plugins/signals/knip-report.md +++ b/plugins/signals/knip-report.md @@ -8,13 +8,17 @@ | @material-ui/lab | package.json | error | | react-use | package.json | error | -## Unused devDependencies (5) +## Unused devDependencies (3) | Name | Location | Severity | | :-------------------------- | :----------- | :------- | | @testing-library/user-event | package.json | error | | @backstage/core-app-api | package.json | error | -| @testing-library/react | package.json | error | -| @backstage/test-utils | package.json | error | | msw | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/techdocs-addons-test-utils/knip-report.md b/plugins/techdocs-addons-test-utils/knip-report.md index ca9b81ba38..97a4eafef0 100644 --- a/plugins/techdocs-addons-test-utils/knip-report.md +++ b/plugins/techdocs-addons-test-utils/knip-report.md @@ -12,3 +12,9 @@ | :------------------- | :----------- | :------- | | @testing-library/dom | package.json | error | +## Referenced optional peerDependencies (1) + +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | + diff --git a/plugins/techdocs-module-addons-contrib/knip-report.md b/plugins/techdocs-module-addons-contrib/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/plugins/techdocs-module-addons-contrib/knip-report.md +++ b/plugins/techdocs-module-addons-contrib/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/techdocs-react/knip-report.md b/plugins/techdocs-react/knip-report.md index 0f171c4d8f..8288639520 100644 --- a/plugins/techdocs-react/knip-report.md +++ b/plugins/techdocs-react/knip-report.md @@ -1,8 +1,15 @@ # Knip report -## Unused dependencies (1) +## Unused dependencies (2) + +| Name | Location | Severity | +| :------------------------- | :----------- | :------- | +| @backstage/core-components | package.json | error | +| react-helmet | package.json | error | + +## Referenced optional peerDependencies (1) | Name | Location | Severity | | :----------- | :----------- | :------- | -| react-helmet | package.json | error | +| @types/react | package.json | error | diff --git a/plugins/techdocs/knip-report.md b/plugins/techdocs/knip-report.md index a2b0f2bc7d..fb09d78e25 100644 --- a/plugins/techdocs/knip-report.md +++ b/plugins/techdocs/knip-report.md @@ -6,10 +6,9 @@ | :-- | :----------- | :------- | | jss | package.json | error | -## Unused devDependencies (2) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | -| canvas | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | diff --git a/plugins/user-settings-backend/knip-report.md b/plugins/user-settings-backend/knip-report.md index 2661c35327..9a27c5bad6 100644 --- a/plugins/user-settings-backend/knip-report.md +++ b/plugins/user-settings-backend/knip-report.md @@ -1,2 +1,9 @@ # Knip report +## Unused dependencies (2) + +| Name | Location | Severity | +| :---------------- | :----------- | :------- | +| @backstage/config | package.json | error | +| yn | package.json | error | + diff --git a/plugins/user-settings/knip-report.md b/plugins/user-settings/knip-report.md index 8d850953c7..27a145e95c 100644 --- a/plugins/user-settings/knip-report.md +++ b/plugins/user-settings/knip-report.md @@ -1,8 +1,8 @@ # Knip report -## Unused devDependencies (1) +## Referenced optional peerDependencies (1) -| Name | Location | Severity | -| :------------------- | :----------- | :------- | -| @testing-library/dom | package.json | error | +| Name | Location | Severity | +| :----------- | :----------- | :------- | +| @types/react | package.json | error | From 84fd95c76973f7daded3107fa57bbe728b5d9dc4 Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Wed, 15 Jan 2025 15:04:38 +0100 Subject: [PATCH 3/8] Ignore optional peerDependencies Let's figure out later what triggers those Signed-off-by: Axel Hecht --- packages/app-defaults/knip-report.md | 6 ------ packages/app-next-example-plugin/knip-report.md | 6 ------ packages/core-app-api/knip-report.md | 6 ------ packages/core-compat-api/knip-report.md | 6 ------ packages/core-components/knip-report.md | 6 ------ packages/core-plugin-api/knip-report.md | 6 ------ packages/dev-utils/knip-report.md | 6 ------ packages/frontend-app-api/knip-report.md | 6 ------ packages/frontend-plugin-api/knip-report.md | 6 ------ packages/frontend-test-utils/knip-report.md | 6 ------ packages/integration-react/knip-report.md | 6 ------ .../repo-tools/src/commands/knip-reports/knip-extractor.ts | 4 ++++ packages/test-utils/knip-report.md | 7 ------- packages/theme/knip-report.md | 6 ------ packages/version-bridge/knip-report.md | 6 ------ plugins/api-docs-module-protoc-gen-doc/knip-report.md | 6 ------ plugins/api-docs/knip-report.md | 6 ------ plugins/app-visualizer/knip-report.md | 6 ------ plugins/catalog-graph/knip-report.md | 6 ------ plugins/catalog-import/knip-report.md | 6 ------ plugins/catalog-react/knip-report.md | 6 ------ plugins/catalog-unprocessed-entities/knip-report.md | 6 ------ plugins/catalog/knip-report.md | 6 ------ plugins/config-schema/knip-report.md | 6 ------ plugins/devtools/knip-report.md | 6 ------ plugins/example-todo-list/knip-report.md | 6 ------ plugins/home-react/knip-report.md | 6 ------ plugins/home/knip-report.md | 6 ------ plugins/kubernetes-cluster/knip-report.md | 6 ------ plugins/kubernetes-react/knip-report.md | 6 ------ plugins/kubernetes/knip-report.md | 6 ------ plugins/notifications/knip-report.md | 6 ------ plugins/org-react/knip-report.md | 6 ------ plugins/org/knip-report.md | 6 ------ plugins/permission-react/knip-report.md | 6 ------ plugins/scaffolder-node-test-utils/knip-report.md | 3 +-- plugins/scaffolder-react/knip-report.md | 6 ------ plugins/scaffolder/knip-report.md | 6 ------ plugins/search-react/knip-report.md | 6 ------ plugins/search/knip-report.md | 6 ------ plugins/signals-react/knip-report.md | 6 ------ plugins/signals/knip-report.md | 6 ------ plugins/techdocs-addons-test-utils/knip-report.md | 6 ------ plugins/techdocs-module-addons-contrib/knip-report.md | 6 ------ plugins/techdocs-react/knip-report.md | 6 ------ plugins/techdocs/knip-report.md | 6 ------ plugins/user-settings/knip-report.md | 6 ------ 47 files changed, 5 insertions(+), 273 deletions(-) diff --git a/packages/app-defaults/knip-report.md b/packages/app-defaults/knip-report.md index 27a145e95c..2661c35327 100644 --- a/packages/app-defaults/knip-report.md +++ b/packages/app-defaults/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/app-next-example-plugin/knip-report.md b/packages/app-next-example-plugin/knip-report.md index 54dc66d59e..e01cd18c93 100644 --- a/packages/app-next-example-plugin/knip-report.md +++ b/packages/app-next-example-plugin/knip-report.md @@ -7,9 +7,3 @@ | cross-fetch | package.json | error | | msw | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/core-app-api/knip-report.md b/packages/core-app-api/knip-report.md index 010afdd8b4..76616b6d60 100644 --- a/packages/core-app-api/knip-report.md +++ b/packages/core-app-api/knip-report.md @@ -16,9 +16,3 @@ | react-router-stable | package.json | error | | react-router-beta | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/core-compat-api/knip-report.md b/packages/core-compat-api/knip-report.md index d6bf611772..7d3c3a7c03 100644 --- a/packages/core-compat-api/knip-report.md +++ b/packages/core-compat-api/knip-report.md @@ -8,12 +8,6 @@ | @backstage/types | package.json | error | | zod | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (2) | Name | Location | Severity | diff --git a/packages/core-components/knip-report.md b/packages/core-components/knip-report.md index 6d8a954385..69f3218a24 100644 --- a/packages/core-components/knip-report.md +++ b/packages/core-components/knip-report.md @@ -14,12 +14,6 @@ | @types/google-protobuf | package.json | error | | cross-fetch | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (3) | Name | Location | Severity | diff --git a/packages/core-plugin-api/knip-report.md b/packages/core-plugin-api/knip-report.md index 20e35cfd7d..169117772f 100644 --- a/packages/core-plugin-api/knip-report.md +++ b/packages/core-plugin-api/knip-report.md @@ -6,9 +6,3 @@ | :-------------------------- | :----------- | :------- | | @testing-library/user-event | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/dev-utils/knip-report.md b/packages/dev-utils/knip-report.md index 27a145e95c..2661c35327 100644 --- a/packages/dev-utils/knip-report.md +++ b/packages/dev-utils/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/frontend-app-api/knip-report.md b/packages/frontend-app-api/knip-report.md index 517e88ce8a..69cb2648b7 100644 --- a/packages/frontend-app-api/knip-report.md +++ b/packages/frontend-app-api/knip-report.md @@ -8,12 +8,6 @@ | @backstage/errors | package.json | error | | zod | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (3) | Name | Location | Severity | diff --git a/packages/frontend-plugin-api/knip-report.md b/packages/frontend-plugin-api/knip-report.md index 43b78effd5..462c1fd03b 100644 --- a/packages/frontend-plugin-api/knip-report.md +++ b/packages/frontend-plugin-api/knip-report.md @@ -14,12 +14,6 @@ | :-------------------------- | :----------- | :------- | | @backstage/frontend-app-api | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (5) | Name | Location | Severity | diff --git a/packages/frontend-test-utils/knip-report.md b/packages/frontend-test-utils/knip-report.md index 5b35805816..5d03d0a51c 100644 --- a/packages/frontend-test-utils/knip-report.md +++ b/packages/frontend-test-utils/knip-report.md @@ -7,12 +7,6 @@ | @backstage/version-bridge | package.json | error | | zod | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (1) | Name | Location | Severity | diff --git a/packages/integration-react/knip-report.md b/packages/integration-react/knip-report.md index 9ccde91b0c..551f19ab2f 100644 --- a/packages/integration-react/knip-report.md +++ b/packages/integration-react/knip-report.md @@ -7,9 +7,3 @@ | @testing-library/dom | package.json | error | | msw | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index 645d084017..5238a951c9 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -83,6 +83,10 @@ async function generateKnipConfig({ knipConfigPath }: KnipConfigOptions) { '{fixtures,migrations,templates}/**', ], ignoreDependencies: [ + // these is reported as a referenced optional peerDependencies + // TBD: investigate what triggers these + '@types/react', + '@types/jest', '@backstage/cli', // everything depends on this for its package.json commands '@backstage/theme', // this uses `declare module` in .d.ts so is implicitly used whenever extensions are needed ], diff --git a/packages/test-utils/knip-report.md b/packages/test-utils/knip-report.md index 1b48f703a1..2661c35327 100644 --- a/packages/test-utils/knip-report.md +++ b/packages/test-utils/knip-report.md @@ -1,9 +1,2 @@ # Knip report -## Referenced optional peerDependencies (2) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | -| @types/jest | package.json | error | - diff --git a/packages/theme/knip-report.md b/packages/theme/knip-report.md index c5f148fbd3..9aa0ae1d46 100644 --- a/packages/theme/knip-report.md +++ b/packages/theme/knip-report.md @@ -6,9 +6,3 @@ | :--------------- | :----------- | :------- | | react-router-dom | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/packages/version-bridge/knip-report.md b/packages/version-bridge/knip-report.md index c5f148fbd3..9aa0ae1d46 100644 --- a/packages/version-bridge/knip-report.md +++ b/packages/version-bridge/knip-report.md @@ -6,9 +6,3 @@ | :--------------- | :----------- | :------- | | react-router-dom | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/api-docs-module-protoc-gen-doc/knip-report.md b/plugins/api-docs-module-protoc-gen-doc/knip-report.md index 9743763ca9..63b63f2f9b 100644 --- a/plugins/api-docs-module-protoc-gen-doc/knip-report.md +++ b/plugins/api-docs-module-protoc-gen-doc/knip-report.md @@ -7,9 +7,3 @@ | react-router-dom | package.json | error | | react-dom | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/api-docs/knip-report.md b/plugins/api-docs/knip-report.md index 27e141bb8c..3c06f53535 100644 --- a/plugins/api-docs/knip-report.md +++ b/plugins/api-docs/knip-report.md @@ -15,9 +15,3 @@ | @backstage/core-app-api | package.json | error | | @types/highlightjs | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/app-visualizer/knip-report.md b/plugins/app-visualizer/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/app-visualizer/knip-report.md +++ b/plugins/app-visualizer/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/catalog-graph/knip-report.md b/plugins/catalog-graph/knip-report.md index 118e62e2f9..6b1996899f 100644 --- a/plugins/catalog-graph/knip-report.md +++ b/plugins/catalog-graph/knip-report.md @@ -6,9 +6,3 @@ | :------ | :----------- | :------- | | p-limit | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/catalog-import/knip-report.md b/plugins/catalog-import/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/catalog-import/knip-report.md +++ b/plugins/catalog-import/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/catalog-react/knip-report.md b/plugins/catalog-react/knip-report.md index 74674f0a9f..4b33c02956 100644 --- a/plugins/catalog-react/knip-report.md +++ b/plugins/catalog-react/knip-report.md @@ -6,9 +6,3 @@ | :------------------ | :----------- | :------- | | react-test-renderer | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/catalog-unprocessed-entities/knip-report.md b/plugins/catalog-unprocessed-entities/knip-report.md index 5fb4eceb90..ccda99cbfb 100644 --- a/plugins/catalog-unprocessed-entities/knip-report.md +++ b/plugins/catalog-unprocessed-entities/knip-report.md @@ -6,9 +6,3 @@ | :--------------------- | :----------- | :------- | | @testing-library/react | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/catalog/knip-report.md b/plugins/catalog/knip-report.md index 466d6bf8bf..7ceae94282 100644 --- a/plugins/catalog/knip-report.md +++ b/plugins/catalog/knip-report.md @@ -6,9 +6,3 @@ | :------ | :----------- | :------- | | history | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/config-schema/knip-report.md b/plugins/config-schema/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/config-schema/knip-report.md +++ b/plugins/config-schema/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/devtools/knip-report.md b/plugins/devtools/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/devtools/knip-report.md +++ b/plugins/devtools/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/example-todo-list/knip-report.md b/plugins/example-todo-list/knip-report.md index 8265f7cba3..ba9c1dddc5 100644 --- a/plugins/example-todo-list/knip-report.md +++ b/plugins/example-todo-list/knip-report.md @@ -7,9 +7,3 @@ | @testing-library/react | package.json | error | | @testing-library/dom | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/home-react/knip-report.md b/plugins/home-react/knip-report.md index 5eba7ba786..c9f34521a3 100644 --- a/plugins/home-react/knip-report.md +++ b/plugins/home-react/knip-report.md @@ -6,9 +6,3 @@ | :----------------------- | :----------- | :------- | | @types/react-grid-layout | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/home/knip-report.md b/plugins/home/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/home/knip-report.md +++ b/plugins/home/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/kubernetes-cluster/knip-report.md b/plugins/kubernetes-cluster/knip-report.md index 4dfb36f550..a5a618460d 100644 --- a/plugins/kubernetes-cluster/knip-report.md +++ b/plugins/kubernetes-cluster/knip-report.md @@ -16,9 +16,3 @@ | :------------------- | :----------- | :------- | | @testing-library/dom | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/kubernetes-react/knip-report.md b/plugins/kubernetes-react/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/kubernetes-react/knip-report.md +++ b/plugins/kubernetes-react/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/kubernetes/knip-report.md b/plugins/kubernetes/knip-report.md index 3bbc096084..a20fd463aa 100644 --- a/plugins/kubernetes/knip-report.md +++ b/plugins/kubernetes/knip-report.md @@ -16,9 +16,3 @@ | luxon | package.json | error | | xterm | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/notifications/knip-report.md b/plugins/notifications/knip-report.md index 07352d7523..da21340672 100644 --- a/plugins/notifications/knip-report.md +++ b/plugins/notifications/knip-report.md @@ -14,9 +14,3 @@ | @testing-library/user-event | package.json | error | | @backstage/core-app-api | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/org-react/knip-report.md b/plugins/org-react/knip-report.md index 61b4972034..b5cb1d8b22 100644 --- a/plugins/org-react/knip-report.md +++ b/plugins/org-react/knip-report.md @@ -6,9 +6,3 @@ | :------------------------ | :----------- | :------- | | @backstage/catalog-client | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/org/knip-report.md b/plugins/org/knip-report.md index 7a38c994a2..d1f8bacd53 100644 --- a/plugins/org/knip-report.md +++ b/plugins/org/knip-report.md @@ -6,9 +6,3 @@ | :---------------------------------- | :----------- | :------- | | @backstage/plugin-permission-common | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/permission-react/knip-report.md b/plugins/permission-react/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/permission-react/knip-report.md +++ b/plugins/permission-react/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/scaffolder-node-test-utils/knip-report.md b/plugins/scaffolder-node-test-utils/knip-report.md index 0459ceb140..beefceb792 100644 --- a/plugins/scaffolder-node-test-utils/knip-report.md +++ b/plugins/scaffolder-node-test-utils/knip-report.md @@ -1,11 +1,10 @@ # Knip report -## Unused devDependencies (4) +## Unused devDependencies (3) | Name | Location | Severity | | :--------------- | :----------- | :------- | | react-router-dom | package.json | error | -| @types/react | package.json | error | | react-dom | package.json | error | | react | package.json | error | diff --git a/plugins/scaffolder-react/knip-report.md b/plugins/scaffolder-react/knip-report.md index 7893bde707..982fa32d6f 100644 --- a/plugins/scaffolder-react/knip-report.md +++ b/plugins/scaffolder-react/knip-report.md @@ -12,12 +12,6 @@ | :------------------------------- | :----------- | :------- | | @backstage/plugin-catalog-common | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (3) | Name | Location | Severity | diff --git a/plugins/scaffolder/knip-report.md b/plugins/scaffolder/knip-report.md index 7eff004077..caf0b9001b 100644 --- a/plugins/scaffolder/knip-report.md +++ b/plugins/scaffolder/knip-report.md @@ -8,12 +8,6 @@ | @rjsf/material-ui | package.json | error | | git-url-parse | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - ## Unlisted dependencies (1) | Name | Location | Severity | diff --git a/plugins/search-react/knip-report.md b/plugins/search-react/knip-report.md index 95b2395ba4..7f6a4d4ccd 100644 --- a/plugins/search-react/knip-report.md +++ b/plugins/search-react/knip-report.md @@ -6,9 +6,3 @@ | :-------------------------- | :----------- | :------- | | @backstage/frontend-app-api | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/search/knip-report.md b/plugins/search/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/search/knip-report.md +++ b/plugins/search/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/signals-react/knip-report.md b/plugins/signals-react/knip-report.md index 55c2adfde5..0d1d20d45b 100644 --- a/plugins/signals-react/knip-report.md +++ b/plugins/signals-react/knip-report.md @@ -13,9 +13,3 @@ | @testing-library/react | package.json | error | | @backstage/test-utils | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/signals/knip-report.md b/plugins/signals/knip-report.md index 785af5a7c6..e9c2c167b1 100644 --- a/plugins/signals/knip-report.md +++ b/plugins/signals/knip-report.md @@ -16,9 +16,3 @@ | @backstage/core-app-api | package.json | error | | msw | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/techdocs-addons-test-utils/knip-report.md b/plugins/techdocs-addons-test-utils/knip-report.md index 97a4eafef0..ca9b81ba38 100644 --- a/plugins/techdocs-addons-test-utils/knip-report.md +++ b/plugins/techdocs-addons-test-utils/knip-report.md @@ -12,9 +12,3 @@ | :------------------- | :----------- | :------- | | @testing-library/dom | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/techdocs-module-addons-contrib/knip-report.md b/plugins/techdocs-module-addons-contrib/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/techdocs-module-addons-contrib/knip-report.md +++ b/plugins/techdocs-module-addons-contrib/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/techdocs-react/knip-report.md b/plugins/techdocs-react/knip-report.md index 8288639520..651079cda4 100644 --- a/plugins/techdocs-react/knip-report.md +++ b/plugins/techdocs-react/knip-report.md @@ -7,9 +7,3 @@ | @backstage/core-components | package.json | error | | react-helmet | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/techdocs/knip-report.md b/plugins/techdocs/knip-report.md index fb09d78e25..2b0b502c71 100644 --- a/plugins/techdocs/knip-report.md +++ b/plugins/techdocs/knip-report.md @@ -6,9 +6,3 @@ | :-- | :----------- | :------- | | jss | package.json | error | -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - diff --git a/plugins/user-settings/knip-report.md b/plugins/user-settings/knip-report.md index 27a145e95c..2661c35327 100644 --- a/plugins/user-settings/knip-report.md +++ b/plugins/user-settings/knip-report.md @@ -1,8 +1,2 @@ # Knip report -## Referenced optional peerDependencies (1) - -| Name | Location | Severity | -| :----------- | :----------- | :------- | -| @types/react | package.json | error | - From 5e66a252a6130038e0db5eb0f31ef53a3e02887f Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Wed, 15 Jan 2025 15:56:09 +0100 Subject: [PATCH 4/8] Move ignores into workspace to fix templates Signed-off-by: Axel Hecht --- packages/cli/knip-report.md | 20 +++---- packages/create-app/knip-report.md | 59 ------------------- .../commands/knip-reports/knip-extractor.ts | 18 +++--- 3 files changed, 16 insertions(+), 81 deletions(-) diff --git a/packages/cli/knip-report.md b/packages/cli/knip-report.md index 60ccba435c..bfb5537db1 100644 --- a/packages/cli/knip-report.md +++ b/packages/cli/knip-report.md @@ -58,18 +58,12 @@ | @rspack/dev-server | package.json | error | | @rspack/core | package.json | error | -## Unlisted dependencies (10) +## Unlisted dependencies (4) -| Name | Location | Severity | -| :--------------------- | :----------------------------------------------------------------------------------------------- | :------- | -| @testing-library/react | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.test.tsx | error | -| react | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.test.tsx | error | -| @material-ui/core | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.tsx | error | -| react | templates/default-react-plugin-package/src/components/ExampleComponent/ExampleComponent.tsx | error | -| supertest | templates/default-backend-plugin/src/router.test.ts | error | -| express-promise-router | templates/default-backend-plugin/src/router.ts | error | -| react-dom | src/lib/bundler/hasReactDomClient.ts | error | -| react-dom | src/lib/bundler/config.ts | error | -| react | src/lib/bundler/config.ts | error | -| react | src/lib/bundler/server.ts | error | +| Name | Location | Severity | +| :-------- | :----------------------------------- | :------- | +| react-dom | src/lib/bundler/hasReactDomClient.ts | error | +| react-dom | src/lib/bundler/config.ts | error | +| react | src/lib/bundler/config.ts | error | +| react | src/lib/bundler/server.ts | error | diff --git a/packages/create-app/knip-report.md b/packages/create-app/knip-report.md index 0afa527ab0..864a5956ef 100644 --- a/packages/create-app/knip-report.md +++ b/packages/create-app/knip-report.md @@ -6,62 +6,3 @@ | :-------------------- | :----------- | :------- | | @types/command-exists | package.json | error | -## Unlisted dependencies (54) - -| Name | Location | Severity | -| :----------------------------------------------- | :----------------------------------------------------------------------- | :------- | -| @backstage/plugin-techdocs-module-addons-contrib | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-techdocs-react | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-catalog-graph | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-kubernetes | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-api-docs | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-techdocs | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/core-components | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-catalog | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/catalog-model | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-org | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @material-ui/core | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| react | templates/default-app/packages/app/src/components/catalog/EntityPage.tsx | error | -| @backstage/plugin-catalog-react | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @backstage/plugin-search-react | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @backstage/plugin-techdocs | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @backstage/core-components | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @backstage/core-plugin-api | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @backstage/plugin-catalog | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @backstage/plugin-search | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @material-ui/core | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| react | templates/default-app/packages/app/src/components/search/SearchPage.tsx | error | -| @material-ui/core | templates/default-app/packages/app/src/components/Root/LogoFull.tsx | error | -| @material-ui/core | templates/default-app/packages/app/src/components/Root/LogoIcon.tsx | error | -| react | templates/default-app/packages/app/src/components/Root/LogoFull.tsx | error | -| react | templates/default-app/packages/app/src/components/Root/LogoIcon.tsx | error | -| @backstage/plugin-user-settings | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| @backstage/core-components | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| @backstage/plugin-search | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| @backstage/plugin-org | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| @material-ui/icons | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| @material-ui/core | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| react | templates/default-app/packages/app/src/components/Root/Root.tsx | error | -| @testing-library/react | templates/default-app/packages/app/src/App.test.tsx | error | -| react | templates/default-app/packages/app/src/App.test.tsx | error | -| @backstage/plugin-techdocs-module-addons-contrib | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-permission-react | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-catalog-import | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-techdocs-react | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-catalog-common | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-user-settings | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-catalog-graph | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-scaffolder | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/integration-react | templates/default-app/packages/app/src/apis.ts | error | -| @backstage/plugin-api-docs | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-techdocs | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/core-components | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/core-plugin-api | templates/default-app/packages/app/src/apis.ts | error | -| @backstage/plugin-catalog | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-search | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/app-defaults | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/core-app-api | templates/default-app/packages/app/src/App.tsx | error | -| @backstage/plugin-org | templates/default-app/packages/app/src/App.tsx | error | -| react-router-dom | templates/default-app/packages/app/src/App.tsx | error | -| react | templates/default-app/packages/app/src/App.tsx | error | - diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index 5238a951c9..019d6a7ebc 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -68,20 +68,20 @@ async function generateKnipConfig({ knipConfigPath }: KnipConfigOptions) { 'src/routes.ts', 'src/run.ts', ], + ignore: [ + '.eslintrc.js', + 'config.d.ts', + 'knexfile.js', + 'node_modules/**', + 'dist/**', + '{fixtures,migrations,templates}/**', + ], }, }, jest: { - entry: ['src/setupTests.ts', '**/*.test.{ts,tsx}'], + entry: ['src/setupTests.ts', 'src/**/*.test.{ts,tsx}'], }, storybook: { entry: 'src/components/**/*.stories.tsx' }, - ignore: [ - '.eslintrc.js', - 'config.d.ts', - 'knexfile.js', - 'node_modules/**', - 'dist/**', - '{fixtures,migrations,templates}/**', - ], ignoreDependencies: [ // these is reported as a referenced optional peerDependencies // TBD: investigate what triggers these From f4d0a921430a113ab571dfaf8c4c20525fdea0a0 Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Wed, 15 Jan 2025 21:29:19 +0100 Subject: [PATCH 5/8] Slim down entries, relying on package.json exports Signed-off-by: Axel Hecht --- .../src/commands/knip-reports/knip-extractor.ts | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index 019d6a7ebc..d7110eec5b 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -61,13 +61,7 @@ async function generateKnipConfig({ knipConfigPath }: KnipConfigOptions) { workspaces: { '.': {}, '{packages,plugins}/*': { - entry: [ - 'dev/index.{ts,tsx}', - 'src/index.{ts,tsx}', - 'src/alpha.{ts,tsx}', - 'src/routes.ts', - 'src/run.ts', - ], + entry: ['dev/index.{ts,tsx}', 'src/index.{ts,tsx}'], ignore: [ '.eslintrc.js', 'config.d.ts', From 2815a4bb069b92f3689d7056c4168ea518e22fc8 Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Thu, 16 Jan 2025 11:12:40 +0100 Subject: [PATCH 6/8] Ignore internal non-published packages Using just the naming convention for now. Signed-off-by: Axel Hecht --- packages/core-compat-api/knip-report.md | 7 ------- packages/frontend-app-api/knip-report.md | 8 -------- packages/frontend-plugin-api/knip-report.md | 10 ---------- packages/frontend-test-utils/knip-report.md | 6 ------ .../src/commands/knip-reports/knip-extractor.ts | 1 + plugins/scaffolder-react/knip-report.md | 8 -------- plugins/scaffolder/knip-report.md | 6 ------ 7 files changed, 1 insertion(+), 45 deletions(-) diff --git a/packages/core-compat-api/knip-report.md b/packages/core-compat-api/knip-report.md index 7d3c3a7c03..a5794ee10b 100644 --- a/packages/core-compat-api/knip-report.md +++ b/packages/core-compat-api/knip-report.md @@ -8,10 +8,3 @@ | @backstage/types | package.json | error | | zod | package.json | error | -## Unlisted dependencies (2) - -| Name | Location | Severity | -| :----------------- | :------------------------------- | :------- | -| @internal/frontend | src/collectLegacyRoutes.test.tsx | error | -| @internal/frontend | src/convertLegacyPlugin.test.tsx | error | - diff --git a/packages/frontend-app-api/knip-report.md b/packages/frontend-app-api/knip-report.md index 69cb2648b7..b050438d7c 100644 --- a/packages/frontend-app-api/knip-report.md +++ b/packages/frontend-app-api/knip-report.md @@ -8,11 +8,3 @@ | @backstage/errors | package.json | error | | zod | package.json | error | -## Unlisted dependencies (3) - -| Name | Location | Severity | -| :----------------- | :---------------------------------- | :------- | -| @internal/frontend | src/wiring/createSpecializedApp.tsx | error | -| @internal/frontend | src/tree/resolveAppNodeSpecs.ts | error | -| @internal/frontend | src/routing/collectRouteIds.ts | error | - diff --git a/packages/frontend-plugin-api/knip-report.md b/packages/frontend-plugin-api/knip-report.md index 462c1fd03b..402fea3d47 100644 --- a/packages/frontend-plugin-api/knip-report.md +++ b/packages/frontend-plugin-api/knip-report.md @@ -14,13 +14,3 @@ | :-------------------------- | :----------- | :------- | | @backstage/frontend-app-api | package.json | error | -## Unlisted dependencies (5) - -| Name | Location | Severity | -| :----------------- | :------------------------------------------- | :------- | -| @internal/frontend | src/wiring/createExtensionBlueprint.test.tsx | error | -| @internal/frontend | src/wiring/resolveExtensionDefinition.ts | error | -| @internal/frontend | src/wiring/createFrontendModule.ts | error | -| @internal/frontend | src/wiring/createFrontendPlugin.ts | error | -| @internal/frontend | src/wiring/createExtension.ts | error | - diff --git a/packages/frontend-test-utils/knip-report.md b/packages/frontend-test-utils/knip-report.md index 5d03d0a51c..50f4f788e3 100644 --- a/packages/frontend-test-utils/knip-report.md +++ b/packages/frontend-test-utils/knip-report.md @@ -7,9 +7,3 @@ | @backstage/version-bridge | package.json | error | | zod | package.json | error | -## Unlisted dependencies (1) - -| Name | Location | Severity | -| :----------------- | :-------------------------------- | :------- | -| @internal/frontend | src/app/createExtensionTester.tsx | error | - diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index d7110eec5b..f2cd6e513e 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -81,6 +81,7 @@ async function generateKnipConfig({ knipConfigPath }: KnipConfigOptions) { // TBD: investigate what triggers these '@types/react', '@types/jest', + '@internal/.*', // internal packages are not published and inlined '@backstage/cli', // everything depends on this for its package.json commands '@backstage/theme', // this uses `declare module` in .d.ts so is implicitly used whenever extensions are needed ], diff --git a/plugins/scaffolder-react/knip-report.md b/plugins/scaffolder-react/knip-report.md index 982fa32d6f..38b9eed013 100644 --- a/plugins/scaffolder-react/knip-report.md +++ b/plugins/scaffolder-react/knip-report.md @@ -12,11 +12,3 @@ | :------------------------------- | :----------- | :------- | | @backstage/plugin-catalog-common | package.json | error | -## Unlisted dependencies (3) - -| Name | Location | Severity | -| :------------------- | :--------------------------------------------------- | :------- | -| @internal/scaffolder | src/next/extensions/createScaffolderFormDecorator.ts | error | -| @internal/scaffolder | src/next/blueprints/FormFieldBlueprint.tsx | error | -| @internal/scaffolder | src/next/api/FormFieldsApi.ts | error | - diff --git a/plugins/scaffolder/knip-report.md b/plugins/scaffolder/knip-report.md index caf0b9001b..02a697ddb0 100644 --- a/plugins/scaffolder/knip-report.md +++ b/plugins/scaffolder/knip-report.md @@ -8,9 +8,3 @@ | @rjsf/material-ui | package.json | error | | git-url-parse | package.json | error | -## Unlisted dependencies (1) - -| Name | Location | Severity | -| :------------------- | :----------------------------------- | :------- | -| @internal/scaffolder | src/alpha/hooks/useFormDecorators.ts | error | - From 45c37538f55d9ccb4cb8d563bb2d584e3f3554c8 Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Thu, 16 Jan 2025 11:53:15 +0100 Subject: [PATCH 7/8] Add missing reports, only skip canon Canon uses a different setup for storybook Signed-off-by: Axel Hecht --- packages/frontend-defaults/knip-report.md | 2 ++ packages/frontend-internal/knip-report.md | 20 ++++++++++++++ packages/opaque-internal/knip-report.md | 9 +++++++ .../commands/knip-reports/knip-extractor.ts | 6 +++-- packages/scaffolder-internal/knip-report.md | 2 ++ .../techdocs-cli-embedded-app/knip-report.md | 26 ++++++++----------- packages/yarn-plugin/knip-report.md | 2 ++ plugins/app/knip-report.md | 19 ++++++++++++++ .../knip-report.md | 2 ++ .../knip-report.md | 8 ++++++ .../knip-report.md | 9 +++++++ .../knip-report.md | 8 ++++++ plugins/auth-react/knip-report.md | 8 ++++++ .../knip-report.md | 8 ++++++ .../knip-report.md | 2 ++ .../knip-report.md | 2 ++ .../knip-report.md | 8 ++++++ plugins/proxy-node/knip-report.md | 9 +++++++ .../knip-report.md | 8 ++++++ plugins/techdocs-common/knip-report.md | 2 ++ plugins/user-settings-common/knip-report.md | 2 ++ 21 files changed, 145 insertions(+), 17 deletions(-) create mode 100644 packages/frontend-defaults/knip-report.md create mode 100644 packages/frontend-internal/knip-report.md create mode 100644 packages/opaque-internal/knip-report.md create mode 100644 packages/scaffolder-internal/knip-report.md create mode 100644 packages/yarn-plugin/knip-report.md create mode 100644 plugins/app/knip-report.md create mode 100644 plugins/auth-backend-module-auth0-provider/knip-report.md create mode 100644 plugins/auth-backend-module-azure-easyauth-provider/knip-report.md create mode 100644 plugins/auth-backend-module-bitbucket-provider/knip-report.md create mode 100644 plugins/auth-backend-module-bitbucket-server-provider/knip-report.md create mode 100644 plugins/auth-react/knip-report.md create mode 100644 plugins/catalog-backend-module-gitlab-org/knip-report.md create mode 100644 plugins/catalog-backend-module-logs/knip-report.md create mode 100644 plugins/catalog-unprocessed-entities-common/knip-report.md create mode 100644 plugins/notifications-backend-module-email/knip-report.md create mode 100644 plugins/proxy-node/knip-report.md create mode 100644 plugins/scaffolder-backend-module-notifications/knip-report.md create mode 100644 plugins/techdocs-common/knip-report.md create mode 100644 plugins/user-settings-common/knip-report.md diff --git a/packages/frontend-defaults/knip-report.md b/packages/frontend-defaults/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/packages/frontend-defaults/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/packages/frontend-internal/knip-report.md b/packages/frontend-internal/knip-report.md new file mode 100644 index 0000000000..d5272513d9 --- /dev/null +++ b/packages/frontend-internal/knip-report.md @@ -0,0 +1,20 @@ +# Knip report + +## Unused dependencies (3) + +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @backstage/version-bridge | package.json | error | +| @backstage/types | package.json | error | +| zod | package.json | error | + +## Unused devDependencies (5) + +| Name | Location | Severity | +| :----------------------------- | :----------- | :------- | +| @backstage/frontend-test-utils | package.json | error | +| @backstage/frontend-app-api | package.json | error | +| @testing-library/jest-dom | package.json | error | +| @testing-library/react | package.json | error | +| @backstage/test-utils | package.json | error | + diff --git a/packages/opaque-internal/knip-report.md b/packages/opaque-internal/knip-report.md new file mode 100644 index 0000000000..b131ea8948 --- /dev/null +++ b/packages/opaque-internal/knip-report.md @@ -0,0 +1,9 @@ +# Knip report + +## Unused devDependencies (2) + +| Name | Location | Severity | +| :------------------------ | :----------- | :------- | +| @testing-library/jest-dom | package.json | error | +| @testing-library/react | package.json | error | + diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index f2cd6e513e..b8bf720bb8 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -21,8 +21,10 @@ import fs from 'fs-extra'; import type { KnipConfig } from 'knip'; import { createBinRunner } from '../util'; -// Ignore this due to Knip error: Error: ENAMETOOLONG: name too long, scandir -const ignoredPackages = ['packages/techdocs-cli-embedded-app']; +// Ignore these +const ignoredPackages = [ + 'packages/canon', // storybook config is different from the rest +]; interface KnipExtractionOptions { packageDirs: string[]; diff --git a/packages/scaffolder-internal/knip-report.md b/packages/scaffolder-internal/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/packages/scaffolder-internal/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/packages/techdocs-cli-embedded-app/knip-report.md b/packages/techdocs-cli-embedded-app/knip-report.md index 87e4093961..96aa361a87 100644 --- a/packages/techdocs-cli-embedded-app/knip-report.md +++ b/packages/techdocs-cli-embedded-app/knip-report.md @@ -1,21 +1,17 @@ # Knip report -## Unused dependencies (4) +## Unused dependencies (2) -| Name | Location | -|:----------------------|:-------------| -| history | package.json | -| react-use | package.json | -| @backstage/theme | package.json | -| @backstage/test-utils | package.json | +| Name | Location | Severity | +| :-------- | :----------- | :------- | +| react-use | package.json | error | +| history | package.json | error | -## Unused devDependencies (5) +## Unused devDependencies (3) -| Name | Location | -|:----------------------------|:-------------| -| cross-env | package.json | -| @testing-library/dom | package.json | -| @testing-library/react | package.json | -| @testing-library/jest-dom | package.json | -| @testing-library/user-event | package.json | +| Name | Location | Severity | +| :-------------------------- | :----------- | :------- | +| @testing-library/user-event | package.json | error | +| @testing-library/dom | package.json | error | +| cross-env | package.json | error | diff --git a/packages/yarn-plugin/knip-report.md b/packages/yarn-plugin/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/packages/yarn-plugin/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/plugins/app/knip-report.md b/plugins/app/knip-report.md new file mode 100644 index 0000000000..2206dd4cf9 --- /dev/null +++ b/plugins/app/knip-report.md @@ -0,0 +1,19 @@ +# Knip report + +## Unused dependencies (2) + +| Name | Location | Severity | +| :--------------- | :----------- | :------- | +| @material-ui/lab | package.json | error | +| react-use | package.json | error | + +## Unused devDependencies (5) + +| Name | Location | Severity | +| :----------------------------- | :----------- | :------- | +| @backstage/frontend-test-utils | package.json | error | +| @testing-library/user-event | package.json | error | +| @testing-library/react | package.json | error | +| @backstage/dev-utils | package.json | error | +| msw | package.json | error | + diff --git a/plugins/auth-backend-module-auth0-provider/knip-report.md b/plugins/auth-backend-module-auth0-provider/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/plugins/auth-backend-module-auth0-provider/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/plugins/auth-backend-module-azure-easyauth-provider/knip-report.md b/plugins/auth-backend-module-azure-easyauth-provider/knip-report.md new file mode 100644 index 0000000000..d5c786591f --- /dev/null +++ b/plugins/auth-backend-module-azure-easyauth-provider/knip-report.md @@ -0,0 +1,8 @@ +# Knip report + +## Unused dependencies (1) + +| Name | Location | Severity | +| :----------------------- | :----------- | :------- | +| @backstage/catalog-model | package.json | error | + diff --git a/plugins/auth-backend-module-bitbucket-provider/knip-report.md b/plugins/auth-backend-module-bitbucket-provider/knip-report.md new file mode 100644 index 0000000000..6624c86073 --- /dev/null +++ b/plugins/auth-backend-module-bitbucket-provider/knip-report.md @@ -0,0 +1,9 @@ +# Knip report + +## Unused dependencies (2) + +| Name | Location | Severity | +| :------- | :----------- | :------- | +| passport | package.json | error | +| express | package.json | error | + diff --git a/plugins/auth-backend-module-bitbucket-server-provider/knip-report.md b/plugins/auth-backend-module-bitbucket-server-provider/knip-report.md new file mode 100644 index 0000000000..4386ad9c4a --- /dev/null +++ b/plugins/auth-backend-module-bitbucket-server-provider/knip-report.md @@ -0,0 +1,8 @@ +# Knip report + +## Unused dependencies (1) + +| Name | Location | Severity | +| :------- | :----------- | :------- | +| passport | package.json | error | + diff --git a/plugins/auth-react/knip-report.md b/plugins/auth-react/knip-report.md new file mode 100644 index 0000000000..df86118651 --- /dev/null +++ b/plugins/auth-react/knip-report.md @@ -0,0 +1,8 @@ +# Knip report + +## Unused devDependencies (1) + +| Name | Location | Severity | +| :-- | :----------- | :------- | +| msw | package.json | error | + diff --git a/plugins/catalog-backend-module-gitlab-org/knip-report.md b/plugins/catalog-backend-module-gitlab-org/knip-report.md new file mode 100644 index 0000000000..cb5315967a --- /dev/null +++ b/plugins/catalog-backend-module-gitlab-org/knip-report.md @@ -0,0 +1,8 @@ +# Knip report + +## Unused devDependencies (1) + +| Name | Location | Severity | +| :---- | :----------- | :------- | +| luxon | package.json | error | + diff --git a/plugins/catalog-backend-module-logs/knip-report.md b/plugins/catalog-backend-module-logs/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/plugins/catalog-backend-module-logs/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/plugins/catalog-unprocessed-entities-common/knip-report.md b/plugins/catalog-unprocessed-entities-common/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/plugins/catalog-unprocessed-entities-common/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/plugins/notifications-backend-module-email/knip-report.md b/plugins/notifications-backend-module-email/knip-report.md new file mode 100644 index 0000000000..38a3e4ea8a --- /dev/null +++ b/plugins/notifications-backend-module-email/knip-report.md @@ -0,0 +1,8 @@ +# Knip report + +## Unused dependencies (1) + +| Name | Location | Severity | +| :------------- | :----------- | :------- | +| @aws-sdk/types | package.json | error | + diff --git a/plugins/proxy-node/knip-report.md b/plugins/proxy-node/knip-report.md new file mode 100644 index 0000000000..e7921e52b4 --- /dev/null +++ b/plugins/proxy-node/knip-report.md @@ -0,0 +1,9 @@ +# Knip report + +## Unused devDependencies (2) + +| Name | Location | Severity | +| :---------------------------- | :----------- | :------- | +| @backstage/backend-test-utils | package.json | error | +| @backstage/config | package.json | error | + diff --git a/plugins/scaffolder-backend-module-notifications/knip-report.md b/plugins/scaffolder-backend-module-notifications/knip-report.md new file mode 100644 index 0000000000..9ac53047a7 --- /dev/null +++ b/plugins/scaffolder-backend-module-notifications/knip-report.md @@ -0,0 +1,8 @@ +# Knip report + +## Unused dependencies (1) + +| Name | Location | Severity | +| :------ | :----------- | :------- | +| octokit | package.json | error | + diff --git a/plugins/techdocs-common/knip-report.md b/plugins/techdocs-common/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/plugins/techdocs-common/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + diff --git a/plugins/user-settings-common/knip-report.md b/plugins/user-settings-common/knip-report.md new file mode 100644 index 0000000000..2661c35327 --- /dev/null +++ b/plugins/user-settings-common/knip-report.md @@ -0,0 +1,2 @@ +# Knip report + From 281ef4007a269d6984752c9705ac2bea5b182b1b Mon Sep 17 00:00:00 2001 From: Axel Hecht Date: Thu, 16 Jan 2025 15:38:47 +0100 Subject: [PATCH 8/8] Fix for cli packaging tests Signed-off-by: Axel Hecht --- packages/repo-tools/src/commands/knip-reports/knip-extractor.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts index b8bf720bb8..763989f9c7 100644 --- a/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts +++ b/packages/repo-tools/src/commands/knip-reports/knip-extractor.ts @@ -71,6 +71,7 @@ async function generateKnipConfig({ knipConfigPath }: KnipConfigOptions) { 'node_modules/**', 'dist/**', '{fixtures,migrations,templates}/**', + 'src/tests/transforms/__fixtures__/**', // cli packaging tests ], }, },