land: catalog-trace-trust

Merged
AF
2h ago
+294-24

Accepted agent delivery; branch-keyed search descriptor included.


Summary by cubic

Traces slow provider reads (e.g. tool.catalog) and retires the stale first-party trust id alfonso-core.

  • Provider reads now record full phase timings through the existing slow-call log, with the session included in each trace line.
  • Failed tool.catalog replies (write errors, route release, serialization/frame build failures) now emit warning logs with session and correlation data.
  • The index census parser now accepts the new session= field in slow-call lines, with a regression test pinned to the CI cost-gate step.
  • alfonso-core no longer resolves to first-party trust; it returns Untrusted like any other retired module id.
  • Adds a benchmark slice descriptor for land/catalog-trace-trust.

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge; the follow-up fixes the earlier parser issue without introducing a new failure.

Summary

Adds session-tagged timing logs for provider reads, warning logs for failed catalog replies, and removes first-party trust for alfonso-core.

  • Provider reads now show their timing and session in slow-call logs.
  • The retired alfonso-core id no longer receives first-party trust.
  • The slow-call census reads logs with or without a session field.
  • The benchmark harness now has a descriptor for this catalog-trace slice.

Reviews (2) · Last reviewed commit: "telemetry: parse slow tool_call lines th..." · Reviewed by Greptile