fix: correct SQL syntax errors in schema.sql

[yogeshkumawat2027]

🐛 Bug Fix

Fixed 5 SQL syntax errors in supabase/schemas/schema.sql that were preventing database setup.

📝 Changes Made

• therapist table: Removed trailing comma after license_number
• observation_master table: Removed trailing comma after applicable_therapies
• activity_master table: Removed trailing comma after applicable_therapies
• daily_activities table: Removed trailing comma after days_of_week
• daily_activity_logs table: Added missing comma after activity_items

✅ Testing

• All 14 database tables now create successfully without errors

Fixes #[issue_number]

Summary by CodeRabbit

• Chores
  • Resolved minor database schema formatting inconsistencies to improve code clarity and maintainability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

This pull request corrects SQL syntax in the schema file by repositioning trailing commas in table column definitions for the therapist and daily_activity_logs tables. No functional changes or data structure alterations are introduced.

Changes

Cohort / File(s)	Summary
SQL Schema Syntax Cleanup supabase/schemas/schema.sql	Removed extraneous trailing comma after license_number TEXT in therapist table; repositioned trailing comma in daily_activity_logs table from end of activity_items JSONB NOT NULL line to properly precede the next column definition.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Syntax-only corrections with no logic or structural changes
• Single file modification affecting only punctuation

Poem

🐰 A comma here, a comma there,
The schema's fixed with punctuation care,
No logic bent, no data lost,
Just syntax clean—a worthwhile cost!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: correct SQL syntax errors in schema.sql' directly and accurately summarizes the main change—fixing SQL syntax errors in the schema file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a2d6eb and 4d9ae96.

📒 Files selected for processing (1)

• supabase/schemas/schema.sql (4 hunks)

🔇 Additional comments (5)

supabase/schemas/schema.sql (5)

36-36: Trailing comma removed correctly from therapist table.

The removal of the trailing comma after license_number TEXT is correct since it's the last column in the table.

---

123-123: Trailing comma removed correctly from observation_master table.

The removal of the trailing comma after applicable_therapies UUID[] NOT NULL is correct since it's the last column in the table.

---

139-139: Trailing comma removed correctly from activity_master table.

The removal of the trailing comma after applicable_therapies UUID[] NOT NULL is correct since it's the last column in the table.

---

153-153: Trailing comma removed correctly from daily_activities table.

The removal of the trailing comma after days_of_week INT2[] is correct since it's the last column in the table.

---

160-161: Comma placement corrected in daily_activity_logs table.

The comma is properly placed after activity_items JSONB NOT NULL, with patient_id as the final column.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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