docs: weekly drift audit (2026-05-04)#119
Open
github-actions[bot] wants to merge 1 commit intomainfrom
Open
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated weekly docs audit. See workflow:
.github/workflows/docs-audit.yml.Fixes committed in this PR
marketing/src/app/docs/android-sdk/page.tsx— Removed dead-code callrift.checkDeferredDeepLink(clipboardText = null)and its misleading comments from the Android deferred deep linking example.client/mobile/ffi/src/lib.rs:510-511shows that callingcheck_deferred_deep_linkwithNoneimmediately returnsOk(None)without performing any attribution or link fetch. The comment above the call ("One call: reports attribution + fetches link data") was factually wrong. The correct Android pattern (reportAttributionForLink+getLink) was already present two lines below — only the dead code and its framing were removed.README.md— Removedworker/andworker-slack/from the architecture directory listing. Neither directory exists in the repository (lsat repo root confirms;git log -- worker/ worker-slack/shows they were removed in commitd57d8f1). Listing non-existent directories as part of the repo structure is factually wrong.Judgment calls for human review (NOT committed)
marketing/src/app/docs/webhooks/page.tsx:177— "Failed deliveries are retried 4 times".server/src/services/webhooks/dispatcher.rs:169showslet delays = [0, 1, 5, 25];— 4 total attempts including the first (delay=0), so strictly 3 retries. However CLAUDE.md itself says "4 retries" using the same counting convention, so this may be intentional terminology. Not fixing without human confirmation.marketing/src/lib/agent-docs.ts:152— References${siteUrl}/mcp/server.jsonas "MCP Manifest" in Integrations. The actual MCP manifest route in the server is/.well-known/mcp.json(perapi/mod.rs). This path discrepancy exists but theagent-docs.tsfile is thellms.txtcontent generator — if/mcp/server.jsonis a valid marketing-site Next.js route, it may be intentionally separate from the API. Needs human review.Out of scope (flagged, not fixed)
No OpenAPI/utoipa annotation issues found. The
api-reference/page is auto-rendered from the OpenAPI spec and was not edited.