[shyim]

Move Shopmon integration management from the controller into ShopmonIntegrationService. Installation and updates refresh the dedicated ACL role and create an integration automatically when both SHOPMON_ACCESS_KEY and SHOPMON_ACCESS_SECRET are set.

Adopt existing FroshShopmon integrations and their stored setup data without rotating credentials or creating duplicates. Repeated manual setup also preserves credentials. Removal cleans up the integration, role, and configuration for both plugins.

The README documents environment setup and the requirement to keep user data when uninstalling FroshShopmon, whose uninstall routine otherwise deletes the adopted integration.

Validation: full PHP suite passed on Shopware 6.7 / PHP 8.3: 439 tests, 5,066 assertions, one skipped. Includes regression coverage for legacy adoption, credential preservation, ACL updates, environment-based setup, malformed configuration, and removal.

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The changes since the previous review appear safe to merge.

Summary

Moves Shopmon setup, status, and removal into ShopmonIntegrationService.

• Shopmon setup keeps matching credentials and adopts legacy integrations.
• FroshTools initializes Shopmon during install and update.

Since the previous review, the only change extracts the existing permissions list into PERMISSIONS. No new issues were found. No previous review threads were supplied.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Plugin installation or update] --> B[Initialize Shopmon]
    B --> C{Existing integration?}
    C -->|Yes| D[Refresh permissions and preserve credentials]
    C -->|No| E{Both environment credentials set?}
    E -->|Yes| F[Create integration and save setup data]
    E -->|No| G[Leave shop unconfigured]
    H[Manual setup] --> I[Reuse stored credentials or create credentials]
    J[Remove integration] --> K[Delete integrations, roles, and both plugins' settings]

Loading

_{Reviews (4) · Last reviewed commit: "refactor: define Shopmon permissions as ..." · Reviewed by Greptile}

[shyim]

@frosh-ci fix-cs

[frosh-automation]

Code style has been fixed and pushed to the branch.