alchemy-effect 0.4.0 → 0.6.0

package/lib/apply.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"apply.d.ts","sourceRoot":"","sources":["../src/apply.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,MAAM,MAAM,eAAe,CAAC;AACxC,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,EAAE,GAAG,EAAE,MAAM,UAAU,CAAC;AAE/B,OAAO,EAEL,GAAG,EAEJ,MAAM,kBAAkB,CAAC;AAI1B,OAAO,EAIL,KAAK,MAAM,EACX,KAAK,UAAU,EACf,KAAK,KAAK,EACV,KAAK,SAAS,EACf,MAAM,WAAW,CAAC;AACnB,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAC5C,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAC3D,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC/C,OAAO,EASL,KAAK,EACL,eAAe,EAChB,MAAM,YAAY,CAAC;AAIpB,MAAM,MAAM,WAAW,CACrB,CAAC,SAAS,KAAK,EACf,GAAG,GAAG,KAAK,EACX,GAAG,GAAG,KAAK,IACT,MAAM,CAAC,MAAM,CACf;KACG,CAAC,IAAI,MAAM,WAAW,CAAC,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;CAC/C,EACD,GAAG,EACH,GAAG,CACJ,CAAC;AAEF,MAAM,MAAM,WAAW,CAAC,CAAC,SAAS,KAAK,IAAI;KACxC,EAAE,IAAI,MAAM,CAAC,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC,SAC5C,MAAM,CAAC,QAAQ,CAAC,GAChB,SAAS,GACT,KAAK,GACL,KAAK,GACL,QAAQ,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC;CACrD,CAAC;AAEF,eAAO,MAAM,KAAK,GAChB,KAAK,CAAC,SAAS,SAAS,CAAC,UAAU,GAAG,WAAW,CAAC,EAAE,GAAG,KAAK,EAE5D,GAAG,WAAW,SAAS,KACtB,WAAW,CACZ,UAAU,CAAC,QAAQ,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,EACvC,KAAK,EACL,KAAK,GAAG,SAAS,CAAC,QAAQ,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,CAKtC,CAAC;AAEX,eAAO,MAAM,SAAS,GAAI,CAAC,SAAS,KAAK,EAAE,MAAM,CAAC,sBAqB3C,CAAC,gGAEJ,CAAC"}
+{"version":3,"file":"apply.d.ts","sourceRoot":"","sources":["../src/apply.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,MAAM,MAAM,eAAe,CAAC;AACxC,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,EAAE,GAAG,EAAE,MAAM,UAAU,CAAC;AAE/B,OAAO,EAGL,GAAG,EACJ,MAAM,kBAAkB,CAAC;AAI1B,OAAO,EAGL,KAAK,MAAM,EACX,KAAK,UAAU,EACf,KAAK,KAAK,EACV,KAAK,SAAS,EAEf,MAAM,WAAW,CAAC;AACnB,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAE5C,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAC3D,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC/C,OAAO,EASL,KAAK,EACL,eAAe,EAChB,MAAM,YAAY,CAAC;AAGpB,MAAM,MAAM,WAAW,CACrB,CAAC,SAAS,KAAK,EACf,GAAG,GAAG,KAAK,EACX,GAAG,GAAG,KAAK,IACT,MAAM,CAAC,MAAM,CACf;KACG,CAAC,IAAI,MAAM,WAAW,CAAC,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;CAC/C,EACD,GAAG,EACH,GAAG,CACJ,CAAC;AAEF,MAAM,MAAM,WAAW,CAAC,CAAC,SAAS,KAAK,IAAI;KACxC,EAAE,IAAI,MAAM,CAAC,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC,SAC5C,MAAM,CAAC,QAAQ,CAAC,GAChB,SAAS,GACT,KAAK,GACL,KAAK,GACL,QAAQ,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC;CACrD,CAAC;AAEF,eAAO,MAAM,KAAK,GAChB,KAAK,CAAC,SAAS,SAAS,CAAC,UAAU,GAAG,WAAW,CAAC,EAAE,GAAG,KAAK,EAE5D,GAAG,WAAW,SAAS,KACtB,WAAW,CACZ,UAAU,CAAC,QAAQ,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,EACvC,KAAK,EACL,KAAK,GAAG,SAAS,CAAC,QAAQ,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,CAKtC,CAAC;AAEX,eAAO,MAAM,SAAS,GAAI,CAAC,SAAS,KAAK,EAAE,MAAM,CAAC,sBAqB3C,CAAC,gGAEJ,CAAC"}

package/lib/apply.js

@@ -1,13 +1,14 @@
+import { Layer } from "effect";
 import * as Context from "effect/Context";
 import * as Effect from "effect/Effect";
 import { App } from "./app.js";
 import { CLI, } from "./cli/service.js";
-import { generateInstanceId } from "./instance-id.js";
+import { generateInstanceId, InstanceId } from "./instance-id.js";
 import * as Output from "./output.js";
 import { plan, } from "./plan.js";
+import { getProviderByType } from "./provider.js";
 import { State, StateStoreError, } from "./state.js";
 import { asEffect } from "./util.js";
-import { getProviderByType } from "./provider.js";
 export const apply = (...resources) => plan(...resources).pipe(Effect.flatMap((p) => applyPlan(p)));
 export const applyPlan = (plan) => Effect.gen(function* () {
     const cli = yield* CLI;
@@ -101,7 +102,7 @@ const expandAndPivot = Effect.fnUntraced(function* (plan, session) {
             };
         }
         return oldBindingOutput;
-    })));
+    })), { concurrency: "unbounded" });
     const apply = (node) => Effect.gen(function* () {
         const commit = (value) => state.set({
             stack: app.name,
@@ -130,7 +131,7 @@ const expandAndPivot = Effect.fnUntraced(function* (plan, session) {
                 return node.state.attr;
             }
             // resolve upstream dependencies before committing any changes to state
-            const upstream = Object.fromEntries(yield* Effect.all(Object.entries(Output.resolveUpstream(node.props)).map(([id]) => resolveUpstream(id).pipe(Effect.map(({ upstreamAttr }) => [id, upstreamAttr])))));
+            const upstream = Object.fromEntries(yield* Effect.all(Object.entries(Output.resolveUpstream(node.props)).map(([id]) => resolveUpstream(id).pipe(Effect.map(({ upstreamAttr }) => [id, upstreamAttr]))), { concurrency: "unbounded" }));
             const instanceId = yield* Effect.gen(function* () {
                 if (node.action === "create" && !node.state?.instanceId) {
                     const instanceId = yield* generateInstanceId();
@@ -174,281 +175,285 @@ const expandAndPivot = Effect.fnUntraced(function* (plan, session) {
                 // this should never happen
                 return yield* Effect.dieMessage(`Instance ID not found for resource '${id}' and action is '${node.action}'`);
             });
-            if (node.action === "create") {
-                const news = (yield* Output.evaluate(node.props, upstream));
-                const checkpoint = (attr) => commit({
-                    status: "creating",
-                    logicalId: id,
-                    instanceId,
-                    resourceType: node.resource.type,
-                    props: news,
-                    attr,
-                    providerVersion: node.provider.version ?? 0,
-                    bindings: node.bindings,
-                    downstream: node.downstream,
-                });
-                if (!node.state) {
-                    yield* checkpoint(undefined);
-                }
-                let attr;
-                if (node.action === "create" &&
-                    node.provider.precreate &&
-                    // pre-create is only designed to ensure the resource exists, if we have state.attr, then it already exists and should be skipped
-                    node.state?.attr === undefined) {
-                    yield* report("pre-creating");
-                    // stub the resource prior to resolving upstream resources or bindings if a stub is available
-                    attr = yield* node.provider.precreate({
+            const apply = Effect.gen(function* () {
+                if (node.action === "create") {
+                    const news = (yield* Output.evaluate(node.props, upstream));
+                    const checkpoint = (attr) => commit({
+                        status: "creating",
+                        logicalId: id,
+                        instanceId,
+                        resourceType: node.resource.type,
+                        props: news,
+                        attr,
+                        providerVersion: node.provider.version ?? 0,
+                        bindings: node.bindings,
+                        downstream: node.downstream,
+                    });
+                    if (!node.state) {
+                        yield* checkpoint(undefined);
+                    }
+                    let attr;
+                    if (node.action === "create" &&
+                        node.provider.precreate &&
+                        // pre-create is only designed to ensure the resource exists, if we have state.attr, then it already exists and should be skipped
+                        node.state?.attr === undefined) {
+                        yield* report("pre-creating");
+                        // stub the resource prior to resolving upstream resources or bindings if a stub is available
+                        attr = yield* node.provider.precreate({
+                            id,
+                            news: node.props,
+                            session: scopedSession,
+                            instanceId,
+                        });
+                        yield* checkpoint(attr);
+                    }
+                    yield* report("attaching");
+                    let bindingOutputs = yield* attachBindings({
+                        resource,
+                        bindings: node.bindings,
+                        target: {
+                            id,
+                            props: news,
+                            attr,
+                        },
+                    });
+                    yield* report("creating");
+                    attr = yield* node.provider.create({
                         id,
-                        news: node.props,
-                        session: scopedSession,
+                        news,
                         instanceId,
+                        bindings: bindingOutputs,
+                        session: scopedSession,
                     });
                     yield* checkpoint(attr);
-                }
-                yield* report("attaching");
-                let bindingOutputs = yield* attachBindings({
-                    resource,
-                    bindings: node.bindings,
-                    target: {
-                        id,
+                    yield* report("post-attach");
+                    bindingOutputs = yield* postAttachBindings({
+                        resource,
+                        bindings: node.bindings,
+                        bindingOutputs,
+                        target: {
+                            id,
+                            props: news,
+                            attr,
+                        },
+                    });
+                    yield* commit({
+                        status: "created",
+                        logicalId: id,
+                        instanceId,
+                        resourceType: node.resource.type,
                         props: news,
                         attr,
-                    },
-                });
-                yield* report("creating");
-                attr = yield* node.provider.create({
-                    id,
-                    news,
-                    instanceId,
-                    bindings: bindingOutputs,
-                    session: scopedSession,
-                });
-                yield* checkpoint(attr);
-                yield* report("post-attach");
-                bindingOutputs = yield* postAttachBindings({
-                    resource,
-                    bindings: node.bindings,
-                    bindingOutputs,
-                    target: {
+                        bindings: node.bindings.map((binding, i) => ({
+                            ...binding,
+                            attr: bindingOutputs[i],
+                        })),
+                        providerVersion: node.provider.version ?? 0,
+                        downstream: node.downstream,
+                    });
+                    yield* report("created");
+                    return attr;
+                }
+                else if (node.action === "update") {
+                    const upstream = Object.fromEntries(yield* Effect.all(Object.entries(Output.resolveUpstream(node.props)).map(([id]) => resolveUpstream(id).pipe(Effect.map(({ upstreamAttr }) => [id, upstreamAttr]))), { concurrency: "unbounded" }));
+                    const news = (yield* Output.evaluate(node.props, upstream));
+                    const checkpoint = (attr) => {
+                        if (node.state.status === "replaced") {
+                            return commit({
+                                ...node.state,
+                                attr,
+                                props: news,
+                            });
+                        }
+                        else {
+                            return commit({
+                                status: "updating",
+                                logicalId: id,
+                                instanceId,
+                                resourceType: node.resource.type,
+                                props: news,
+                                attr,
+                                providerVersion: node.provider.version ?? 0,
+                                bindings: node.bindings,
+                                downstream: node.downstream,
+                                old: node.state.status === "updating"
+                                    ? node.state.old
+                                    : node.state,
+                            });
+                        }
+                    };
+                    yield* checkpoint(node.state.attr);
+                    yield* report("attaching");
+                    let bindingOutputs = yield* attachBindings({
+                        resource,
+                        bindings: node.bindings,
+                        target: {
+                            id,
+                            props: news,
+                            attr: node.state.attr,
+                        },
+                    });
+                    yield* report("updating");
+                    const attr = yield* node.provider.update({
                         id,
-                        props: news,
-                        attr,
-                    },
-                });
-                yield* commit({
-                    status: "created",
-                    logicalId: id,
-                    instanceId,
-                    resourceType: node.resource.type,
-                    props: news,
-                    attr,
-                    bindings: node.bindings.map((binding, i) => ({
-                        ...binding,
-                        attr: bindingOutputs[i],
-                    })),
-                    providerVersion: node.provider.version ?? 0,
-                    downstream: node.downstream,
-                });
-                yield* report("created");
-                return attr;
-            }
-            else if (node.action === "update") {
-                const upstream = Object.fromEntries(yield* Effect.all(Object.entries(Output.resolveUpstream(node.props)).map(([id]) => resolveUpstream(id).pipe(Effect.map(({ upstreamAttr }) => [id, upstreamAttr])))));
-                const news = (yield* Output.evaluate(node.props, upstream));
-                const checkpoint = (attr) => {
+                        news,
+                        instanceId,
+                        bindings: bindingOutputs,
+                        session: scopedSession,
+                        olds: node.state.status === "created" ||
+                            node.state.status === "updated" ||
+                            node.state.status === "replaced"
+                            ? node.state.props
+                            : node.state.old.props,
+                        output: node.state.attr,
+                    });
+                    yield* checkpoint(attr);
+                    yield* report("post-attach");
+                    bindingOutputs = yield* postAttachBindings({
+                        resource,
+                        bindings: node.bindings,
+                        bindingOutputs,
+                        target: {
+                            id,
+                            props: news,
+                            attr,
+                        },
+                    });
                     if (node.state.status === "replaced") {
-                        return commit({
+                        yield* commit({
                             ...node.state,
                             attr,
                             props: news,
                         });
                     }
                     else {
-                        return commit({
-                            status: "updating",
+                        yield* commit({
+                            status: "updated",
                             logicalId: id,
                             instanceId,
                             resourceType: node.resource.type,
                             props: news,
                             attr,
+                            bindings: node.bindings.map((binding, i) => ({
+                                ...binding,
+                                attr: bindingOutputs[i],
+                            })),
                             providerVersion: node.provider.version ?? 0,
-                            bindings: node.bindings,
                             downstream: node.downstream,
-                            old: node.state.status === "updating"
-                                ? node.state.old
-                                : node.state,
                         });
                     }
-                };
-                yield* checkpoint(node.state.attr);
-                yield* report("attaching");
-                let bindingOutputs = yield* attachBindings({
-                    resource,
-                    bindings: node.bindings,
-                    target: {
-                        id,
-                        props: news,
-                        attr: node.state.attr,
-                    },
-                });
-                yield* report("updating");
-                const attr = yield* node.provider.update({
-                    id,
-                    news,
-                    instanceId,
-                    bindings: bindingOutputs,
-                    session: scopedSession,
-                    olds: node.state.status === "created" ||
-                        node.state.status === "updated" ||
-                        node.state.status === "replaced"
-                        ? node.state.props
-                        : node.state.old.props,
-                    output: node.state.attr,
-                });
-                yield* checkpoint(attr);
-                yield* report("post-attach");
-                bindingOutputs = yield* postAttachBindings({
-                    resource,
-                    bindings: node.bindings,
-                    bindingOutputs,
-                    target: {
-                        id,
-                        props: news,
-                        attr,
-                    },
-                });
-                if (node.state.status === "replaced") {
-                    yield* commit({
-                        ...node.state,
-                        attr,
-                        props: news,
-                    });
+                    yield* report("updated");
+                    return attr;
                 }
-                else {
-                    yield* commit({
-                        status: "updated",
+                else if (node.action === "replace") {
+                    if (node.state.status === "replaced") {
+                        // we've already created the replacement resource, return the output
+                        return node.state.attr;
+                    }
+                    let state;
+                    if (node.state.status !== "replacing") {
+                        yield* commit((state = {
+                            status: "replacing",
+                            logicalId: id,
+                            instanceId,
+                            resourceType: node.resource.type,
+                            props: node.props,
+                            attr: node.state.attr,
+                            providerVersion: node.provider.version ?? 0,
+                            deleteFirst: node.deleteFirst,
+                            old: node.state,
+                            downstream: node.downstream,
+                        }));
+                    }
+                    else {
+                        state = node.state;
+                    }
+                    const upstream = Object.fromEntries(yield* Effect.all(Object.entries(Output.resolveUpstream(node.props)).map(([id]) => resolveUpstream(id).pipe(Effect.map(({ upstreamAttr }) => [id, upstreamAttr]))), { concurrency: "unbounded" }));
+                    const news = (yield* Output.evaluate(node.props, upstream));
+                    const checkpoint = ({ status, attr, bindings, }) => commit({
+                        status,
                         logicalId: id,
                         instanceId,
                         resourceType: node.resource.type,
                         props: news,
                         attr,
-                        bindings: node.bindings.map((binding, i) => ({
-                            ...binding,
-                            attr: bindingOutputs[i],
-                        })),
                         providerVersion: node.provider.version ?? 0,
+                        bindings: bindings ?? node.bindings,
                         downstream: node.downstream,
-                    });
-                }
-                yield* report("updated");
-                return attr;
-            }
-            else if (node.action === "replace") {
-                if (node.state.status === "replaced") {
-                    // we've already created the replacement resource, return the output
-                    return node.state.attr;
-                }
-                let state;
-                if (node.state.status !== "replacing") {
-                    yield* commit((state = {
-                        status: "replacing",
-                        logicalId: id,
-                        instanceId,
-                        resourceType: node.resource.type,
-                        props: node.props,
-                        attr: node.state.attr,
-                        providerVersion: node.provider.version ?? 0,
+                        old: state.old,
                         deleteFirst: node.deleteFirst,
-                        old: node.state,
-                        downstream: node.downstream,
-                    }));
-                }
-                else {
-                    state = node.state;
-                }
-                const upstream = Object.fromEntries(yield* Effect.all(Object.entries(Output.resolveUpstream(node.props)).map(([id]) => resolveUpstream(id).pipe(Effect.map(({ upstreamAttr }) => [id, upstreamAttr])))));
-                const news = (yield* Output.evaluate(node.props, upstream));
-                const checkpoint = ({ status, attr, bindings, }) => commit({
-                    status,
-                    logicalId: id,
-                    instanceId,
-                    resourceType: node.resource.type,
-                    props: news,
-                    attr,
-                    providerVersion: node.provider.version ?? 0,
-                    bindings: bindings ?? node.bindings,
-                    downstream: node.downstream,
-                    old: state.old,
-                    deleteFirst: node.deleteFirst,
-                });
-                let attr;
-                if (node.provider.precreate &&
-                    // pre-create is only designed to ensure the resource exists, if we have state.attr, then it already exists and should be skipped
-                    node.state?.attr === undefined) {
-                    yield* report("pre-creating");
-                    // stub the resource prior to resolving upstream resources or bindings if a stub is available
-                    attr = yield* node.provider.precreate({
+                    });
+                    let attr;
+                    if (node.provider.precreate &&
+                        // pre-create is only designed to ensure the resource exists, if we have state.attr, then it already exists and should be skipped
+                        node.state?.attr === undefined) {
+                        yield* report("pre-creating");
+                        // stub the resource prior to resolving upstream resources or bindings if a stub is available
+                        attr = yield* node.provider.precreate({
+                            id,
+                            news: node.props,
+                            session: scopedSession,
+                            instanceId,
+                        });
+                        yield* checkpoint({
+                            status: "replacing",
+                            attr,
+                        });
+                    }
+                    yield* report("attaching");
+                    let bindingOutputs = yield* attachBindings({
+                        resource,
+                        bindings: node.bindings,
+                        target: {
+                            id,
+                            props: news,
+                            attr,
+                        },
+                    });
+                    yield* report("creating replacement");
+                    attr = yield* node.provider.create({
                         id,
-                        news: node.props,
-                        session: scopedSession,
+                        news,
                         instanceId,
+                        bindings: bindingOutputs,
+                        session: scopedSession,
                     });
                     yield* checkpoint({
                         status: "replacing",
                         attr,
                     });
-                }
-                yield* report("attaching");
-                let bindingOutputs = yield* attachBindings({
-                    resource,
-                    bindings: node.bindings,
-                    target: {
-                        id,
-                        props: news,
-                        attr,
-                    },
-                });
-                yield* report("creating replacement");
-                attr = yield* node.provider.create({
-                    id,
-                    news,
-                    instanceId,
-                    bindings: bindingOutputs,
-                    session: scopedSession,
-                });
-                yield* checkpoint({
-                    status: "replacing",
-                    attr,
-                });
-                yield* report("post-attach");
-                bindingOutputs = yield* postAttachBindings({
-                    resource,
-                    bindings: node.bindings,
-                    bindingOutputs,
-                    target: {
-                        id,
-                        props: news,
+                    yield* report("post-attach");
+                    bindingOutputs = yield* postAttachBindings({
+                        resource,
+                        bindings: node.bindings,
+                        bindingOutputs,
+                        target: {
+                            id,
+                            props: news,
+                            attr,
+                        },
+                    });
+                    yield* checkpoint({
+                        status: "replaced",
                         attr,
-                    },
-                });
-                yield* checkpoint({
-                    status: "replaced",
-                    attr,
-                    bindings: node.bindings.map((binding, i) => ({
-                        ...binding,
-                        attr: bindingOutputs[i],
-                    })),
-                });
-                yield* report("created");
-                return attr;
-            }
-            // @ts-expect-error
-            return yield* Effect.dieMessage(`Unknown action: ${node.action}`);
+                        bindings: node.bindings.map((binding, i) => ({
+                            ...binding,
+                            attr: bindingOutputs[i],
+                        })),
+                    });
+                    yield* report("created");
+                    return attr;
+                }
+                // @ts-expect-error
+                return yield* Effect.dieMessage(`Unknown action: ${node.action}`);
+            });
+            // provide the resource-specific context (InstanceId, etc.)
+            return yield* apply.pipe(Effect.provide(Layer.succeed(InstanceId, instanceId)));
         })));
     });
     return Object.fromEntries(yield* Effect.all(Object.entries(plan.resources).map(Effect.fn(function* ([id, node]) {
         return [id, yield* apply(node)];
-    }))));
+    })), { concurrency: "unbounded" }));
 });
 const collectGarbage = Effect.fnUntraced(function* (plan, session) {
     const state = yield* State;
@@ -507,7 +512,7 @@ const collectGarbage = Effect.fnUntraced(function* (plan, session) {
         return yield* (deletions[logicalId] ??= yield* Effect.cached(Effect.gen(function* () {
             yield* Effect.all(downstream.map((dep) => dep in deletionGraph
                 ? deleteResource(deletionGraph[dep])
-                : Effect.void));
+                : Effect.void), { concurrency: "unbounded" });
             yield* report("deleting");
             if (isDeleteNode(node)) {
                 yield* commit({
@@ -522,14 +527,16 @@ const collectGarbage = Effect.fnUntraced(function* (plan, session) {
                     bindings: node.bindings,
                 });
             }
-            yield* provider.delete({
-                id: logicalId,
-                instanceId,
-                olds: props,
-                output: attr,
-                session: scopedSession,
-                bindings: [],
-            });
+            if (attr !== undefined) {
+                yield* provider.delete({
+                    id: logicalId,
+                    instanceId,
+                    olds: props,
+                    output: attr,
+                    session: scopedSession,
+                    bindings: [],
+                });
+            }
             if (isDeleteNode(node)) {
                 // TODO(sam): should we commit a tombstone instead? and then clean up tombstones after all deletions are complete?
                 yield* state.delete({
@@ -553,10 +560,10 @@ const collectGarbage = Effect.fnUntraced(function* (plan, session) {
                 });
                 yield* report("replaced");
             }
-        })));
+        }).pipe(Effect.provide(Layer.succeed(InstanceId, instanceId)))));
     });
     yield* Effect.all(Object.values(deletionGraph)
         .filter((node) => node !== undefined)
-        .map(deleteResource));
+        .map(deleteResource), { concurrency: "unbounded" });
 });
 //# sourceMappingURL=apply.js.map