Skip to content

fix: show new billing accounts in the activity feed#67

Merged
scotwells merged 1 commit into
mainfrom
fix/activity-policy-generatename-summary
Jun 25, 2026
Merged

fix: show new billing accounts in the activity feed#67
scotwells merged 1 commit into
mainfrom
fix/activity-policy-generatename-summary

Conversation

@scotwells

@scotwells scotwells commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What this does

When a billing account is created, it should appear in the activity feed — but in some cases that entry was quietly going missing. This makes new billing accounts show up reliably, and applies the same fix to billing account bindings so they don't hit it either.

What changed

  • Create summaries now use the account's assigned name, so they always show up.
  • Updates and deletes are untouched.

The BillingAccount create-fallback activity rule linked
audit.objectRef.name, which is empty for generateName-created accounts.
CEL summary evaluation then errored ("no such key: name"), the activity
entry was dead-lettered, and the create never appeared in the activity
feed. Two production failures were observed (error_type=cel_summary).

Switch the linked label to audit.responseObject.metadata.name, which is
populated for both name and generateName creates, while keeping
audit.objectRef as the link target. Apply the same swap preventively to
the BillingAccountBinding create-fallback rule, which shares the
unguarded pattern (no production failures observed).

Update and delete rules are untouched: audit.objectRef.name is reliably
present there.

Claude-Session: https://claude.ai/code/session_01KnYuL5Pf1R5ysZoxxNkKiu
@scotwells scotwells changed the title fix(activity): show generateName BillingAccount creates in the activity feed fix: show new billing accounts in the activity feed Jun 25, 2026
@scotwells scotwells marked this pull request as ready for review June 25, 2026 16:54
@scotwells scotwells merged commit e46d783 into main Jun 25, 2026
8 checks passed
@scotwells scotwells deleted the fix/activity-policy-generatename-summary branch June 25, 2026 16:56
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.

3 participants