fix(group_chat_context): use .get() to avoid KeyError when config profile lacks image_caption_prompt

[tsaitang404]

Description

When group_chat_context.py accesses the config via cfg["provider_settings"]["image_caption_prompt"], it raises KeyError if the config profile was created before this field was added.

Root Cause

group_chat_context.py:59 uses direct dict indexing:

image_caption_prompt = cfg["provider_settings"]["image_caption_prompt"]

cfg comes from get_config(umo=...) which may return an older profile without this key.

Fix

Use .get() with default fallback:

image_caption_prompt = cfg.get("provider_settings", {}).get("image_caption_prompt", "Please describe the image using Chinese.")

Summary by Sourcery

Bug Fixes:

• Avoid KeyError when loading configs that lack the image_caption_prompt by using a safe lookup with a default prompt.

[gemini-code-assist]

Code Review

This pull request modifies group_chat_context.py to safely retrieve image_caption_prompt from the configuration using .get() with a default value. The reviewer identified a potential AttributeError if provider_settings is explicitly set to None and provided a safer alternative using (cfg.get("provider_settings") or {}) to handle this case.

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.

[gemini-code-assist]

If provider_settings is explicitly configured as null (or parsed as None), cfg.get("provider_settings", {}) will return None instead of {}. Calling .get() on None will then raise an AttributeError. Using (cfg.get("provider_settings") or {}) is safer and prevents potential runtime crashes.

Suggested change

	image_caption_prompt = cfg.get("provider_settings", {}).get("image_caption_prompt", "Please describe the image using Chinese.")
	image_caption_prompt = (cfg.get("provider_settings") or {}).get("image_caption_prompt", "Please describe the image using Chinese.")

[sourcery-ai]

Hey - I've left some high level feedback:

• You still access cfg["provider_ltm_settings"] with direct indexing; if older configs can also miss this key, consider using .get() or validating the structure before use to avoid similar KeyErrors.
• The hard-coded fallback prompt "Please describe the image using Chinese." may not be appropriate for all deployments; consider pulling this default from a shared constant or configuration so language and phrasing can be adjusted centrally.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- You still access `cfg["provider_ltm_settings"]` with direct indexing; if older configs can also miss this key, consider using `.get()` or validating the structure before use to avoid similar `KeyError`s.
- The hard-coded fallback prompt "Please describe the image using Chinese." may not be appropriate for all deployments; consider pulling this default from a shared constant or configuration so language and phrasing can be adjusted centrally.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	astrbot-docs	d0e5e68	Jul 12 2026, 08:24 AM