feat(coding-agent): disable thinking block rendering - #4
Conversation
This comment has been minimized.
This comment has been minimized.
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
| - name: Build offline | ||
| run: npm run build:offline |
There was a problem hiding this comment.
The switch to build:offline lets stale model data pass CI indefinitely: validateGeneratedModelData (packages/ai/scripts/model-data.ts:209-210) checks generatedAt for parseability only, never age or upstream. Add a freshness guard to check-model-data or a scheduled workflow that runs npm run hydrate:model-data and opens a PR when the baseline is outdated.
- name: Build offline
run: npm run build:offline
- name: Verify model data freshness
run: node packages/ai/scripts/check-model-data.ts --require-freshPrompt for LLM
File .github/workflows/ci.yml:
Line 35 to 36:
The switch to build:offline lets stale model data pass CI indefinitely: validateGeneratedModelData (packages/ai/scripts/model-data.ts:209-210) checks generatedAt for parseability only, never age or upstream. Add a freshness guard to check-model-data or a scheduled workflow that runs npm run hydrate:model-data and opens a PR when the baseline is outdated.
Suggested Code:
- name: Build offline
run: npm run build:offline
- name: Verify model data freshness
run: node packages/ai/scripts/check-model-data.ts --require-fresh
Talk to Kody by mentioning @kody
Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction.
Summary
Adds a new
disableThinkingBlocksetting that completely prevents thinking blocks from being rendered in assistant output, rather than just replacing them with a short label like the existinghideThinkingBlockoption.Changes
disableThinkingBlock(default:true) — when enabled, thinking blocks are omitted entirely from the UI, with no placeholder label shown. This means thinking content is hidden by default unless a user explicitly disables this setting.hideThinkingBlocksemantics — the existing option now only controls whether a short "Thinking..." label is shown in place of thinking blocks. It has no effect whendisableThinkingBlockis enabled.