Skip to content

fix(erlang): resolve selective imports by arity - #1622

Open
danusha2345 wants to merge 18 commits into
colbymchenry:mainfrom
danusha2345:fix/1610-erlang-import-arity-main
Open

danusha2345 wants to merge 18 commits into
colbymchenry:mainfrom
danusha2345:fix/1610-erlang-import-arity-main

Conversation

@danusha2345

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #1615 for the remaining #1610 resolution correctness gaps.

  • resolve -import(Module, [f/N]) to the exact exported module, function, and arity
  • keep bare Erlang calls scoped to the current module so BIFs and out-of-repo calls cannot bind to unrelated project functions
  • bump the extraction version so existing Erlang indexes are rebuilt
  • add regressions for selective imports and same-named BIF collisions

This intentionally does not include the comment-arity change from #1619.

Verification

  • npm run build:kernel
  • npm run build
  • npm test -- --reporter=dot — 194 files, 3231 passed, 10 skipped
  • focused Erlang suite — 10 passed

@danusha2345
danusha2345 force-pushed the fix/1610-erlang-import-arity-main branch from ab7b791 to d38f6e8 Compare August 28, 2026 07:03
@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (3adf067, post-#1770) into this branch: head e3a1e15. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (43271f3) into this branch: head cb2ba35. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (85550eb) into this branch: head e09dd54. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (8733c28) into this branch: head d0660c6. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (8c04734) into this branch: head e347d7b. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (ee83636) into this branch: head 871f90f. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (e720f6c) into this branch: head 3cda01f. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (de5adba) into this branch: head ad25340. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (cece072) into this branch: head 554a465. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (040ba38) into this branch: head 1f0d677. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (374b3b4) into this branch: head b854f53. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (71d049c) into this branch: head 50430a2. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (9181dd1) into this branch: head fc3dd29. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (64bd45c) into this branch: head 4fd0647. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

@danusha2345

Copy link
Copy Markdown
Contributor Author

Merged current main (748311f) into this branch: head cbf8175. The only conflict was the CHANGELOG entry; no source conflict. tsc clean, the branch's tests pass.

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