Skip to content

Rename *Controller to *Adapter - #4

Merged
jimmy-phantom merged 2 commits into
mainfrom
rename-controller-to-adapter
Apr 7, 2026
Merged

Rename *Controller to *Adapter#4
jimmy-phantom merged 2 commits into
mainfrom
rename-controller-to-adapter

Conversation

@jimmy-phantom

@jimmy-phantom jimmy-phantom commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Renames QueryController, RESTQueryController, and TopicQueryController to QueryAdapter, RESTQueryAdapter, and TopicQueryAdapter
  • Renames static controllerstatic adapter, controllersadapters, controllerClassadapterClass, getController()getAdapter()
  • Updates all source files, tests, and documentation to match
  • Includes a minor changeset (breaking change, pre-1.0)

Test plan

  • npm run check-types passes
  • All 1124 unit tests pass
  • CI passes

🤖 Generated with Claude Code

Breaking change: QueryController, RESTQueryController, and
TopicQueryController are now QueryAdapter, RESTQueryAdapter, and
TopicQueryAdapter. The `static controller` property on Query/Mutation
classes is now `static adapter`, and the `controllers` option on
QueryClient is now `adapters`. Also removes stale v1.0.0 note from README.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jimmy-phantom
jimmy-phantom force-pushed the rename-controller-to-adapter branch from 853ab88 to d6cee29 Compare April 7, 2026 19:45
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jimmy-phantom
jimmy-phantom requested a review from pzuraq April 7, 2026 20:23
@jimmy-phantom
jimmy-phantom merged commit 2107b32 into main Apr 7, 2026
1 check passed
@jimmy-phantom
jimmy-phantom deleted the rename-controller-to-adapter branch April 7, 2026 21:14
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.

2 participants