Skip to content

Rescript v12#37

Merged
zth merged 13 commits into
mainfrom
rescript-v12
Mar 9, 2026
Merged

Rescript v12#37
zth merged 13 commits into
mainfrom
rescript-v12

Conversation

@zth

@zth zth commented Feb 13, 2026

Copy link
Copy Markdown
Owner

No description provided.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 18c695e995

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/ml/Cli.ml
Comment on lines 17 to 21
| [_; "generate-schema"; sourceFolder; outputFolder] ->
GenerateSchema.generateSchema ~writeStateFile:true ~sourceFolder
~debug:false ~outputFolder ~writeSdlFile:false ~printToStdOut:true
| [_; "completion"; path; line; col; currentFile] ->
Completion.completion ~debug:false ~path
~pos:(int_of_string line, int_of_string col)
~currentFile
| [_; "hover"; path; line; col] ->
Hover.hover ~path ~pos:(int_of_string line, int_of_string col) ~debug:false
| [_; "hover-graphql"; path; hoverHint] ->
Hover.hoverGraphQL ~path ~hoverHint |> print_endline
| [_; "definition-graphql"; path; definitionHint] ->
Hover.definitionGraphQL ~path ~definitionHint |> print_endline
| [_; "test"; path] -> Commands.test ~path
run_generate ~sourceFolder ~outputFolder ~writeSdlFile:false
| args when List.mem "-h" args || List.mem "--help" args -> prerr_endline help
| _ ->
prerr_endline help;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Restore private CLI handlers for LSP subcommands

This command parser now accepts only generate-schema, but the editor/LSP path still invokes the private binary with completion, hover, hover-graphql, and definition-graphql (via Utils.callPrivateCli in the CLI layer). In this state those requests fall into the default branch, return exit code 1, and make execFileSync throw, so hover/completion/definition requests fail at runtime instead of returning JSON results.

Useful? React with 👍 / 👎.

@zth zth merged commit ff63f93 into main Mar 9, 2026
5 checks passed
@zth zth deleted the rescript-v12 branch March 9, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant