Merge pull request #1073 from constructive-io/fix/resolve-storage-config-codec-name

fix(presigned-url): codec table name matching + ownerId type resolution

Author: pyramation

graphile/graphile-presigned-url-plugin/src/plugin.ts

@@ -184,6 +184,12 @@ export function createPresignedUrlPlugin(
                 continue;
               }
 
+              // Resolve the GraphQL type for ownerId from the codec's attribute codec
+              // (e.g. UUID scalar instead of String) so Grafast's type matching works.
+              const ownerIdType = hasOwnerId
+                ? (build as any).getGraphQLTypeByPgCodec(codec.attributes.owner_id.codec, 'input')
+                : null;
+
               log.debug(`Adding mutation entry point "${fieldName}" for bucket type ${typeName} (entity-scoped=${hasOwnerId})`);
 
               newFields[fieldName] = context.fieldWithHooks(
@@ -194,7 +200,7 @@ export function createPresignedUrlPlugin(
                   args: {
                     key: { type: new GraphQLNonNull(GraphQLString), description: 'Bucket key (e.g., "public", "private")' },
                     ...(hasOwnerId
-                      ? { ownerId: { type: new GraphQLNonNull(GraphQLString), description: 'Owner entity ID (required for entity-scoped buckets)' } }
+                      ? { ownerId: { type: new GraphQLNonNull(ownerIdType || GraphQLString), description: 'Owner entity ID (required for entity-scoped buckets)' } }
                       : {}),
                   },
                   plan(_$mutation: any, fieldArgs: any) {

graphile/graphile-presigned-url-plugin/src/storage-module-cache.ts

@@ -367,11 +367,11 @@ export async function loadAllStorageModules(
  * @returns The matching StorageModuleConfig or null
  */
 export function resolveStorageConfigFromCodec(
-  pgCodec: { name: string; extensions?: { pg?: { schemaName?: string } }; sqlType?: string },
+  pgCodec: { name: string; extensions?: { pg?: { schemaName?: string; name?: string } }; sqlType?: string },
   allConfigs: StorageModuleConfig[],
 ): StorageModuleConfig | null {
   const schemaName = pgCodec.extensions?.pg?.schemaName;
-  const tableName = pgCodec.name;
+  const tableName = pgCodec.extensions?.pg?.name ?? pgCodec.name;
 
   if (!schemaName || !tableName) return null;