Skip to content

refactor: remove schema helper API - #137

Merged
PsiACE merged 1 commit into
mainfrom
refactor/helpers-as-convenience
Aug 30, 2026
Merged

refactor: remove schema helper API#137
PsiACE merged 1 commit into
mainfrom
refactor/helpers-as-convenience

Conversation

@PsiACE

@PsiACE PsiACE commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove acp.helpers and its top-level re-exports.
  • Construct generated schema models directly in examples, contrib code, and golden acceptance cases.
  • Update current docs to make direct model construction the standard path.

Related issues

Closes #47

Testing

  • make gen-check
  • make check
  • make test (197 passed, 1 skipped)
  • uv run mkdocs build --strict
  • Loaded every script under examples/ with the http extra.

No schema regeneration was needed.

Docs & screenshots

README, quickstart, and examples were updated. No visual changes.

Checklist

  • Conventional Commit title (e.g. feat:, fix:).
  • Tests cover the change or are not required (explain above).
  • Docs/examples updated when behaviour is user-facing.
  • Schema regenerations (make gen-all) are called out if applicable.

@PsiACE
PsiACE marked this pull request as ready for review August 30, 2026 19:34
@PsiACE
PsiACE merged commit 713cbc1 into main Aug 30, 2026
16 checks passed
@PsiACE
PsiACE deleted the refactor/helpers-as-convenience branch August 30, 2026 19:37
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.

feat: improve schema usability type instantiation

1 participant