Skip to content

Add logger newline boundary regression fixture#13

Closed
h1065153539-create wants to merge 2 commits into
Soengkit:mainfrom
h1065153539-create:codex/frailbox-logger-newline-boundary-20260620214629
Closed

Add logger newline boundary regression fixture#13
h1065153539-create wants to merge 2 commits into
Soengkit:mainfrom
h1065153539-create:codex/frailbox-logger-newline-boundary-20260620214629

Conversation

@h1065153539-create

Copy link
Copy Markdown

Summary

  • Adds a focused C fixture for legacy frailbox logger newline boundary behavior.
  • Covers messages with no caller newline, one caller newline, multiple trailing newlines, and a message that crosses the internal log buffer limit.
  • Keeps the public logger API compatible while ensuring truncated records still end with a newline before the next record starts.
  • Documents the fixture target in docs/OPERATIONS.md.

Validation

  • make -C frailbox clean logger-newline-fixtures passed.
  • make -C frailbox all passed.
  • python3 build.py -m frailbox compiled frailbox successfully in the API-sourced worktree; the encrypted diagnostic log records the expected non-git diagnostic commit limitation for this local checkout.

Diagnostic bundle

  • diagnostic/build-abef3d5c.logd
  • diagnostic/build-abef3d5c.json

Closes #4.

@Soengkit

Copy link
Copy Markdown
Owner

Closing this PR because the linked fork issue is only a closed payout tracker for an already submitted upstream PR. This fork is not an active bounty intake or payment authority, and it is not accepting external submissions or payment details here.

@Soengkit Soengkit closed this Jun 23, 2026
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.

Payout tracking for upstream PR #414

2 participants