chore(cleanup): remove cluster orfao useProductRegistration* (F1-7.11)#98
Merged
adm01-debug merged 2 commits intoMay 8, 2026
Merged
Conversation
Cluster fechado de 3 hooks que NAO eram importados por ninguem fora do proprio cluster. Eram filhos da ProductRegistrationPage.tsx, deletada no PR #97 (commit 741af39). TOPOLOGIA DO CLUSTER (validada via grep recursivo): - useProductRegistration.ts (228 linhas) - exportava PRODUCT_FIELDS, BulkImportRow, ColumnMapping. Importado APENAS por useBulkImportFile. - useBulkImportFile.ts (157 linhas) - logica de import em massa. ZERO importadores externos. - useProductRegistrationImport.ts (128 linhas) - 'logica de importacao'. ZERO importadores externos. VALIDACAO PREVIA: - grep recursivo em src/, scripts/, e2e/, supabase/ (.ts, .tsx, .json, .md, .mjs, .js, .html) → zero refs externas - zero imports dinamicos (lazy, import()) - zero tests/specs/stories - pagina que usava (ProductRegistrationPage) ja morta no PR #97 Arquivos removidos: 3 (513 linhas) Refs auditoria: F1-7.11
MUDANCAS
========
1. docs/FUNCIONALIDADES_E_FERRAMENTAS.md
- Secao 3.4 'Registro de Produtos' removida inteira (apontava
pro hook deletado no commit anterior)
- Renumeracao em cascata: 3.5->3.4, 3.6->3.5, 3.7->3.6, 3.8->3.7,
3.9->3.8, 3.10->3.9, 3.11->3.10, 3.12->3.11
- Sequencia 3.1 a 3.11 sem buracos
2. docs/AUDITORIA_2026-05-07.md
- F1-7.11 marcada como CONCLUIDA (cluster useProductRegistration*)
- F1-7.12 marcada como CONCLUIDA (secao 3.4 removida)
- Contador da Fase 1: 16/19 (84%) -> 18/19 (95%)
REFS
====
- F1-7.11, F1-7.12 (versionados em 07/05/2026 sessao de noite)
- Resolve achado bonus do PR #97
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Warning
|
| Layer / File(s) | Summary |
|---|---|
Audit & Feature Documentation docs/AUDITORIA_2026-05-07.md, docs/FUNCIONALIDADES_E_FERRAMENTAS.md |
Audit task counter increments from 16/19 to 18/19. Product registration section is removed from feature documentation; remaining sections (3.4–3.11) are renumbered sequentially. |
Removed Type Definitions and Exports src/hooks/useBulkImportFile.ts, src/hooks/useProductRegistration.ts, src/hooks/useProductRegistrationImport.ts |
Removed exported types: ImportMode, ImportStep, ParsedData, ProductFormData, ProductImageInput, BulkImportRow, ImportValidationResult, ColumnMapping. Removed hooks: useBulkImportFile(), useProductRegistration(), useProductImport(). Removed field descriptor: PRODUCT_FIELDS. |
useBulkImportFile Hook Removal src/hooks/useBulkImportFile.ts |
Parsing logic for CSV/XLSX files via papaparse and dynamic XLSX import, column mapping utilities for field matching and transformation, and file input/drag-and-drop event handlers deleted. |
useProductRegistration Hook Removal src/hooks/useProductRegistration.ts |
Module imports, reference-data loading via parallel fetches for categories/suppliers/techniques/tags, product and image creation logic, audit log insertion, toast notifications, and delegation to import helper deleted. |
useProductImport Hook Removal src/hooks/useProductRegistrationImport.ts |
Import progress state management, row validation including numeric parsing and supplier/category ID resolution, sequential product import with per-row error handling, progress tracking, success/error toast notifications, and CSV template generation with browser download deleted. |
🎯 4 (Complex) | ⏱️ ~45 minutes
🐰 A cluster cleaned with care,
Three hooks that once thrived in the air,
Now pass into rest,
Their purpose addressed,
The code breathes again, fresh and fair! 🌿
Note
🎁 Summarized by CodeRabbit Free
Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.
Comment @coderabbitai help to get the list of available commands and usage tips.
There was a problem hiding this comment.
Pull request overview
Remove um “cluster órfão” de hooks relacionado ao antigo fluxo de registro/importação de produtos (já desativado com a remoção da ProductRegistrationPage no PR #97), e atualiza a documentação/auditoria para refletir essa limpeza.
Changes:
- Remove 3 hooks não referenciados (
useProductRegistration*) que só se importavam entre si. - Remove a seção correspondente no doc de funcionalidades e renumera as subseções seguintes.
- Atualiza a auditoria para marcar as tarefas F1-7.11 e F1-7.12 como concluídas (e ajustar o percentual da Fase 1).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/hooks/useProductRegistration.ts | Deleta hook órfão de cadastro de produto (inclui tipos/export PRODUCT_FIELDS). |
| src/hooks/useBulkImportFile.ts | Deleta hook órfão de parsing/mapping de arquivo de importação em massa. |
| src/hooks/useProductRegistrationImport.ts | Deleta hook órfão de validação/importação de linhas e geração de template CSV. |
| docs/FUNCIONALIDADES_E_FERRAMENTAS.md | Remove seção “Registro de Produtos” e renumera as seções 3.4+ mantendo a formatação consistente. |
| docs/AUDITORIA_2026-05-07.md | Marca F1-7.11/F1-7.12 como concluídas e ajusta progresso da Fase 1. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
adm01-debug
added a commit
that referenced
this pull request
May 8, 2026
CONTEUDO ======== 1. docs/AUDITORIA_2026-05-07.md - Adicionada secao 3.0 'Triagem decisoria (concluida 07/05/2026)' - 7 decisoes registradas (F3-0.1 a F3-0.8) - 3 acoes pendentes do Joaquim (F3-0.9 a F3-0.11) - Status Fase 3: NAO INICIADA -> PRE-TRABALHO COMPLETO (8/11, 73%) - Changelog v1.4 (PR #98) e v1.5 (triagem) adicionados 2. docs/HANDOFF-2026-05-07-SESSAO-NOITE.md - Resumo de uma frase atualizado (84% -> 95%, 5 PRs) - Nova secao 'Triagem decisoria da Fase 3' - Premissa-chave revelada: sistema NAO em producao real - Tabela proximas opcoes atualizada (A e B marcadas concluidas) - F3-0.9 a F3-0.11 documentadas como bloqueadores PREMISSA-CHAVE REVELADA NA SESSAO ================================== Joaquim confirmou explicitamente: o sistema NAO esta em producao real. Todos os dados existentes sao de teste. Isso simplificou as 7 decisoes drasticamente (descartar historico = zero impacto). DECISOES REGISTRADAS ==================== 1. admin_audit_log_* -> DESCARTA (a) 2. product_views -> CONTINUA (a) 3. Expert IA -> CONTINUA (a) 4. MFA step-up -> DESCARTA (b) 5. Bitrix24 -> CONTINUA (a) 6. ai_usage_* -> CONTINUA (a) 7. Voz / Voice Agent -> CONTINUA (a) EFEITOS COLATERAIS NOTAVEIS ============================ - 3 integracoes pagas continuam ativas: API IA, ElevenLabs, Bitrix24 - ai_usage (#6) protege contra explosao de custos das outras 3 - 2 recomendacoes minhas foram contrariadas (#5, #7) — decisao do Joaquim vale PROXIMOS PASSOS =============== Pra DESBLOQUEAR execucao da Fase 3, Joaquim precisa: - F3-0.9: transferir Supabase Lovable pra conta propria (3 cliques) - F3-0.10: enable_signup = false no painel - F3-0.11: enable_anonymous_sign_ins = false no painel GOVERNANCA ========== Mudanca Pequena (2 arquivos docs, ~100 linhas) -> merge direto na main. REFS ==== - F3-0.1 a F3-0.11 versionados - Sessao da tarde: docs/HANDOFF-2026-05-07.md (intacto) - Sessao de noite atualizada: docs/HANDOFF-2026-05-07-SESSAO-NOITE.md
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.
Em portugues claro pro Joaquim
Problema: durante o PR #97 descobri 3 hooks que NAO eram importados por ninguem fora do proprio cluster — orfaos sobreviventes da `ProductRegistrationPage` que ja deletei. Este PR limpa essa sobra.
Solucao: apaga os 3 hooks (-513 linhas) + remove secao 3.4 do doc `FUNCIONALIDADES` que apontava pra eles.
Topologia do cluster (validada)
```
useProductRegistration --> (so importado por useBulkImportFile)
useBulkImportFile --> (NINGUEM importa)
useProductRegistrationImport --> (NINGUEM importa)
```
Todos os 3 sao filhos da `ProductRegistrationPage.tsx` deletada no PR #97 (commit `741af39aa`).
Arquivos alterados
Validacoes
Red team antes de executar
Cobri os mesmos vetores do PR #97:
Bug encontrado durante execucao: minha primeira tentativa de renumeracao do FUNCIONALIDADES pulou seções 3.10-3.12 (passei na 3.9 e parei). Detectei e corrigi no proprio commit antes do push.
Risco
Refs
Summary by CodeRabbit
Removed Features
Documentation