fix: use context.push() instead of context.go() for sub-page navigation

lib/main.dart

@@ -383,7 +383,7 @@ class FavoritesScreen extends StatelessWidget {
                 return DrinkCard(
                   key: ValueKey(drink.id),
                   drink: drink,
-                  onTap: () => context.go(buildDrinkDetailPath(festivalId, drink.id)),
+                  onTap: () => context.push(buildDrinkDetailPath(festivalId, drink.id)),
                   onFavoriteTap: () => provider.toggleFavorite(drink),
                 );
               },

lib/screens/drink_detail_screen.dart

@@ -239,7 +239,7 @@ class _DrinkDetailScreenState extends State<DrinkDetailScreen> {
     unawaited(provider.analyticsService.logStyleViewed(style));
 
     // Navigate to style screen
-    context.go(buildStylePath(widget.festivalId, style));
+    context.push(buildStylePath(widget.festivalId, style));
   }
 
   Widget _buildDescription(BuildContext context, Drink drink, ThemeData theme) {
@@ -305,7 +305,7 @@ class _DrinkDetailScreenState extends State<DrinkDetailScreen> {
                     ? Text(drink.breweryLocation)
                     : null,
                 trailing: const Icon(Icons.chevron_right),
-                onTap: () => context.go(buildBreweryPath(widget.festivalId, drink.producer.id)),
+                onTap: () => context.push(buildBreweryPath(widget.festivalId, drink.producer.id)),
               ),
             ),
           ),

lib/screens/drinks_screen.dart

@@ -332,7 +332,7 @@ class _DrinksScreenState extends State<DrinksScreen> {
         hint: 'Double tap for more details',
         button: true,
         child: InkWell(
-          onTap: () => context.go(buildFestivalInfoPath(widget.festivalId)),
+          onTap: () => context.push(buildFestivalInfoPath(widget.festivalId)),
           child: Padding(
             padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 8),
             child: Row(
@@ -489,7 +489,7 @@ class _DrinksScreenState extends State<DrinksScreen> {
   }
 
   void _navigateToDetail(BuildContext context, String drinkId) {
-    context.go(buildDrinkDetailPath(widget.festivalId, drinkId));
+    context.push(buildDrinkDetailPath(widget.festivalId, drinkId));
   }
 
   void _showCategoryFilter(BuildContext context, BeerProvider provider) {

lib/widgets/drink_list_section.dart

@@ -68,7 +68,7 @@ class DrinkListSection {
             return DrinkCard(
               key: ValueKey(drink.id),
               drink: drink,
-              onTap: () => context.go(buildDrinkDetailPath(festivalId, drink.id)),
+              onTap: () => context.push(buildDrinkDetailPath(festivalId, drink.id)),
               onFavoriteTap: () => provider.toggleFavorite(drink),
             );
           },
@@ -119,7 +119,7 @@ class DrinkListSection {
               key: ValueKey(drink.id),
               drink: drink,
               subtitle: subtitle,
-              onTap: () => context.go(buildDrinkDetailPath(festivalId, drink.id)),
+              onTap: () => context.push(buildDrinkDetailPath(festivalId, drink.id)),
               onFavoriteTap: () => provider.toggleFavorite(drink),
             );
           },

lib/widgets/festival_menu_sheets.dart

@@ -215,8 +215,9 @@ class FestivalSelectorSheet extends StatelessWidget {
                             router?.go(targetPath);
                           },
                           onInfoTap: () {
+                            final router = GoRouter.maybeOf(context);
                             Navigator.pop(context);
-                            context.go(buildFestivalInfoPath(festival.id));
+                            router?.push(buildFestivalInfoPath(festival.id));
                           },
                         ),
                       );

lib/widgets/overflow_menu.dart

@@ -60,6 +60,6 @@ void _handleMenuSelection(BuildContext context, String value) {
     case 'settings':
       showSettingsSheet(context);
     case 'about':
-      context.go('/about');
+      context.push('/about');
   }
 }

test/drink_detail_screen_test.dart

@@ -523,7 +523,7 @@ void main() {
         // Verify similar drink card exists
         expect(find.text('Similar IPA'), findsOneWidget);
 
-        // NOTE: Navigation uses go_router's context.go() which requires GoRouter 
+        // NOTE: Navigation uses go_router's context.push() which requires GoRouter 
         // in the widget tree. This is tested in E2E tests instead of unit tests.
       });