fix: respect defaultValue when arg is AstNodeType.EMPTY

[marcin-kordas-hoc]

Problem

When a user writes an empty argument (e.g. =VLOOKUP(2,A1:B3,2,)), the parser
produces AstNodeType.EMPTY which evaluates to EmptyValue (a Symbol sentinel).

The guard in FunctionPlugin.coerceArgumentsToRequiredTypes checked only for
undefined, so EmptyValue passed through and was coerced to 0/false instead
of using the parameter's declared defaultValue.

Affected functions: any plugin parameter with defaultValue ≠ 0/false
(e.g. LOG base=10, VLOOKUP/HLOOKUP sorted=true, MATCH matchType=1, ADDRESS abs=1).

Fix

In FunctionPlugin.coerceArgumentsToRequiredTypes: when the runtime value is
EmptyValue and the parameter declares an explicit defaultValue, substitute
defaultValue instead of passing EmptyValue to type coercion.

Tests

Regression tests in companion PR in handsontable/hyperformula-tests
(branch fix/empty-default-value): covers VLOOKUP, HLOOKUP, MATCH, ADDRESS, LOG.

---

Note

Medium Risk
Touches central interpreter argument-coercion logic, which can subtly change results for many functions when users pass empty trailing arguments.

Overview
Fixes function argument handling so an explicitly empty argument (parsed as AstNodeType.EMPTY and evaluated to EmptyValue) now uses the parameter’s declared defaultValue instead of being coerced as a real value.

This updates FunctionPlugin.coerceArgumentsToRequiredTypes to substitute defaultValue when rawArg === EmptyValue (and a defaultValue is defined), aligning behavior with omitted arguments across functions like VLOOKUP/MATCH/LOG that rely on non-zero defaults.

^{Written by Cursor Bugbot for commit f68ecd8. This will update automatically on new commits. Configure here.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.18%. Comparing base (4042b04) to head (f68ecd8).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #1631   +/-   ##
========================================
  Coverage    97.18%   97.18%           
========================================
  Files          172      172           
  Lines        14836    14841    +5     
  Branches      3258     3262    +4     
========================================
+ Hits         14418    14423    +5     
  Misses         418      418           

Files with missing lines	Coverage Δ
src/interpreter/plugin/FunctionPlugin.ts	99.04% <100.00%> (+0.02%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

Performance comparison of head (f68ecd8) vs base (4042b04)

                                     testName |   base |   head |  change
-------------------------------------------------------------------------
                                      Sheet A | 487.97 | 510.13 |  +4.54%
                                      Sheet B | 157.51 | 165.76 |  +5.24%
                                      Sheet T | 139.65 | 142.29 |  +1.89%
                                Column ranges | 471.82 | 472.25 |  +0.09%
Sheet A:  change value, add/remove row/column |  15.76 |  17.53 | +11.23%
 Sheet B: change value, add/remove row/column | 138.56 | 138.14 |  -0.30%
                   Column ranges - add column | 152.63 | 156.85 |  +2.76%
                Column ranges - without batch | 452.06 | 471.83 |  +4.37%
                        Column ranges - batch | 113.42 | 117.22 |  +3.35%