kasy-cli 1.32.0 → 1.34.0

package/templates/firebase/lib/core/rating/widgets/review_popup.dart

@@ -1,5 +1,5 @@
+import 'package:flutter/foundation.dart' show kIsWeb;
 import 'package:flutter/material.dart';
-import 'package:flutter_animate/flutter_animate.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:go_router/go_router.dart';
 import 'package:kasy_kit/components/components.dart';
@@ -7,17 +7,25 @@ import 'package:kasy_kit/core/data/api/analytics_api.dart';
 import 'package:kasy_kit/core/rating/providers/rating_repository.dart';
 import 'package:kasy_kit/core/states/user_state_notifier.dart';
 import 'package:kasy_kit/core/theme/theme.dart';
-import 'package:kasy_kit/core/widgets/responsive_layout.dart';
 import 'package:kasy_kit/i18n/translations.g.dart';
 import 'package:logger/logger.dart';
 
 /// Shows the in-app review dialog. Prefer a stable [context] (not a sheet that
 /// will be popped before the async work finishes).
+///
+/// A clean [KasyDialog]: a title, a short message and two stacked actions
+/// (write a review / suggest improvements). Dismissing via the dialog's own
+/// close button just defers the next ask and keeps the user where they are.
 Future<bool> showReviewDialog(
   BuildContext context,
   WidgetRef ref, {
   bool force = false,
 }) async {
+  // Store reviews are a native-only concept (App Store / Play Store). On web
+  // there is nowhere to send the user, so the prompt never shows there.
+  if (kIsWeb) {
+    return false;
+  }
   if (!context.mounted) {
     return false;
   }
@@ -41,97 +49,42 @@ Future<bool> showReviewDialog(
     barrierDismissible: false,
     builder: (dialogContext) {
       ratingRepository.delay();
-      return Animate(
-        effects: const [
-          FadeEffect(
-            delay: Duration(milliseconds: 100),
-            duration: Duration(milliseconds: 300),
-          ),
-          MoveEffect(
-            delay: Duration(milliseconds: 100),
-            duration: Duration(milliseconds: 450),
-            curve: Curves.easeOut,
-            begin: Offset(0, 50),
-            end: Offset.zero,
-          ),
-        ],
-        child: DeviceSizeBuilder(
-          builder: (device) {
-            final maxWidth = switch (device) {
-              DeviceType.medium =>
-                MediaQuery.of(dialogContext).size.width - KasySpacing.xl,
-              _ => 550.0,
-            };
-            final translations = Translations.of(dialogContext).review_popup;
-            return KasyDialog(
-              title: translations.title,
-              titleCentered: true,
-              showCloseButton: false,
-              body: Column(
-                crossAxisAlignment: CrossAxisAlignment.stretch,
-                mainAxisSize: MainAxisSize.min,
-                children: [
-                  Stack(
-                    clipBehavior: Clip.none,
-                    children: [
-                      ClipRRect(
-                        borderRadius: BorderRadius.circular(KasyRadius.md),
-                        child: Image.asset(
-                          'assets/images/review.png',
-                          fit: BoxFit.fitWidth,
-                          width: maxWidth,
-                        ),
-                      ),
-                      Positioned(
-                        top: KasySpacing.sm,
-                        left: KasySpacing.sm,
-                        child: CloseIcon(
-                          onExit: () {
-                            analytics.logEvent('rating_popup_close', {});
-                            rating.delay().then((_) {
-                              if (!dialogContext.mounted) return;
-                              Navigator.of(dialogContext).pop();
-                            });
-                          },
-                        ),
-                      ),
-                    ],
-                  ),
-                  const SizedBox(height: KasySpacing.md),
-                  Text(
-                    translations.description,
-                    textAlign: TextAlign.center,
-                    style: Theme.of(dialogContext).textTheme.bodyMedium,
-                  ),
-                ],
-              ),
-              footer: Column(
-                crossAxisAlignment: CrossAxisAlignment.stretch,
-                children: [
-                  KasyButton(
-                    label: translations.rate_button,
-                    expand: true,
-                    onPressed: () {
-                      analytics.logEvent('rating_popup_show', {});
-                      ratingRepository.rate().then((_) => rating.review()).then(
-                        (_) {
-                          if (!dialogContext.mounted) return;
-                          Navigator.of(dialogContext).pop();
-                        },
-                      );
-                    },
-                  ),
-                  const SizedBox(height: KasySpacing.sm),
-                  KasyButton(
-                    label: translations.cancel_button,
-                    variant: KasyButtonVariant.soft,
-                    expand: true,
-                    onPressed: () => Navigator.of(dialogContext).pop(true),
-                  ),
-                ],
-              ),
-            );
-          },
+      final translations = Translations.of(dialogContext).review_popup;
+      return KasyDialog(
+        leadingIcon: KasyIcons.star,
+        iconTone: KasyDialogIconTone.info,
+        title: translations.title,
+        titleCentered: true,
+        message: translations.description,
+        onClose: () {
+          analytics.logEvent('rating_popup_close', {});
+          rating.delay();
+          Navigator.of(dialogContext).pop();
+        },
+        footer: Column(
+          crossAxisAlignment: CrossAxisAlignment.stretch,
+          children: [
+            KasyButton(
+              label: translations.rate_button,
+              expand: true,
+              onPressed: () {
+                analytics.logEvent('rating_popup_show', {});
+                ratingRepository.rate().then((_) => rating.review()).then(
+                  (_) {
+                    if (!dialogContext.mounted) return;
+                    Navigator.of(dialogContext).pop();
+                  },
+                );
+              },
+            ),
+            const SizedBox(height: KasySpacing.sm),
+            KasyButton(
+              label: translations.cancel_button,
+              variant: KasyButtonVariant.soft,
+              expand: true,
+              onPressed: () => Navigator.of(dialogContext).pop(true),
+            ),
+          ],
         ),
       );
     },
@@ -142,34 +95,3 @@ Future<bool> showReviewDialog(
   }
   return true;
 }
-
-class CloseIcon extends StatelessWidget {
-  final VoidCallback onExit;
-
-  const CloseIcon({super.key, required this.onExit});
-
-  @override
-  Widget build(BuildContext context) {
-    return ClipOval(
-      child: Material(
-        color: Colors.transparent,
-        child: InkWell(
-          onTap: () => onExit.call(),
-          child: Ink(
-            width: 32,
-            height: 32,
-            decoration: BoxDecoration(
-              color: context.colors.background,
-              shape: BoxShape.circle,
-            ),
-            child: Icon(
-              KasyIcons.close,
-              color: context.colors.onBackground,
-              size: KasyIconSize.lg,
-            ),
-          ),
-        ),
-      ),
-    );
-  }
-}

package/templates/firebase/lib/core/widgets/update_bottom_sheet.dart

@@ -3,17 +3,19 @@ import 'package:kasy_kit/components/components.dart';
 import 'package:kasy_kit/core/theme/theme.dart';
 import 'package:kasy_kit/i18n/translations.g.dart';
 
-/// Shows a modal bottom sheet displaying app update information
+/// Shows the "what's new" bottom sheet, built on [KasyBottomSheet].
+///
+/// Dismissing it — the Continue button, the drag handle, or tapping the dim
+/// barrier outside — simply closes the sheet and returns the user to the screen
+/// they were on; it never navigates anywhere. The version is already recorded as
+/// seen before this is shown (see `MaybeShowUpdateBottomSheet`), so it won't pop
+/// up again for the same version.
 Future<void> showUpdateBottomSheet({
   required BuildContext context,
   required String version,
-  bool useRootNavigator = true,
 }) async {
-  await showModalBottomSheet(
+  await showKasyBottomSheet<void>(
     context: context,
-    useSafeArea: true,
-    useRootNavigator: useRootNavigator,
-    barrierColor: context.colors.background.withValues(alpha: 0.90),
     isScrollControlled: true,
     builder: (context) => _UpdateBottomSheet(version: version),
   );
@@ -28,130 +30,31 @@ class _UpdateBottomSheet extends StatelessWidget {
   Widget build(BuildContext context) {
     final translations = Translations.of(context).update_bottom_sheet;
 
-    return Container(
-      decoration: BoxDecoration(
-        color: context.colors.surface,
-        borderRadius: const BorderRadius.only(
-          topLeft: Radius.circular(KasyRadius.xl),
-          topRight: Radius.circular(KasyRadius.xl),
+    return KasyBottomSheet(
+      icon: KasyIcons.star,
+      title: translations.title,
+      message: 'Version $version',
+      body: ConstrainedBox(
+        constraints: BoxConstraints(
+          maxHeight: MediaQuery.sizeOf(context).height * 0.5,
         ),
-        border: Border.all(
-          color: context.colors.primary.withValues(alpha: 0.2),
-          strokeAlign: BorderSide.strokeAlignOutside,
-          width: 2,
+        child: ListView.separated(
+          shrinkWrap: true,
+          padding: EdgeInsets.zero,
+          itemBuilder: (context, index) =>
+              _UpdateHighlightTile(highlight: translations.highlights[index]),
+          separatorBuilder: (context, index) =>
+              const SizedBox(height: KasySpacing.sm),
+          itemCount: translations.highlights.length,
         ),
       ),
-      child: SafeArea(
-        minimum: const EdgeInsets.only(bottom: KasySpacing.md),
-        child: Column(
-          mainAxisSize: MainAxisSize.min,
-          crossAxisAlignment: CrossAxisAlignment.start,
-          children: [
-            Flexible(
-              flex: 0,
-              child: Align(
-                child: ClipRRect(
-                  borderRadius: const BorderRadius.only(
-                    topLeft: Radius.circular(KasyRadius.xl),
-                    topRight: Radius.circular(KasyRadius.xl),
-                  ),
-                  child: Image.asset(
-                    'assets/images/update.png',
-                    fit: BoxFit.cover,
-                  ),
-                ),
-              ),
-            ),
-            Padding(
-              padding: const EdgeInsets.fromLTRB(
-                KasySpacing.lg,
-                KasySpacing.lg,
-                KasySpacing.md,
-                0,
-              ),
-              child: Row(
-                children: [
-                  Container(
-                    width: 48,
-                    height: 48,
-                    decoration: BoxDecoration(
-                      color: context.colors.primary.withValues(alpha: 0.1),
-                      borderRadius: KasyRadius.mdBorderRadius,
-                    ),
-                    child: Icon(
-                      KasyIcons.star,
-                      color: context.colors.primary,
-                      size: KasyIconSize.xl,
-                    ),
-                  ),
-                  const SizedBox(width: KasySpacing.md),
-                  Expanded(
-                    child: Column(
-                      crossAxisAlignment: CrossAxisAlignment.start,
-                      children: [
-                        Text(
-                          translations.title,
-                          style: context.textTheme.headlineSmall?.copyWith(
-                            color: context.colors.onSurface,
-                            fontWeight: FontWeight.w700,
-                          ),
-                        ),
-                        Text(
-                          "Version $version",
-                          style: context.textTheme.bodyMedium?.copyWith(
-                            color: context.colors.muted,
-                            fontWeight: FontWeight.w500,
-                          ),
-                        ),
-                      ],
-                    ),
-                  ),
-                  KasyButton.iconOnly(
-                    icon: KasyIcons.close,
-                    variant: KasyButtonVariant.ghost,
-                    foregroundColor: context.colors.muted,
-                    onPressed: () => Navigator.of(context).pop(),
-                    semanticLabel: translations.title,
-                  ),
-                ],
-              ),
-            ),
-            const SizedBox(height: KasySpacing.lg),
-            Flexible(
-              child: ConstrainedBox(
-                constraints: BoxConstraints(
-                  maxHeight: MediaQuery.of(context).size.height * 0.6,
-                ),
-                child: ListView.separated(
-                  shrinkWrap: true,
-                  padding: const EdgeInsets.symmetric(
-                    horizontal: KasySpacing.lg,
-                  ),
-                  itemBuilder: (context, index) {
-                    final highlight = translations.highlights[index];
-                    return _UpdateHighlightTile(highlight: highlight);
-                  },
-                  separatorBuilder: (context, index) =>
-                      const SizedBox(height: KasySpacing.sm),
-                  itemCount: translations.highlights.length,
-                ),
-              ),
-            ),
-            const SizedBox(height: KasySpacing.xl),
-            Padding(
-              padding: const EdgeInsets.symmetric(horizontal: KasySpacing.lg),
-              child: SizedBox(
-                width: double.infinity,
-                child: KasyButton(
-                label: translations.continue_button,
-                expand: true,
-                onPressed: () => Navigator.of(context).pop(),
-              ),
-              ),
-            ),
-          ],
+      actions: [
+        KasyButton(
+          label: translations.continue_button,
+          expand: true,
+          onPressed: () => Navigator.of(context).pop(),
         ),
-      ),
+      ],
     );
   }
 }

package/templates/firebase/lib/features/ai_chat/ai_chat_page.dart

@@ -115,13 +115,8 @@ class _DetailContainer extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    return Container(
-      clipBehavior: Clip.antiAlias,
-      decoration: BoxDecoration(
-        color: context.colors.surface,
-        borderRadius: BorderRadius.circular(24),
-        border: Border.all(color: context.colors.border, width: 0.5),
-      ),
+    return KasyCard(
+      borderRadius: BorderRadius.circular(KasyRadius.xl),
       child: child,
     );
   }
@@ -143,6 +138,15 @@ class _DetailPlaceholder extends StatelessWidget {
               showShadow: true,
             ),
             const SizedBox(height: KasySpacing.md),
+            Text(
+              t.home.cards.assistant_title,
+              textAlign: TextAlign.center,
+              style: context.textTheme.titleMedium?.copyWith(
+                fontWeight: FontWeight.w600,
+                color: context.colors.onBackground,
+              ),
+            ),
+            const SizedBox(height: KasySpacing.xs),
             Text(
               t.ai_chat.no_conversation_selected,
               textAlign: TextAlign.center,

package/templates/firebase/lib/features/ai_chat/ui/widgets/ai_chat_composer.dart

@@ -30,6 +30,12 @@ class AiChatComposer extends StatefulWidget {
 class _AiChatComposerState extends State<AiChatComposer> {
   bool _canSend = false;
 
+  // Owns the field focus so we can intercept hardware Enter: plain Enter sends,
+  // Shift+Enter inserts a newline (the standard chat-composer behaviour). On
+  // touch keyboards (no hardware key event) the return key keeps inserting a
+  // newline and the send orb is used, so mobile is unchanged.
+  late final FocusNode _focusNode = FocusNode(onKeyEvent: _onKeyEvent);
+
   @override
   void initState() {
     super.initState();
@@ -40,9 +46,23 @@ class _AiChatComposerState extends State<AiChatComposer> {
   @override
   void dispose() {
     widget.controller.removeListener(_onTextChanged);
+    _focusNode.dispose();
     super.dispose();
   }
 
+  KeyEventResult _onKeyEvent(FocusNode node, KeyEvent event) {
+    if (event is! KeyDownEvent) return KeyEventResult.ignored;
+    final bool isEnter = event.logicalKey == LogicalKeyboardKey.enter ||
+        event.logicalKey == LogicalKeyboardKey.numpadEnter;
+    if (!isEnter) return KeyEventResult.ignored;
+    // Shift+Enter falls through to the field and inserts a newline.
+    if (HardwareKeyboard.instance.isShiftPressed) {
+      return KeyEventResult.ignored;
+    }
+    _handleSend();
+    return KeyEventResult.handled;
+  }
+
   bool _hasSendableText(String value) {
     final String trimmed = value.trim();
     return trimmed.isNotEmpty && trimmed.length <= kAiChatMaxMessageLength;
@@ -88,6 +108,7 @@ class _AiChatComposerState extends State<AiChatComposer> {
             Expanded(
               child: KasyTextField(
                 controller: widget.controller,
+                focusNode: _focusNode,
                 enabled: enabled,
                 variant: KasyTextFieldVariant.embedded,
                 hint: t.ai_chat.hint,

package/templates/firebase/lib/features/ai_chat/ui/widgets/ai_conversation_list.dart

@@ -208,7 +208,7 @@ class _Header extends StatelessWidget {
                 maxLines: 1,
                 overflow: TextOverflow.ellipsis,
                 style: context.textTheme.titleLarge?.copyWith(
-                  fontWeight: FontWeight.w800,
+                  fontWeight: FontWeight.w700,
                   color: context.colors.onBackground,
                 ),
               ),

package/templates/firebase/lib/features/authentication/api/authentication_api.dart

@@ -392,6 +392,46 @@ class FirebaseAuthenticationApi implements AuthenticationApi {
   /// Falls back to regular sign-in if the Facebook account already exists.
   @override
   Future<Credentials> signupFromAnonymousWithFacebook() async {
+    // Web: use the Firebase popup (link to the anonymous user when present), same
+    // pattern as Apple/Google on web.
+    if (kIsWeb) {
+      final facebookProvider = FacebookAuthProvider();
+      facebookProvider.addScope('email');
+      final currentUser = _auth.currentUser;
+      if (currentUser == null) {
+        try {
+          final result = await _popupOrCancel(
+            () => _auth.signInWithPopup(facebookProvider),
+          );
+          return Credentials(id: result.user!.uid, token: result.credential?.token.toString() ?? '');
+        } on FirebaseAuthException catch (e) {
+          if (_isUserCancelledPopup(e.code)) throw const UserCancelledSignInException();
+          rethrow;
+        }
+      }
+      try {
+        final result = await _popupOrCancel(() => currentUser.linkWithPopup(facebookProvider));
+        return Credentials(id: result.user!.uid, token: result.credential?.token.toString() ?? '');
+      } on FirebaseAuthException catch (e) {
+        if (_isUserCancelledPopup(e.code)) throw const UserCancelledSignInException();
+        if (e.code == 'credential-already-in-use') {
+          final anonymousUser = _auth.currentUser;
+          if (anonymousUser != null && anonymousUser.isAnonymous) {
+            await anonymousUser.delete();
+          }
+          final cred = e.credential;
+          if (cred != null) {
+            final result = await _auth.signInWithCredential(cred);
+            return Credentials(id: result.user!.uid, token: result.credential?.token.toString() ?? '');
+          }
+          final result = await _popupOrCancel(
+            () => _auth.signInWithPopup(facebookProvider),
+          );
+          return Credentials(id: result.user!.uid, token: result.credential?.token.toString() ?? '');
+        }
+        rethrow;
+      }
+    }
     final loginResult = await FacebookAuth.instance.login();
     if (loginResult.status == LoginStatus.cancelled) throw const UserCancelledSignInException();
     if (loginResult.status != LoginStatus.success || loginResult.accessToken == null) {
@@ -429,6 +469,27 @@ class FirebaseAuthenticationApi implements AuthenticationApi {
 
   @override
   Future<Credentials> signinWithFacebook() async {
+    // Web has no native Facebook SDK flow: use the Firebase popup, same as Google
+    // and Apple on web. Requires the Facebook provider enabled in Firebase
+    // (kasy facebook) and the redirect URI registered on Meta.
+    if (kIsWeb) {
+      final facebookProvider = FacebookAuthProvider();
+      facebookProvider.addScope('email');
+      try {
+        final value = await _popupOrCancel(
+          () => _auth.signInWithPopup(facebookProvider),
+        );
+        return Credentials(
+          id: value.user!.uid,
+          token: value.credential?.token.toString() ?? '',
+        );
+      } on FirebaseAuthException catch (e) {
+        if (_isUserCancelledPopup(e.code)) {
+          throw const UserCancelledSignInException();
+        }
+        rethrow;
+      }
+    }
     final LoginResult loginResult = await FacebookAuth.instance.login();
     if (loginResult.status == LoginStatus.cancelled) throw const UserCancelledSignInException();
     if (loginResult.status != LoginStatus.success || loginResult.accessToken == null) {

package/templates/firebase/lib/features/authentication/ui/signin_page.dart

@@ -49,7 +49,7 @@ class SigninPage extends ConsumerWidget {
                 const TextStyle())
             .copyWith(
               color: context.colors.muted,
-              fontSize: 13,
+              fontSize: 13, // design-check: ignore — small "forgot password" link
               fontWeight: FontWeight.w500,
             );
     return PopScope(
@@ -237,6 +237,10 @@ class _SocialSigninRow extends ConsumerWidget {
         ? withAppleWebSignin
         : (defaultTargetPlatform == TargetPlatform.iOS ||
             defaultTargetPlatform == TargetPlatform.macOS);
+    // Facebook on web works on the Firebase backend (signInWithPopup); on Supabase
+    // the web flow isn't wired yet (roadmap), so withFacebookWebSignin ships false
+    // there. Native (iOS/Android) always shows it.
+    const bool showFacebook = !kIsWeb || withFacebookWebSignin;
     return Row(
       children: [
         Expanded(
@@ -263,22 +267,24 @@ class _SocialSigninRow extends ConsumerWidget {
             ),
           ),
         ],
-        const SizedBox(width: KasySpacing.sm),
-        Expanded(
-          child: _SocialSigninTile(
-            label: t.auth.signin.facebook,
-            icon: Image.asset(
-              'assets/icons/facebook.png',
-              width: 20,
-              height: 20,
+        if (showFacebook) ...[
+          const SizedBox(width: KasySpacing.sm),
+          Expanded(
+            child: _SocialSigninTile(
+              label: t.auth.signin.facebook,
+              icon: Image.asset(
+                'assets/icons/facebook.png',
+                width: 20,
+                height: 20,
+              ),
+              onPressed: isSending
+                  ? null
+                  : () => ref
+                        .read(signinStateProvider.notifier)
+                        .signinWithFacebook(),
             ),
-            onPressed: isSending
-                ? null
-                : () => ref
-                      .read(signinStateProvider.notifier)
-                      .signinWithFacebook(),
           ),
-        ),
+        ],
       ],
     );
   }

package/templates/firebase/lib/features/authentication/ui/signup_page.dart

@@ -216,6 +216,10 @@ class _SocialSignupRow extends ConsumerWidget {
         ? withAppleWebSignin
         : (defaultTargetPlatform == TargetPlatform.iOS ||
             defaultTargetPlatform == TargetPlatform.macOS);
+    // Facebook on web works on the Firebase backend (signInWithPopup); on Supabase
+    // the web flow isn't wired yet (roadmap), so withFacebookWebSignin ships false
+    // there. Native (iOS/Android) always shows it.
+    const bool showFacebook = !kIsWeb || withFacebookWebSignin;
     return Row(
       children: [
         Expanded(
@@ -242,22 +246,24 @@ class _SocialSignupRow extends ConsumerWidget {
             ),
           ),
         ],
-        const SizedBox(width: KasySpacing.sm),
-        Expanded(
-          child: _SocialSignupTile(
-            label: t.auth.signin.facebook,
-            icon: Image.asset(
-              'assets/icons/facebook.png',
-              width: 20,
-              height: 20,
+        if (showFacebook) ...[
+          const SizedBox(width: KasySpacing.sm),
+          Expanded(
+            child: _SocialSignupTile(
+              label: t.auth.signin.facebook,
+              icon: Image.asset(
+                'assets/icons/facebook.png',
+                width: 20,
+                height: 20,
+              ),
+              onPressed: isSending
+                  ? null
+                  : () => ref
+                        .read(signinStateProvider.notifier)
+                        .signinWithFacebook(),
             ),
-            onPressed: isSending
-                ? null
-                : () => ref
-                      .read(signinStateProvider.notifier)
-                      .signinWithFacebook(),
           ),
-        ),
+        ],
       ],
     );
   }

package/templates/firebase/lib/features/feedbacks/ui/widgets/feature_card.dart

@@ -60,7 +60,6 @@ class _FeatureCardState extends State<FeatureCard> {
                 Text(
                   widget.title,
                   style: context.textTheme.titleMedium?.copyWith(
-                    fontSize: 16,
                     fontWeight: FontWeight.w700,
                   ),
                 ),
@@ -251,7 +250,7 @@ class _VoteCardState extends State<VoteCard>
                     key: ValueKey('votes-${widget.id}-${widget.votes}'),
                     widget.votes.toString(),
                     style: context.textTheme.labelLarge?.copyWith(
-                      fontSize: 15,
+                      fontSize: 15, // design-check: ignore — vote-count badge
                       height: 1,
                       fontWeight: FontWeight.w800,
                       color: widget.textColor,