Skip to content

Refactor JanusQL parsing around an AST#12

Merged
argahsuknesib merged 1 commit intomainfrom
codex/janusql-ast-parser
Apr 9, 2026
Merged

Refactor JanusQL parsing around an AST#12
argahsuknesib merged 1 commit intomainfrom
codex/janusql-ast-parser

Conversation

@argahsuknesib
Copy link
Copy Markdown
Collaborator

Summary

  • replace regex-based JanusQL clause parsing with an explicit AST-backed parser
  • keep the existing ParsedJanusQuery output shape so the rest of Janus continues to work
  • add focused AST parser tests for register parsing, multiline window clauses, ON LOG sliding windows, and nested WHERE window bodies

Validation

  • cargo fmt
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test --all-features

@argahsuknesib argahsuknesib merged commit fcd869d into main Apr 9, 2026
14 checks passed
@argahsuknesib argahsuknesib deleted the codex/janusql-ast-parser branch April 9, 2026 08:11
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