Skip to content

Fix compilation of local lambdas without type annotations - #56

Merged
Y-Nak merged 5 commits into
argotorg:mainfrom
micahscopes:fix/local-lambda-hull
Oct 22, 2026
Merged

Y-Nak merged 5 commits into
argotorg:mainfrom
micahscopes:fix/local-lambda-hull

Conversation

@micahscopes

Copy link
Copy Markdown
Collaborator

Local lambdas without an explicit function type can pass type checking but fail later during compilation. This change carries their inferred callable signatures into specialization, preserving call resolution and capture behavior. Recursive signatures produce a diagnostic rather than overflowing the stack.

Added new compilation and execution fixtures covering the six reported failures, captured values, nested lambdas, and generic calls. All 169 specialization/Hull tests pass, along with the new lambda execution tests and the existing closure execution tests.

A possible follow-up is a shared, context-aware view of a checked body, so later stages consistently access types, callable signatures, and resolution information without separately assembling that context.

@micahscopes
micahscopes requested a review from Y-Nak October 13, 2026 15:55
@micahscopes
micahscopes marked this pull request as draft October 13, 2026 20:45
@Y-Nak Y-Nak mentioned this pull request Oct 21, 2026
@micahscopes
micahscopes marked this pull request as ready for review October 22, 2026 02:10

@Y-Nak Y-Nak left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Y-Nak added 2 commits October 22, 2026 15:26
Preserve both Hull regression test additions and format the new Solcore fixtures for the current repository checks.
Apply the repository formatter to the fixture imported from main so the merged branch passes the fixture formatting check.
@Y-Nak
Y-Nak merged commit 75d6261 into argotorg:main Oct 22, 2026
12 checks passed
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.

2 participants