Skip to content

chore(cleanup): remove arquivos gerados versionados (F1-1.x)#103

Merged
adm01-debug merged 1 commit into
mainfrom
cleanup/files-zumbi
May 9, 2026
Merged

chore(cleanup): remove arquivos gerados versionados (F1-1.x)#103
adm01-debug merged 1 commit into
mainfrom
cleanup/files-zumbi

Conversation

@adm01-debug
Copy link
Copy Markdown
Owner

@adm01-debug adm01-debug commented May 9, 2026

Plano

Remove 2 arquivos auto-gerados que não deveriam estar versionados, e adiciona entries no .gitignore pra prevenir regressão.

Arquivos removidos

Arquivo Tamanho Motivo
vite.config.d.ts 171 B Auto-gerado pelo TypeScript ao compilar vite.config.ts (declarations)
triage-edge-typecheck.json 2.3 KB Output do script scripts/triage-edge-typecheck.mjs (auditoria local de edge functions) — não pertence ao repo

.gitignore — entries adicionadas

# Generated files (auto-emit) — NÃO commitar
vite.config.d.ts

# Outputs de auditoria/triage — gerados localmente, não devem versionar
triage-edge-typecheck.json
triage-*.json

Risk

🟢 Zero. Validado pré-commit:

  • grep -r vite.config.d.ts0 referências em *.json, *.ts, *.js, *.mjs, *.yaml
  • triage-edge-typecheck.json é referenciado apenas por scripts/triage-edge-typecheck.mjs (o script que GERA o arquivo) — CI não consome o JSON
  • .github/ workflows não dependem desses arquivos

Test plan

  • npm run build continua funcionando (Vite regenera vite.config.d.ts localmente, mas agora ignorado)
  • node scripts/triage-edge-typecheck.mjs continua gerando o JSON local (gitignored)
  • Pre-push hook ESLint passou: drift positivo (1571 → 1563 erros, -8)
  • CI verde (aguardando)
  • CodeRabbit revisão

Contexto

Faxina F1 — Onda 1 ('arquivos zumbi'). Próximas ondas relacionadas:

  • cleanup/branches-lovable — apaga 759 branches lovable-sync-* no remoto
  • cleanup/remove-lovable-auth — remove integração Lovable Auth (já aprovado)
  • docs/auth-sso-todo — documenta reativação SSO Google via Supabase nativo
  • refactor/tsconfig-consolidation (futuro PR separado) — consolidar tsconfig.json + tsconfig.node.json

Summary by CodeRabbit

Release Notes

  • Chores
    • Atualização de padrões de exclusão de arquivos gerados e artefatos de build.
    • Limpeza de arquivos temporários de configuração do projeto.

Review Change Stack

- Remove `vite.config.d.ts` (auto-gerado pelo TypeScript ao compilar `vite.config.ts`)
- Remove `triage-edge-typecheck.json` (output do script `scripts/triage-edge-typecheck.mjs`,
  ferramenta de auditoria local — não pertence ao repo)
- Adiciona entries no `.gitignore` pra prevenir regressão (incluindo wildcard `triage-*.json`)

**Risk**: zero. Validado: 0 referências externas a `vite.config.d.ts`;
`triage-edge-typecheck.json` só é referenciado pelo script que o gera.

**Test plan**:
- `npm run build` continua funcionando
- `node scripts/triage-edge-typecheck.mjs` continua gerando o JSON local (agora gitignored)
- Nenhum import TS/JS quebra (sem refs a esses arquivos)
Copilot AI review requested due to automatic review settings May 9, 2026 14:39
@vercel
Copy link
Copy Markdown

vercel Bot commented May 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
promo-gifts Ready Ready Preview, Comment May 9, 2026 2:40pm

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 9, 2026

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: String must contain at most 250 character(s) at "tone_instructions"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 99e6d1e7-8825-449b-afc5-11447c73845b

📥 Commits

Reviewing files that changed from the base of the PR and between 7e29078 and fec34da.

📒 Files selected for processing (3)
  • .gitignore
  • triage-edge-typecheck.json
  • vite.config.d.ts
💤 Files with no reviewable changes (2)
  • vite.config.d.ts
  • triage-edge-typecheck.json

📝 Walkthrough

Resumo

A solicitação de pull remove artefatos gerados e dados de auditoria temporários, atualizando o .gitignore para impedir o rastreamento futuro desses arquivos.

Alterações

Limpeza de Artefatos Temporários e Gerados

Camada / Arquivo(s) Resumo
Remoção de Declarações de Tipo
vite.config.d.ts
Remove a declaração de tipo padrão do módulo Vite (tipado como import("vite").UserConfigFnObject).
Limpeza de Resultados de Auditoria
triage-edge-typecheck.json
Remove os dados de resultados de triage armazenados anteriormente, incluindo metadata de versão, modo de execução e listas de módulos.
Atualização de Padrões de Exclusão
.gitignore
Adiciona novos padrões para ignorar vite.config.d.ts, triage-edge-typecheck.json e arquivos correspondentes a triage-*.json.

Estimativa de Esforço de Revisão de Código

🎯 1 (Trivial) | ⏱️ ~3 minutos

Poema

🐰 Papéis gerados, agora sumidos,
Triage limpo, auditorias abolidas,
Gitignore abraça o que não deveria existir,
Artefatos temporários, finalmente partir!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed O título descreve claramente o objetivo principal do PR: remover arquivos gerados que estão versionados, o que corresponde exatamente às mudanças no changeset.
Description check ✅ Passed A descrição do PR é bastante completa com plano detalhado, justificativa de risco, teste executado e contexto, embora não siga exatamente a template com todas as seções obrigatórias.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes two locally-generated artifacts that were mistakenly committed to the repository and adds .gitignore rules to prevent them from being versioned again.

Changes:

  • Removed vite.config.d.ts (TypeScript declaration artifact for vite.config.ts).
  • Removed triage-edge-typecheck.json (local output from scripts/triage-edge-typecheck.mjs).
  • Added .gitignore entries for these generated/triage outputs.

Reviewed changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated no comments.

File Description
vite.config.d.ts Deleted generated TypeScript declaration file that shouldn’t be tracked.
triage-edge-typecheck.json Deleted generated triage output JSON that shouldn’t be tracked.
.gitignore Ignores the above generated files (and triage-*.json) to avoid regressions.

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

@adm01-debug adm01-debug merged commit ff11cdd into main May 9, 2026
18 of 19 checks passed
@adm01-debug adm01-debug deleted the cleanup/files-zumbi branch May 9, 2026 14:51
adm01-debug added a commit that referenced this pull request May 9, 2026
24 erros eliminados nos PRs da Faxina F1 desta semana (Onda 1, #103, #104,
#99, etc) que entraram em main como drift positivo. O `pre-push` hook já
sugeria "Considere atualizar o baseline" desde a Onda 1.

**Antes:**
- baseline: 1571 erros
- atual:    1547 erros (drift positivo de -24)

**Depois:**
- baseline: 1547 erros (congelados)
- atual:    1547 erros (sem drift, sem regressão)

**Distribuição dos 1547 erros congelados:**
- 615 arquivos com pelo menos 1 violação ignorada
- Maiores ofensores continuam sendo no-unused-vars, no-explicit-any,
  no-undef, eqeqeq (mesmas categorias mapeadas no relatório de auditoria
  — elas serão atacadas em PRs futuros da Fase 6)

**Risk**: 🟢 zero. Apenas reduz o teto que o CI permite — torna o gate
mais rigoroso, não menos. Qualquer regressão futura de lint vai falhar mais cedo.

**Test plan**:
- [x] `npm run lint:baseline:update` (regenerou)
- [x] `npm run lint:baseline` (gate passa: atual=baseline=1547)
- [ ] CI verde
- [ ] CodeRabbit OK
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants