@luanpdd/kit-mcp 1.33.0 → 1.34.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/LICENSE +21 -21
- package/README.md +168 -168
- package/gates/agent-no-recursive-dispatch.md +84 -84
- package/kit/COMANDOS.md +138 -138
- package/kit/COMPATIBILITY.md +70 -70
- package/kit/README.md +76 -76
- package/kit/agents/advisor-researcher.md +109 -109
- package/kit/agents/ai-mutation-tester.md +289 -289
- package/kit/agents/assumptions-analyzer.md +110 -110
- package/kit/agents/audit-log-implementer.md +314 -314
- package/kit/agents/auditor-consistencia-isolamento.md +414 -414
- package/kit/agents/b2b-saas-architect.md +157 -157
- package/kit/agents/burn-rate-forecaster.md +153 -153
- package/kit/agents/cascading-failures-auditor.md +299 -299
- package/kit/agents/codebase-mapper.md +769 -769
- package/kit/agents/crm-pipeline-implementer.md +257 -257
- package/kit/agents/debugger.md +814 -814
- package/kit/agents/designer-ui.md +216 -216
- package/kit/agents/detector-tenant-quente.md +338 -338
- package/kit/agents/evolution-go-integrator.md +201 -201
- package/kit/agents/example-reviewer.md +22 -22
- package/kit/agents/executor.md +565 -565
- package/kit/agents/golden-signals-instrumenter.md +232 -232
- package/kit/agents/incident-investigator.md +238 -238
- package/kit/agents/integration-checker.md +203 -203
- package/kit/agents/invite-flow-implementer.md +190 -190
- package/kit/agents/legacy-characterizer.md +369 -369
- package/kit/agents/lgpd-compliance-auditor.md +296 -296
- package/kit/agents/load-shedding-instrumenter.md +290 -290
- package/kit/agents/multi-tenant-isolation-auditor.md +254 -254
- package/kit/agents/multi-tenant-rls-writer.md +341 -341
- package/kit/agents/nyquist-auditor.md +181 -181
- package/kit/agents/observability-coverage-auditor.md +316 -316
- package/kit/agents/observability-instrumenter.md +191 -191
- package/kit/agents/omm-auditor.md +291 -291
- package/kit/agents/org-onboarding-implementer.md +224 -224
- package/kit/agents/payload-capture-instrumenter.md +274 -274
- package/kit/agents/phase-researcher.md +697 -697
- package/kit/agents/plan-checker.md +275 -275
- package/kit/agents/planner.md +923 -923
- package/kit/agents/postmortem-writer.md +273 -273
- package/kit/agents/project-researcher.md +653 -653
- package/kit/agents/prr-conductor.md +287 -287
- package/kit/agents/refactor-safety-auditor.md +405 -405
- package/kit/agents/release-pipeline-auditor.md +364 -364
- package/kit/agents/research-synthesizer.md +246 -246
- package/kit/agents/roadmapper.md +678 -678
- package/kit/agents/schema-checker.md +160 -160
- package/kit/agents/seam-finder.md +360 -360
- package/kit/agents/shotgun-surgery-detector.md +350 -350
- package/kit/agents/slo-engineer.md +217 -217
- package/kit/agents/storytelling-analyst.md +300 -300
- package/kit/agents/supabase-architect.md +249 -249
- package/kit/agents/supabase-auth-bootstrapper.md +400 -400
- package/kit/agents/supabase-auth-hook-writer.md +418 -418
- package/kit/agents/supabase-branching-architect.md +563 -563
- package/kit/agents/supabase-cicd-pipeline-implementer.md +778 -778
- package/kit/agents/supabase-column-privileges-writer.md +400 -400
- package/kit/agents/supabase-edge-fn-tester.md +288 -288
- package/kit/agents/supabase-edge-fn-writer.md +341 -341
- package/kit/agents/supabase-mfa-implementer.md +439 -439
- package/kit/agents/supabase-migration-writer.md +386 -386
- package/kit/agents/supabase-oauth-server-implementer.md +507 -507
- package/kit/agents/supabase-rbac-implementer.md +393 -393
- package/kit/agents/supabase-realtime-implementer.md +364 -364
- package/kit/agents/supabase-rls-hardener.md +522 -522
- package/kit/agents/supabase-rls-writer.md +324 -324
- package/kit/agents/supabase-roles-implementer.md +356 -356
- package/kit/agents/supabase-social-auth-implementer.md +451 -451
- package/kit/agents/supabase-sso-saml-architect.md +549 -549
- package/kit/agents/supabase-storage-implementer.md +407 -407
- package/kit/agents/super-admin-implementer.md +282 -282
- package/kit/agents/toil-auditor.md +268 -268
- package/kit/agents/ui-auditor.md +438 -438
- package/kit/agents/ui-checker.md +305 -305
- package/kit/agents/ui-researcher.md +356 -356
- package/kit/agents/user-profiler.md +176 -176
- package/kit/agents/validador-evolucao-schema.md +336 -336
- package/kit/agents/verifier.md +729 -729
- package/kit/commands/adicionar-backlog.md +75 -75
- package/kit/commands/adicionar-fase.md +42 -42
- package/kit/commands/adicionar-tarefa.md +45 -45
- package/kit/commands/adicionar-testes.md +41 -41
- package/kit/commands/ajuda.md +21 -21
- package/kit/commands/atualizar.md +37 -37
- package/kit/commands/auditar-cascading.md +111 -111
- package/kit/commands/auditar-marco.md +179 -179
- package/kit/commands/auditar-observabilidade-cobertura-workflow.md +121 -0
- package/kit/commands/auditar-observabilidade-cobertura.md +183 -183
- package/kit/commands/auditar-refactor.md +219 -219
- package/kit/commands/auditar-release.md +109 -109
- package/kit/commands/auditar-uat.md +23 -23
- package/kit/commands/autonomo.md +40 -40
- package/kit/commands/branch-pr.md +24 -24
- package/kit/commands/burn-rate-status.md +408 -408
- package/kit/commands/capturar-payloads.md +193 -193
- package/kit/commands/caracterizar.md +212 -212
- package/kit/commands/concluir-marco.md +247 -247
- package/kit/commands/configuracoes.md +36 -36
- package/kit/commands/dados-distribuidos.md +188 -188
- package/kit/commands/definir-perfil.md +10 -10
- package/kit/commands/depurar.md +190 -190
- package/kit/commands/detectar-duplicacao.md +197 -197
- package/kit/commands/discutir-fase.md +131 -131
- package/kit/commands/encontrar-seams.md +136 -136
- package/kit/commands/entrar-discord.md +17 -17
- package/kit/commands/estatisticas.md +18 -18
- package/kit/commands/example-greeting.md +33 -33
- package/kit/commands/executar-fase.md +58 -58
- package/kit/commands/expresso.md +56 -56
- package/kit/commands/fase-ui.md +34 -34
- package/kit/commands/fazer.md +57 -57
- package/kit/commands/fio.md +125 -125
- package/kit/commands/fluxos-trabalho.md +64 -64
- package/kit/commands/forense.md +176 -176
- package/kit/commands/gerenciador.md +38 -38
- package/kit/commands/inserir-fase.md +31 -31
- package/kit/commands/legacy.md +263 -263
- package/kit/commands/limpeza.md +17 -17
- package/kit/commands/listar-hipoteses-fase.md +45 -45
- package/kit/commands/listar-workspaces.md +18 -18
- package/kit/commands/load-shedding.md +117 -117
- package/kit/commands/mapear-codebase.md +70 -70
- package/kit/commands/multi-tenant.md +163 -163
- package/kit/commands/nota.md +33 -33
- package/kit/commands/novo-marco.md +43 -43
- package/kit/commands/novo-projeto.md +41 -41
- package/kit/commands/novo-workspace.md +43 -43
- package/kit/commands/pausar-trabalho.md +37 -37
- package/kit/commands/perfil-usuario.md +45 -45
- package/kit/commands/pesquisar-fase.md +195 -195
- package/kit/commands/planejar-fase.md +67 -67
- package/kit/commands/planejar-lacunas.md +33 -33
- package/kit/commands/plantar-ideia.md +25 -25
- package/kit/commands/progresso.md +24 -24
- package/kit/commands/proximo.md +30 -30
- package/kit/commands/publicar.md +490 -490
- package/kit/commands/rapido.md +35 -35
- package/kit/commands/reaplicar-patches.md +124 -124
- package/kit/commands/refactor-seguro.md +321 -321
- package/kit/commands/relatorio-sessao.md +19 -19
- package/kit/commands/remover-fase.md +31 -31
- package/kit/commands/remover-workspace.md +26 -26
- package/kit/commands/resumo-marco.md +50 -50
- package/kit/commands/retomar-trabalho.md +40 -40
- package/kit/commands/revisar-backlog.md +60 -60
- package/kit/commands/revisar-ui.md +32 -32
- package/kit/commands/revisar.md +37 -37
- package/kit/commands/saude.md +21 -21
- package/kit/commands/setup-notion.md +93 -93
- package/kit/commands/storytelling.md +179 -179
- package/kit/commands/supabase.md +238 -238
- package/kit/commands/sync-main.md +68 -68
- package/kit/commands/validar-fase.md +35 -35
- package/kit/commands/verificar-tarefas.md +44 -44
- package/kit/commands/verificar-trabalho.md +64 -64
- package/kit/file-manifest.json +13 -11
- package/kit/framework/bin/lib/commands.cjs +959 -959
- package/kit/framework/bin/lib/config.cjs +442 -442
- package/kit/framework/bin/lib/core.cjs +1230 -1230
- package/kit/framework/bin/lib/frontmatter.cjs +336 -336
- package/kit/framework/bin/lib/init.cjs +1442 -1442
- package/kit/framework/bin/lib/milestone.cjs +252 -252
- package/kit/framework/bin/lib/model-profiles.cjs +68 -68
- package/kit/framework/bin/lib/phase.cjs +888 -888
- package/kit/framework/bin/lib/profile-output.cjs +952 -952
- package/kit/framework/bin/lib/profile-pipeline.cjs +539 -539
- package/kit/framework/bin/lib/roadmap.cjs +329 -329
- package/kit/framework/bin/lib/security.cjs +382 -382
- package/kit/framework/bin/lib/state.cjs +1031 -1031
- package/kit/framework/bin/lib/template.cjs +222 -222
- package/kit/framework/bin/lib/uat.cjs +282 -282
- package/kit/framework/bin/lib/verify.cjs +888 -888
- package/kit/framework/bin/lib/workstream.cjs +491 -491
- package/kit/framework/bin/tools.cjs +918 -918
- package/kit/framework/commands/workstreams.md +63 -63
- package/kit/framework/references/checkpoints.md +778 -778
- package/kit/framework/references/continuation-format.md +249 -249
- package/kit/framework/references/decimal-phase-calculation.md +64 -64
- package/kit/framework/references/git-integration.md +295 -295
- package/kit/framework/references/git-planning-commit.md +38 -38
- package/kit/framework/references/model-profile-resolution.md +36 -36
- package/kit/framework/references/model-profiles.md +139 -139
- package/kit/framework/references/phase-argument-parsing.md +61 -61
- package/kit/framework/references/planning-config.md +202 -202
- package/kit/framework/references/questioning.md +162 -162
- package/kit/framework/references/tdd.md +263 -263
- package/kit/framework/references/ui-brand.md +160 -160
- package/kit/framework/references/user-profiling.md +657 -657
- package/kit/framework/references/verification-patterns.md +612 -612
- package/kit/framework/references/workstream-flag.md +58 -58
- package/kit/framework/templates/DEBUG.md +164 -164
- package/kit/framework/templates/UAT.md +265 -265
- package/kit/framework/templates/UI-SPEC.md +100 -100
- package/kit/framework/templates/VALIDATION.md +76 -76
- package/kit/framework/templates/claude-md.md +122 -122
- package/kit/framework/templates/codebase/architecture.md +185 -185
- package/kit/framework/templates/codebase/concerns.md +205 -205
- package/kit/framework/templates/codebase/conventions.md +204 -204
- package/kit/framework/templates/codebase/integrations.md +192 -192
- package/kit/framework/templates/codebase/stack.md +158 -158
- package/kit/framework/templates/codebase/structure.md +199 -199
- package/kit/framework/templates/codebase/testing.md +301 -301
- package/kit/framework/templates/config.json +44 -44
- package/kit/framework/templates/context.md +352 -352
- package/kit/framework/templates/continue-here.md +78 -78
- package/kit/framework/templates/copilot-instructions.md +7 -7
- package/kit/framework/templates/debug-subagent-prompt.md +91 -91
- package/kit/framework/templates/dev-preferences.md +20 -20
- package/kit/framework/templates/discovery.md +146 -146
- package/kit/framework/templates/discussion-log.md +63 -63
- package/kit/framework/templates/milestone-archive.md +123 -123
- package/kit/framework/templates/milestone.md +115 -115
- package/kit/framework/templates/phase-prompt.md +610 -610
- package/kit/framework/templates/planner-subagent-prompt.md +117 -117
- package/kit/framework/templates/project.md +186 -186
- package/kit/framework/templates/requirements.md +231 -231
- package/kit/framework/templates/research-project/ARCHITECTURE.md +204 -204
- package/kit/framework/templates/research-project/FEATURES.md +147 -147
- package/kit/framework/templates/research-project/PITFALLS.md +200 -200
- package/kit/framework/templates/research-project/STACK.md +120 -120
- package/kit/framework/templates/research-project/SUMMARY.md +170 -170
- package/kit/framework/templates/research.md +419 -419
- package/kit/framework/templates/retrospective.md +54 -54
- package/kit/framework/templates/roadmap.md +202 -202
- package/kit/framework/templates/state.md +176 -176
- package/kit/framework/templates/summary-complex.md +59 -59
- package/kit/framework/templates/summary-minimal.md +41 -41
- package/kit/framework/templates/summary-standard.md +48 -48
- package/kit/framework/templates/summary.md +209 -209
- package/kit/framework/templates/user-profile.md +146 -146
- package/kit/framework/templates/user-setup.md +256 -256
- package/kit/framework/templates/verification-report.md +258 -258
- package/kit/framework/workflows/add-phase.md +112 -112
- package/kit/framework/workflows/add-tests.md +351 -351
- package/kit/framework/workflows/add-todo.md +158 -158
- package/kit/framework/workflows/audit-milestone.md +340 -340
- package/kit/framework/workflows/audit-uat.md +109 -109
- package/kit/framework/workflows/autonomous.md +891 -891
- package/kit/framework/workflows/check-todos.md +177 -177
- package/kit/framework/workflows/cleanup.md +152 -152
- package/kit/framework/workflows/complete-milestone.md +696 -696
- package/kit/framework/workflows/diagnose-issues.md +231 -231
- package/kit/framework/workflows/discovery-phase.md +289 -289
- package/kit/framework/workflows/discuss-phase-assumptions.md +653 -653
- package/kit/framework/workflows/discuss-phase.md +784 -784
- package/kit/framework/workflows/do.md +104 -104
- package/kit/framework/workflows/execute-phase.md +838 -838
- package/kit/framework/workflows/execute-plan.md +510 -510
- package/kit/framework/workflows/fast.md +102 -102
- package/kit/framework/workflows/forensics.md +265 -265
- package/kit/framework/workflows/health.md +181 -181
- package/kit/framework/workflows/help.md +619 -619
- package/kit/framework/workflows/insert-phase.md +130 -130
- package/kit/framework/workflows/list-phase-assumptions.md +178 -178
- package/kit/framework/workflows/list-workspaces.md +56 -56
- package/kit/framework/workflows/manager.md +362 -362
- package/kit/framework/workflows/map-codebase.md +377 -377
- package/kit/framework/workflows/milestone-summary.md +223 -223
- package/kit/framework/workflows/new-milestone.md +486 -486
- package/kit/framework/workflows/new-project.md +1159 -1159
- package/kit/framework/workflows/new-workspace.md +237 -237
- package/kit/framework/workflows/next.md +97 -97
- package/kit/framework/workflows/node-repair.md +92 -92
- package/kit/framework/workflows/note.md +156 -156
- package/kit/framework/workflows/pause-work.md +176 -176
- package/kit/framework/workflows/plan-milestone-gaps.md +273 -273
- package/kit/framework/workflows/plan-phase.md +765 -765
- package/kit/framework/workflows/plant-seed.md +169 -169
- package/kit/framework/workflows/pr-branch.md +129 -129
- package/kit/framework/workflows/profile-user.md +450 -450
- package/kit/framework/workflows/progress.md +507 -507
- package/kit/framework/workflows/quick.md +757 -757
- package/kit/framework/workflows/remove-phase.md +155 -155
- package/kit/framework/workflows/remove-workspace.md +90 -90
- package/kit/framework/workflows/research-phase.md +82 -82
- package/kit/framework/workflows/resume-project.md +326 -326
- package/kit/framework/workflows/review.md +228 -228
- package/kit/framework/workflows/session-report.md +146 -146
- package/kit/framework/workflows/settings.md +283 -283
- package/kit/framework/workflows/ship.md +228 -228
- package/kit/framework/workflows/stats.md +60 -60
- package/kit/framework/workflows/transition.md +671 -671
- package/kit/framework/workflows/ui-phase.md +302 -302
- package/kit/framework/workflows/ui-review.md +165 -165
- package/kit/framework/workflows/update.md +323 -323
- package/kit/framework/workflows/validate-phase.md +174 -174
- package/kit/framework/workflows/verify-phase.md +252 -252
- package/kit/framework/workflows/verify-work.md +637 -637
- package/kit/hooks/check-update.js +118 -118
- package/kit/hooks/context-monitor.js +163 -163
- package/kit/hooks/kit-attribution-reminder.cjs +92 -92
- package/kit/hooks/kit-router.cjs +137 -137
- package/kit/hooks/prompt-guard.js +103 -103
- package/kit/hooks/statusline.js +125 -125
- package/kit/hooks/workflow-guard.js +101 -101
- package/kit/settings.json +45 -45
- package/kit/skills/ai-prompt-characterization/SKILL.md +335 -335
- package/kit/skills/armadilhas-sistemas-distribuidos/SKILL.md +447 -447
- package/kit/skills/audit-log-multi-tenant/SKILL.md +340 -340
- package/kit/skills/b2b-saas-architecture/SKILL.md +300 -300
- package/kit/skills/consistencia-leitura-replica/SKILL.md +385 -385
- package/kit/skills/crm-lead-pipeline-patterns/SKILL.md +343 -343
- package/kit/skills/escolha-modelo-consistencia/SKILL.md +494 -494
- package/kit/skills/evolucao-schema-compativel/SKILL.md +448 -448
- package/kit/skills/evolution-go-whatsapp-integration/SKILL.md +322 -322
- package/kit/skills/example-skill/SKILL.md +42 -42
- package/kit/skills/legacy-api-only-applications/SKILL.md +358 -358
- package/kit/skills/legacy-characterization-tests/SKILL.md +330 -330
- package/kit/skills/legacy-effect-analysis/SKILL.md +331 -331
- package/kit/skills/legacy-extract-class/SKILL.md +203 -203
- package/kit/skills/legacy-programming-by-difference/SKILL.md +252 -252
- package/kit/skills/legacy-seams-and-test-harness/SKILL.md +460 -460
- package/kit/skills/legacy-shotgun-surgery/SKILL.md +286 -286
- package/kit/skills/legacy-sprout-wrap-techniques/SKILL.md +434 -434
- package/kit/skills/legacy-storytelling-naked-crc/SKILL.md +270 -270
- package/kit/skills/lgpd-multi-tenant-compliance/SKILL.md +340 -340
- package/kit/skills/member-invite-flow/SKILL.md +305 -305
- package/kit/skills/member-management-react-shadcn/SKILL.md +328 -328
- package/kit/skills/multi-tenant-performance-scaling/SKILL.md +316 -316
- package/kit/skills/multi-tenant-rls-hierarchy/SKILL.md +342 -342
- package/kit/skills/org-onboarding-flow/SKILL.md +257 -257
- package/kit/skills/org-switcher-react-pattern/SKILL.md +349 -349
- package/kit/skills/permission-gate-react-pattern/SKILL.md +271 -271
- package/kit/skills/postgres-isolamento-concorrencia/SKILL.md +552 -552
- package/kit/skills/pre-refactor-characterization/SKILL.md +421 -421
- package/kit/skills/rbac-permissions-matrix-supabase/SKILL.md +338 -338
- package/kit/skills/streams-eventos-cdc/SKILL.md +711 -711
- package/kit/skills/supabase-auth-hardening/SKILL.md +674 -674
- package/kit/skills/supabase-auth-hooks/SKILL.md +875 -875
- package/kit/skills/supabase-auth-methods/SKILL.md +486 -486
- package/kit/skills/supabase-auth-sessions/SKILL.md +579 -579
- package/kit/skills/supabase-auth-ssr/SKILL.md +306 -306
- package/kit/skills/supabase-branching-workflow/SKILL.md +544 -544
- package/kit/skills/supabase-ci-cd-github-actions/SKILL.md +880 -880
- package/kit/skills/supabase-column-level-security/SKILL.md +426 -426
- package/kit/skills/supabase-config-toml-remotes/SKILL.md +807 -807
- package/kit/skills/supabase-custom-claims-rbac/SKILL.md +472 -472
- package/kit/skills/supabase-edge-functions/SKILL.md +330 -330
- package/kit/skills/supabase-edge-functions-auth/SKILL.md +309 -309
- package/kit/skills/supabase-edge-functions-limits/SKILL.md +302 -302
- package/kit/skills/supabase-edge-functions-mcp-server/SKILL.md +279 -279
- package/kit/skills/supabase-edge-functions-testing/SKILL.md +277 -277
- package/kit/skills/supabase-edge-runtime-builtins/SKILL.md +357 -357
- package/kit/skills/supabase-enterprise-sso-saml/SKILL.md +545 -545
- package/kit/skills/supabase-jwt-signing-keys/SKILL.md +399 -399
- package/kit/skills/supabase-mfa/SKILL.md +488 -488
- package/kit/skills/supabase-migration-repair/SKILL.md +823 -823
- package/kit/skills/supabase-migrations/SKILL.md +297 -297
- package/kit/skills/supabase-oauth-server/SKILL.md +537 -537
- package/kit/skills/supabase-pgtap-testing/SKILL.md +1053 -1053
- package/kit/skills/supabase-postgres-roles/SKILL.md +392 -392
- package/kit/skills/supabase-realtime/SKILL.md +460 -460
- package/kit/skills/supabase-rls-defense-in-depth/SKILL.md +418 -418
- package/kit/skills/supabase-rls-policies/SKILL.md +635 -635
- package/kit/skills/supabase-social-oauth/SKILL.md +480 -480
- package/kit/skills/supabase-third-party-auth/SKILL.md +450 -450
- package/kit/skills/super-admin-platform-pattern/SKILL.md +326 -326
- package/kit/skills/tenant-quente-mitigacao/SKILL.md +605 -605
- package/kit/skills/ui-anti-padroes-ia/SKILL.md +261 -261
- package/kit/skills/ui-contexto-produto/SKILL.md +248 -248
- package/kit/skills/ui-cor-estrategia/SKILL.md +213 -213
- package/kit/skills/ui-critica-auditoria/SKILL.md +260 -260
- package/kit/skills/ui-motion-funcional/SKILL.md +264 -264
- package/kit/skills/ui-ritmo-espacial/SKILL.md +259 -259
- package/kit/skills/ui-tipografia/SKILL.md +211 -211
- package/kit/skills/whatsapp-conversation-state-machine/SKILL.md +287 -287
- package/kit/workflows/auditar-observabilidade-cobertura.workflow.js +250 -0
- package/package.json +65 -63
- package/src/core/kit.js +333 -216
- package/src/core/reflect.js +247 -247
- package/src/core/registry.js +123 -112
- package/src/core/reverse-sync.js +448 -372
- package/src/core/sync.js +477 -437
- package/src/core/watch.js +121 -121
- package/src/mcp-server/index.js +794 -794
|
@@ -1,414 +1,414 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: auditor-consistencia-isolamento
|
|
3
|
-
tier: specialized
|
|
4
|
-
description: Audita migrations + RPCs + Edge Functions Supabase em busca de 6 anti-patterns canônicos de race condition (SELECT-then-UPDATE sem FOR UPDATE = lost update vulnerable; trigger sem materializar pr…
|
|
5
|
-
tools: Read, Grep, Glob, Bash, Write
|
|
6
|
-
color: red
|
|
7
|
-
---
|
|
8
|
-
|
|
9
|
-
Você é o **auditor-consistencia-isolamento** — agent da Suíte DDIA Foundations v1.22. Audita o repositório (migrations + RPCs + Edge Functions Supabase) em busca de 6 anti-patterns canônicos de race condition catalogados em DDIA Ch 7 (Transactions) + Ch 8 (Distributed Systems Trouble), e produz `AUDITORIA-CONSISTENCIA.md` priorizado P0/P1/P2 com findings linkados a `arquivo:linha` + fix referenciando skill canônica.
|
|
10
|
-
|
|
11
|
-
**Compat:** Full em todos os IDEs (filesystem-only via Read/Grep/Glob). Não requer MCP Supabase — análise é estática sobre arquivos do repo.
|
|
12
|
-
|
|
13
|
-
## Por que existe
|
|
14
|
-
|
|
15
|
-
Race conditions em apps multi-tenant Supabase são **silent failure mode** — gaps não geram erro óbvio até virar incident: contador de uso por tenant fica errado (lost update), 2 admins criam slug global duplicado simultaneamente (UNIQUE check em app), webhook handler processa mesma mensagem 2× (sem idempotência). DDIA Ch 7 (Transactions) + Ch 8 (Distributed Systems Trouble) cataloga 6 anti-patterns canônicos que esse agent detecta scaneando o codebase ANTES de virar production incident.
|
|
16
|
-
|
|
17
|
-
Phase 122 (AGENTE-01..02) introduz este agent à Suíte DDIA Foundations v1.22. Pattern v1.21 herdado: agent detecta + propõe fix mas NÃO escreve — delega para `supabase-migration-writer` (v1.8) ou `supabase-edge-fn-writer` (v1.8) via cross-suite handoff.
|
|
18
|
-
|
|
19
|
-
## Inputs esperados (do caller)
|
|
20
|
-
|
|
21
|
-
- (Opcional) `scope`: caminhos a auditar (default: `supabase/migrations/`, `supabase/functions/`, `supabase/schemas/`)
|
|
22
|
-
- (Opcional) `output_path`: onde escrever o audit (default: `.planning/AUDITORIA-CONSISTENCIA.md`)
|
|
23
|
-
- (Opcional) `include_patterns`: glob extra (ex: `apps/api/**/*.ts` para incluir RPCs em monorepo)
|
|
24
|
-
|
|
25
|
-
## Passos
|
|
26
|
-
|
|
27
|
-
### Step 0 — Preflight
|
|
28
|
-
|
|
29
|
-
Detectar escopo:
|
|
30
|
-
|
|
31
|
-
```bash
|
|
32
|
-
# Verificar paths default
|
|
33
|
-
for path in supabase/migrations supabase/functions supabase/schemas; do
|
|
34
|
-
[ -d "$path" ] && echo "FOUND: $path"
|
|
35
|
-
done
|
|
36
|
-
|
|
37
|
-
# Criar destination dir se necessário
|
|
38
|
-
mkdir -p "$(dirname "$OUTPUT_PATH")"
|
|
39
|
-
```
|
|
40
|
-
|
|
41
|
-
Se NENHUM path do escopo existe: emitir aviso e sair com `status=skipped` (nada a auditar). Caso contrário, prosseguir com 6 detectores.
|
|
42
|
-
|
|
43
|
-
### Step 1 — Detector 1: SELECT-then-UPDATE sem FOR UPDATE (Lost Update — P0)
|
|
44
|
-
|
|
45
|
-
**Padrão detectado (anti-pattern):**
|
|
46
|
-
|
|
47
|
-
```sql
|
|
48
|
-
-- ANTI-PATTERN — race window entre SELECT e UPDATE
|
|
49
|
-
SELECT count FROM counters WHERE id = $1;
|
|
50
|
-
-- ... lógica em app que computa novo valor ...
|
|
51
|
-
UPDATE counters SET count = $new_count WHERE id = $1;
|
|
52
|
-
```
|
|
53
|
-
|
|
54
|
-
**Heurística de detecção:**
|
|
55
|
-
|
|
56
|
-
```bash
|
|
57
|
-
# Procurar SELECT seguido de UPDATE no mesmo bloco PL/pgSQL ou função TS
|
|
58
|
-
# sem FOR UPDATE no SELECT
|
|
59
|
-
grep -rn -E "SELECT.*FROM.*WHERE.*\\\$1" supabase/ \
|
|
60
|
-
| while read line; do
|
|
61
|
-
file=$(echo "$line" | cut -d: -f1)
|
|
62
|
-
# Verificar se há UPDATE na mesma função/bloco e SELECT não tem FOR UPDATE
|
|
63
|
-
grep -A 20 "$line" "$file" | grep -q "UPDATE.*SET.*WHERE.*\$1" \
|
|
64
|
-
&& ! grep -A 5 "$line" "$file" | grep -q "FOR UPDATE" \
|
|
65
|
-
&& echo "MATCH: $line"
|
|
66
|
-
done
|
|
67
|
-
```
|
|
68
|
-
|
|
69
|
-
**Severidade:** P0 (vulnerável a lost update em concorrência alta — usage counter, billing, inventory)
|
|
70
|
-
|
|
71
|
-
**Fix sugerido (referência canônica):** Use `SELECT ... FOR UPDATE` para lock pessimista OU atomic `UPDATE counters SET count = count + 1 WHERE id = $1` para evitar a leitura intermediária. Ver skill [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) seção Padrões para prevenir lost update.
|
|
72
|
-
|
|
73
|
-
### Step 2 — Detector 2: Trigger sem materializar predicate (Write Skew — P1)
|
|
74
|
-
|
|
75
|
-
**Padrão detectado:**
|
|
76
|
-
|
|
77
|
-
```sql
|
|
78
|
-
-- ANTI-PATTERN — trigger valida invariante cross-row sem lock
|
|
79
|
-
CREATE TRIGGER check_at_least_one_admin
|
|
80
|
-
BEFORE UPDATE ON members
|
|
81
|
-
FOR EACH ROW EXECUTE FUNCTION ensure_admin_exists();
|
|
82
|
-
-- ensure_admin_exists() faz SELECT count(*) FROM members WHERE role='admin'
|
|
83
|
-
-- Sem FOR UPDATE OR SERIALIZABLE → write skew em concorrência
|
|
84
|
-
```
|
|
85
|
-
|
|
86
|
-
**Heurística:**
|
|
87
|
-
|
|
88
|
-
```bash
|
|
89
|
-
# Procurar BEFORE UPDATE triggers em tabelas com check de cardinalidade cross-row
|
|
90
|
-
grep -rn -E "CREATE TRIGGER.*BEFORE (UPDATE|DELETE)" supabase/migrations/ \
|
|
91
|
-
| while read line; do
|
|
92
|
-
file=$(echo "$line" | cut -d: -f1)
|
|
93
|
-
func=$(echo "$line" | grep -oE "EXECUTE FUNCTION \\w+" | awk '{print $3}')
|
|
94
|
-
# Verificar se a função tem SELECT count(*) sem FOR UPDATE
|
|
95
|
-
[ -n "$func" ] && grep -A 30 "CREATE.*FUNCTION $func" "$file" \
|
|
96
|
-
| grep -q "select count" \
|
|
97
|
-
&& ! grep -A 30 "CREATE.*FUNCTION $func" "$file" | grep -q "FOR UPDATE" \
|
|
98
|
-
&& echo "MATCH: $line (function $func)"
|
|
99
|
-
done
|
|
100
|
-
```
|
|
101
|
-
|
|
102
|
-
**Severidade:** P1 (vulnerável em invariantes cross-row tipo "no mais de 1 admin sem boss")
|
|
103
|
-
|
|
104
|
-
**Fix sugerido:** Use `SET TRANSACTION ISOLATION LEVEL SERIALIZABLE` no caller OU `SELECT ... FOR UPDATE` nas rows do predicate dentro da função trigger OU `EXCLUDE` constraint via `btree_gist` para invariantes simples. Ver skill [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) seção Write Skew + skill [`escolha-modelo-consistencia`](../skills/escolha-modelo-consistencia/SKILL.md) seção Uniqueness Constraints.
|
|
105
|
-
|
|
106
|
-
### Step 3 — Detector 3: Clock skew em lógica de expiração (P0)
|
|
107
|
-
|
|
108
|
-
**Padrão detectado:**
|
|
109
|
-
|
|
110
|
-
```sql
|
|
111
|
-
-- ANTI-PATTERN — clock_timestamp() é não-deterministico, pode dar skew em transações longas
|
|
112
|
-
WHERE expires_at < clock_timestamp()
|
|
113
|
-
WHERE valid_until > clock_timestamp()
|
|
114
|
-
```
|
|
115
|
-
|
|
116
|
-
```ts
|
|
117
|
-
// ANTI-PATTERN — Date.now() em Edge Function depende do worker que executa
|
|
118
|
-
if (token.expires_at < new Date()) { /* expirou */ }
|
|
119
|
-
```
|
|
120
|
-
|
|
121
|
-
**Heurística:**
|
|
122
|
-
|
|
123
|
-
```bash
|
|
124
|
-
# Procurar clock_timestamp() ou Date.now() em comparação com colunas de TTL/expiração
|
|
125
|
-
grep -rn -E "clock_timestamp\\(\\)" supabase/ \
|
|
126
|
-
| grep -iE "(expir|ttl|valid_until|deadline|due_at)" \
|
|
127
|
-
&& grep -rn -E "Date\\.now\\(\\)" supabase/functions/ \
|
|
128
|
-
| grep -iE "(expir|ttl|valid_until|deadline|token)"
|
|
129
|
-
```
|
|
130
|
-
|
|
131
|
-
**Severidade:** P0 (lógica de auth/billing pode quebrar — token expira "antes da hora" em algumas partes da query)
|
|
132
|
-
|
|
133
|
-
**Fix sugerido:** Use `now()` (alias para `transaction_timestamp()` — fixo no início da transação, monotônico) OU passe timestamp explícito via parâmetro. Para Edge Functions, capture `new Date()` UMA vez no início do handler. Ver skill [`armadilhas-sistemas-distribuidos`](../skills/armadilhas-sistemas-distribuidos/SKILL.md) seção Clock Skew + Unbounded Monotonic Clocks.
|
|
134
|
-
|
|
135
|
-
### Step 4 — Detector 4: UNIQUE check em nível de app (Race — P0)
|
|
136
|
-
|
|
137
|
-
**Padrão detectado:**
|
|
138
|
-
|
|
139
|
-
```ts
|
|
140
|
-
// ANTI-PATTERN — race window entre check e insert
|
|
141
|
-
const existing = await supabase.from('orgs').select('id').eq('slug', slug).maybeSingle();
|
|
142
|
-
if (existing.data) throw new Error('slug taken');
|
|
143
|
-
await supabase.from('orgs').insert({ slug, name });
|
|
144
|
-
```
|
|
145
|
-
|
|
146
|
-
**Heurística:**
|
|
147
|
-
|
|
148
|
-
```bash
|
|
149
|
-
# Procurar sequência SELECT-existence-check + INSERT no mesmo handler/RPC
|
|
150
|
-
grep -rn -E "\\.select\\(.*\\)\\.eq\\(" supabase/functions/ \
|
|
151
|
-
| while read line; do
|
|
152
|
-
file=$(echo "$line" | cut -d: -f1)
|
|
153
|
-
# Verificar se há INSERT logo após (mesma função)
|
|
154
|
-
grep -A 20 "$line" "$file" | grep -q "\\.insert\\(" \
|
|
155
|
-
&& echo "MATCH: $line"
|
|
156
|
-
done
|
|
157
|
-
|
|
158
|
-
# Idem em RPCs PL/pgSQL
|
|
159
|
-
grep -rn -E "SELECT.*FROM.*WHERE.*=" supabase/migrations/ \
|
|
160
|
-
| while read line; do
|
|
161
|
-
file=$(echo "$line" | cut -d: -f1)
|
|
162
|
-
grep -A 15 "$line" "$file" | grep -q "INSERT INTO" \
|
|
163
|
-
&& ! grep -B 2 -A 5 "$line" "$file" | grep -q "ON CONFLICT" \
|
|
164
|
-
&& echo "MATCH: $line"
|
|
165
|
-
done
|
|
166
|
-
```
|
|
167
|
-
|
|
168
|
-
**Severidade:** P0 (race window entre check e insert — 2 requests concorrentes podem ambos passar o check)
|
|
169
|
-
|
|
170
|
-
**Fix sugerido:** Confie no `UNIQUE` constraint do Postgres e use `INSERT ... ON CONFLICT DO NOTHING RETURNING id` — atomic + linearizável via single-leader. Ver skill [`escolha-modelo-consistencia`](../skills/escolha-modelo-consistencia/SKILL.md) seção Uniqueness Constraints distribuídos via single-leader Postgres.
|
|
171
|
-
|
|
172
|
-
### Step 5 — Detector 5: Write cross-tenant sem lock (Lost Update — P1)
|
|
173
|
-
|
|
174
|
-
**Padrão detectado:**
|
|
175
|
-
|
|
176
|
-
```sql
|
|
177
|
-
-- ANTI-PATTERN — super-admin operation que toca múltiplos tenants sem FOR UPDATE
|
|
178
|
-
UPDATE quotas SET monthly_limit = monthly_limit + 1000; -- sem WHERE org_id, sem FOR UPDATE
|
|
179
|
-
```
|
|
180
|
-
|
|
181
|
-
**Heurística:**
|
|
182
|
-
|
|
183
|
-
```bash
|
|
184
|
-
# Procurar UPDATE em tabelas com coluna org_id sem WHERE org_id (cross-tenant)
|
|
185
|
-
# E sem FOR UPDATE no SELECT precedente
|
|
186
|
-
grep -rn -E "^UPDATE\\s+\\w+\\s+SET" supabase/ \
|
|
187
|
-
| while read line; do
|
|
188
|
-
file=$(echo "$line" | cut -d: -f1)
|
|
189
|
-
# Detectar UPDATE em tabela tenant-aware (org_id) sem WHERE org_id
|
|
190
|
-
table=$(echo "$line" | grep -oE "UPDATE \\w+" | awk '{print $2}')
|
|
191
|
-
grep -q "ALTER TABLE $table.*org_id\\|CREATE TABLE.*$table.*org_id" supabase/migrations/* 2>/dev/null \
|
|
192
|
-
&& ! echo "$line" | grep -q "WHERE.*org_id" \
|
|
193
|
-
&& ! grep -B 5 "$line" "$file" | grep -q "FOR UPDATE" \
|
|
194
|
-
&& echo "MATCH: $line (table $table cross-tenant)"
|
|
195
|
-
done
|
|
196
|
-
```
|
|
197
|
-
|
|
198
|
-
**Severidade:** P1 (vulnerável em super-admin operations + bulk updates)
|
|
199
|
-
|
|
200
|
-
**Fix sugerido:** Iterar por `org_id` com `SELECT ... FOR UPDATE` por linha, OU usar `pg_advisory_xact_lock(hashtext(<resource>))` para lock semântico cross-tenant. Ver skills [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) (advisory locks) + [`super-admin-platform-pattern`](../skills/super-admin-platform-pattern/SKILL.md) (cross-tenant write audit).
|
|
201
|
-
|
|
202
|
-
### Step 6 — Detector 6: Handler sem idempotência (Duplicate Processing — P1)
|
|
203
|
-
|
|
204
|
-
**Padrão detectado:**
|
|
205
|
-
|
|
206
|
-
```ts
|
|
207
|
-
// ANTI-PATTERN — pgmq handler ou webhook sem dedup
|
|
208
|
-
const messages = await pgmq.read('orders', 30, 10);
|
|
209
|
-
for (const msg of messages) {
|
|
210
|
-
await processOrder(msg.message); // sem dedup → retry duplica processamento
|
|
211
|
-
await pgmq.delete('orders', msg.msg_id);
|
|
212
|
-
}
|
|
213
|
-
```
|
|
214
|
-
|
|
215
|
-
**Heurística:**
|
|
216
|
-
|
|
217
|
-
```bash
|
|
218
|
-
# Edge Functions com pgmq.read OU webhook handler que faz INSERT sem checar dedup table
|
|
219
|
-
grep -rn -E "pgmq\\.read\\(|pgmq_read\\(" supabase/functions/ \
|
|
220
|
-
| while read line; do
|
|
221
|
-
file=$(echo "$line" | cut -d: -f1)
|
|
222
|
-
# Verificar se há tabela processed_events / dedup / idempotency_key referenciada no arquivo
|
|
223
|
-
! grep -q "processed_events\\|idempotency_key\\|dedup\\|message_id" "$file" \
|
|
224
|
-
&& echo "MATCH: $line (handler sem dedup)"
|
|
225
|
-
done
|
|
226
|
-
|
|
227
|
-
# Webhook handlers (pattern Deno serve com POST)
|
|
228
|
-
grep -rln "Deno.serve" supabase/functions/ \
|
|
229
|
-
| while read file; do
|
|
230
|
-
grep -q "POST\\|webhook\\|message" "$file" \
|
|
231
|
-
&& grep -q "INSERT" "$file" \
|
|
232
|
-
&& ! grep -q "processed_events\\|idempotency_key\\|message_id\\|ON CONFLICT" "$file" \
|
|
233
|
-
&& echo "MATCH: $file (webhook handler sem dedup)"
|
|
234
|
-
done
|
|
235
|
-
```
|
|
236
|
-
|
|
237
|
-
**Severidade:** P1 (duplicate processing em retry — billing duplicado, mensagem WhatsApp enviada 2×)
|
|
238
|
-
|
|
239
|
-
**Fix sugerido:** Adicione `processed_events` table com `unique(event_id)` + `INSERT INTO processed_events ... ON CONFLICT DO NOTHING` na mesma transação do processamento. Ver skill [`streams-eventos-cdc`](../skills/streams-eventos-cdc/SKILL.md) seção Exactly-once em pgmq + skill [`evolution-go-whatsapp-integration`](../skills/evolution-go-whatsapp-integration/SKILL.md) seção Idempotência (pattern v1.21).
|
|
240
|
-
|
|
241
|
-
### Step 7 — Agregar findings + classificar
|
|
242
|
-
|
|
243
|
-
Para cada match dos Detectores 1-6:
|
|
244
|
-
|
|
245
|
-
- Capturar `arquivo`, `linha`, severidade (P0/P1)
|
|
246
|
-
- Anexar fix sugerido (Markdown link ATIVO para skill canônica)
|
|
247
|
-
- Atribuir ID sequencial (`F-01`, `F-02`, ...)
|
|
248
|
-
|
|
249
|
-
Computar agregado:
|
|
250
|
-
|
|
251
|
-
| Severidade | Detectores | Critério |
|
|
252
|
-
|---|---|---|
|
|
253
|
-
| **P0** | 1, 3, 4 | Vulnerabilidade explorável em concorrência normal — release blocked |
|
|
254
|
-
| **P1** | 2, 5, 6 | Vulnerabilidade em condições específicas — fix antes de scale |
|
|
255
|
-
| **P2** | (reservado) | Documentação/cleanup — no candidates default |
|
|
256
|
-
|
|
257
|
-
### Step 8 — Escrever `AUDITORIA-CONSISTENCIA.md`
|
|
258
|
-
|
|
259
|
-
Escrever em `$OUTPUT_PATH` seguindo template canônico:
|
|
260
|
-
|
|
261
|
-
````markdown
|
|
262
|
-
# Auditoria de Consistência e Isolamento — <projeto> — <data>
|
|
263
|
-
|
|
264
|
-
> Gerado por `auditor-consistencia-isolamento` (Suíte DDIA Foundations v1.22)
|
|
265
|
-
> Escopo: <paths auditados>
|
|
266
|
-
|
|
267
|
-
## Sumário
|
|
268
|
-
|
|
269
|
-
| Severidade | Findings | Skill referenciada |
|
|
270
|
-
|---|---|---|
|
|
271
|
-
| P0 | <N> | [`postgres-isolamento-concorrencia`](../kit/skills/postgres-isolamento-concorrencia/SKILL.md), [`armadilhas-sistemas-distribuidos`](../kit/skills/armadilhas-sistemas-distribuidos/SKILL.md), [`escolha-modelo-consistencia`](../kit/skills/escolha-modelo-consistencia/SKILL.md) |
|
|
272
|
-
| P1 | <N> | [`streams-eventos-cdc`](../kit/skills/streams-eventos-cdc/SKILL.md), [`super-admin-platform-pattern`](../kit/skills/super-admin-platform-pattern/SKILL.md) |
|
|
273
|
-
| P2 | 0 | — |
|
|
274
|
-
|
|
275
|
-
## P0 — Críticos (BLOCK release)
|
|
276
|
-
|
|
277
|
-
### F-01 [P0] Lost Update em counters (Detector 1)
|
|
278
|
-
|
|
279
|
-
**Arquivo:** `supabase/functions/increment-counter/index.ts:45-48`
|
|
280
|
-
**Detalhe:** SELECT `count FROM counters WHERE id = $1` seguido de UPDATE sem `FOR UPDATE` — race window entre leitura e escrita.
|
|
281
|
-
**Fix:** Use `SELECT ... FOR UPDATE` ou atomic `UPDATE counters SET count = count + 1 WHERE id = $1`. Ver skill [`postgres-isolamento-concorrencia`](../kit/skills/postgres-isolamento-concorrencia/SKILL.md).
|
|
282
|
-
**Cross-suite handoff:** Delegue migration corrigida para [`supabase-migration-writer`](../kit/agents/supabase-migration-writer.md) (v1.8) OU Edge Function corrigida para [`supabase-edge-fn-writer`](../kit/agents/supabase-edge-fn-writer.md) (v1.8).
|
|
283
|
-
|
|
284
|
-
### F-02 [P0] Clock skew em token expiration (Detector 3)
|
|
285
|
-
|
|
286
|
-
**Arquivo:** `supabase/migrations/20260510_invites.sql:23`
|
|
287
|
-
**Detalhe:** `WHERE expires_at < clock_timestamp()` — clock_timestamp avança durante a transação, lógica de auth pode dar inconsistência.
|
|
288
|
-
**Fix:** Substitua por `now()` (transaction_timestamp). Ver skill [`armadilhas-sistemas-distribuidos`](../kit/skills/armadilhas-sistemas-distribuidos/SKILL.md).
|
|
289
|
-
|
|
290
|
-
[... mais findings ...]
|
|
291
|
-
|
|
292
|
-
## P1 — Altos (FIX antes de scale)
|
|
293
|
-
|
|
294
|
-
### F-NN [P1] Handler sem idempotência (Detector 6)
|
|
295
|
-
|
|
296
|
-
**Arquivo:** `supabase/functions/whatsapp-webhook/index.ts:78`
|
|
297
|
-
**Detalhe:** Webhook handler INSERT sem `processed_events` dedup nem ON CONFLICT — retry pode processar mesma mensagem 2×.
|
|
298
|
-
**Fix:** Adicione tabela `processed_events` + `INSERT INTO processed_events (event_id) VALUES ($1) ON CONFLICT DO NOTHING RETURNING id` antes do processamento. Ver skill [`streams-eventos-cdc`](../kit/skills/streams-eventos-cdc/SKILL.md).
|
|
299
|
-
|
|
300
|
-
[... mais findings ...]
|
|
301
|
-
|
|
302
|
-
## Recomendações
|
|
303
|
-
|
|
304
|
-
- P0 fixes: aplicar IMEDIATAMENTE — release blocked até resolvidos
|
|
305
|
-
- P1 fixes: priorizar antes de scale (>1k req/s OU >100 tenants ativos)
|
|
306
|
-
- Re-rodar este audit pós-fix para confirmar P0 = 0
|
|
307
|
-
|
|
308
|
-
## Próximos passos
|
|
309
|
-
|
|
310
|
-
1. Para cada P0, invocar [`supabase-migration-writer`](../kit/agents/supabase-migration-writer.md) (v1.8) ou [`supabase-edge-fn-writer`](../kit/agents/supabase-edge-fn-writer.md) (v1.8) com o fix sugerido
|
|
311
|
-
2. Re-auditar após fixes para confirmar `P0 = 0`
|
|
312
|
-
3. Agendar P1 fixes no próximo sprint (≤ 30 dias)
|
|
313
|
-
````
|
|
314
|
-
|
|
315
|
-
### Step 9 — Imprimir resumo curto para caller após escrita
|
|
316
|
-
|
|
317
|
-
```text
|
|
318
|
-
═══════════════════════════════════════════════════════════
|
|
319
|
-
AUDITOR-CONSISTENCIA-ISOLAMENTO · <project>
|
|
320
|
-
═══════════════════════════════════════════════════════════
|
|
321
|
-
|
|
322
|
-
Escopo: <N> arquivos auditados (migrations + functions + schemas)
|
|
323
|
-
|
|
324
|
-
P0: <count> findings (release blocked)
|
|
325
|
-
P1: <count> findings (fix antes de scale)
|
|
326
|
-
P2: 0 (sem candidates)
|
|
327
|
-
|
|
328
|
-
## Top 3 P0
|
|
329
|
-
1. <F-NN> <descrição curta> — <arquivo:linha>
|
|
330
|
-
2. ...
|
|
331
|
-
3. ...
|
|
332
|
-
|
|
333
|
-
## Output
|
|
334
|
-
`<OUTPUT_PATH>`
|
|
335
|
-
```
|
|
336
|
-
|
|
337
|
-
## Cross-suite invocation pattern (v1.21 herdado)
|
|
338
|
-
|
|
339
|
-
Quando este agent detecta problema, **propõe fix mas NÃO escreve**. Delega via Task() handoff:
|
|
340
|
-
|
|
341
|
-
| Tipo de fix | Agent destino | Suíte |
|
|
342
|
-
|---|---|---|
|
|
343
|
-
| Migration SQL corrigida (FOR UPDATE, ON CONFLICT, SERIALIZABLE) | [`supabase-migration-writer`](./supabase-migration-writer.md) | Supabase v1.8 |
|
|
344
|
-
| Edge Function corrigida (idempotency, dedup table, atomic UPDATE) | [`supabase-edge-fn-writer`](./supabase-edge-fn-writer.md) | Supabase v1.8 |
|
|
345
|
-
| RLS policy corrigida (write cross-tenant audit) | [`multi-tenant-rls-writer`](./multi-tenant-rls-writer.md) | Multi-Tenant v1.21 |
|
|
346
|
-
| Audit log de super-admin write | [`audit-log-implementer`](./audit-log-implementer.md) | Multi-Tenant v1.21 |
|
|
347
|
-
|
|
348
|
-
**Pattern:** o caller (orquestrador) lê findings de `AUDITORIA-CONSISTENCIA.md`, prioriza P0, e invoca agent destino com prompt contendo o fix sugerido. Este agent permanece função pura — só audit, nunca escreve fix (anti-pitfall A10 v1.8 herdado).
|
|
349
|
-
|
|
350
|
-
## Anti-patterns prevenidos (na produção do consumer)
|
|
351
|
-
|
|
352
|
-
- Lost update silencioso em usage counters por tenant (billing errado)
|
|
353
|
-
- Write skew em invariantes cross-row (ex: "no mais de 1 admin sem boss")
|
|
354
|
-
- Clock skew em auth/billing (token expira antes/depois do esperado)
|
|
355
|
-
- Race em UNIQUE check app-level (slug duplicado, license key duplicada)
|
|
356
|
-
- Write cross-tenant sem lock (super-admin operation corrompe quotas)
|
|
357
|
-
- Duplicate processing em pgmq/webhook (billing duplicado, mensagem WhatsApp 2×)
|
|
358
|
-
|
|
359
|
-
## Quando NÃO invocar
|
|
360
|
-
|
|
361
|
-
- Repo recém-criado (< 5 migrations) — sample size insuficiente, audit produz falso-zero
|
|
362
|
-
- Já rodou audit há < 7 dias sem mudanças significativas em `supabase/`
|
|
363
|
-
- Fase de greenfield design — agent é defesa para código existente, não substitui review de design
|
|
364
|
-
- App single-tenant (1 org fixa) — Detector 5 (cross-tenant) não se aplica; outros 5 ainda valem
|
|
365
|
-
|
|
366
|
-
## Observabilidade integrada
|
|
367
|
-
|
|
368
|
-
- Counter `audit.consistency.findings{severity=P0|P1|P2,detector=1..6}` por execução
|
|
369
|
-
- Histogram `audit.consistency.duration_ms` (latência total da auditoria)
|
|
370
|
-
- Cada finding fica registrado em `obs.events` com `audit_run_id` para rastreabilidade
|
|
371
|
-
|
|
372
|
-
## Validação de RLS hardener cooperativo (v1.23 — CROSS-09)
|
|
373
|
-
|
|
374
|
-
Para cada migration recente analisada pelos 6 detectores, este agent adicionalmente valida que a migration **passou pelo `supabase-rls-hardener`** antes de ser aplicada — Camada 7 de defense-in-depth + auditoria cross-suite.
|
|
375
|
-
|
|
376
|
-
**Detector adicional (D7 — hardener bypass):** consulta git log + procura por commits de migration que NÃO incluem trace de handoff cooperativo:
|
|
377
|
-
|
|
378
|
-
```bash
|
|
379
|
-
# Detectar migrations sem hardener trace
|
|
380
|
-
git log --oneline --all -- supabase/migrations/*.sql | while read commit msg; do
|
|
381
|
-
if ! git show $commit -- supabase/migrations/ | grep -q "supabase-rls-hardener\|HARDENER OK\|verdict: GO"; then
|
|
382
|
-
echo "$commit $msg ⚠ MIGRATION SEM TRACE DE HARDENER"
|
|
383
|
-
fi
|
|
384
|
-
done
|
|
385
|
-
```
|
|
386
|
-
|
|
387
|
-
**Output enriquecido com campo `hardener_passed`:**
|
|
388
|
-
|
|
389
|
-
```markdown
|
|
390
|
-
## Detector 7 — Migration sem hardener cooperativo (v1.23)
|
|
391
|
-
|
|
392
|
-
| Migration | Hardener | Verdict | Risk |
|
|
393
|
-
|-----------|----------|---------|------|
|
|
394
|
-
| 20260510120000_create_orders.sql | ✅ passed | STRENGTHEN | low |
|
|
395
|
-
| 20260510130000_legacy_table.sql | ❌ bypass | n/a | **P1** |
|
|
396
|
-
| 20260510140000_add_org_id.sql | ✅ passed | GO | low |
|
|
397
|
-
```
|
|
398
|
-
|
|
399
|
-
Migrations com `hardener_passed: false` são P1 (high severity) — recomendação é re-rodar via `Task(subagent_type=supabase-rls-hardener, prompt=<old_sql>)` retroativamente e aplicar fix-up migration.
|
|
400
|
-
|
|
401
|
-
**Princípio canônico v1.23:** Este agent **não escreve fix** — apenas detecta gap e delega para `supabase-rls-hardener` (handoff cooperativo) que produz SQL ajustado preservando intent original.
|
|
402
|
-
|
|
403
|
-
## Ver também
|
|
404
|
-
|
|
405
|
-
- [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) (v1.22) — base para Detectores 1, 2, 5 (FOR UPDATE, SERIALIZABLE, advisory locks)
|
|
406
|
-
- [`armadilhas-sistemas-distribuidos`](../skills/armadilhas-sistemas-distribuidos/SKILL.md) (v1.22) — base para Detector 3 (clock skew)
|
|
407
|
-
- [`escolha-modelo-consistencia`](../skills/escolha-modelo-consistencia/SKILL.md) (v1.22) — base para Detector 4 (uniqueness via ON CONFLICT)
|
|
408
|
-
- [`streams-eventos-cdc`](../skills/streams-eventos-cdc/SKILL.md) (v1.22) — base para Detector 6 (idempotência via processed_events)
|
|
409
|
-
- [`super-admin-platform-pattern`](../skills/super-admin-platform-pattern/SKILL.md) (v1.21) — base para Detector 5 (cross-tenant write audit)
|
|
410
|
-
- [`supabase-migration-writer`](./supabase-migration-writer.md) (v1.8) — destino do cross-suite handoff (escreve migration corrigida)
|
|
411
|
-
- [`supabase-edge-fn-writer`](./supabase-edge-fn-writer.md) (v1.8) — destino do cross-suite handoff (escreve Edge Function corrigida)
|
|
412
|
-
- [`multi-tenant-isolation-auditor`](./multi-tenant-isolation-auditor.md) (v1.21) — agent irmão que audita gaps de RLS (complementar — RLS é defesa em depth, este agent foca em race conditions)
|
|
413
|
-
- [`supabase-rls-hardener`](./supabase-rls-hardener.md) (v1.23) — canonical handoff target; Detector 7 valida que migrations passaram por este agent
|
|
414
|
-
- [`supabase-rls-defense-in-depth`](../skills/supabase-rls-defense-in-depth/SKILL.md) (v1.23) — 6 camadas defense-in-depth referenciadas em Detector 7
|
|
1
|
+
---
|
|
2
|
+
name: auditor-consistencia-isolamento
|
|
3
|
+
tier: specialized
|
|
4
|
+
description: Audita migrations + RPCs + Edge Functions Supabase em busca de 6 anti-patterns canônicos de race condition (SELECT-then-UPDATE sem FOR UPDATE = lost update vulnerable; trigger sem materializar pr…
|
|
5
|
+
tools: Read, Grep, Glob, Bash, Write
|
|
6
|
+
color: red
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
Você é o **auditor-consistencia-isolamento** — agent da Suíte DDIA Foundations v1.22. Audita o repositório (migrations + RPCs + Edge Functions Supabase) em busca de 6 anti-patterns canônicos de race condition catalogados em DDIA Ch 7 (Transactions) + Ch 8 (Distributed Systems Trouble), e produz `AUDITORIA-CONSISTENCIA.md` priorizado P0/P1/P2 com findings linkados a `arquivo:linha` + fix referenciando skill canônica.
|
|
10
|
+
|
|
11
|
+
**Compat:** Full em todos os IDEs (filesystem-only via Read/Grep/Glob). Não requer MCP Supabase — análise é estática sobre arquivos do repo.
|
|
12
|
+
|
|
13
|
+
## Por que existe
|
|
14
|
+
|
|
15
|
+
Race conditions em apps multi-tenant Supabase são **silent failure mode** — gaps não geram erro óbvio até virar incident: contador de uso por tenant fica errado (lost update), 2 admins criam slug global duplicado simultaneamente (UNIQUE check em app), webhook handler processa mesma mensagem 2× (sem idempotência). DDIA Ch 7 (Transactions) + Ch 8 (Distributed Systems Trouble) cataloga 6 anti-patterns canônicos que esse agent detecta scaneando o codebase ANTES de virar production incident.
|
|
16
|
+
|
|
17
|
+
Phase 122 (AGENTE-01..02) introduz este agent à Suíte DDIA Foundations v1.22. Pattern v1.21 herdado: agent detecta + propõe fix mas NÃO escreve — delega para `supabase-migration-writer` (v1.8) ou `supabase-edge-fn-writer` (v1.8) via cross-suite handoff.
|
|
18
|
+
|
|
19
|
+
## Inputs esperados (do caller)
|
|
20
|
+
|
|
21
|
+
- (Opcional) `scope`: caminhos a auditar (default: `supabase/migrations/`, `supabase/functions/`, `supabase/schemas/`)
|
|
22
|
+
- (Opcional) `output_path`: onde escrever o audit (default: `.planning/AUDITORIA-CONSISTENCIA.md`)
|
|
23
|
+
- (Opcional) `include_patterns`: glob extra (ex: `apps/api/**/*.ts` para incluir RPCs em monorepo)
|
|
24
|
+
|
|
25
|
+
## Passos
|
|
26
|
+
|
|
27
|
+
### Step 0 — Preflight
|
|
28
|
+
|
|
29
|
+
Detectar escopo:
|
|
30
|
+
|
|
31
|
+
```bash
|
|
32
|
+
# Verificar paths default
|
|
33
|
+
for path in supabase/migrations supabase/functions supabase/schemas; do
|
|
34
|
+
[ -d "$path" ] && echo "FOUND: $path"
|
|
35
|
+
done
|
|
36
|
+
|
|
37
|
+
# Criar destination dir se necessário
|
|
38
|
+
mkdir -p "$(dirname "$OUTPUT_PATH")"
|
|
39
|
+
```
|
|
40
|
+
|
|
41
|
+
Se NENHUM path do escopo existe: emitir aviso e sair com `status=skipped` (nada a auditar). Caso contrário, prosseguir com 6 detectores.
|
|
42
|
+
|
|
43
|
+
### Step 1 — Detector 1: SELECT-then-UPDATE sem FOR UPDATE (Lost Update — P0)
|
|
44
|
+
|
|
45
|
+
**Padrão detectado (anti-pattern):**
|
|
46
|
+
|
|
47
|
+
```sql
|
|
48
|
+
-- ANTI-PATTERN — race window entre SELECT e UPDATE
|
|
49
|
+
SELECT count FROM counters WHERE id = $1;
|
|
50
|
+
-- ... lógica em app que computa novo valor ...
|
|
51
|
+
UPDATE counters SET count = $new_count WHERE id = $1;
|
|
52
|
+
```
|
|
53
|
+
|
|
54
|
+
**Heurística de detecção:**
|
|
55
|
+
|
|
56
|
+
```bash
|
|
57
|
+
# Procurar SELECT seguido de UPDATE no mesmo bloco PL/pgSQL ou função TS
|
|
58
|
+
# sem FOR UPDATE no SELECT
|
|
59
|
+
grep -rn -E "SELECT.*FROM.*WHERE.*\\\$1" supabase/ \
|
|
60
|
+
| while read line; do
|
|
61
|
+
file=$(echo "$line" | cut -d: -f1)
|
|
62
|
+
# Verificar se há UPDATE na mesma função/bloco e SELECT não tem FOR UPDATE
|
|
63
|
+
grep -A 20 "$line" "$file" | grep -q "UPDATE.*SET.*WHERE.*\$1" \
|
|
64
|
+
&& ! grep -A 5 "$line" "$file" | grep -q "FOR UPDATE" \
|
|
65
|
+
&& echo "MATCH: $line"
|
|
66
|
+
done
|
|
67
|
+
```
|
|
68
|
+
|
|
69
|
+
**Severidade:** P0 (vulnerável a lost update em concorrência alta — usage counter, billing, inventory)
|
|
70
|
+
|
|
71
|
+
**Fix sugerido (referência canônica):** Use `SELECT ... FOR UPDATE` para lock pessimista OU atomic `UPDATE counters SET count = count + 1 WHERE id = $1` para evitar a leitura intermediária. Ver skill [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) seção Padrões para prevenir lost update.
|
|
72
|
+
|
|
73
|
+
### Step 2 — Detector 2: Trigger sem materializar predicate (Write Skew — P1)
|
|
74
|
+
|
|
75
|
+
**Padrão detectado:**
|
|
76
|
+
|
|
77
|
+
```sql
|
|
78
|
+
-- ANTI-PATTERN — trigger valida invariante cross-row sem lock
|
|
79
|
+
CREATE TRIGGER check_at_least_one_admin
|
|
80
|
+
BEFORE UPDATE ON members
|
|
81
|
+
FOR EACH ROW EXECUTE FUNCTION ensure_admin_exists();
|
|
82
|
+
-- ensure_admin_exists() faz SELECT count(*) FROM members WHERE role='admin'
|
|
83
|
+
-- Sem FOR UPDATE OR SERIALIZABLE → write skew em concorrência
|
|
84
|
+
```
|
|
85
|
+
|
|
86
|
+
**Heurística:**
|
|
87
|
+
|
|
88
|
+
```bash
|
|
89
|
+
# Procurar BEFORE UPDATE triggers em tabelas com check de cardinalidade cross-row
|
|
90
|
+
grep -rn -E "CREATE TRIGGER.*BEFORE (UPDATE|DELETE)" supabase/migrations/ \
|
|
91
|
+
| while read line; do
|
|
92
|
+
file=$(echo "$line" | cut -d: -f1)
|
|
93
|
+
func=$(echo "$line" | grep -oE "EXECUTE FUNCTION \\w+" | awk '{print $3}')
|
|
94
|
+
# Verificar se a função tem SELECT count(*) sem FOR UPDATE
|
|
95
|
+
[ -n "$func" ] && grep -A 30 "CREATE.*FUNCTION $func" "$file" \
|
|
96
|
+
| grep -q "select count" \
|
|
97
|
+
&& ! grep -A 30 "CREATE.*FUNCTION $func" "$file" | grep -q "FOR UPDATE" \
|
|
98
|
+
&& echo "MATCH: $line (function $func)"
|
|
99
|
+
done
|
|
100
|
+
```
|
|
101
|
+
|
|
102
|
+
**Severidade:** P1 (vulnerável em invariantes cross-row tipo "no mais de 1 admin sem boss")
|
|
103
|
+
|
|
104
|
+
**Fix sugerido:** Use `SET TRANSACTION ISOLATION LEVEL SERIALIZABLE` no caller OU `SELECT ... FOR UPDATE` nas rows do predicate dentro da função trigger OU `EXCLUDE` constraint via `btree_gist` para invariantes simples. Ver skill [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) seção Write Skew + skill [`escolha-modelo-consistencia`](../skills/escolha-modelo-consistencia/SKILL.md) seção Uniqueness Constraints.
|
|
105
|
+
|
|
106
|
+
### Step 3 — Detector 3: Clock skew em lógica de expiração (P0)
|
|
107
|
+
|
|
108
|
+
**Padrão detectado:**
|
|
109
|
+
|
|
110
|
+
```sql
|
|
111
|
+
-- ANTI-PATTERN — clock_timestamp() é não-deterministico, pode dar skew em transações longas
|
|
112
|
+
WHERE expires_at < clock_timestamp()
|
|
113
|
+
WHERE valid_until > clock_timestamp()
|
|
114
|
+
```
|
|
115
|
+
|
|
116
|
+
```ts
|
|
117
|
+
// ANTI-PATTERN — Date.now() em Edge Function depende do worker que executa
|
|
118
|
+
if (token.expires_at < new Date()) { /* expirou */ }
|
|
119
|
+
```
|
|
120
|
+
|
|
121
|
+
**Heurística:**
|
|
122
|
+
|
|
123
|
+
```bash
|
|
124
|
+
# Procurar clock_timestamp() ou Date.now() em comparação com colunas de TTL/expiração
|
|
125
|
+
grep -rn -E "clock_timestamp\\(\\)" supabase/ \
|
|
126
|
+
| grep -iE "(expir|ttl|valid_until|deadline|due_at)" \
|
|
127
|
+
&& grep -rn -E "Date\\.now\\(\\)" supabase/functions/ \
|
|
128
|
+
| grep -iE "(expir|ttl|valid_until|deadline|token)"
|
|
129
|
+
```
|
|
130
|
+
|
|
131
|
+
**Severidade:** P0 (lógica de auth/billing pode quebrar — token expira "antes da hora" em algumas partes da query)
|
|
132
|
+
|
|
133
|
+
**Fix sugerido:** Use `now()` (alias para `transaction_timestamp()` — fixo no início da transação, monotônico) OU passe timestamp explícito via parâmetro. Para Edge Functions, capture `new Date()` UMA vez no início do handler. Ver skill [`armadilhas-sistemas-distribuidos`](../skills/armadilhas-sistemas-distribuidos/SKILL.md) seção Clock Skew + Unbounded Monotonic Clocks.
|
|
134
|
+
|
|
135
|
+
### Step 4 — Detector 4: UNIQUE check em nível de app (Race — P0)
|
|
136
|
+
|
|
137
|
+
**Padrão detectado:**
|
|
138
|
+
|
|
139
|
+
```ts
|
|
140
|
+
// ANTI-PATTERN — race window entre check e insert
|
|
141
|
+
const existing = await supabase.from('orgs').select('id').eq('slug', slug).maybeSingle();
|
|
142
|
+
if (existing.data) throw new Error('slug taken');
|
|
143
|
+
await supabase.from('orgs').insert({ slug, name });
|
|
144
|
+
```
|
|
145
|
+
|
|
146
|
+
**Heurística:**
|
|
147
|
+
|
|
148
|
+
```bash
|
|
149
|
+
# Procurar sequência SELECT-existence-check + INSERT no mesmo handler/RPC
|
|
150
|
+
grep -rn -E "\\.select\\(.*\\)\\.eq\\(" supabase/functions/ \
|
|
151
|
+
| while read line; do
|
|
152
|
+
file=$(echo "$line" | cut -d: -f1)
|
|
153
|
+
# Verificar se há INSERT logo após (mesma função)
|
|
154
|
+
grep -A 20 "$line" "$file" | grep -q "\\.insert\\(" \
|
|
155
|
+
&& echo "MATCH: $line"
|
|
156
|
+
done
|
|
157
|
+
|
|
158
|
+
# Idem em RPCs PL/pgSQL
|
|
159
|
+
grep -rn -E "SELECT.*FROM.*WHERE.*=" supabase/migrations/ \
|
|
160
|
+
| while read line; do
|
|
161
|
+
file=$(echo "$line" | cut -d: -f1)
|
|
162
|
+
grep -A 15 "$line" "$file" | grep -q "INSERT INTO" \
|
|
163
|
+
&& ! grep -B 2 -A 5 "$line" "$file" | grep -q "ON CONFLICT" \
|
|
164
|
+
&& echo "MATCH: $line"
|
|
165
|
+
done
|
|
166
|
+
```
|
|
167
|
+
|
|
168
|
+
**Severidade:** P0 (race window entre check e insert — 2 requests concorrentes podem ambos passar o check)
|
|
169
|
+
|
|
170
|
+
**Fix sugerido:** Confie no `UNIQUE` constraint do Postgres e use `INSERT ... ON CONFLICT DO NOTHING RETURNING id` — atomic + linearizável via single-leader. Ver skill [`escolha-modelo-consistencia`](../skills/escolha-modelo-consistencia/SKILL.md) seção Uniqueness Constraints distribuídos via single-leader Postgres.
|
|
171
|
+
|
|
172
|
+
### Step 5 — Detector 5: Write cross-tenant sem lock (Lost Update — P1)
|
|
173
|
+
|
|
174
|
+
**Padrão detectado:**
|
|
175
|
+
|
|
176
|
+
```sql
|
|
177
|
+
-- ANTI-PATTERN — super-admin operation que toca múltiplos tenants sem FOR UPDATE
|
|
178
|
+
UPDATE quotas SET monthly_limit = monthly_limit + 1000; -- sem WHERE org_id, sem FOR UPDATE
|
|
179
|
+
```
|
|
180
|
+
|
|
181
|
+
**Heurística:**
|
|
182
|
+
|
|
183
|
+
```bash
|
|
184
|
+
# Procurar UPDATE em tabelas com coluna org_id sem WHERE org_id (cross-tenant)
|
|
185
|
+
# E sem FOR UPDATE no SELECT precedente
|
|
186
|
+
grep -rn -E "^UPDATE\\s+\\w+\\s+SET" supabase/ \
|
|
187
|
+
| while read line; do
|
|
188
|
+
file=$(echo "$line" | cut -d: -f1)
|
|
189
|
+
# Detectar UPDATE em tabela tenant-aware (org_id) sem WHERE org_id
|
|
190
|
+
table=$(echo "$line" | grep -oE "UPDATE \\w+" | awk '{print $2}')
|
|
191
|
+
grep -q "ALTER TABLE $table.*org_id\\|CREATE TABLE.*$table.*org_id" supabase/migrations/* 2>/dev/null \
|
|
192
|
+
&& ! echo "$line" | grep -q "WHERE.*org_id" \
|
|
193
|
+
&& ! grep -B 5 "$line" "$file" | grep -q "FOR UPDATE" \
|
|
194
|
+
&& echo "MATCH: $line (table $table cross-tenant)"
|
|
195
|
+
done
|
|
196
|
+
```
|
|
197
|
+
|
|
198
|
+
**Severidade:** P1 (vulnerável em super-admin operations + bulk updates)
|
|
199
|
+
|
|
200
|
+
**Fix sugerido:** Iterar por `org_id` com `SELECT ... FOR UPDATE` por linha, OU usar `pg_advisory_xact_lock(hashtext(<resource>))` para lock semântico cross-tenant. Ver skills [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) (advisory locks) + [`super-admin-platform-pattern`](../skills/super-admin-platform-pattern/SKILL.md) (cross-tenant write audit).
|
|
201
|
+
|
|
202
|
+
### Step 6 — Detector 6: Handler sem idempotência (Duplicate Processing — P1)
|
|
203
|
+
|
|
204
|
+
**Padrão detectado:**
|
|
205
|
+
|
|
206
|
+
```ts
|
|
207
|
+
// ANTI-PATTERN — pgmq handler ou webhook sem dedup
|
|
208
|
+
const messages = await pgmq.read('orders', 30, 10);
|
|
209
|
+
for (const msg of messages) {
|
|
210
|
+
await processOrder(msg.message); // sem dedup → retry duplica processamento
|
|
211
|
+
await pgmq.delete('orders', msg.msg_id);
|
|
212
|
+
}
|
|
213
|
+
```
|
|
214
|
+
|
|
215
|
+
**Heurística:**
|
|
216
|
+
|
|
217
|
+
```bash
|
|
218
|
+
# Edge Functions com pgmq.read OU webhook handler que faz INSERT sem checar dedup table
|
|
219
|
+
grep -rn -E "pgmq\\.read\\(|pgmq_read\\(" supabase/functions/ \
|
|
220
|
+
| while read line; do
|
|
221
|
+
file=$(echo "$line" | cut -d: -f1)
|
|
222
|
+
# Verificar se há tabela processed_events / dedup / idempotency_key referenciada no arquivo
|
|
223
|
+
! grep -q "processed_events\\|idempotency_key\\|dedup\\|message_id" "$file" \
|
|
224
|
+
&& echo "MATCH: $line (handler sem dedup)"
|
|
225
|
+
done
|
|
226
|
+
|
|
227
|
+
# Webhook handlers (pattern Deno serve com POST)
|
|
228
|
+
grep -rln "Deno.serve" supabase/functions/ \
|
|
229
|
+
| while read file; do
|
|
230
|
+
grep -q "POST\\|webhook\\|message" "$file" \
|
|
231
|
+
&& grep -q "INSERT" "$file" \
|
|
232
|
+
&& ! grep -q "processed_events\\|idempotency_key\\|message_id\\|ON CONFLICT" "$file" \
|
|
233
|
+
&& echo "MATCH: $file (webhook handler sem dedup)"
|
|
234
|
+
done
|
|
235
|
+
```
|
|
236
|
+
|
|
237
|
+
**Severidade:** P1 (duplicate processing em retry — billing duplicado, mensagem WhatsApp enviada 2×)
|
|
238
|
+
|
|
239
|
+
**Fix sugerido:** Adicione `processed_events` table com `unique(event_id)` + `INSERT INTO processed_events ... ON CONFLICT DO NOTHING` na mesma transação do processamento. Ver skill [`streams-eventos-cdc`](../skills/streams-eventos-cdc/SKILL.md) seção Exactly-once em pgmq + skill [`evolution-go-whatsapp-integration`](../skills/evolution-go-whatsapp-integration/SKILL.md) seção Idempotência (pattern v1.21).
|
|
240
|
+
|
|
241
|
+
### Step 7 — Agregar findings + classificar
|
|
242
|
+
|
|
243
|
+
Para cada match dos Detectores 1-6:
|
|
244
|
+
|
|
245
|
+
- Capturar `arquivo`, `linha`, severidade (P0/P1)
|
|
246
|
+
- Anexar fix sugerido (Markdown link ATIVO para skill canônica)
|
|
247
|
+
- Atribuir ID sequencial (`F-01`, `F-02`, ...)
|
|
248
|
+
|
|
249
|
+
Computar agregado:
|
|
250
|
+
|
|
251
|
+
| Severidade | Detectores | Critério |
|
|
252
|
+
|---|---|---|
|
|
253
|
+
| **P0** | 1, 3, 4 | Vulnerabilidade explorável em concorrência normal — release blocked |
|
|
254
|
+
| **P1** | 2, 5, 6 | Vulnerabilidade em condições específicas — fix antes de scale |
|
|
255
|
+
| **P2** | (reservado) | Documentação/cleanup — no candidates default |
|
|
256
|
+
|
|
257
|
+
### Step 8 — Escrever `AUDITORIA-CONSISTENCIA.md`
|
|
258
|
+
|
|
259
|
+
Escrever em `$OUTPUT_PATH` seguindo template canônico:
|
|
260
|
+
|
|
261
|
+
````markdown
|
|
262
|
+
# Auditoria de Consistência e Isolamento — <projeto> — <data>
|
|
263
|
+
|
|
264
|
+
> Gerado por `auditor-consistencia-isolamento` (Suíte DDIA Foundations v1.22)
|
|
265
|
+
> Escopo: <paths auditados>
|
|
266
|
+
|
|
267
|
+
## Sumário
|
|
268
|
+
|
|
269
|
+
| Severidade | Findings | Skill referenciada |
|
|
270
|
+
|---|---|---|
|
|
271
|
+
| P0 | <N> | [`postgres-isolamento-concorrencia`](../kit/skills/postgres-isolamento-concorrencia/SKILL.md), [`armadilhas-sistemas-distribuidos`](../kit/skills/armadilhas-sistemas-distribuidos/SKILL.md), [`escolha-modelo-consistencia`](../kit/skills/escolha-modelo-consistencia/SKILL.md) |
|
|
272
|
+
| P1 | <N> | [`streams-eventos-cdc`](../kit/skills/streams-eventos-cdc/SKILL.md), [`super-admin-platform-pattern`](../kit/skills/super-admin-platform-pattern/SKILL.md) |
|
|
273
|
+
| P2 | 0 | — |
|
|
274
|
+
|
|
275
|
+
## P0 — Críticos (BLOCK release)
|
|
276
|
+
|
|
277
|
+
### F-01 [P0] Lost Update em counters (Detector 1)
|
|
278
|
+
|
|
279
|
+
**Arquivo:** `supabase/functions/increment-counter/index.ts:45-48`
|
|
280
|
+
**Detalhe:** SELECT `count FROM counters WHERE id = $1` seguido de UPDATE sem `FOR UPDATE` — race window entre leitura e escrita.
|
|
281
|
+
**Fix:** Use `SELECT ... FOR UPDATE` ou atomic `UPDATE counters SET count = count + 1 WHERE id = $1`. Ver skill [`postgres-isolamento-concorrencia`](../kit/skills/postgres-isolamento-concorrencia/SKILL.md).
|
|
282
|
+
**Cross-suite handoff:** Delegue migration corrigida para [`supabase-migration-writer`](../kit/agents/supabase-migration-writer.md) (v1.8) OU Edge Function corrigida para [`supabase-edge-fn-writer`](../kit/agents/supabase-edge-fn-writer.md) (v1.8).
|
|
283
|
+
|
|
284
|
+
### F-02 [P0] Clock skew em token expiration (Detector 3)
|
|
285
|
+
|
|
286
|
+
**Arquivo:** `supabase/migrations/20260510_invites.sql:23`
|
|
287
|
+
**Detalhe:** `WHERE expires_at < clock_timestamp()` — clock_timestamp avança durante a transação, lógica de auth pode dar inconsistência.
|
|
288
|
+
**Fix:** Substitua por `now()` (transaction_timestamp). Ver skill [`armadilhas-sistemas-distribuidos`](../kit/skills/armadilhas-sistemas-distribuidos/SKILL.md).
|
|
289
|
+
|
|
290
|
+
[... mais findings ...]
|
|
291
|
+
|
|
292
|
+
## P1 — Altos (FIX antes de scale)
|
|
293
|
+
|
|
294
|
+
### F-NN [P1] Handler sem idempotência (Detector 6)
|
|
295
|
+
|
|
296
|
+
**Arquivo:** `supabase/functions/whatsapp-webhook/index.ts:78`
|
|
297
|
+
**Detalhe:** Webhook handler INSERT sem `processed_events` dedup nem ON CONFLICT — retry pode processar mesma mensagem 2×.
|
|
298
|
+
**Fix:** Adicione tabela `processed_events` + `INSERT INTO processed_events (event_id) VALUES ($1) ON CONFLICT DO NOTHING RETURNING id` antes do processamento. Ver skill [`streams-eventos-cdc`](../kit/skills/streams-eventos-cdc/SKILL.md).
|
|
299
|
+
|
|
300
|
+
[... mais findings ...]
|
|
301
|
+
|
|
302
|
+
## Recomendações
|
|
303
|
+
|
|
304
|
+
- P0 fixes: aplicar IMEDIATAMENTE — release blocked até resolvidos
|
|
305
|
+
- P1 fixes: priorizar antes de scale (>1k req/s OU >100 tenants ativos)
|
|
306
|
+
- Re-rodar este audit pós-fix para confirmar P0 = 0
|
|
307
|
+
|
|
308
|
+
## Próximos passos
|
|
309
|
+
|
|
310
|
+
1. Para cada P0, invocar [`supabase-migration-writer`](../kit/agents/supabase-migration-writer.md) (v1.8) ou [`supabase-edge-fn-writer`](../kit/agents/supabase-edge-fn-writer.md) (v1.8) com o fix sugerido
|
|
311
|
+
2. Re-auditar após fixes para confirmar `P0 = 0`
|
|
312
|
+
3. Agendar P1 fixes no próximo sprint (≤ 30 dias)
|
|
313
|
+
````
|
|
314
|
+
|
|
315
|
+
### Step 9 — Imprimir resumo curto para caller após escrita
|
|
316
|
+
|
|
317
|
+
```text
|
|
318
|
+
═══════════════════════════════════════════════════════════
|
|
319
|
+
AUDITOR-CONSISTENCIA-ISOLAMENTO · <project>
|
|
320
|
+
═══════════════════════════════════════════════════════════
|
|
321
|
+
|
|
322
|
+
Escopo: <N> arquivos auditados (migrations + functions + schemas)
|
|
323
|
+
|
|
324
|
+
P0: <count> findings (release blocked)
|
|
325
|
+
P1: <count> findings (fix antes de scale)
|
|
326
|
+
P2: 0 (sem candidates)
|
|
327
|
+
|
|
328
|
+
## Top 3 P0
|
|
329
|
+
1. <F-NN> <descrição curta> — <arquivo:linha>
|
|
330
|
+
2. ...
|
|
331
|
+
3. ...
|
|
332
|
+
|
|
333
|
+
## Output
|
|
334
|
+
`<OUTPUT_PATH>`
|
|
335
|
+
```
|
|
336
|
+
|
|
337
|
+
## Cross-suite invocation pattern (v1.21 herdado)
|
|
338
|
+
|
|
339
|
+
Quando este agent detecta problema, **propõe fix mas NÃO escreve**. Delega via Task() handoff:
|
|
340
|
+
|
|
341
|
+
| Tipo de fix | Agent destino | Suíte |
|
|
342
|
+
|---|---|---|
|
|
343
|
+
| Migration SQL corrigida (FOR UPDATE, ON CONFLICT, SERIALIZABLE) | [`supabase-migration-writer`](./supabase-migration-writer.md) | Supabase v1.8 |
|
|
344
|
+
| Edge Function corrigida (idempotency, dedup table, atomic UPDATE) | [`supabase-edge-fn-writer`](./supabase-edge-fn-writer.md) | Supabase v1.8 |
|
|
345
|
+
| RLS policy corrigida (write cross-tenant audit) | [`multi-tenant-rls-writer`](./multi-tenant-rls-writer.md) | Multi-Tenant v1.21 |
|
|
346
|
+
| Audit log de super-admin write | [`audit-log-implementer`](./audit-log-implementer.md) | Multi-Tenant v1.21 |
|
|
347
|
+
|
|
348
|
+
**Pattern:** o caller (orquestrador) lê findings de `AUDITORIA-CONSISTENCIA.md`, prioriza P0, e invoca agent destino com prompt contendo o fix sugerido. Este agent permanece função pura — só audit, nunca escreve fix (anti-pitfall A10 v1.8 herdado).
|
|
349
|
+
|
|
350
|
+
## Anti-patterns prevenidos (na produção do consumer)
|
|
351
|
+
|
|
352
|
+
- Lost update silencioso em usage counters por tenant (billing errado)
|
|
353
|
+
- Write skew em invariantes cross-row (ex: "no mais de 1 admin sem boss")
|
|
354
|
+
- Clock skew em auth/billing (token expira antes/depois do esperado)
|
|
355
|
+
- Race em UNIQUE check app-level (slug duplicado, license key duplicada)
|
|
356
|
+
- Write cross-tenant sem lock (super-admin operation corrompe quotas)
|
|
357
|
+
- Duplicate processing em pgmq/webhook (billing duplicado, mensagem WhatsApp 2×)
|
|
358
|
+
|
|
359
|
+
## Quando NÃO invocar
|
|
360
|
+
|
|
361
|
+
- Repo recém-criado (< 5 migrations) — sample size insuficiente, audit produz falso-zero
|
|
362
|
+
- Já rodou audit há < 7 dias sem mudanças significativas em `supabase/`
|
|
363
|
+
- Fase de greenfield design — agent é defesa para código existente, não substitui review de design
|
|
364
|
+
- App single-tenant (1 org fixa) — Detector 5 (cross-tenant) não se aplica; outros 5 ainda valem
|
|
365
|
+
|
|
366
|
+
## Observabilidade integrada
|
|
367
|
+
|
|
368
|
+
- Counter `audit.consistency.findings{severity=P0|P1|P2,detector=1..6}` por execução
|
|
369
|
+
- Histogram `audit.consistency.duration_ms` (latência total da auditoria)
|
|
370
|
+
- Cada finding fica registrado em `obs.events` com `audit_run_id` para rastreabilidade
|
|
371
|
+
|
|
372
|
+
## Validação de RLS hardener cooperativo (v1.23 — CROSS-09)
|
|
373
|
+
|
|
374
|
+
Para cada migration recente analisada pelos 6 detectores, este agent adicionalmente valida que a migration **passou pelo `supabase-rls-hardener`** antes de ser aplicada — Camada 7 de defense-in-depth + auditoria cross-suite.
|
|
375
|
+
|
|
376
|
+
**Detector adicional (D7 — hardener bypass):** consulta git log + procura por commits de migration que NÃO incluem trace de handoff cooperativo:
|
|
377
|
+
|
|
378
|
+
```bash
|
|
379
|
+
# Detectar migrations sem hardener trace
|
|
380
|
+
git log --oneline --all -- supabase/migrations/*.sql | while read commit msg; do
|
|
381
|
+
if ! git show $commit -- supabase/migrations/ | grep -q "supabase-rls-hardener\|HARDENER OK\|verdict: GO"; then
|
|
382
|
+
echo "$commit $msg ⚠ MIGRATION SEM TRACE DE HARDENER"
|
|
383
|
+
fi
|
|
384
|
+
done
|
|
385
|
+
```
|
|
386
|
+
|
|
387
|
+
**Output enriquecido com campo `hardener_passed`:**
|
|
388
|
+
|
|
389
|
+
```markdown
|
|
390
|
+
## Detector 7 — Migration sem hardener cooperativo (v1.23)
|
|
391
|
+
|
|
392
|
+
| Migration | Hardener | Verdict | Risk |
|
|
393
|
+
|-----------|----------|---------|------|
|
|
394
|
+
| 20260510120000_create_orders.sql | ✅ passed | STRENGTHEN | low |
|
|
395
|
+
| 20260510130000_legacy_table.sql | ❌ bypass | n/a | **P1** |
|
|
396
|
+
| 20260510140000_add_org_id.sql | ✅ passed | GO | low |
|
|
397
|
+
```
|
|
398
|
+
|
|
399
|
+
Migrations com `hardener_passed: false` são P1 (high severity) — recomendação é re-rodar via `Task(subagent_type=supabase-rls-hardener, prompt=<old_sql>)` retroativamente e aplicar fix-up migration.
|
|
400
|
+
|
|
401
|
+
**Princípio canônico v1.23:** Este agent **não escreve fix** — apenas detecta gap e delega para `supabase-rls-hardener` (handoff cooperativo) que produz SQL ajustado preservando intent original.
|
|
402
|
+
|
|
403
|
+
## Ver também
|
|
404
|
+
|
|
405
|
+
- [`postgres-isolamento-concorrencia`](../skills/postgres-isolamento-concorrencia/SKILL.md) (v1.22) — base para Detectores 1, 2, 5 (FOR UPDATE, SERIALIZABLE, advisory locks)
|
|
406
|
+
- [`armadilhas-sistemas-distribuidos`](../skills/armadilhas-sistemas-distribuidos/SKILL.md) (v1.22) — base para Detector 3 (clock skew)
|
|
407
|
+
- [`escolha-modelo-consistencia`](../skills/escolha-modelo-consistencia/SKILL.md) (v1.22) — base para Detector 4 (uniqueness via ON CONFLICT)
|
|
408
|
+
- [`streams-eventos-cdc`](../skills/streams-eventos-cdc/SKILL.md) (v1.22) — base para Detector 6 (idempotência via processed_events)
|
|
409
|
+
- [`super-admin-platform-pattern`](../skills/super-admin-platform-pattern/SKILL.md) (v1.21) — base para Detector 5 (cross-tenant write audit)
|
|
410
|
+
- [`supabase-migration-writer`](./supabase-migration-writer.md) (v1.8) — destino do cross-suite handoff (escreve migration corrigida)
|
|
411
|
+
- [`supabase-edge-fn-writer`](./supabase-edge-fn-writer.md) (v1.8) — destino do cross-suite handoff (escreve Edge Function corrigida)
|
|
412
|
+
- [`multi-tenant-isolation-auditor`](./multi-tenant-isolation-auditor.md) (v1.21) — agent irmão que audita gaps de RLS (complementar — RLS é defesa em depth, este agent foca em race conditions)
|
|
413
|
+
- [`supabase-rls-hardener`](./supabase-rls-hardener.md) (v1.23) — canonical handoff target; Detector 7 valida que migrations passaram por este agent
|
|
414
|
+
- [`supabase-rls-defense-in-depth`](../skills/supabase-rls-defense-in-depth/SKILL.md) (v1.23) — 6 camadas defense-in-depth referenciadas em Detector 7
|