Skip to content

[Plan 001] Establish an application verification baseline - #10

Merged
prof-ramos merged 1 commit into
mainfrom
agent/001-verification-baseline
Aug 3, 2026
Merged

[Plan 001] Establish an application verification baseline#10
prof-ramos merged 1 commit into
mainfrom
agent/001-verification-baseline

Conversation

@prof-ramos

@prof-ramos prof-ramos commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Implements Plan 001 and closes #5 when merged.

What changed

  • Pins the repository to pnpm 10.34.5 and adds Vitest configuration tests for Supabase environment handling.
  • Makes pnpm check run lint, typecheck, app tests, and the production build.
  • Adds a safe .env.example, local database CI coverage, and onboarding documentation.
  • Versions the five implementation plans under plans/ so the follow-up PRs share the reviewed execution context.

Why

The repository had no application test runner or CI gate. The following authorization and financial migrations need a repeatable app and local-database verification path.

Checks

  • pnpm check
  • pnpm supabase:test against local Supabase: 22 assertions passed
  • git diff --check

Review notes

This is the base PR for the remaining four stacked PRs. It is intentionally a draft and should not be merged automatically.

Summary by CodeRabbit

  • Novos recursos

    • Adicionado suporte a testes automatizados da aplicação.
    • Incluídos exemplos de configuração das variáveis públicas do Supabase.
    • Criado pipeline de integração contínua para validar a aplicação e o banco local.
  • Documentação

    • Atualizadas as instruções de configuração local, scripts disponíveis e status de cobertura.
    • Documentados planos de segurança, auditoria, pedidos, eventos e ledger financeiro.
  • Melhorias

    • O comando de verificação agora inclui testes, lint, tipagem e build.
    • Fixada a versão recomendada do pnpm para execução consistente.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 22601ede-781a-4921-b106-c959c7bb97e0

📥 Commits

Reviewing files that changed from the base of the PR and between 216e687 and 3ed8788.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml, !pnpm-lock.yaml
📒 Files selected for processing (14)
  • .env.example
  • .github/workflows/ci.yml
  • .gitignore
  • CLAUDE.md
  • README.md
  • package.json
  • plans/001-establish-verification-baseline.md
  • plans/002-restrict-audit-log-writes.md
  • plans/003-remove-order-pii-from-general-surface.md
  • plans/004-enforce-event-lifecycle.md
  • plans/005-enforce-ledger-invariants.md
  • plans/README.md
  • src/lib/supabase/config.test.ts
  • vitest.config.ts

Walkthrough

A mudança estabelece uma linha de base de verificação com Vitest, scripts atualizados, variáveis de ambiente de exemplo e CI para a aplicação e o Supabase local. Também adiciona documentação para os planos de segurança do banco e atualiza o setup do projeto.

Changes

Linha de base de verificação

Layer / File(s) Summary
Testes e comandos da aplicação
vitest.config.ts, src/lib/supabase/config.test.ts, package.json
Configura o Vitest, adiciona testes para a configuração do Supabase, fixa o pnpm e inclui os scripts test, test:watch e check.
Ambiente e workflow de CI
.env.example, .gitignore, .github/workflows/ci.yml
Adiciona variáveis públicas de ambiente e dois jobs de CI. O segundo job inicia e encerra o Supabase local.
Documentação da verificação
README.md, CLAUDE.md, plans/001-establish-verification-baseline.md
Documenta o setup local, os comandos de teste, os gates de CI e o plano de verificação.

Planos de segurança do banco

Layer / File(s) Summary
Planos de acesso e auditoria
plans/002-restrict-audit-log-writes.md, plans/003-remove-order-pii-from-general-surface.md
Define restrições planejadas para escritas em audit_logs e leituras de PII em orders.
Planos de ciclo de vida e ledger
plans/004-enforce-event-lifecycle.md, plans/005-enforce-ledger-invariants.md, plans/README.md
Define controles planejados para transições de eventos, histórico de status e invariantes do ledger. Documenta a ordem e o estado dos planos.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/001-verification-baseline

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@prof-ramos
prof-ramos marked this pull request as ready for review August 3, 2026 08:43
@prof-ramos
prof-ramos merged commit 9207929 into main Aug 3, 2026
2 of 3 checks passed
@prof-ramos
prof-ramos deleted the agent/001-verification-baseline branch August 3, 2026 09:18
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.

[Plan 001] Establish an application verification baseline

1 participant