check-links accepts --stale-days=N and then ignores it (365 is used)

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#250 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
78/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
javascript, node.js
Bereich
cli, tooling

Rechercherichtung

Lies site/check-links.mjs im Bereich der Zeilen 99–105 und site/lib/link-check.mjs im Bereich der Zeilen 54–63 und verfolge dann, wie die geparsten Flags staleDaysFrom erreichen. Führe die relevanten Tests für --stale-days=30 und --stale-days 30 sowie für einen ungültigen Wert aus. Fertig ist es, wenn beide gültigen Formen einen Grenzwert von 30 Tagen verwenden und ungültige Werte weiterhin fehlschlagen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug good first issue priority: low

Summary

check-links parses argv twice: the strict cliArgs parse accepts --stale-days=30, then staleDaysFrom(process.argv) scans for the exact token --stale-days, misses the = form and falls back to 365 days without a warning.

Where

  • site/check-links.mjs:99-105
  • site/lib/link-check.mjs:54-63 (staleDaysFrom)

Details

readArgs returns { "stale-days": "30" } while staleDaysFrom(['node','x','--stale-days=30']) returns { days: 365 }. This is the last script that still hand-parses argv after the strict-flags change.

Impact

A maintainer asking for a shorter staleness window locally silently gets the 365-day report. The weekly workflow passes no flag, so CI is unaffected.

Steps to reproduce

  1. node site/check-links.mjs --stale-days=30 --report r.md
  2. The stale list uses a 365-day cutoff.

Suggested fix

Validate the parsed value (flags["stale-days"]) in staleDaysFrom instead of re-reading process.argv, and drop the second parse.

Acceptance criteria

  • --stale-days=30 and --stale-days 30 both give a 30-day cutoff
  • Invalid values still fail as they do today

Getting started

  • Relevant files: site/check-links.mjs, site/lib/link-check.mjs
  • Run the tests: npm test; the full local gate is npm run check (Node 22+, no install step)
  • Related docs: CONTRIBUTING.md, site/README.md

Related: #82

Filed by repo-audit/full-audit at 4adb944. Severity: low. Effort: S. audit-key: bug|site/check-links.mjs|staleDaysFrom

Vorherrschende Sprache
JavaScript
Sterne
58
Forks
15
Ø Merge
1 T. 15 Std.
Gemergte PRs (30 T.)
12

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus TMHSDigital/Free-Game-Dev-Assets

Alle Issues in TMHSDigital/Free-Game-Dev-Assets

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.