Skip to content

Reduce debug trace noise and add safe outcomes - #437

Merged
Wibias merged 7 commits into
mainfrom
fix/trace-efficiency-diagnostics
Sep 9, 2026
Merged

Reduce debug trace noise and add safe outcomes#437
Wibias merged 7 commits into
mainfrom
fix/trace-efficiency-diagnostics

Conversation

@Wibias

@Wibias Wibias commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

Addresses the final trace-efficiency findings from the Grok delivery run:

  • coalesces adjacent reasoning-summary deltas with the same trace identity instead of writing thousands of tiny JSONL records;
  • preserves ordering by flushing at identity and non-reasoning event boundaries;
  • enriches sanitised item_completed events with bounded outcome, durationMs, and errorKind metadata;
  • keeps raw tool inputs, outputs, error messages, paths, and other sensitive payloads out of the trace.

Regression evidence

RED-first coverage reproduced both the record explosion and missing completion diagnostics. Provider and recorder tests also verify that sensitive terminal payloads remain excluded.

Verification

Exact head: f06c22ce85db09c818e4107e3ce6d4b0184089de

  • CI: success
  • Dependency Review: success
  • Windows trace bins: success
  • CodeQL: success

Scope

Debug trace efficiency and diagnostics only. No workflow, publication, mutation, or controller semantics are changed here.

Ready for review. Do not merge without explicit instruction.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 734f933f-3b77-462f-91df-88b8f64bd62d


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Wibias
Wibias merged commit 4f0c670 into main Sep 9, 2026
12 checks passed
@Wibias
Wibias deleted the fix/trace-efficiency-diagnostics branch September 9, 2026 06:16
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