Developer by Franklyn-R-Silva · Pull Request #12 · Franklyn-R-Silva/Portfolio

Franklyn-R-Silva · 2025-12-11T02:45:16Z

This pull request introduces several significant improvements to the project, focusing on enhanced documentation, robust error logging, architectural transparency, and database schema evolution. The main changes include the implementation of a centralized logging utility with Supabase integration, improved error handling across the data and utility layers, expanded and clarified documentation (including a detailed architecture section), and updates to the database schema to support logging and skill icon assets.

Architecture & Documentation Enhancements:

The README.md has been substantially expanded to include a detailed architecture section, clearly explaining the MVVM + Repository Pattern, data flow, SOLID principles, design patterns, and the benefits of the chosen structure. The stack and feature descriptions have also been updated for clarity and completeness. [1] [2]

Centralized Logging & Error Handling:

Introduced a new AppLogger utility (lib/core/utils/app_logger.dart) that logs errors and events to a new app_logs table in Supabase, with support for different log levels and optional user association.
Updated error handling in AppUtils and SupabaseRepository to use AppLogger for logging exceptions, including stack traces, improving observability and debugging. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Database Schema Updates:

Added an icon_asset column to the skills table to support custom icons.
Created a new app_logs table with appropriate row-level security (RLS) policies to store application logs, supporting both authenticated and anonymous inserts, and granular read access for users and admins. [1] [2]

Controller Improvements:

Enhanced AuthController to check for an existing session on initialization, ensuring the UI reflects the current authentication state immediately.

Miscellaneous:

Added a constant for the GitHub repositories URL in AppStrings.

These changes collectively improve the maintainability, observability, and clarity of the project, laying a stronger foundation for future development and team collaboration.

…ulo_flutter into developer

…ção na HomePage

…ra usuários

…artes do código

…gs com userId do usuário autenticado

…tCard

…stes na stack tecnológica

…tura do README.md

…ion.dart

- Updated constructor parameter order in CustomTextField, MagneticElement, SocialButton, TechAutocompleteField, TechChip, TypewriterText, CertificateForm, ExperienceForm, ProjectForm, SkillForm, CertificateCard, ExperienceCard, ProjectCard, CertificatesSection, ExperienceSection, GlassHeader, HeroSection, IntroOverlay, ProjectsSection, SkillsSection, and widget tests to place `super.key` at the end. - Changed the build method signature to include `final` keyword for parameters in multiple widgets for improved readability. - Replaced string literals with single quotes in various places for consistency. - Adjusted some numeric values for clarity and consistency.

Copilot

Pull request overview

This pull request introduces a comprehensive set of improvements focused on production-readiness, observability, and user experience. The PR implements centralized error logging with Supabase integration, enhances the admin workflow with better authentication state management, improves UI/UX with interactive project cards, and provides extensive architectural documentation.

Key Changes:

Centralized Logging System: New AppLogger utility that logs errors and events to a Supabase app_logs table, integrated across data, presentation, and utility layers
Enhanced Admin Experience: AuthController now checks for existing sessions on initialization, plus a new floating action button on the home page for quick admin dashboard access
Improved Project Cards: Added live demo buttons, refined spacing/sizing, and better visual hierarchy with hover effects

Reviewed changes

Copilot reviewed 42 out of 53 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
`lib/core/utils/app_logger.dart`	New centralized logging utility with Supabase integration and helper methods for different log levels
`lib/core/utils/app_utils.dart`	Added AppLogger integration for error tracking in URL launch failures
`lib/core/constants/app_constants.dart`	Added GitHub repositories URL constant
`lib/data/repositories/supabase_repository.dart`	Integrated AppLogger for all error scenarios in data fetching operations
`lib/presentation/controllers/auth_controller.dart`	Added session check on initialization to maintain authentication state
`lib/presentation/controllers/portfolio_controller.dart`	Added AppLogger integration for error tracking in data loading
`lib/presentation/pages/home_page.dart`	Added admin FAB for quick dashboard access when logged in
`lib/presentation/pages/admin/admin_dashboard_page.dart`	Added AppLogger integration for delete operation errors
`lib/presentation/widgets/forms/*.dart`	Added AppLogger integration in all form error handlers
`lib/presentation/widgets/molecules/project_card.dart`	Added live demo button, refined layout with better spacing, and improved hover interactions
`lib/presentation/widgets/organisms/projects_section.dart`	Implemented GitHub repositories link functionality
`assets/docs/create.sql`	Added `app_logs` table with RLS policies and `icon_asset` column to skills table
`README.md`	Extensive documentation updates including detailed architecture diagrams, SOLID principles, and technology stack

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-11T02:52:19Z

lib/presentation/widgets/organisms/projects_section.dart

+  Future<void> _launchGitHub() async {
+    final url = Uri.parse(AppStrings.gitHubRepositoriesUrl);
+
+    if (!await launchUrl(url, mode: LaunchMode.externalApplication)) {
+      throw 'Could not launch $url';
+    }
+  }


The error handling in _launchGitHub throws a string exception instead of providing user feedback. This exception may not be caught properly, resulting in an unhandled exception crash. Consider handling the error gracefully by showing a SnackBar to the user instead of throwing, similar to how AppUtils.launchURL handles failures.

Copilot · 2025-12-11T02:52:20Z

lib/presentation/widgets/molecules/project_card.dart

+                          onPressed: () =>
+                              launchUrl(Uri.parse(widget.project.liveUrl!)),


The launchUrl call on line 196 lacks error handling. If the URL fails to open, the user receives no feedback. Consider wrapping this call with try-catch or using AppUtils.launchURL which provides proper error handling and user feedback via SnackBar.

Copilot · 2025-12-11T02:52:20Z

lib/presentation/widgets/molecules/project_card.dart

+                        onEnter: (_) => _isHovered.value = false,
+                        onExit: (_) {},


Setting _isHovered.value to false on mouse enter of the demo button prevents the card hover effect from being visible when hovering over the button area. While this appears intentional, the onExit handler is empty and doesn't restore the hover state. Consider either restoring the hover state on exit or documenting why this behavior is desired.

Copilot · 2025-12-11T02:52:20Z

lib/core/utils/app_logger.dart

+  static Future<void> log({
+    required final String level,
+    required final String message,
+    final String? stack,
+    final String? userId,
+  }) async {
+    try {
+      // Se não passar userId, tenta pegar o usuário autenticado
+      final currentUserId = userId ?? supabase.auth.currentUser?.id;
+
+      await supabase.from('app_logs').insert({
+        'level': level,
+        'message': message,
+        'stack': stack,
+        'user_id': currentUserId,
+      });
+    } catch (e) {
+      // evita que falha no logging quebre o app
+      // Use apenas print em dev, em produção considere remover ou usar serviço externo
+      print('⚠️ Erro ao gravar log no Supabase: $e');
+    }
+  }


The AppLogger.log method and its helper methods (info, debug, error, warning) lack test coverage. Given that the repository already has comprehensive test coverage for other data layer components (as seen in test/data/repositories/supabase_repository_test.dart), consider adding unit tests to verify logging behavior, error handling in the catch block, and that failed logging doesn't break the application.

Copilot · 2025-12-11T02:52:21Z

lib/presentation/controllers/auth_controller.dart

+  AuthController(this.repository) {
+    // Verifica se já existe uma sessão ativa ao inicializar
+    _checkSession();
+  }

  bool get isLogged => repository.isAuthenticated;

-  Future<bool> login(String email, String pass) async {
+  // Verifica a sessão existente
+  void _checkSession() {
+    // O Supabase automaticamente restaura a sessão se houver uma válida
+    if (repository.isAuthenticated) {
+      notifyListeners();
+    }
+  }


The AuthController constructor with _checkSession initialization logic lacks test coverage. Given that the repository has comprehensive test coverage for the data layer, consider adding tests to verify that the controller properly checks for existing sessions on initialization and correctly notifies listeners when a session is detected.

Copilot · 2025-12-11T02:52:21Z

assets/docs/create.sql

+using (auth.uid() IN (
+  -- Adicione aqui os UUIDs dos usuários admin
+  -- Exemplo: 'uuid-do-admin-1', 'uuid-do-admin-2'
+  select id from auth.users where email = 'seu-email-admin@exemplo.com'
+));


The admin policy relies on a hardcoded email check ('seu-email-admin@exemplo.com') which is clearly a placeholder. This should be updated with actual admin email(s) or better yet, documented with clearer instructions on how to configure admin access. Consider adding a comment explaining that this needs to be updated before deployment, or use a more robust admin role system.

Suggested change

using (auth.uid() IN (

-- Adicione aqui os UUIDs dos usuários admin

-- Exemplo: 'uuid-do-admin-1', 'uuid-do-admin-2'

select id from auth.users where email = 'seu-email-admin@exemplo.com'

));

using (

-- ATENÇÃO: Substitua os valores abaixo pelos UUIDs dos usuários admin antes de implantar em produção.

-- Você pode obter os UUIDs dos admins na tabela auth.users.

auth.uid() IN ('uuid-do-admin-1', 'uuid-do-admin-2')

)

Copilot · 2025-12-11T02:52:21Z

assets/docs/create.sql

+create policy "Qualquer um pode inserir logs"
+on app_logs
+for insert
+with check (true);


The RLS policy allows anyone (authenticated or anonymous) to insert logs with "with check (true)". While this supports the logging functionality for unauthenticated users, it could be exploited to flood the database with excessive log entries. Consider adding rate limiting at the application level or implementing a more restrictive policy that limits the number of log entries per IP/session.

Suggested change

create policy "Qualquer um pode inserir logs"

on app_logs

for insert

with check (true);

create policy "Limitar inserção de logs por usuário/sessão"

on app_logs

for insert

with check (

(

-- Para usuários autenticados, limite a 10 logs por 10 minutos

user_id is not null AND (

(select count(*) from app_logs

where user_id = app_logs.user_id

and timestamp > (now() - interval '10 minutes')

) < 10

)

)

OR

(

-- Para usuários anônimos, limite a 10 logs por 10 minutos

user_id is null AND (

(select count(*) from app_logs

where user_id is null

and timestamp > (now() - interval '10 minutes')

) < 10

)

)

);

Copilot · 2025-12-11T02:52:22Z

lib/presentation/pages/home_page.dart

+        // Botão de Download CV (sempre visível)
+        FloatingActionButton.extended(
+          heroTag: 'download_fab',
+          onPressed: () => launchUrl(Uri.parse(AppAssets.cvPtBr)),


The launchUrl call lacks error handling. If the URL fails to open, the user receives no feedback. Consider using AppUtils.launchURL which already provides proper error handling and user feedback.

Copilot · 2025-12-11T02:52:23Z

lib/core/utils/app_logger.dart

+    } catch (e) {
+      // evita que falha no logging quebre o app
+      // Use apenas print em dev, em produção considere remover ou usar serviço externo
+      print('⚠️ Erro ao gravar log no Supabase: $e');


Using print() for error logging in production is not recommended. Consider using debugPrint() which is more Flutter-friendly and can be conditionally disabled in release mode, or integrate with a proper logging framework like logger package. The comment mentions considering removal in production, but a clearer strategy would be beneficial.

Copilot · 2025-12-11T02:52:23Z

lib/presentation/widgets/molecules/project_card.dart

+                        Flexible(
+                          child: Text(
+                            widget.project.description,
+                            maxLines: 2,
+                            overflow: TextOverflow.ellipsis,
+                            style: theme.textTheme.bodyMedium?.copyWith(
+                              height: 1.4,
+                              fontSize: 13,
+                              color: theme.textTheme.bodyMedium?.color
+                                  ?.withValues(alpha: 0.8),
+                            ),
                          ),
                        ),


Using Flexible widget directly inside a Column without an Expanded ancestor or a constrained parent may cause layout issues. The Column has mainAxisSize: MainAxisSize.min, but the Flexible widget expects to know how much space is available. Consider using Expanded if you want the text to take available space, or remove Flexible if a fixed maxLines constraint is sufficient.

Franklyn-R-Silva added 13 commits December 10, 2025 22:52

feat: adicionar link para repositórios do GitHub na seção de projetos

89a92db

Merge branch 'main' of https://github.com/Franklyn-R-Silva/meu_curric…

842de21

…ulo_flutter into developer

feat: adicionar verificação de sessão no AuthController e botões de a…

2d3cbab

…ção na HomePage

feat: adicionar tabela de logs de aplicativo e políticas de acesso pa…

939b847

…ra usuários

feat: adicionar sistema de logging de erros com AppLogger em várias p…

2057d91

…artes do código

feat: adicionar suporte para logs de aviso e ajuste na inserção de lo…

5f11d6e

…gs com userId do usuário autenticado

feat: adicionar coluna opcional icon_asset na tabela de habilidades

e5d7280

feat: ajustar layout do ProjectCard e adicionar botão para Live Demo

17fda3b

feat: simplificar chamada de função para abrir URL no botão do Projec…

5f215ef

…tCard

feat: atualizar README.md com melhorias na descrição do projeto e aju…

efe357b

…stes na stack tecnológica

feat: melhorar formatação da tabela de benefícios na seção de arquite…

a50475f

…tura do README.md

feat: reorganizar imports nos arquivos app_utils.dart e projects_sect…

183d3af

…ion.dart

Franklyn-R-Silva self-assigned this Dec 11, 2025

Copilot AI review requested due to automatic review settings December 11, 2025 02:45

Franklyn-R-Silva had a problem deploying to github-pages December 11, 2025 02:45 — with GitHub Actions Failure

Copilot started reviewing on behalf of Franklyn-R-Silva December 11, 2025 02:45 View session

Franklyn-R-Silva had a problem deploying to github-pages December 11, 2025 02:46 — with GitHub Actions Failure

Franklyn-R-Silva had a problem deploying to github-pages December 11, 2025 02:51 — with GitHub Actions Error

Franklyn-R-Silva merged commit f192763 into main Dec 11, 2025
0 of 2 checks passed

Copilot AI reviewed Dec 11, 2025

View reviewed changes

Franklyn-R-Silva temporarily deployed to github-pages January 20, 2026 17:27 — with GitHub Pages Inactive

Franklyn-R-Silva deployed to github-pages January 20, 2026 17:35 — with GitHub Pages View deployment

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Developer#12

Developer#12
Franklyn-R-Silva merged 13 commits intomainfrom
developer

Franklyn-R-Silva commented Dec 11, 2025

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Copilot AI Dec 11, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		onPressed: () =>
		launchUrl(Uri.parse(widget.project.liveUrl!)),

-create policy "Qualquer um pode inserir logs"
-on app_logs
-for insert
-with check (true);
+create policy "Limitar inserção de logs por usuário/sessão"
+on app_logs
+for insert
+with check (
+  (
+    -- Para usuários autenticados, limite a 10 logs por 10 minutos
+    user_id is not null AND (
+      (select count(*) from app_logs
+        where user_id = app_logs.user_id
+          and timestamp > (now() - interval '10 minutes')
+      ) < 10
+    )
+  )
+  OR
+  (
+    -- Para usuários anônimos, limite a 10 logs por 10 minutos
+    user_id is null AND (
+      (select count(*) from app_logs
+        where user_id is null
+          and timestamp > (now() - interval '10 minutes')
+      ) < 10
+    )
+  )
+);

Conversation

Franklyn-R-Silva commented Dec 11, 2025

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant