[eslint-refiner] ESLint Refiner daily report -- 2026-09-10 #59893
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by ESLint Refiner. A newer discussion is available at Discussion #60198. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Overview
Daily ESLint Refiner pass over
eslint-factory(62 registered rules, up from 61 yesterday —require-fs-chmod-try-catchshipped since the last run). Filed 2 issues, both grounded in live code inactions/setup/js/**.Key finding: issues can expire unfixed
Two prior findings — #57868 (shared try/catch suggestion builder skips
VariableDeclarationcall sites) and #57869 (no-json-stringify-equalitymisses indirect template-literal comparisons) — were auto-closed on 2026-09-09 by the tracking bot's TTL (state_reason: not_planned), not because they were fixed or rejected. Re-checking the current source confirmed both defects are still present and still grounded in live code, so both were re-filed as explicitly-labeled recurrences rather than silently dropped.Issues filed
SAFE_WRAPPABLE_STATEMENT_TYPESintry-catch-rule-utils.tsstill only coversExpressionStatement/ReturnStatement, so the dominant real-world shape (const x = fs.someSync(y);,const parsed = JSON.parse(raw);) gets a diagnostic but no autofix, across 14 rules that share this util (one more than at original filing —require-fs-chmod-try-catchinherits it too).trace_graders.cjs:357-366(gradeLoops) still compares two template-literal-embeddedJSON.stringify(...)results (key === prevKey) unnoticed, since the rule only matches directJSON.stringify(a) === JSON.stringify(b)call expressions.Rules reviewed this run with no defect found
require-fs-chmod-try-catch(new, 62nd rule) — inherits the two family-wide known gaps above (and the already-tracked eslint-factory: isIdentifierBoundToFsModule in try-catch-rule-utils.ts does not check for reassignment #59377 reassignment blind spot) but has no chmod-specific defect of its own; all 18 livechmodSync/fchmodSynccall sites in the corpus are already inside try blocks.require-return-after-core-setfailed— extremely mature and well-tested (JSDoc-annotated DI parameters, switch fall-through, braceless control bodies, cross-block/loop continuation are all already covered); no gap found.no-math-minmax-array-spread— zero liveMath.min(...)/Math.max(...)spread call sites exist anywhere inactions/setup/js/**, so no grounded finding was possible (a theoretical bare-destructured-call gap exists but isn't worth filing without a live occurrence).Next actions
hasSuggestions: trueand heavy live corpus usage.Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
api.anthropic.comTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.
All reactions