[CONTENT SPRINT] Identity Stack Cookbook: Auth0, Okta, Entra ID, and Keycloak#18962
[CONTENT SPRINT] Identity Stack Cookbook: Auth0, Okta, Entra ID, and Keycloak#18962sicarul wants to merge 4 commits into
Conversation
Social Media Reviewcontent/blog/identity-stack-cookbook-auth0-okta-azuread-keycloak/index.mdX — PASSLinkedIn — PASSBluesky — PASSSuggestions (advisory)These are stylistic notes — they don't block the post. X
Updated for commit |
Docs review — Identity Stack CookbookThanks for putting this together, Pablo! Here is a review focused on what is user-visible and the contents of the PR itself. The blog post itself is solid in shape, but the PR currently bundles a lot of files that should not ship with it, and the prose and code examples are inconsistent with each other on a few points. Blocking issues1. PR is shipping internal tooling and scratch files The PR includes a large
Please scope this PR down to just the blog post and the five example programs before merging. 2. Naming inconsistency — "Azure AD" vs "AzureAD" vs "Entra ID" The title and
"AzureAD" as one word is not a Microsoft product name; it only exists as the Pulumi provider package name ( Note that this also affects the directory slug Content issues3. Code samples do not demonstrate what the prose promises ( The "The pattern" section says the cookbook will show four elements: Application, Group, SCIM Provisioning, and Custom Claims. None of the four code samples actually implement SCIM or custom claims:
Either trim the "pattern" list to match what the code actually shows (Application + Group), or add SCIM and a custom-claim mapper to each example. The current shape is a credibility risk for the post. 4. Awkward sentence (
"While using different stacks" is ambiguous (different IdP stacks? Different cloud stacks?). Suggested rewrite for the whole opening paragraph: 5. Example programs export values that do not appear in the blog snippets Each 6. Minor: Auth0 prerequisites are terse ( "Configured with the tenant domain, client ID, and client secret" is brief for a cookbook entry. Either link to the provider's installation/configuration docs, or note that these are typically set via Nits
If you would like another pass after the scope-down and the AzureAD/Entra ID + SCIM cleanup, mention @claude and I will re-review. |
|
Your site preview for commit 2b2e3c1 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-18962-2b2e3c15.s3-website.us-west-2.amazonaws.com |
Lighthouse Performance ReportCommit: 8a5bbfc | Metric definitions
|
8a5bbfc to
65e892e
Compare
65e892e to
d0df473
Compare
|
@claude please review again. I rewrote this branch history to a single clean commit on current |
|
Claude finished @sicarul's task in 1m 43s —— View job Docs re-review — Identity Stack CookbookThanks for the cleanup, Pablo! The PR scope is now clean (just the three blog files), the social frontmatter is correctly split, and prerequisites are in place for all four providers. Most of the structural feedback is addressed. But the rename pass introduced a real bug in the code, and the "Azure AD / Entra ID" naming is still inconsistent. There are also a couple of items from the prior review that are still open.
✅ Fixed
❌ Blocking — broken code sample
signInAudience: "Microsoft Entra IDMyOrg",That value does not exist. This looks like a global ❌ Naming still inconsistentThe title,
Recommended approach: pick one — "Microsoft Entra ID" on first mention with a single
|
|
@claude please review again. Branch history is now a single clean commit on current origin/master. The PR diff contains only content/blog/identity-stack-cookbook-auth0-okta-azuread-keycloak/index.md plus feature.png and meta.png. Social-review is passing, and actionable content feedback has been addressed. |
|
Claude finished @sicarul's task in 1m 44s —— View job Docs re-review — Identity Stack CookbookThanks for the rebase, Pablo! The PR scope is now clean and the social frontmatter looks good. However, the content fixes from the prior review do not appear to have landed — the same blocking issues are still present in the file at commit
✅ Fixed since prior review
❌ Still blocking — broken code sample at line 119The signInAudience: "Microsoft Entra IDMyOrg",That string is not a valid Microsoft API value. ❌ Still blocking — naming still inconsistentTitle/
Recommended: pick one — "Microsoft Entra ID" on first mention with a single
|
|
@claude please review again. This request is after the latest pushed head commit |
|
Claude finished @sicarul's task in 1m 11s —— View job Docs re-review — Identity Stack Cookbook
Hi Pablo — the PR is still pointing at the same commit ( ❌ Still blocking — broken
|
d0df473 to
2a82b56
Compare
|
@claude please review again. This PR is now marked as draft, the needs-design label has been removed, and the latest pushed head is |
|
@claude please retry the review. The previous review request appears to have failed or stalled due to GitHub rate limits. Please review the latest HEAD and confirm whether the prior feedback is addressed. |
|
Claude finished @sicarul's task in 1m 16s —— View job Docs re-review — Identity Stack Cookbook
Thanks Pablo — reviewed at ✅ Fixed since prior review
|
|
@claude please re-review the latest normal commit on top. I addressed the remaining pending feedback, including optional polish, for this PR. |
|
Claude finished @sicarul's task in 1m 28s —— View job Docs re-review — Identity Stack Cookbook
Thanks Pablo — reviewed at ✅ Fixed since prior review
All earlier blockers — broken Nits (non-blocking)
Summary: All prior blockers and polish items addressed. Auth0 and Keycloak samples are now properly wired (not just disconnected resources), names won't collide if pasted side-by-side, and the AzureAD comment lands well. Suggest a one-line |
|
@claude #new-review |
a9474fe to
b9f669d
Compare
|
🤖 @sicarul — pinned review cleared; regenerating from scratch. |
|
@claude #new-review |
|
🤖 @sicarul — pinned review cleared; regenerating from scratch. |
b9f669d to
2b2e3c1
Compare
|
@claude #new-review |
Pre-merge Review — Last updated 2026-05-19T03:22:45ZTip Summary: New blog post ( Review confidence:
Investigation log
🔍 Verification trail57 claims extracted · 29 verified · 17 unverifiable · 1 contradicted
Claim verification reported errors — some verdicts may be incomplete; spot-check the affected claims in-review. 📊 Editorial balanceSingle-subject post; balance check N/A. 🚨 Outstanding in this PRThese must be resolved or refuted before merging. No outstanding issues.
|
|
🤖 Review regenerated on @sicarul's request. |
Why this content is interesting
What we took into account
Why it is useful to an end user
Design need
needs-designbecause the post needs a final meta image before publication.feature.pngandmeta.pngfiles are placeholders unless Design chooses to reuse or adapt them.Metadata
🤖 Generated with OpenCode