Conversation
There was a problem hiding this comment.
💡 Codex Review
rescript/compiler/syntax/src/res_core.ml
Line 4719 in 7e75974
When an external parameter is itself a bare arrow type that starts with an identifier, this branch constructs the left-hand side and then calls parse_arrow_type_rest without current_type_name_path or inline_types_context. For example, external withTransform: (int => {id: string}) => unit = ... will parse the nested {id: string} return with no inline-record context, so it falls back to object-type parsing instead of synthesizing withTransform.arg0.return.type. Top-level inline records work, but callback parameters like this still fail.
ℹ️ 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".
rescript
@rescript/darwin-arm64
@rescript/darwin-x64
@rescript/linux-arm64
@rescript/linux-x64
@rescript/runtime
@rescript/win32-x64
commit: |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d2d95f3a72
ℹ️ 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".
Support inline record definitions in externals: