Repository navigation
fix(doctor): point COOKIE_DOMAIN_NOT_SET at a real docs URL - #159
sheldon-workos wants to merge 2 commits into
Conversation
The remediation linked to https://workos.com/docs/authkit/cookie-domain, which 404s. Repoint to the authkit-nextjs README's "Optional configuration" section, which is where WORKOS_COOKIE_DOMAIN is actually documented. Co-authored-by: Cursor <[email protected]>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates the documentation URL reference in the doctor module's issue definitions. The ChangesDocumentation URL Update
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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. Comment |
[Medium impact] The PR appears safe to merge, with the previous documentation mismatch addressed. SummaryUpdates cookie-domain guidance in
The previous finding is addressed. No new actionable issues were found. Reviews (2) · Last reviewed commit: "fix(doctor): only warn about WORKOS_COOK..." · Reviewed by Greptile |
COOKIE_DOMAIN_NOT_SET fired for every AuthKit SDK, but only authkit-nextjs and authkit-tanstack-start read WORKOS_COOKIE_DOMAIN. Gate the warning on those two and link each to its first-party docs page under /docs/sdks/ instead of the Next.js GitHub README.
Summary
workos doctoremits the warning below when AuthKit is detected andWORKOS_COOKIE_DOMAINisn't set:That docs URL 404s — there is no
/authkit/cookie-domainpage in the WorkOS docs. The env var is canonically documented in theauthkit-nextjsREADME ("Optional configuration" section). This PR repoints thedocsUrlthere so the recommendation is actionable.Test plan
pnpm typecheckpassespnpm testpasses (1800/1800)pnpm buildpasseshttps://github.com/workos/authkit-nextjs#optional-configurationresolves to the table that documentsWORKOS_COOKIE_DOMAINNote on related broken URLs
While verifying, I noticed a few other
workos.com/docs/authkit/*URLs in this file (and insrc/doctor/checks/auth-patterns.ts) that also 404 today:https://workos.com/docs/authkit/redirect-uri(issues.ts:42, auth-patterns.ts:324)https://workos.com/docs/authkit/sign-out(auth-patterns.ts:139)https://workos.com/docs/authkit/nextjs/middleware(auth-patterns.ts:194, 216)https://workos.com/docs/authkit/nextjs/setup(auth-patterns.ts:254)Left out of this PR to keep the change surgical — happy to do a follow-up that audits all
docsUrlfields against the live docs site if that's wanted.Made with Cursor
Summary by CodeRabbit