@wtfalch/foundry 0.26.1 → 0.26.2

package/dist/plan/executor.d.ts

@@ -135,7 +135,6 @@ export interface CoolifyPort {
         name: string;
         postgres_db: string;
         postgres_user: string;
-        postgres_initdb_args?: string;
     }): Promise<{
         uuid: string;
         internal_db_url: string;
@@ -2220,6 +2219,14 @@ export declare class FoundryStepExecutor implements StepExecutor, StepDestroyer
      * `system:kind:key` slot of its own for the same reason.
      */
     private ensureCallsDatabase;
+    /**
+     * Sets `POSTGRES_INITDB_ARGS` on a Postgres database, once: the official
+     * image's entrypoint passes it to `initdb`, which Coolify's own
+     * `postgres_initdb_args` field never does ({@link CALLS_SYNAPSE_DB_INITDB_ARGS}).
+     * `/databases/{uuid}/envs` is not a typed port method yet, so this goes
+     * through `raw`. Nothing here is a secret.
+     */
+    private ensureDatabaseInitdbArgs;
     /**
      * Reads back {@link CALLS_SECRET_ENV_KEYS} from the MAS app's own envs
      * before minting fresh ones -- this stack's only memory across a re-run

package/dist/plan/executor.js

@@ -567,21 +567,17 @@ const CALLS_MAS_DB_SCOPE_KEY = 'coolify_calls_mas_database';
  * Synapse refuses to start against a database whose `LC_COLLATE`/`LC_CTYPE`
  * is not `C` (https://element-hq.github.io/synapse/latest/postgres.html);
  * Coolify's official postgres image otherwise creates one with `en_US.utf8`.
- * `createPostgresDatabase` forwards this to Coolify's own
- * `postgres_initdb_args` field -- but Coolify stores that field and reads it
- * back unchanged without ever handing it to `initdb`: its Postgres start
- * action builds the container's env from `postgres_user`/`postgres_password`/
- * `postgres_db` and runtime envs only
- * (https://github.com/coollabsio/coolify/blob/main/app/Actions/Database/StartPostgresql.php).
- * A database created this way still comes up `en_US.utf8` and crash-loops
- * every Synapse app that reads it; live evidence and the real fix (not yet
- * chosen) are tracked in #494. This constant and the field it feeds stay in
- * place, inert, until that lands -- collation cannot be migrated onto an
- * existing database under any of the candidate fixes either. MAS has no
- * such requirement, so this is passed for the Synapse database only. Not
- * using Synapse's own `allow_unsafe_locale` escape hatch regardless of
- * which fix lands: that trades a loud startup refusal for silent index
- * corruption after a future glibc upgrade.
+ * Coolify's `postgres_initdb_args` field is stored and never handed to
+ * `initdb` (its Postgres start action builds the container's env from
+ * `postgres_user`/`postgres_password`/`postgres_db` and runtime envs only,
+ * https://github.com/coollabsio/coolify/blob/main/app/Actions/Database/StartPostgresql.php),
+ * so foundry no longer sends it (#494). The official `postgres` image's
+ * entrypoint does read the `POSTGRES_INITDB_ARGS` env: `ensureCallsDatabase`
+ * sets it on the Synapse database before that database first starts. It
+ * cannot change a database whose data directory `initdb` already made; MAS
+ * has no such requirement, so the env goes on the Synapse database only. Not
+ * using Synapse's own `allow_unsafe_locale` escape hatch: that trades a loud
+ * startup refusal for silent index corruption after a future glibc upgrade.
  */
 const CALLS_SYNAPSE_DB_INITDB_ARGS = '--locale=C --encoding=UTF8';
 /**
@@ -3298,13 +3294,23 @@ export class FoundryStepExecutor {
      * database, whose {@link CALLS_SERVICE_DB_SCOPE_KEY} has no
      * `system:kind:key` slot of its own for the same reason.
      */
-    async ensureCallsDatabase(org, context, projectUuid, environment, name, postgresDb, scopeKey, postgresInitdbArgs) {
+    async ensureCallsDatabase(org, context, projectUuid, environment, name, postgresDb, scopeKey, initdbArgs) {
+        // A database found by id or name may be one a failed run created and never
+        // started (its env call threw), so the env is checked on every path. It
+        // only takes effect before the first start; the callers start it later
+        // (`getDatabaseStarted`).
         const existing = context.recordedIds()[scopeKey];
-        if (existing)
+        if (existing) {
+            if (initdbArgs)
+                await this.ensureDatabaseInitdbArgs(existing, initdbArgs);
             return existing;
+        }
         const found = (await this.ports.coolify.listDatabases()).find((d) => d.name === name && d.environment_id === environment.id);
-        if (found)
+        if (found) {
+            if (initdbArgs)
+                await this.ensureDatabaseInitdbArgs(found.uuid, initdbArgs);
             return found.uuid;
+        }
         const server = await this.resolveServer(org, context);
         const created = await this.ports.coolify.createPostgresDatabase({
             project_uuid: projectUuid,
@@ -3314,10 +3320,27 @@ export class FoundryStepExecutor {
             name,
             postgres_db: postgresDb,
             postgres_user: postgresDb,
-            ...(postgresInitdbArgs ? { postgres_initdb_args: postgresInitdbArgs } : {}),
         });
+        if (initdbArgs)
+            await this.ensureDatabaseInitdbArgs(created.uuid, initdbArgs);
         return created.uuid;
     }
+    /**
+     * Sets `POSTGRES_INITDB_ARGS` on a Postgres database, once: the official
+     * image's entrypoint passes it to `initdb`, which Coolify's own
+     * `postgres_initdb_args` field never does ({@link CALLS_SYNAPSE_DB_INITDB_ARGS}).
+     * `/databases/{uuid}/envs` is not a typed port method yet, so this goes
+     * through `raw`. Nothing here is a secret.
+     */
+    async ensureDatabaseInitdbArgs(dbId, args) {
+        const envs = (await this.ports.coolify.raw('GET', `/databases/${dbId}/envs`));
+        if (envs.some((e) => e.key === 'POSTGRES_INITDB_ARGS'))
+            return;
+        await this.ports.coolify.raw('POST', `/databases/${dbId}/envs`, {
+            key: 'POSTGRES_INITDB_ARGS',
+            value: args,
+        });
+    }
     /**
      * Reads back {@link CALLS_SECRET_ENV_KEYS} from the MAS app's own envs
      * before minting fresh ones -- this stack's only memory across a re-run

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wtfalch/foundry",
-  "version": "0.26.1",
+  "version": "0.26.2",
   "description": "Foundry's SDK: the declared org's types, the plan and apply contracts, and the state store's interface.",
   "repository": {
     "type": "git",