[segunmailerlite]

Why?

• The Java SDK only had e-commerce models for automation triggers, so it couldn't reach shops, orders, variants or the revenue reports from the API - Ecommerce & Revenue project.

Changes

• New ml.ecommerce() with shops, products, variants, categories, customers, orders, carts, cart items, imports and settings, using the SDK's retriever/builder pattern.
• ecommerceReport() on campaigns, subscribers, automations and shops for revenue stats and attributed orders.

[mailerlite-kritika]

Confidence Score: 4/5

[Medium risk] The review's account says it read no sources outside the repository, so the endpoint paths and query parameter names (for example ecommerce_shop_id on automation order-activity and the /ecommerce/shops routes) are unverified against the MailerLite API and rest only on the tests' expected URLs, which the same author wrote. The earlier null-endpoint problem in orders() is fixed in the diff, so it no longer lowers the score. The change is additive SDK code with no auth, secret or …

Findings

Earlier findings (3 resolved)

• src/main/java/com/mailerlite/sdk/automations/Automations.java:62 ecommerceEmailReport exposes orders() that throws NullPointerException · correctness · resolved
• src/main/java/com/mailerlite/sdk/ecommerce/EcommerceReportRetriever.java:86 orders() throws NullPointerException when no orders endpoint was given · correctness · resolved
• src/test/java/com/mailerlite/sdk/tests/EcommerceTest.java:67 Capturing client is left installed in the global factory after each test · tests · resolved

Summary

Adds ecommerce API objects and revenue reports to the Java SDK

The change adds ml.ecommerce() with shops, products, variants, categories, customers, orders, carts, cart items, imports and settings. It also adds ecommerceReport() on campaigns, subscribers, automations and shops, all built on a shared EcommerceReportRetriever. The latest commits make orders() throw a MailerLiteException when a report has no orders endpoint, and restore the HTTP client after each test.

What's good

• Tests check method, URL and JSON body for nearly every endpoint without hitting the network.
• EcommerceRequest URL-encodes query parameters, so bracketed filter names are safe.

Review context · 0/5 commands

• Commands run: none

_{Reviews (2) · Last reviewed commit: "ECO-600: Nits" · claude-sonnet-5-5}