vintasend 0.4.17 → 0.5.0-alpha2

package/dist/services/attachment-manager/base-attachment-manager.d.ts

@@ -1,21 +1,17 @@
-import type { AttachmentFile, AttachmentFileRecord, FileAttachment } from '../../types/attachment';
+import type { AttachmentFile, AttachmentFileRecord, FileAttachment, StorageIdentifiers } from '../../types/attachment';
 export declare abstract class BaseAttachmentManager {
     /**
      * Upload a file and return file record (reusable across notifications)
      */
     abstract uploadFile(file: FileAttachment, filename: string, contentType?: string): Promise<AttachmentFileRecord>;
     /**
-     * Get file metadata by ID
+     * Reconstruct AttachmentFile from storage identifiers
      */
-    abstract getFile(fileId: string): Promise<AttachmentFileRecord | null>;
+    abstract reconstructAttachmentFile(storageIdentifiers: StorageIdentifiers): AttachmentFile;
     /**
-     * Delete a file (only if not referenced by any notifications)
+     * Delete a file from storage using its identifiers
      */
-    abstract deleteFile(fileId: string): Promise<void>;
-    /**
-     * Reconstruct AttachmentFile from storage metadata
-     */
-    abstract reconstructAttachmentFile(storageMetadata: Record<string, unknown>): AttachmentFile;
+    abstract deleteFileByIdentifiers(storageIdentifiers: StorageIdentifiers): Promise<void>;
     /**
      * Detect content type from filename
      * Public to allow backends and custom implementations to use it

package/dist/services/attachment-manager/local-file-attachment-manager.d.ts

@@ -1,4 +1,4 @@
-import type { AttachmentFile, AttachmentFileRecord, FileAttachment } from '../../types/attachment';
+import type { AttachmentFile, AttachmentFileRecord, FileAttachment, StorageIdentifiers } from '../../types/attachment';
 import { BaseAttachmentManager } from './base-attachment-manager';
 /**
  * Configuration options for LocalFileAttachmentManager
@@ -56,7 +56,11 @@ export declare class LocalFileAttachmentManager extends BaseAttachmentManager {
      */
     deleteFile(fileId: string): Promise<void>;
     /**
-     * Reconstruct an AttachmentFile interface from stored metadata
+     * Delete a file from the local filesystem using storage identifiers
      */
-    reconstructAttachmentFile(storageMetadata: Record<string, unknown>): AttachmentFile;
+    deleteFileByIdentifiers(storageIdentifiers: StorageIdentifiers): Promise<void>;
+    /**
+     * Reconstruct an AttachmentFile interface from stored identifiers
+     */
+    reconstructAttachmentFile(storageIdentifiers: StorageIdentifiers): AttachmentFile;
 }

package/dist/services/attachment-manager/local-file-attachment-manager.js

@@ -119,7 +119,8 @@ class LocalFileAttachmentManager extends base_attachment_manager_1.BaseAttachmen
             contentType: finalContentType,
             size,
             checksum,
-            storageMetadata: {
+            storageIdentifiers: {
+                id: fileId,
                 path: filePath,
                 backend: 'local-filesystem',
             },
@@ -147,7 +148,8 @@ class LocalFileAttachmentManager extends base_attachment_manager_1.BaseAttachmen
                 contentType: metadata.contentType,
                 size: metadata.size,
                 checksum: metadata.checksum,
-                storageMetadata: {
+                storageIdentifiers: {
+                    id: metadata.id,
                     path: filePath,
                     backend: 'local-filesystem',
                 },
@@ -169,7 +171,8 @@ class LocalFileAttachmentManager extends base_attachment_manager_1.BaseAttachmen
                     contentType: 'application/octet-stream', // Unknown
                     size: stats.size,
                     checksum,
-                    storageMetadata: {
+                    storageIdentifiers: {
+                        id: fileId,
                         path: filePath,
                         backend: 'local-filesystem',
                     },
@@ -210,13 +213,28 @@ class LocalFileAttachmentManager extends base_attachment_manager_1.BaseAttachmen
         }
     }
     /**
-     * Reconstruct an AttachmentFile interface from stored metadata
+     * Delete a file from the local filesystem using storage identifiers
      */
-    reconstructAttachmentFile(storageMetadata) {
-        if (!storageMetadata.path || typeof storageMetadata.path !== 'string') {
-            throw new Error('Invalid storage metadata: missing path');
+    async deleteFileByIdentifiers(storageIdentifiers) {
+        if (storageIdentifiers.id && typeof storageIdentifiers.id === 'string') {
+            await this.deleteFile(storageIdentifiers.id);
+            return;
         }
-        const filePath = storageMetadata.path;
+        if (storageIdentifiers.path && typeof storageIdentifiers.path === 'string') {
+            const fileId = path.basename(storageIdentifiers.path);
+            await this.deleteFile(fileId);
+            return;
+        }
+        throw new Error('Invalid storage identifiers: missing id or path');
+    }
+    /**
+     * Reconstruct an AttachmentFile interface from stored identifiers
+     */
+    reconstructAttachmentFile(storageIdentifiers) {
+        if (!storageIdentifiers.path || typeof storageIdentifiers.path !== 'string') {
+            throw new Error('Invalid storage identifiers: missing path');
+        }
+        const filePath = storageIdentifiers.path;
         return new LocalAttachmentFile(filePath, this);
     }
 }

package/dist/services/notification-backends/base-notification-backend.d.ts

@@ -34,6 +34,19 @@ export interface BaseNotificationBackend<Config extends BaseNotificationTypeConf
      */
     injectLogger?(logger: BaseLogger): void;
     /**
+     * Store attachment file record in database.
+     * Called after AttachmentManager.uploadFile() returns storageIdentifiers.
+     * Backend persists file metadata and storage identifiers for later retrieval.
+     */
+    storeAttachmentFileRecord?(record: AttachmentFileRecord): Promise<void>;
+    /**
+     * Get attachment file record from database by ID.
+     * Returns the file metadata and storage identifiers needed to reconstruct file access.
+     * Used by AttachmentManager.reconstructAttachmentFile() to get file content.
+     */
+    getAttachmentFileRecord?(fileId: string): Promise<AttachmentFileRecord | null>;
+    /**
+     * @deprecated Use getAttachmentFileRecord instead.
      * Get an attachment file record by ID
      */
     getAttachmentFile?(fileId: string): Promise<AttachmentFileRecord | null>;
@@ -64,7 +77,8 @@ export interface BaseNotificationBackend<Config extends BaseNotificationTypeConf
  * Type guard to check if backend supports attachment operations
  */
 export declare function supportsAttachments<Config extends BaseNotificationTypeConfig>(backend: BaseNotificationBackend<Config>): backend is BaseNotificationBackend<Config> & {
-    getAttachmentFile(fileId: string): Promise<AttachmentFileRecord | null>;
+    storeAttachmentFileRecord(record: AttachmentFileRecord): Promise<void>;
+    getAttachmentFileRecord(fileId: string): Promise<AttachmentFileRecord | null>;
     findAttachmentFileByChecksum(checksum: string): Promise<AttachmentFileRecord | null>;
     deleteAttachmentFile(fileId: string): Promise<void>;
     getOrphanedAttachmentFiles(): Promise<AttachmentFileRecord[]>;

package/dist/services/notification-backends/base-notification-backend.js

@@ -5,7 +5,8 @@ exports.supportsAttachments = supportsAttachments;
  * Type guard to check if backend supports attachment operations
  */
 function supportsAttachments(backend) {
-    return (typeof backend.getAttachmentFile === 'function' &&
+    return (typeof backend.storeAttachmentFileRecord === 'function' &&
+        typeof backend.getAttachmentFileRecord === 'function' &&
         typeof backend.findAttachmentFileByChecksum === 'function' &&
         typeof backend.deleteAttachmentFile === 'function' &&
         typeof backend.getOrphanedAttachmentFiles === 'function' &&

package/dist/types/attachment.d.ts

@@ -18,15 +18,49 @@ export interface AttachmentFile {
     url(expiresIn?: number): Promise<string>;
     delete(): Promise<void>;
 }
+/**
+ * Generic storage identifiers returned by AttachmentManagers.
+ * Backends persist these as opaque data and pass them back for file reconstruction.
+ * Each AttachmentManager implementation defines its own specific structure.
+ *
+ * @example MedplumStorageIdentifiers
+ * ```typescript
+ * {
+ *   id: "media-123",
+ *   medplumBinaryId: "binary-456",
+ *   medplumMediaId: "media-123",
+ *   url: "Binary/binary-456"
+ * }
+ * ```
+ *
+ * @example S3StorageIdentifiers
+ * ```typescript
+ * {
+ *   id: "file-123",
+ *   awsS3Bucket: "my-bucket",
+ *   awsS3Key: "uploads/file-123.pdf",
+ *   awsS3Region: "us-east-1"
+ * }
+ * ```
+ */
+export interface StorageIdentifiers {
+    id: string;
+    [key: string]: unknown;
+}
+/**
+ * Complete file record returned after upload.
+ * Contains both file metadata and storage identifiers.
+ * Storage identifiers are opaque to backends - used only for reconstruction.
+ */
 export interface AttachmentFileRecord {
     id: string;
     filename: string;
     contentType: string;
     size: number;
     checksum: string;
-    storageMetadata: Record<string, unknown>;
     createdAt: Date;
     updatedAt: Date;
+    storageIdentifiers: StorageIdentifiers;
 }
 export interface StoredAttachment {
     id: string;
@@ -38,5 +72,5 @@ export interface StoredAttachment {
     createdAt: Date;
     file: AttachmentFile;
     description?: string;
-    storageMetadata: Record<string, unknown>;
+    storageMetadata: StorageIdentifiers;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vintasend",
-  "version": "0.4.17",
+  "version": "0.5.0-alpha2",
   "main": "dist/index.js",
   "files": [
     "dist"
@@ -19,6 +19,7 @@
     "release:bump": "node scripts/release-bump.js",
     "release:bump:patch": "node scripts/release-bump.js --bump=patch",
     "release:bump:minor": "node scripts/release-bump.js --bump=minor",
+    "release:bump:alpha": "node scripts/release-bump.js --bump=alpha",
     "release:publish": "node scripts/release-publish.js"
   },
   "devDependencies": {