Warn when deployed studies use default Firebase config

[JackWilb]

Summary

• add a shared helper for detecting ReVISit's default Firebase project and suppressing default-deployment warnings on local/ReVISit-controlled hosts
• add a parser warning category for default Firebase config so study cards use the existing warning UI
• show a non-blocking Default Firebase badge in the direct study header
• cover helper, parser, warning renderer, and header behavior with Vitest tests

Fixes #1202

Verification

• yarn unittest --run src/utils/tests/defaultFirebaseConfig.spec.ts src/parser/tests/parser.spec.ts src/components/tests/ErrorLoadingConfig.spec.tsx src/components/interface/tests/AppHeader.spec.tsx
• yarn eslint src/utils/defaultFirebaseConfig.ts src/utils/tests/defaultFirebaseConfig.spec.ts src/parser/parser.ts src/parser/types.ts src/parser/tests/parser.spec.ts src/components/ErrorLoadingConfig.tsx src/components/tests/ErrorLoadingConfig.spec.tsx src/components/interface/AppHeader.tsx src/components/interface/tests/AppHeader.spec.tsx
• git diff --check -- src/utils/defaultFirebaseConfig.ts src/utils/tests/defaultFirebaseConfig.spec.ts src/parser/parser.ts src/parser/types.ts src/parser/tests/parser.spec.ts src/components/ErrorLoadingConfig.tsx src/components/tests/ErrorLoadingConfig.spec.tsx src/components/interface/AppHeader.tsx src/components/interface/tests/AppHeader.spec.tsx

Note

• yarn typecheck is currently blocked by unrelated unstaged storage-engine changes in the worktree: src/tests/utils.ts does not implement _updateSnapshotParticipantCounts from StorageEngine.

[github-actions]

🪓 PR closed, deleted preview.

[jaykim1213]

Looks great. I was wondering if we should open an issue for Supabase as well, e.g., warning when the default VITE_SUPABASE_URL is used with VITE_STORAGE_ENGINE="supabase".

[JackWilb]

Addressed the review feedback in 85aba38:

• Removed the alert icon from the Default Firebase header badge so it matches the other status badges.
• Added Firebase storageBucket detection for revisit-utah.appspot.com.
• Added default Supabase warnings when VITE_STORAGE_ENGINE=supabase and VITE_SUPABASE_URL is on revisit.dev or a revisit.dev subdomain.

Local verification passed: focused Vitest suite, scoped ESLint, git diff --check, and yarn typecheck.

[JackWilb]

Follow-up on the review feedback:

• Removed IconAlertTriangle from the direct-study header badge. The Default Firebase and Default Supabase badges now match the existing text-only status badges like Demo Mode and Storage Disconnected.
• Expanded the Firebase default-project detector to also check storageBucket equals revisit-utah.appspot.com, in addition to projectId and authDomain.
• Added Supabase default-backend warnings. The rule is intentionally host-based: when VITE_STORAGE_ENGINE is supabase, we parse VITE_SUPABASE_URL and warn if its hostname is revisit.dev or any revisit.dev subdomain.
• Wired the Supabase warning through the same parser warning UI on study cards and the same direct-study header badge path.
• Added focused coverage for the new Firebase storage bucket case, Supabase URL detection, parser warning behavior, warning rendering, and direct header badge display.

Verification after the changes:

• yarn unittest --run src/utils/tests/defaultFirebaseConfig.spec.ts src/parser/tests/parser.spec.ts src/components/tests/ErrorLoadingConfig.spec.tsx src/components/interface/tests/AppHeader.spec.tsx
• scoped yarn eslint on the touched files
• git diff --check
• yarn typecheck

[JackWilb]

Follow-up on the review feedback:

• Removed the from the direct-study header badge. The Default Firebase and Default Supabase badges now match the existing text-only status badges like Demo Mode and Storage Disconnected.
• Expanded the Firebase default-project detector to also check , in addition to and .
• Added Supabase default-backend warnings. The rule is intentionally host-based: when , we parse and warn if its hostname is or any subdomain.
• Wired the Supabase warning through the same parser warning UI on study cards and the same direct-study header badge path.
• Added focused coverage for the new Firebase storage bucket case, Supabase URL detection, parser warning behavior, warning rendering, and direct header badge display.

Verification after the changes:

• yarn run v1.22.22
  $ vitest --run src/utils/tests/defaultFirebaseConfig.spec.ts src/parser/tests/parser.spec.ts src/components/tests/ErrorLoadingConfig.spec.tsx src/components/interface/tests/AppHeader.spec.tsx

RUN v4.1.0 /Users/jwilburn/Projects/revisit-study

Test Files 4 passed (4)
Tests 67 passed (67)
Start at 23:45:07
Duration 2.09s (transform 545ms, setup 164ms, import 1.92s, tests 93ms, environment 2.61s)

Done in 2.65s.

• scoped yarn run v1.22.22
  $ /Users/jwilburn/Projects/revisit-study/node_modules/.bin/eslint
  info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. on the touched files
• yarn run v1.22.22
  $ tsc --noEmit
  Done in 11.03s.

[jaykim1213]

Approved.
One small suggestion: could we rename defaultFirebaseConfig.ts to something likedefaultStorageConfig.ts since it now handles both Firebase and Supabase? Thank you!