@firestartr/cli 1.49.0-snapshot-8 → 1.49.0-snapshot-9

package/build/index.js

@@ -302667,15 +302667,20 @@ function validatePermissionsUniqueness(crs) {
             const permissions = cr.spec.permissions;
             cdk8s_renderer_src_logger.debug(`Validating Permissions Uniqueness of ${crkey}`);
             if (permissions) {
-                const seen = new Set();
+                const rolesByIdentifier = new Map();
                 for (const perm of permissions) {
                     const identifier = perm.collaborator
                         ? perm.collaborator
                         : `ref:${perm.ref.kind}:${perm.ref.name}`;
-                    if (seen.has(identifier)) {
-                        throw new Error(`Duplicate permission reference found in FirestartrGithubRepository ${crkey}: ${identifier}\nFull CR: ${JSON.stringify(cr, null, 2)}`);
+                    const role = perm.role;
+                    if (rolesByIdentifier.has(identifier)) {
+                        const prevRole = rolesByIdentifier.get(identifier);
+                        if (prevRole !== role) {
+                            throw new Error(`Conflicting permission role in FirestartrGithubRepository ${crkey}: ${identifier} has roles "${prevRole}" and "${role}".\nFull CR: ${JSON.stringify(cr, null, 2)}`);
+                        }
+                        continue;
                     }
-                    seen.add(identifier);
+                    rolesByIdentifier.set(identifier, role);
                 }
             }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@firestartr/cli",
-  "version": "1.49.0-snapshot-8",
+  "version": "1.49.0-snapshot-9",
   "private": false,
   "description": "Commandline tool",
   "main": "build/main.js",