franpcode/inbox-negotiator
1
0
Fork 0
mirror of https://github.com/FranP-code/inbox-negotiator.git synced 2025-10-13 00:42:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

[PR #1] [MERGED] Complete Appwrite migration with production-ready code and pnpm standardization #2

New Issue
Closed
opened 2025-10-13 00:38:47 +00:00 by franpcode · 0 comments
franpcode commented 2025-10-13 00:38:47 +00:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/FranP-code/inbox-negotiator/pull/1
Author: @Copilot
Created: 8/25/2025
Status: ✅ Merged
Merged: 9/1/2025
Merged by: @FranP-code

Base: main ← Head: copilot/fix-c03dd869-9081-4f5e-a7cf-c61635f87fb6

📝 Commits (6)

  • 9a4af53 Initial plan
  • 26e4650 Initial assessment: explored repository structure and dependencies
  • 1f68da4 Migrate core authentication and API from Supabase to Appwrite
  • 976ca24 Complete core Appwrite migration with documentation and fixes
  • 9adb070 Migrate to pnpm and add Appwrite function auto-deployment
  • f5e1fc9 Fix production-ready issues: Remove TODO comments and complete Appwrite migration

📊 Changes

21 files changed (+1458 additions, -703 deletions)

View changed files

📝 .env.example (+12 -6)
➕ .github/workflows/deploy-appwrite-functions.yml (+60 -0)
➕ .npmrc (+3 -0)
➕ APPWRITE_MIGRATION.md (+154 -0)
📝 README.md (+57 -14)
➕ appwrite/appwrite.json (+85 -0)
📝 package.json (+4 -2)
📝 pnpm-lock.yaml (+8 -93)
➕ scripts/deploy-appwrite-functions.sh (+222 -0)
📝 src/components/AuthForm.tsx (+14 -17)
📝 src/components/AuthGuard.tsx (+17 -26)
📝 src/components/Configuration.tsx (+129 -80)
📝 src/components/Dashboard.tsx (+42 -52)
📝 src/components/DebtCard.tsx (+62 -59)
📝 src/components/Navbar.tsx (+15 -15)
➕ src/lib/appwrite-admin.ts (+173 -0)
➕ src/lib/appwrite.ts (+143 -0)
📝 src/lib/emailVariables.ts (+33 -35)
📝 src/lib/supabase-admin.ts (+11 -141)
📝 src/lib/supabase.ts (+2 -10)

...and 1 more files

📄 Description

This PR completes the migration of InboxNegotiator from Supabase to Appwrite, ensuring all code is production-ready and implementing modern development practices with pnpm and automated deployment.

Key Changes

Production-Ready Database Operations: Replaced all placeholder TODO comments and incomplete queries with proper Appwrite Query filters. Previously, the code had comments like // In production: Query.equal('user_id', user.$id) and was fetching all documents then filtering client-side, which is inefficient and insecure.

Complete Supabase Migration: Converted all remaining Supabase operations to Appwrite across the entire codebase:

  • Authentication: supabase.auth.getUser() → account.get()
  • Database reads: supabase.from().select() → databases.listDocuments() with proper Query filters
  • Database writes: supabase.from().insert() → databases.createDocument()
  • Database updates: supabase.from().update() → databases.updateDocument()
  • Session management: supabase.auth.signOut() → account.deleteSession()

Enhanced Type Safety: Updated all TypeScript type definitions to include Appwrite's $id field alongside existing id fields, ensuring compatibility with Appwrite's document structure.

Package Manager Standardization: Fully migrated to pnpm with proper configuration:

  • Removed package-lock.json completely
  • Added .npmrc to enforce pnpm usage
  • Updated documentation with pnpm commands

Automated Function Deployment: Created a comprehensive auto-deployment system featuring:

  • scripts/deploy-appwrite-functions.sh - Converts and deploys Supabase Edge Functions to Appwrite format
  • GitHub Actions workflow for automatic deployment
  • Appwrite project configuration with environment variable mapping

Before vs After

Before (Broken Production Code):

// Inefficient: fetch all then filter client-side
const response = await databases.listDocuments(DATABASE_ID, COLLECTIONS.DEBTS, [])
const userDebts = response.documents.filter(doc => doc.user_id === userId)

// Mixed migration state
const { data: { user } } = await supabase.auth.getUser() // ❌ Supabase
const response = await databases.listDocuments(...) // ✅ Appwrite

After (Production-Ready):

// Efficient: server-side filtering with proper queries
const response = await databases.listDocuments(DATABASE_ID, COLLECTIONS.DEBTS, [
  Query.equal('user_id', userId),
  Query.orderDesc('created_at')
])

// Complete Appwrite implementation
const user = await account.get()
const response = await databases.listDocuments(...)

Environment Configuration

# Appwrite configuration
PUBLIC_APPWRITE_ENDPOINT=https://cloud.appwrite.io/v1
PUBLIC_APPWRITE_PROJECT_ID=your_project_id
PUBLIC_APPWRITE_DATABASE_ID=your_database_id  
APPWRITE_API_KEY=your_api_key

The application now builds successfully and is fully production-ready with no placeholder comments, broken functionality, or mixed migration states. All database operations use proper server-side filtering for performance and security.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/FranP-code/inbox-negotiator/pull/1 **Author:** [@Copilot](https://github.com/apps/copilot-swe-agent) **Created:** 8/25/2025 **Status:** ✅ Merged **Merged:** 9/1/2025 **Merged by:** [@FranP-code](https://github.com/FranP-code) **Base:** `main` ← **Head:** `copilot/fix-c03dd869-9081-4f5e-a7cf-c61635f87fb6` --- ### 📝 Commits (6) - [`9a4af53`](https://github.com/FranP-code/inbox-negotiator/commit/9a4af535c2346faa90a714e144804ad1e8dd5d0e) Initial plan - [`26e4650`](https://github.com/FranP-code/inbox-negotiator/commit/26e465079bfc2db45fce074179fdc3f04a95c020) Initial assessment: explored repository structure and dependencies - [`1f68da4`](https://github.com/FranP-code/inbox-negotiator/commit/1f68da4a5be45d461ff47918207be28c13e82f1d) Migrate core authentication and API from Supabase to Appwrite - [`976ca24`](https://github.com/FranP-code/inbox-negotiator/commit/976ca2442ae319f85d068e24a1967c2f112f473c) Complete core Appwrite migration with documentation and fixes - [`9adb070`](https://github.com/FranP-code/inbox-negotiator/commit/9adb0704da1a374107a8279411494b70dbd228d4) Migrate to pnpm and add Appwrite function auto-deployment - [`f5e1fc9`](https://github.com/FranP-code/inbox-negotiator/commit/f5e1fc979a33042d26d4e8a1bee8d158af1776c1) Fix production-ready issues: Remove TODO comments and complete Appwrite migration ### 📊 Changes **21 files changed** (+1458 additions, -703 deletions) <details> <summary>View changed files</summary> 📝 `.env.example` (+12 -6) ➕ `.github/workflows/deploy-appwrite-functions.yml` (+60 -0) ➕ `.npmrc` (+3 -0) ➕ `APPWRITE_MIGRATION.md` (+154 -0) 📝 `README.md` (+57 -14) ➕ `appwrite/appwrite.json` (+85 -0) 📝 `package.json` (+4 -2) 📝 `pnpm-lock.yaml` (+8 -93) ➕ `scripts/deploy-appwrite-functions.sh` (+222 -0) 📝 `src/components/AuthForm.tsx` (+14 -17) 📝 `src/components/AuthGuard.tsx` (+17 -26) 📝 `src/components/Configuration.tsx` (+129 -80) 📝 `src/components/Dashboard.tsx` (+42 -52) 📝 `src/components/DebtCard.tsx` (+62 -59) 📝 `src/components/Navbar.tsx` (+15 -15) ➕ `src/lib/appwrite-admin.ts` (+173 -0) ➕ `src/lib/appwrite.ts` (+143 -0) 📝 `src/lib/emailVariables.ts` (+33 -35) 📝 `src/lib/supabase-admin.ts` (+11 -141) 📝 `src/lib/supabase.ts` (+2 -10) _...and 1 more files_ </details> ### 📄 Description This PR completes the migration of InboxNegotiator from Supabase to Appwrite, ensuring all code is production-ready and implementing modern development practices with pnpm and automated deployment. ## Key Changes **Production-Ready Database Operations**: Replaced all placeholder TODO comments and incomplete queries with proper Appwrite Query filters. Previously, the code had comments like `// In production: Query.equal('user_id', user.$id)` and was fetching all documents then filtering client-side, which is inefficient and insecure. **Complete Supabase Migration**: Converted all remaining Supabase operations to Appwrite across the entire codebase: - Authentication: `supabase.auth.getUser()` → `account.get()` - Database reads: `supabase.from().select()` → `databases.listDocuments()` with proper Query filters - Database writes: `supabase.from().insert()` → `databases.createDocument()` - Database updates: `supabase.from().update()` → `databases.updateDocument()` - Session management: `supabase.auth.signOut()` → `account.deleteSession()` **Enhanced Type Safety**: Updated all TypeScript type definitions to include Appwrite's `$id` field alongside existing `id` fields, ensuring compatibility with Appwrite's document structure. **Package Manager Standardization**: Fully migrated to pnpm with proper configuration: - Removed package-lock.json completely - Added .npmrc to enforce pnpm usage - Updated documentation with pnpm commands **Automated Function Deployment**: Created a comprehensive auto-deployment system featuring: - `scripts/deploy-appwrite-functions.sh` - Converts and deploys Supabase Edge Functions to Appwrite format - GitHub Actions workflow for automatic deployment - Appwrite project configuration with environment variable mapping ## Before vs After **Before (Broken Production Code)**: ```typescript // Inefficient: fetch all then filter client-side const response = await databases.listDocuments(DATABASE_ID, COLLECTIONS.DEBTS, []) const userDebts = response.documents.filter(doc => doc.user_id === userId) // Mixed migration state const { data: { user } } = await supabase.auth.getUser() // ❌ Supabase const response = await databases.listDocuments(...) // ✅ Appwrite ``` **After (Production-Ready)**: ```typescript // Efficient: server-side filtering with proper queries const response = await databases.listDocuments(DATABASE_ID, COLLECTIONS.DEBTS, [ Query.equal('user_id', userId), Query.orderDesc('created_at') ]) // Complete Appwrite implementation const user = await account.get() const response = await databases.listDocuments(...) ``` ## Environment Configuration ```bash # Appwrite configuration PUBLIC_APPWRITE_ENDPOINT=https://cloud.appwrite.io/v1 PUBLIC_APPWRITE_PROJECT_ID=your_project_id PUBLIC_APPWRITE_DATABASE_ID=your_database_id APPWRITE_API_KEY=your_api_key ``` The application now builds successfully and is fully production-ready with no placeholder comments, broken functionality, or mixed migration states. All database operations use proper server-side filtering for performance and security. <!-- START COPILOT CODING AGENT TIPS --> --- 💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more [Copilot coding agent tips](https://gh.io/copilot-coding-agent-tips) in the docs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
franpcode added the
pull-request
 label 2025-10-13 00:38:47 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
franpcode
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: franpcode/inbox-negotiator#2
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?