feat(tui): classify error reasons on DOWN monitors #54

Merged

lerko merged 3 commits from feat/error-classification into main 2026-06-03 21:29:05 +00:00

Conversation 0 Commits 3 Files Changed 6

+473 -11

lerko commented 2026-06-03 20:35:48 +00:00

Owner

Copy Link

Copy Source

Summary

• Error classifier categorizes raw error strings into DNS/TCP/TLS/HTTP/ICMP/TMO/PRIV
• Status column shows DOWN:DNS, DOWN:TLS, DOWN:HTTP, etc. for instant triage
• Inline NAME column errors prefixed with category tag ([DNS] no such h...)
• Detail panel shows connection chain checklist for HTTP monitors pinpointing failure layer
• All display-side only — no database or model changes

Visual

Status column: DOWN → DOWN:DNS / DOWN:TCP / DOWN:TLS / DOWN:HTTP / DOWN:TMO

Detail panel connection chain:

  ✓ DNS resolve
  ✓ TCP connect
  ✗ TLS handshake    certificate expired
  · HTTP response    (skipped)

Test plan

• go build ./... passes
• go test ./... — all pass, 0 regressions
• 22 new test cases: TestClassifyError, TestCategoryTag, TestConnectionChain, TestFmtStatus_ErrorCategory
• Manual TUI test: trigger DOWN monitor, verify status badge + inline error + detail chain
• Verify narrow terminal (80 cols) renders correctly

## Summary - Error classifier categorizes raw error strings into DNS/TCP/TLS/HTTP/ICMP/TMO/PRIV - Status column shows `DOWN:DNS`, `DOWN:TLS`, `DOWN:HTTP`, etc. for instant triage - Inline NAME column errors prefixed with category tag (`[DNS] no such h...`) - Detail panel shows connection chain checklist for HTTP monitors pinpointing failure layer - All display-side only — no database or model changes ## Visual **Status column:** `DOWN` → `DOWN:DNS` / `DOWN:TCP` / `DOWN:TLS` / `DOWN:HTTP` / `DOWN:TMO` **Detail panel connection chain:** ``` ✓ DNS resolve ✓ TCP connect ✗ TLS handshake certificate expired · HTTP response (skipped) ``` ## Test plan - [x] `go build ./...` passes - [x] `go test ./...` — all pass, 0 regressions - [x] 22 new test cases: `TestClassifyError`, `TestCategoryTag`, `TestConnectionChain`, `TestFmtStatus_ErrorCategory` - [x] Manual TUI test: trigger DOWN monitor, verify status badge + inline error + detail chain - [x] Verify narrow terminal (80 cols) renders correctly

lerko added 1 commit 2026-06-03 20:35:49 +00:00

feat(tui): classify error reasons on DOWN monitors ... 3d7ab5a49e

Categorize raw error strings into DNS/TCP/TLS/HTTP/ICMP/TMO/PRIV
so users get instant triage from the monitor list without opening
the detail panel.

- Status column shows DOWN:DNS, DOWN:TLS, DOWN:HTTP, etc.
- Inline NAME column errors prefixed with category tag [DNS], [TLS]
- Detail panel shows connection chain checklist for HTTP monitors
  (✓ DNS → ✓ TCP → ✗ TLS → · HTTP) pinpointing failure layer
- All display-side only — no database or model changes

lerko added 1 commit 2026-06-03 20:57:30 +00:00

fix(tui): remove error truncation from detail panel ... c25614c098

Error row now word-wraps to terminal width instead of hard-truncating.
Probe results and state change errors show full text.

lerko added 1 commit 2026-06-03 21:24:34 +00:00

fix(tui): classify safedial "failed to connect" as TCP ... c0ad51af9c

Error from safedial.go fell through to ErrCatUnknown, showing plain
DOWN instead of DOWN:TCP.

lerko merged commit c0ad51af9c into main 2026-06-03 21:29:05 +00:00

lerko deleted branch feat/error-classification 2026-06-03 21:29:05 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

distribution

Packaging, publishing, branding

feature

New functionality

polish

UX/visual refinement

refactor

Code quality, no behavior change

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

lerko (Tyler Koenig)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: lerkolabs/uptop#54