fix: Bump up realtime; add integration tests

[filipecabaco]

What kind of change does this PR introduce?

Bump up realtime; add integration tests

[vilnius1998]

Hey, do we have an estimated time when this will be merged? Thanks!

[filipecabaco]

We're doing some integration testing to ensure we don't break things but hopefully this should get merged in the next couple of weeks

[BrandontMitchell]

following 👀 looking forward to using!

[Copilot]

Pull Request Overview

This PR bumps the realtime dependency version and adds comprehensive integration tests for Supabase functionality. Key changes include:

• Implementation of integration tests covering PostgREST, authentication, and realtime channels.
• Migration scripts for setting up todos tables and realtime RLS policies.
• Updates to helper methods (renaming stripTrailingSlash to cleanUrl) and test commands in package.json and README.
• CI workflow now includes Supabase CLI setup.

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
test/integration.test.ts	Added integration tests for multiple Supabase functionalities
test/helper.test.ts	Updated tests to use the new cleanUrl function
supabase/seed/seed_todos.sql	Added seed data for the todos table
supabase/migrations/20250423000000_realtime_rls_setup.sql	Added migration for realtime RLS policies
supabase/migrations/20250422000000_create_todos_table.sql	Added migration to create todos table
supabase/.temp/cli-latest	Updated Supabase CLI version in temporary file
supabase/.branches/_current_branch	Confirmed branch configuration
src/lib/helpers.ts	Renamed stripTrailingSlash to cleanUrl and updated its functionality
src/SupabaseClient.ts	Updated to use cleanUrl instead of stripTrailingSlash
package.json	Modified test commands and bumped realtime-js version
README.md	Added documentation for running integration tests
.github/workflows/ci.yml	Added step to set up Supabase CLI in CI

[Copilot]

Pull Request Overview

This PR bumps the realtime library version and adds integration tests while refactoring URL cleaning logic. The key changes include:

• Adding comprehensive integration tests for Supabase functionalities.
• Replacing the stripTrailingSlash function with a more robust cleanUrl implementation.
• Updating package.json and CI workflow to support integration tests and the new realtime version.

Reviewed Changes

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

Show a summary per file

File	Description
test/integration.test.ts	Adds integration tests for PostgREST, authentication, and realtime functionality.
test/helper.test.ts	Updates tests to validate the new cleanUrl behavior for trimming URLs.
supabase/seed/seed_todos.sql	Adds seed data for the todos table.
supabase/migrations/*.sql	Introduces new migration scripts for realtime RLS setup and todos table creation.
supabase/.temp/cli-latest	Updates the Supabase CLI version.
supabase/.branches/_current_branch	Confirms the current branch.
src/lib/helpers.ts	Refactors the URL cleaning function from stripTrailingSlash to cleanUrl.
src/SupabaseClient.ts	Updates usage of the URL cleaning function.
package.json	Updates realtime-js version and adjusts test scripts for integration testing.
README.md	Adds documentation for running unit and integration tests.
.github/workflows/ci.yml	Adds steps to set up the Supabase CLI in the CI workflow.

[Copilot]

The updated cleanUrl function now removes both trailing slashes and whitespace. Consider adding a docstring to clarify its enhanced behavior compared to the old stripTrailingSlash implementation.

Suggested change

	/**
	* Cleans a URL by removing trailing slashes and whitespace.
	*
	* This function enhances the behavior of the old `stripTrailingSlash` implementation
	* by also removing any trailing whitespace in addition to slashes.
	*
	* @param url - The URL string to clean.
	* @returns The cleaned URL string without trailing slashes or whitespace.
	*/

[Copilot]

[nitpick] The loop uses a magic number (50) for the subscription timeout. Extract this value into a named constant to improve readability and maintainability.

Suggested change

	if (attempts > 50) throw new Error('Timeout waiting for subscription')
	if (attempts > SUBSCRIPTION_TIMEOUT_ATTEMPTS) throw new Error('Timeout waiting for subscription')

[Copilot]

[nitpick] The message waiting loop also uses the magic number (50) for timeout. Consider refactoring to use a named constant for this timeout threshold for clarity.

Suggested change

	if (attempts > 50) throw new Error('Timeout waiting for message')
	if (attempts > TIMEOUT_THRESHOLD) throw new Error('Timeout waiting for message')