chore(deps): migrate eslint config to flat format for v10 compat#9222
chore(deps): migrate eslint config to flat format for v10 compat#9222
Conversation
ESLint v10 removed support for the legacy .eslintrc.* format, so render:fig was failing with "ESLint couldn't find an eslint.config file" after the v10 bump in #9200. @fig/eslint-config-autocomplete is eslint-v8-only (its v2.0.0 peerDep caps at ^8.56.0 and it still imports LegacyESLint which is gone in v10), so swap it for a flat config that uses the modern typescript-eslint package, eslint-plugin-compat v7, and keeps the fig-linter rules we were actually using. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Code Review
This pull request migrates the project to a flat ESLint configuration by introducing eslint.config.mjs and updating dependencies, including the addition of typescript-eslint and eslint-plugin-compat. Feedback indicates that @withfig/eslint-plugin-fig-linter is missing from devDependencies despite being used in the new configuration. Additionally, it is recommended to avoid disabling core safety rules such as no-var and no-unused-vars globally within the ESLint configuration.
| "@types/node": "^25.0.0", | ||
| "@withfig/autocomplete-tools": "^2.11.0", | ||
| "@withfig/autocomplete-types": "^1.31.0", | ||
| "eslint": "^10.0.0", |
There was a problem hiding this comment.
The package @withfig/eslint-plugin-fig-linter is imported in eslint.config.mjs but is missing from devDependencies. It was previously a transitive dependency of @fig/eslint-config-autocomplete, which has been removed. This will cause the linting process to fail because the module cannot be found.
| "eslint": "^10.0.0", | |
| "@withfig/eslint-plugin-fig-linter": "^1.4.1", | |
| "eslint": "^10.0.0", |
| "no-unused-vars": "off", | ||
| "no-var": "off", | ||
| "@typescript-eslint/no-unused-vars": "off", |
There was a problem hiding this comment.
Disabling no-unused-vars, no-var, and @typescript-eslint/no-unused-vars globally for these files reduces code quality and safety. It is recommended to keep these rules enabled to prevent the use of legacy var declarations and to identify dead code. If certain variables must remain unused, consider prefixing them with an underscore (e.g., _unused) and configuring the rule to ignore that pattern, or use specific inline disable comments where necessary.
| "no-unused-vars": "off", | |
| "no-var": "off", | |
| "@typescript-eslint/no-unused-vars": "off", | |
| rules: { | |
| "@typescript-eslint/explicit-module-boundary-types": "off", | |
| "@withfig/fig-linter/no-malicious-script": "error", | |
| }, |
Greptile SummaryMigrates the Fig completion linting setup from the legacy Confidence Score: 5/5Safe to merge — tooling-only change with no runtime impact and all findings are P2 or lower. All changed files are dev tooling (ESLint config, lockfile, fig generator source cleanup). The previous P1 concern about the missing No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["mise run render:fig"] --> B["bun run lint-fig"]
B --> C["eslint.config.mjs"]
C --> D["typescript-eslint v8 recommended"]
C --> E["eslint-plugin-compat v7 flat/recommended"]
C --> F["@withfig/eslint-plugin-fig-linter rules"]
F --> F1["no-malicious-script"]
F --> F2["no-useless-insertvalue"]
F --> F3["no-empty-array-values"]
F --> F4["no-name-equals"]
C --> G["xtasks/fig/**/*.ts"]
G --> H["generators.ts"]
G --> I["src/mise.ts"]
Reviews (2): Last reviewed commit: "chore(deps): add @withfig/eslint-plugin-..." | Re-trigger Greptile |
| @@ -0,0 +1,38 @@ | |||
| import compat from "eslint-plugin-compat"; | |||
| import figLinter from "@withfig/eslint-plugin-fig-linter"; | |||
There was a problem hiding this comment.
Missing dependency for
@withfig/eslint-plugin-fig-linter
@withfig/eslint-plugin-fig-linter is imported here but is not listed in package.json devDependencies and has been removed from bun.lock. Any developer running bun install from scratch will get a module-not-found error when bun run lint-fig (or mise run render:fig) hits this import. The package needs to be added explicitly since it was previously a transitive dependency of @fig/eslint-config-autocomplete (now removed).
Add "@withfig/eslint-plugin-fig-linter": "^1.4.1" to devDependencies in package.json and regenerate the lockfile.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d2a74f6. Configure here.
Hyperfine Performance
|
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.4.16 x -- echo |
21.7 ± 0.4 | 20.8 | 24.4 | 1.00 |
mise x -- echo |
22.4 ± 0.7 | 21.3 | 29.1 | 1.03 ± 0.04 |
mise env
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.4.16 env |
21.2 ± 0.6 | 20.4 | 27.7 | 1.00 |
mise env |
21.7 ± 0.4 | 20.8 | 23.8 | 1.02 ± 0.04 |
mise hook-env
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.4.16 hook-env |
22.1 ± 0.4 | 21.3 | 24.5 | 1.00 |
mise hook-env |
22.6 ± 0.6 | 21.4 | 26.7 | 1.02 ± 0.03 |
mise ls
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.4.16 ls |
19.3 ± 0.5 | 18.4 | 21.0 | 1.00 |
mise ls |
20.0 ± 0.5 | 18.8 | 21.5 | 1.03 ± 0.03 |
xtasks/test/perf
| Command | mise-2026.4.16 | mise | Variance |
|---|---|---|---|
| install (cached) | 143ms | 148ms | -3% |
| ls (cached) | 76ms | 77ms | -1% |
| bin-paths (cached) | 80ms | 82ms | -2% |
| task-ls (cached) | 794ms | 797ms | +0% |
Previously pulled in transitively via @fig/eslint-config-autocomplete which this PR removed. Without an explicit entry, fresh installs fail with module-not-found when eslint.config.mjs imports it. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
### 🐛 Bug Fixes - **(backend)** respect install_before in latest lookup by @risu729 in [#9193](#9193) - **(backend)** route explicit latest through stable lookup by @risu729 in [#9228](#9228) - **(backends)** deprecate b shorthand by @risu729 in [#9234](#9234) - **(config)** warn for deprecated env keys by @risu729 in [#9205](#9205) - **(config)** treat enable_tools empty as disable-all by @risu729 in [#9108](#9108) - **(github)** avoid auth on release asset downloads by @risu729 in [#9060](#9060) - **(gitlab)** warn when glab OAuth2 token is expired by @stanhu in [#9195](#9195) - **(npm)** honor install_before without day drift by @risu729 in [#9157](#9157) - **(npm)** warn on old bun and pnpm for install_before by @risu729 in [#9232](#9232) - **(pipx)** honor install_before for uv and pipx installs by @risu729 in [#9190](#9190) - **(registry)** allow shfmt on Windows by @zeitlinger in [#9191](#9191) ### 🚜 Refactor - **(backend)** remove unused rolling release helper by @risu729 in [#9175](#9175) - **(backend)** use file util for removals by @risu729 in [#9206](#9206) ### 📚 Documentation - **(config)** clarify always_keep_download behavior by @risu729 in [#9235](#9235) - **(configuration)** add rust to idiomatic version files by @jjt in [#9233](#9233) - **(contributing)** expand contribution guide introduction by @marianwolf in [#9208](#9208) - **(github)** document multiple release assets workaround by @risu729 in [#9236](#9236) ### 📦️ Dependency Updates - update actions/setup-node action to v6 by @renovate[bot] in [#9183](#9183) - update dependency @types/node to v25 by @renovate[bot] in [#9187](#9187) - update crazy-max/ghaction-import-gpg action to v7 by @renovate[bot] in [#9186](#9186) - update actions/cache action to v5 by @renovate[bot] in [#9181](#9181) - update amannn/action-semantic-pull-request action to v6 by @renovate[bot] in [#9184](#9184) - update apple-actions/import-codesign-certs action to v6 by @renovate[bot] in [#9185](#9185) - update dependency eslint to v10 by @renovate[bot] in [#9200](#9200) - update dependency toml to v4 by @renovate[bot] in [#9201](#9201) - update rust crate reqwest to 0.13 by @renovate[bot] in [#9171](#9171) - update ghcr.io/jdx/mise:deb docker digest to 523d826 by @renovate[bot] in [#9198](#9198) - update ghcr.io/jdx/mise:alpine docker digest to 05617e0 by @renovate[bot] in [#9196](#9196) - update ghcr.io/jdx/mise:rpm docker digest to c1992f9 by @renovate[bot] in [#9199](#9199) - update ghcr.io/jdx/mise:copr docker digest to 90db6cd by @renovate[bot] in [#9197](#9197) - update taiki-e/install-action digest to 58e8625 by @renovate[bot] in [#9209](#9209) - update fedora docker tag to v45 by @renovate[bot] in [#9213](#9213) - update docker/setup-buildx-action action to v4 by @renovate[bot] in [#9212](#9212) - update docker/metadata-action action to v6 by @renovate[bot] in [#9211](#9211) - update docker/login-action action to v4 by @renovate[bot] in [#9210](#9210) - update dependency typescript to v6 by @renovate[bot] in [#9202](#9202) - update docker/build-push-action action to v7 by @renovate[bot] in [#9203](#9203) - update github artifact actions (major) by @renovate[bot] in [#9215](#9215) - update rust crate duct to v1 by @renovate[bot] in [#9220](#9220) - update rust crate demand to v2 by @renovate[bot] in [#9219](#9219) - update rust crate clx to v2 by @renovate[bot] in [#9218](#9218) - update nick-fields/retry action to v4 by @renovate[bot] in [#9217](#9217) - update jdx/mise-action action to v4 by @renovate[bot] in [#9216](#9216) - update rust crate self_update to 0.44 by @renovate[bot] in [#9174](#9174) - migrate eslint config to flat format for v10 compat by @jdx in [#9222](#9222) - update actions/checkout action to v6 by @renovate[bot] in [#9182](#9182) - update rust crate toml to v1 by @renovate[bot] in [#9225](#9225) - update rust crate versions to v7 by @renovate[bot] in [#9226](#9226) - update rust crate which to v8 by @renovate[bot] in [#9227](#9227) - update rust crate rmcp to v1 by @renovate[bot] in [#9221](#9221) ### 📦 Registry - add sheldon by @3w36zj6 in [#9104](#9104) - add pocketbase by @ranfdev in [#9123](#9123) - add worktrunk ([aqua:max-sixty/worktrunk, cargo:worktrunk](https://github.com/max-sixty/worktrunk, cargo:worktrunk))#1 by @edouardr in [#8796](#8796) - add dependency-check ([aqua:dependency-check/DependencyCheck](https://github.com/dependency-check/DependencyCheck)) by @kapitoshka438 in [#9204](#9204) - add janet by @ranfdev in [#9241](#9241) ### New Contributors - @ranfdev made their first contribution in [#9241](#9241) - @jjt made their first contribution in [#9233](#9233) - @marianwolf made their first contribution in [#9208](#9208) - @edouardr made their first contribution in [#8796](#8796) ## 📦 Aqua Registry Updates #### New Packages (3) - [`LargeModGames/spotatui`](https://github.com/LargeModGames/spotatui) - [`android-sms-gateway/cli`](https://github.com/android-sms-gateway/cli) - [`velero-io/velero`](https://github.com/velero-io/velero) #### Updated Packages (1) - [`skim-rs/skim`](https://github.com/skim-rs/skim)

Summary
mise run render:figbecause v10 removed support for legacy.eslintrc.*configs and requires a flateslint.config.js.@fig/eslint-config-autocompleteis v8-only (peerDep caps at^8.56.0, loadsLegacyESLintwhich was removed in v10), so it's swapped out.eslint.config.mjsuses the modern unifiedtypescript-eslintpackage +eslint-plugin-compat@7(both support v10), and keeps the@withfig/fig-linterrules that were actually doing work (no-malicious-script,no-useless-insertvalue,no-empty-array-values,no-name-equals).eslint --fixalso removed an unusedeslint-disable @withfig/fig-linter/conventional-descriptionsdirective fromgenerators.tsandsrc/mise.ts; the re-render is stable.Test plan
mise run render:figcompletes cleanly locallymise run render:figproduces no further diff (idempotent)bun run lint-figexits 0🤖 Generated with Claude Code
Note
Low Risk
Low risk: changes are limited to development lint tooling/config and dependency updates, though misconfiguration could cause
lint-fig/render:figto fail in CI.Overview
Switches ESLint configuration to the v10 flat config by removing
.eslintrc.cjsand addingeslint.config.mjsscoped toxtasks/fig/**/*.ts, usingtypescript-eslint,eslint-plugin-compat, and@withfig/eslint-plugin-fig-linterrules.Updates
package.json/bun.lockto drop@fig/eslint-config-autocomplete, add the new lint deps, and bumps related transitive packages; also removes now-unnecessaryeslint-disabledirectives in the Fig generator sources.Reviewed by Cursor Bugbot for commit 4c3a66f. Bugbot is set up for automated code reviews on this repo. Configure here.