refactor: make OPENCODE_CONFIG_CONTENT a dynamic Flag getter

ariane-emory · ariane-emory · commit dff8bbf2b514 · 2026-02-12T17:17:41.000-05:00
Converts OPENCODE_CONFIG_CONTENT to a dynamic getter on the Flag object,
matching the pattern used for OPENCODE_CONFIG_DIR and OPENCODE_CLIENT.
This ensures env var changes are reflected at access time.
diff --git a/packages/opencode/src/config/config.ts b/packages/opencode/src/config/config.ts
@@ -178,10 +178,10 @@ export namespace Config {
     // Route through load() to enable {env:} and {file:} token substitution.
     // Use a path within Instance.directory so relative {file:} paths resolve correctly.
     // The filename "OPENCODE_CONFIG_CONTENT" appears in error messages for clarity.
-    if (process.env.OPENCODE_CONFIG_CONTENT) {
+    if (Flag.OPENCODE_CONFIG_CONTENT) {
       result = mergeConfigConcatArrays(
         result,
-        await load(process.env.OPENCODE_CONFIG_CONTENT, path.join(Instance.directory, "OPENCODE_CONFIG_CONTENT")),
+        await load(Flag.OPENCODE_CONFIG_CONTENT, path.join(Instance.directory, "OPENCODE_CONFIG_CONTENT")),
       )
       log.debug("loaded custom config from OPENCODE_CONFIG_CONTENT")
     }
diff --git a/packages/opencode/src/flag/flag.ts b/packages/opencode/src/flag/flag.ts
@@ -8,7 +8,7 @@ export namespace Flag {
   export const OPENCODE_GIT_BASH_PATH = process.env["OPENCODE_GIT_BASH_PATH"]
   export const OPENCODE_CONFIG = process.env["OPENCODE_CONFIG"]
   export declare const OPENCODE_CONFIG_DIR: string | undefined
-  export const OPENCODE_CONFIG_CONTENT = process.env["OPENCODE_CONFIG_CONTENT"]
+  export declare const OPENCODE_CONFIG_CONTENT: string | undefined
   export const OPENCODE_DISABLE_AUTOUPDATE = truthy("OPENCODE_DISABLE_AUTOUPDATE")
   export const OPENCODE_DISABLE_PRUNE = truthy("OPENCODE_DISABLE_PRUNE")
   export const OPENCODE_DISABLE_TERMINAL_TITLE = truthy("OPENCODE_DISABLE_TERMINAL_TITLE")
@@ -91,3 +91,14 @@ Object.defineProperty(Flag, "OPENCODE_CLIENT", {
   enumerable: true,
   configurable: false,
 })
+
+// Dynamic getter for OPENCODE_CONFIG_CONTENT
+// This must be evaluated at access time, not module load time,
+// because external tooling may set this env var at runtime
+Object.defineProperty(Flag, "OPENCODE_CONFIG_CONTENT", {
+  get() {
+    return process.env["OPENCODE_CONFIG_CONTENT"]
+  },
+  enumerable: true,
+  configurable: false,
+})

Original file line number	Diff line number	Diff line change
`@@ -178,10 +178,10 @@ export namespace Config {`
`178`	`178`	`// Route through load() to enable {env:} and {file:} token substitution.`
`179`	`179`	`// Use a path within Instance.directory so relative {file:} paths resolve correctly.`
`180`	`180`	`// The filename "OPENCODE_CONFIG_CONTENT" appears in error messages for clarity.`
`181`		`- if (process.env.OPENCODE_CONFIG_CONTENT) {`
	`181`	`+ if (Flag.OPENCODE_CONFIG_CONTENT) {`
`182`	`182`	`result = mergeConfigConcatArrays(`
`183`	`183`	`result,`
`184`		`- await load(process.env.OPENCODE_CONFIG_CONTENT, path.join(Instance.directory, "OPENCODE_CONFIG_CONTENT")),`
	`184`	`+ await load(Flag.OPENCODE_CONFIG_CONTENT, path.join(Instance.directory, "OPENCODE_CONFIG_CONTENT")),`
`185`	`185`	`)`
`186`	`186`	`log.debug("loaded custom config from OPENCODE_CONFIG_CONTENT")`
`187`	`187`	`}`