Skip to content

fix: strict pipes and AOT clippy hygiene - #75

Merged
gRoussac merged 1 commit into
Interchouette-ITC:devfrom
Interchouette:feat/strict-pipes-aot-clippy
Sep 7, 2026
Merged

gRoussac merged 1 commit into
Interchouette-ITC:devfrom
Interchouette:feat/strict-pipes-aot-clippy

Conversation

@Interchouette

Copy link
Copy Markdown
Member

Summary

  • Builtin and demo/parity pipes now return real EvalError::TypeMismatch on bad types/args; drop unnecessary_wraps allows while keeping PipeFn as Result.
  • Remove fn-level AOT needless_pass_by_value / redundant_clone allows; fix for-loop emission (row / item move into Arc) and avoid redundant get_value().clone().
  • Document pipe type rules in docs/SPEC.md (display still maps eval failure to empty via existing unwrap_or).

Test plan

  • make lint
  • make demo-check
  • make demo-build
  • make test
  • make doc
  • make audit && make deny

Make builtin and demo pipes return real TypeMismatch errors, and fix
generated for-loop emission so blanket needless_pass_by_value /
redundant_clone allows are no longer needed.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Sep 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.77419% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
crates/rangular-expr/src/pipe.rs 96.42% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@gRoussac
gRoussac merged commit b0acf61 into Interchouette-ITC:dev Sep 7, 2026
9 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.

2 participants