Add auth #6

Shitanshukumar607 · 2025-10-04T22:33:31Z

add google login

vercel · 2025-10-04T22:33:36Z

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

Project	Deployment	Preview	Comments	Updated (UTC)
img-to-svg	Ready	Preview	Comment	Oct 4, 2025 11:00pm

Copilot

Pull Request Overview

Adds Supabase-based authentication, including Google/GitHub OAuth login, and integrates auth UI elements into the navbar.

Introduces Supabase SSR/browser clients and an OAuth callback route
Adds login page with OAuth and email/password, plus new UI primitives (card, field, input, label, separator)
Updates navbar to show auth buttons

Reviewed Changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 15 comments.

Show a summary per file

File	Description
package.json	Adds Radix Label/Separator and @supabase/ssr dependencies to support new UI and auth.
lib/supabase/server.ts	Server-side Supabase client for cookie-based session handling.
lib/supabase/middleware.ts	Middleware helper for cookie syncing (naming/return issues noted).
lib/supabase/client.ts	Browser Supabase client for client-side auth flows.
components/ui/separator.tsx	New separator UI component.
components/ui/label.tsx	New label UI component.
components/ui/input.tsx	New input UI component (should be client component).
components/ui/field.tsx	Field composition primitives for forms.
components/ui/card.tsx	Card UI component (should be client component).
components/ui/button.tsx	Adds cursor-pointer to button base styles.
components/NavbarComponents/Navbar.tsx	Hooks up AuthButtons to navbar.
components/NavbarComponents/AuthButtons.tsx	Renders login/logout controls (missing sign-out action).
app/signup/page.tsx	Placeholder signup page.
app/login/page.tsx	Login page with OAuth and email/password flows (callback URL issues, minor import issues).
app/api/auth/callback/route.ts	OAuth callback route to exchange code for session and redirect.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-04T22:37:43Z

app/login/page.tsx

+      const { error } = await supabase.auth.signInWithOAuth({
+        provider,
+        options: {
+          redirectTo: `${process.env.SITE_URL}/auth/callback`,


redirectTo points to /auth/callback and uses process.env.SITE_URL in a client component. Client-side env vars must be prefixed with NEXT_PUBLIC_, and your callback route is implemented at /api/auth/callback. Update to use the correct path and a client-safe origin, e.g.: redirectTo: ${window.location.origin}/api/auth/callback.

Suggested change

redirectTo: `${process.env.SITE_URL}/auth/callback`,

redirectTo: `${window.location.origin}/api/auth/callback`,

Copilot · 2025-10-04T22:37:44Z

lib/supabase/middleware.ts

+      setAll(cookiesToSet) {
+        cookiesToSet.forEach(({ name, value, options }) =>
+          request.cookies.set(name, value),
+        );
+        supabaseResponse = NextResponse.next({
+          request,
+        });
+        cookiesToSet.forEach(({ name, value, options }) =>
+          supabaseResponse.cookies.set(name, value, options),
+        );
+      },


In middleware, request.cookies is read-only; setting cookies on the request has no effect. Also, NextResponse.next expects request: { headers: Headers }, not the full request object. Remove the request.cookies.set call and construct NextResponse.next with headers only, then set cookies on the response.

Copilot · 2025-10-04T22:37:44Z

lib/supabase/middleware.ts

+const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL;
+const supabaseKey = process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY;
+
+export const createClient = (request: NextRequest) => {


[nitpick] This function is named createClient but returns a NextResponse, not the Supabase client. Either return the Supabase client (and/or a tuple like { supabase, response }) or rename this to reflect its purpose (e.g., updateSession) and clearly document the return shape.

Suggested change

export const createClient = (request: NextRequest) => {

/**

* Updates the session cookies and returns a NextResponse object.

* @param {NextRequest} request - The incoming Next.js request.

* @returns {NextResponse} The modified NextResponse with updated cookies.

*/

export const updateSessionResponse = (request: NextRequest) => {

Copilot · 2025-10-04T22:37:44Z

lib/supabase/middleware.ts

+  return supabaseResponse;
+};


[nitpick] This function is named createClient but returns a NextResponse, not the Supabase client. Either return the Supabase client (and/or a tuple like { supabase, response }) or rename this to reflect its purpose (e.g., updateSession) and clearly document the return shape.

Copilot · 2025-10-04T22:37:44Z

components/ui/input.tsx

+import * as React from "react";
+
+import { cn } from "@/lib/utils";


This component is imported by client components (e.g., the login page). Without 'use client' at the top, it'll be treated as a Server Component and cannot be imported into Client Components. Add 'use client' to avoid build/runtime errors.

Copilot · 2025-10-04T22:37:46Z

components/NavbarComponents/AuthButtons.tsx

+        <button className="hidden sm:flex text-xs border border-emerald-500 dark:border-purple-500 px-3 py-1.5 rounded-md hover:bg-emerald-100 dark:hover:bg-violet-900 transition-colors duration-200 items-center gap-1.5">
+          <span>Logout</span>
+          {!loading && (
+            <span className="text-xs opacity-70">{`(${user.user_metadata.name})`}</span>
+          )}
+        </button>


The Logout button has no onClick handler, so it won't sign the user out. Add a sign-out call, e.g., onClick={async () => { await supabase.auth.signOut(); location.reload(); }} or use router.refresh()/push after sign-out.

components/NavbarComponents/AuthButtons.tsx

app/login/page.tsx

Copilot · 2025-10-04T22:37:46Z

lib/supabase/client.ts

+export function createClient() {
+  return createBrowserClient(
+    process.env.NEXT_PUBLIC_SUPABASE_URL!,
+    process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY!,


[nitpick] The env var name NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY is non-standard and ambiguous. Prefer the conventional NEXT_PUBLIC_SUPABASE_ANON_KEY to reduce misconfiguration risk and align with Supabase docs.

Suggested change

process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY!,

process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY!,

Copilot · 2025-10-04T22:37:46Z

lib/supabase/server.ts

+import { cookies } from "next/headers";
+
+const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL;
+const supabaseKey = process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY;


[nitpick] Use a clear, conventional name for the anon key, e.g., NEXT_PUBLIC_SUPABASE_ANON_KEY. This avoids confusion with the service role key and aligns with Supabase examples.

Suggested change

const supabaseKey = process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY;

const supabaseKey = process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY;

Co-authored-by: Copilot <[email protected]>

into add-auth

Shitanshukumar607 added 3 commits October 4, 2025 22:31

add login and signup files

9d7a385

add google login w supabase and style the login page

52ae4fd

format using prettier

c227001

Copilot AI review requested due to automatic review settings October 4, 2025 22:33

Copilot AI reviewed Oct 4, 2025

View reviewed changes

Update lib/supabase/server.ts

6b14ba6

Co-authored-by: Copilot <[email protected]>

vercel bot had a problem deploying to Preview October 4, 2025 22:39 Failure

Shitanshukumar607 and others added 2 commits October 5, 2025 04:09

Update components/NavbarComponents/AuthButtons.tsx

224f1e9

Co-authored-by: Copilot <[email protected]>

Update app/login/page.tsx

115a954

Co-authored-by: Copilot <[email protected]>

vercel bot had a problem deploying to Preview October 4, 2025 22:40 Failure

Shitanshukumar607 added 2 commits October 5, 2025 04:23

fix eslint and prettier

50448d8

Merge branch 'add-auth' of https://github.com/Shitanshukumar607/imgToSvg

235fab5

into add-auth

vercel bot deployed to Preview October 4, 2025 22:55 View deployment

fix not-found page

1345ce8

vercel bot deployed to Preview October 4, 2025 22:59 View deployment

Merge branch 'main' into add-auth

851c89a

vercel bot deployed to Preview October 4, 2025 23:00 View deployment

Shitanshukumar607 merged commit 6a0efb9 into main Oct 4, 2025
5 of 6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add auth #6

Add auth #6

Uh oh!

Shitanshukumar607 commented Oct 4, 2025

Uh oh!

vercel bot commented Oct 4, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Uh oh!

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Copilot AI Oct 4, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	redirectTo: `${process.env.SITE_URL}/auth/callback`,
	redirectTo: `${window.location.origin}/api/auth/callback`,

-export const createClient = (request: NextRequest) => {
+/**
+ * Updates the session cookies and returns a NextResponse object.
+ * @param {NextRequest} request - The incoming Next.js request.
+ * @returns {NextResponse} The modified NextResponse with updated cookies.
+ */
+export const updateSessionResponse = (request: NextRequest) => {

		import * as React from "react";

		import { cn } from "@/lib/utils";

	process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY!,
	process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY!,

	const supabaseKey = process.env.NEXT_PUBLIC_SUPABASE_PUBLISHABLE_OR_ANON_KEY;
	const supabaseKey = process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY;

Add auth #6

Add auth #6

Uh oh!

Conversation

Shitanshukumar607 commented Oct 4, 2025

Uh oh!

vercel bot commented Oct 4, 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 Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel bot commented Oct 4, 2025 •

edited

Loading