@jaypie/constructs 1.2.49 → 1.2.51

package/dist/esm/JaypieGitHubDeployRole.d.ts

@@ -1,9 +1,11 @@
 import { Role } from "aws-cdk-lib/aws-iam";
 import { Construct } from "constructs";
 export interface JaypieGitHubDeployRoleProps {
+    ecr?: boolean;
     oidcProviderArn?: string;
     output?: boolean | string;
     repoRestriction?: string;
+    sponsor?: string;
 }
 export declare class JaypieGitHubDeployRole extends Construct {
     private readonly _role;

package/dist/esm/__tests__/JaypieGitHubDeployRole.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/esm/index.js

@@ -945,12 +945,16 @@ function exportEnvName$1(name, env = process.env) {
 }
 class JaypieEnvSecret extends Construct {
     constructor(scope, idOrEnvKey, props) {
-        // Check if idOrEnvKey should be treated as envKey:
-        // - No props provided OR props.envKey is not set
-        // - AND idOrEnvKey exists as a non-empty string in process.env
+        // Shorthand detection: treat idOrEnvKey as envKey when envKey prop is
+        // not set and idOrEnvKey either looks like a SCREAMING_SNAKE_CASE env
+        // var name or is already present in process.env. Convention-based
+        // detection ensures missing env vars still go through envKey validation
+        // instead of silently creating an empty secret.
+        const looksLikeEnvKey = /^[A-Z][A-Z0-9_]*$/.test(idOrEnvKey);
         const treatAsEnvKey = (!props || props.envKey === undefined) &&
-            typeof process.env[idOrEnvKey] === "string" &&
-            process.env[idOrEnvKey] !== "";
+            (looksLikeEnvKey ||
+                (typeof process.env[idOrEnvKey] === "string" &&
+                    process.env[idOrEnvKey] !== ""));
         const id = treatAsEnvKey ? `EnvSecret_${idOrEnvKey}` : idOrEnvKey;
         super(scope, id);
         const { consumer = checkEnvIsConsumer$1(), envKey: envKeyProp, export: exportParam, generateSecretString, provider = checkEnvIsProvider$1(), removalPolicy, roleTag, vendorTag, value, } = props || {};
@@ -3259,23 +3263,33 @@ class JaypieExpressLambda extends JaypieLambda {
     }
 }
 
+const ECR_PUSH_ACTIONS = [
+    "ecr:BatchCheckLayerAvailability",
+    "ecr:BatchGetImage",
+    "ecr:CompleteLayerUpload",
+    "ecr:CreateRepository",
+    "ecr:DescribeRepositories",
+    "ecr:InitiateLayerUpload",
+    "ecr:PutImage",
+    "ecr:UploadLayerPart",
+];
 class JaypieGitHubDeployRole extends Construct {
     constructor(scope, id = "GitHubDeployRole", props = {}) {
         super(scope, id);
-        const { oidcProviderArn = Fn.importValue(CDK$2.IMPORT.OIDC_PROVIDER), output = true, repoRestriction: propsRepoRestriction, } = props;
+        const { ecr = true, oidcProviderArn = Fn.importValue(CDK$2.IMPORT.OIDC_PROVIDER), output = true, repoRestriction: propsRepoRestriction, sponsor: propsSponsor, } = props;
         // Extract account ID from the scope
         const accountId = Stack.of(this).account;
-        // Resolve repoRestriction from props or environment variables
+        // Resolve repoRestriction and sponsor from props or environment variables
+        const envRepo = process.env.CDK_ENV_REPO || process.env.PROJECT_REPO;
+        const envRepoOrganization = envRepo ? envRepo.split("/")[0] : undefined;
         let repoRestriction = propsRepoRestriction;
         if (!repoRestriction) {
-            const envRepo = process.env.CDK_ENV_REPO || process.env.PROJECT_REPO;
-            if (!envRepo) {
+            if (!envRepoOrganization) {
                 throw new ConfigurationError("No repoRestriction provided. Set repoRestriction prop, CDK_ENV_REPO, or PROJECT_REPO environment variable");
             }
-            // Extract organization from owner/repo format and create org-wide restriction
-            const organization = envRepo.split("/")[0];
-            repoRestriction = `repo:${organization}/*:*`;
+            repoRestriction = `repo:${envRepoOrganization}/*:*`;
         }
+        const sponsor = propsSponsor || process.env.PROJECT_SPONSOR || envRepoOrganization;
         // Create the IAM role
         this._role = new Role(this, "GitHubActionsRole", {
             assumedBy: new FederatedPrincipal(oidcProviderArn, {
@@ -3323,6 +3337,22 @@ class JaypieGitHubDeployRole extends Construct {
                 "arn:aws:iam::*:role/cdk-readOnlyRole",
             ],
         }));
+        // Grant ECR auth + push scoped to <sponsor>-* repositories
+        if (ecr) {
+            if (!sponsor) {
+                throw new ConfigurationError("Cannot grant default ECR permissions without a sponsor. Set sponsor prop, PROJECT_SPONSOR, CDK_ENV_REPO, or PROJECT_REPO, or pass `ecr: false`");
+            }
+            this._role.addToPolicy(new PolicyStatement({
+                actions: ["ecr:GetAuthorizationToken"],
+                effect: Effect.ALLOW,
+                resources: ["*"],
+            }));
+            this._role.addToPolicy(new PolicyStatement({
+                actions: ECR_PUSH_ACTIONS,
+                effect: Effect.ALLOW,
+                resources: [`arn:aws:ecr:*:${accountId}:repository/${sponsor}-*`],
+            }));
+        }
         // Export the ARN of the role
         if (output !== false) {
             const outputId = typeof output === "string" ? output : "GitHubActionsRoleArn";