mirror of
https://github.com/kucherenko/jscpd.git
synced 2026-09-19 08:11:03 +08:00
apk
f516710e28
Copilot's review of #1062 raised seven points; each checked out and each is fixed with a test that fails without it. - A glob keeps its pattern. `import.meta.glob('./locales/*.js')` and import(`./pages/${name}.vue`) used to keep every file under the directory alive; they now reach what the pattern matches, `*` staying within one segment as it does for the bundler. On Firefly an image glob, '../../**/*.{png,jpg}', had been keeping all of src/ alive. - Every pattern of an `import.meta.glob([...])` array is kept, not the first. - A declared alias is not second-guessed: when `@` points at src/ and the file is missing there, the `@/` project-root fallback no longer finds a different file. A catch-all `*` path claims no prefix and does not count. - Astro client <script> blocks are read as the modules Astro bundles them into, so what they import is used. A script with any attribute but `src` is left to the browser, as Astro leaves it. - import('./x') in markup is found only inside expressions, not in comments or plain text. - A quoted attribute value is an expression only on a directive (`:prop`, `@click`, `#slot`, `v-*`); elsewhere only its `{...}` interpolations are, so `class="card"` no longer credits a declaration named `card`. - An object key in markup, `{ label: value }`, is not a read. Re-measuring on the thirty trending projects also exposed `?raw` imports, as in `import script from './panzoom.js?raw'`: a bundler query now is stripped before resolving. On one snapshot of those projects the previous release reports 1093 unused files and this branch 481, with the share some import names falling from 83% to 62%; the CHANGELOG now quotes those numbers instead of ones measured against a development baseline. Components used only from .mdx content are still reported, and the docs say so.