[eslint-refiner] ESLint Refiner — daily report (2026-09-22) #62561
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by ESLint Refiner. A newer discussion is available at Discussion #62880. |
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.
1 issue filed on
eslint-factorycustom rules today. Ruleset remains stable at 62 rules; the codebase has now been through a full multi-week exhaustive review cycle, so daily output has shifted from broad rule-by-rule sweeps to targeted re-verification of expired findings.Key result
#60757(auto-expirednot_plannedon 2026-09-21, unfixed). Re-grounded live atactions/setup/js/safeoutputs_cli.cjs:46-48:const err = error ?? {}followed bytypeof err.message === "string" ? err.message : String(error)is flagged as "stringifies the wrong container", buterris a type-narrowed alias oferror— whenerroris truthy (the common case) they're reference-identical, and when falsy the existing code is actually more correct than the rule's implied fix. Refiled since this is only the theme's 1st expiry.Strategy this run
HEADhad moved since the last 3 stale runs — it had (2b90c3a), butgit show --statshowed 880 files as pure insertions (362,896 insertions, 0 deletions), confirming this is another shallow-clone squash artifact, not a real diff. Git-log-based "new call sites" detection is now conclusively unusable across 5 consecutive runs.#60757had dropped off. Re-grounded and refiled it (see above).#59647(no-duplicate-constant-values string-threshold theory, 1 expiry, technically eligible) but declined — the 2026-09-18 run had already re-grepped the corpus and found zero live triggers for that shape, and nothing has changed since, so re-filing a purely latent finding would just repeat a call already made.Open issues snapshot (2026-09-22)
CHILD_PROCESS_OBJECTSname allowlist missescpalias.catch()/.then()hides the body-read callERROR_CODE_PATTERNmisses non-ERR_/E0xxexportsshim.cjsdual-mode standalone filesAll 6 issues carried over from 2026-09-21 remain open with no fix uptake yet.
Chronic themes not re-filed (2+ expiries, no PR-capability in this workflow)
try-catch-rule-utils.tssharedVariableDeclaration-suggestion gap — affects ~14 fs-sync/child-process-family rules. Filed/expired twice (#57868,#59891). A source fix (extend the shared suggestion builder to also matchconst x = fs.readFileSync(...)declarator forms, not just expression/return statements) would resolve all of these at once, but this workflow has nocreate_pull_requestsafe-output — flagging for a human or PR-capable workflow to pick up.no-json-stringify-equalityindirect template-literal comparison gap — filed/expired twice (#57869,#59892). Same status.Next actions
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