Skip to content

feat: add neutral SCM observability#46

Merged
illegalcall merged 1 commit into
feat/27from
feat/45
May 31, 2026
Merged

feat: add neutral SCM observability#46
illegalcall merged 1 commit into
feat/27from
feat/45

Conversation

@whoisasx

Copy link
Copy Markdown
Collaborator

Summary

  • Add internal/scm/logging for provider-neutral SCM field names, event names, correlation IDs, and safe diagnostic helpers.
  • Instrument SCM observer and command service flows with started/completed/failed, cache, audit, refresh, snapshot, backoff, and rate-limit logs.
  • Instrument GitHub REST/GraphQL transport with neutral request/response/failure/rate-limit logs while avoiding tokens, request bodies, GraphQL queries, comment bodies, response bodies, and CI log tails.
  • Keep durable diagnostics concise by attaching normalized failure summaries and avoiding successful transport-call diagnostics on snapshots/results.

Stack

Stacked on #28 / feat/27 because the SCM provider code this issue updates is introduced there.

Closes #45

Tests

  • cd backend && go test -count=1 ./...

@whoisasx whoisasx requested review from Copilot and illegalcall and removed request for Copilot May 30, 2026 19:34
@illegalcall illegalcall merged commit f8f37a1 into feat/27 May 31, 2026
3 checks passed
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.

2 participants