@microagi/alchemy-gcp 0.11.6 → 0.11.8

package/src/Iam/WorkloadIdentityPoolProvider.ts

@@ -0,0 +1,429 @@
+import { ConfigError } from "@distilled.cloud/gcp";
+import * as iam from "@distilled.cloud/gcp/unstable/iam_v1";
+import { Resource } from "alchemy";
+import { Unowned } from "alchemy/AdoptPolicy";
+import { isResolved } from "alchemy/Diff";
+import * as Provider from "alchemy/Provider";
+import * as Effect from "effect/Effect";
+import {
+  descriptionHasAlchemyMarker,
+  gcpAlchemyDescription,
+  normalizeStringMap,
+  stripAlchemyMarker,
+} from "../Tags.ts";
+import type * as GCP from "../Providers.ts";
+import { makeAwaitOperation } from "./Operations.ts";
+import { poolResourceName } from "./WorkloadIdentityPool.ts";
+
+/**
+ * An OIDC **provider** inside a
+ * {@link import("./WorkloadIdentityPool.ts").WorkloadIdentityPool} — the
+ * thing that actually declares "I trust tokens issued by this URL".
+ *
+ * The canonical use is keyless federation from another cloud: point
+ * `oidc.issuerUri` at an EKS/AKS/GKE cluster's OIDC issuer, and pods holding
+ * a projected ServiceAccount token can impersonate a GSA with no key material
+ * anywhere in the system.
+ *
+ * ### `attributeMapping` is required and `google.subject` is mandatory
+ *
+ * The mapping turns claims on the incoming token into Google attributes. At
+ * minimum `google.subject` must be mapped; it is what `principal://` members
+ * match on. For a Kubernetes issuer the subject claim is
+ * `system:serviceaccount:{namespace}:{name}`, so:
+ *
+ * ```typescript
+ * attributeMapping: { "google.subject": "assertion.sub" }
+ * ```
+ *
+ * ### Set `allowedAudiences`, or the default is surprising
+ *
+ * When `oidc.allowedAudiences` is empty, GCP accepts only the *default*
+ * audience — the full provider resource name prefixed with
+ * `https://iam.googleapis.com/`. A Kubernetes projected token minted for
+ * audience `sts.googleapis.com` (or anything else) is then rejected with a
+ * generic audience error. Set this explicitly to whatever audience the
+ * token is actually minted for.
+ *
+ * ### Constrain trust with `attributeCondition`
+ *
+ * An issuer-only trust accepts **every** identity that issuer can mint — on
+ * a Kubernetes cluster, that is every ServiceAccount in every namespace. Pair
+ * the provider with an `attributeCondition` (and a narrow `principal://`
+ * member on the GSA binding) so a token from an unrelated namespace cannot
+ * impersonate anything.
+ *
+ * @section Creating a WorkloadIdentityPoolProvider
+ * @example Trust one Kubernetes ServiceAccount on an EKS cluster
+ * ```typescript
+ * const provider = yield* GCP.WorkloadIdentityPoolProvider("EksOidc", {
+ *   project: "123456789",
+ *   poolId: pool.poolId,
+ *   providerId: "eks-oidc",
+ *   oidc: {
+ *     issuerUri: "https://oidc.eks.us-east-1.amazonaws.com/id/ABC123",
+ *     allowedAudiences: ["sts.googleapis.com"],
+ *   },
+ *   attributeMapping: { "google.subject": "assertion.sub" },
+ *   attributeCondition:
+ *     "assertion.sub == 'system:serviceaccount:admin:eks-auth-reconciler'",
+ * });
+ * ```
+ */
+export type WorkloadIdentityPoolProviderProps = {
+  /** Project that owns the pool. Prefer the project **number**. */
+  project: string;
+  /** ID of the enclosing pool. */
+  poolId: string;
+  /**
+   * Provider ID, 4–32 characters of `[a-z0-9-]`. Immutable — changing it
+   * replaces the provider. The `gcp-` prefix is reserved by Google.
+   */
+  providerId: string;
+  /** The OIDC issuer this provider trusts. */
+  oidc: {
+    /** Issuer URL. Must be HTTPS and serve an OIDC discovery document. */
+    issuerUri: string;
+    /**
+     * Acceptable `aud` values. Leave unset only if the token is minted for
+     * the provider's default audience — see the note above.
+     */
+    allowedAudiences?: string[];
+    /**
+     * Inline JWKS, for issuers whose keys are not publicly reachable. When
+     * unset, GCP fetches `jwks_uri` from the issuer's discovery document,
+     * which requires the issuer to be reachable from Google's network.
+     */
+    jwksJson?: string;
+  };
+  /**
+   * Claim-to-attribute mapping. Must include `google.subject`.
+   */
+  attributeMapping: Record<string, string>;
+  /** CEL expression further restricting which credentials are accepted. */
+  attributeCondition?: string;
+  /** Display name, max 32 characters. */
+  displayName?: string;
+  /** Description, max 256 characters. */
+  description?: string;
+  /** Disable the provider without deleting it. */
+  disabled?: boolean;
+};
+
+export type WorkloadIdentityPoolProviderAttributes = {
+  /** Full resource name, `…/workloadIdentityPools/{pool}/providers/{id}`. */
+  name: string;
+  project: string;
+  poolId: string;
+  providerId: string;
+  issuerUri: string | undefined;
+  // `readonly` because that is how the SDK models repeated fields; widening it
+  // to a mutable array here would only force a copy at every use site.
+  allowedAudiences: readonly string[] | undefined;
+  attributeMapping: Record<string, string> | undefined;
+  attributeCondition: string | undefined;
+  displayName: string | undefined;
+  description: string | undefined;
+  state: string | undefined;
+  disabled: boolean | undefined;
+  /**
+   * The audience string to put in a credential configuration, i.e.
+   * `//iam.googleapis.com/{name}`. Provided because assembling it by hand is
+   * easy to get subtly wrong (the leading `//`, and no scheme).
+   */
+  audience: string;
+};
+
+export interface WorkloadIdentityPoolProvider
+  extends Resource<
+    "GCP.WorkloadIdentityPoolProvider",
+    WorkloadIdentityPoolProviderProps,
+    WorkloadIdentityPoolProviderAttributes,
+    never,
+    GCP.Providers
+  > {}
+
+export const WorkloadIdentityPoolProvider =
+  Resource<WorkloadIdentityPoolProvider>("GCP.WorkloadIdentityPoolProvider");
+
+export const providerResourceName = (
+  project: string,
+  poolId: string,
+  providerId: string,
+) => `${poolResourceName(project, poolId)}/providers/${providerId}`;
+
+/**
+ * Compare two claim mappings by CONTENT, not serialisation.
+ *
+ * `JSON.stringify` preserves key insertion order, and the order GCP returns
+ * these keys in need not match the order they were sent in. A raw stringify
+ * comparison therefore reports drift for two identical mappings and fires a
+ * patch LRO on every single deploy.
+ */
+const sameMapping = (
+  a: Record<string, string> | undefined,
+  b: Record<string, string> | undefined,
+) => {
+  const entries = (r: Record<string, string> | undefined) =>
+    Object.entries(r ?? {}).sort(([x], [y]) => (x < y ? -1 : x > y ? 1 : 0));
+  return JSON.stringify(entries(a)) === JSON.stringify(entries(b));
+};
+
+/**
+ * Compare audience allow-lists as SETS.
+ *
+ * `allowedAudiences` is an allow-list: membership is what matters, order is
+ * not meaningful, and GCP is free to return it in any order. Comparing it
+ * positionally would make a reordering look like drift forever.
+ */
+const sameAudiences = (
+  a: readonly string[] | undefined,
+  b: readonly string[] | undefined,
+) => {
+  const sorted = (v: readonly string[] | undefined) => [...(v ?? [])].sort();
+  return JSON.stringify(sorted(a)) === JSON.stringify(sorted(b));
+};
+
+/**
+ * Treat absent and empty-string as the same value.
+ *
+ * The patch below uses a FIXED `updateMask` while the request body omits
+ * unset optional fields — which is deliberate: under a field mask, an omitted
+ * field means "clear it", so dropping `displayName` from the props really
+ * should clear it on GCP. The hazard is only in the comparison afterwards. If
+ * GCP echoes a cleared field back as `""` rather than omitting it, a raw
+ * `!==` against `undefined` would report drift on every single deploy and
+ * churn a long-running patch operation forever.
+ */
+const sameText = (a: string | undefined, b: string | undefined) =>
+  (a ?? "") === (b ?? "");
+
+export const WorkloadIdentityPoolProviderProvider = () =>
+  Provider.effect(
+    WorkloadIdentityPoolProvider,
+    Effect.gen(function* () {
+      const getProvider =
+        yield* iam.getProjectsLocationsWorkloadIdentityPoolsProviders;
+      const createProvider =
+        yield* iam.createProjectsLocationsWorkloadIdentityPoolsProviders;
+      const patchProvider =
+        yield* iam.patchProjectsLocationsWorkloadIdentityPoolsProviders;
+      const deleteProvider =
+        yield* iam.deleteProjectsLocationsWorkloadIdentityPoolsProviders;
+      const undeleteProvider =
+        yield* iam.undeleteProjectsLocationsWorkloadIdentityPoolsProviders;
+      const getOperations =
+        yield* iam.getProjectsLocationsWorkloadIdentityPoolsProvidersOperations;
+      const awaitOperation = makeAwaitOperation(
+        getOperations,
+        "Workload Identity provider",
+      );
+
+      const observeProvider = (
+        project: string,
+        poolId: string,
+        providerId: string,
+      ) =>
+        getProvider({
+          name: providerResourceName(project, poolId, providerId),
+        }).pipe(
+          Effect.catchTag("NotFound", () =>
+            Effect.succeed(
+              undefined as iam.WorkloadIdentityPoolProvider | undefined,
+            ),
+          ),
+          Effect.catchTag("Forbidden", () =>
+            Effect.succeed(
+              undefined as iam.WorkloadIdentityPoolProvider | undefined,
+            ),
+          ),
+        );
+
+      const toAttrs = (
+        project: string,
+        poolId: string,
+        providerId: string,
+        provider: iam.WorkloadIdentityPoolProvider,
+      ): WorkloadIdentityPoolProviderAttributes => {
+        const name =
+          provider.name ?? providerResourceName(project, poolId, providerId);
+        return {
+          name,
+          project,
+          poolId,
+          providerId,
+          issuerUri: provider.oidc?.issuerUri,
+          allowedAudiences: provider.oidc?.allowedAudiences,
+          attributeMapping: normalizeStringMap(provider.attributeMapping),
+          attributeCondition: provider.attributeCondition,
+          displayName: provider.displayName,
+          description: stripAlchemyMarker(provider.description),
+          state: provider.state,
+          disabled: provider.disabled,
+          audience: `//iam.googleapis.com/${name}`,
+        };
+      };
+
+      return {
+        stables: ["name", "project", "poolId", "providerId", "audience"],
+        diff: Effect.fn(function* ({ olds = {}, news, output }) {
+          if (!isResolved(news)) return undefined;
+          const oldProps = olds as Partial<WorkloadIdentityPoolProviderProps>;
+          // Prefer live attributes over persisted props, and fall back to the
+          // DESIRED value when neither is known — see the equivalent comment
+          // in WorkloadIdentityPool.ts. Defaulting to `undefined` would make
+          // an adoption replace a live, correct provider.
+          //
+          // All three fields are part of the resource name; the API cannot
+          // move a provider between pools or rename it.
+          const currentProject = output?.project || oldProps.project || news.project;
+          const currentPoolId = output?.poolId || oldProps.poolId || news.poolId;
+          const currentProviderId =
+            output?.providerId || oldProps.providerId || news.providerId;
+          if (
+            currentProject !== news.project ||
+            currentPoolId !== news.poolId ||
+            currentProviderId !== news.providerId
+          ) {
+            return { action: "replace" } as const;
+          }
+        }),
+        read: Effect.fn(function* ({ id, olds, output }) {
+          const project = output?.project || olds?.project;
+          const poolId = output?.poolId || olds?.poolId;
+          const providerId = output?.providerId || olds?.providerId;
+          if (!project || !poolId || !providerId) return undefined;
+          const provider = yield* observeProvider(project, poolId, providerId);
+          if (!provider) return undefined;
+          const attrs = toAttrs(project, poolId, providerId, provider);
+          return (yield* descriptionHasAlchemyMarker(id, provider.description))
+            ? attrs
+            : Unowned(attrs);
+        }),
+        reconcile: Effect.fn(function* ({ id, news, session }) {
+          const { project, poolId, providerId } = news;
+          if (!news.attributeMapping["google.subject"]) {
+            // Caught here rather than at the API, which rejects it with a
+            // message that does not name the missing key.
+            return yield* new ConfigError({
+              message: `Workload Identity provider ${providerId} must map 'google.subject' in attributeMapping.`,
+            });
+          }
+          const description = yield* gcpAlchemyDescription(
+            id,
+            news.description,
+          );
+          // Scalars are TOTAL over the updateMask below — always present,
+          // explicit empty/default when unset — so clearing is stated in code
+          // rather than implied by field-mask semantics.
+          //
+          // `attributeCondition` and `oidc.jwksJson` are the exception and
+          // stay conditional: both are validated formats (a CEL expression
+          // and a JWKS document), and sending "" risks a parse rejection
+          // where omission is unambiguous. They are still cleared correctly
+          // when unset — `oidc` is masked as a whole message, so the body's
+          // `oidc` replaces it outright.
+          const body: iam.WorkloadIdentityPoolProvider = {
+            displayName: news.displayName ?? "",
+            description,
+            disabled: news.disabled ?? false,
+            attributeMapping: news.attributeMapping,
+            ...(news.attributeCondition
+              ? { attributeCondition: news.attributeCondition }
+              : {}),
+            oidc: {
+              issuerUri: news.oidc.issuerUri,
+              allowedAudiences: news.oidc.allowedAudiences ?? [],
+              ...(news.oidc.jwksJson ? { jwksJson: news.oidc.jwksJson } : {}),
+            },
+          };
+
+          let provider = yield* observeProvider(project, poolId, providerId);
+
+          // Same soft-delete reasoning as the pool: the ID is held until the
+          // purge, so undelete rather than strand the deploy.
+          if (provider?.state === "DELETED") {
+            yield* session.note(
+              `Undeleting soft-deleted Workload Identity provider ${providerId}…`,
+            );
+            const op = yield* undeleteProvider({
+              name: providerResourceName(project, poolId, providerId),
+              body: {},
+            });
+            if (op.name) yield* awaitOperation(op.name, session);
+            provider = yield* observeProvider(project, poolId, providerId);
+          }
+
+          if (!provider) {
+            const op = yield* createProvider({
+              parent: poolResourceName(project, poolId),
+              workloadIdentityPoolProviderId: providerId,
+              body,
+            });
+            if (op.name) yield* awaitOperation(op.name, session);
+            provider = yield* observeProvider(project, poolId, providerId);
+            if (!provider) {
+              return yield* new ConfigError({
+                message: `Workload Identity provider ${providerId} in pool ${poolId} was not readable after create.`,
+              });
+            }
+          } else {
+            const needsPatch =
+              !sameText(provider.displayName, news.displayName) ||
+              !sameText(provider.description, description) ||
+              (provider.disabled ?? false) !== (news.disabled ?? false) ||
+              !sameText(provider.attributeCondition, news.attributeCondition) ||
+              !sameMapping(
+                normalizeStringMap(provider.attributeMapping),
+                news.attributeMapping,
+              ) ||
+              !sameText(provider.oidc?.issuerUri, news.oidc.issuerUri) ||
+              !sameAudiences(
+                provider.oidc?.allowedAudiences,
+                news.oidc.allowedAudiences,
+              ) ||
+              !sameText(provider.oidc?.jwksJson, news.oidc.jwksJson);
+            if (needsPatch) {
+              const op = yield* patchProvider({
+                name: providerResourceName(project, poolId, providerId),
+                updateMask:
+                  "displayName,description,disabled,attributeMapping,attributeCondition,oidc",
+                body,
+              });
+              if (op.name) yield* awaitOperation(op.name, session);
+              provider =
+                (yield* observeProvider(project, poolId, providerId)) ??
+                provider;
+            }
+          }
+
+          yield* session.note(
+            providerResourceName(project, poolId, providerId),
+          );
+          return toAttrs(project, poolId, providerId, provider);
+        }),
+        delete: Effect.fn(function* ({ olds, output, session }) {
+          // Props fallback for the same reason as the pool: incomplete
+          // attributes must not turn destroy into a silent no-op that leaks a
+          // soft-deleted-name-holding resource.
+          // `||` not `??`: an empty string is never a valid identity here,
+          // and a persisted `""` must fall through to the next source rather
+          // than be taken as real. With `??` a corrupt/partial state entry
+          // would short-circuit the fallback and turn destroy into a silent
+          // no-op (or make diff replace a live resource).
+          const project = output?.project || olds?.project;
+          const poolId = output?.poolId || olds?.poolId;
+          const providerId = output?.providerId || olds?.providerId;
+          if (!project || !poolId || !providerId) return;
+          const op = yield* deleteProvider({
+            name: providerResourceName(project, poolId, providerId),
+          }).pipe(
+            Effect.catchTag("NotFound", () =>
+              Effect.succeed({ name: undefined } as iam.Operation),
+            ),
+          );
+          if (op.name) yield* awaitOperation(op.name, session);
+        }),
+      };
+    }),
+  );

package/src/Iam/index.ts

@@ -6,3 +6,13 @@ export {
   ServiceAccountKeyProvider,
 } from "./ServiceAccountKey.ts";
 export type { ServiceAccountKeyProps } from "./ServiceAccountKey.ts";
+export {
+  WorkloadIdentityPool,
+  WorkloadIdentityPoolResourceProvider,
+} from "./WorkloadIdentityPool.ts";
+export type { WorkloadIdentityPoolProps } from "./WorkloadIdentityPool.ts";
+export {
+  WorkloadIdentityPoolProvider,
+  WorkloadIdentityPoolProviderProvider,
+} from "./WorkloadIdentityPoolProvider.ts";
+export type { WorkloadIdentityPoolProviderProps } from "./WorkloadIdentityPoolProvider.ts";

package/src/Kubernetes/Secret.ts

@@ -123,7 +123,10 @@ type SecretObject = {
     namespace?: string;
     uid?: string;
     resourceVersion?: string;
-    labels?: Record<string, string>;
+    // Distilled's generated `…ObjectMetaLabelsMap` index signature includes
+    // `undefined`, so the observed object is only assignable to this local
+    // subset if we mirror that.
+    labels?: Record<string, string | undefined>;
   };
   type?: string;
 };

package/src/ManagedLustre/Instance.ts

@@ -1,5 +1,5 @@
 import { ConfigError } from "@distilled.cloud/gcp";
-import * as lustre from "@distilled.cloud/gcp/lustre-v1";
+import * as lustre from "@distilled.cloud/gcp/lustre_v1";
 import { Resource } from "alchemy";
 import { Unowned } from "alchemy/AdoptPolicy";
 import type { ScopedPlanStatusSession } from "alchemy/Cli/Cli";
@@ -11,7 +11,7 @@ import * as Duration from "effect/Duration";
 import * as Effect from "effect/Effect";
 import * as Schedule from "effect/Schedule";
 import type { NetworkRefById } from "../Compute/Network.ts";
-import { gcpInternalLabels, hasAlchemyLabels } from "../Tags.ts";
+import { gcpInternalLabels, hasAlchemyLabels, normalizeStringMap } from "../Tags.ts";
 import type * as GCP from "../Providers.ts";
 
 /**
@@ -221,7 +221,7 @@ const toAttributes = (
   perUnitStorageThroughput: i.perUnitStorageThroughput,
   gkeSupportEnabled: i.gkeSupportEnabled,
   kmsKey: i.kmsKey,
-  labels: { ...(i.labels ?? {}) },
+  labels: normalizeStringMap(i.labels) ?? {},
 });
 
 export const ManagedLustreInstanceProvider = () =>
@@ -295,7 +295,7 @@ export const ManagedLustreInstanceProvider = () =>
         if ((args.observed.description ?? undefined) !== args.desired.description) {
           updateMaskFields.push("description");
         }
-        const observedLabels = { ...(args.observed.labels ?? {}) };
+        const observedLabels = normalizeStringMap(args.observed.labels) ?? {};
         const labelDiff = diffTags(observedLabels, args.desired.labels);
         if (labelDiff.removed.length > 0 || labelDiff.upsert.length > 0) {
           updateMaskFields.push("labels");
@@ -434,7 +434,7 @@ export const ManagedLustreInstanceProvider = () =>
               ),
             );
             for (const candidate of page?.instances ?? []) {
-              if (yield* hasAlchemyLabels(id, candidate.labels)) {
+              if (yield* hasAlchemyLabels(id, normalizeStringMap(candidate.labels))) {
                 observed = candidate;
                 break;
               }
@@ -450,7 +450,7 @@ export const ManagedLustreInstanceProvider = () =>
             instanceId,
             network,
           });
-          return (yield* hasAlchemyLabels(id, observed.labels))
+          return (yield* hasAlchemyLabels(id, normalizeStringMap(observed.labels)))
             ? attrs
             : Unowned(attrs);
         }),

package/src/Providers.ts

@@ -39,6 +39,14 @@ import {
   ServiceAccountKey,
   ServiceAccountKeyProvider,
 } from "./Iam/ServiceAccountKey.ts";
+import {
+  WorkloadIdentityPool,
+  WorkloadIdentityPoolResourceProvider,
+} from "./Iam/WorkloadIdentityPool.ts";
+import {
+  WorkloadIdentityPoolProvider,
+  WorkloadIdentityPoolProviderProvider,
+} from "./Iam/WorkloadIdentityPoolProvider.ts";
 import {
   HelmRelease,
   HelmReleaseProvider,
@@ -123,6 +131,8 @@ export const providers = () =>
       ArtifactRegistryRepositoryIamMember,
       ServiceAccount,
       ServiceAccountKey,
+      WorkloadIdentityPool,
+      WorkloadIdentityPoolProvider,
       KubernetesSecret,
       KubernetesManifest,
       HelmRelease,
@@ -156,6 +166,8 @@ export const providers = () =>
         ArtifactRegistryRepositoryIamMemberProvider(),
         ServiceAccountProvider(),
         ServiceAccountKeyProvider(),
+        WorkloadIdentityPoolResourceProvider(),
+        WorkloadIdentityPoolProviderProvider(),
         KubernetesSecretProvider(),
         KubernetesManifestProvider(),
         HelmReleaseProvider(),

package/src/Run/IamSync.ts

@@ -1,4 +1,4 @@
-import type * as run from "@distilled.cloud/gcp/run-v2";
+import type * as run from "@distilled.cloud/gcp/run_v2";
 import type { ResourceBinding } from "alchemy/Resource";
 import * as Duration from "effect/Duration";
 import * as Effect from "effect/Effect";

package/src/Run/Job.ts

@@ -1,4 +1,4 @@
-import * as run from "@distilled.cloud/gcp/run-v2";
+import * as run from "@distilled.cloud/gcp/run_v2";
 import { Resource } from "alchemy";
 import { Unowned } from "alchemy/AdoptPolicy";
 import type { ScopedPlanStatusSession } from "alchemy/Cli/Cli";
@@ -10,7 +10,7 @@ import * as Duration from "effect/Duration";
 import * as Effect from "effect/Effect";
 import * as Schedule from "effect/Schedule";
 import type * as GCP from "../Providers.ts";
-import { gcpInternalLabels, hasAlchemyLabels } from "../Tags.ts";
+import { gcpInternalLabels, hasAlchemyLabels, normalizeStringMap } from "../Tags.ts";
 import { makeSyncIam, type RunIamBindingContract } from "./IamSync.ts";
 import { makeAwaitOperation } from "./Operations.ts";
 import {
@@ -34,7 +34,7 @@ import {
  * converge-to-desired-state model) and is left to user-side code:
  *
  * ```typescript
- * import * as run from "@distilled.cloud/gcp/run-v2";
+ * import * as run from "@distilled.cloud/gcp/run_v2";
  * const job = yield* GCP.Job("Nightly", { ... });
  * const runJob = yield* run.runProjectsLocationsJobs;
  * yield* runJob({
@@ -234,7 +234,7 @@ const toAttributes = (
   latestCreatedExecution: j.latestCreatedExecution,
   reconciling: j.reconciling,
   etag: j.etag,
-  labels: { ...(j.labels ?? {}) },
+  labels: normalizeStringMap(j.labels) ?? {},
   createTime: j.createTime,
   updateTime: j.updateTime,
 });
@@ -260,7 +260,7 @@ const jobMutated = (
   desiredLabels: Record<string, string>,
 ): boolean => {
   const labelDiff = diffTags(
-    { ...(observed.labels ?? {}) },
+    normalizeStringMap(observed.labels) ?? {},
     desiredLabels,
   );
   if (labelDiff.removed.length > 0 || labelDiff.upsert.length > 0) return true;
@@ -448,7 +448,7 @@ export const JobProvider = () =>
           const observed = yield* observe(project, location, name);
           if (!observed) return undefined;
           const attrs = toAttributes(observed, { project, location, name });
-          return (yield* hasAlchemyLabels(id, observed.labels))
+          return (yield* hasAlchemyLabels(id, normalizeStringMap(observed.labels)))
             ? attrs
             : Unowned(attrs);
         }),

package/src/Run/Operations.ts

@@ -1,5 +1,5 @@
 import { ConfigError } from "@distilled.cloud/gcp";
-import * as run from "@distilled.cloud/gcp/run-v2";
+import * as run from "@distilled.cloud/gcp/run_v2";
 import type { ScopedPlanStatusSession } from "alchemy/Cli/Cli";
 import * as Duration from "effect/Duration";
 import * as Effect from "effect/Effect";

package/src/Run/Service.ts

@@ -1,4 +1,4 @@
-import * as run from "@distilled.cloud/gcp/run-v2";
+import * as run from "@distilled.cloud/gcp/run_v2";
 import { Resource } from "alchemy";
 import { Unowned } from "alchemy/AdoptPolicy";
 import type { ScopedPlanStatusSession } from "alchemy/Cli/Cli";
@@ -10,7 +10,7 @@ import * as Duration from "effect/Duration";
 import * as Effect from "effect/Effect";
 import * as Schedule from "effect/Schedule";
 import type * as GCP from "../Providers.ts";
-import { gcpInternalLabels, hasAlchemyLabels } from "../Tags.ts";
+import { gcpInternalLabels, hasAlchemyLabels, normalizeStringMap } from "../Tags.ts";
 import { makeSyncIam, type RunIamBindingContract } from "./IamSync.ts";
 import { makeAwaitOperation } from "./Operations.ts";
 import {
@@ -245,8 +245,10 @@ const toServiceBody = (
     ? { defaultUriDisabled: news.defaultUriDisabled }
     : {}),
   ...(news.scaling ? { scaling: news.scaling } : {}),
-  ...(news.traffic ? { traffic: news.traffic } : {}),
-  ...(news.customAudiences ? { customAudiences: news.customAudiences } : {}),
+  ...(news.traffic ? { traffic: [...news.traffic] } : {}),
+  ...(news.customAudiences
+    ? { customAudiences: [...news.customAudiences] }
+    : {}),
   ...(news.binaryAuthorization
     ? { binaryAuthorization: news.binaryAuthorization }
     : {}),
@@ -271,7 +273,7 @@ const toAttributes = (
   terminalCondition: s.terminalCondition,
   trafficStatuses: s.trafficStatuses ?? [],
   etag: s.etag,
-  labels: { ...(s.labels ?? {}) },
+  labels: normalizeStringMap(s.labels) ?? {},
   createTime: s.createTime,
   updateTime: s.updateTime,
 });
@@ -314,7 +316,7 @@ export const ServiceProvider = () =>
           updateMaskFields.push("description");
         }
         const labelDiff = diffTags(
-          { ...(args.observed.labels ?? {}) },
+          normalizeStringMap(args.observed.labels) ?? {},
           args.desiredLabels,
         );
         if (labelDiff.removed.length > 0 || labelDiff.upsert.length > 0) {
@@ -536,7 +538,7 @@ export const ServiceProvider = () =>
           const observed = yield* observe(project, location, name);
           if (!observed) return undefined;
           const attrs = toAttributes(observed, { project, location, name });
-          return (yield* hasAlchemyLabels(id, observed.labels))
+          return (yield* hasAlchemyLabels(id, normalizeStringMap(observed.labels)))
             ? attrs
             : Unowned(attrs);
         }),

package/src/Run/index.ts

@@ -1,4 +1,4 @@
-import type * as run from "@distilled.cloud/gcp/run-v2";
+import type * as run from "@distilled.cloud/gcp/run_v2";
 
 export { Job, JobProvider } from "./Job.ts";
 export type { JobAttributes, JobProps } from "./Job.ts";
@@ -9,7 +9,7 @@ export type { ServiceAttributes, ServiceProps } from "./Service.ts";
 
 /**
  * Clean aliases for the most commonly-used Cloud Run v2 SDK types,
- * re-exported from `@distilled.cloud/gcp/run-v2` so consumers don't
+ * re-exported from `@distilled.cloud/gcp/run_v2` so consumers don't
  * need to import from the underlying SDK directly. The aliases are
  * type-only — they share identity with the distilled types, so values
  * are interchangeable.