Skip to content

Fix parser bugs: underscore placeholder and spread pipe operator precedence - #72

Merged
mcclowes merged 1 commit into
mainfrom
claude/bug-bash-s8ucs
Dec 21, 2025
Merged

mcclowes merged 1 commit into
mainfrom
claude/bug-bash-s8ucs

Conversation

@mcclowes

Copy link
Copy Markdown
Owner
  • Allow underscore '_' as a placeholder expression in function call arguments
    (e.g., add(3, _) where _ represents the piped value)

  • Fix spread pipe operator precedence so that pipes inside function bodies
    are correctly parsed when the function is a spread pipe operand
    (e.g., list />>> (t) -> t.amount /> double now works correctly)

  • Add inPipelineLiteral context flag to distinguish between pipeline
    literal context (where pipes are stage separators) and regular expression
    context (where pipes inside function bodies belong to the function)

  • Rename test variable 'flatten' to 'flattenLists' to avoid conflict with
    builtin flatten function

…edence

- Allow underscore '_' as a placeholder expression in function call arguments
  (e.g., `add(3, _)` where `_` represents the piped value)

- Fix spread pipe operator precedence so that pipes inside function bodies
  are correctly parsed when the function is a spread pipe operand
  (e.g., `list />>> (t) -> t.amount /> double` now works correctly)

- Add inPipelineLiteral context flag to distinguish between pipeline
  literal context (where pipes are stage separators) and regular expression
  context (where pipes inside function bodies belong to the function)

- Rename test variable 'flatten' to 'flattenLists' to avoid conflict with
  builtin flatten function
@vercel

vercel Bot commented Dec 21, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
lea Ready Ready Preview, Comment Dec 21, 2025 0:45am

@mcclowes
mcclowes merged commit d891b0b into main Dec 21, 2025
2 of 6 checks passed
@mcclowes
mcclowes deleted the claude/bug-bash-s8ucs branch December 21, 2025 00:45

This branch was successfully deployed

1 active deployment
Preview — 184160fd Deployed Dec 21, 2025 by vercel[bot]
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.

2 participants