@adobe/helix-config-storage 2.0.3 → 2.0.4

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [2.0.4](https://github.com/adobe/helix-config-storage/compare/v2.0.3...v2.0.4) (2025-03-04)
+
+
+### Bug Fixes
+
+* relax features schema ([#100](https://github.com/adobe/helix-config-storage/issues/100)) ([558a046](https://github.com/adobe/helix-config-storage/commit/558a046a36363bc9c80cc1879209f4bf4a709dad))
+
 ## [2.0.3](https://github.com/adobe/helix-config-storage/compare/v2.0.2...v2.0.3) (2025-03-04)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-config-storage",
-  "version": "2.0.3",
+  "version": "2.0.4",
   "description": "Helix Config Storage",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/config-store.js

@@ -11,7 +11,6 @@
  */
 /* eslint-disable no-param-reassign */
 import crypto from 'crypto';
-import { isDeepStrictEqual } from 'util';
 import { HelixStorage } from '@adobe/helix-shared-storage';
 import { StatusCodeError } from './status-code-error.js';
 import {
@@ -19,7 +18,7 @@ import {
   migrateToken,
   updateCodeSource,
   updateContentSource,
-  deepGetOrCreate, deepPut, prune, createSecret, migrateSecret,
+  deepGetOrCreate, deepPut, prune, createSecret, migrateSecret, isDeepEqual,
 } from './utils.js';
 import { validate as validateSchema } from './config-validator.js';
 import { getMergedConfig } from './config-merge.js';
@@ -438,12 +437,12 @@ export class ConfigStore {
     if (this.type !== 'org') {
       const oldFeatures = oldConfig?.features ?? {};
       const newFeatures = newConfig?.features ?? {};
-      if (!isDeepStrictEqual(oldFeatures, newFeatures)) {
+      if (!isDeepEqual(oldFeatures, newFeatures)) {
         throw new StatusCodeError(403, 'not allowed to modify features');
       }
       const oldLimits = oldConfig?.limits ?? {};
       const newLimits = newConfig?.limits ?? {};
-      if (!isDeepStrictEqual(oldLimits, newLimits)) {
+      if (!isDeepEqual(oldLimits, newLimits)) {
         throw new StatusCodeError(403, 'not allowed to modify limits');
       }
     }
@@ -505,7 +504,7 @@ export class ConfigStore {
     const frag = getFragmentInfo(this.type, relPath);
     let config = data;
     // set config to null if empty object
-    if (isDeepStrictEqual(config, {})) {
+    if (isDeepEqual(config, {})) {
       config = null;
     }
 

package/src/schemas/features.schema.json

@@ -8,14 +8,6 @@
   "patternProperties": {
     "^[a-zA-Z0-9-_=]+$": {
       "type": "object",
-      "properties": {
-          "enabled": {
-            "type": "boolean"
-          },
-          "description": {
-            "type": "string"
-          }
-      },
       "required": [
           "enabled"
       ],

package/src/utils.js

@@ -305,3 +305,14 @@ export function deepPut(obj, path, value) {
   }
   return obj;
 }
+
+/**
+ * Lazy implementation of deep equals. Note that isDeepStrictEquals() is not suited for our case,
+ * as object with null prototypes are not equal to objects with default prototype.
+ * @param {object} o0
+ * @param {object} o1
+ * @returns {boolean}
+ */
+export function isDeepEqual(o0, o1) {
+  return JSON.stringify(o0) === JSON.stringify(o1);
+}