@byline/db-postgres 3.1.0 → 3.2.0

package/dist/modules/storage/storage-queries.d.ts

@@ -100,8 +100,9 @@ export declare class DocumentQueries implements IDocumentQueries {
     /**
      * Batch-fetch the version-locale availability sets from the
      * `byline_document_version_locales` ledger. For each version returns the
-     * concrete locales its content is complete in (`availableLocales`, sorted),
-     * or `localeAgnostic: true` when the version carries only the `'all'`
+     * concrete locales its content is complete in (`availableLocales`, in
+     * configured content-locale order), or `localeAgnostic: true` when the
+     * version carries only the `'all'`
      * sentinel (no localized content → renders identically in every locale).
      * Drives the `_availableVersionLocales` read metadata. One indexed query per call.
      */
@@ -109,8 +110,9 @@ export declare class DocumentQueries implements IDocumentQueries {
     /**
      * Batch-fetch the editorial advertised-locale sets from
      * `byline_document_available_locales` (document-grain). For each logical
-     * document returns the sorted set of locales the editor has elected to
-     * advertise. Surfaced on reads as `availableLocales` — the deliberate
+     * document returns the set of locales the editor has elected to advertise,
+     * in configured content-locale order. Surfaced on reads as
+     * `availableLocales` — the deliberate
      * counterpart to the version-grain `_availableVersionLocales` ledger fact;
      * the public advertised set is their intersection. One indexed query per
      * call. See docs/I18N.md.

package/dist/modules/storage/storage-queries.js

@@ -9,7 +9,7 @@
 // constructs query/command classes before initBylineCore() wires up the Pino
 // logger. A future refactor could inject the logger at construction time by
 // either deferring adapter construction or accepting a lazy logger parameter.
-import { ERR_DATABASE, ERR_NOT_FOUND, getLogger } from '@byline/core';
+import { ERR_DATABASE, ERR_NOT_FOUND, getLogger, orderByContentLocale } from '@byline/core';
 import { and, desc, eq, inArray, isNotNull, sql } from 'drizzle-orm';
 import { collections, currentDocumentsView, currentPublishedDocumentsView, documentAvailableLocales, documentPaths, documents, documentVersionLocales, documentVersions, metaStore, } from '../../database/schema/index.js';
 import { extractFlattenedFieldValue, restoreFieldSetData } from './storage-restore.js';
@@ -133,8 +133,9 @@ export class DocumentQueries {
     /**
      * Batch-fetch the version-locale availability sets from the
      * `byline_document_version_locales` ledger. For each version returns the
-     * concrete locales its content is complete in (`availableLocales`, sorted),
-     * or `localeAgnostic: true` when the version carries only the `'all'`
+     * concrete locales its content is complete in (`availableLocales`, in
+     * configured content-locale order), or `localeAgnostic: true` when the
+     * version carries only the `'all'`
      * sentinel (no localized content → renders identically in every locale).
      * Drives the `_availableVersionLocales` read metadata. One indexed query per call.
      */
@@ -160,15 +161,20 @@ export class DocumentQueries {
             else
                 entry.availableLocales.push(row.locale);
         }
-        for (const entry of result.values())
-            entry.availableLocales.sort();
+        // Stable, config-driven order — `_availableVersionLocales` is a set
+        // consumed via membership, so this just keeps its array order canonical
+        // (matching `availableLocales`). Falls back to a–z when no config.
+        for (const entry of result.values()) {
+            entry.availableLocales = orderByContentLocale(entry.availableLocales);
+        }
         return result;
     }
     /**
      * Batch-fetch the editorial advertised-locale sets from
      * `byline_document_available_locales` (document-grain). For each logical
-     * document returns the sorted set of locales the editor has elected to
-     * advertise. Surfaced on reads as `availableLocales` — the deliberate
+     * document returns the set of locales the editor has elected to advertise,
+     * in configured content-locale order. Surfaced on reads as
+     * `availableLocales` — the deliberate
      * counterpart to the version-grain `_availableVersionLocales` ledger fact;
      * the public advertised set is their intersection. One indexed query per
      * call. See docs/I18N.md.
@@ -192,8 +198,12 @@ export class DocumentQueries {
             }
             arr.push(row.locale);
         }
-        for (const arr of result.values())
-            arr.sort();
+        // Project `availableLocales` in configured content-locale order (the set
+        // is order-insensitive). Read-time only — nothing persisted changes, and
+        // unknown/removed codes sort last rather than throwing. Falls back to a–z
+        // when no server config is registered (e.g. isolated storage tests).
+        for (const [did, arr] of result)
+            result.set(did, orderByContentLocale(arr));
         return result;
     }
     /**

package/package.json

@@ -2,7 +2,7 @@
   "name": "@byline/db-postgres",
   "private": false,
   "license": "MPL-2.0",
-  "version": "3.1.0",
+  "version": "3.2.0",
   "engines": {
     "node": ">=20.9.0"
   },
@@ -57,9 +57,9 @@
     "pg": "^8.21.0",
     "uuid": "^14.0.0",
     "zod": "^4.4.3",
-    "@byline/core": "3.1.0",
-    "@byline/auth": "3.1.0",
-    "@byline/admin": "3.1.0"
+    "@byline/core": "3.2.0",
+    "@byline/admin": "3.2.0",
+    "@byline/auth": "3.2.0"
   },
   "devDependencies": {
     "@biomejs/biome": "2.4.15",