Repository navigation
docs(permissions): state each read-audit degradation's real reporting granularity - #19817
Merged
objectstack-fleet[bot] merged 1 commit intoSep 24, 2026
Merged
Conversation
… granularity The record-view auditing page said both read-audit degradations are reported once per process. Buffer overflow still is (a process-wide boolean in read-audit.ts); a failed ledger write is reported once per distinct cause (keyed by the error code via auditFailureCauseKey). Split the lead-in so each bullet carries its own granularity. Claude-Session: https://claude.ai/code/session_01VDtqoecgES7ScQYGbFVDRv Co-authored-by: Claude <noreply@anthropic.com>
This was referenced Sep 23, 2026
Contributor
Author
Contract reviewServed-tier: ① Derived judgments
② Semver levelDocs-only, one ③ Boundary flags
Implemented-by: VERDICT: PASS Isolated at-tier reviewer, adopted by the Generated by Claude Code |
objectstack-fleet
Bot
deleted the
claude/issue-19216-read-audit-once-per-cause
branch
September 24, 2026 15:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #19216
Clause-②: no
What
content/docs/permissions/record-view-auditing.mdx, section "Failure posture": the lead-in said both read-audit degradations are reported once per process. That was true of one and false of the other, measured onorigin/main502f179cc:packages/plugins/plugin-audit/src/read-audit.ts):240let overflowReported = false;(one boolean in the batcher closure; repeats go todebug):542const reportedReadAuditFailureCauses = new Set<string>();, keyed byauditFailureCauseKey('sys_audit_log', err)(the errorcode, uncoded errors collapse to one bucket); docblock at:499says once per CAUSEThe lead-in now says only what holds for both (neither is retried; each is reported once), and each bullet states its own granularity. The ledger-write bullet uses the same definition of "cause" as the sibling page
content/docs/kernel/runtime-services/audit-service.mdx(recordAuthEvent), so the two pages agree. The implementation matches its own docblock, so no code changed. Frontmatter and headings are untouched.Changeset
Docs-only. The one changed path is under
content/docs/**, and no published package source orfiles[]entry is touched, so this isskip-changesetterritory.Verification
Gates derived with
node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --commands(change set from merge base502f179cc: 1 path). All 44 derived families ran on headceefc645and every one exited 0, includingpnpm check:doc-authoring,pnpm check:doc-anchorsandpnpm check:nul-bytes. Reconciliation with--ran(exit codes recorded): "44 derived, 44 run, 0 NOT-MEASURED, 0 UNRUN". Two gates first answered PREREQUISITE NOT MET (exit 3:check:docs-transcript-drift, which needs@objectstack/lintdist, andspec check:skill-examples, which needs@objectstack/client-reactdist). After those closures were built under the verify lock, both were re-run and exited 0. The control-byte self-scan of the changed file found nothing.Acceptance notes
installReadAuditWritercreates one batcher per install. In practice that is one per process, so the page keeps the wording the card accepted.Generated by Claude Code