docs: surface 80% coverage gate and scripts/debug runners

[senamakel]

Summary

• Document the existing diff-cover ≥ 80% merge gate (Vitest + cargo-llvm-cov) in the PR template, both issue templates, CLAUDE.md, and AGENTS.md so agents know it blocks merge.
• Document the scripts/debug/ wrappers (pnpm debug unit/e2e/rust/logs) in CLAUDE.md and AGENTS.md — bounded stdout, full output teed to target/debug-logs/.

Problem

Coverage gate enforced by .github/workflows/coverage.yml (#1104) and the agent-friendly debug runners (#1105) shipped recently, but neither was reflected in the agent-facing docs or the PR/issue templates. Agents writing PRs didn't know about the 80% changed-lines requirement and reached for raw Vitest/WDIO/cargo invocations instead of the bounded wrappers.

Solution

• PR template and both issue templates: new acceptance-criteria checklist item linking to coverage.yml.
• CLAUDE.md and AGENTS.md: new "Agent debug runners" + "Coverage requirement (merge gate)" sections with the full pnpm debug surface.

Docs-only — no behavior change.

Submission Checklist

• Tests added or updated (happy path + at least one failure / edge case) per docs/TESTING-STRATEGY.md — N/A: docs-only change
• Diff coverage ≥ 80% — N/A: docs-only change, no executable lines
• Coverage matrix updated — N/A: behaviour-only / docs-only change
• All affected feature IDs from the matrix are listed in the PR description under ## Related — N/A: docs-only change
• No new external network dependencies introduced — N/A: docs-only change
• Manual smoke checklist updated if this touches release-cut surfaces — N/A: docs-only change
• Linked issue closed via Closes #NNN in the ## Related section — N/A: no linked issue

Impact

Documentation only. No runtime, build, or test behavior changes.

Related

• Closes:
• Follow-up PR(s)/TODOs:

Summary by CodeRabbit

• Documentation

  • Added comprehensive debug-runner docs for running tests with captured logs and examples
  • Documented coverage standard requiring ≥80% coverage on changed lines

• Chores

  • Updated issue templates to require ≥80% diff coverage for bug and feature reports
  • Updated pull request template with a mandatory coverage checklist
  • Set workflow default shell to bash for CI coverage steps

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0a640a87-62c3-478c-91cd-ca62e0a5fb41

📥 Commits

Reviewing files that changed from the base of the PR and between 14f45fa and 91476e5.

📒 Files selected for processing (1)

• .github/workflows/coverage.yml

---

📝 Walkthrough

Walkthrough

Documentation and repository templates were updated to require a "Diff coverage ≥ 80%" gate for changed lines, documentation for local debug runners was added, and the CI coverage workflow default shell was set to bash.

Changes

Coverage gate + agent debug runners

Layer / File(s)	Summary
Templates — Acceptance Criteria .github/ISSUE_TEMPLATE/bug.md, .github/ISSUE_TEMPLATE/feature.md	Adds checklist item - [ ] **Diff coverage ≥ 80%** linking to the coverage workflow.
PR Submission Checklist .github/PULL_REQUEST_TEMPLATE.md	Adds checklist requiring Diff coverage ≥ 80% for changed lines and documents the tooling (Vitest + cargo-llvm-cov merged via diff-cover) and local commands.
Developer Docs — Debug Runners AGENTS.md, CLAUDE.md	Adds “Agent debug runners (scripts/debug/)” documentation describing pnpm debug wrappers for Vitest, WDIO E2E, and Rust tests, and log inspection helpers.
Developer Docs — Coverage Requirement AGENTS.md, CLAUDE.md	Documents the ≥80% changed-lines coverage merge gate, the CI coverage artifacts (app/coverage/lcov.info + cargo-llvm-cov), and local verification commands (pnpm test:coverage, pnpm test:rust).
CI Workflow Defaults .github/workflows/coverage.yml	Adds top-level defaults.run.shell: bash so all run: steps use bash semantics (e.g., set -o pipefail).

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Dev as Developer
  participant Repo as Repository (PR)
  participant CI as CI Workflow
  participant Vitest as Vitest (JS coverage)
  participant RustCov as cargo-llvm-cov (Rust coverage)
  participant MergeTool as diff-cover

  Dev->>Repo: Open PR + run `pnpm test:coverage` / `pnpm test:rust`
  Repo->>CI: Push triggers `coverage.yml` (bash)
  CI->>Vitest: Run JS tests -> produce `app/coverage/lcov.info`
  CI->>RustCov: Run Rust coverage -> produce merged lcov
  CI->>MergeTool: Merge coverage outputs, run diff-cover vs PR diff
  MergeTool-->>CI: Coverage report (diff coverage %)
  CI->>Repo: Pass/fail merge gate (requires ≥80% diff coverage)

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• ci: pull pnpm into CI image, drop redundant setup steps #1107 — Related edits to .github/workflows/coverage.yml (other CI adjustments to coverage job).
• ci: add diff-aware 80% coverage gate (Vitest + cargo-llvm-cov) #1104 — Implements or adjusts the coverage workflow referenced by the new templates and docs.

Poem

🐰 I hop through patches, tests in paw,

Eighty percent—our tidy law.
Debug runners hum, logs tucked away,
Coverage gates guard the merge today.
A tiny rabbit cheers: "Run tests, hooray!"

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main changes: documenting the 80% coverage gate requirement and the agent debug runner scripts across templates and documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Review rate limit: 1/5 review remaining, refill in 44 minutes and 5 seconds.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}