Conversation
…multi-tenant evolution of Cal.diy
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_94656575-8043-47c7-9240-f0e4d1b6dafe) |
There was a problem hiding this comment.
Code Review
This pull request updates the README.md to rebrand the project to Crove Cal, detailing its enterprise features, system architecture, Model Context Protocol (MCP) server tools, and deployment instructions. The review feedback highlights rendering issues in the document, specifically pointing out invalid HTML nesting in the header and a syntax error in the Mermaid architecture diagram that prevents it from rendering correctly.
| <p align="center"> | ||
| <a href="https://github.com/calcom/cal.diy"> | ||
| <img src="https://user-images.githubusercontent.com/8019099/210054112-5955e812-a76e-4160-9ddd-58f2c72f1cce.png" alt="Logo"> | ||
| </a> | ||
|
|
||
| <h3 align="center">Cal.diy</h3> | ||
|
|
||
| <img src="https://user-images.githubusercontent.com/8019099/210054112-5955e812-a76e-4160-9ddd-58f2c72f1cce.png" alt="Crove Cal Logo" width="120" /> | ||
| <h2 align="center">Crove Cal</h2> | ||
| <p align="center"> | ||
| The community-driven, open-source scheduling platform. | ||
| <strong>The Enterprise-Grade, Multi-Tenant, AI-Agent-Powered Scheduling Platform</strong> | ||
| <br /> | ||
| <a href="https://github.com/calcom/cal.diy"><strong>GitHub</strong></a> | ||
| <br /> | ||
| <a href="https://github.com/calcom/cal.diy/issues">Issues</a> | ||
| · | ||
| <a href="./CONTRIBUTING.md">Contributing</a> | ||
| <em>An advanced open evolution of Cal.diy with full native Organizations, Teams, MCP Agent Protocol & DOS ID SSO.</em> | ||
| </p> | ||
| </p> |
There was a problem hiding this comment.
The outer <p> tag contains block-level elements (<h2> and <p>), which is invalid HTML according to the HTML specification (a <p> element cannot contain other block-level elements). This can cause rendering and layout issues in some markdown parsers. Replacing the outer <p> with a <div> and removing redundant align="center" attributes (since they inherit from the parent div) resolves this issue.
| <p align="center"> | |
| <a href="https://github.com/calcom/cal.diy"> | |
| <img src="https://user-images.githubusercontent.com/8019099/210054112-5955e812-a76e-4160-9ddd-58f2c72f1cce.png" alt="Logo"> | |
| </a> | |
| <h3 align="center">Cal.diy</h3> | |
| <img src="https://user-images.githubusercontent.com/8019099/210054112-5955e812-a76e-4160-9ddd-58f2c72f1cce.png" alt="Crove Cal Logo" width="120" /> | |
| <h2 align="center">Crove Cal</h2> | |
| <p align="center"> | |
| The community-driven, open-source scheduling platform. | |
| <strong>The Enterprise-Grade, Multi-Tenant, AI-Agent-Powered Scheduling Platform</strong> | |
| <br /> | |
| <a href="https://github.com/calcom/cal.diy"><strong>GitHub</strong></a> | |
| <br /> | |
| <a href="https://github.com/calcom/cal.diy/issues">Issues</a> | |
| · | |
| <a href="./CONTRIBUTING.md">Contributing</a> | |
| <em>An advanced open evolution of Cal.diy with full native Organizations, Teams, MCP Agent Protocol & DOS ID SSO.</em> | |
| </p> | |
| </p> | |
| <div align="center"> | |
| <img src="https://user-images.githubusercontent.com/8019099/210054112-5955e812-a76e-4160-9ddd-58f2c72f1cce.png" alt="Crove Cal Logo" width="120" /> | |
| <h2>Crove Cal</h2> | |
| <p> | |
| <strong>The Enterprise-Grade, Multi-Tenant, AI-Agent-Powered Scheduling Platform</strong> | |
| <br /> | |
| <em>An advanced open evolution of Cal.diy with full native Organizations, Teams, MCP Agent Protocol & DOS ID SSO.</em> | |
| </p> | |
| </div> |
| SupabaseAuth[Supabase OIDC Server\nRS256 / ES256 PKCE] | ||
| CustomHook[Custom Access Token Hook\nInjects 'organizations' & 'role'] | ||
| end | ||
|
|
||
| subgraph CroveCalApp [Crove Cal Service - cal.crove.com] | ||
| NextAuth[NextAuth OIDC Provider\nDosIdProvider] | ||
| JIT[JIT Sync Logic\nsyncDosOrganizations] | ||
| WebhookEndpoint[/api/webhooks/dos-org-sync\nHMAC-SHA256 Signed] |
There was a problem hiding this comment.
The Mermaid diagram contains two issues:
- The node
WebhookEndpoint[/api/webhooks/dos-org-sync\nHMAC-SHA256 Signed]uses[/which Mermaid interprets as the start of a parallelogram shape, but it is closed with a standard]instead of/]. This unbalanced syntax causes a Mermaid parsing error and prevents the diagram from rendering. - Literal
\ncharacters are used for line breaks inside node labels, which are not universally supported or rendered correctly in all Mermaid versions. Using standard HTML<br/>tags inside double quotes is the recommended way to define multi-line labels.
| SupabaseAuth[Supabase OIDC Server\nRS256 / ES256 PKCE] | |
| CustomHook[Custom Access Token Hook\nInjects 'organizations' & 'role'] | |
| end | |
| subgraph CroveCalApp [Crove Cal Service - cal.crove.com] | |
| NextAuth[NextAuth OIDC Provider\nDosIdProvider] | |
| JIT[JIT Sync Logic\nsyncDosOrganizations] | |
| WebhookEndpoint[/api/webhooks/dos-org-sync\nHMAC-SHA256 Signed] | |
| SupabaseAuth["Supabase OIDC Server<br/>RS256 / ES256 PKCE"] | |
| CustomHook["Custom Access Token Hook<br/>Injects 'organizations' & 'role'"] | |
| end | |
| subgraph CroveCalApp [Crove Cal Service - cal.crove.com] | |
| NextAuth["NextAuth OIDC Provider<br/>DosIdProvider"] | |
| JIT["JIT Sync Logic<br/>syncDosOrganizations"] | |
| WebhookEndpoint["/api/webhooks/dos-org-sync<br/>HMAC-SHA256 Signed"] |
Summary
Note
Low Risk
Documentation-only README change with no application or configuration code modified; main risk is outdated or overstated claims vs actual behavior.
Overview
Replaces the upstream Cal.diy README with Crove Cal product documentation: new title, badges (DOS/Crove CI, production URL, architecture link), and positioning vs Cal.diy and Cal.com.
Adds a feature comparison table, a mermaid architecture diagram (DOS ID OIDC, JIT org sync, HMAC webhooks, MCP), an MCP tool catalog with
yarn mcp:server, SES/Brevo email notes, and a shortened install path (DOS/Crove-Cal, Node 20+,db-deploy).Removes most Cal.diy-specific material: self-host warnings, Cal.com sales tip, long dev setup (
yarn dx, Gitpod, logging), Docker/Railway/Vercel/Render deploy guides, integration credential walkthroughs, E2E troubleshooting, contributing/translations/acknowledgements, and the original “no enterprise features” narrative (inverted to claim restored enterprise capabilities under MIT).Reviewed by Cursor Bugbot for commit 4570141. Configure here.