Skip to content

feat: manufacturer description generation (P0 #3) - #7

Merged
roboshyim merged 1 commit into
mainfrom
feat/manufacturer-copy
Jul 21, 2026
Merged

feat: manufacturer description generation (P0 #3)#7
roboshyim merged 1 commit into
mainfrom
feat/manufacturer-copy

Conversation

@roboshyim

Copy link
Copy Markdown
Collaborator

What

Adds AI-generated manufacturer descriptions — the manufacturer half of P0 roadmap item #3. Completes P0 #3 (category & manufacturer copy). Mirrors the just-merged category description feature (#6) exactly.

Changes

  • ManufacturerDescriptionGenerator (src/Feature/ManufacturerDescription/): feature-flag gates, language resolution, shared tone presets, manufacturer-specific prompt (name, website link, existing description), Capability::TextGeneration, HTML sanitization. Reuses ProductDescriptionToneStore.
  • ManufacturerDescriptionController: POST /api/_action/frosh-ai/manufacturer/generate-description + GET /api/_action/frosh-ai/manufacturer/description-status.
  • FeatureId::ManufacturerDescription + featureManufacturerDescription config checkbox (en + de-DE).
  • services.xml registrations.

Tests

ManufacturerDescriptionGeneratorTest (5 tests): master-off / missing-name gates, tone+language+HTML sanitization, locale resolution, default tone. Updated FeatureFlagsTest (feature count 6→7).

106 tests / 416 assertions, green — locally and in the frosh-ai-dev container (Shopware 6.7 / PHP 8.3).

Notes

Admin UI button is a follow-up (same as category). This lands generator + API + flag + tests.

…g, tests)

Mirrors the category description feature for product manufacturers:
- ManufacturerDescriptionGenerator (name/link/existing description context)
- ManufacturerDescriptionController (generate + status endpoints)
- FeatureId::ManufacturerDescription + featureManufacturerDescription config flag
- Unit tests mirroring CategoryDescriptionGeneratorTest
@roboshyim
roboshyim merged commit 972ff22 into main Jul 21, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant