feat: implement connection system #65

Closed

exe-dev-bot wants to merge 7 commits from (deleted):feat/connections-system into main

pull from: (deleted):feat/connections-system

merge into: kwila:main

kwila:main

kwila:exe-dev-bot-feat/newsletter-delivery

kwila:exe-dev-bot-feat/connections-system

kwila:add-invite-validation-system

kwila:set-up-supabase-email-auth

kwila:setup-supabase-storage

kwila:feat/db-migrations

kwila:feat-docker-compose

kwila:mvp-add-docker-compose

kwila:claude/mvp-spec-next-01JjffeFb97yuuUWk8o6J2VV

Conversation 0 Commits 7 Files changed 23 +1549 -194

exe-dev-bot commented 2026-01-10 21:02:23 -05:00

Contributor

Copy link

Summary

Implements the connection request system (spec 002) that allows users to establish trust relationships.

Changes

Database

• Rename user_a/user_b columns to initiator_id/receiver_id for clarity
• Add can_create_connection_request() function for rate limiting (5/day)
• Add get_connection_status() RPC for checking connection state
• Update RLS policies for role-based delete permissions

API Endpoints

• POST /api/connections - Create connection request (with rate limiting)
• PATCH /api/connections/[id] - Accept/decline request (receiver only)
• DELETE /api/connections/[id] - Withdraw pending or delete declined

UI Components

• ConnectButton - Connect button with confirmation dialog and trust checkboxes
• ConnectionActions - Accept/Decline/Withdraw buttons
• ProfileShare - QR code and Web Share API for profile sharing

Pages

• Profile page: Added Connect button with status display
• Connections page: Separated incoming/outgoing pending requests
• Dashboard: Added profile sharing card with QR code

Testing

• All unit tests pass
• All lint and type checks pass
• Manual testing pending (spec tasks marked incomplete)

Related

• Closes: spec 002-connections

## Summary Implements the connection request system (spec 002) that allows users to establish trust relationships. ## Changes ### Database - Rename `user_a`/`user_b` columns to `initiator_id`/`receiver_id` for clarity - Add `can_create_connection_request()` function for rate limiting (5/day) - Add `get_connection_status()` RPC for checking connection state - Update RLS policies for role-based delete permissions ### API Endpoints - `POST /api/connections` - Create connection request (with rate limiting) - `PATCH /api/connections/[id]` - Accept/decline request (receiver only) - `DELETE /api/connections/[id]` - Withdraw pending or delete declined ### UI Components - `ConnectButton` - Connect button with confirmation dialog and trust checkboxes - `ConnectionActions` - Accept/Decline/Withdraw buttons - `ProfileShare` - QR code and Web Share API for profile sharing ### Pages - **Profile page**: Added Connect button with status display - **Connections page**: Separated incoming/outgoing pending requests - **Dashboard**: Added profile sharing card with QR code ## Testing - All unit tests pass - All lint and type checks pass - Manual testing pending (spec tasks marked incomplete) ## Related - Closes: spec 002-connections

exe-dev-bot added 1 commit 2026-01-10 21:02:24 -05:00

feat: implement connection system ... 076f80e442

- Rename connection columns user_a/user_b to initiator_id/receiver_id
- Add rate limiting function can_create_connection_request()
- Add get_connection_status() RPC for checking connection state
- Update RLS policies for role-based delete permissions
- Create API endpoints for connections (POST, PATCH, DELETE)
- Add ConnectButton component with confirmation dialog
- Add ConnectionActions for accept/decline/withdraw
- Add ProfileShare with QR code and Web Share API
- Update connections page with incoming/outgoing sections
- Add connection status display on profile pages
- Update all queries to use new column names

Co-authored-by: Shelley <shelley@exe.dev>

addison added 1 commit 2026-01-16 16:01:42 -05:00

chore: re-enable CI workflow bc52e58845

exe-dev-bot added 1 commit 2026-01-16 16:49:11 -05:00

fix(ci): use npx astro dev instead of just for E2E tests ... 1ad62e5e29

The CI environment doesn't have 'just' installed, causing E2E tests
to fail with exit code 127 (command not found).

Co-authored-by: Shelley <shelley@exe.dev>

exe-dev-bot added 1 commit 2026-01-16 16:50:38 -05:00

perf(ci): use container images with Node.js pre-installed ... 453929d5cf

- Use node:20-bookworm for lint, type-check, format-check, unit-tests
- Use mcr.microsoft.com/playwright:v1.52.0-noble for E2E tests
- Remove setup-node steps and Playwright browser install steps
- This eliminates Node.js download/setup time in each job

Co-authored-by: Shelley <shelley@exe.dev>

exe-dev-bot added 1 commit 2026-01-16 16:51:28 -05:00

revert(ci): remove container images, keep original workflow ... 338d197aef

Container syntax may not work with Forgejo act runner.

Co-authored-by: Shelley <shelley@exe.dev>

exe-dev-bot added 1 commit 2026-01-16 16:52:23 -05:00

Revert "revert(ci): remove container images, keep original workflow" ... dc26fe3e00

This reverts commit 338d197aef.

exe-dev-bot added 1 commit 2026-01-16 17:10:26 -05:00

fix(ci): update Playwright image to v1.56.1-noble ... 40224ff75c

Playwright was updated to 1.56.1 but the CI was using v1.52.0-noble
Docker image, causing browser executable not found errors.

Co-authored-by: Shelley <shelley@exe.dev>

addison closed this pull request 2026-01-16 21:02:28 -05:00

Some checks failed

Hide all checks

CI / Type Check (pull_request) Successful in 1m35s

Required

Details

CI / Format Check (pull_request) Successful in 2m46s

Required

Details

CI / Lint (pull_request) Successful in 3m8s

Required

Details

CI / Unit Tests (pull_request) Successful in 3m4s

Required

Details

CI / E2E Tests (pull_request) Failing after 5m44s

Required

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

kwila/market!65

No description provided.