From bc7d14eeebc2c4dbd16e0c40e657d7794d4abc3a Mon Sep 17 00:00:00 2001
From: shenlong-tanwen <139912620+shalong-tanwen@users.noreply.github.com>
Date: Mon, 29 Jun 2026 02:41:18 +0530
Subject: [PATCH] review changes

---
 mobile/lib/domain/services/asset.service.dart                | 2 +-
 mobile/lib/presentation/actions/stack.action.dart            | 2 +-
 mobile/test/unit/mocks.dart                                  | 2 +-
 mobile/test/unit/presentation/actions/stack_action_test.dart | 4 ++--
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/mobile/lib/domain/services/asset.service.dart b/mobile/lib/domain/services/asset.service.dart
index ed4d5c6c5a..5050b0f547 100644
--- a/mobile/lib/domain/services/asset.service.dart
+++ b/mobile/lib/domain/services/asset.service.dart
@@ -87,7 +87,7 @@ class AssetService {
     await _remoteRepository.stack(userId, stack);
   }
 
-  Future<void> unStack(List<String> stackIds) async {
+  Future<void> unstack(List<String> stackIds) async {
     if (stackIds.isEmpty) {
       return;
     }
diff --git a/mobile/lib/presentation/actions/stack.action.dart b/mobile/lib/presentation/actions/stack.action.dart
index 8973786087..dc23a74bc6 100644
--- a/mobile/lib/presentation/actions/stack.action.dart
+++ b/mobile/lib/presentation/actions/stack.action.dart
@@ -33,7 +33,7 @@ class StackAction extends AssetAction<RemoteAsset> {
     if (shouldStack) {
       await service.stack(scope.authUser.id, assets.map((asset) => asset.id).toList(growable: false));
     } else {
-      await service.unStack(assets.map((asset) => asset.stackId).nonNulls.toList(growable: false));
+      await service.unstack(assets.map((asset) => asset.stackId).nonNulls.toList(growable: false));
     }
 
     final message = shouldStack
diff --git a/mobile/test/unit/mocks.dart b/mobile/test/unit/mocks.dart
index a030109542..6501d5fe60 100644
--- a/mobile/test/unit/mocks.dart
+++ b/mobile/test/unit/mocks.dart
@@ -174,7 +174,7 @@ extension type const AssetServiceStub(MockAssetService service) implements Stub<
       () => service.stack(any(), any());
 
   Future<void> Function() get unStack =>
-      () => service.unStack(any());
+      () => service.unstack(any());
 }
 
 extension type const NativeSyncApiStub(MockNativeSyncApi api) implements Stub<MockNativeSyncApi> {
diff --git a/mobile/test/unit/presentation/actions/stack_action_test.dart b/mobile/test/unit/presentation/actions/stack_action_test.dart
index bab186de9e..ca7aba41eb 100644
--- a/mobile/test/unit/presentation/actions/stack_action_test.dart
+++ b/mobile/test/unit/presentation/actions/stack_action_test.dart
@@ -38,7 +38,7 @@ void main() {
 
       await tester.pumpTestAction(context, StackAction(assets: [asset]));
 
-      verify(() => assetService.unStack(['stack'])).called(1);
+      verify(() => assetService.unstack(['stack'])).called(1);
     });
 
     testWidgets('prioritizes stack when mixed state', (tester) async {
@@ -66,7 +66,7 @@ void main() {
 
       await tester.pumpTestAction(context, StackAction(assets: [first, second]));
 
-      verify(() => assetService.unStack(['stack-1', 'stack-2'])).called(1);
+      verify(() => assetService.unstack(['stack-1', 'stack-2'])).called(1);
     });
 
     testWidgets('shows a confirmation snackbar on success', (tester) async {