Skip to content

land: catalog-trace-trust - #557

Merged
ualtinok merged 3 commits into
mainfrom
land/catalog-trace-trust
Oct 18, 2026
Merged

ualtinok merged 3 commits into
mainfrom
land/catalog-trace-trust

Conversation

@aft-alfonso

@aft-alfonso aft-alfonso Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

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.

Written for commit 3a582d4. Summary will update on new commits.

View guided diff Turn on auto-fix

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

ualtinok and others added 2 commits October 18, 2026 20:27
Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
Comment thread crates/aft/src/logging.rs
The daemon now logs session= between name= and channel=. The census
regex required them adjacent and silently dropped every new slow-call
line. Accept both forms and pin them with a unittest in the cost-gate CI
step.

Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
@ualtinok
ualtinok added this pull request to the merge queue Oct 18, 2026
@ualtinok
ualtinok removed this pull request from the merge queue due to a manual request Oct 18, 2026
@ualtinok
ualtinok added this pull request to the merge queue Oct 18, 2026
@ualtinok
ualtinok removed this pull request from the merge queue due to a manual request Oct 18, 2026
@ualtinok
ualtinok added this pull request to the merge queue Oct 18, 2026
Merged via the queue into main with commit 2609e04 Oct 18, 2026
49 checks passed
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