fix: include created_at on AddInvoice SmallOrder

[arkanoider]

Summary

• Set SmallOrder.created_at from order.created_at in set_waiting_invoice_status when sending Action::AddInvoice to the buyer, matching the payload shape used in flow::hold_invoice_paid so clients (e.g. Mostrix) can show order creation time after a take-sell flow without an invoice in the take message.

Summary by CodeRabbit

• Bug Fixes

  • Buyer invoice notifications now include the order creation timestamp.

• Tests

  • Test helpers updated to include the anti-abuse bond field for environment-variable override scenarios.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ad653676-153f-4b45-9f85-f5ceb62cee5c

📥 Commits

Reviewing files that changed from the base of the PR and between d6d89f7 and 820c304.

📒 Files selected for processing (1)

• src/config/util.rs

---

Walkthrough

Adds the order creation timestamp to the buyer invoice notification payload and updates a test Settings fixture to include anti_abuse_bond: None.

Changes

Invoice payload and test fixture

Layer / File(s)	Summary
Invoice payload includes creation timestamp src/util.rs	set_waiting_invoice_status now sets the SmallOrder field to Some(order.created_at) when enqueuing the AddInvoice payload.
Test Settings fixture src/config/util.rs	make_settings test helper now explicitly sets anti_abuse_bond: None in the Settings struct literal.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• MostroP2P/mostro#524: Also modifies set_waiting_invoice_status payload construction in src/util.rs to fix buyer amount calculation using saturating_sub.
• MostroP2P/mostro#464: Adjusts which active_order/SmallOrder data is packaged and returned in invoice-related responses.

Suggested reviewers

• grunch
• Catrya

Poem

🐰 A tiny timestamp hops inside,
The invoice keeps when orders arrived,
A test gets tidy, fields set right,
Small changes make the garden bright. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	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 specifically describes the main change: adding the created_at field to the SmallOrder payload in the AddInvoice action, which matches the primary objective of the pull request.
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.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/add-invoice-payload-created-at

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[grunch]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[grunch]

LGTM