Add partner integration guide and mermaid diagrams#45730
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
6 Skipped Deployments
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
github-actions
Bot
added
the
documentation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughAdds a Mermaid renderer component and dependency, wires it into MDX for inline diagrams, introduces a partner integration guide with simple and signed-JWT redirect flows, and updates the docs navigation to link the new guide. ChangesMermaid Diagram Support & Partner Integration Guide
Sequence DiagramsequenceDiagram
participant Browser
participant MDX as MDX renderer
participant Mermaid as Mermaid component
participant Renderer as beautiful-mermaid
participant DOM
Browser->>MDX: render MDX with `Mermaid` node
MDX->>Mermaid: pass `chart` prop
Mermaid->>Renderer: renderMermaidSVG(chart, theme)
Renderer-->>Mermaid: SVG markup
Mermaid->>DOM: injectSVG(innerHTML)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
♻️ Duplicate comments (1)
apps/docs/content/guides/integrations/partner-integration-guide.mdx (1)
205-205:⚠️ Potential issue | 🟠 Major | ⚡ Quick winFix docs_lint blocker:
keypairspelling at Line 205
docs_lint / supa-mdx-lintfails on this exact token (Rule003Spelling), so CI remains red until this is corrected.Suggested patch
-| `kid` | Yes | The key ID identifying the keypair. Use this to pick the correct public key when verifying the signature. | +| `kid` | Yes | The key ID identifying the key pair. Use this to pick the correct public key when verifying the signature. |🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@apps/docs/content/guides/integrations/partner-integration-guide.mdx` at line 205, The docs lint failure is caused by the token "keypair" in the table cell for `kid`; update the wording to the correct spelling "key pair" (e.g., change "The key ID identifying the keypair..." to "The key ID identifying the key pair...") so the `kid` row no longer triggers Rule003Spelling; ensure only that token is changed and no other table formatting is altered.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Duplicate comments:
In `@apps/docs/content/guides/integrations/partner-integration-guide.mdx`:
- Line 205: The docs lint failure is caused by the token "keypair" in the table
cell for `kid`; update the wording to the correct spelling "key pair" (e.g.,
change "The key ID identifying the keypair..." to "The key ID identifying the
key pair...") so the `kid` row no longer triggers Rule003Spelling; ensure only
that token is changed and no other table formatting is altered.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro
Run ID: 577fd6bf-9133-4a84-b3bc-7d552c81b005
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (5)
apps/docs/components/Mermaid.tsxapps/docs/components/Navigation/NavigationMenu/NavigationMenu.constants.tsapps/docs/content/guides/integrations/partner-integration-guide.mdxapps/docs/features/docs/MdxBase.shared.tsxapps/docs/package.json
🎭 Playwright Test Results
Details
Flaky testsFeatures › table-editor.spec.ts › table editor › should show rls accordingly Skipped testsFeatures › auth-users.spec.ts › should show web3 users as enabled when the matching web3 provider is enabled |
Braintrust eval report
|
3 participants
Summary by CodeRabbit
New Features
Documentation
Chores