@adobe/helix-config 5.0.0 → 5.1.1

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [5.1.1](https://github.com/adobe/helix-config/compare/v5.1.0...v5.1.1) (2025-04-05)
+
+
+### Bug Fixes
+
+* resolve groups correctly ([#258](https://github.com/adobe/helix-config/issues/258)) ([7971230](https://github.com/adobe/helix-config/commit/797123067f6028828657eececfc48fbf1d6ae484))
+
+# [5.1.0](https://github.com/adobe/helix-config/compare/v5.0.0...v5.1.0) (2025-04-04)
+
+
+### Features
+
+* support group references w/o .json ([#257](https://github.com/adobe/helix-config/issues/257)) ([013cb2a](https://github.com/adobe/helix-config/commit/013cb2a6f11cf8881966f97470c2a8eaeee81eb4)), closes [#256](https://github.com/adobe/helix-config/issues/256)
+
 # [5.0.0](https://github.com/adobe/helix-config/compare/v4.14.4...v5.0.0) (2025-04-03)
 
 

package/package.json

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

package/src/config-view.js

@@ -406,15 +406,21 @@ function computeSiteAdminRoles(admin, orgConfig, configGroups = {}) {
     for (const /* @type string */ entry of role) {
       if (entry.indexOf('@') > 0) {
         users.add(entry);
-      } else if (entry.startsWith('groups/') && entry.endsWith('.json')) {
+      } else if (entry.startsWith('groups/')) {
         // keep entry in list so that admin can auto-migrate updates to the groups json
         // TODO: remove after v4 EOL
         users.add(entry);
-        for (const email of resolveGroup(configGroups, entry.substring(7, entry.length - 5))) {
+        const reference = entry.endsWith('.json')
+          ? entry.substring(7, entry.length - 5)
+          : entry.substring(7);
+        for (const email of resolveGroup(configGroups, reference)) {
           users.add(email);
         }
       } else if (entry.startsWith('/groups/')) {
-        for (const email of resolveGroup(orgGroups, entry.substring(8, entry.length - 5))) {
+        const reference = entry.endsWith('.json')
+          ? entry.substring(8, entry.length - 5)
+          : entry.substring(8);
+        for (const email of resolveGroup(orgGroups, reference)) {
           users.add(email);
         }
       }