alchemy 0.62.3 → 0.63.0

package/src/sentry/team.ts

@@ -10,8 +10,10 @@ import { SentryApi } from "./api.ts";
 export interface TeamProps {
   /**
    * The name for the team
+   *
+   * @default ${app}-${stage}-${id}
    */
-  name: string;
+  name?: string;
 
   /**
    * Uniquely identifies a team and is used for the interface
@@ -46,6 +48,11 @@ export interface Team extends Resource<"sentry::Team">, TeamProps {
    */
   id: string;
 
+  /**
+   * Name of the Team.
+   */
+  name: string;
+
   /**
    * Time at which the team was created
    */
@@ -127,11 +134,36 @@ export const Team = Resource(
   "sentry::Team",
   async function (
     this: Context<Team>,
-    _id: string,
+    id: string,
     props: TeamProps,
   ): Promise<Team> {
     const api = new SentryApi({ authToken: props.authToken });
 
+    // it's possible that `this.output.name` is undefined because a previous version
+    // of alchemy had a bug where it didn't set the name on the output
+    // so, we try to find the key by ID and use the name from the API response
+    const lookupName = async () => {
+      if (!this.output) {
+        return undefined;
+      } else if (this.output?.name) {
+        return this.output.name;
+      }
+      const name = await getTeamName(api, props.organization, this.output.id);
+      if (name) {
+        this.output.name = name;
+      }
+      return name;
+    };
+
+    const teamName =
+      props.name ?? (await lookupName()) ?? this.scope.createPhysicalName(id);
+
+    if (this.phase === "update" && this.output.name !== teamName) {
+      // TODO(sam): can we rename without destroying?
+      // -> no: https://docs.sentry.io/api/teams/update-a-team/
+      this.replace();
+    }
+
     if (this.phase === "delete") {
       try {
         if (this.output?.id) {
@@ -164,22 +196,22 @@ export const Team = Resource(
           } catch (error) {
             // Check if this is a "team already exists" error and adopt is enabled
             if (
-              props.adopt &&
+              (props.adopt ?? this.scope.adopt) &&
               error instanceof Error &&
               error.message.includes("already exists")
             ) {
               logger.log(
-                `Team '${props.slug || props.name}' already exists, adopting it`,
+                `Team '${props.slug || teamName}' already exists, adopting it`,
               );
               // Find the existing team by slug
               const existingTeam = await findTeamBySlug(
                 api,
                 props.organization,
-                props.slug || props.name,
+                props.slug || teamName,
               );
               if (!existingTeam) {
                 throw new Error(
-                  `Failed to find existing team '${props.slug || props.name}' for adoption`,
+                  `Failed to find existing team '${props.slug || teamName}' for adoption`,
                 );
               }
               response = await api.get(
@@ -192,13 +224,16 @@ export const Team = Resource(
         }
 
         if (!response.ok) {
-          throw new Error(`API error: ${response.statusText}`);
+          throw new Error(
+            `API error: ${response.statusText} ${await response.text()}`,
+          );
         }
 
         const data = (await response.json()) as Omit<Team, keyof TeamProps>;
         return this({
           ...props,
           id: data.id,
+          name: teamName,
           dateCreated: data.dateCreated,
           isMember: data.isMember,
           teamRole: data.teamRole,
@@ -234,3 +269,16 @@ async function findTeamBySlug(
   const team = teams.find((t) => t.slug === slug);
   return team ? { id: team.id, slug: team.slug } : null;
 }
+
+async function getTeamName(
+  api: SentryApi,
+  organization: string,
+  slug: string,
+): Promise<string | undefined> {
+  const response = await api.get(`/teams/${organization}/${slug}/`);
+  if (!response.ok) {
+    throw new Error(`API error: ${response.statusText}`);
+  }
+
+  return ((await response.json()) as { name: string }).name;
+}

package/src/serde.ts

@@ -1,10 +1,4 @@
 import { decryptWithKey, encrypt } from "./encrypt.ts";
-import {
-  InnerResourceScope,
-  ResourceFQN,
-  ResourceKind,
-  type Resource,
-} from "./resource.ts";
 import { isScope, type Scope } from "./scope.ts";
 import { isSecret, Secret } from "./secret.ts";
 
@@ -65,44 +59,6 @@ export type Serialized<T> = T extends
                       }
                     : T;
 
-export type SerializedScope = {
-  [fqn: ResourceFQN]: Serialized<Resource>;
-};
-
-export async function serializeScope(scope: Scope): Promise<SerializedScope> {
-  const map: SerializedScope = {};
-
-  return serializeScope(scope);
-
-  async function serializeScope(scope: Scope): Promise<SerializedScope> {
-    await Promise.all(
-      Array.from(scope.resources.values()).map(async (resource) => {
-        if (resource[ResourceKind] === "alchemy::Scope") {
-          return;
-        }
-        map[resource[ResourceFQN]] = await serialize(scope, await resource, {
-          transform: (value) => {
-            if (isSecret(value)) {
-              return {
-                "@secret-env": value.name,
-              };
-            }
-            return value;
-          },
-        });
-        const innerScope = await resource[InnerResourceScope];
-        if (innerScope) {
-          await serializeScope(innerScope);
-        }
-      }),
-    );
-    await Promise.all(
-      Array.from(scope.children.values()).map((scope) => serializeScope(scope)),
-    );
-    return map;
-  }
-}
-
 export async function serialize(
   scope: Scope,
   value: any,

package/src/state/d1-state-store.ts

@@ -1,13 +1,16 @@
-import { drizzle, type RemoteCallback } from "drizzle-orm/sqlite-proxy";
+import type { RemoteCallback } from "drizzle-orm/sqlite-proxy";
 import assert from "node:assert";
 import { extractCloudflareResult } from "../cloudflare/api-response.ts";
-import type { CloudflareApi, CloudflareApiOptions } from "../cloudflare/api.ts";
+import {
+  createCloudflareApi,
+  type CloudflareApi,
+  type CloudflareApiOptions,
+} from "../cloudflare/api.ts";
 import type { Scope } from "../scope.ts";
 import { memoize } from "../util/memoize.ts";
+import { importPeer } from "../util/peer.ts";
 import { MIGRATIONS_DIRECTORY } from "./migrations.ts";
-import { SQLiteStateStoreOperations } from "./operations.ts";
 import { StateStoreProxy } from "./proxy.ts";
-import * as schema from "./schema.ts";
 
 export interface D1StateStoreOptions extends CloudflareApiOptions {
   databaseName?: string;
@@ -26,6 +29,7 @@ export class D1StateStore extends StateStoreProxy {
 
   async provision(): Promise<StateStoreProxy.Dispatch> {
     const db = await createDatabaseClient(this.options);
+    const { SQLiteStateStoreOperations } = await import("./operations.js");
     const operations = new SQLiteStateStoreOperations(db, {
       chain: this.scope.chain,
     });
@@ -34,7 +38,10 @@ export class D1StateStore extends StateStoreProxy {
 }
 
 const createDatabaseClient = memoize(async (options: D1StateStoreOptions) => {
-  const { createCloudflareApi } = await import("../cloudflare/api.ts");
+  const { drizzle } = await importPeer(
+    import("drizzle-orm/sqlite-proxy"),
+    "D1StateStore",
+  );
   const api = await createCloudflareApi(options);
   const database = await upsertDatabase(
     api,
@@ -63,7 +70,7 @@ const createDatabaseClient = memoize(async (options: D1StateStoreOptions) => {
     };
   };
   return drizzle(remoteCallback, {
-    schema,
+    schema: await import("./schema.js"),
   });
 });
 

package/src/state/sqlite-state-store.ts

@@ -3,10 +3,9 @@ import fs from "node:fs";
 import path from "node:path";
 import type { Scope } from "../scope.ts";
 import { memoize } from "../util/memoize.ts";
+import { importPeer } from "../util/peer.ts";
 import { MIGRATIONS_DIRECTORY } from "./migrations.ts";
-import { SQLiteStateStoreOperations } from "./operations.ts";
 import { StateStoreProxy } from "./proxy.ts";
-import * as schema from "./schema.ts";
 
 interface BunSQLiteStateStoreOptions {
   /**
@@ -82,6 +81,7 @@ export class SQLiteStateStore extends StateStoreProxy {
 
   async provision(): Promise<StateStoreProxy.Dispatch> {
     const db = await createDatabase(this.options);
+    const { SQLiteStateStoreOperations } = await import("./operations.js");
     const operations = new SQLiteStateStoreOperations(db, {
       chain: this.scope.chain,
     });
@@ -125,12 +125,12 @@ async function createBunSQLiteDatabase(
     ".alchemy/state.sqlite";
   ensureDirectory(filename);
   const { Database } = await import("bun:sqlite");
-  const { drizzle } = await import("drizzle-orm/bun-sqlite").catch(() => {
-    throw new Error(
-      "[SQLiteStateStore] Missing `drizzle-orm` peer dependency. Please `bun install drizzle-orm`.",
-    );
-  });
+  const { drizzle } = await importPeer(
+    import("drizzle-orm/bun-sqlite"),
+    "SQLiteStateStore",
+  );
   const { migrate } = await import("drizzle-orm/bun-sqlite/migrator");
+  const schema = await import("./schema.js");
   // Bun's constructor throws if we pass in an empty object or if extraneous
   // options are passed in, so here's some ugly destructuring!
   const { engine: _engine, filename: _filename, ...rest } = options ?? {};
@@ -154,17 +154,16 @@ async function createLibSQLDatabase(
   if (filename) {
     ensureDirectory(filename);
   }
-  const { createClient } = await import("@libsql/client").catch(() => {
-    throw new Error(
-      "[SQLiteStateStore] Missing `@libsql/client` peer dependency. Please `npm install @libsql/client`.",
-    );
-  });
-  const { drizzle } = await import("drizzle-orm/libsql").catch(() => {
-    throw new Error(
-      "[SQLiteStateStore] Missing `drizzle-orm` peer dependency. Please `npm install drizzle-orm`.",
-    );
-  });
+  const { drizzle } = await importPeer(
+    import("drizzle-orm/libsql"),
+    "SQLiteStateStore",
+  );
+  const { createClient } = await importPeer(
+    import("@libsql/client"),
+    "SQLiteStateStore (libsql)",
+  );
   const { migrate } = await import("drizzle-orm/libsql/migrator");
+  const schema = await import("./schema.js");
   const client = createClient({ url, ...options });
   await client.execute("PRAGMA journal_mode = WAL;");
   const db = drizzle(client, {

package/src/stripe/card.ts

@@ -209,7 +209,7 @@ export const Card = Resource(
     _id: string,
     props: CardProps,
   ): Promise<Card> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -279,7 +279,10 @@ export const Card = Resource(
             createParams,
           )) as Stripe.Card;
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (
+            isStripeConflictError(error) &&
+            (props.adopt ?? this.scope.adopt)
+          ) {
             throw new Error(
               "Card adoption is not supported - cards cannot be uniquely identified for adoption",
             );

package/src/stripe/client.ts

@@ -1,13 +1,20 @@
-import Stripe from "stripe";
+import type Stripe from "stripe";
 import type { Secret } from "../secret.ts";
-import { withExponentialBackoff } from "../util/retry.ts";
 import { logger } from "../util/logger.ts";
+import { importPeer } from "../util/peer.ts";
+import { withExponentialBackoff } from "../util/retry.ts";
 
 export interface StripeClientOptions {
   apiKey?: Secret | string;
 }
 
-export function createStripeClient(options: StripeClientOptions = {}): Stripe {
+export async function createStripeClient(
+  options: StripeClientOptions = {},
+): Promise<Stripe> {
+  const { default: Stripe } = await importPeer(
+    import("stripe"),
+    "Stripe resources",
+  );
   let apiKey: string;
 
   if (options.apiKey) {

package/src/stripe/coupon.ts

@@ -163,7 +163,8 @@ export const Coupon = Resource(
     _id: string,
     props: CouponProps,
   ): Promise<Coupon> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -217,7 +218,7 @@ export const Coupon = Resource(
           try {
             coupon = await stripe.coupons.create(createParams);
           } catch (error) {
-            if (isStripeConflictError(error) && props.adopt) {
+            if (isStripeConflictError(error) && adopt) {
               if (props.id) {
                 const existingCoupon = await stripe.coupons.retrieve(props.id);
                 const updateParams: Stripe.CouponUpdateParams = {
@@ -236,7 +237,7 @@ export const Coupon = Resource(
             }
           }
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (isStripeConflictError(error) && adopt) {
             if (props.id) {
               const existingCoupon = await stripe.coupons.retrieve(props.id);
               const updateParams: Stripe.CouponUpdateParams = {

package/src/stripe/customer.ts

@@ -282,7 +282,7 @@ export const Customer = Resource(
     _id: string,
     props: CustomerProps,
   ): Promise<Customer> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -385,7 +385,7 @@ export const Customer = Resource(
           };
         }
 
-        if (props.adopt && props.email) {
+        if ((props.adopt ?? this.scope.adopt) && props.email) {
           const existingCustomers = await stripe.customers.list({
             email: props.email,
             limit: 1,

package/src/stripe/entitlements-feature.ts

@@ -101,7 +101,7 @@ export const EntitlementsFeature = Resource(
     _id: string,
     props: EntitlementsFeatureProps,
   ): Promise<EntitlementsFeature> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       return this.destroy();
@@ -130,7 +130,10 @@ export const EntitlementsFeature = Resource(
         try {
           feature = await stripe.entitlements.features.create(createParams);
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (
+            isStripeConflictError(error) &&
+            (props.adopt ?? this.scope.adopt)
+          ) {
             const existingFeatures = await stripe.entitlements.features.list({
               lookup_key: props.lookupKey,
               limit: 1,

package/src/stripe/file.ts

@@ -153,7 +153,8 @@ export const File = Resource(
     _id: string,
     props: FileProps,
   ): Promise<File> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       return this.destroy();
@@ -171,7 +172,7 @@ export const File = Resource(
             purpose: props.purpose,
           });
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (isStripeConflictError(error) && adopt) {
             throw new Error(
               "File adoption is not supported - files are immutable and cannot be adopted",
             );

package/src/stripe/meter.ts

@@ -155,7 +155,7 @@ export const Meter = Resource(
     _logicalId: string,
     props: MeterProps,
   ): Promise<Meter> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     const currentOutputId = this.output?.id;
 
@@ -303,7 +303,7 @@ export const Meter = Resource(
         stripeAPIResponse = await stripe.billing.meters.create(createParams);
       } catch (error) {
         logger.warn("Error creating/updating meter:", error);
-        if (isStripeConflictError(error) && props.adopt) {
+        if (isStripeConflictError(error) && (props.adopt ?? this.scope.adopt)) {
           throw new Error(
             "Meter adoption is not supported - meters cannot be uniquely identified for adoption",
           );

package/src/stripe/portal-configuration.ts

@@ -312,7 +312,7 @@ export const PortalConfiguration = Resource(
     _id: string,
     props: PortalConfigurationProps,
   ): Promise<PortalConfiguration> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -499,7 +499,10 @@ export const PortalConfiguration = Resource(
           configuration =
             await stripe.billingPortal.configurations.create(createParams);
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (
+            isStripeConflictError(error) &&
+            (props.adopt ?? this.scope.adopt)
+          ) {
             throw new Error(
               "PortalConfiguration adoption is not supported - portal configurations cannot be uniquely identified for adoption",
             );

package/src/stripe/price.ts

@@ -353,7 +353,8 @@ export const Price = Resource(
     _id: string,
     props: PriceProps,
   ): Promise<Price> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -475,7 +476,7 @@ export const Price = Resource(
             limit: 1,
           });
           if (existingPrices.data.length > 0) {
-            if (props.adopt) {
+            if (adopt) {
               const existingPrice = existingPrices.data[0];
               const updateParams: Stripe.PriceUpdateParams & {
                 expand?: string[];

package/src/stripe/product-feature.ts

@@ -97,7 +97,7 @@ export const ProductFeature = Resource(
     _id: string,
     props: ProductFeatureProps,
   ): Promise<ProductFeature> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -124,7 +124,10 @@ export const ProductFeature = Resource(
             entitlement_feature: props.entitlementFeature,
           });
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (
+            isStripeConflictError(error) &&
+            (props.adopt ?? this.scope.adopt)
+          ) {
             throw new Error(
               "ProductFeature adoption is not supported - product features cannot be uniquely identified for adoption",
             );

package/src/stripe/product.ts

@@ -157,7 +157,8 @@ export const Product = Resource(
     _id: string,
     props: ProductProps,
   ): Promise<Product> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -202,7 +203,7 @@ export const Product = Resource(
           metadata: props.metadata,
           tax_code: props.taxCode,
         };
-        if (props.adopt) {
+        if (adopt) {
           const existingProducts = await stripe.products.list({
             limit: 100,
           });

package/src/stripe/promotion-code.ts

@@ -156,7 +156,8 @@ export const PromotionCode = Resource(
     _id: string,
     props: PromotionCodeProps,
   ): Promise<PromotionCode> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -203,7 +204,7 @@ export const PromotionCode = Resource(
         try {
           promotionCode = await stripe.promotionCodes.create(createParams);
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (isStripeConflictError(error) && adopt) {
             if (props.code) {
               const existingPromotionCodes = await stripe.promotionCodes.list({
                 code: props.code,

package/src/stripe/shipping-rate.ts

@@ -209,7 +209,8 @@ export const ShippingRate = Resource(
     _id: string,
     props: ShippingRateProps,
   ): Promise<ShippingRate> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -286,7 +287,7 @@ export const ShippingRate = Resource(
           try {
             shippingRate = await stripe.shippingRates.create(createParams);
           } catch (error) {
-            if (isStripeConflictError(error) && props.adopt) {
+            if (isStripeConflictError(error) && adopt) {
               throw new Error(
                 "ShippingRate adoption is not supported - shipping rates cannot be uniquely identified for adoption",
               );
@@ -295,7 +296,7 @@ export const ShippingRate = Resource(
             }
           }
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (isStripeConflictError(error) && adopt) {
             throw new Error(
               "ShippingRate adoption is not supported - shipping rates cannot be uniquely identified for adoption",
             );

package/src/stripe/tax-rate.ts

@@ -143,7 +143,8 @@ export const TaxRate = Resource(
     _id: string,
     props: TaxRateProps,
   ): Promise<TaxRate> {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -185,7 +186,7 @@ export const TaxRate = Resource(
             tax_type: props.taxType,
           });
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (isStripeConflictError(error) && adopt) {
             throw new Error(
               "TaxRate adoption is not supported - tax rates cannot be uniquely identified for adoption",
             );

package/src/stripe/webhook.ts

@@ -155,7 +155,8 @@ export const WebhookEndpoint = Resource(
     _id: string,
     props: WebhookEndpointProps,
   ) {
-    const stripe = createStripeClient({ apiKey: props.apiKey });
+    const adopt = props.adopt ?? this.scope.adopt;
+    const stripe = await createStripeClient({ apiKey: props.apiKey });
 
     if (this.phase === "delete") {
       try {
@@ -197,7 +198,7 @@ export const WebhookEndpoint = Resource(
           try {
             webhook = await stripe.webhookEndpoints.create(createParams);
           } catch (error) {
-            if (isStripeConflictError(error) && props.adopt) {
+            if (isStripeConflictError(error) && adopt) {
               const existingWebhooks = await stripe.webhookEndpoints.list({
                 limit: 100,
               });
@@ -223,7 +224,7 @@ export const WebhookEndpoint = Resource(
             }
           }
         } catch (error) {
-          if (isStripeConflictError(error) && props.adopt) {
+          if (isStripeConflictError(error) && adopt) {
             const existingWebhooks = await stripe.webhookEndpoints.list({
               limit: 100,
             });

package/src/test/vitest.ts

@@ -131,6 +131,7 @@ export function test(
     phase: "up",
     telemetryClient: new NoopTelemetryClient(),
     quiet: defaultOptions.quiet,
+    password: process.env.ALCHEMY_PASSWORD,
   });
 
   test.beforeAll = (fn: (scope: Scope) => Promise<void>) => {