Skip to content

docs: organize Python helpers and refresh README#334

Merged
kevincodex1 merged 4 commits intoGitlawb:mainfrom
Vasanthdev2004:feat/repo-root-cleanup-readme
Apr 4, 2026
Merged

docs: organize Python helpers and refresh README#334
kevincodex1 merged 4 commits intoGitlawb:mainfrom
Vasanthdev2004:feat/repo-root-cleanup-readme

Conversation

@Vasanthdev2004
Copy link
Copy Markdown
Collaborator

Summary

  • move the root-level Python helper files into python/
  • move the root-level Python tests into python/tests/
  • update the Python tests so they still import correctly after the move
  • refresh README.md into a cleaner structure that reflects current main features and the new repo layout

Verification

  • py -3 -m pytest python/tests/test_atomic_chat_provider.py python/tests/test_ollama_provider.py python/tests/test_smart_router.py
  • git grep -n -e "atomic_chat_provider.py" -e "ollama_provider.py" -e "smart_router.py" -e "test_atomic_chat_provider.py" -e "test_ollama_provider.py" -e "test_smart_router.py" -- . ':(exclude)docs/superpowers/**'
  • bun run build
  • bun run smoke

Scope

This is intentionally a low-risk organization/documentation pass only:

  • no runtime behavior changes to the TypeScript CLI
  • no extension path move
  • no broader repo reorg beyond the Python helper/test files

@Vasanthdev2004 Vasanthdev2004 marked this pull request as ready for review April 4, 2026 11:33
Copy link
Copy Markdown
Collaborator

@gnanam1990 gnanam1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review

Python test imports

Each test file adds sys.path.insert(0, str(Path(__file__).resolve().parents[1])) individually. This should be handled once in a conftest.py at python/tests/. Also missing python/__init__.py and python/tests/__init__.py — without these, pytest discovery is inconsistent across environments.

The file moves themselves are clean.


Provenance section

The legal notice was removed and replaced with a single vague line. If the goal is to reduce Claude Code references in the README, rewrite it to a short factual statement rather than removing it entirely — for example:

OpenClaude originated from the Claude Code codebase and has since been substantially modified to support multiple providers and open use. "Claude" and "Claude Code" are trademarks of Anthropic PBC. See LICENSE for details.

Removing the disclosure entirely is a maintainer-level decision and needs explicit sign-off before this merges.

Copy link
Copy Markdown
Contributor

@kevincodex1 kevincodex1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good for me

@kevincodex1 kevincodex1 merged commit 80a2f14 into Gitlawb:main Apr 4, 2026
1 check passed
@Vasanthdev2004 Vasanthdev2004 deleted the feat/repo-root-cleanup-readme branch April 5, 2026 07:51
euxaristia pushed a commit to euxaristia/openclaude that referenced this pull request Apr 13, 2026
* docs: organize Python helpers and refresh README

* docs: add README status badges

* test: centralize Python helper test imports

* docs: add short provenance disclaimer
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.

3 participants