feat: Add settings to control diagnostic messages (#5524)#5582

Merged

mrubens merged 31 commits intomainfrom

feat/issue-5524-diagnostic-settings

Jul 23, 2025

Commits on Jul 23, 2025

feat: Add settings to control diagnostic messages (#5524)

hannesrudolph
authored and
daniel-lxs
committed
fix: address PR #5582 review feedback

hannesrudolph
authored and
daniel-lxs
committed
fix: ensure 'Include diagnostic messages in context' checkbox persists unchecked state

hannesrudolph
authored and
daniel-lxs
committed
fix: preserve explicit false values in diagnostic settings synchronization

hannesrudolph
authored and
daniel-lxs
committed
feat: diagnostic settings working correctly - checkbox state persists, slider adjustments maintain checkbox state

hannesrudolph
authored and
daniel-lxs
committed
feat: enhance diagnostic settings UI with reset button and unlimited option

hannesrudolph
authored and
daniel-lxs
committed
feat: improve unlimited diagnostics UI with integrated slider design

hannesrudolph
authored and
daniel-lxs
committed
test: fix insertContentTool mock to match actual implementation

hannesrudolph
authored and
daniel-lxs
committed
fix: respect maxDiagnosticMessages setting in workspace problems mention

hannesrudolph
authored and
daniel-lxs
committed
fix: update diagnostic slider description dynamically based on value

hannesrudolph
authored and
daniel-lxs
committed
chore(i18n): remove unused translation keys

hannesrudolph
authored and
daniel-lxs
committed
fix: make diagnostic settings description static

hannesrudolph
authored and
daniel-lxs
committed
fix: resolve localization issues in ContextManagementSettings

hannesrudolph
authored and
daniel-lxs
committed
fix: refactor diagnostic settings to follow project patterns

hannesrudolph
authored and
daniel-lxs
committed
fix: add null checks to integration test to fix TypeScript errors

hannesrudolph
authored and
daniel-lxs
committed
fix: correct parameter name mismatch for diagnostic settings

hannesrudolph
authored and
daniel-lxs
committed
chore: remove temporary analysis files

hannesrudolph
authored and
daniel-lxs
committed
chore: remove diagnostics integration test (out of scope for this PR)

hannesrudolph
authored and
daniel-lxs
committed
fix: address bot review comments

hannesrudolph
authored and
daniel-lxs
committed
feat: implement size-based limiting for diagnostic messages

roomote
authored and
daniel-lxs
committed
fix: update ContextManagementSettings tests to handle UI component mocks correctly

roomote
authored and
daniel-lxs
committed
fix: change diagnostic limiting from size-based to count-based

hannesrudolph
authored and
daniel-lxs
committed
fix: add Button component import to ContextManagementSettings
daniel-lxs
committed
fix: address PR #5582 review feedback for diagnostic messages feature
daniel-lxs
committed
fix: add missing writeDelayMs prop to ContextManagementSettings test
daniel-lxs
committed
revert: remove unintended whitespace formatting change in insertContentTool.ts
daniel-lxs
committed
fix: update Content Security Policy to remove redundant webview.cspSource references
daniel-lxs
committed
fix: revert unrelated CSP changes in ClineProvider.ts
daniel-lxs
committed
feat: add missing translations for diagnostic settings
daniel-lxs
committed
Delete src/core/webview/__tests__/diagnosticSettingsBugFix.spec.ts
daniel-lxs
authored
Delete src/core/webview/__tests__/diagnosticSettingsPersistence.spec.ts
daniel-lxs
authored

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Add settings to control diagnostic messages (#5524)#5582

feat: Add settings to control diagnostic messages (#5524)#5582
mrubens merged 31 commits intomainfrom
feat/issue-5524-diagnostic-settings

Commits on Jul 23, 2025

feat: Add settings to control diagnostic messages (#5524)

fix: address PR #5582 review feedback

fix: ensure 'Include diagnostic messages in context' checkbox persists unchecked state

fix: preserve explicit false values in diagnostic settings synchronization

feat: diagnostic settings working correctly - checkbox state persists, slider adjustments maintain checkbox state

feat: enhance diagnostic settings UI with reset button and unlimited option

feat: improve unlimited diagnostics UI with integrated slider design

test: fix insertContentTool mock to match actual implementation

fix: respect maxDiagnosticMessages setting in workspace problems mention

fix: update diagnostic slider description dynamically based on value

chore(i18n): remove unused translation keys

fix: make diagnostic settings description static

fix: resolve localization issues in ContextManagementSettings

fix: refactor diagnostic settings to follow project patterns

fix: add null checks to integration test to fix TypeScript errors

fix: correct parameter name mismatch for diagnostic settings

chore: remove temporary analysis files

chore: remove diagnostics integration test (out of scope for this PR)

fix: address bot review comments

feat: implement size-based limiting for diagnostic messages

fix: update ContextManagementSettings tests to handle UI component mocks correctly

fix: change diagnostic limiting from size-based to count-based

fix: add Button component import to ContextManagementSettings

fix: address PR #5582 review feedback for diagnostic messages feature

fix: add missing writeDelayMs prop to ContextManagementSettings test

revert: remove unintended whitespace formatting change in insertContentTool.ts

fix: update Content Security Policy to remove redundant webview.cspSource references

fix: revert unrelated CSP changes in ClineProvider.ts

feat: add missing translations for diagnostic settings

Delete src/core/webview/tests/diagnosticSettingsBugFix.spec.ts

Delete src/core/webview/tests/diagnosticSettingsPersistence.spec.ts

feat: Add settings to control diagnostic messages (#5524)#5582mrubens merged 31 commits intomainRooCodeInc/Roo-Code:mainfrom feat/issue-5524-diagnostic-settingsRooCodeInc/Roo-Code:feat/issue-5524-diagnostic-settingsCopy head branch name to clipboard

Commits

Commits on Jul 23, 2025

feat: Add settings to control diagnostic messages (#5524)#5582
mrubens merged 31 commits intomainfrom
feat/issue-5524-diagnostic-settings