Skip to content

feat(): adopt official ESLint MCP; remove ds/angular-eslint tool #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 20, 2025

Conversation

poltorak
Copy link
Collaborator

Summary

Switch to the official ESLint MCP integration and remove the in-house DS Angular ESLint tool. This reduces maintenance, unifies linting, and simplifies configuration via .cursor/mcp.json.

What changed

  • Removed custom DS Angular ESLint tool:
    • Deleted packages/angular-mcp-server/src/lib/tools/ds/angular-eslint/**
  • Updated DS tooling aggregation:
    • Edited packages/angular-mcp-server/src/lib/tools/ds/tools.ts
  • Config and docs:
    • Edited .cursor/mcp.json (local)
    • Added .cursor/mcp.json.example
    • Edited .cursor/flows/ds-refactoring-flow/04-validate-changes.mdc
    • Edited .cursor/rules/04-validate-changes.mdc
    • Edited README.md
    • Edited .gitignore to ignore local .cursor/mcp.json
    • Updated package-lock.json

Counts: +1 added, ~7 modified, -11 removed

Motivation

  • Standardize on the official ESLint MCP server
  • Reduce bespoke code and future maintenance
  • Provide a clear local config example

Breaking changes

  • Removed ds.angular-eslint tool. Consumers should use the official ESLint MCP integration.

Migration

  1. Copy .cursor/mcp.json.example to .cursor/mcp.json and adjust to your environment.
  2. Replace any ds.angular-eslint usages with the official ESLint MCP tool commands.

How to test

  • Ensure .cursor/mcp.json exists locally (use the provided example).
  • Start the MCP server and verify ESLint actions are available and functioning via MCP.

Affected areas

  • packages/angular-mcp-server (tools registration)
  • .cursor/* (MCP config + docs)
  • Root docs and lockfile

Notes

  • Local config (.cursor/mcp.json) is now ignored by Git; example file is provided for reference.

@poltorak poltorak merged commit 7fcd5d6 into main Aug 20, 2025
1 check passed
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.

2 participants