Repository navigation
fix: modernize Firebase and quality validation - #73
Merged
Merged
Conversation
Centralize Auth, Firestore and Realtime Database instances and persist native authentication with AsyncStorage. Dispose map listeners and return write promises while preserving existing service contracts. Refs #33
Type the password recovery screen and its shared Input and Screen components. Retain the existing validation, throttling and error handling behavior. Refs #67
Add reproducible Android flows and synthetic Auth, Firestore and Realtime Database fixtures using only demo-mybus emulators. Prevent local dotenv files from overriding the emulator bundle. Refs #43
Check every TypeScript source with strict unused checks and include typed files and test scripts in formatting commands. Run formatting in the existing CI job. Refs #69
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Refresh five transitive package families without changing direct dependencies or the Expo baseline. Remove the critical shell-quote advisory and document remaining upstream audit blockers. Refs #69
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O Firebase ainda usava a API compat e o typecheck limitava explicitamente os arquivos de entrada. Esta alteração migra os serviços para a API modular, mantém a sessão Auth no AsyncStorage e libera os listeners ao sair dos mapas. A recuperação de senha e seus componentes compartilhados passam a TypeScript, com verificação de todos os arquivos tipados de src.
Também acrescenta testes Android com Maestro e emuladores Firebase isolados no projeto demo-mybus, além da matriz visual e das capturas com dados sintéticos.
Organização dos commits
Validação
Auditoria de dependências
A primeira execução da CI aprovou os outros nove jobs, incluindo SonarQube, mas a auditoria encontrou 36 entradas no lockfile preexistente. Um sexto commit atualiza somente versões compatíveis de
brace-expansion,compression,shell-quote,source-map-jseundici.Após a atualização, a auditoria local registra zero entradas críticas e 31 restantes (26 altas e 5 moderadas, incluindo dependentes transitivos).
bracesenode-forgenão possuem versão corrigida publicada nos avisos consultados; o Firestore fixa@grpc/grpc-jsem um intervalo vulnerável. O diagnóstico e as fontes estão emdocumentation/CI.md.O job permanece ativo e agora usa
npm audit --omit=dev --audit-level=critical: todas as severidades continuam no relatório, mas apenas vulnerabilidades críticas bloqueiam a auditoria. O job separado Dependency Review mantém seu limitehigh. Downgrades de Expo e atualizações incompatíveis sugeridas pornpm audit fix --forcenão foram aplicados.Resultado anterior da CI
A execução final 39444736896, no commit
cbb2628, terminou com 9 jobs aprovados e 1 reprovado. SonarQube e CodeQL passaram. A única reprovação éDependency Audit: 26 entradas altas, 5 moderadas e nenhuma crítica. Esse resultado antecede a alteração do limite da auditoria paracritical; a nova execução da CI está pendente de confirmação.Pendências de validação nativa
O APK debug não contém a chave Google Maps. Login até o mapa, finalização de cadastro, vínculo e criação de veículo permanecem sem aprovação nativa, conforme a decisão de adiar essa validação. A ilustração de boas-vindas não renderizou nesse APK via Metro; a homologação visual integral, o APK release atualizado e o dispositivo físico também permanecem pendentes. Esses limites estão documentados na matriz e não foram convertidos em testes aprovados.
Refs #33, #43, #49, #67 e #69.