Skip to content

Added ESLint for linting service as a Toolcall for linting the output #12

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Kitenite
Copy link

Description

Added ESLint for linting the output of the llm response

Related Issues

fixes onlook-dev#1570

Type of Change

Added a file for linting script in the packages/foundation/src/linting

  • Bug fix
  • New feature
  • Documentation update
  • Release
  • Refactor
  • Other (please describe):

Important

Adds ESLint integration for linting and auto-fixing LLM-generated project files using a new LintingService.

  • Behavior:
    • Integrates LintingService in index.ts to lint and fix generated project files.
    • Emits progress messages for linting results, including errors, warnings, and auto-fixes.
  • Dependencies:
    • Adds eslint, @typescript-eslint/parser, @typescript-eslint/eslint-plugin, eslint-plugin-react, and eslint-plugin-react-hooks to package.json.
  • Linting Service:
    • Adds LintingService class in index.ts to perform linting and auto-fixing using ESLint.
    • Provides lintAndFix() and lintProject() methods for file and project-level linting.

This description was created by Ellipsis for 31de4f0. You can customize this summary. It will automatically update as commits are pushed.

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.

[feat] Add a toolcall for linting the output that the model can use
2 participants