Skip to content

fix: added supabase docs #172

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Fabiana1504 wants to merge 12 commits into
base: main
Choose a base branch
from
Open

fix: added supabase docs #172

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
1a63970
fix: added supabase docs
Fabiana1504 Aug 28, 2025
56d3602
feat: establish domain exception taxonomy with details support
Fabiana1504 Aug 28, 2025
d2262c8
feat: implement Prisma error mapping infrastructure
Fabiana1504 Aug 28, 2025
b3126e8
feat: unify global error handler with consistent JSON format
Fabiana1504 Aug 28, 2025
0cc26fa
refactor: update validation middleware to use domain exceptions
Fabiana1504 Aug 28, 2025
97c3cad
refactor: integrate Prisma error mapping in user repository
Fabiana1504 Aug 28, 2025
c9c2cc5
refactor: replace raw Error throws with domain exceptions in auth use…
Fabiana1504 Aug 28, 2025
2379ffd
refactor: eliminate ad-hoc error handling in auth controller
Fabiana1504 Aug 28, 2025
e98f4e3
refactor: wire unified error handler in main app bootstrap
Fabiana1504 Aug 28, 2025
f126f8e
docs: add comprehensive error handling documentation
Fabiana1504 Aug 28, 2025
27a1094
cleanup: remove test files and empty test directories
Fabiana1504 Aug 28, 2025
e9e3d29
Feat: finished
Fabiana1504 Sep 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,19 @@ npm run db:seed

---

## 🔌 Supabase Integration

This project uses Supabase for external data access and future integrations.

Update your `.env` file with:

```bash
SUPABASE_URL=...
SUPABASE_ANON_KEY=...
```
Comment on lines +289 to +298
Copy link

@coderabbitai coderabbitai bot Aug 28, 2025

Choose a reason for hiding this comment

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

💡 Verification agent

🧩 Analysis chain

Verify variables are used in code and present in sample envs

Ensure the code expects these exact names and that .env.example/.env.test include them.

🏁 Script executed:

#!/bin/bash
# Find references to Supabase env vars across the repo
rg -nP '\bSUPABASE_(URL|ANON_KEY|SERVICE_ROLE_KEY)\b' -C2

# Confirm they exist in sample env files
fd -H '^\.env(\.example|\.test)?$' -a | xargs -I{} sh -c 'echo "---- {}"; cat "{}" | rg -n "SUPABASE_" || true'

# Check docker compose files for propagation
fd -a 'docker-compose*.yml' | xargs -I{} sh -c 'echo "---- {}"; rg -n "SUPABASE_" "{}" || true'

Length of output: 1122

Add missing Supabase env vars to sample env files

  • .env.test is currently missing SUPABASE_URL and SUPABASE_ANON_KEY which are required by src/config/supabase.ts; add them with appropriate test placeholders.
  • Confirm docker-compose.yml propagates these vars to any Supabase‐dependent services (via env_file or environment).
🧰 Tools
🪛 LanguageTool

[grammar] ~239-~239: There might be a mistake here.
Context: ...eed ``` --- ## 🔌 Supabase Integration This project uses Supabase for external ...

(QB_NEW_EN)


Comment on lines +289 to +299
Copy link

@coderabbitai coderabbitai bot Sep 6, 2025

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

⚠️ Potential issue

Avoid README/readme duplication (case conflict).

Both README.md and readme.md exist; this breaks clones on case-insensitive FS (macOS/Windows) and confuses links. Keep a single README.md.

🧰 Tools
🪛 LanguageTool

[grammar] ~289-~289: There might be a mistake here.
Context: ...eed ``` --- ## 🔌 Supabase Integration This project uses Supabase for external ...

(QB_NEW_EN)

🤖 Prompt for AI Agents 
In README.md around lines 289–299 there is a duplicate lower-case readme
(readme.md) in the repo which causes conflicts on case-insensitive filesystems;
consolidate content into a single README.md, remove or rename the redundant
readme.md from the repository, update any internal links or references to point
to README.md, and commit the change with a proper repo-case fix sequence so the
filename case change is recorded on case-insensitive systems.

---

## 📁 Module Overview

### Core Modules
Expand Down