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

Aberta Para iniciantes
  • Algo errado?
#75,596 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
72/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
eslint, javascript, node.js
Domínio
testing, tooling

Direção de pesquisa

Comece por eslint-factory/eslint.config.cjs e examine como as exclusões dele são aplicadas a actions/setup/js. Confira os arquivos existentes nesse diretório em relação à exclusão proposta e, em seguida, execute make lint:setup-js para confirmar que o arquivo de terceiros incorporado não gera diagnósticos de gh-aw-custom. A tarefa estará concluída quando a exclusão abranger o arquivo de terceiros incorporado sem excluir arquivos criados pela equipe.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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
Linguagem predominante
Go
Estrelas
5.6k
Forks
615
Merge médio
5h 44min
PRs com merge (30d)
808

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de github/gh-aw

Todas as issues de github/gh-aw

Issues semelhantes

Mais issues de Go

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.