feat(webui): Sentry/GlitchTip observability console & correlation view (Closes #649) #917

Merged
sysadmin merged 1 commits from feat/issue-649-sentry-console-correlation into master 2026-07-27 15:23:36 -05:00
Owner

Closes #649

Implements Web Console Phase 4 observability surface:

  • Provider connection health widgets (Sentry & GlitchTip)
  • Error correlation links between provider incidents and Gitea issues
  • Gated reconcile & link controls
  • Unit test suite and ADR architecture docs
Closes #649 Implements Web Console Phase 4 observability surface: - Provider connection health widgets (Sentry & GlitchTip) - Error correlation links between provider incidents and Gitea issues - Gated reconcile & link controls - Unit test suite and ADR architecture docs
jcwalker3 added 1 commit 2026-07-25 17:48:49 -05:00
Owner

repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #917
issue: #649
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: prgs-reviewer-55686-d18373f9
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr917-649
phase: claimed
candidate_head: f2dbf30e81
target_branch: master
target_branch_sha: 2b4e43042a
last_activity: 2026-07-25T23:08:16Z
expires_at: 2026-07-25T23:18:16Z
blocker: none

<!-- mcp-review-lease:v1 --> repo: Scaled-Tech-Consulting/Gitea-Tools pr: #917 issue: #649 reviewer_identity: sysadmin profile: prgs-reviewer session_id: prgs-reviewer-55686-d18373f9 worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr917-649 phase: claimed candidate_head: f2dbf30e81cb0a8f645b708db8433713260d1b3d target_branch: master target_branch_sha: 2b4e43042a34f4e29617378ae79a7f5a3d312688 last_activity: 2026-07-25T23:08:16Z expires_at: 2026-07-25T23:18:16Z blocker: none
sysadmin approved these changes 2026-07-25 18:09:59 -05:00
sysadmin left a comment
Owner

Formal review — PR #917 (head f2dbf30) · Closes #649

Verdict: APPROVE

Scope

On-issue Phase 4 console work for Sentry/GlitchTip observability. 10 files, +721 lines. No unrelated churn. Mergeable into master.

Acceptance criteria mapping (#649)

  1. Connection health visible — ProviderHealth cards (configured / missing_token / disabled / healthy).
  2. Linked issues for fingerprints — list_incident_links + correlation table.
  3. Create/link gated + dry-runable — gated actions registered; MVP attempt_action fails closed.
  4. Tests with mocks — 7 unit/route/authz tests; redaction asserted.
  5. Docs reference ADR — docs/observability/webui-observability-console.md + authz matrix Phase 4 rows.

Safety

  • Tokens never returned; credentials_present only; console_redaction on outbound dicts.
  • Raw incidents not allocator work.
  • Author jcwalker3 / reviewer sysadmin (not self-review).

Validation

pytest tests/test_webui_observability.py → 7 passed. Worktree HEAD f2dbf30 matches pin.

Canonical PR State

STATE: approved-awaiting-merge
WHO_IS_NEXT: merger
NEXT_ACTION: merge PR #917 at pinned head f2dbf30e81 when merge policy allows
NEXT_PROMPT:

Merger session: call gitea_allocate_next_work(apply=true, role='merger') for prgs/Scaled-Tech-Consulting/Gitea-Tools. If PR #917 is assigned, merge only that PR at head f2dbf30e81cb0a8f645b708db8433713260d1b3d after merge preflight. Do not self-select. Do not force-push.

WHAT_HAPPENED: Formal APPROVE submitted for PR #917 after full code review and tests.
WHY: #649 acceptance criteria met; redaction and fail-closed gates verified; scope clean.
ISSUE: #649
HEAD_SHA: f2dbf30e81
REVIEW_STATUS: approved / approval_at_current_head
MERGE_READY: true
BLOCKERS: none
VALIDATION: pytest tests/test_webui_observability.py → 7 passed; reviewer approved at head f2dbf30e81
NATIVE_REVIEW_PROOF: transport=native_mcp; entrypoint=mcp_server; tool=gitea_submit_pr_review
LAST_UPDATED_BY: sysadmin / prgs-reviewer

## Formal review — PR #917 (head f2dbf30) · Closes #649 **Verdict: APPROVE** ### Scope On-issue Phase 4 console work for Sentry/GlitchTip observability. 10 files, +721 lines. No unrelated churn. Mergeable into master. ### Acceptance criteria mapping (#649) 1. Connection health visible — ProviderHealth cards (configured / missing_token / disabled / healthy). 2. Linked issues for fingerprints — list_incident_links + correlation table. 3. Create/link gated + dry-runable — gated actions registered; MVP attempt_action fails closed. 4. Tests with mocks — 7 unit/route/authz tests; redaction asserted. 5. Docs reference ADR — docs/observability/webui-observability-console.md + authz matrix Phase 4 rows. ### Safety - Tokens never returned; credentials_present only; console_redaction on outbound dicts. - Raw incidents not allocator work. - Author jcwalker3 / reviewer sysadmin (not self-review). ### Validation pytest tests/test_webui_observability.py → 7 passed. Worktree HEAD f2dbf30 matches pin. ## Canonical PR State STATE: approved-awaiting-merge WHO_IS_NEXT: merger NEXT_ACTION: merge PR #917 at pinned head f2dbf30e81cb0a8f645b708db8433713260d1b3d when merge policy allows NEXT_PROMPT: ```text Merger session: call gitea_allocate_next_work(apply=true, role='merger') for prgs/Scaled-Tech-Consulting/Gitea-Tools. If PR #917 is assigned, merge only that PR at head f2dbf30e81cb0a8f645b708db8433713260d1b3d after merge preflight. Do not self-select. Do not force-push. ``` WHAT_HAPPENED: Formal APPROVE submitted for PR #917 after full code review and tests. WHY: #649 acceptance criteria met; redaction and fail-closed gates verified; scope clean. ISSUE: #649 HEAD_SHA: f2dbf30e81cb0a8f645b708db8433713260d1b3d REVIEW_STATUS: approved / approval_at_current_head MERGE_READY: true BLOCKERS: none VALIDATION: pytest tests/test_webui_observability.py → 7 passed; reviewer approved at head f2dbf30e81cb0a8f645b708db8433713260d1b3d NATIVE_REVIEW_PROOF: transport=native_mcp; entrypoint=mcp_server; tool=gitea_submit_pr_review LAST_UPDATED_BY: sysadmin / prgs-reviewer
Owner

repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #917
issue: #649
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 18881-b7fc208a0863
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr917-649
phase: claimed
candidate_head: f2dbf30e81
target_branch: master
target_branch_sha: c30b381eb2
last_activity: 2026-07-27T20:23:07Z
expires_at: 2026-07-27T20:33:07Z
blocker: none

<!-- mcp-review-lease:v1 --> repo: Scaled-Tech-Consulting/Gitea-Tools pr: #917 issue: #649 reviewer_identity: sysadmin profile: prgs-merger session_id: 18881-b7fc208a0863 worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr917-649 phase: claimed candidate_head: f2dbf30e81cb0a8f645b708db8433713260d1b3d target_branch: master target_branch_sha: c30b381eb2756c4ba9cbb2bf7c3f2d6f31b701e7 last_activity: 2026-07-27T20:23:07Z expires_at: 2026-07-27T20:33:07Z blocker: none
sysadmin merged commit dc5d2c8caa into master 2026-07-27 15:23:36 -05:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Scaled-Tech-Consulting/Gitea-Tools#917