Skip to content

Use public logo URL to avoid binary diff and harden auth loading state#83

Merged
fuzziecoder merged 1 commit intocodex/fix-remaining-issues-and-raise-prfrom
codex/fix-loading-issue-cmfb7u
Feb 24, 2026
Merged

Use public logo URL to avoid binary diff and harden auth loading state#83
fuzziecoder merged 1 commit intocodex/fix-remaining-issues-and-raise-prfrom
codex/fix-loading-issue-cmfb7u

Conversation

@fuzziecoder
Copy link
Copy Markdown
Owner

@fuzziecoder fuzziecoder commented Feb 24, 2026

Motivation

  • Avoid including a binary image in the PR while keeping logo resolution robust for non-root deployments.
  • Prevent the auth hook from remaining stuck in loading if Supabase session retrieval hangs or emits stale updates.

Description

  • Replaced the bundled binary-based logo approach with a public URL export by adding src/lib/assets.ts that exports logoUrl = ${import.meta.env.BASE_URL}logo.jpg`` and removed src/assets/logo.jpg so the PR contains no binary diffs.
  • Updated UI to import and use logoUrl instead of "/logo.jpg" in Login, Signup, Sidebar and FlexibleAI so the logo remains centralized and works under arbitrary base paths.
  • Improved useAuth in src/hooks/useAuth.ts by adding an isMounted guard, a 5s fallback loadingTimeout, explicit .catch/.finally handling for getSession(), and cleanup logic that clears the timeout and unsubscribes the Supabase subscription on unmount so loading always resolves and no stale state updates occur.

Testing

  • Ran the production build with npm run build (which runs tsc -b && vite build) and the build completed successfully.

Codex Task


Open with Devin

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 24, 2026

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

Project Deployment Actions Updated (UTC)
flexi-roaster Ready Ready Preview, Comment Feb 24, 2026 3:20pm

@fuzziecoder fuzziecoder self-assigned this Feb 24, 2026
@fuzziecoder fuzziecoder merged commit 7079b81 into codex/fix-remaining-issues-and-raise-pr Feb 24, 2026
5 of 6 checks passed
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant