fix: error screen in main.dart

[tqha1011]

Summary by CodeRabbit

• Refactor
  • Simplified the Sort filter button UI by removing mock content for a cleaner interface
  • Updated app initialization to implement authentication-first navigation flow
  • Reorganized internal state management architecture for improved provider registration

[coderabbitai]

📝 Walkthrough

Walkthrough

Three distinct updates across the codebase: removing annotation comments from UserStatisticsModel, cleaning up mock UI elements from the Sort button in the home screen, and restructuring the app's root navigation to use AuthGate for authentication-first flow while explicitly registering TaskViewModel in the provider hierarchy.

Changes

Cohort / File(s)	Summary
Statistics Model Cleanup src/lib/features/statistics/model/StatisticsModel.dart	Removed inline comment markers/annotations from the dailyCounts field and constructor parameter; no logic or type changes.
Home Screen UI Refactoring src/lib/features/tasks/view/screens/home_screen.dart	Removed mock TaskCard UI block from Sort filter button, including stacked CircleAvatar widgets and "add" icon overlay; sort button now renders icon only.
Authentication & Provider Configuration src/lib/main.dart	Changed app root from MainScreen to AuthGate for authentication-first navigation; explicitly registered TaskViewModel in MultiProvider via ChangeNotifierProvider<TaskViewModel>; removed unused login_view.dart import and reorganized provider imports.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Implement priority and tag selection in task creation version 1 #31 — Both PRs register TaskViewModel with Provider in main.dart to make the view model globally available at the app root.
• Implement complete auth UI and integrate Focus feature set #27 — Both PRs modify app authentication and navigation setup; this PR switches the root to AuthGate while the related PR implements/removes auth UI including login_view.

Suggested reviewers

• hoanghaoz

Poem

🐰 The app hops forward with auth in sight,
AuthGate guards the entrance, holding tight.
Providers whisper their tasks with care,
While mock UI vanishes into thin air. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title 'fix: error screen in main.dart' references only one specific aspect (error screen in main.dart), but the changeset includes multiple substantial modifications across three files: comment removal in StatisticsModel, UI cleanup in home_screen, and authentication flow restructuring plus provider registration in main.dart. The title does not capture the main architectural change (switching to AuthGate and registering TaskViewModel), making it partially related but not representing the primary changes.	Update the title to reflect the primary architectural change, such as: 'feat: initialize auth flow and register TaskViewModel globally' or 'refactor: switch to AuthGate and setup provider structure', which better represents the core changes across the files.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/errorWhenStart

---

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.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/lib/features/tasks/view/screens/home_screen.dart (1)

154-184: ⚠️ Potential issue | 🟡 Minor

Use an accessible button widget for the icon-only sort action.

The sort button (lines 154–184) currently uses GestureDetector with no semantic markup or tooltip, reducing clarity for screen readers and keyboard navigation. Replace with IconButton and add a tooltip label:

Proposed refactor

-                      GestureDetector(
-                        onTap: () => viewModel.toggleSortByPriority(),
-                        child: AnimatedContainer(
+                      AnimatedContainer(
                           duration: const Duration(milliseconds: 200),
                           padding: const EdgeInsets.symmetric(
                             horizontal: 12,
                             vertical: 7,
                           ),
                           decoration: BoxDecoration(
                             color: viewModel.sortByPriority
                                 ? AppColors.primaryBlue
                                 : Colors.white,
                             borderRadius: BorderRadius.circular(20),
                             border: Border.all(
                               color: AppColors.primaryBlue,
                               width: 1,
                             ),
                           ),
-                          child: Row(
-                            children: [
-                              Icon(
-                                Icons.sort,
-                                size: 16,
-                                color: viewModel.sortByPriority
-                                    ? Colors.white
-                                    : AppColors.primaryBlue,
-                              ),
-                            ]
-                          ),
-                        )
-                      ),
+                          child: IconButton(
+                            tooltip: 'Sort by priority',
+                            onPressed: () => viewModel.toggleSortByPriority(),
+                            icon: Icon(
+                              Icons.sort,
+                              size: 16,
+                              color: viewModel.sortByPriority
+                                  ? Colors.white
+                                  : AppColors.primaryBlue,
+                            ),
+                            visualDensity: VisualDensity.compact,
+                            constraints: const BoxConstraints(),
+                            padding: EdgeInsets.zero,
+                          ),
+                        ),

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/lib/features/tasks/view/screens/home_screen.dart` around lines 154 - 184,
Replace the GestureDetector wrapping the AnimatedContainer (the icon-only sort
control) with an accessible IconButton so it supports keyboard focus, semantics
and a tooltip; wire IconButton.onPressed to viewModel.toggleSortByPriority(),
provide a meaningful tooltip/semanticLabel like "Sort by priority", and preserve
the existing visual state by applying the same conditional colors
(viewModel.sortByPriority ? AppColors.primaryBlue : Colors.white for background
and the inverse for Icon color) and the rounded border/animation (keep the
AnimatedContainer or wrap IconButton with a decorated/animated parent) so
behavior of sortByPriority and visual styling remain unchanged.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@src/lib/features/tasks/view/screens/home_screen.dart`:
- Around line 154-184: Replace the GestureDetector wrapping the
AnimatedContainer (the icon-only sort control) with an accessible IconButton so
it supports keyboard focus, semantics and a tooltip; wire IconButton.onPressed
to viewModel.toggleSortByPriority(), provide a meaningful tooltip/semanticLabel
like "Sort by priority", and preserve the existing visual state by applying the
same conditional colors (viewModel.sortByPriority ? AppColors.primaryBlue :
Colors.white for background and the inverse for Icon color) and the rounded
border/animation (keep the AnimatedContainer or wrap IconButton with a
decorated/animated parent) so behavior of sortByPriority and visual styling
remain unchanged.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 60acb9c8-bde2-4223-aad0-1def92ffd149

📥 Commits

Reviewing files that changed from the base of the PR and between ae8e44c and f3dfe0f.

📒 Files selected for processing (3)

• src/lib/features/statistics/model/StatisticsModel.dart
• src/lib/features/tasks/view/screens/home_screen.dart
• src/lib/main.dart