@onlineapps/content-resolver 1.1.5 → 1.1.8

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@onlineapps/content-resolver",
-  "version": "1.1.5",
+  "version": "1.1.8",
   "description": "Automatic conversion between text content and storage references with Content Descriptor pattern",
   "main": "src/index.js",
   "scripts": {
-    "test": "jest"
+    "test": "jest --passWithNoTests"
   },
   "keywords": [
     "content",
@@ -16,9 +16,11 @@
   "author": "OnlineApps",
   "license": "ISC",
   "dependencies": {
-    "@onlineapps/conn-base-storage": "^1.0.0"
+    "@onlineapps/conn-base-storage": "1.0.4",
+    "@onlineapps/runtime-config": "1.0.2"
   },
   "peerDependencies": {
     "@onlineapps/conn-base-storage": "^1.0.0"
-  }
+  },
+  "devDependencies": {}
 }

package/src/config.js

@@ -0,0 +1,35 @@
+'use strict';
+
+/**
+ * Runtime configuration schema for @onlineapps/content-resolver.
+ *
+ * Uses @onlineapps/runtime-config for unified priority:
+ * 1. Explicit config (passed to constructor)
+ * 2. Environment variable
+ * 3. Module-owned defaults (from ./defaults.js)
+ *
+ * IMPORTANT: MinIO topology is FAIL-FAST (no defaults).
+ */
+
+const { createRuntimeConfig } = require('@onlineapps/runtime-config');
+const DEFAULTS = require('./defaults');
+
+const runtimeCfg = createRuntimeConfig({
+  defaults: DEFAULTS,
+  schema: {
+    threshold: { env: 'CONTENT_RESOLVER_THRESHOLD', defaultKey: 'thresholdBytes', type: 'number' },
+
+    // MinIO topology (FAIL-FAST)
+    endPoint: { env: 'MINIO_ENDPOINT', required: true },
+    port: { env: 'MINIO_PORT', required: true, type: 'number' },
+    accessKey: { env: 'MINIO_ACCESS_KEY', required: true },
+    secretKey: { env: 'MINIO_SECRET_KEY', required: true },
+    useSSL: { env: 'MINIO_USE_SSL', defaultKey: 'useSSL', type: 'boolean' },
+
+    defaultBucket: { env: 'MINIO_DEFAULT_BUCKET', defaultKey: 'defaultBucket' },
+  }
+});
+
+module.exports = runtimeCfg;
+
+

package/src/defaults.js

@@ -0,0 +1,15 @@
+'use strict';
+
+/**
+ * Module-owned defaults for @onlineapps/content-resolver.
+ *
+ * NOTE: Infrastructure topology (MinIO endpoint/credentials) has NO defaults.
+ */
+
+module.exports = {
+  thresholdBytes: 16 * 1024,
+  defaultBucket: 'workflow',
+  useSSL: false,
+};
+
+

package/src/index.js

@@ -11,9 +11,7 @@
 
 const StorageConnector = require('@onlineapps/conn-base-storage');
 const crypto = require('crypto');
-
-// Default threshold: 16KB (16384 bytes)
-const DEFAULT_THRESHOLD = 16 * 1024;
+const runtimeCfg = require('./config');
 
 // Reference patterns
 const MINIO_REF_PATTERN = /^minio:\/\/([^/]+)\/(.+)$/;
@@ -88,7 +86,7 @@ function parseReference(ref) {
   const internalMatch = ref.match(INTERNAL_REF_PATTERN);
   if (internalMatch) {
     // Internal references use default bucket
-    return { bucket: 'workflow', path: internalMatch[1] };
+    return { bucket: runtimeCfg.get('defaultBucket'), path: internalMatch[1] };
   }
   
   return null;
@@ -106,20 +104,26 @@ class ContentResolver {
    * @param {Object} [options.logger] - Logger instance
    */
   constructor(options = {}) {
-    this.threshold = options.threshold || DEFAULT_THRESHOLD;
+    const resolved = runtimeCfg.resolve(options);
+    this.threshold = resolved.threshold;
     this.logger = options.logger || console;
     
-    // Storage can be passed as instance or config
+    // Storage can be passed as instance or config (no fallbacks - fail-fast)
     if (options.storage instanceof StorageConnector) {
       this.storage = options.storage;
+    } else if (options.storage) {
+      // Config passed explicitly
+      this.storageConfig = options.storage;
+      this.storage = null;
     } else {
-      this.storageConfig = options.storage || {
-        endPoint: process.env.MINIO_HOST || 'api_shared_storage',
-        port: parseInt(process.env.MINIO_PORT || '9000'),
-        useSSL: false,
-        accessKey: process.env.MINIO_ACCESS_KEY || 'minioadmin',
-        secretKey: process.env.MINIO_SECRET_KEY || 'minioadmin',
-        defaultBucket: 'workflow'
+      // Must have runtime config (resolved from env) if no explicit storage config passed
+      this.storageConfig = {
+        endPoint: resolved.endPoint,
+        port: resolved.port,
+        useSSL: resolved.useSSL,
+        accessKey: resolved.accessKey,
+        secretKey: resolved.secretKey,
+        defaultBucket: resolved.defaultBucket
       };
       this.storage = null;
     }