Skip to content

fix(parser): allow map shorthand after open expressions - #108

Merged
Seddryck merged 1 commit into
mainfrom
fix/map-shorthand-open-pipeline
Aug 25, 2026
Merged

fix(parser): allow map shorthand after open expressions#108
Seddryck merged 1 commit into
mainfrom
fix/map-shorthand-open-pipeline

Conversation

@Seddryck

Copy link
Copy Markdown
Owner

Summary

  • allow map shorthand nodes to follow operations in open-expression pipelines
  • preserve the authored |> form and existing map_shorthand CST shape
  • add grammar and typed C# regression coverage for filter(even) |> add(1)

Validation

  • npx tree-sitter test --file-name map-shorthand.txt (10 passed)
  • dotnet test bindings/csharp/Expressif.Syntax.Tests/Expressif.Syntax.Tests.csproj --framework net8.0 --no-restore --nologo -m:1 (174 passed)

Close #107

@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: 93a051ab-efe2-4973-8ade-678e8d5a736b


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 force-pushed the fix/map-shorthand-open-pipeline branch from 17a96db to f0434eb Compare August 25, 2026 22:46
@Seddryck
Seddryck merged commit 02fcc15 into main Aug 25, 2026
25 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.

Map shorthand should compose after open expressions

1 participant