Skip to content

audit-service.mdx documents the audit-write failure report as once per process, which PR #17450 makes false — it becomes once per CAUSE #17451

Description

@claude

Filed by the domain:services execution seat (session session_01ToDPcx9AESFubJkDiFMtKW) out of the docs-drift advisory on PR #17450. ⛔ Not graded, no domain:* — that is triage's. Expected landing point: content/docs/**domain:devx. ⛔ The services seat did not edit the page: content/docs/** is outside its lane.

The falsified sentence

content/docs/kernel/runtime-services/audit-service.mdx:76, verbatim on origin/main:

  • A failed ledger insert is reported at error level once per process — every later
    failure drops to debug — and the row is simply lost. Nothing retries it.

Why it becomes false

PR #17450 (card #15166, maintainer-ruled direction) re-keys reportAuditWriteFailure's dedupe from a process-wide boolean to a per-cause key — the error code (or its absence) plus the object. After it lands:

  • a repeat of an already-reported cause still degrades to debug (that half of the sentence survives);
  • a new cause reports at error once, rather than dropping to debug.

⇒ 「once per process」 and 「every later failure drops to debug」 are both wrong as stated. The page will describe a behaviour the platform no longer has, on the page that is the audit service's own contract documentation.

What is asked

One sentence corrected on that page to state the actual granularity: once per cause, where a cause is the error code plus the object. ⛔ Nothing else on the page is implicated by this change.

⛔ What is NOT asked — measured, so nobody widens this

content/docs/permissions/record-view-auditing.mdx:217 also says 「Both degradations are reported once per process」 and it was checked. It is NOT falsified: it documents the record-view (read) audit buffer, implemented in packages/plugins/plugin-audit/src/read-audit.ts, which PR #17450 does not touch. ⇒ Leave it alone.

The other four pages the advisory listed (production-readiness.mdx, plugins/packages.mdx, protocol/kernel/config-resolution.mdx, ui/setup-app.mdx) were checked for the same phrasing and return 0 hits; a positive control confirmed the phrase does occur elsewhere in content/docs, so those zeros are readings rather than a broken query.

Blocking

⚠️ Blocked-by: PR #17450 — the sentence is TRUE today and only becomes false when that PR merges. ⛔ Do not correct it before then, or the docs will be wrong in the other direction. Re-check with git log origin/main --oneline | grep '(#17450)' on a NON-shallow checkout (a shallow tree answers that 0 silently).

Refs: #15166 (the ruled card) · PR #17450 · content/docs/kernel/runtime-services/audit-service.mdx

Blocked-by: #15166


Generated by Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions