feat: typed builder

[kewynakshlley]

No description provided.

[cubic-dev-ai]

3 issues found across 50 files

Confidence score: 3/5

• There is a concrete regression risk in src/main/java/com/resend/services/automations/model/SendEmailStepBuilder.java: build() can emit template.id even when templateId is null, which may produce invalid email-step payloads at runtime.
• src/main/java/com/resend/services/automations/model/AbstractStepBuilder.java currently shares the mutable config map with built objects; without a defensive copy, later builder mutations can unexpectedly alter previously built AutomationStep instances.
• src/main/java/com/resend/services/automations/model/ContactUpdateStepBuilder.java should copy the input map in properties(...) to prevent aliasing and potential failures when property(...) mutates an immutable map.
• Pay close attention to src/main/java/com/resend/services/automations/model/SendEmailStepBuilder.java, src/main/java/com/resend/services/automations/model/AbstractStepBuilder.java, src/main/java/com/resend/services/automations/model/ContactUpdateStepBuilder.java - null template payloads and mutable map aliasing are the main correctness risks before merge.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/main/java/com/resend/services/automations/model/ContactUpdateStepBuilder.java">

<violation number="1" location="src/main/java/com/resend/services/automations/model/ContactUpdateStepBuilder.java:64">
P2: Copy the input map in `properties(...)` to avoid aliasing and immutable-map mutation failures during fluent `property(...)` calls.</violation>
</file>

<file name="src/main/java/com/resend/services/automations/model/AbstractStepBuilder.java">

<violation number="1" location="src/main/java/com/resend/services/automations/model/AbstractStepBuilder.java:66">
P2: Build should pass a defensive copy of `config` to avoid mutating already-built `AutomationStep` instances through later builder changes.</violation>
</file>

<file name="src/main/java/com/resend/services/automations/model/SendEmailStepBuilder.java">

<violation number="1" location="src/main/java/com/resend/services/automations/model/SendEmailStepBuilder.java:98">
P1: `build()` unconditionally adds `template.id` even when `templateId` is null, which can produce invalid `template` payloads.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

The merge-base changed after approval.