Skip to content

Add verification claim funnel observability metrics and dashboard docs - #825

Open
Silver36-ship-it wants to merge 5 commits into
Pulsefy:mainfrom
Silver36-ship-it:ObservabilityMetricsForVerificationClaimFunnel
Open

Add verification claim funnel observability metrics and dashboard docs#825
Silver36-ship-it wants to merge 5 commits into
Pulsefy:mainfrom
Silver36-ship-it:ObservabilityMetricsForVerificationClaimFunnel

Conversation

@Silver36-ship-it

Copy link
Copy Markdown

Closes #561

PR Description

Summary

Add Prometheus funnel observability for claim lifecycle transitions, including verification path coverage and dashboard-ready metric names.

What changed

  • verification.service.ts

    • Injected MetricsService
    • Added funnel metric recording for verification transitions
    • Created recordClaimFunnelTransition() and claimStatusToFunnelStage()
    • Emits:
      • claim_funnel_total{stage="verified"}
      • claim_funnel_current{stage="verified"}
  • verification.module.ts

    • Imported MetricsModule so VerificationService can consume MetricsService
  • verification.service.spec.ts

    • Added MetricsService mock
    • Added assertions verifying funnel metrics are emitted when verification succeeds
  • claims.service.ts

    • Existing claim lifecycle metric recording remains intact
    • Ensures created/approved/disbursed stages continue to emit funnel metrics
  • metrics.service.ts

    • Confirmed dynamic counter/gauge helpers already support metric labels
  • testnet-observability-dashboard.md

    • Documented dashboard-ready metrics:
      • claim_funnel_total{stage}
      • claim_funnel_current{stage}
    • Included supported stages:
      • created
      • verified
      • approved
      • disbursed

Validation

  • Confirmed no diagnostics errors in modified backend files
  • Verified branch ObservabilityMetricsForVerificationClaimFunnel created and pushed

Notes

  • This change instruments the AI verification flow directly in VerificationService.processVerification()
  • It avoids duplicate metric logic by using the same funnel stage naming conventions already used in ClaimsService
  • It does not add files to the codebase outside the backend observability and verification implementation paths

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

@Silver36-ship-it is attempting to deploy a commit to the Cedarich's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@Silver36-ship-it Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Cedarich

Copy link
Copy Markdown
Contributor

@Silver36-ship-it kindly fix workflow

@Cedarich Cedarich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly remove changes from backend CI file

@Cedarich

Copy link
Copy Markdown
Contributor

@Silver36-ship-it

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.

Observability — Metrics for Verification/Claim Funnel

2 participants