Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Build-test fixtures that no longer test what they claim

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

Maintainer antworten meist innerhalb von 1 Tag

@cknitt arbeitet bereits daran.

Seit 12.10.2026.

  • #8838 von @cknitt — offen

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, ocaml
Bereich
testing

Rechercherichtung

Lies tests/build_tests/rescript_tools_binary_path/input.js und die Einrichtung der zugehörigen Fixtures und untersuche dann die rescript.json-Dateien von weird_deps und weird_devdeps. Füge eine Fixture-package.json für rescript_tools_binary_path hinzu und setze die Sources der anderen Fixtures auf eine leere Liste. Fertig ist die Aufgabe, wenn die Fixtures das beabsichtigte Verhalten beim Import von Packages prüfen und keine gelöschten Quellordner mehr auflisten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Found by adversarial experiments on semantic analysts.

Each item below is a build-test fixture whose setup no longer matches its purpose.

  • rescript_tools_binary_path passes with the import it guards against.
    • input.js runs the compiled src/Main.mjs and src/Main.cjs inside the repository.
    • The fixture has no package.json, so Node resolves # specifiers in those files against the repository root package.json, which defines #cli/*.
    • The bug fixed by #8694 (@module("#cli/bins") emitted into user code) therefore does not reproduce here. The commit message of 45446347c says the test uses its own package.json, but none is checked in.
    • Reproduction at 8d51b9ce7 in tests/build_tests/rescript_tools_binary_path, with node_modules/rescript linked to the repo root as input.js does. The two files contain the JS that bsc emits for the pre-#8694 external: 
      printf 'import * as Bins from "#cli/bins";\nconsole.log(Bins.rescript_tools_exe);\n' > src/Old.mjs
printf 'let Bins = require("#cli/bins");\nconsole.log(Bins.rescript_tools_exe);\n' > src/Old.cjs
node src/Old.mjs; node src/Old.cjs   # both print …/bin/rescript-tools.exe
echo '{"name":"user-project","private":true}' > package.json
node src/Old.mjs   # ERR_PACKAGE_IMPORT_NOT_DEFINED
node src/Old.cjs   # Error: Cannot find module '#cli/bins'
    • With that package.json in place, the current Main.mjs and Main.cjs (importing rescript/tools) still print the binary path.
    • Fix: check in a fixture package.json.
  • weird_deps and weird_devdeps list deleted source folders.
Vorherrschende Sprache
OCaml
Sterne
7.5k
Forks
484
Ø Merge
18 Std. 19 Min.
Gemergte PRs (30 T.)
116

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

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 rescript-lang/rescript

Alle Issues in rescript-lang/rescript

Ähnliche Issues

Weitere Issues zu Testing & QA

Neue Issues direkt in Ihr Postfach

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