Skip to content

Refactor/egui native desktop#12

Open
diegodev18 wants to merge 6 commits into
MrArcher23:mainfrom
diegodev18:refactor/egui-native-desktop
Open

Refactor/egui native desktop#12
diegodev18 wants to merge 6 commits into
MrArcher23:mainfrom
diegodev18:refactor/egui-native-desktop

Conversation

@diegodev18

Copy link
Copy Markdown

Summary

Related issue

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that changes existing behavior)
  • Refactor / internal cleanup (no behavior change)
  • Documentation only

How tested

  • cargo test --release passes
  • For parser changes: verified against real ~/.claude data (e.g. the /verify-parser skill, or ./target/release/arrow --list)
  • For UI changes: checked in the web dev server (npm run dev) and/or the Tauri app (cargo tauri dev)

Checklist

  • cargo fmt --all --check leaves no changes
  • cargo clippy --all-targets --all-features -- -D warnings passes (CI treats any clippy warning as an error)
  • New code comments, identifiers, and UI text are in English
  • Change respects arrow's honesty principle (does not claim more than the native data knows; only Edit/Write/MultiEdit edits are captured, never Bash changes)
  • Documentation updated where needed (README.md / ROADMAP.md / SPEC.md / CLAUDE.md)
  • Commit messages are in the imperative mood

Copilot AI review requested due to automatic review settings June 6, 2026 07:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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