graphile-presigned-url-plugin 1.16.0 → 1.16.1

package/esm/physical-bucket.js

@@ -9,6 +9,7 @@
  * environment-level bucket standing in for a tenant's.
  */
 import { Logger } from '@pgpmjs/logger';
+import { recordPhysicalName } from 'graphile-storage-registry';
 import { withRequestPgClient } from './request-pg-client';
 import { s3FailureError } from './s3-failure';
 import { isS3BucketProvisioned, markS3BucketProvisioned } from './storage-module-cache';
@@ -108,12 +109,7 @@ export async function provisionAndRecordPhysicalBucket(options, withPgClient, st
     // guard keeps this idempotent and race-safe across concurrent first uploads.
     // The catalog-sync trigger on this UPDATE needs `jwt.claims.database_id`, so the
     // write runs under the resolved database claim (privileged role preserved).
-    await withRequestPgClient(withPgClient, { 'jwt.claims.database_id': databaseId }, (client) => client.query({
-        text: `UPDATE ${storageConfig.bucketsQualifiedName}
-             SET physical_name = $1
-             WHERE id = $2 AND physical_name IS NULL`,
-        values: [s3BucketName, bucket.id],
-    }));
+    await withRequestPgClient(withPgClient, { 'jwt.claims.database_id': databaseId }, (client) => recordPhysicalName((query) => client.query(query), storageConfig.bucketsQualifiedName, bucket.id, s3BucketName));
     bucket.physical_name = s3BucketName;
     log.info(`Recorded physical_name="${s3BucketName}" on bucket ${bucket.id}`);
     return s3BucketName;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "graphile-presigned-url-plugin",
-  "version": "1.16.0",
+  "version": "1.16.1",
   "description": "Presigned URL upload plugin for PostGraphile v5 — requestUploadUrl mutation and downloadUrl computed field",
   "author": "Constructive <developers@constructive.io>",
   "homepage": "https://github.com/constructive-io/constructive",
@@ -44,7 +44,7 @@
     "@aws-sdk/s3-request-presigner": "^3.1052.0",
     "@pgpmjs/logger": "^2.25.0",
     "@pgsql/quotes": "^18.2.4",
-    "graphile-storage-registry": "^0.1.0",
+    "graphile-storage-registry": "^0.1.1",
     "lru-cache": "^11.2.7",
     "mime-bytes": "^0.32.0"
   },
@@ -62,5 +62,5 @@
     "@types/node": "^22.19.11",
     "makage": "^0.3.0"
   },
-  "gitHead": "df6b6613c844b505813e5d02d3a4afd918b74eab"
+  "gitHead": "7e6cc68f5659d12ec852835a7d8fcca6b05a8f50"
 }

package/physical-bucket.js

@@ -15,6 +15,7 @@ exports.mintPhysicalBucketName = mintPhysicalBucketName;
 exports.resolveS3ForDatabase = resolveS3ForDatabase;
 exports.provisionAndRecordPhysicalBucket = provisionAndRecordPhysicalBucket;
 const logger_1 = require("@pgpmjs/logger");
+const graphile_storage_registry_1 = require("graphile-storage-registry");
 const request_pg_client_1 = require("./request-pg-client");
 const s3_failure_1 = require("./s3-failure");
 const storage_module_cache_1 = require("./storage-module-cache");
@@ -114,12 +115,7 @@ async function provisionAndRecordPhysicalBucket(options, withPgClient, storageCo
     // guard keeps this idempotent and race-safe across concurrent first uploads.
     // The catalog-sync trigger on this UPDATE needs `jwt.claims.database_id`, so the
     // write runs under the resolved database claim (privileged role preserved).
-    await (0, request_pg_client_1.withRequestPgClient)(withPgClient, { 'jwt.claims.database_id': databaseId }, (client) => client.query({
-        text: `UPDATE ${storageConfig.bucketsQualifiedName}
-             SET physical_name = $1
-             WHERE id = $2 AND physical_name IS NULL`,
-        values: [s3BucketName, bucket.id],
-    }));
+    await (0, request_pg_client_1.withRequestPgClient)(withPgClient, { 'jwt.claims.database_id': databaseId }, (client) => (0, graphile_storage_registry_1.recordPhysicalName)((query) => client.query(query), storageConfig.bucketsQualifiedName, bucket.id, s3BucketName));
     bucket.physical_name = s3BucketName;
     log.info(`Recorded physical_name="${s3BucketName}" on bucket ${bucket.id}`);
     return s3BucketName;