cl: compileCallExpr convKwargs bugfix - #2685

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request significantly improves the maintainability and correctness of function call compilation by refactoring keyword argument processing into a standalone utility function. This change isolates complex logic, making it easier to understand and test, and includes new tests to cover specific error conditions related to variadic functions and keyword arguments. Additionally, a minor but important fix was applied to a type initialization method.

Highlights

• Refactoring: The logic for converting keyword arguments in function calls has been extracted from compileCallExpr into a new, dedicated helper function named convKwargs.
• Testing: Two new test cases, TestConvKwargs1 and TestConvKwargs2, were added to builtin_test.go to specifically validate the error handling of the convKwargs function, particularly for variadic functions.
• Bugfix: A subtle bug in fnType.initFuncs related to pointer assignment within a loop was corrected to ensure proper type chain initialization.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Codecov Report

❌ Patch coverage is 92.30769% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.11%. Comparing base (c62581b) to head (fbe11df).
⚠️ Report is 4 commits behind head on main.

@@            Coverage Diff             @@
##             main    #2685      +/-   ##
==========================================
+ Coverage   94.09%   94.11%   +0.02%     
==========================================
  Files          32       32              
  Lines       10045    10046       +1     
==========================================
+ Hits         9452     9455       +3     
+ Misses        422      421       -1     
+ Partials      171      170       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Good refactor — extracting convKwargs and moving it inside the overload loop is the right approach for correct per-candidate kwargs conversion. The initFuncs cleanup (p = fn instead of p = p.next) is a nice clarity improvement.

Main concerns: (1) convKwargs panics inside the overload loop, which prevents fallthrough to other candidates unlike compileCallArgs errors. (2) The recordCallExpr change from fnt to fn.sig and from v to nv alters IDE recording semantics — worth verifying with recorder tests.