Skip to content

fix: handle nullable message metadata#74

Merged
steipete merged 1 commit into
mainfrom
codex/fix-nullable-message-reads
Jul 4, 2026
Merged

fix: handle nullable message metadata#74
steipete merged 1 commit into
mainfrom
codex/fix-nullable-message-reads

Conversation

@steipete-oai

Copy link
Copy Markdown
Contributor

Summary

  • normalize nullable message metadata to empty strings across search, message listing, and thread-context reads
  • cover nullable user, thread, reply, and subtype fields with a regression test
  • document the fix for the next patch release

Verification

  • GOWORK=off go test ./...
  • structured Codex autoreview: clean, no accepted/actionable findings

@clawsweeper

clawsweeper Bot commented Jul 4, 2026

Copy link
Copy Markdown

ClawSweeper status: review started.

I am starting a fresh review of this pull request: fix: handle nullable message metadata This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@steipete steipete left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the null-normalization paths and regression coverage. Full Go tests, CI, CodeQL, secret scanning, and structured autoreview are clean.

@steipete steipete merged commit 6814742 into main Jul 4, 2026
12 checks passed
@steipete steipete deleted the codex/fix-nullable-message-reads branch July 4, 2026 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants