Skip to content

chore(workflow): adiciona input extraEdges em delete-orphan-edges.yml#134

Merged
adm01-debug merged 1 commit into
mainfrom
chore/delete-2-extra-edges
May 10, 2026
Merged

chore(workflow): adiciona input extraEdges em delete-orphan-edges.yml#134
adm01-debug merged 1 commit into
mainfrom
chore/delete-2-extra-edges

Conversation

@adm01-debug
Copy link
Copy Markdown
Owner

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

O que faz

Estende o workflow .github/workflows/delete-orphan-edges.yml (já em main pelo PR #133) com um input opcional extraEdges (CSV) que permite deletar edges adicionais além das 6 órfãs originais.

Por quê (caso de uso imediato)

Os PRs #131 e #132 foram mergeados removendo o código dos stubs quote-public-view e generate-mockup-nanobanana, mas as edges em prod continuam ativas (deleção operacional pendente).

Em vez de criar um workflow novo só pra essas 2 edges, ou editar o ORPHANS hardcoded a cada limpeza pontual, o input extraEdges resolve isso de forma genérica e reusável:

gh workflow run delete-orphan-edges.yml \
  -f dryRun=false \
  -f confirm=DELETE \
  -f extraEdges="quote-public-view,generate-mockup-nanobanana"

Mudança

Apenas .github/workflows/delete-orphan-edges.yml:

  • Novo input extraEdges (string, opcional, default vazio)
  • Bash split por vírgula com trim de whitespace
  • Adiciona ao array ORPHANS dinamicamente
  • Log mostra Extra edges: <none> ou os nomes passados
  • Comportamento default 100% inalterado (mesma lista de 6 quando extraEdges vazio)

Risco

Mínimo. Mudança isolada num único arquivo de workflow, com:

  • Gate de confirmação DELETE mantido
  • dryRun=true por default mantido
  • Trim de whitespace evita acidentes com espaços extras

[DECISION] Mergear com --admin esperado: o "Lint, Typecheck & Test" da main está com timeout 25min sistemático (problema pré-existente em tests/.../external-db mocks faltando export invokeExternalDb — investigação separada). Como esta mudança toca apenas YAML de workflow e não código TS, é seguro fazer override.

Summary by CodeRabbit

  • Chores
    • O fluxo de limpeza de funções Edge agora permite especificar funções adicionais para exclusão através de um novo parâmetro de entrada, oferecendo maior flexibilidade no gerenciamento de recursos.

Review Change Stack

Estende delete-orphan-edges.yml com um input opcional `extraEdges` (CSV) que
permite passar uma lista de edges adicionais a serem deletadas além das 6
órfãs originais. Útil para limpezas pontuais que vêm depois (ex.: stubs
removidos via PR mas com a edge ainda viva em prod).

Uso: Run workflow → extraEdges = "edge1,edge2" + dryRun=false + confirm=DELETE

Caso de uso imediato: deletar quote-public-view (PR #131 mergeado, edge ainda
em prod) e generate-mockup-nanobanana (PR #132 mergeado, edge ainda em prod).
Copilot AI review requested due to automatic review settings May 10, 2026 15:15
@vercel
Copy link
Copy Markdown

vercel Bot commented May 10, 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 10, 2026 3:15pm

@adm01-debug adm01-debug merged commit 0143702 into main May 10, 2026
13 of 14 checks passed
@adm01-debug adm01-debug deleted the chore/delete-2-extra-edges branch May 10, 2026 15:15
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 10, 2026

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3b8ae1a2-b8ee-4e20-a5c4-ff918d0034da

📥 Commits

Reviewing files that changed from the base of the PR and between a9c1a7b and d538097.

📒 Files selected for processing (1)
  • .github/workflows/delete-orphan-edges.yml

Visão Geral

O workflow Delete Orphan Edge Functions agora aceita um input CSV extensível (extraEdges) via workflow_dispatch, fazendo parse em tempo de execução e adicionando entries dinâmicas à lista padrão de orphans. Mensagens de saída e validação foram atualizadas para refletir totais dinâmicos.

Mudanças

Extensibilidade do Workflow de Deleção

Layer / Arquivo(s) Resumo
Contrato de Entrada
.github/workflows/delete-orphan-edges.yml
Novo input extraEdges (string CSV, opcional, padrão vazio) adicionado como workflow_dispatch input.
Parsing e Extensão da Lista
.github/workflows/delete-orphan-edges.yml
Input wired para variável de ambiente; parsing de CSV com trim de whitespace; entries não-vazias apendidas ao array ORPHANS; set -euo pipefail introduzido para shell safety.
Logging e Validação
.github/workflows/delete-orphan-edges.yml
Output dinâmico mostra ref, modo dry-run, valores de edges extras (ou <none>), contagem total; texto de validação pós-deleção generalizado de "6 edges" para "the removed edges" com nota datada.

Esforço Estimado de Review

🎯 2 (Simples) | ⏱️ ~10 minutos

PRs Possivelmente Relacionadas

  • adm01-debug/Promo_Gifts#133: PR principal que estende o mesmo workflow delete-orphan-edges.yml adicionando o input extraEdges e parsing dele na lista de deleção—mudanças diretamente relacionadas.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/delete-2-extra-edges

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

Extends the existing GitHub Actions workflow for deleting “orphan” Supabase Edge Functions by allowing operators to optionally pass additional edge function names via a new extraEdges CSV input, keeping the default behavior unchanged when not provided.

Changes:

  • Adds a new optional workflow_dispatch input extraEdges (CSV string) and wires it into the job environment.
  • Parses extraEdges, trims entries, and appends them to the default ORPHANS list before iterating deletions.
  • Updates usage/docs/comments and post-delete hints to reflect the expanded capability.

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

Comment on lines +96 to +97
# Trim whitespace
trimmed="$(echo "$extra" | xargs)"
fi
done
fi

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