Skip to content

refactor: change thoughtLogging default from true to false - #4

Merged
wei840222 merged 1 commit into
mainfrom
refactor/thought-logging-default-change
Jun 21, 2026
Merged

refactor: change thoughtLogging default from true to false#4
wei840222 merged 1 commit into
mainfrom
refactor/thought-logging-default-change

Conversation

@ani6439walc

@ani6439walc ani6439walc commented Jun 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • Changed thoughtLogging default value from true to false
  • Updated all related configurations and tests
  • Modified openclaw.plugin.json, src/config.ts, src/tool.ts, and test files

Changes

  • Default behavior is now quieter (no console logging unless explicitly enabled)
  • All related tests updated to reflect new default
  • Documentation updated

Closes #NA

@github-actions

Copy link
Copy Markdown

Jules PR Review

Summary

This PR changes the default value of the thoughtLogging configuration option from true to false. The updates are applied consistently across the README documentation, plugin schema definition (openclaw.plugin.json), Zod configuration schema (src/config.ts), test suite (src/config.test.ts), and the tool's runtime fallback (src/tool.ts). Overall, this is a clean and straightforward change that will help reduce default console noise.

🔴 Blocking Issues

No blocking issues found.

🟡 Suggestions

  • Backwards Compatibility: Changing a default value is technically a breaking change for users who implicitly relied on thoughtLogging being enabled by default. Consider whether this repository follows strict semantic versioning. Ensure this change is communicated clearly in the release notes (or released as a major version bump if appropriate) so users know they must explicitly set "thoughtLogging": true in their configuration if they still want to see the logs.

🧪 Tests & Verification

Test coverage looks good.

Verdict

✅ Approve
The PR implements the intended default change cleanly and consistently across code, tests, and documentation.

Jules session: https://jules.google.com/session/12721304354978828369


Reviewed by Jules via GitHub Actions.

@wei840222
wei840222 merged commit d783775 into main Jun 21, 2026
1 check passed
@wei840222
wei840222 deleted the refactor/thought-logging-default-change branch June 21, 2026 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants