@fjall/components-infrastructure 3.4.1 → 3.6.0

package/dist/lib/patterns/aws/apexDomainPattern.js

@@ -1,5 +1,5 @@
 import { HostedZone } from "../../resources/aws/networking/hostedZone.js";
-import { DomainCertificate } from "../../resources/aws/networking/domainCertificate.js";
+import { composeDomainCertificates } from "./domainCertificateComposer.js";
 import { composeTypedDnsRecords } from "./dnsRecordComposer.js";
 import { toPascalCase, getSafeZoneName } from "../../utils/capitaliseString.js";
 /**
@@ -27,19 +27,17 @@ export function composeApexDomain(scope, props) {
         costAllocationEnvironment: props.costAllocationEnvironment,
         costAllocationDomain: props.zoneName
     });
-    const certificates = new Map();
-    (props.certificates ?? []).forEach((cert, index) => {
-        const normalised = normaliseCertificate(cert);
-        const safeCertName = toPascalCase(normalised.domainName.split(".").join(""));
-        const dc = new DomainCertificate(scope, `${safeZone}${safeCertName}Cert${index}`, {
-            domainName: normalised.domainName,
-            subjectAlternativeNames: normalised.subjectAlternativeNames,
-            transparencyLogging: normalised.transparencyLogging,
-            hostedZone: hostedZoneConstruct.hostedZone,
-            costAllocationEnvironment: props.costAllocationEnvironment,
-            costAllocationDomain: props.zoneName
-        });
-        certificates.set(normalised.domainName, dc.certificate);
+    // Regional entries mint beside the zone as before; cloudFront entries
+    // mint in us-east-1 (in-stack here, or in the domain-paired
+    // UsEast1Certificates stack) and publish the zone-level export (D3).
+    const certificates = composeDomainCertificates(scope, {
+        certificates: props.certificates,
+        effectiveZoneName: props.zoneName,
+        safeZone,
+        hostedZone: hostedZoneConstruct.hostedZone,
+        hostedZoneId: props.hostedZoneId,
+        costAllocationEnvironment: props.costAllocationEnvironment,
+        costAllocationDomain: props.zoneName
     });
     if (props.records && props.records.length > 0) {
         composeTypedDnsRecords(scope, hostedZoneConstruct.hostedZone, props.zoneName, props.records);
@@ -55,13 +53,3 @@ export function composeApexDomain(scope, props) {
         manualRecords: []
     };
 }
-function normaliseCertificate(cert) {
-    if (typeof cert === "string") {
-        return { domainName: cert };
-    }
-    return {
-        domainName: cert.domainName,
-        subjectAlternativeNames: cert.subjectAlternativeNames,
-        transparencyLogging: cert.transparencyLogging
-    };
-}

package/dist/lib/patterns/aws/buildkite/buildkite.d.ts

@@ -23,15 +23,24 @@ export declare class Buildkite extends Construct {
     readonly secretsBucketName: string;
     readonly autoScalingGroupName: string;
     constructor(scope: Construct, id: string, props: BuildkiteConstructProps);
+    /**
+     * Derive an agent-cluster secret path from the app namespace — the same
+     * `buildParameterPath` derivation `fjall secrets` and the ECS pattern use,
+     * so derived parameters stay manageable by the shared tooling. Explicit
+     * props bypass this entirely; it throws only when derivation is the last
+     * resort and no valid `applicationId` exists to derive from.
+     */
+    private deriveAgentSecretPath;
     /**
      * Ship the per-job `env` hook into the managed secrets bucket. The
      * s3-secrets-hooks plugin sources this file at the start of EVERY job, so
-     * FJALL_API_KEY is read fresh from SSM per job — rotation takes effect on
-     * the next job with no instance replacement (design § D4(iii)). The script
-     * is configuration, not a secret: shipping it through a CDK asset is fine;
-     * the secret VALUE only ever moves SSM → instance at job runtime.
+     * FJALL_API_KEY and the per-pipeline job secrets are read fresh from SSM
+     * per job — rotation takes effect on the next job with no instance
+     * replacement (design § D4(iii)). The script is configuration, not a
+     * secret: shipping it through a CDK asset is fine; secret VALUES only ever
+     * move SSM → instance at job runtime.
      */
-    private addFjallApiKeyEnvHook;
+    private addAgentSecretsEnvHook;
 }
 /**
  * Object key the s3-secrets-hooks plugin sources at the start of every job —
@@ -39,8 +48,44 @@ export declare class Buildkite extends Construct {
  */
 export declare const FJALL_ENV_HOOK_OBJECT_KEY = "env";
 /**
- * The per-job env hook's script content (design § D4(iii)): FJALL_API_KEY is
- * read fresh from SSM per job, so rotation takes effect on the next job with
- * no instance replacement. Pure — the unit test pins the exact content.
+ * Cluster segment of derived agent secret paths
+ * (`/<applicationId>/agents/<name>`) — the namespace the live estate already
+ * uses for its explicitly-provisioned parameters.
+ */
+export declare const BUILDKITE_SECRETS_CLUSTER = "agents";
+/** Derived secret name for the cluster-scoped Buildkite agent token. */
+export declare const AGENT_TOKEN_SECRET_NAME = "agent-token";
+/** Derived secret name for the FJALL_API_KEY env-hook parameter. */
+export declare const FJALL_API_KEY_SECRET_NAME = "fjall-api-key";
+/** Inputs to {@link buildAgentSecretsEnvHookScript}; omitted members skip their block. */
+export interface AgentSecretsEnvHookOptions {
+    /** Resolved FJALL_API_KEY parameter path (explicit prop or derived). */
+    readonly fjallApiKeyParameterName?: string;
+    /**
+     * Namespace root for per-pipeline job secrets (`/<applicationId>/agents`);
+     * the hook lists `<prefix>/<BUILDKITE_PIPELINE_SLUG>` per job. Jobs read
+     * the whole `/<app>/agents/*` subtree via the paired IAM grant — identical
+     * flatness to the previous whole-bucket S3 read; isolation stays per-fleet.
+     */
+    readonly pipelineSecretsPrefix?: string;
+    readonly region: string;
+}
+/**
+ * The per-job env hook's script content (design § D4(iii) + Phase B):
+ * FJALL_API_KEY and the per-pipeline job secrets are read fresh from SSM per
+ * job, so rotation takes effect on the next job with no instance replacement.
+ * Pure — the unit tests byte-pin both emitted variants and `sh -n` them.
+ *
+ * The script is eval'd by the s3secrets plugin under bash
+ * `set -Eeuo pipefail` + an ERR trap that exits 53 (fails the job) +
+ * `set -o allexport` (every assignment auto-exports) + `nounset` — pinned in
+ * `lib/__tests__/fixtures/buildkite/upstream-hooks-v6.68.1/`. Hence: every
+ * fallible command is guarded, every env read uses `${VAR:-}`, every temp
+ * variable is unset after use, and the script never exits. `private_ssh_key`
+ * loads into an ephemeral ssh-agent (stdin → agent memory, never disk or
+ * argv); the upstream pre-exit hook kills whatever `SSH_AGENT_PID` is
+ * exported, so our agent's lifecycle is owned upstream. Every fetched value
+ * is registered with the agent redactor (stdin form) before use — the
+ * upstream helper cannot see values we fetch at eval time.
  */
-export declare function buildFjallApiKeyEnvHookScript(parameterName: string, region: string): string;
+export declare function buildAgentSecretsEnvHookScript(options: AgentSecretsEnvHookOptions): string;

package/dist/lib/patterns/aws/buildkite/buildkite.js

@@ -1,4 +1,5 @@
-import { CfnOutput, Duration, RemovalPolicy, Stack, Token } from "aws-cdk-lib";
+import { Annotations, CfnOutput, Duration, RemovalPolicy, Stack, Token } from "aws-cdk-lib";
+import { SSM_COMPONENT_PATTERN, buildParameterPath } from "@fjall/util";
 import { Monitoring } from "aws-cdk-lib/aws-autoscaling";
 import { InstanceArchitecture, InstanceType, MachineImage, SubnetType } from "aws-cdk-lib/aws-ec2";
 import { PolicyStatement } from "aws-cdk-lib/aws-iam";
@@ -38,6 +39,37 @@ export class Buildkite extends Construct {
         super(scope, id);
         const { vpc, ...plainProps } = props;
         const config = validateBuildkiteProps(plainProps);
+        // Deliberate asymmetry: the agent token is load-bearing (boot + scaler),
+        // so underivable throws; the api-key hook and per-pipeline job secrets
+        // are capabilities, so underivable — including a pattern-invalid
+        // applicationId, which pre-derivation callers could always pass —
+        // degrades to "not shipped" with a warning.
+        const agentTokenParameterName = config.agentTokenSsmParameterName ??
+            this.deriveAgentSecretPath(config.applicationId, AGENT_TOKEN_SECRET_NAME);
+        const derivableApplicationId = config.applicationId !== undefined &&
+            config.applicationId !== "" &&
+            SSM_COMPONENT_PATTERN.test(config.applicationId)
+            ? config.applicationId
+            : undefined;
+        const fjallApiKeyParameterName = config.fjallApiKeySsmParameterName ??
+            (derivableApplicationId !== undefined
+                ? this.deriveAgentSecretPath(derivableApplicationId, FJALL_API_KEY_SECRET_NAME)
+                : undefined);
+        const pipelineSecretsPrefix = derivableApplicationId !== undefined
+            ? buildParameterPath({
+                app: derivableApplicationId,
+                cluster: BUILDKITE_SECRETS_CLUSTER
+            })
+            : undefined;
+        if (derivableApplicationId === undefined &&
+            config.applicationId !== undefined &&
+            config.applicationId !== "") {
+            Annotations.of(this).addWarningV2("fjall:buildkite:fjall-api-key-hook-skipped", `applicationId '${config.applicationId}' is not a valid SSM path ` +
+                `component, so namespace-derived agent secrets are disabled: ` +
+                `per-pipeline job secrets will not load, and the FJALL_API_KEY env ` +
+                `hook is not shipped unless fjallApiKeySsmParameterName is set ` +
+                `explicitly. Rename the app or set the parameter-name props.`);
+        }
         const stack = Stack.of(this);
         if (Token.isUnresolved(stack.region)) {
             throw new Error("Buildkite requires a concrete env region at synth (the pinned AMI " +
@@ -61,17 +93,27 @@ export class Buildkite extends Construct {
         // and accidental overwrite; the artifact bucket is scratch output and
         // keeps the wrapper's env-aware default (DESTROY + pre-empty off prod).
         const managedSecretsBucket = new S3Bucket(this, `${id}ManagedSecretsBucket`, { versioned: true, removalPolicy: RemovalPolicy.RETAIN });
-        if (config.fjallApiKeySsmParameterName !== undefined) {
-            this.addFjallApiKeyEnvHook(id, managedSecretsBucket, config.fjallApiKeySsmParameterName, region);
+        if (fjallApiKeyParameterName !== undefined ||
+            pipelineSecretsPrefix !== undefined) {
+            this.addAgentSecretsEnvHook(id, managedSecretsBucket, {
+                ...(fjallApiKeyParameterName !== undefined && {
+                    fjallApiKeyParameterName
+                }),
+                ...(pipelineSecretsPrefix !== undefined && { pipelineSecretsPrefix }),
+                region
+            });
         }
         const parameterArn = (name) => `arn:${stack.partition}:ssm:${region}:${stack.account}:parameter${name}`;
         const agentRole = buildAgentRole(this, `${id}AgentRole`, {
-            agentTokenParameterArn: parameterArn(config.agentTokenSsmParameterName),
+            agentTokenParameterArn: parameterArn(agentTokenParameterName),
             ...(config.agentTokenKmsKeyArn !== undefined && {
                 agentTokenKmsKeyArn: config.agentTokenKmsKeyArn
             }),
-            ...(config.fjallApiKeySsmParameterName !== undefined && {
-                fjallApiKeyParameterArn: parameterArn(config.fjallApiKeySsmParameterName)
+            ...(fjallApiKeyParameterName !== undefined && {
+                fjallApiKeyParameterArn: parameterArn(fjallApiKeyParameterName)
+            }),
+            ...(pipelineSecretsPrefix !== undefined && {
+                pipelineSecretsPrefixArn: parameterArn(pipelineSecretsPrefix)
             }),
             secretsBucketArn: managedSecretsBucket.bucketArn,
             artifactBucketArn: artifactBucket.bucketArn,
@@ -82,7 +124,8 @@ export class Buildkite extends Construct {
             stackName: stack.stackName,
             region,
             secretsBucketName: managedSecretsBucket.bucketName,
-            artifactBucketName: artifactBucket.bucketName
+            artifactBucketName: artifactBucket.bucketName,
+            agentTokenParameterName
         });
         // Deliberate deviation from upstream's InstanceScaleInProtection
         // (design § D7): the wrapper's `newInstancesProtectedFromScaleIn: false`
@@ -146,7 +189,7 @@ export class Buildkite extends Construct {
             applicationId: resolveScalerSarApplicationArn(architecture),
             semanticVersion: BUILDKITE_STACK_PINS.scalerVersion,
             parameters: {
-                BuildkiteAgentTokenParameter: config.agentTokenSsmParameterName,
+                BuildkiteAgentTokenParameter: agentTokenParameterName,
                 ...(config.agentTokenKmsKeyArn !== undefined && {
                     BuildkiteAgentTokenParameterStoreKMSKey: config.agentTokenKmsKeyArn
                 }),
@@ -201,18 +244,43 @@ export class Buildkite extends Construct {
             value: artifactBucket.bucketName
         });
     }
+    /**
+     * Derive an agent-cluster secret path from the app namespace — the same
+     * `buildParameterPath` derivation `fjall secrets` and the ECS pattern use,
+     * so derived parameters stay manageable by the shared tooling. Explicit
+     * props bypass this entirely; it throws only when derivation is the last
+     * resort and no valid `applicationId` exists to derive from.
+     */
+    deriveAgentSecretPath(applicationId, secretName) {
+        if (applicationId === undefined || applicationId === "") {
+            throw new Error(`Buildkite: no explicit SSM parameter name was set for '${secretName}' ` +
+                `and no applicationId is available to derive ` +
+                `'/<applicationId>/${BUILDKITE_SECRETS_CLUSTER}/${secretName}' from. ` +
+                `Construct via App.addBuildkite (which injects applicationId) or set ` +
+                `the parameter-name prop explicitly.`);
+        }
+        if (!SSM_COMPONENT_PATTERN.test(applicationId)) {
+            throw new Error(`Buildkite: cannot derive the '${secretName}' SSM parameter path — ` +
+                `applicationId '${applicationId}' is not a valid SSM path component ` +
+                `(must start with a letter; letters, numbers, periods, hyphens, ` +
+                `underscores only). Rename the app or set the parameter-name prop ` +
+                `explicitly.`);
+        }
+        return buildParameterPath({ app: applicationId, cluster: BUILDKITE_SECRETS_CLUSTER }, secretName);
+    }
     /**
      * Ship the per-job `env` hook into the managed secrets bucket. The
      * s3-secrets-hooks plugin sources this file at the start of EVERY job, so
-     * FJALL_API_KEY is read fresh from SSM per job — rotation takes effect on
-     * the next job with no instance replacement (design § D4(iii)). The script
-     * is configuration, not a secret: shipping it through a CDK asset is fine;
-     * the secret VALUE only ever moves SSM → instance at job runtime.
+     * FJALL_API_KEY and the per-pipeline job secrets are read fresh from SSM
+     * per job — rotation takes effect on the next job with no instance
+     * replacement (design § D4(iii)). The script is configuration, not a
+     * secret: shipping it through a CDK asset is fine; secret VALUES only ever
+     * move SSM → instance at job runtime.
      */
-    addFjallApiKeyEnvHook(id, secretsBucket, parameterName, region) {
+    addAgentSecretsEnvHook(id, secretsBucket, options) {
         new BucketDeployment(this, `${id}EnvHookDeployment`, {
             sources: [
-                Source.data(FJALL_ENV_HOOK_OBJECT_KEY, buildFjallApiKeyEnvHookScript(parameterName, region))
+                Source.data(FJALL_ENV_HOOK_OBJECT_KEY, buildAgentSecretsEnvHookScript(options))
             ],
             destinationBucket: secretsBucket,
             // The secrets bucket also holds out-of-band objects the deployment
@@ -230,17 +298,57 @@ export class Buildkite extends Construct {
  */
 export const FJALL_ENV_HOOK_OBJECT_KEY = "env";
 /**
- * The per-job env hook's script content (design § D4(iii)): FJALL_API_KEY is
- * read fresh from SSM per job, so rotation takes effect on the next job with
- * no instance replacement. Pure — the unit test pins the exact content.
+ * Cluster segment of derived agent secret paths
+ * (`/<applicationId>/agents/<name>`) — the namespace the live estate already
+ * uses for its explicitly-provisioned parameters.
+ */
+export const BUILDKITE_SECRETS_CLUSTER = "agents";
+/** Derived secret name for the cluster-scoped Buildkite agent token. */
+export const AGENT_TOKEN_SECRET_NAME = "agent-token";
+/** Derived secret name for the FJALL_API_KEY env-hook parameter. */
+export const FJALL_API_KEY_SECRET_NAME = "fjall-api-key";
+/**
+ * The per-job env hook's script content (design § D4(iii) + Phase B):
+ * FJALL_API_KEY and the per-pipeline job secrets are read fresh from SSM per
+ * job, so rotation takes effect on the next job with no instance replacement.
+ * Pure — the unit tests byte-pin both emitted variants and `sh -n` them.
+ *
+ * The script is eval'd by the s3secrets plugin under bash
+ * `set -Eeuo pipefail` + an ERR trap that exits 53 (fails the job) +
+ * `set -o allexport` (every assignment auto-exports) + `nounset` — pinned in
+ * `lib/__tests__/fixtures/buildkite/upstream-hooks-v6.68.1/`. Hence: every
+ * fallible command is guarded, every env read uses `${VAR:-}`, every temp
+ * variable is unset after use, and the script never exits. `private_ssh_key`
+ * loads into an ephemeral ssh-agent (stdin → agent memory, never disk or
+ * argv); the upstream pre-exit hook kills whatever `SSH_AGENT_PID` is
+ * exported, so our agent's lifecycle is owned upstream. Every fetched value
+ * is registered with the agent redactor (stdin form) before use — the
+ * upstream helper cannot see values we fetch at eval time.
  */
-export function buildFjallApiKeyEnvHookScript(parameterName, region) {
-    return [
-        `FJALL_API_KEY="$(aws ssm get-parameter --name '${parameterName}' --with-decryption --query Parameter.Value --output text --region '${region}')"`,
-        "export FJALL_API_KEY",
-        ""
-    ].join("\n");
+export function buildAgentSecretsEnvHookScript(options) {
+    const { fjallApiKeyParameterName, pipelineSecretsPrefix, region } = options;
+    const lines = ["set +x", "set +v"];
+    if (fjallApiKeyParameterName !== undefined) {
+        lines.push(`FJALL_API_KEY="$(aws ssm get-parameter --name '${fjallApiKeyParameterName}' --with-decryption --query Parameter.Value --output text --region '${region}' 2>/dev/null)" || true`, 'if [ -n "${FJALL_API_KEY:-}" ]; then', `  printf '%s' "$FJALL_API_KEY" | buildkite-agent redactor add >/dev/null 2>&1 || true`, "  export FJALL_API_KEY", "else", "  unset FJALL_API_KEY", `  echo 'fjall-secrets: could not fetch ${fjallApiKeyParameterName}' >&2`, "fi");
+    }
+    if (pipelineSecretsPrefix !== undefined) {
+        lines.push('if [ -n "${BUILDKITE_PIPELINE_SLUG:-}" ]; then', `  _fjall_names="$(aws ssm get-parameters-by-path --path "${pipelineSecretsPrefix}/\${BUILDKITE_PIPELINE_SLUG:-}" --query 'Parameters[].Name' --output text --region '${region}' 2>/dev/null)" || true`, "  for _fjall_param in ${_fjall_names:-}; do", '    _fjall_name="${_fjall_param##*/}"', `    _fjall_value="$(aws ssm get-parameter --name "$_fjall_param" --with-decryption --query Parameter.Value --output text --region '${region}' 2>/dev/null)" || true`, '    if [ -z "${_fjall_value:-}" ]; then', '      echo "fjall-secrets: could not fetch $_fjall_param" >&2', "      unset _fjall_value", "      continue", "    fi", `    printf '%s' "$_fjall_value" | buildkite-agent redactor add >/dev/null 2>&1 || true`, `    if [ "$_fjall_name" = 'private_ssh_key' ]; then`, `      _fjall_agent_out="$(ssh-agent -s 2>/dev/null)" || true`, '      if [ -n "${_fjall_agent_out:-}" ]; then', '        eval "$_fjall_agent_out" >/dev/null 2>&1 || true', `        printf '%s\\n' "$_fjall_value" | ssh-add - >/dev/null 2>&1 || echo 'fjall-secrets: ssh-add failed for private_ssh_key' >&2`, "      else", `        echo 'fjall-secrets: ssh-agent unavailable - private_ssh_key not loaded' >&2`, "      fi", "      unset _fjall_agent_out", `    elif printf '%s' "$_fjall_name" | grep -Eq '${JOB_ENV_NAME_ALLOW_REGEX}' && ! printf '%s' "$_fjall_name" | grep -Eq '${JOB_ENV_NAME_DENY_REGEX}'; then`, `      export "$_fjall_name=$_fjall_value" 2>/dev/null || echo "fjall-secrets: could not export $_fjall_name" >&2`, "    else", '      echo "fjall-secrets: skipping non-conforming secret name $_fjall_name" >&2', "    fi", "    unset _fjall_value", "  done", "  unset _fjall_names", "  unset _fjall_param", "  unset _fjall_name", "fi");
+    }
+    lines.push("");
+    return lines.join("\n");
 }
+/**
+ * Job-env eligibility for per-pipeline secret names: the allow shape is a
+ * portable shell identifier; the deny list removes the bash-readonly names
+ * (`export UID=…` fails "readonly variable" — under the upstream ERR trap
+ * that would exit-53 EVERY job on the pipeline until the parameter is
+ * deleted), the ssh-agent lifecycle vars the D-B3 flow owns, the
+ * loader-injection vector vars, and the `BUILDKITE_`/`AWS_` prefixes.
+ * `private_ssh_key` is handled before this filter; denied/non-conforming
+ * names are skipped with an identifier-only warning.
+ */
+const JOB_ENV_NAME_ALLOW_REGEX = "^[A-Za-z_][A-Za-z0-9_]*$";
+const JOB_ENV_NAME_DENY_REGEX = "^(PATH|HOME|IFS|LD_PRELOAD|LD_LIBRARY_PATH|BASH_ENV|SSH_AUTH_SOCK|SSH_AGENT_PID|UID|EUID|PPID|SHELLOPTS|BASHOPTS|BASH_VERSINFO)$|^(BUILDKITE_|AWS_)";
 function deriveCpuArchitecture(instanceTypeIdentifier) {
     const architecture = new InstanceType(instanceTypeIdentifier).architecture;
     return architecture === InstanceArchitecture.ARM_64 ? "arm64" : "amd64";

package/dist/lib/patterns/aws/buildkite/iam.d.ts

@@ -11,11 +11,18 @@ import { Role } from "../../../resources/aws/iam/index.js";
  * subset of this list, so an upstream bump (or a future edit) that grows
  * permissions fails loudly instead of shipping silently.
  */
-export declare const BUILDKITE_AGENT_IAM_ACTION_ALLOWLIST: readonly ["ssm:GetParameter", "kms:Decrypt", "s3:GetObject", "s3:ListBucket", "s3:GetObjectVersion", "s3:PutObject", "s3:PutObjectAcl", "s3:PutObjectVersionAcl", "logs:CreateLogGroup", "logs:CreateLogStream", "logs:PutLogEvents", "logs:DescribeLogGroups", "logs:DescribeLogStreams", "logs:PutRetentionPolicy", "ssm:DescribeInstanceProperties", "ssm:ListAssociations", "ssm:PutInventory", "ssm:UpdateInstanceInformation", "ssmmessages:CreateControlChannel", "ssmmessages:CreateDataChannel", "ssmmessages:OpenControlChannel", "ssmmessages:OpenDataChannel", "ec2messages:AcknowledgeMessage", "ec2messages:DeleteMessage", "ec2messages:FailMessage", "ec2messages:GetEndpoint", "ec2messages:GetMessages", "ec2messages:SendReply", "autoscaling:DescribeAutoScalingInstances", "autoscaling:SetInstanceHealth", "autoscaling:TerminateInstanceInAutoScalingGroup", "cloudwatch:PutMetricData", "cloudformation:DescribeStackResource", "ec2:DescribeTags"];
+export declare const BUILDKITE_AGENT_IAM_ACTION_ALLOWLIST: readonly ["ssm:GetParameter", "ssm:GetParametersByPath", "kms:Decrypt", "s3:GetObject", "s3:ListBucket", "s3:GetObjectVersion", "s3:PutObject", "s3:PutObjectAcl", "s3:PutObjectVersionAcl", "logs:CreateLogGroup", "logs:CreateLogStream", "logs:PutLogEvents", "logs:DescribeLogGroups", "logs:DescribeLogStreams", "logs:PutRetentionPolicy", "ssm:DescribeInstanceProperties", "ssm:ListAssociations", "ssm:PutInventory", "ssm:UpdateInstanceInformation", "ssmmessages:CreateControlChannel", "ssmmessages:CreateDataChannel", "ssmmessages:OpenControlChannel", "ssmmessages:OpenDataChannel", "ec2messages:AcknowledgeMessage", "ec2messages:DeleteMessage", "ec2messages:FailMessage", "ec2messages:GetEndpoint", "ec2messages:GetMessages", "ec2messages:SendReply", "autoscaling:DescribeAutoScalingInstances", "autoscaling:SetInstanceHealth", "autoscaling:TerminateInstanceInAutoScalingGroup", "cloudwatch:PutMetricData", "cloudformation:DescribeStackResource", "ec2:DescribeTags"];
 export interface BuildkiteAgentRoleParams {
     readonly agentTokenParameterArn: string;
     readonly agentTokenKmsKeyArn?: string;
     readonly fjallApiKeyParameterArn?: string;
+    /**
+     * `arn:…:parameter/<app>/agents` — per-pipeline job-secrets subtree root;
+     * grants list + read on `<root>/*`. Jobs can read the whole subtree
+     * (identical flatness to the whole-bucket S3 read this replaces);
+     * isolation stays per-fleet.
+     */
+    readonly pipelineSecretsPrefixArn?: string;
     readonly secretsBucketArn: string;
     readonly artifactBucketArn: string;
     /** Own-stack ARN — `cloudformation:DescribeStackResource` scope. */

package/dist/lib/patterns/aws/buildkite/iam.js

@@ -13,6 +13,9 @@ import { Role } from "../../../resources/aws/iam/index.js";
  */
 export const BUILDKITE_AGENT_IAM_ACTION_ALLOWLIST = [
     "ssm:GetParameter",
+    // Read-only, scoped to the /<app>/agents/* subtree (per-pipeline job
+    // secrets) — never account-wide.
+    "ssm:GetParametersByPath",
     "kms:Decrypt",
     "s3:GetObject",
     "s3:ListBucket",
@@ -77,6 +80,14 @@ function buildAgentPolicyDocuments(params) {
                             : [])
                     ]
                 }),
+                ...(params.pipelineSecretsPrefixArn !== undefined
+                    ? [
+                        new PolicyStatement({
+                            actions: ["ssm:GetParameter", "ssm:GetParametersByPath"],
+                            resources: [`${params.pipelineSecretsPrefixArn}/*`]
+                        })
+                    ]
+                    : []),
                 ...(params.agentTokenKmsKeyArn !== undefined
                     ? [
                         new PolicyStatement({

package/dist/lib/patterns/aws/buildkite/userData.d.ts

@@ -11,6 +11,13 @@ export interface BuildkiteUserDataContext {
     readonly region: string;
     readonly secretsBucketName: string;
     readonly artifactBucketName: string;
+    /**
+     * RESOLVED agent-token parameter path (explicit prop or namespace-derived)
+     * — never `props.agentTokenSsmParameterName`, which is optional since the
+     * derivation landed and would render the literal string "undefined" into
+     * BUILDKITE_AGENT_TOKEN_PATH through the template literal.
+     */
+    readonly agentTokenParameterName: string;
 }
 /**
  * The pinned AMI's boot scripts run under `set -Eeuo pipefail` and
@@ -42,7 +49,7 @@ export declare function buildBuildkiteDockerCommands(props: BuildkiteProps): str
  * `bk-install-elastic-stack.sh`, mirroring the upstream block key-for-key in
  * upstream order (so a fixture-bump diff reads line-by-line). Keys with no
  * schema knob carry the upstream v6.68.1 parameter default verbatim, with
- * four deliberate deviations:
+ * five deliberate deviations:
  *
  * - `BUILDKITE_S3_ACL`: our artifact bucket is BucketOwnerEnforced (ACLs
  *   disabled), where S3 rejects every canned ACL EXCEPT
@@ -54,6 +61,11 @@ export declare function buildBuildkiteDockerCommands(props: BuildkiteProps): str
  *   is out of scope for the Phase-1 deploy fleet; a future signing knob must
  *   land WITH the kms:Sign/Verify/GetPublicKey grants it needs (the prior
  *   knob shipped without them — a dead toggle that broke agents when set).
+ * - `BUILDKITE_SECRETS_PLUGIN_SKIP_SSH_KEY_NOT_FOUND_WARNING`: hard-pinned
+ *   `true` (upstream default `false`). Since the Phase-B migration the git
+ *   deploy key lives in SSM and is loaded by our env hook — the plugin's S3
+ *   `private_ssh_key` object is intentionally absent, so its per-job
+ *   not-found warning is pure noise.
  * - `ENABLE_EC2_LOG_RETENTION_POLICY`: hard-pinned `true` (upstream default
  *   `false`, "preserve all logs"). Fjall's log-group lifecycle posture is
  *   bounded retention everywhere: each booting instance applies

package/dist/lib/patterns/aws/buildkite/userData.js

@@ -149,7 +149,7 @@ export function buildBuildkiteDockerCommands(props) {
  * `bk-install-elastic-stack.sh`, mirroring the upstream block key-for-key in
  * upstream order (so a fixture-bump diff reads line-by-line). Keys with no
  * schema knob carry the upstream v6.68.1 parameter default verbatim, with
- * four deliberate deviations:
+ * five deliberate deviations:
  *
  * - `BUILDKITE_S3_ACL`: our artifact bucket is BucketOwnerEnforced (ACLs
  *   disabled), where S3 rejects every canned ACL EXCEPT
@@ -161,6 +161,11 @@ export function buildBuildkiteDockerCommands(props) {
  *   is out of scope for the Phase-1 deploy fleet; a future signing knob must
  *   land WITH the kms:Sign/Verify/GetPublicKey grants it needs (the prior
  *   knob shipped without them — a dead toggle that broke agents when set).
+ * - `BUILDKITE_SECRETS_PLUGIN_SKIP_SSH_KEY_NOT_FOUND_WARNING`: hard-pinned
+ *   `true` (upstream default `false`). Since the Phase-B migration the git
+ *   deploy key lives in SSM and is loaded by our env hook — the plugin's S3
+ *   `private_ssh_key` object is intentionally absent, so its per-job
+ *   not-found warning is pure noise.
  * - `ENABLE_EC2_LOG_RETENTION_POLICY`: hard-pinned `true` (upstream default
  *   `false`, "preserve all logs"). Fjall's log-group lifecycle posture is
  *   bounded retention everywhere: each booting instance applies
@@ -181,11 +186,11 @@ export function buildBuildkiteInstallCommands(props, context) {
         `BUILDKITE_SCALE_IN_IDLE_PERIOD='${props.scaleInIdlePeriodSeconds}' \\`,
         `BUILDKITE_SECRETS_BUCKET='${context.secretsBucketName}' \\`,
         `BUILDKITE_SECRETS_BUCKET_REGION='${context.region}' \\`,
-        "BUILDKITE_SECRETS_PLUGIN_SKIP_SSH_KEY_NOT_FOUND_WARNING='false' \\",
+        "BUILDKITE_SECRETS_PLUGIN_SKIP_SSH_KEY_NOT_FOUND_WARNING='true' \\",
         `BUILDKITE_ARTIFACTS_BUCKET='${context.artifactBucketName}' \\`,
         `BUILDKITE_S3_DEFAULT_REGION='${context.region}' \\`,
         "BUILDKITE_S3_ACL='bucket-owner-full-control' \\",
-        `BUILDKITE_AGENT_TOKEN_PATH='${props.agentTokenSsmParameterName}' \\`,
+        `BUILDKITE_AGENT_TOKEN_PATH='${context.agentTokenParameterName}' \\`,
         `BUILDKITE_AGENTS_PER_INSTANCE='${props.agentsPerInstance}' \\`,
         "BUILDKITE_AGENT_ENDPOINT='https://agent-edge.buildkite.com/v3' \\",
         `BUILDKITE_AGENT_TAGS='${props.buildkiteAgentTags}' \\`,

package/dist/lib/patterns/aws/buildkite.d.ts

@@ -4,7 +4,7 @@
  * Construct instantiated via `App.addBuildkite()`; design:
  * `aiDocs/designs/2026-07-18-buildkite-selfhosted-agents.md`.
  */
-export { Buildkite, FJALL_ENV_HOOK_OBJECT_KEY, buildFjallApiKeyEnvHookScript, type BuildkiteConstructProps } from "./buildkite/buildkite.js";
+export { AGENT_TOKEN_SECRET_NAME, BUILDKITE_SECRETS_CLUSTER, Buildkite, FJALL_API_KEY_SECRET_NAME, FJALL_ENV_HOOK_OBJECT_KEY, buildAgentSecretsEnvHookScript, type AgentSecretsEnvHookOptions, type BuildkiteConstructProps } from "./buildkite/buildkite.js";
 export { BUILDKITE_AMI_OWNER_ACCOUNT_ID, BUILDKITE_CPU_ARCHITECTURES, BUILDKITE_STACK_PINS, resolvePinnedAmiId, resolveScalerSarApplicationArn, type BuildkiteCpuArchitecture } from "./buildkite/pins.js";
 export { BuildkitePropsSchema, validateBuildkiteProps, type BuildkiteProps, type BuildkitePropsInput } from "./buildkite/schema.js";
 export { BuildkiteFactory, type BuildkiteFactoryFn } from "./buildkite/factory.js";

package/dist/lib/patterns/aws/buildkite.js

@@ -4,7 +4,7 @@
  * Construct instantiated via `App.addBuildkite()`; design:
  * `aiDocs/designs/2026-07-18-buildkite-selfhosted-agents.md`.
  */
-export { Buildkite, FJALL_ENV_HOOK_OBJECT_KEY, buildFjallApiKeyEnvHookScript } from "./buildkite/buildkite.js";
+export { AGENT_TOKEN_SECRET_NAME, BUILDKITE_SECRETS_CLUSTER, Buildkite, FJALL_API_KEY_SECRET_NAME, FJALL_ENV_HOOK_OBJECT_KEY, buildAgentSecretsEnvHookScript } from "./buildkite/buildkite.js";
 export { BUILDKITE_AMI_OWNER_ACCOUNT_ID, BUILDKITE_CPU_ARCHITECTURES, BUILDKITE_STACK_PINS, resolvePinnedAmiId, resolveScalerSarApplicationArn } from "./buildkite/pins.js";
 export { BuildkitePropsSchema, validateBuildkiteProps } from "./buildkite/schema.js";
 export { BuildkiteFactory } from "./buildkite/factory.js";

package/dist/lib/patterns/aws/database.d.ts

@@ -69,6 +69,12 @@ export interface AuroraDatabaseProps extends BaseDatabaseProps {
     clusterIdentifier?: string;
     /** Physical name of the generated master-credentials secret. */
     credentialsSecretName?: string;
+    /**
+     * KMS alias for the CMK encrypting the master-credentials secret. Must be
+     * app-scoped when the same `databaseName` deploys once per app in a shared
+     * account (aliases are account+region-unique).
+     */
+    credentialsKmsAliasName?: string;
     /** Serverless-v2 floor in ACU. 0 enables scale-to-zero (auto-pause). */
     serverlessV2MinCapacity?: number;
     /** Serverless-v2 ceiling in ACU. */

package/dist/lib/patterns/aws/database.js

@@ -16,7 +16,7 @@ import { Duration } from "aws-cdk-lib";
 export { isAwsManagedKey, isCMKRequested, AWS_MANAGED, USE_CMK } from "../../utils/databaseTypes.js";
 export const DATABASE_ENGINE_CONFIG = Object.freeze({
     postgresql: {
-        aurora: { version: "16.6", majorVersion: "16" },
+        aurora: { version: "16.10", majorVersion: "16" },
         instance: { version: "17.5", majorVersion: "17" },
         defaultUsername: "postgres",
         sslParameters: { "rds.force_ssl": "1" }
@@ -372,6 +372,7 @@ export class RelationalDatabase extends Construct {
             engineConfig,
             clusterIdentifier: props.clusterIdentifier,
             credentialsSecretName: props.credentialsSecretName,
+            credentialsKmsAliasName: props.credentialsKmsAliasName,
             serverlessV2MinCapacity: props.serverlessV2MinCapacity,
             serverlessV2MaxCapacity: props.serverlessV2MaxCapacity,
             serverlessV2AutoPauseDuration: props.serverlessV2AutoPauseDuration,

package/dist/lib/patterns/aws/delegatedDomainPattern.d.ts

@@ -10,8 +10,12 @@ export interface DelegatedDomainPatternResult {
 }
 /**
  * Composition for `registrar: "external-delegated"` — the delegated-child
- * topology. Creates a hosted zone for `{delegatedSubdomain}.{zoneName}` and,
- * when `parentDelegationRoleArn` is present, UPSERTs the child NS into the
+ * topology. Creates a hosted zone for `{delegatedSubdomain}.{zoneName}` —
+ * or ADOPTS an existing one when `hostedZoneId` + `adoptedNameServers` are
+ * set (both-or-neither, validated): the zone is imported instead of created
+ * and the delegation UPSERT carries the adopted NS literals, since an
+ * imported zone exposes no `hostedZoneNameServers` attribute. When
+ * `parentDelegationRoleArn` is present, the child NS are UPSERTed into the
  * parent zone via `CrossAccountZoneDelegationRecord` (D8 child-writes
  * delegation; same shape as devSubstrate Case 1). The role ARN MUST be a
  * literal — validated in `validateDomainProps` (VD2-a). When absent, the