chore: quality improvement pass — tests, coverage, architecture (#3)

* docs: add quality improvement design spec

Covers architecture refactoring (extract input.rs), test coverage
tooling (cargo-llvm-cov, proptest, MockBackend), and code quality
cleanup (dead_code audit, tokio feature slimming).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: add quality improvement implementation plan

10-task plan covering: MockBackend tests, input.rs extraction,
dead_code cleanup, proptest parsers, CI hardening with coverage.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: slim tokio features, add proptest dev-dep

* test: add MockBackend and Collector unit tests

* refactor: extract input handling from main.rs into input.rs

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test: add input handler unit tests

* docs: clarify why std::thread::sleep is used in swap_reset

* chore: remove unnecessary #[allow(dead_code)] suppressions

* test: add proptest property-based tests for parsers

Add property-based tests using proptest to linux.rs (parse_proc_swaps)
and proc_reader.rs (parse_status, parse_stat_cpu_ticks) covering valid
inputs, malformed inputs, and byte-conversion invariants. Also refactor
resolve_key to use a KeyContext struct to resolve pre-existing clippy
too_many_arguments warning.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* ci: add fmt check and coverage with cargo-llvm-cov

* style: apply cargo fmt

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: fix cargo coverage alias and ci llvm-cov regex exclusion

- Switch .cargo/config.toml alias to array syntax so the
  --ignore-filename-regex flag is passed as a discrete argument
  (string form silently dropped the regex under some cargo versions)
- CI coverage step already uses the same exclusion flag to skip
  untestable UI/TUI/event-loop files; line coverage is now 82% on
  testable code, well above the 70% threshold

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix ci coverage

* fix(ci): restore corrupted cache key in actions/cache step

The key: field and end of the path block were mangled into a single
line, causing "Input required and not supplied: key" in CI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: ricsdeol

.cargo/config.toml

@@ -0,0 +1,4 @@
+[alias]
+# Run coverage excluding untestable UI/TUI/event-loop code.
+# Usage: cargo coverage
+coverage = ["llvm-cov", "--ignore-filename-regex", "src/(main|tui|ui/.*)", "--fail-under-lines", "70"]

.github/workflows/ci.yml

@@ -25,7 +25,7 @@ jobs:
         uses: dtolnay/rust-toolchain@stable
         with:
           toolchain: "1.94.1"
-          components: rustfmt, clippy
+          components: rustfmt, clippy, llvm-tools-preview
 
       - name: Cache cargo registry and build artifacts
         uses: actions/cache@v4
@@ -39,6 +39,9 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-cargo-
 
+      - name: Check formatting
+        run: cargo fmt --check
+
       - name: Build
         run: cargo build
 
@@ -47,3 +50,8 @@ jobs:
 
       - name: Tests
         run: cargo test
+
+      - uses: taiki-e/install-action@cargo-llvm-cov
+
+      - name: Coverage
+        run: cargo llvm-cov --ignore-filename-regex "src/(main|tui|ui/.*)" --fail-under-lines 70

.rtk/filters.toml

@@ -0,0 +1,13 @@
+# Project-local RTK filters — commit this file with your repo.
+# Filters here override user-global and built-in filters.
+# Docs: https://github.com/rtk-ai/rtk#custom-filters
+schema_version = 1
+
+# Example: suppress build noise from a custom tool
+# [filters.my-tool]
+# description = "Compact my-tool output"
+# match_command = "^my-tool\\s+build"
+# strip_ansi = true
+# strip_lines_matching = ["^\\s*$", "^Downloading", "^Installing"]
+# max_lines = 30
+# on_empty = "my-tool: ok"

CLAUDE.md

@@ -5,12 +5,12 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 ## Commands
 
 ```bash
-cargo build                   # build debug
-cargo build --release         # build release
-cargo run                     # run (requires Linux for full functionality)
-cargo test                    # run all tests
-cargo test <test_name>        # run a single test
-cargo clippy -- -D warnings   # lint
+rtk cargo build                   # build debug
+rtk cargo build --release         # build release
+rtk cargo run                     # run (requires Linux for full functionality)
+rtk cargo test                    # run all tests
+rtk cargo test <test_name>        # run a single test
+rtk cargo clippy -- -D warnings   # lint
 ```
 
 ## Project: swaptop
@@ -74,9 +74,9 @@ Phase 5 → create-swap wizard, tokio::process::Command, disk space validation
 Run these and fix any issues after implement all phases:
 
 ```bash
-cargo build          # must compile clean (zero warnings)
-cargo clippy -- -D warnings  # must pass with no warnings
-cargo test           # all tests must pass
+rtk cargo build          # must compile clean (zero warnings)
+rtk cargo clippy -- -D warnings  # must pass with no warnings
+rtk cargo test           # all tests must pass
 ```
 
 ## Key constraints
@@ -86,3 +86,95 @@ cargo test           # all tests must pass
 - `swapon`/`swapoff` require root — check `nix::unistd::geteuid() == 0` before calling
 - History is in-memory only (no persistence between sessions) — `VecDeque` capped at `max_history`
 - Chart widget expects `Vec<(f64, f64)>` — convert `Instant` to seconds-since-start when feeding charts
+
+<!-- rtk-instructions v2 -->
+# RTK (Rust Token Killer) - Token-Optimized Commands
+
+## Golden Rule
+
+**Always prefix commands with `rtk`**. If RTK has a dedicated filter, it uses it. If not, it passes through unchanged. This means RTK is always safe to use.
+
+**Important**: Even in command chains with `&&`, use `rtk`:
+```bash
+# ❌ Wrong
+git add . && git commit -m "msg" && git push
+
+# ✅ Correct
+rtk git add . && rtk git commit -m "msg" && rtk git push
+```
+
+## RTK Commands by Workflow
+
+### Build & Compile (80-90% savings)
+```bash
+rtk cargo build         # Cargo build output
+rtk cargo check         # Cargo check output
+rtk cargo clippy        # Clippy warnings grouped by file (80%)
+```
+
+### Test (90-99% savings)
+```bash
+rtk cargo test          # Cargo test failures only (90%)
+```
+
+### Git (59-80% savings)
+```bash
+rtk git status          # Compact status
+rtk git log             # Compact log (works with all git flags)
+rtk git diff            # Compact diff (80%)
+rtk git show            # Compact show (80%)
+rtk git add             # Ultra-compact confirmations (59%)
+rtk git commit          # Ultra-compact confirmations (59%)
+rtk git push            # Ultra-compact confirmations
+rtk git pull            # Ultra-compact confirmations
+rtk git branch          # Compact branch list
+rtk git fetch           # Compact fetch
+rtk git stash           # Compact stash
+rtk git worktree        # Compact worktree
+```
+
+Note: Git passthrough works for ALL subcommands, even those not explicitly listed.
+
+### GitHub (26-87% savings)
+```bash
+rtk gh pr view <num>    # Compact PR view (87%)
+rtk gh pr checks        # Compact PR checks (79%)
+rtk gh run list         # Compact workflow runs (82%)
+rtk gh issue list       # Compact issue list (80%)
+rtk gh api              # Compact API responses (26%)
+```
+
+### Files & Search (60-75% savings)
+```bash
+rtk ls <path>           # Tree format, compact (65%)
+rtk read <file>         # Code reading with filtering (60%)
+rtk grep <pattern>      # Search grouped by file (75%)
+rtk find <pattern>      # Find grouped by directory (70%)
+```
+
+### Analysis & Debug (70-90% savings)
+```bash
+rtk err <cmd>           # Filter errors only from any command
+rtk log <file>          # Deduplicated logs with counts
+rtk json <file>         # JSON structure without values
+rtk deps                # Dependency overview
+rtk env                 # Environment variables compact
+rtk summary <cmd>       # Smart summary of command output
+rtk diff                # Ultra-compact diffs
+```
+
+### Network (65-70% savings)
+```bash
+rtk curl <url>          # Compact HTTP responses (70%)
+rtk wget <url>          # Compact download output (65%)
+```
+
+### Meta Commands
+```bash
+rtk gain                # View token savings statistics
+rtk gain --history      # View command history with savings
+rtk discover            # Analyze Claude Code sessions for missed RTK usage
+rtk proxy <cmd>         # Run command without filtering (for debugging)
+rtk init                # Add RTK instructions to CLAUDE.md
+```
+<!-- /rtk-instructions -->

Cargo.toml

@@ -6,11 +6,14 @@ edition = "2024"
 [dependencies]
 ratatui     = "0.30.0"
 crossterm   = { version = "0.29.0", features = ["event-stream"] }
-tokio       = { version = "1.51.1", features = ["full"] }
+tokio       = { version = "1.51.1", features = ["macros", "rt-multi-thread", "time", "sync", "signal"] }
 tokio-util  = "0.7.18"
 futures     = "0.3.32"
 sysinfo     = "0.38.4"
 nix         = { version = "0.31.2", features = ["process", "signal", "mount", "user"] }
 color-eyre  = "0.6.5"
 human_bytes = "0.4.3"
 glob        = "0.3.3"
+
+[dev-dependencies]
+proptest = "1"