eslint-factory: no ignores entry for vendored actions/setup/js/work_queue_yaml.cjs ([email protected] copy) causes unactionable lint no

オープン 初心者向け
  • 何か問題がありますか？
#75,596 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
72/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
eslint, javascript, node.js
領域
testing, tooling

調査の方向性

eslint-factory/eslint.config.cjs から始め、ignores が actions/setup/js にどのように適用されるかを確認してください。そのディレクトリにある既存ファイルを提案された除外設定と照らし合わせた後、make lint:setup-js を実行し、ベンダー提供ファイルが gh-aw-custom の診断を出さないことを確認してください。除外設定がベンダー提供ファイルを対象にしつつ、チームが作成したファイルを除外しなければ完了です。

索引モデルが issue の本文から書いたものです。

説明

cookie eslint
Summary

actions/setup/js/work_queue_yaml.cjs opens with:

// @ts-nocheck
// Generated by scripts/work_queue_yaml/build.mjs from [email protected].

It is a byte-for-byte vendored copy of the yaml npm package, not team-authored code. eslint-factory/eslint.config.cjs's only ignores entry is ["**/*.test.cjs", "**/*.test.js"] (confirmed by direct read) — there is no exclusion for vendored/generated files, and the lint:setup-js Makefile target runs ESLint (all ~70 gh-aw-custom/* rules) against the whole actions/setup/js directory with this config, so every rule lints this file.

Live grounding

Grepped the full non-test corpus for a @ts-nocheck + Generated by vendoring marker: work_queue_yaml.cjs is currently the only file carrying this convention. It alone accounts for 13 live .substr() call sites (no-deprecated-string-substr would warn on all of them — re-confirmed today, line numbers: 998, 1025 ×2, 1026, 2044, 2248, 2420, 3635, 3689, 3694, 4325, 4351, 4672, 4746), plus whatever other gh-aw-custom/* rules independently fire on the same 4700+-line file. None of this is actionable: fixing a vendored upstream copy would just be overwritten by the next build.mjs regeneration from [email protected], and the file is explicitly marked @ts-nocheck for exactly this reason.

Suggested fix

Add an ignores entry to eslint-factory/eslint.config.cjs scoped to vendored/generated files — either an explicit path ("**/work_queue_yaml.cjs") or, more durably, a convention-based exclusion that detects the // Generated by + // @ts-nocheck marker pattern so future vendored files are automatically excluded without a config change each time.

Acceptance criteria
  • actions/setup/js/work_queue_yaml.cjs (or any file matching the vendored-file convention) is excluded from gh-aw-custom/* linting via eslint.config.cjs.
  • make lint:setup-js (or equivalent) produces zero gh-aw-custom/* diagnostics for this file after the change.
  • The exclusion does not widen to cover genuinely team-authored files — verify no other actions/setup/js/*.cjs file currently matches the chosen ignore pattern.

Generated by 🤖 ESLint Refiner · claude · agent · ~122 AIC · ⌖ ~1 AIC · ⊞ 4.2K · ◷
Claim finished

  • expires on Nov 13, 2026, 7:34 PM UTC-08:00
主要言語
Go
スター
5.6k
フォーク
615
平均マージ
5時間 44分
マージ済み PR（30日）
808

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

github/gh-aw のほかの issue

github/gh-aw の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。