Pass `opts` to wrapped Pulumi resources about ion HOT 8 CLOSED

This would also allow a workaround for #617

from ion.

@jayair , since there is a transform options for Pulumi args you could (breaking) change to transformArgs and transformOpts. This will solve the case for multiple resources inside a component and be much clearer (in my opinion).

from ion.

Hmm interesting. We'll need to consider the case when there are multiple resources inside a component.

from ion.

@remembell-root Which opts field are you looking to pass to child resources?

so, aws.sns.Topic only gets { parent } as opts.

Pulumi passes a subset of opts to child resources by default, ie. retainOnDelete

from ion.

In the latest version of SST, u can transform opts like this

new sst.aws.Bucket("MyBucket", {
  transform: {
    bucket: (args, opts) => {
      opts.retainOnDelete = true;
    },
  },
});

from ion.

@fwang, thanks for implementing this.

I'm sorry, I don't understand why use function transform and not pass the entire opts from the constructor (as suggested).

from ion.

Cos the bucket component has multiple resources in it

from ion.

@jayair , could you please make notes for next major release to discuss the api change I suggested? It will make my code much cleaner and simpler to understand.

from ion.