From beb83e253cad1593593c8240c876b8afea838153 Mon Sep 17 00:00:00 2001 From: jaymar921 Date: Thu, 13 Aug 2026 23:13:18 +0800 Subject: [PATCH] fix(lint): benchmark output is not this project's source MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `eslint .` reported 861 errors on a clean checkout of `main`, every one of them from `.ignore/` — the scratch tree where benchmark runs leave the todo apps the agent built. Vite configs and React hooks parsed as CommonJS, plus 845 from one minified `dist/` bundle. None of it is authored here, and none of it says anything about HirayaCoder. `npm run lint` never saw it because that script names `app test`, so CI stayed green while anyone running the linter directly — or just opening the repo with the ESLint extension, which lints the whole workspace — saw a repository that looked broken. `.ignore/` now sits with the other generated trees the config already skips. It is the same set `.gitignore` refuses to track, for the same reason. `npm run lint` is unchanged at 0 errors; `eslint .` now exits clean. --- eslint.config.js | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/eslint.config.js b/eslint.config.js index 3447961..fd8e022 100644 --- a/eslint.config.js +++ b/eslint.config.js @@ -15,7 +15,24 @@ const noUnsanitized = require('eslint-plugin-no-unsanitized'); module.exports = [ { - ignores: ['node_modules/**', 'out/**', 'dist/**', 'builds/**', '.vscode-test/**'], + // Everything here is generated or downloaded, never authored — the same set the + // repository already refuses to track. + // + // `.ignore/` is where benchmark runs leave the projects the agent built (Vite + // configs, React hooks, and a minified `dist/` bundle). It is model output, not + // this project's source: `npm run lint` never sees it because that script names + // `app test`, but a bare `eslint .` — and the ESLint editor extension, which lints + // the whole workspace — reported 861 errors from it and made the repository look + // broken on a clean checkout of `main`. Linting it says nothing about HirayaCoder, + // and re-running a benchmark would change the count anyway. + ignores: [ + 'node_modules/**', + 'out/**', + 'dist/**', + 'builds/**', + '.vscode-test/**', + '.ignore/**', + ], }, { files: ['**/*.js'],