Skip to content

[nightshift] lint-fix: resolve 29 ESLint parsing errors + 4 lint warnings#3

Open
Microck wants to merge 1 commit intomainfrom
nightshift/lint-fix
Open

[nightshift] lint-fix: resolve 29 ESLint parsing errors + 4 lint warnings#3
Microck wants to merge 1 commit intomainfrom
nightshift/lint-fix

Conversation

@Microck
Copy link
Copy Markdown
Owner

@Microck Microck commented Apr 1, 2026

Automated by Nightshift v3 (GLM 5.1).

Task: lint-fix
Category: pr

Changes

Fixed 29 ESLint parsing errors by adding allowDefaultProject configuration to eslint.config.mjs for test files, scripts, and config files outside the tsconfig src/ include paths. Also fixed 4 actual lint warnings discovered in newly-parseable files:

  1. eslint.config.mjs — Expanded projectService: true to explicit config with allowDefaultProject glob list for all 29 files outside tsconfig scope
  2. packages/core/test/timing.test.ts — Removed unnecessary escape \\/ in regex
  3. scripts/benchmark.ts — Removed unused join import
  4. scripts/publish-smoke.ts — Removed unnecessary escape \\" in template literal (2 instances)

Final result: 0 errors, 0 warnings. pnpm lint passes clean.

Merge if useful, close if not.

- Add allowDefaultProject config to eslint.config.mjs for test files,
  scripts, and config files outside src/ include paths
- Set maximumDefaultProjectFileMatchCount to 30 to accommodate all files
- Fix unnecessary escape character in timing.test.ts regex
- Remove unused 'join' import in scripts/benchmark.ts
- Fix unnecessary escape characters in scripts/publish-smoke.ts template literal

Resolves 29 parsing errors and 4 warnings, achieving 0 errors/0 warnings.
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