fix: harden dashboard improve panel #79

Merged
jesse merged 2 commits from phase-8/dashboard-ui-pr6-improve into main 2026-05-13 17:48:18 +02:00
Owner

Summary

  • Harden the dashboard Improve panel against stale renders, scoped tool denials, malformed exports, and hard tool failures.
  • Scope analysis/summarization calls to the selected session and add regression coverage for explicit session handling.
  • Split Improve e2e coverage into its own spec and update Phase 8 dashboard docs.

Tests

  • cargo fmt --check
  • cargo test -p cognix-server --features ui,sse test_pr6_
  • cargo test -p cognix-server --features ui,sse handle_analyze_reasoning
  • cargo test -p cognix-server --features ui,sse --test ui_tool_schema_shapes improve
  • npm test --prefix e2e/dashboard -- improve.spec.ts
  • pre-commit hook: secret detection, cargo fmt --check, cargo clippy
## Summary - Harden the dashboard Improve panel against stale renders, scoped tool denials, malformed exports, and hard tool failures. - Scope analysis/summarization calls to the selected session and add regression coverage for explicit session handling. - Split Improve e2e coverage into its own spec and update Phase 8 dashboard docs. ## Tests - cargo fmt --check - cargo test -p cognix-server --features ui,sse test_pr6_ - cargo test -p cognix-server --features ui,sse handle_analyze_reasoning - cargo test -p cognix-server --features ui,sse --test ui_tool_schema_shapes improve - npm test --prefix e2e/dashboard -- improve.spec.ts - pre-commit hook: secret detection, cargo fmt --check, cargo clippy
fix: harden dashboard improve panel
All checks were successful
CI / Detect Changes (pull_request) Successful in 12s
CI / Format (pull_request) Successful in 16s
CI / Integration Tests (pull_request) Has been skipped
CI / Benchmarks (pull_request) Has been skipped
CI / Conventional Validation (pull_request) Successful in 39s
CI / Clean Build Sample 1 (pull_request) Has been skipped
CI / Clean Build Sample 2 (pull_request) Has been skipped
CI / Clean Build Sample 3 (pull_request) Has been skipped
CI / Clean Build Summary (pull_request) Has been skipped
CI / Security Scan (pull_request) Successful in 21s
CI / Check file lengths (pull_request) Successful in 32s
CI / Documentation (pull_request) Successful in 1m51s
CI / Clippy (pull_request) Successful in 2m34s
CI / Check (linux-aarch64 compile-validation) (pull_request) Successful in 2m28s
CI / Deny (pull_request) Successful in 5m38s
CI / Audit (CVEs) (pull_request) Successful in 6m24s
CI / D-02 Clean Build Gate (pull_request) Successful in 6m32s
CI / Test (pull_request) Successful in 7m31s
CI / Coverage (80% gate) (pull_request) Successful in 8m0s
CI / RSS gate (P-15) (pull_request) Successful in 5m52s
CI / Build (release) (pull_request) Successful in 7m11s
CI / PR Size Check (pull_request) Successful in 12s
CI / CI Report (pull_request) Successful in 5s
e1ff21d0a1
jesse merged commit a260b48e55 into main 2026-05-13 17:48:18 +02:00
jesse deleted branch phase-8/dashboard-ui-pr6-improve 2026-05-13 17:48:18 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jesse/cognix!79
No description provided.