Skip to content

feat(engine): implement systematic debugging mode#69

Merged
robertotru merged 1 commit into
mainfrom
067-systematic-debugging
Jun 3, 2026
Merged

feat(engine): implement systematic debugging mode#69
robertotru merged 1 commit into
mainfrom
067-systematic-debugging

Conversation

@robertotru
Copy link
Copy Markdown
Contributor

What

Implemented the systematic debugging mode within canon-engine to enable a recommendation-only, green-zone workflow for codebase troubleshooting, incident isolation, and validation.

Why

This change is needed to address roadmap feature 01-systematic-debugging, establishing a structured five-step gate system to ensure incidents undergo rigorous verification and post-incident root-cause analysis before closure.

How

  • Registered the new debugging mode in the domain model and classifier.
  • Defined and implemented the debugging mode gatekeeper logic in crates/canon-engine/src/modes/debugging.rs.
  • Enforced a 5-phase artifact verification contract:
    • 01-context (Incident background and initial assumptions)
    • 02-reproduction (Test harness, script, or environment setups)
    • 03-root-cause (Diagnostic isolation and analysis evidence)
    • 04-fix-decision (Remediation choices and tradeoffs)
    • 05-verification (Evidence demonstrating red-to-green state transition)
  • Bumps workspace crates and compatibility manifests to 0.65.0.
  • Removed the completed feature document from roadmap/features/ and updated modes guides (tech-docs/guides/modes.md and docs/guide/canon-modes.md).

Validation

  • Added a full integration test suite in tests/debugging_run.rs verifying gate checks, missing artifact blocks, and transition paths.
  • Ran cargo test and cargo nextest run to ensure all existing and new tests pass cleanly.
  • Validated workspace constraints with cargo clippy --workspace --all-targets --all-features -- -D warnings and cargo fmt.

Checklist

  • The change is scoped to one real problem and avoids unrelated cleanup.
  • I updated docs, specs, or release-facing surfaces when behavior changed.
  • I ran the relevant validation for the surfaces I touched.
  • If I touched install, packaging, or release behavior, I kept those surfaces aligned.
  • If I touched skills or assistant package surfaces, I validated those paths too.

@robertotru robertotru self-assigned this Jun 3, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2026

Codecov Report

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

Files with missing lines Patch % Lines
...non-engine/src/orchestrator/service/mode_change.rs 98.18% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@robertotru robertotru force-pushed the 067-systematic-debugging branch 3 times, most recently from 4ee1c51 to 2a74a6b Compare June 3, 2026 19:40
Bumps workspace versions to 0.65.0, registers the systematic debugging mode, and enforces the five-phase artifact gate checklist (01-context, 02-reproduction, 03-root-cause, 04-fix-decision, 05-verification) for resolving and tracking codebase incidents.
@robertotru robertotru force-pushed the 067-systematic-debugging branch from 2a74a6b to 520ee2d Compare June 3, 2026 20:01
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 3, 2026

@robertotru robertotru merged commit b6f3701 into main Jun 3, 2026
27 checks passed
@robertotru robertotru deleted the 067-systematic-debugging branch June 3, 2026 20:09
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.

1 participant