Skip to content

feat(review): add local ESLint quality findings - #263

Open
LarryHu0217 wants to merge 3 commits into
johannesjo:mainfrom
LarryHu0217:codex/eslint-quality-provider-260
Open

feat(review): add local ESLint quality findings#263
LarryHu0217 wants to merge 3 commits into
johannesjo:mainfrom
LarryHu0217:codex/eslint-quality-provider-260

Conversation

@LarryHu0217

Copy link
Copy Markdown
Contributor

Summary

  • run the project's existing local ESLint only for changed JavaScript and TypeScript files
  • map ESLint JSON findings into the shared review finding contract and reuse the existing sidebar/send-to-agent flow
  • degrade silently when no ESLint config applies, while surfacing actionable provider failures
  • add an explicit Refresh findings control for the current diff

Validation

  • npm run check
  • npm test (116 files passed, 2 skipped; 1,861 tests passed, 23 skipped)
  • Electron IPC finding tests and client review tests

Fixes #260

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.

Add a local ESLint-backed quality-finding provider

1 participant