Skip to content

ENG-1408 - Suppress env errors if SUPABASE_USE_DB="none" #760

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mdroidian wants to merge 4 commits into
base: main
Choose a base branch
from
Open

ENG-1408 - Suppress env errors if SUPABASE_USE_DB="none" #760

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
555ec40
Enhance Supabase integration by adding SUPABASE_USE_DB environment va…
mdroidian Feb 9, 2026
e4ef40b
Update compile script to conditionally set SUPABASE_USE_DB environmen…
mdroidian Feb 9, 2026
0085199
Disable ESLint naming convention rule in compile script to allow for …
mdroidian Feb 9, 2026
6f0c55a
Update envContents function to include SUPABASE_USE_DB variable from …
mdroidian Feb 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions apps/roam/scripts/compile.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
/* eslint-disable @typescript-eslint/naming-convention */
import esbuild from "esbuild";
import fs from "fs";
import path from "path";
Expand Down Expand Up @@ -159,6 +160,9 @@ export const compile = ({
bundle: true,
format,
define: {
"process.env.SUPABASE_USE_DB": dbEnv.SUPABASE_USE_DB
? `"${dbEnv.SUPABASE_USE_DB}"`
: "null",
"process.env.SUPABASE_URL": dbEnv.SUPABASE_URL
? `"${dbEnv.SUPABASE_URL}"`
: "null",
Expand Down
2 changes: 1 addition & 1 deletion apps/roam/src/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ export default runExtension(async (onloadArgs) => {
text: "(BETA) Suggestive Mode Enabled",
}).value;

if (isSuggestiveModeEnabled) {
if (isSuggestiveModeEnabled && process.env.SUPABASE_USE_DB !== "none") {
initializeSupabaseSync();
}

Expand Down
8 changes: 7 additions & 1 deletion packages/database/src/dbDotEnv.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
/* eslint-disable @typescript-eslint/naming-convention */
import { readFileSync, existsSync } from "node:fs";
import { join, dirname, basename } from "node:path";
import { fileURLToPath } from "node:url";
Expand Down Expand Up @@ -69,18 +70,23 @@ export const envFilePath = () => {
};

export const envContents = () => {
const variant = getVariant();
const path = envFilePath();
if (!path) {
// Fallback to process.env when running in production environments
const raw = {
SUPABASE_USE_DB: variant,
SUPABASE_URL: process.env.SUPABASE_URL,
SUPABASE_ANON_KEY: process.env.SUPABASE_ANON_KEY,
NEXT_API_ROOT: process.env.NEXT_API_ROOT,
};
return Object.fromEntries(Object.entries(raw).filter(([, v]) => !!v));
}
const data = readFileSync(path, "utf8");
return dotenv.parse(data);
return {
...dotenv.parse(data),
SUPABASE_USE_DB: variant,
};
};

let configDone = false;
Expand Down