Skip to content

refactor: make context configuration consumer-owned - #26

Merged
codeforester merged 1 commit into
mainfrom
enhancement/23-20260731-generalize-context-config-api
Aug 1, 2026
Merged

refactor: make context configuration consumer-owned#26
codeforester merged 1 commit into
mainfrom
enhancement/23-20260731-generalize-context-config-api

Conversation

@codeforester

Copy link
Copy Markdown
Contributor

Summary

  • remove Base-shaped configuration dataclasses from the generic base_cli.config module
  • make Context.user_config opaque and project workspace-root projection through CliProfile
  • update the generic API, tests, README, consumer-profile docs, and changelog for the new boundary

Validation

  • uv run --extra dev pytest -q
  • all base-cli tests pass

The coordinated Base consumer migration is in basefoundry/base#1841.

Fixes #23

@codeforester
codeforester marked this pull request as ready for review August 1, 2026 02:32
@codeforester
codeforester merged commit 8c3854f into main Aug 1, 2026
11 checks passed
@codeforester
codeforester deleted the enhancement/23-20260731-generalize-context-config-api branch August 1, 2026 02:32
codeforester added a commit to basefoundry/base that referenced this pull request Aug 1, 2026
## Summary

- move Base-owned `UserConfig` dataclasses and parsing types into
`base_cli_adapters`
- adapt Base's profile to pass opaque consumer configuration and project
its workspace root through the generic profile callback
- replace Base's reads of the removed `Context.base_home` field with
neutral `Context.application_home`

## Validation

-
`PYTHONPATH=/Users/rameshhp/work/base-cli-worktrees/23-generalize-context-config-api/lib/python:cli/python
/Users/rameshhp/.base.d/base/.venv/bin/python -m pytest cli/python -q`
- `841 passed, 1 skipped, 248 subtests passed`
- `git diff --check`

Depends on basefoundry/base-cli#26.

Fixes #1841
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.

Generalize the remaining base-cli context and configuration API

1 participant