Skip to content

[codex] bump lint-staged to 17#7

Merged
xdanger merged 2 commits into
mainfrom
codex/dependency-sweep-lint-staged-17
May 6, 2026
Merged

[codex] bump lint-staged to 17#7
xdanger merged 2 commits into
mainfrom
codex/dependency-sweep-lint-staged-17

Conversation

@xdanger
Copy link
Copy Markdown
Member

@xdanger xdanger commented May 6, 2026

Summary

  • Bump lint-staged from ^16.4.0 to ^17.0.2.
  • Refresh pnpm-lock.yaml for the new lint-staged dependency graph.

Breaking-change risk

  • lint-staged 17 requires Node >=22.22.1 and Git >=2.32.0. This checkout validated with Node v24.14.0 and Git 2.53.0.
  • The release notes also call out YAML config handling; this repo uses .lintstagedrc.mjs, so no migration was needed.

Validation

  • pnpm install --frozen-lockfile
  • pnpm run lint
  • pnpm run lint:staged
  • pnpm outdated --format json returned {}

- 🔧 keep the scaffold on the current lint-staged release

- 🔧 accept the Node 22.22.1 floor now required by lint-staged 17
@github-actions github-actions Bot requested a review from Copilot May 6, 2026 13:04
@xdanger xdanger review requested due to automatic review settings May 6, 2026 13:04
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 6, 2026

Greptile Summary

lint-staged^16.4.0 升级至 ^17.0.2,并在 package.json 中新增 engines 字段以声明 Node >=22.22.1 的最低版本要求。lockfile 随之刷新,传递依赖图已更新。

  • lint-staged@17 移除了对 commandercolorette 的直接依赖,将 yaml 改为 optional 依赖,并将 listr29.x 升级至 10.x(要求 Node >=22.13.0,在新 engines 约束范围内)。
  • 本仓库使用 .lintstagedrc.mjs 配置,yaml 可选化不影响现有功能。
  • 之前评审中提出的缺少 engines 字段问题,已在本 PR 中通过新增 "engines": { "node": ">=22.22.1" } 解决。

Confidence Score: 5/5

这是一个纯依赖升级 PR,变更范围明确,lockfile 与 package.json 保持一致,无破坏性代码逻辑改动,可安全合并。

变更仅涉及 lint-staged 版本号和 engines 字段的声明,传递依赖图已在 lockfile 中正确反映,yaml 改为可选依赖与本仓库使用 .mjs 配置完全兼容,无功能代码修改。

无文件需要特别关注。

Important Files Changed

Filename Overview
package.json 将 lint-staged 从 ^16.4.0 升级至 ^17.0.2,并新增 engines 字段声明 Node >=22.22.1 要求
pnpm-lock.yaml lockfile 与 package.json 变更保持一致:lint-staged 17.0.2、listr2 10.2.1、移除 colorette/commander、yaml 改为 optional 依赖、新增 wrap-ansi 10.0.0

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    LS17["lint-staged@17.0.2\n(node>=22.22.1)"]
    L2["listr2@10.2.1\n(node>=22.13.0)"]
    WA10["wrap-ansi@10.0.0"]
    YAML["yaml@2.8.4\n(optional)"]
    PICOM["picomatch@4.0.4"]
    SARGV["string-argv@0.3.2"]
    TEXEC["tinyexec@1.1.2"]

    LS17 --> L2
    LS17 --> PICOM
    LS17 --> SARGV
    LS17 --> TEXEC
    LS17 -.->|optional| YAML
    L2 --> WA10

    LS16["lint-staged@16.4.0\n(node>=20.17)"]:::removed
    CMD["commander@14.0.3"]:::removed
    COL["colorette@2.0.20"]:::removed
    L29["listr2@9.0.5"]:::removed
    WA9["wrap-ansi@9.0.2"]:::removed

    LS16 --> CMD
    LS16 --> L29
    L29 --> COL
    L29 --> WA9

    classDef removed fill:#fdd,stroke:#d44,color:#900
Loading

Reviews (2): Last reviewed commit: "🔧 chore(deps): declare node floor" | Re-trigger Greptile

Comment thread package.json
- 🔧 make the lint-staged 17 runtime requirement explicit
@github-actions github-actions Bot requested a review from Copilot May 6, 2026 13:08
@xdanger xdanger review requested due to automatic review settings May 6, 2026 13:08
@xdanger xdanger merged commit a49bd7d into main May 6, 2026
5 checks passed
@xdanger xdanger deleted the codex/dependency-sweep-lint-staged-17 branch May 6, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant