Skip to content

[codex] add connector runtime latency metrics#31319

Open
mzeng-openai wants to merge 1 commit into
mainfrom
dev/mzeng/apps-runtime-metrics
Open

[codex] add connector runtime latency metrics#31319
mzeng-openai wants to merge 1 commit into
mainfrom
dev/mzeng/apps-runtime-metrics

Conversation

@mzeng-openai

Copy link
Copy Markdown
Collaborator

Summary

  • measure successful legacy app/list latency with codex.apps.installed.duration_ms, segmented by path=legacy and reload
  • measure successful host-owned codex_apps startup and explicit refresh latency with codex.apps.refresh.duration_ms
  • add the refresh trigger to successful codex.mcp.tools.fetch_uncached.duration_ms samples for codex_apps without changing other MCP-server samples

Why

This establishes a small latency baseline for the current connector path before ConnectorRuntimeManager, app/installed, and app/read land. Error-rate and broader runtime-state instrumentation are intentionally deferred.

This is telemetry-only and does not change connector behavior.

Validation

  • just test -p codex-mcp (94 passed)
  • just test -p codex-app-server list_apps (13 passed)
  • just fix -p codex-mcp
  • just fix -p codex-app-server
  • just fmt
  • git diff --check

@mzeng-openai mzeng-openai force-pushed the dev/mzeng/apps-runtime-metrics branch from 796153d to 2378474 Compare July 7, 2026 00:02
@mzeng-openai mzeng-openai marked this pull request as ready for review July 7, 2026 00:47
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.

1 participant