Skip to content

Add a top-level example configuration file - #79

Merged
glenjbarber merged 1 commit into
mainfrom
claude/relaxed-knuth-1frpkt
Oct 13, 2026
Merged

glenjbarber merged 1 commit into
mainfrom
claude/relaxed-knuth-1frpkt

Conversation

@glenjbarber

Copy link
Copy Markdown
Owner

What

Adds orcli.json.example at the repository root: every top-level member Config reads, plus every optional block (cloudflare, notion, github, apiary, mcp), in one copy-pasteable file. Previously a reader had to piece the full shape together from scattered snippets in README.md.

Why

Requested as a quick addition alongside other work in this session: a single example config makes first-run setup (cp orcli.json.example ~/.orcli.json, fill in values, chmod 0600) much easier than assembling the shape from README prose.

Testing

Added TestExampleConfigLoadsThroughTheRealParser in internal/config/load_test.go, which reads the example straight from the repository root and runs it through the real Parse/CloudflareAPIKey/ApiarySettings/MCPServers path - so a future field added to Config without a matching update to the example fails CI instead of silently going stale.

make test passes clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Nxr4mW9rMNzn9kThjdJpZt


Generated by Claude Code

orcli.json.example shows every top-level member Config reads, plus
every optional block (cloudflare, notion, github, apiary, mcp), in one
copy-pasteable file - previously a reader had to piece the full shape
together from scattered snippets in README.md.

Added TestExampleConfigLoadsThroughTheRealParser, which reads the
example straight from the repository root and runs it through the
real Parse/CloudflareAPIKey/ApiarySettings/MCPServers path, so a
future field added to Config without a matching update to the example
fails CI instead of silently going stale.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Nxr4mW9rMNzn9kThjdJpZt
@glenjbarber
glenjbarber marked this pull request as ready for review October 13, 2026 17:30
@glenjbarber
glenjbarber merged commit f6fff65 into main Oct 13, 2026
1 check passed
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