fix: infer MCP tool-call status from level/error when status field is absent

[Copilot]

Post-OTel-collector migration, gateway.jsonl entries stopped emitting an explicit "status" string field. The parser blindly copied entry.Status (empty "") into MCPToolCall.Status, which downstream aggregators rendered as "unknown" — breaking error-rate, cost-per-tool, and retry analytics for all 165+ tool calls in the audit.

Changes

pkg/cli/gateway_logs_mcp.go — extractToolCallsFromGatewayLog

• When entry.Status == "", infer "error" if entry.Error != "" or entry.Level == "error", otherwise default to "success":

status:=entry.Statusifstatus=="" {
ifentry.Error!=""||entry.Level=="error" {
status="error"
} else {
status="success"
}
}

pkg/cli/gateway_logs_parsing.go — processGatewayLogEntry

• Extend error-tracking predicate to include entry.Level == "error" so TotalErrors/ErrorCount fields aren't under-counted for the new log format:

// beforeifentry.Status=="error"||entry.Error!="" {
// afterifentry.Status=="error"||entry.Error!=""||entry.Level=="error" {

Tests

• TestExtractToolCallsStatusInference: 5 sub-cases — explicit status pass-through, level:"error" inference, error field inference, and level:"info" → success default.
• TestProcessGatewayLogEntryLevelErrorCounting: verifies error counters increment when only level:"error" is present.

[Copilot]

Pull request overview

This PR updates the MCP gateway log parser to handle post-OTel format drift where gateway.jsonl tool-call entries may omit the "status" field, ensuring tool-call status and error metrics are still computed correctly.

Changes:

• Infer MCPToolCall.Status as "error" when level:"error" or error is present, otherwise default to "success" when status is absent.
• Count log entries as errors when level:"error" is present (to avoid under-counting errors with missing status/error fields).
• Add unit tests covering status inference and level-based error counting.
• Update GitHub Actions lock/workflow files (includes a schedule change for the developer-docs consolidator workflow).

Show a summary per file

File	Description
pkg/cli/gateway_logs_mcp.go	Infer per-tool-call status when status is missing.
pkg/cli/gateway_logs_parsing.go	Treat level:"error" entries as errors for metrics counting.
pkg/cli/gateway_logs_test.go	Add tests for status inference and level-based error counting.
.github/workflows/release.lock.yml	Update pinned docker/setup-buildx-action reference.
.github/workflows/developer-docs-consolidator.lock.yml	Change schedule to weekly and update lock content.
.github/aw/actions-lock.json	Add lock entry for docker/setup-buildx-action@v4.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 6/6 changed files
• Comments generated: 2