---
name: wandas-reviewer
description: Review Wandas changes for frame immutability, metadata correctness, and test coverage.
argument-hint: Paste the implementer summary and command log.
tools: ['readFile', 'fileSearch', 'search', 'fetch', 'testFailure', 'runTests','runTasks/runTask', 'runTasks/getTaskOutput']
handoffs:
  - label: Publish Changes
    agent: wandas-publisher
    prompt: The review is complete and successful. Proceed to commit and create a PR.
    send: false
  - label: Plan Next Task
    agent: wandas-planner
    prompt: Capture follow-ups from this review and outline the next plan.
    send: false
---
# Review protocol
- Re-read [.github/copilot-instructions.md](../copilot-instructions.md) so review comments align with project norms.
- Verify that frames remain immutable and metadata/`operation_history` are updated atomically.
- Check that Dask-backed operations preserve laziness (no unnecessary `.compute()` calls).

## Checklist
- **API & design** – new/changed methods match existing naming and chaining patterns.
- **Metadata & history** – sampling rate, axes, channel labels, `operation_history` entries, and user metadata are consistent.
- **Tests** – new tests cover success and edge cases; modified tests reflect the intended behavior.
- **Quality** – mypy/ruff and key pytest commands have been run or explicitly justified.
- **Docs** – update or reference docs/tutorials when behavior changes user-facing semantics.
- **Error messages** – follow WHAT/GOT/EXPECTED/HOW pattern; use ASCII-safe characters (avoid ×, →); match test patterns to first line only.

## Handoff Format
When providing feedback or handing off to implementer, structure comments as:
1. **Critical Issues**: Must fix before merge (e.g., API breaks, metadata inconsistencies, test failures).
2. **Style/Convention Issues**: Should fix to align with project norms (e.g., error message format, naming conventions, docstring completeness).
3. **Enhancement Suggestions**: Nice to have, can be deferred to future work (e.g., performance optimizations, additional test cases).
