Skip to content

docs: include Writing in the diagnostic sink map - #2419

Merged
r3dbars merged 1 commit into
mainfrom
codex/review-docs-19
Nov 12, 2026
Merged

r3dbars merged 1 commit into
mainfrom
codex/review-docs-19

Conversation

@r3dbars

@r3dbars r3dbars commented Nov 10, 2026

Copy link
Copy Markdown
Owner

The observability inventory omits Writing’s DiagnosticsLog. Add its source location, default writing-diagnostics.log path and existing 4 MB / one .1 generation rotation, then update the documented sink count from five to six.

Verified against the full logger implementation and the storage map. This records existing behavior; it changes no logger, payload, retention setting or privacy policy. AppLogSink and feedback documentation corrections are outside this PR.

Validation on e26d8a887: all three mapped bash check.sh documentation checks passed; preflight passed; verify-change.sh reported no applicable app check. Independent full two-hunk diff/source review against origin/main: GREEN from the coordinator before commit. No logger instantiation/runtime or real log reads.

This extracts only DOCS-19’s sink inventory correction from broad pending #2336, which remains untouched and unmerged. Preserve its unrelated hunks and apply this portion once during integration. No merge requested.

@r3dbars
r3dbars merged commit 7178256 into main Nov 12, 2026
8 of 11 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