Publish the parser as a standalone OCaml library
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- ocaml
- Ambito
- build-system
Direzione di ricerca
Start by reading the dune graph and the listed blocker issues #8567 and #8624, then trace the parser’s dependency closure with ocamldep. The proposed work spans AST boundaries, dependencies, namespacing, global state, C stubs, packaging, and CI, so it is not a small standalone change. Done means a clean-switch dune build -p rescript-syntax succeeds with only ocaml and dune dependencies.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Goal
Publish the ReScript parser/printer as a reusable OCaml library that needs only ocaml and dune.
Current state
The dune graph is syntax → ml → {ext, flow_parser}. flow_parser is the only part with heavy OPAM dependencies (ppxlib, ppx_deriving, ppx_gen_rec, wtf8, plus a vendored sedlex ppx). The parser pulls all of them in.
Transitive module closure of Res_core / Res_printer / Res_driver (via ocamldep):
| library | modules reached |
|---|---|
syntax/src |
16 / 24 |
ml |
~18 / 91 |
ext |
20 / 66 |
flow_parser |
27 / 27 |
The parser needs only a small part of ml/ext. A few edges drag in the rest.
Blockers
- The AST layer depends on the Flow JS parser.
ml/ast_payload.mlcallsParser_flowandClassify_functionto classify%rawpayloads.ml/ast_helper.mlusesAst_payload, so the plain AST builder depends onflow_parser.- Related: #8567 (replacing/vendoring the Flow parser). This issue needs Flow out of the parser's dependency closure either way.
- Compiler semantics live in
Parsetree.Prim_ffi of {spec: External_ffi_types.t}andPrim_inline_consttie the syntax tree to FFI/codegen types. Tracked in #8624 (section C / step B). - No library boundary.
syntaxdepends on all ofml(the type checker). - No namespacing. All libraries are
(wrapped false). Module names clash withcompiler-libs/ ppxlib (Location,Misc,Warnings,Ident,Parsetree,Asttypes,Longident,Ast_helper,Printast,Pprintast) and are generic (Literals,Map_gen,Identifiable). - Global mutable state.
Location→Clflags/Warnings/Misc. Diagnostics go through global refs, stderr and exceptions. - C stubs and build profiles.
exthas C stubs reachable from the parser (Ext_platform_primitives). Native vs. playground behavior is chosen with thebrowserprofile and copy rules instead ofselector virtual libraries.
Proposed plan
- Split
Ast_payloadinto a pure payload-inspection part and a%raw/%reclassification part. Move the classification tofrontend/core, and makeAst_helperflow-free. This alone drops ppxlib/ppx_deriving/ppx_gen_rec/wtf8 from the parser. - Move
Prim_ffi/Prim_inline_constout of the parsetree (see #8624). - Extract a small
rescript_astlibrary (Parsetree, Asttypes, Longident, Location, Ast_helper, pure Ast_payload, neededexthelpers). Makesyntaxdepend on it instead ofml. Decide wherejsx_ppxandRes_outcome_printer(Outcometree) go. - Make the AST library free of C stubs, or give it a pure-OCaml fallback, so it works under js_of_ocaml without a custom profile.
- Wrap the published libraries (
(wrapped true)or(wrapped (transition …))). - Expose a parser API that returns diagnostics and takes explicit config instead of relying on global state.
- Add an opam package (e.g.
rescript-syntax) withpublic_names. Its only dependencies should beocamlanddune. - Align with #8624 (parsetree v1) for API/AST stability.
- Confirm licensing for the package: compiler-derived files are LGPL with the OCaml linking exception.
- Add a CI job that builds the package alone in a clean switch (
dune build -p rescript-syntax) to catch dependency regressions.
- Lingua principale
- OCaml
- Stelle
- 7.5k
- Fork
- 485
- Merge medio
- 19h 40m
- PR unite (30g)
- 54
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di rescript-lang/rescript
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 22/100
rescript-lang/rescript#8733 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
rescript-lang/rescript#8726 ·
I maintainer di solito rispondono entro 1 giorno
-
Integer range patterns ending at 2147483647 generate incorrect JavaScriptForse già presa @fhammerschmidt l’ha presa 5 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 70/100
rescript-lang/rescript#8716 ·
I maintainer di solito rispondono entro 1 giorno
-
Integer range patterns on record fields match values below the rangeForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 76/100
rescript-lang/rescript#8713 ·
I maintainer di solito rispondono entro 1 giorno
-
A regex literal that starts a statement is parsed as division, and the formatter removes the parentheses that prevent itForse già presa @fhammerschmidt l’ha presa 5 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 68/100
rescript-lang/rescript#8688 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di rescript-lang/rescript
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
lightpanda-io/browser#3852 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
hiero-ledger/hiero-sdk-js#4476 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 85/100
pytest-dev/pluggy#757 ·
I maintainer di solito rispondono entro 1 giorno
-
status: team-only type: dependency-upgrade
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
spring-projects/spring-boot#52098 ·
I maintainer di solito rispondono entro 1 giorno
-
Bump Firebase JS SDK (12.19.0 → 13.0.0)Forse già presa @SelaseKay l’ha presa oggi. ApertaNeeds Attention type: enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
invertase/react-native-firebase#9364 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno