Exception analysis does not know the Temporal `fromStringOrThrow` functions
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 72/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- ocaml
- Domaine
- devtools
Piste de recherche
La table des fonctions qui lèvent une exception se trouve dans analysis/reanalyze/src/exn_lib.ml, environ aux lignes 114-190, et le ticket mentionne les huit externals Temporal fromStringOrThrow ajoutés dans #9094 qui n'y figurent pas. Commence par lire comment les entrées *OrThrow existantes y sont écrites, puis exécute rescript-tools reanalyze -exception sur l'extrait de trois lignes du ticket. C'est terminé lorsque les trois appels Temporal sont signalés comme levant JsExn, et non seulement big.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Found by differential semantic drift analysis from b123b7c3 to c10223e8.
What is broken
let big = () => BigInt.fromStringOrThrow("x")
let date = () => Temporal.PlainDate.fromStringOrThrow("x")
let dur = () => Temporal.Duration.fromStringOrThrow("x")
rescript-tools reanalyze -exception reports only big might throw JsExn. #9094 added eight fromStringOrThrow externals (Duration, Instant, PlainDate, PlainDateTime, PlainMonthDay, PlainTime, PlainYearMonth, ZonedDateTime); none is in the table of throwing library functions in exn_lib.ml, which lists every other stdlib *OrThrow function.
Temporal functions that throw without the OrThrow suffix (arithmetic out of range, invalid arguments to constructors) were not checked.
Possible cause (speculation only)
The table is maintained by hand next to the stdlib, and nothing checks that a new *OrThrow external has an entry.
🤖 Generated with Claude Code
- Langage dominant
- OCaml
- Étoiles
- 7.5k
- Forks
- 484
- Merge moyen
- 17 h 2 min
- PR mergées (30 j)
- 249
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de rescript-lang/rescript
-
Issues from differential semantic drift analysis (b123b7c3 to c10223e8)Peut-être pris @cknitt l’a pris aujourd’hui. Ouverte
rescript-lang/rescript#9186 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
rescript-lang/rescript#9185 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 28/100
rescript-lang/rescript#9184 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
rescript-lang/rescript#9183 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
rescript-lang/rescript#9181 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de rescript-lang/rescript
Issues similaires
-
[bug] A prompt queued during a run is assessed without its textPeut-être pris @jwiegley l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
DevMortimer/pi-warden#211 ·
Les mainteneurs répondent en général sous 1 jour
-
check-relative-paths reports links to existing directories as missingPeut-être pris @reidbaker-agent l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
google/skills_lint.dart#202 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
HorusGoul/trizum#559 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
get_all_subclasses must deduplicate by class identityPeut-être pris @Gambit-Checkmate l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 15 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
versatiles-org/versatiles-rs#306 ·