Skip to content

feat(parser): add ellipsis shorthand for incoming values - #106

Merged
Seddryck merged 1 commit into
mainfrom
codex/105-ellipsis-shorthand
Aug 25, 2026
Merged

feat(parser): add ellipsis shorthand for incoming values#106
Seddryck merged 1 commit into
mainfrom
codex/105-ellipsis-shorthand

Conversation

@Seddryck

Copy link
Copy Markdown
Owner

Summary

  • allow a spread argument to omit its value
  • interpret the omitted value as the existing ellipsis-at-underscore shorthand semantics
  • add corpus coverage for array ellipsis and remove its invalid snapshot

Validation

  • Focused Tree-sitter implicit incoming-value spread test passes
  • Full Tree-sitter corpus: new case passes; seven unrelated existing CRLF snapshot mismatches remain on Windows
  • npm dependency install reaches a pre-existing node-gyp Release-Win32 configuration error

Close #105

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 94890f3c-2274-4dae-a423-479275e1b534


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Seddryck
Seddryck merged commit a38c27b into main Aug 25, 2026
27 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.

Add ellipsis shorthand for the current value

1 participant