71bb9a6295
Two bugs surfaced by the first dev smoke test: 1. Profile JSON declared "llm": "gemini" (lowercase). llm_config's dispatcher compares model_name == "Gemini" case-sensitively (matches the rest of the codebase), so the check fell through to "Invalid model selected" and never reached the API. Every other profile uses "Gemini" with capital G. Spec mistake — fixed. 2. get_client_from_profile() resolves the per-report output folder from the profile_id via hardcoded prefix matches. No 'hp_' branch existed, so hp_copy_review reports landed under output-dev/general/ instead of output-dev/hp/ — the UI then couldn't find them. Added 'hp_' → 'hp' alongside the existing mappings. The check itself works correctly otherwise: profile_source was user_selected, brand resolved to 'hp', and the reference asset was successfully attached. Bug 1 just prevented Gemini from being called. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
14 lines
313 B
JSON
14 lines
313 B
JSON
{
|
|
"name": "HP Copy Review",
|
|
"description": "Marketing copy graded against canonical HP Source Messaging",
|
|
"mode": "asset",
|
|
"visibility": "client_specific",
|
|
"visible_to_clients": ["hp"],
|
|
"checks": {
|
|
"hp_copy_review": {
|
|
"weight": 10.0,
|
|
"llm": "Gemini",
|
|
"enabled": true
|
|
}
|
|
}
|
|
}
|