Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 14, 2025

Description

Adds ESLint with TypeScript and React support to enforce code quality standards across the monorepo.

Key Changes:

  • ESLint Configuration (eslint.config.js)

    • Flat config with TypeScript parser and React/React Hooks plugins
    • File-specific rules for .ts/.tsx, .cjs, .mjs, and scripts
    • Balanced strictness: errors for bugs (missing keys), warnings for style (any usage)
  • CI Integration (.github/workflows/lint.yml)

    • Runs Prettier + ESLint on PRs and main branch
    • Minimal permissions for security
  • Scripts (added to package.json)

    npm run lint        # Check for issues
    npm run lint:fix    # Auto-fix where possible
  • Code Fixes

    • Resolved 5 missing React key errors in SourcesTable.tsx and Clients.tsx
    • Auto-fixed 6 style issues (prefer-const, redundant negations)
    • Result: 0 errors, 30 non-critical warnings
  • Documentation (README.md)

    • Added "Code Quality Tools" section with usage instructions

Contributor Checklist

Original prompt

This section details on the original issue you should resolve

<issue_title>Add linter</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)

@Martinsos Sure @shemzz give it a try! One thing to note, and sorry for assuming too much possibly, but the message above sounds like it was generated by LLM / AI. I don't mind you assisting yourself with the AI, but I do care about you being able to verifying the results it produces, confirming they are the best choice, that it all makes sense, ... . What I am trying to say is, if result is mostly/purely produced by AI and goes beyond your own knowledge, then I will not be inclined to accept the PR as it demands quite some work from us. If instead the PR will be guided by your own experience and research, then I don't mind if it was AI assisted.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add linter to project Add ESLint linting infrastructure Nov 14, 2025
Copilot AI requested a review from vincanger November 14, 2025 13:43
Copilot finished work on behalf of vincanger November 14, 2025 13:43
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 linter

2 participants