docs(config): add missing duckduckgo, exec, and qq sections to example config#514
Merged
yinwm merged 1 commit intosipeed:mainfrom Feb 20, 2026
Conversation
…e config config.example.json was missing three sections that exist in the Go config structs and defaults: - tools.web.duckduckgo: DuckDuckGo is enabled by default in defaults.go and requires no API key (free search provider), but users who copy the example config silently lose it since the section was omitted. - tools.exec: The ExecConfig struct supports enable_deny_patterns and custom_deny_patterns for security hardening, but users had no way to discover these options from the example. - channels.qq: The QQ channel was the only channel in ChannelsConfig missing from the example while all others were present. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
|
@CrisisAlpha Great catch on the missing config sections! Especially the DuckDuckGo one being the only free search provider enabled by default, users starting from the example config would have lost web search without knowing. Really helpful fix. We have the PicoClaw Dev Group on Discord where contributors connect. If you're interested, email |
hyperwd
pushed a commit
to hyperwd/picoclaw
that referenced
this pull request
Mar 5, 2026
…d-missing-sections docs(config): add missing duckduckgo, exec, and qq sections to example config
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Description
config.example.jsonwas missing three sections that exist in the Go config structs (pkg/config/config.go) and have defaults inpkg/config/defaults.go:tools.web.duckduckgo— DuckDuckGo search is enabled by default indefaults.goand requires no API key (it's the only free web search provider). Users who copy the example config as their starting point silently lose web search because this section was omitted entirely.tools.exec— TheExecConfigstruct supportsenable_deny_patternsandcustom_deny_patternsfor security hardening, but the example config didn't surface these options.channels.qq— QQ was the only channel defined inChannelsConfigthat was missing from the example while all 9 other channels were present.🗣️ Type of Change
🤖 AI Code Generation
🔗 Related Issue
Addresses the Documentation & DX goal in ROADMAP.md (Section 5): "Creating clear, user-friendly documentation for both setup and development."
📚 Technical Context (Skip for Docs)
pkg/config/defaults.golines 245-265 (DuckDuckGo default),pkg/config/config.golines 392-421 (struct definitions)🧪 Test Environment
make fmt && make vet && make testall pass☑️ Checklist
Made with Cursor