@adobe/helix-config-storage 1.2.1 → 1.2.3

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.2.3](https://github.com/adobe/helix-config-storage/compare/v1.2.2...v1.2.3) (2024-08-24)
+
+
+### Bug Fixes
+
+* **deps:** update adobe fixes ([3be774d](https://github.com/adobe/helix-config-storage/commit/3be774d5c930c1c4e7186bd719af6ed026f26d7a))
+
+## [1.2.2](https://github.com/adobe/helix-config-storage/compare/v1.2.1...v1.2.2) (2024-08-22)
+
+
+### Bug Fixes
+
+* export required property check ([#7](https://github.com/adobe/helix-config-storage/issues/7)) ([de7b093](https://github.com/adobe/helix-config-storage/commit/de7b0933651f39c8e9068b7e190253243bc3272f))
+
 ## [1.2.1](https://github.com/adobe/helix-config-storage/compare/v1.2.0...v1.2.1) (2024-08-22)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-config-storage",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "Helix Config Storage",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -43,15 +43,15 @@
     "ajv-cli": "5.0.0",
     "c8": "10.1.2",
     "eslint": "8.57.0",
-    "husky": "9.1.4",
-    "json-schema-to-typescript": "15.0.0",
+    "husky": "9.1.5",
+    "json-schema-to-typescript": "15.0.1",
     "junit-report-builder": "5.0.0",
-    "lint-staged": "15.2.8",
+    "lint-staged": "15.2.9",
     "mocha": "10.7.3",
     "mocha-multi-reporters": "1.5.1",
     "mocha-suppress-logs": "0.5.1",
-    "nock": "13.5.4",
-    "semantic-release": "24.0.0",
+    "nock": "13.5.5",
+    "semantic-release": "24.1.0",
     "xml2js": "0.6.2"
   },
   "lint-staged": {
@@ -60,12 +60,12 @@
   },
   "dependencies": {
     "@adobe/fetch": "4.1.8",
-    "@adobe/helix-shared-config": "10.6.6",
+    "@adobe/helix-shared-config": "10.6.7",
     "@adobe/helix-shared-git": "3.0.13",
-    "@adobe/helix-shared-storage": "1.0.5",
+    "@adobe/helix-shared-storage": "1.0.6",
     "@adobe/helix-shared-utils": "3.0.2",
     "ajv": "8.17.1",
     "ajv-formats": "3.0.1",
-    "jose": "5.6.3"
+    "jose": "5.7.0"
   }
 }

package/src/config-merge.js

@@ -43,22 +43,31 @@ const FORCED_TYPES = {
 /**
  * properties that are required by the schema and must not be removed for empty values
  */
-const REQUIRED_PROPERTIES = {
-  '.cdn.prod.host': true,
-  '.cdn.prod.type': true,
-  '.cdn.prod.endpoint': true,
-  '.cdn.prod.clientSecret': true,
-  '.cdn.prod.clientToken': true,
-  '.cdn.prod.accessToken': true,
-  '.cdn.prod.plan': true,
-  '.cdn.prod.zoneId': true,
-  '.cdn.prod.apiToken': true,
-  '.cdn.prod.distributionId': true,
-  '.cdn.prod.accessKeyId': true,
-  '.cdn.prod.secretAccessKey': true,
-  '.cdn.prod.serviceId': true,
-  '.cdn.prod.authToken': true,
-};
+const REQUIRED_PROPERTIES = [
+  '.cdn.prod.host',
+  '.cdn.prod.type',
+  '.cdn.prod.endpoint',
+  '.cdn.prod.clientSecret',
+  '.cdn.prod.clientToken',
+  '.cdn.prod.accessToken',
+  '.cdn.prod.plan',
+  '.cdn.prod.zoneId',
+  '.cdn.prod.apiToken',
+  '.cdn.prod.distributionId',
+  '.cdn.prod.accessKeyId',
+  '.cdn.prod.secretAccessKey',
+  '.cdn.prod.serviceId',
+  '.cdn.prod.authToken',
+];
+
+/**
+ * properties that are required by the schema and must not be removed for empty values
+ * @param propertyPath
+ * @returns {boolean}
+ */
+export function isRequiredProperty(propertyPath) {
+  return REQUIRED_PROPERTIES.includes(propertyPath);
+}
 
 /**
  * Merges the `src` object into the `dst` object.
@@ -75,7 +84,7 @@ const REQUIRED_PROPERTIES = {
 function merge(dst, src, path, isModifier) {
   for (const [key, value] of Object.entries(src)) {
     const itemPath = `${path}.${key}`;
-    if (value === '' && !REQUIRED_PROPERTIES[itemPath]) {
+    if (value === '' && !isRequiredProperty(itemPath)) {
       // remove key if value is empty string (keep false, 0)
       delete dst[key];
     } else if (typeof value !== 'object') {