wavefnd / wavefnd/Wave

Migrate postfix and aggregate expression errors to structured diagnostics

Open
#399 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue needs testing
Dominant language
Rust
Stars
53
Forks
16
Avg merge
4h 22m
Merged PRs (30d)
46

Description

Tracked by #346.

## Why

Method calls, indexing, qualified calls, grouped expressions, arrays, and struct literals still have ad-hoc parser prints in `front/parser/src/expr/`.

## Scope

- Convert missing member names, closing delimiters, struct field separators, and qualified-name failures to structured parser errors.
- Cover postfix calls/indexing and primary aggregate expressions.
- Add focused negative tests that assert the construct context and source location.
- Do not change expression precedence, method dispatch, or aggregate semantics.

## Completion criteria

- [ ] Converted expression paths return errors without direct output.
- [ ] Group, call, index, array, struct literal, and qualified-name failures are covered.
- [ ] Valid nested postfix expressions still parse.
- [ ] Human and JSON error modes remain consistent.

Contributor guide

Open the contributing guide

Research direction

Begin in front/parser/src/expr/ and trace the postfix, primary aggregate, grouped-expression, call, index, array, struct-literal, and qualified-name parsing paths that still print errors directly. Add focused negative tests for construct context and source location, then verify valid nested postfix expressions and consistency between human and JSON error modes without changing precedence, dispatch, or aggregate semantics.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
compilers
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.