Skip to content

[domain] add createdAt field to message state - #104

Merged
capcom6 merged 1 commit into
masterfrom
domain/message-state-created-at
Oct 17, 2026
Merged

capcom6 merged 1 commit into
masterfrom
domain/message-state-created-at

Conversation

@capcom6

@capcom6 capcom6 commented Aug 30, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features

    • Message states now include an optional creation timestamp.
    • Creation timestamps can be read when available and are preserved as provided, including malformed values.
    • Existing messages without a timestamp continue to work with a null value.
  • Tests

    • Added coverage for timestamp handling when provided, omitted, or read from message data.

@coderabbitai

coderabbitai Bot commented Aug 30, 2026 •

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: dfdea90c-39ad-4b3b-9f71-7354abf6af8a
📥 Commits

Reviewing files that changed from the base of the PR and between 0f0ec8d and 2ee6083.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 73ce9836-27e7-42c9-ad9a-ab8465b18a74

📥 Commits

Reviewing files that changed from the base of the PR and between 468b539 and 3c5b5b0.

📒 Files selected for processing (2)
  • src/Domain/MessageState.php
  • tests/Domain/MessageStateTest.php

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

MessageState now supports an optional createdAt timestamp. The value flows through the constructor and FromObject(), is exposed by CreatedAt(), and is covered by tests for present, absent, default, and malformed values.

Changes: MessageState timestamp

Layer / File(s) Summary
MessageState createdAt contract
src/Domain/MessageState.php
Adds nullable createdAt storage, constructor support, the CreatedAt() getter, and FromObject() mapping.
createdAt behavior tests
tests/Domain/MessageStateTest.php
Tests present, absent, default, and malformed createdAt values.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 3c5b5

The PR adds an optional creation timestamp to message state while preserving existing callers and payloads that omit it. No actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 12 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding the createdAt field to MessageState.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@capcom6 capcom6 added the ready label Aug 31, 2026
@capcom6
capcom6 force-pushed the domain/message-state-created-at branch from 3c5b5b0 to 0f0ec8d Compare October 2, 2026 01:08
@capcom6
capcom6 force-pushed the domain/message-state-created-at branch from 0f0ec8d to 2ee6083 Compare October 16, 2026 02:14
@capcom6
capcom6 merged commit 4047d2d into master Oct 17, 2026
6 checks passed
@capcom6
capcom6 deleted the domain/message-state-created-at branch October 17, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant