Skip to content

A fallback credential is a fact on the call, not a row in the feed - #143

Merged
czpython merged 1 commit into
mainfrom
credential-fallback-noise
Jul 28, 2026
Merged

A fallback credential is a fact on the call, not a row in the feed#143
czpython merged 1 commit into
mainfrom
credential-fallback-noise

Conversation

@czpython

Copy link
Copy Markdown
Owner

Every agent call emitted a credential.fallback event when its login came
from the fallback account rather than the run's own. A run makes one such
call per agent, so a single run wrote a column of identical rows into the
feed — same subject, same harness, same run id — and the feed had no
renderer for the type, so each row read as the raw dotted identifier.

The fact those rows carried is already recorded: agent_calls.account_id
holds the subscription actually charged, on the row that paid it, and it is
already on the wire. The event is dropped; the account snapshot that stamps
the call stays.

The feed test that used this type as its fixture keeps it — logged rows from
before this change still carry the kind, and rendering an unrecognised kind
as itself is what keeps them readable.

@czpython
czpython force-pushed the credential-fallback-noise branch from 8f7c13b to 2bde56e Compare July 28, 2026 17:39
@czpython
czpython enabled auto-merge (squash) July 28, 2026 17:40
Every agent call emitted credential.fallback when its login came from the
fallback account, so a run put one identical row in the feed per call, with
no renderer behind the type. The charged account is already stamped on the
agent call it paid for, and reads there.
@czpython
czpython force-pushed the credential-fallback-noise branch from 2bde56e to d4bd3a8 Compare July 28, 2026 17:43
@czpython
czpython merged commit 46f15a5 into main Jul 28, 2026
2 checks passed
@czpython
czpython deleted the credential-fallback-noise branch July 28, 2026 17:46
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