Skip to content

[client] provide message content in state#44

Merged
capcom6 merged 1 commit intomasterfrom
client/message-content-in-state
Apr 10, 2026
Merged

[client] provide message content in state#44
capcom6 merged 1 commit intomasterfrom
client/message-content-in-state

Conversation

@capcom6
Copy link
Copy Markdown
Member

@capcom6 capcom6 commented Apr 3, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • New Features

    • Added support for messages that include a cryptographic hash.
    • Message state can now optionally include text, data, or hashed payloads for more flexible composition.

  • Bug Fixes

    • Existing state validation behavior preserved: state checks continue to operate as before.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 3, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
smsgateway/domain_messages.go 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 3, 2026 •
edited
Loading

Warning

Rate limit exceeded

@capcom6 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 25 minutes and 30 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 25 minutes and 30 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8e5192a4-8760-4153-850f-c4cbae0abbe6

📥 Commits

Reviewing files that changed from the base of the PR and between 29eea54 and 2098032.

📒 Files selected for processing (1)
  • smsgateway/domain_messages.go

Walkthrough

Added exported HashedMessage with a required SHA256 Hash field and extended exported MessageState to include optional TextMessage, DataMessage, and HashedMessage pointer fields, each with specific JSON keys and omitempty. MessageState validation logic unchanged.

Changes

Cohort / File(s) Summary
MessageState & HashedMessage
smsgateway/domain_messages.go		 Added exported HashedMessage struct (Hash string with json:"hash" and validate:"required,sha256"). Extended exported MessageState with optional pointer fields: TextMessage *TextMessage (json:"textMessage,omitempty"), DataMessage *DataMessage (json:"dataMessage,omitempty"), and HashedMessage *HashedMessage (json:"hashedMessage,omitempty"). MessageState.Validate() unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[client] provide message content in state' directly relates to the main change: adding message content fields (TextMessage, DataMessage, HashedMessage) to the MessageState struct.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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 and usage tips.

@capcom6 capcom6 force-pushed the client/message-content-in-state branch from 04e1098 to d79e90a Compare April 3, 2026 07:39
@capcom6 capcom6 force-pushed the client/message-content-in-state branch from 29eea54 to 2098032 Compare April 9, 2026 02:56
@capcom6 capcom6 merged commit 6e7d60c into master Apr 10, 2026
8 checks passed
@capcom6 capcom6 deleted the client/message-content-in-state branch April 10, 2026 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@capcom6