Skip to content

fix(pds): rename getAccountStatus to checkAccountStatus to match lexicon#123

Merged
ascorbic merged 1 commit intomainfrom
fix/check-account-status-lexicon
Feb 7, 2026
Merged

fix(pds): rename getAccountStatus to checkAccountStatus to match lexicon#123
ascorbic merged 1 commit intomainfrom
fix/check-account-status-lexicon

Conversation

@ascorbic
Copy link
Owner

@ascorbic ascorbic commented Feb 7, 2026

Summary

  • Renames com.atproto.server.getAccountStatus to com.atproto.server.checkAccountStatus to match the official AT Protocol lexicon
  • Updates route, handler, CLI client, and tests
  • Fixes third-party app login failures (e.g. stream.place) caused by the PDS returning HTML 404 instead of JSON when clients called the correct endpoint name

Fixes #110

Test plan

  • All 310 tests pass (226 unit + 84 CLI)

The AT Protocol lexicon defines this endpoint as
com.atproto.server.checkAccountStatus, not getAccountStatus.
@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
atproto-pds db17d9a Feb 07 2026, 10:15 AM

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 7, 2026

Open in StackBlitz

npm i https://pkg.pr.new/create-pds@123
npm i https://pkg.pr.new/@getcirrus/oauth-provider@123
npm i https://pkg.pr.new/@getcirrus/pds@123

commit: db17d9a

@ascorbic ascorbic merged commit 4e3d4e9 into main Feb 7, 2026
5 checks passed
@ascorbic ascorbic deleted the fix/check-account-status-lexicon branch February 7, 2026 10:21
@mixie-bot mixie-bot bot mentioned this pull request Feb 7, 2026
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.

stream.place login fails

1 participant