Skip to content

chore: migrate to biome v2#67

Merged
alexkli merged 1 commit into
mainfrom
biome2
Apr 29, 2026
Merged

chore: migrate to biome v2#67
alexkli merged 1 commit into
mainfrom
biome2

Conversation

@alexkli
Copy link
Copy Markdown
Contributor

@alexkli alexkli commented Apr 29, 2026

Summary

  • Upgrade @biomejs/biome 1.9.4 → 2.4.13
  • Migrate biome.json to v2 schema: includes (with negated globs) replaces ignore, override includeincludes with **/ prefix per v2's new glob semantics, recommended: true replaces removed "all": true, drop noConsoleLog (merged into noConsole), disable noTemplateCurlyInString (false positives on edgly's own ${{VAR}} templating syntax)
  • Remove unused biome-ignore comments and fix <explanation> placeholders that v2 now flags
  • Minor lint autofixes: import sorting, useless regex escapes, parseInt radix

Test plan

  • npm run lint passes
  • npm run lint-ci passes
  • npm test passes (16/16)

🤖 Generated with Claude Code

- upgrade @biomejs/biome 1.9.4 -> 2.4.13
- update biome.json schema: files.includes/formatter.includes with
  negated globs, overrides[].includes with **/ prefix per v2 glob
  semantics, recommended preset (replaces removed "all": true), drop
  noConsoleLog (merged into noConsole), disable noTemplateCurlyInString
  (false positives on edgly's own ${{VAR}} templating syntax)
- remove unused biome-ignore comments and fix <explanation> placeholders
- minor lint autofixes: import sorting, useless regex escapes,
  parseInt radix

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

This PR will trigger no release when merged.

@adobe-bot
Copy link
Copy Markdown

⚠️ Sizewatcher detected a size increase 📈:

git -0.1% (881 kB => 880 kB)
Largest files:
Largest files in repository checkout:

302KiB package-lock.json
182KiB src/secrets/patterns.yaml
102KiB devflow.svg
20KiB README.md
19KiB src/fastly/api/fastly-api.js
17KiB src/fastly/store.js
12KiB test/secrets.test.js
11KiB src/fastly/service-mgr.js
11KiB LICENSE
10KiB src/test/runner.js

Largest files among new changes:

302KiB package-lock.json
19KiB src/fastly/api/fastly-api.js
17KiB src/fastly/store.js
12KiB test/secrets.test.js
10KiB src/test/runner.js
9.5KiB src/secrets/secrets.js
8.7KiB src/fastly/fiddle-mgr.js
6.2KiB src/yargs-ahoy.js
3.8KiB test/test.test.js
3.6KiB src/commands/service/update.js
3.1KiB src/commands/service/get.js
3.0KiB src/commands/service/create.js
1.9KiB package.json
1.7KiB biome.json
⚠️ node_modules +33.0% (163 MB => 217 MB)
Largest production node modules:
@adobe/edgly@1.4.2 (46 deps, 7.73mb, 998 files, ©undefined)
╭────────────────────────────┬──────────────┬──────────┬───────┬───────────┬───────────────┬───────────╮
│ Name │ Dependencies │ Size │ Files │ Native │ License │ Deprec │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ glob@11.1.0 │ 17 │ 4.93mb │ 421 │ │ BlueOak-1.0.0 │ │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ yaml@2.8.3 │ │ 668.57kb │ 233 │ │ ISC │ │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ yargs@17.7.2 │ 17 │ 629.89kb │ 173 │ │ MIT │ │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ @adobe/fetch@4.3.0 │ 4 │ 369.88kb │ 45 │ │ Apache-2.0 │ │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ formdata-node@6.0.3 │ │ 124.64kb │ 16 │ │ MIT │ │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ chalk@5.6.2 │ │ 43.3kb │ 12 │ │ MIT │ │
├────────────────────────────┼──────────────┼──────────┼───────┼───────────┼───────────────┼───────────┤
│ tiny-update-notifier@2.0.2 │ 1 │ 17.24kb │ 14 │ │ MIT │ │
╰────────────────────────────┴──────────────┴──────────┴───────┴───────────┴───────────────┴───────────╯
npm_package -0.3% (66.4 kB => 66.2 kB)
Package contents:
npm notice
📦 @adobe/edgly@1.4.2
Tarball Contents
170B COPYRIGHT
11.3kB LICENSE
20.9kB README.md
1.9kB package.json
2.3kB src/commands/fiddle/clear.js
2.0kB src/commands/fiddle/create.js
1.2kB src/commands/fiddle/fiddle.js
2.6kB src/commands/fiddle/get.js
2.2kB src/commands/fiddle/update.js
3.0kB src/commands/service/create.js
3.2kB src/commands/service/get.js
2.1kB src/commands/service/import.js
1.2kB src/commands/service/service.js
3.7kB src/commands/service/update.js
2.7kB src/commands/test.js
1.2kB src/commands/version.js
1.6kB src/config.js
19.4kB src/fastly/api/fastly-api.js
2.2kB src/fastly/api/fiddle-api.js
8.9kB src/fastly/fiddle-mgr.js
11.7kB src/fastly/service-mgr.js
3.8kB src/fastly/service.js
16.9kB src/fastly/store.js
2.6kB src/index.js
2.8kB src/opts.js
185.9kB src/secrets/patterns.yaml
9.8kB src/secrets/secrets.js
6.8kB src/test/parser.js
10.4kB src/test/runner.js
6.0kB src/util.js
6.4kB src/yargs-ahoy.js
Tarball Details
name: @adobe/edgly
version: 1.4.2
filename: adobe-edgly-1.4.2.tgz
package size: 66.2 kB
unpacked size: 357.2 kB
shasum: 0f3d86e5fe0676966786460fe819b27f48edf54e
integrity: sha512-h2YuGXSZSIslN[...]0D9QPrm+a3xvA==
total files: 31
npm notice
Notes
  • PR branch: biome2 @ 226c65c
  • Base branch: main @ eb17885
  • Sizewatcher v1.4.1
  • Effective Configuration:
limits:
  fail: 100%
  warn: 30%
  ok: '-10%'
report:
  githubComment: true
  githubStatus: false
comparators: {}

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/fastly/fiddle-mgr.js 0.00% 2 Missing ⚠️
src/secrets/secrets.js 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@alexkli alexkli merged commit 2d09735 into main Apr 29, 2026
6 checks passed
@alexkli alexkli deleted the biome2 branch April 29, 2026 01:59
@adobe-bot
Copy link
Copy Markdown

🎉 This PR is included in version 1.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants