lib/l10n/intl_es.arb (1)

139-141: Minor wording consistency: “sats” lowercase.

Most strings elsewhere use “sats”. Consider “La cantidad de sats…”.

lib/l10n/intl_en.arb (1)

140-140: Style nit: use “sats” (lowercase) for consistency.

“The sats amount is out of the allowed range.”

lib/shared/widgets/notification_listener_widget.dart (2)

41-60: Replace long if-else chain with a map for clarity and maintainability.

This reduces branching and makes adding new reasons trivial.

-            if (cantDoReason == 'pending_order_exists') {
-              message = S.of(context)!.pendingOrderExists;
-            } else if (cantDoReason == 'not_allowed_by_status') {
-              message = S.of(context)!.notAllowedByStatus;
-            } else if (cantDoReason == 'invalid_invoice') {
-              message = S.of(context)!.invalidInvoice;
-            } else if (cantDoReason == 'invalid_trade_index') {
-              message = S.of(context)!.invalidTradeIndex;
-            } else if (cantDoReason == 'is_not_your_order') {
-              message = S.of(context)!.isNotYourOrder;
-            } else if (cantDoReason == 'invalid_signature') {
-              message = S.of(context)!.invalidSignature;
-            } else if (cantDoReason == 'invalid_peer') {
-              message = S.of(context)!.invalidPeer;
-            } else if (cantDoReason == 'invalid_pubkey') {
-              message = S.of(context)!.invalidPubkey;
-            } else if (cantDoReason == 'order_already_canceled') {
-              message = S.of(context)!.orderAlreadyCanceled;
-            } else if (cantDoReason == 'out_of_range_sats_amount') {
-              message = S.of(context)!.outOfRangeSatsAmount;
-            } else {
-              // Use generic cant-do message for other reasons
-              message = NotificationMessageMapper.getLocalizedTitle(context, next.action!);
-            }
+            final reasons = <String, String>{
+              'pending_order_exists': S.of(context)!.pendingOrderExists,
+              'not_allowed_by_status': S.of(context)!.notAllowedByStatus,
+              'invalid_invoice': S.of(context)!.invalidInvoice,
+              'invalid_trade_index': S.of(context)!.invalidTradeIndex,
+              'is_not_your_order': S.of(context)!.isNotYourOrder,
+              'invalid_signature': S.of(context)!.invalidSignature,
+              'invalid_peer': S.of(context)!.invalidPeer,
+              'invalid_pubkey': S.of(context)!.invalidPubkey,
+              'order_already_canceled': S.of(context)!.orderAlreadyCanceled,
+              'out_of_range_sats_amount': S.of(context)!.outOfRangeSatsAmount,
+            };
+            message = (cantDoReason != null && reasons.containsKey(cantDoReason))
+                ? reasons[cantDoReason]!
+                : NotificationMessageMapper.getLocalizedTitle(context, next.action!);

---

62-67: Optional: handle additional likely reason ‘out_of_range_fiat_amount’.

If protocol may send it, map to S.of(context)!.outOfRangeFiatAmount to keep parity.

Would you like me to add this mapping now?

lib/features/trades/widgets/mostro_message_detail_widget.dart (1)

241-269: Deduplicate cantDo reason-to-message mapping

Cant-do reason mapping is now in multiple places (this widget and notification listener). Extract a single mapper (e.g., CantDoReasonLocalizer.map(reason, context, values)) and reuse to avoid drift.

I can sketch a small helper and update both call sites if you want.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: CR
PR: MostroP2P/mobile#0
File: CLAUDE.md:0-0
Timestamp: 2025-08-28T20:32:34.625Z
Learning: Applies to lib/l10n/intl_{en,es,it}.arb : Add new localization keys to all three ARB files (en, es, it)

Learnt from: chebizarro
PR: MostroP2P/mobile#74
File: lib/services/mostro_service.dart:70-76
Timestamp: 2025-05-06T15:49:26.443Z
Learning: In the Mostro Mobile codebase, Riverpod code generation is used with `Riverpod` annotations. Providers like `eventStorageProvider` are generated in `.g.dart` files from annotated functions in the main provider files. These providers are accessible by importing the main provider file (e.g., `mostro_service_provider.dart`), not by importing a separate provider file.

Learnt from: CR
PR: MostroP2P/mobile#0
File: CLAUDE.md:0-0
Timestamp: 2025-08-28T20:32:34.625Z
Learning: Use Riverpod for state management with Notifiers encapsulating business logic; access data only through repositories

Learnt from: Catrya
PR: MostroP2P/mobile#270
File: lib/shared/widgets/order_filter.dart:133-135
Timestamp: 2025-08-15T01:37:12.243Z
Learning: The MostroP2P/mobile project requires Flutter >=3.27.0 as specified in pubspec.yaml, which supports the Color.withValues() method, so usage of withValues() throughout the codebase is valid and should not be flagged as a compatibility issue.

Learnt from: CR
PR: MostroP2P/mobile#0
File: CLAUDE.md:0-0
Timestamp: 2025-08-28T20:32:34.625Z
Learning: Applies to test/mocks.mocks.dart : Do not manually modify `test/mocks.mocks.dart` (auto-generated by Mockito)