Skip to content

fix: restore deps, ignores, and lazy lib init after rename#339

Merged
cdoern merged 1 commit intomainfrom
fix-main-after-rename
Apr 30, 2026
Merged

fix: restore deps, ignores, and lazy lib init after rename#339
cdoern merged 1 commit intomainfrom
fix-main-after-rename

Conversation

@cdoern
Copy link
Copy Markdown
Collaborator

@cdoern cdoern commented Apr 30, 2026

Codegen stripped manual edits during the llama_stack_client → ogx_client rename. This restores them and adds defensive measures so the next codegen wipe is less catastrophic.

Changes

  • pyproject.toml deps: re-add tqdm, rich, click, pyaml, prompt_toolkit, pandas, termcolor, fire, requests (mcp_oauth + lib/cli need these).
  • pyproject.toml per-file-ignores: re-add TC004/I/B for examples/, src/ogx_client/lib/, and tests/integration/ (hand-written code, not codegen-sorted).
  • src/ogx_client/lib/init.py: lazy-import get_oauth_token_for_mcp_server via getattr so `import ogx_client` no longer transitively requires requests/fire. Defensive — if codegen strips deps again, the package still imports.
  • .gitignore + delete .claude/settings.local.json: stop tracking Claude Code session state.

Verification

  • `uvx ruff@0.14.13 check .` → clean
  • `uv run python -c 'import ogx_client'` → ok

- pyproject: restore tqdm/rich/click/pyaml/prompt_toolkit/pandas/termcolor/fire/requests deps that codegen stripped during the rename
- pyproject: restore TC004/I/B per-file-ignores for examples/, src/ogx_client/lib/, and tests/integration/ (codegen wiped these too)
- src/ogx_client/lib/__init__.py: lazy-import get_oauth_token_for_mcp_server so 'import ogx_client' doesn't pull requests/fire if codegen wipes deps again
- .gitignore + remove tracked .claude/settings.local.json
@cdoern cdoern merged commit b36f998 into main Apr 30, 2026
56 checks passed
@cdoern cdoern deleted the fix-main-after-rename branch April 30, 2026 17:45
@stainless-app stainless-app Bot mentioned this pull request Apr 30, 2026
cdoern pushed a commit that referenced this pull request May 2, 2026
Automated Release PR
---


## 0.8.0-alpha.1 (2026-05-01)

Full Changelog:
[v0.7.2-alpha.4...v0.8.0-alpha.1](v0.7.2-alpha.4...v0.8.0-alpha.1)

### Features

* **cli:** wire up ogx-client CLI entrypoint
([#341](#341))
([d67b7f3](d67b7f3))


### Bug Fixes

* restore deps, ignores, and lazy lib init after rename
([#339](#339))
([b36f998](b36f998))


### Chores

* drop Meta licensing (hook + headers)
([#342](#342))
([7a863d3](7a863d3))
* **internal:** reformat pyproject.toml
([2e4bf09](2e4bf09))

---
This pull request is managed by Stainless's [GitHub
App](https://github.com/apps/stainless-app).

The [semver version
number](https://semver.org/#semantic-versioning-specification-semver) is
based on included [commit
messages](https://www.conventionalcommits.org/en/v1.0.0/).
Alternatively, you can manually set the version number in the title of
this pull request.

For a better experience, it is recommended to use either rebase-merge or
squash-merge when merging this pull request.

🔗 Stainless [website](https://www.stainlessapi.com)
📚 Read the [docs](https://app.stainlessapi.com/docs)
🙋 [Reach out](mailto:support@stainlessapi.com) for help or questions

---------

Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>
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.

1 participant