Skip to content

Docs: APNs token endpoints route under /device/ not /database/ (#382)#415

Merged
leogdion merged 1 commit into
v1.0.0-beta.3from
docs/cloudkit-apns-endpoint-path
Jul 1, 2026
Merged

Docs: APNs token endpoints route under /device/ not /database/ (#382)#415
leogdion merged 1 commit into
v1.0.0-beta.3from
docs/cloudkit-apns-endpoint-path

Conversation

@leogdion

Copy link
Copy Markdown
Member

Documentation-only. Records that Apple's archived CloudKit Web Services REST reference documents tokens/create / tokens/register under /database/, but the live service only routes them under /device/ (the /database/ path returns 405 Method Not Allowed, Allow: OPTIONS). MistKit already uses /device/; this captures the divergence so it isn't re-discovered.

Added a "Known Endpoint Discrepancies" section to .claude/docs/QUICK_REFERENCE.md (documented-vs-working path table + CloudKit JS setApiModuleName("device") rationale).

Closes #382.

…382)

Apple's archived CloudKit Web Services REST reference documents
tokens/create and tokens/register under /database/, but the live
service only routes them under /device/ (the /database/ path returns
405 Method Not Allowed). MistKit already uses /device/; this captures
the divergence so it isn't re-discovered.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: eeba1055-eba9-49bf-832f-3d017e3a0430

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/cloudkit-apns-endpoint-path

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@leogdion leogdion marked this pull request as ready for review July 1, 2026 00:35
@leogdion leogdion merged commit 7791b09 into v1.0.0-beta.3 Jul 1, 2026
8 checks passed
@leogdion leogdion deleted the docs/cloudkit-apns-endpoint-path branch July 1, 2026 00:35
@claude

claude Bot commented Jul 1, 2026

Copy link
Copy Markdown

Review

Documentation-only PR — small, well-scoped, and accurate. Verified the claims against the current state of the repo:

Minor/optional nits (non-blocking):

  • The comparison tables header row is empty (| | Path |). Purely cosmetic — renders fine on GitHub — but a header like "Source | Path" would read slightly better and avoid an empty header cell.
  • Slight redundancy: the "documented vs actually works" table repeats information already stated in the paragraph immediately above it. Not a real problem, just flagging in case you want to trim.
  • Nice touch calling out that /device/ is container-scoped (no {database} segment) — an easy detail to miss when skimming and worth keeping prominent.

No code changes, no test coverage impact (nothing executable changed), no security or performance concerns. This looks good to merge as-is.

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