Update GitHub link and enhance cursor functionality by jomzxc · Pull Request #14 · jomzxc/WebDrop

jomzxc · 2025-11-06T02:12:35Z

Description

This pull request includes the following updates:

Update GitHub Repository Link:
- Adjusts the GitHub link to correctly point to the WebDrop repository.
Dynamic Cursor Enhancements:
- Adds dynamic cursor styles for better interactivity with user interface elements.
- Removes redundant cursor styles from the unused globals.css.
Documentation Improvements:
- Includes additional updates to the documentation for better clarity and accuracy where applicable.

Checklist

Documentation updated where necessary
Changes comply with contributing guidelines

Removed row-level security and policies for profiles, rooms, peers, and file_transfers tables.

This script enables row-level security on multiple tables and creates various policies for profiles, rooms, peers, and file transfers.

Added foreign key constraint from peers to profiles and created an index for better join performance.

This reverts commit 386bac7.

- Rename SQL migration files for clearer sequencing. - Update foreign key constraints in `peers` and `profiles` tables. - Ensure real-time updates for `peers` and `rooms` functionality. - Update README with new migration order and references.

Fix: Correct SQL Migration Dependency and Naming Errors

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Fix Next.js 16 build failures: migrate to proxy convention and handle missing env vars

Merge eslint.ignoreDuringBuilds into next.config.mjs #VERCEL_SKIP Co-authored-by: JOM <74537369+jomzxc@users.noreply.github.com>

…markdowns Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Consolidate documentation: create comprehensive README and CONTRIBUTING, remove redundant markdown files

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Add MIT License file to repository

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

…nality Add dynamic cursor styles for interactive elements

github-advanced-security · 2025-11-06T02:12:36Z

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

vercel · 2025-11-06T02:12:41Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
webdrop	Ready	Preview	Comment	Nov 6, 2025 2:36am

Copilot

Pull Request Overview

This PR performs a comprehensive project reorganization and documentation update for WebDrop. It refactors the database setup scripts, improves configuration management, and significantly enhances project documentation.

Reorganizes SQL migration scripts into a numbered sequence with clearer separation of concerns
Adds graceful fallbacks for missing Supabase environment variables to enable builds without credentials
Enhances documentation with comprehensive README, CONTRIBUTING guide, and issue templates

Reviewed Changes

Copilot reviewed 20 out of 25 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
`scripts/001_create_tables.sql`	Removed RLS policies from table creation script (moved to separate file)
`scripts/002_create_policies.sql`	New file containing all RLS policies extracted from table creation
`scripts/003_handle_new_user_trigger.sql`	New file for auto-profile creation trigger
`scripts/004_handle_updated_at_trigger.sql`	Renamed trigger to avoid naming conflicts with username sync trigger
`scripts/004_add_peers_profile_fkey.sql`	Deleted (migrated to script 005)
`scripts/005_add_peers_profile_fkey.sql`	New migration adding foreign key constraint with proper schema prefix
`scripts/006_enable_realtime.sql`	New file enabling realtime subscriptions on peers and rooms tables
`scripts/007_avatar_storage.sql`	New file setting up avatar storage bucket with RLS policies
`scripts/008_sync_username_trigger.sql`	Fixed trigger naming conflict by renaming and cleaning up old triggers
`proxy.ts`	Renamed exported function from `middleware` to `proxy`
`lib/supabase/client.ts`	Added environment variable fallbacks for build-time compatibility
`lib/supabase/server.ts`	Added environment variable fallbacks with warning messages
`lib/supabase/middleware.ts`	Added environment variable checks with graceful passthrough
`app/page.tsx`	Updated GitHub link to actual repository URL
`app/layout.tsx`	Removed unused font imports
`app/globals.css`	Added comprehensive cursor styles for interactive elements
`tsconfig.json`	Updated JSX transform and formatting, added dev types path
`README.md`	Complete rewrite with detailed architecture, setup, and troubleshooting
`CONTRIBUTING.md`	New comprehensive contributor guide
`LICENSE`	Added MIT license
`.gitignore`	Refined to allow `.env.example` while ignoring local env files
`SUPABASE_SETUP.md`	Removed (content integrated into README)
`SIGNALING_SETUP.md`	Removed (content integrated into README)
`.github/ISSUE_TEMPLATE/bug_report.md`	Added bug report template
`.github/ISSUE_TEMPLATE/feature_request.md`	Added feature request template

Comments suppressed due to low confidence (1)

proxy.ts:4

The function is named proxy but this file appears to be the Next.js middleware entry point. Next.js middleware requires the exported function to be named middleware, not proxy. This will break the middleware functionality. Either rename the function back to middleware or ensure this file is properly imported and called from an actual middleware.ts file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-11-06T02:16:11Z

lib/supabase/server.ts

+    supabaseUrl || "https://placeholder.supabase.co",
+    supabaseAnonKey || "placeholder-anon-key",


Using placeholder credentials when environment variables are missing creates a security risk. The application will appear to work but will fail silently or unpredictably at runtime when making actual Supabase calls. Consider throwing an error during build time or at least during client creation to fail fast, rather than allowing the app to run with invalid credentials.

@copilot open a new pull request to apply changes based on this feedback

lib/supabase/client.ts

scripts/005_add_peers_profile_fkey.sql

Copilot · 2025-11-06T02:18:06Z

@jomzxc I've opened a new pull request, #15, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Pull Request Overview

Copilot reviewed 20 out of 25 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (2)

proxy.ts:6

The function is named proxy but this file appears to be Next.js middleware (evidenced by the config export with matcher). Next.js expects middleware files to export a function named middleware. Renaming it to proxy may prevent Next.js from recognizing this as middleware. If this file is intended to be middleware.ts at the root, it should export middleware, not proxy.
scripts/001_create_tables.sql:1
The comment '-- Profiles policies' on line 10 (old numbering) is misleading because all the RLS policies have been removed from this file and moved to 002_create_policies.sql. This orphaned comment should be removed as it no longer describes the following code.

-- Create profiles table for user management

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Replace placeholder Supabase credentials with fail-fast error throwing

Copilot

Pull Request Overview

Copilot reviewed 20 out of 25 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

proxy.ts:6

The function name 'proxy' does not accurately describe its purpose. In Next.js, this file serves as middleware and should export a function named 'middleware' for Next.js to recognize it. Renaming from 'middleware' to 'proxy' will break the middleware functionality unless there's a separate middleware.ts file that imports and re-exports this function. If this is intentional refactoring, ensure a middleware.ts file exists that exports this function as 'middleware'.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Merge pull request #14 from jomzxc/develop

jomzxc and others added 30 commits November 5, 2025 14:07

Remove RLS and policies from multiple tables

fe2e499

Removed row-level security and policies for profiles, rooms, peers, and file_transfers tables.

Add row-level security and policies for tables

0e2bee0

This script enables row-level security on multiple tables and creates various policies for profiles, rooms, peers, and file transfers.

Add new SQL script for handling new user trigger

af31675

Rename timestamp trigger script to handle updated_at

33b4c41

Add foreign key constraint and index for peers table

386bac7

Added foreign key constraint from peers to profiles and created an index for better join performance.

Enable realtime for peers and rooms tables

cd686bd

Revert "Add foreign key constraint and index for peers table"

cb1e7d6

This reverts commit 386bac7.

Merge pull request #1

87ab523

Fix: Correct SQL Migration Dependency and Naming Errors

Initial plan

a1643b1

Initial assessment of build issues

c8054ae

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Fix Next.js 16 build errors and add environment variable handling

c135172

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Migrate from middleware.ts to proxy.ts per Next.js 16 convention

b59cb2b

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Update README with improved build instructions and environment setup

ff82d5e

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Merge pull request #7 from jomzxc/copilot/fix-build-error-in-next-config

705b2b7

Fix Next.js 16 build failures: migrate to proxy convention and handle missing env vars

feat: add eslint config to Next.js build

212e83d

Merge eslint.ignoreDuringBuilds into next.config.mjs #VERCEL_SKIP Co-authored-by: JOM <74537369+jomzxc@users.noreply.github.com>

Initial plan

973c0e8

docs: create comprehensive README and CONTRIBUTING, remove redundant …

5239354

…markdowns Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

docs: correct Next.js and React versions to match package.json

80b74e7

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

docs: improve README and CONTRIBUTING based on code review feedback

cf2dbb7

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Merge pull request #9 from jomzxc/copilot/edit-markdowns-remove-extras

4649583

Consolidate documentation: create comprehensive README and CONTRIBUTING, remove redundant markdown files

Update issue templates

97df76c

Initial plan

5b73f6c

Add MIT License file

6b86be8

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Initial plan

602f06c

Merge pull request #10 from jomzxc/copilot/audit-profile-tests-repo-docs

7b17624

Add MIT License file to repository

Add dynamic cursor styles for interactive elements

ee8f4b3

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Remove duplicate cursor styles from unused globals.css

7645b0f

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Merge pull request #12 from jomzxc/copilot/add-dynamic-cursor-functio…

5c41d5c

…nality Add dynamic cursor styles for interactive elements

Update GitHub link to point to WebDrop repository

bf85c19

jomzxc added the enhancement New feature or request label Nov 6, 2025

jomzxc marked this pull request as ready for review November 6, 2025 02:14

jomzxc requested a review from Copilot November 6, 2025 02:14

jomzxc assigned jomzxc and Copilot Nov 6, 2025

Copilot AI reviewed Nov 6, 2025

View reviewed changes

Initial plan

6df0352

Copilot AI mentioned this pull request Nov 6, 2025

Replace placeholder Supabase credentials with fail-fast error throwing #15

Merged

jomzxc requested a review from Copilot November 6, 2025 02:21

Copilot AI reviewed Nov 6, 2025

View reviewed changes

README.md Show resolved Hide resolved

Copilot AI and others added 2 commits November 6, 2025 02:24

Replace placeholder credentials with fail-fast error throwing

728e693

Co-authored-by: jomzxc <74537369+jomzxc@users.noreply.github.com>

Update lib/supabase/client.ts

82ccc2e

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

vercel bot deployed to Preview November 6, 2025 02:31 View deployment

jomzxc added 2 commits November 6, 2025 10:34

Merge branch 'develop' into copilot/sub-pr-14

ab1c0dd

Merge pull request #15 from jomzxc/copilot/sub-pr-14

89c0b02

Replace placeholder Supabase credentials with fail-fast error throwing

jomzxc requested a review from Copilot November 6, 2025 02:36

vercel bot deployed to Preview November 6, 2025 02:36 View deployment

Copilot AI reviewed Nov 6, 2025

View reviewed changes

jomzxc merged commit 72af6fb into main Nov 6, 2025
10 checks passed

jomzxc added a commit that referenced this pull request Nov 6, 2025

Merge pull request #16 from jomzxc/main

b964374

Merge pull request #14 from jomzxc/develop

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

Update GitHub link and enhance cursor functionality#14

Update GitHub link and enhance cursor functionality#14
jomzxc merged 35 commits intomainfrom
develop

jomzxc commented Nov 6, 2025 •

edited

Loading

Uh oh!

github-advanced-security bot commented Nov 6, 2025

Uh oh!

vercel bot commented Nov 6, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Nov 6, 2025

Uh oh!

jomzxc Nov 6, 2025

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Nov 6, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		supabaseUrl \|\| "https://placeholder.supabase.co",
		supabaseAnonKey \|\| "placeholder-anon-key",

Comments

Conversation

jomzxc commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Uh oh!

github-advanced-security bot commented Nov 6, 2025

Uh oh!

vercel bot commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Nov 6, 2025

Choose a reason for hiding this comment

Uh oh!

jomzxc Nov 6, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Nov 6, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

jomzxc commented Nov 6, 2025 •

edited

Loading

vercel bot commented Nov 6, 2025 •

edited

Loading