Skip to content
This repository was archived by the owner on Aug 8, 2026. It is now read-only.

feat(coding-agent): disable thinking block rendering - #4

Merged
cheetahbyte merged 2 commits into
mainfrom
feat/hide-thinking-blocks
Aug 6, 2026
Merged

feat(coding-agent): disable thinking block rendering#4
cheetahbyte merged 2 commits into
mainfrom
feat/hide-thinking-blocks

Conversation

@cheetahbyte

@cheetahbyte cheetahbyte commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Summary

Adds a new disableThinkingBlock setting that completely prevents thinking blocks from being rendered in assistant output, rather than just replacing them with a short label like the existing hideThinkingBlock option.

Changes

  • New setting 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.
  • Updated hideThinkingBlock semantics — the existing option now only controls whether a short "Thinking..." label is shown in place of thinking blocks. It has no effect when disableThinkingBlock is enabled.
  • Interactive settings UI — adds a new "Disable thinking blocks" toggle alongside the existing "Hide thinking" option.
  • Real-time application — the setting is applied when starting the interactive mode, when reloading a session, and when changed from the settings selector. Chat views are rebuilt on toggle so the display updates immediately, including any in-progress streaming responses.
  • Documentation — the settings reference now documents both options and their default values.
  • Tests — added coverage verifying thinking blocks are completely omitted when disabled, and that the setting defaults to enabled while preserving the existing hide setting.

@kody-ai

This comment has been minimized.

Comment thread packages/coding-agent/src/core/settings-manager.ts
@kody-ai

kody-ai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

Comment thread .github/workflows/ci.yml
Comment on lines +35 to +36
- name: Build offline
run: npm run build:offline

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kody code-review Bug high

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-fresh
Prompt 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.

@cheetahbyte
cheetahbyte merged commit 32d74b0 into main Aug 6, 2026
2 checks passed
@cheetahbyte
cheetahbyte deleted the feat/hide-thinking-blocks branch August 6, 2026 04:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant