Skip to content

chore: migrate lint to ESLint CLI for Next 16#55

Merged
BunsDev merged 3 commits intoOpenKnots:mainfrom
clawgreen:codex/atomic-03-lint-script
Mar 9, 2026
Merged

chore: migrate lint to ESLint CLI for Next 16#55
BunsDev merged 3 commits intoOpenKnots:mainfrom
clawgreen:codex/atomic-03-lint-script

Conversation

@clawgreen
Copy link
Copy Markdown
Contributor

@clawgreen clawgreen commented Mar 1, 2026

Summary

  • replace removed next lint usage with ESLint CLI (pnpm exec eslint .)
  • add flat ESLint config (eslint.config.mjs) based on Next core-web-vitals + typescript presets
  • add ESLint dependencies compatible with Next 16 (eslint@^9, eslint-config-next)
  • add temporary rule disables for react-hooks/set-state-in-effect and react/no-unescaped-entities to keep this PR scoped to lint command migration

Why

Next.js 16 removed the next lint command, so the previous script fails by treating lint as a directory argument.

Validation

  • pnpm run lint now executes successfully (warnings only, no errors)

Tracking

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 1, 2026

@clawgreen is attempting to deploy a commit to the 0xBuns Team on Vercel.

A member of the Team first needs to authorize it.

@clawgreen clawgreen marked this pull request as ready for review March 1, 2026 07:41
@BunsDev
Copy link
Copy Markdown
Member

BunsDev commented Mar 9, 2026

Maintainer triage: keeping this open as a high-priority repo-health fix. The current script on still points at , which is stale for this Next 16 setup. Before merge, this branch should be refreshed against so we can validate the final lockfile and any remaining rule disables in the current tree.

@BunsDev BunsDev added blocked-infra Blocked by external tooling or integration setup needs-rebase Branch should be refreshed against main repo-health Repository workflow or infrastructure labels Mar 9, 2026
Copy link
Copy Markdown
Member

@BunsDev BunsDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed after refreshing onto current main. The lint migration is now validated against the current repo state and keeps the branch aligned with the new CI workflow.

@BunsDev BunsDev merged commit 262df30 into OpenKnots:main Mar 9, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked-infra Blocked by external tooling or integration setup needs-rebase Branch should be refreshed against main repo-health Repository workflow or infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants