Skip to content

feat: clarify and simplify switching between cloud and local - #340

Merged
siillee merged 2 commits into
mainfrom
fix-clarify-switching-between-cloud-and-local
Aug 13, 2026
Merged

feat: clarify and simplify switching between cloud and local#340
siillee merged 2 commits into
mainfrom
fix-clarify-switching-between-cloud-and-local

Conversation

@siillee

@siillee siillee commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

This PR clarifies and simplifies how to switch between cloud and local mode in Claude and Codex integrations.

@siillee siillee self-assigned this Aug 13, 2026
@siillee siillee added the core-team Core-team authored/owned PR label Aug 13, 2026
@github-actions

Copy link
Copy Markdown

A 21-file refactor that somehow shipped without a single bug—clearly the work of someone avoiding real problems. 🎯

🟢 No blocking issues found.

Architecture highlights:

  • COGNEE_BACKEND=local correctly scrubs cloud vars from os.environ (not just config), so HTTP hot paths and child processes stay local
  • Forced cloud mode is pinned—no silent fallback to local when misconfigured; status line shows ✕ (missing_cognee_base_url) immediately
  • Plugin-specific vars (COGNEE_CLAUDE_BACKEND / COGNEE_CODEX_BACKEND) correctly beat the shared COGNEE_BACKEND; cross-plugin vars are properly ignored
  • Empty string guards in session-start.py prevent boot attempts when target_url="" (forced cloud, no URL)
  • Test coverage is comprehensive: env scrubbing, child process survival, precedence, status line, doctor diagnostics

Code quality:

  • Clean separation: _env_file.py scrubs environ, config.py sets _forced_backend flag, status line reads both
  • UTF-8 encoding fix for Windows test harness (bonus improvement)
  • Documentation is clear and thorough

See inline comments for details.

No fixes needed — ship it.

@siillee
siillee merged commit 85aa774 into main Aug 13, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core-team Core-team authored/owned PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant