docs(markdown) Adding arguments of agents.gemini.discord.enabled=true and specify Gemini model.

[wesley990]

Adding arguments of agents.gemini.discord.enabled=true and specify Gemini model.
Per https://discord.com/channels/1491295327620169908/1491365158868619404/1498273257214840862

[CHC-Agent]

Review Summary

Verdict: 🟡 Minor issues → ✅ Fixed

What this PR does

Fixes two gaps in docs/gemini.md Helm install instructions:

1. Adds missing --set agents.gemini.discord.enabled=true — without this, the [discord] config block is never generated and the bot token is never injected, so Gemini silently fails to connect to Discord.
2. Documents how to specify a Gemini model via args (optional).

What was fixed in follow-up commit

• Typo: defalut → default
• Formatting: extra leading space before (Optional), missing space before backtick

Verdict

Good to merge. The enabled=true fix addresses a real documentation bug that would block new users.

[masami-agent]

PR Review: #595

Summary

• Problem: docs/gemini.md Helm install example is missing agents.gemini.discord.enabled=true, which means the [discord] config block is never generated — Gemini silently fails to connect to Discord.
• Approach: Add the missing flag and document optional model selection via args.
• Risk level: Low (docs-only change)

Core Assessment

1. Problem clearly stated: ✅ (referenced Discord discussion in PR body)
2. Approach appropriate: ✅
3. Alternatives considered: N/A (straightforward doc fix)
4. Best approach for now: ✅

Findings

Verified against the chart templates:

• configmap.yaml uses {{- if ($cfg.discord).enabled }} — without enabled=true, the entire [discord] block is omitted from config.toml, so the bot token is never configured. The fix is correct and necessary.
• CHC-Agent's follow-up commit fixed the typo (defalut → default) and formatting. Final state looks clean.

Review Summary

🔧 Suggested Changes

• The "Manual config.toml" section below still shows only args = ["--acp"]. Consider adding a brief note there as well showing how to specify a model in TOML format (e.g., args = ["--model", "gemini-3-pro-preview", "--acp"]). This would keep both sections consistent. Not blocking — can be done in a follow-up PR.

⚪ Nits

• PR title could be more concise: docs(gemini): add missing enabled flag and model args (minor, will be rewritten on squash merge anyway).

Verdict

APPROVE — correct and useful doc fix. Good to merge.

[obrutjack]

LGTM — correct and necessary doc fix. The enabled=true flag is required for the configmap template to generate the [discord] block. Clean final state after CHC-Agent's formatting fix.