Skip to content

Update formula errors, bulk results and caching - #4

Open
MudassarTariq wants to merge 1 commit into
mainfrom
fix/sheet-lookups
Open

MudassarTariq wants to merge 1 commit into
mainfrom
fix/sheet-lookups

Conversation

@MudassarTariq

Copy link
Copy Markdown
Member

Bulk formulas now retain blank input positions, and successful lookups return their results when the optional cache is unavailable or a response exceeds its size limit. GET and POST requests share the same error and cache handling, with API keys removed from displayed errors.

The existing formula names, arguments, request formats and shared script-key storage remain unchanged. The README describes shared-key access, blank-row handling and cache limits.

Validation:

  • 21 regression tests pass; the original implementation fails 16 of these tests.
  • 33 README formulas evaluated against a local HTTP simulation using examples from the official API specifications, with GET/POST, per-entry errors, malformed responses and credential-bearing error cases checked.
  • CI runs the regression suite on Node.js 22 and 24. Google services are emulated; actual Google Sheets execution has not been tested.

@MudassarTariq
MudassarTariq marked this pull request as ready for review October 30, 2026 18:27
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