[codex] add Flux cover art skill#3388
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThe PR adds Flux 2 Dev as Melious.ai’s image-generation model, wires it into FTUE/default profile seeding and profile upgrades, introduces a new Flux cover-art skill with compact task-summary prompts, and updates docs, metainfo, changelog, and tests. ChangesMelious Flux cover art and seeding
Sequence Diagram(s)sequenceDiagram
participant JournalAudio
participant availableSkillsForEntityProvider
participant FluxSkill as "Generate Cover Art (Flux)"
participant SkillPromptBuilder
participant Flux2Dev as "Flux 2 Dev"
JournalAudio->>availableSkillsForEntityProvider: requests imageGeneration skills for a task-linked voice note
availableSkillsForEntityProvider->>FluxSkill: exposes the taskSummary cover-art skill
FluxSkill->>SkillPromptBuilder: builds the compact Scene and Mood brief
SkillPromptBuilder->>Flux2Dev: sends the cover-art prompt
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request integrates the Flux 2 Dev image-generation model into Melious.ai, introducing a new 'Generate Cover Art (Flux)' skill that utilizes a compact task summary prompt instead of full task JSON. It updates the onboarding setup, profile seeding, and documentation to support this model. Feedback on the changes points out a potential performance bottleneck and string truncation issue with surrogate pairs in the prompt builder's text compaction helper, suggesting a safer and more efficient implementation.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3388 +/- ##
==========================================
- Coverage 99.20% 99.20% -0.01%
==========================================
Files 1679 1679
Lines 117621 117677 +56
==========================================
+ Hits 116682 116737 +55
- Misses 939 940 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
1 participant
Summary
Generate Cover Art (Flux)skill while keeping the original cover-art skill unchanged.black-forest-labs/flux-2-devas the default image-generation model for new and untouched Melious profiles.Impact
Task-linked text notes and voice notes now show a Flux-specific cover-art action in the AI popup. Users can still choose the image model from the existing provider/model picker, and Melious defaults route image generation to Flux 2 Dev.
Validation
fvm dart format lib/features/ai/helpers/skill_prompt_builder.dart lib/features/ai/skills/built_in_skills.dart lib/features/ai/state/skill_trigger_providers.dart lib/features/ai/ui/settings/services/melious_ftue_setup.dart lib/features/ai/util/known_models.dart lib/features/ai/util/known_models_data.dart lib/features/ai/util/known_models_ftue.dart lib/features/ai/util/profile_seeding_service.dart test/features/ai/helpers/skill_prompt_builder_test.dart test/features/ai/skills/built_in_skills_test.dart test/features/ai/state/skill_trigger_providers_test.dart test/features/ai/ui/settings/services/melious_ftue_setup_test.dart test/features/ai/util/known_models_test.dart test/features/ai/util/profile_seeding_service_slots_test.dart test/features/ai/util/profile_seeding_service_test.dartfvm flutter analyze lib/features/ai/helpers/skill_prompt_builder.dart lib/features/ai/skills/built_in_skills.dart lib/features/ai/state/skill_trigger_providers.dart lib/features/ai/ui/settings/services/melious_ftue_setup.dart lib/features/ai/util/known_models.dart lib/features/ai/util/known_models_data.dart lib/features/ai/util/known_models_ftue.dart lib/features/ai/util/profile_seeding_service.dart test/features/ai/helpers/skill_prompt_builder_test.dart test/features/ai/skills/built_in_skills_test.dart test/features/ai/state/skill_trigger_providers_test.dart test/features/ai/ui/settings/services/melious_ftue_setup_test.dart test/features/ai/util/known_models_test.dart test/features/ai/util/profile_seeding_service_slots_test.dart test/features/ai/util/profile_seeding_service_test.dartfvm flutter test test/features/ai/helpers/skill_prompt_builder_test.dart test/features/ai/skills/built_in_skills_test.dart test/features/ai/state/skill_trigger_providers_test.dart test/features/ai/util/known_models_test.dart test/features/ai/util/profile_seeding_service_test.dart test/features/ai/util/profile_seeding_service_slots_test.dart test/features/ai/ui/settings/services/melious_ftue_setup_test.dartgit diff --cached --checkpython3 -c "import sys, xml.etree.ElementTree as ET; ET.parse(sys.argv[1])" flatpak/com.matthiasn.lotti.metainfo.xmlNote:
xmllintis not installed in this environment, so XML was checked with Python's standard-library parser.Summary by CodeRabbit
New Features
Bug Fixes