Skip to content

[codex] Add service details overview with request stats - #96

Merged
arjunkomath merged 8 commits into
mainfrom
feat/service-details-request-stats
Jul 2, 2026
Merged

arjunkomath merged 8 commits into
mainfrom
feat/service-details-request-stats

Conversation

@techulus-agent

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the service deployment canvas with a compact service details overview.
  • Add an authenticated request-stats API for services backed by VictoriaLogs HTTP access-log aggregation.
  • Add focused tests for request-stat parsing, bucket filling, and range fallback behavior.

Details

  • The overview shows endpoints, service status, servers, resources, running/configured instances, source, created time, and links to existing Logs/Requests tabs.
  • Request stats are aggregated with LogsQL time buckets and 5xx-only error counts, avoiding the capped raw request-log endpoint.
  • Logging-disabled and empty-log states return non-breaking empty stats payloads.

Validation

  • pnpm exec biome check 'components/service/details/service-details-overview.tsx' 'lib/victoria-logs.ts' 'tests/victoria-logs-request-stats.test.ts' 'app/api/services/[id]/request-stats/route.ts' 'app/(dashboard)/dashboard/projects/[slug]/[env]/services/[serviceId]/page.tsx'\n- pnpm exec tsc --noEmit\n- pnpm test\n- pnpm build passed; local build logs include existing Better Auth warnings about missing BETTER_AUTH_URL and default BETTER_AUTH_SECRET in the local environment.\n\n## Notes\n- Rebased on origin/main before push.\n- PR was opened as a draft by default.

@github-actions

github-actions Bot commented Jul 2, 2026 •

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit b646c43.

@techulus-agent
techulus-agent marked this pull request as ready for review July 2, 2026 10:09
@arjunkomath
arjunkomath merged commit c8af1f0 into main Jul 2, 2026
8 checks passed
@arjunkomath
arjunkomath deleted the feat/service-details-request-stats branch July 2, 2026 11:15
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