chore(deps): update dependency jscpd to v5.3.1 - #304
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
This PR contains the following updates:
5.3.0→5.3.1Release Notes
kucherenko/jscpd (jscpd)
v5.3.1Compare Source
New Features
--dashboardlists the largest code files. The Project section now ranks files by lines, with their tokens and size, next to the largest formats — the files worth splitting, beside the most complex ones further down. Only code is ranked: a lockfile, a changelog or a long HTML page is often the longest file in a repository and nothing anyone would refactor, so prose, data and markup files are left out, as they are from the complexity list.--summary-topsets the rows, and thejson,markdownandhtmlreporters carry the same list (project.largestFilesinjscpd-dashboard.json, an added key). Seefixtures/dashboard-demo. (#1086)basta now detects which frameworks a project uses. A framework runs files that nothing imports: a router's pages, a runtime's plugin directories, handlers it finds by convention. basta used to know three such cases, hard-coded: Nuxt, Nitro and WXT. It now reads them from a list of about fifty definitions built into the binary,
frameworks.yaml. The list covers Next.js, Nuxt, Nitro, WXT, Plasmo, Remix, React Router, SvelteKit, Astro, SolidStart, TanStack Start, Qwik City, Gatsby, RedwoodJS, Angular, NestJS, AdonisJS, Sails,@fastify/autoload, Strapi, Medusa, Ember, Quasar, React Native, Expo, Cloudflare Workers, Vercel, Netlify, Serverless, Trigger.dev, Docusaurus, VitePress, Eleventy, Storybook, Histoire, Jest, Vitest, Mocha, AVA, Playwright, Cypress, Cucumber, Prisma, Knex, Sequelize, TypeORM, Hardhat and Create React App.package.jsondependencies, or its section inpackage.json. basta checks every directory that holds scanned files, so each package of a monorepo is treated as its own project.srcDir,entrypointsDir,appDirectory,sourceRootinnest-cli.json, andimports: false.Frameworks: next (apps/web), vitest. Bothbastaandjscpd --dead-codeprint it.bastaflags:--list-frameworksprints the list.--framework <name>treats a framework as present, which helps when the scan starts below itspackage.json.--no-frameworksturns detection off.--frameworks-config <file>loads your own definitions, in YAML or JSON, in the same shape as the built-in list. basta also picks upbasta.frameworks.{yaml,yml,json}from the working directory. Your definitions can add a framework basta does not know, or replace a built-in one that has the same name. If the file fails to load, basta stops with an error, because running without it would report as dead the very files the definitions protect.globals. The built-in list hasgetServerSideProps,generateMetadataand the route segment config for Next;loader,actionandmetafor Remix and React Router;load,actions,prerenderand the hooks for SvelteKit;getStaticPathsfor Astro; the Gatsby Node, browser and SSR APIs; the lifecycle methods of Angular and Nest;onRequest*for Cloudflare Pages Functions;handlerfor Netlify and Serverless functions; and more. basta treats a declaration with such a name as used. It never reports it, including under--include-entry-exportsand as an unused member, and everything the declaration calls stays reachable.globalsapplies to every file of the project. The{ names, files }form limits names to the files the framework reads them from, soloadercounts for Remix under the app directory and is an ordinary name everywhere else. This also fixes a React Router route named fromroutes.ts. basta reaches that file through a string, so it is not an entry point, and itsloaderused to be reported as an unused export.srcDirand treatstasks/as loaded by the framework.fixtures/dead-code-demo. (#1087).jscpd.jsoncan hold dead-code settings in a section of their own. ThedeadCodekey (also spelleddead-codeorbasta) used to be a boolean that turned the mode on. It can now also be an object with these keys:enabled,categories,minConfidence,minLines,entry,ignore,includeTests,includeEntryExports,threshold, and the framework settingsframeworks(definitions written inline, in the shape offrameworks.yaml),frameworksConfig,frameworkandnoFrameworks."enabled": true, so you can keep clone settings and dead-code settings in one file and choose the run on the command line."deadCode": truestill works as the switch.minConfidence,entry,deadCodeCategories, …), which still work.minLinesandthresholdexist only inside the section, because the top-level keys with those names are about clones. The top-levelminLinesis the smallest clone to report and is never applied to a dead-code run. The top-levelthresholdis a limit for duplicated lines. A dead-code run still falls back to it when the section sets nothreshold, as before.jscpd --dead-codeand--dashboardnow also pick upbasta.frameworks.{yaml,yml,json}from the working directory.bastabinary reads the same section from the same file. It has a new-c, --config <file>flag. Without it, basta looks in the working directory for.jscpd.json,.config/jscpd.jsonorpackage.json. Both tools give the same result when you start them in the same directory.fixtures/dead-code-demo. (#1087)Fixes
We ran basta next to knip and fallow on 55 repositories: the GitHub trending lists for JavaScript, TypeScript, Vue, Svelte and Astro, plus the source code of Nuxt, Next.js, Svelte, Gatsby and Astro. The run found these problems in basta, all fixed in [#1087](#1087:
--dead-codewas very slow on projects with many path aliases. basta tested every import against every alias in the project, and it did the slow check first: comparing the importer's path with the directory of the config that declared the alias, one path segment at a time. A monorepo with 150 packages, each listing a few hundredpaths, took 198 s (ever-gauzy). Next.js took 83 s. basta now checks first whether the alias pattern matches the import at all, which fails on the first character for almost every alias. The same two projects now take 3.3 s and 4.4 s, and the findings are byte for byte the same."*": ["./*", "../../node_modules/*"], an import ofreactwent through the resolver from every file, and each miss built and hashed about twenty candidate paths. The module index can now tell in two lookups whether a path could name any module, under any extension or as a directory index, and the answer is exact. LibreChat went from 4.0 s to 0.86 s with identical findings.'@': '/src'alias was read as the root of the file system. In a Vite alias target a leading/means the project root, as it does in a URL, and many projects write the alias that way. basta lost every@/import in such a project and reported its components as unused files at 95% confidence. MoeKoeMusic went from 55 unused files to 0.require(`./x`)with backticks was ignored. A template literal with nothing interpolated is as static as a quoted string, and Gatsby writes every string that way. Gatsby went from 584 unused files to 352.<script on:load={() => { … }} src=…>, basta took the>of=>for the end of the tag and passed the rest of the attribute to the JavaScript parser as the component's script. The file failed to parse, and everything it imports was reported as unused. cobalt went from 37 unused files to 8.admin.py,apps.py, template tags, and the modules that settings name by dotted path. Django, Alembic and Scrapy are now in the framework list, detected bymanage.py,alembic.iniandscrapy.cfg. AdventureLog went from 144 unused Python files to 6.resolve(distDir, 'runtime/handlers/island')is how a framework registers a file it loads itself. It is not an import, and basta does not treat it as one. But a file whose own path ends the same way now gets a new reason,path-appears-in-string("its path appears in a string literal"), which costs 40 points. That puts the file under the default threshold of 60, and it still shows up with--min-confidence 0. Nuxt went from 83 unused files to 46. The reason is a new possible value ofreasonsin the JSON report.--dead-coderead a WXT browser extension as almost entirely dead (#1082): the framework'sentrypoints/directory — background, content scripts, popup pages — was not recognized as entry points, and its@/~→srcDir,@@/~~→ root aliases live only in the generated.wxt/tsconfig.json, which no repository commits, so the whole tree dangled and cascaded (Tencent/BrowserSkill: 27.9% "unused", 115 unused files — now 0.4% and 5). Awxt.config.*now marks the entrypoints and auto-import directories as entries, honoringsrcDir,entrypointsDirandimports: false, and declares the conventional aliases, the same waynuxt.config.*roots Nuxt's directories andsvelte.config.*supplies$lib. (#1083)if,foror a media query'sandas branches — an HTML page could top the "Most complex files" list. These formats now have complexity0, like prose and data files already did, and are therefore no longer counted as code by the health score: an all-markup project reports "no code files" instead of being scored on its markup. Component formats (Vue, Svelte, Astro) still count in full through their script blocks. The markup block of a component file (tokenized ashtml) is now also excluded from the duplication share, matching thecss/scssblocks that already were. (#1081) (#1084)Other
bastaon npm, crates.io and GitHub.Published Packages
basta@0.2.0on crates.iocpd-core@0.1.16on crates.iocpd-finder@0.1.17on crates.iocpd-reporter@0.1.17on crates.iocpd-tokenizer@0.1.17on crates.iojscpd@5.3.1on crates.iocpd@5.3.1on npmjscpd@5.3.1on npmjscpd-darwin-arm64@5.3.1on npmjscpd-darwin-x64@5.3.1on npmjscpd-linux-x64-gnu@5.3.1on npmjscpd-linux-arm64-gnu@5.3.1on npmjscpd-linux-x64-musl@5.3.1on npmjscpd-linux-arm64-musl@5.3.1on npmjscpd-windows-x64-msvc@5.3.1on npmjscpd-windows-arm64-msvc@5.3.1on npmjscpd==5.3.1on PyPIVerify
Archives are signed with Sigstore (keyless,
<asset>.sigstore.json)and carry SLSA build provenance. Replace
jscpd-linux-x64-gnu.tar.gzwith your asset:cosign verify-blob \ --bundle jscpd-linux-x64-gnu.tar.gz.sigstore.json \ --certificate-identity-regexp '^https://github\.com/kucherenko/jscpd/' \ --certificate-oidc-issuer https://token.actions.githubusercontent.com \ jscpd-linux-x64-gnu.tar.gz gh attestation verify jscpd-linux-x64-gnu.tar.gz --repo kucherenko/jscpd sha256sum --check --ignore-missing checksums.txtConfiguration
📅 Schedule: (UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.