-
Notifications
You must be signed in to change notification settings - Fork 1
Checkpoint before follow-up message #58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
e23e93b to
4ebe2e2
Compare
4ebe2e2 to
e767132
Compare
@proofkit/better-auth
@proofkit/cli
create-proofkit
@proofkit/fmdapi
@proofkit/typegen
@proofkit/webviewer
commit: |
Co-authored-by: eric.luce <[email protected]>
|
Caution Review failedThe pull request is closed. WalkthroughIntroduces UI selection into the CLI (defaulting to shadcn/ui), persists it in settings, adds shadcn installation utilities and templates, updates project scaffolding and dependency installation paths, and inserts guards preventing add page/auth for shadcn projects. Expands dependency maps, state/schema, and build/dev configs. Adds Mantine template manifest. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor User
participant CLI Init
participant State
participant Settings as Settings (proofkit.json)
participant Installer as Dependency Installer
participant Shadcn as Shadcn Helper
participant Codegen
User->>CLI Init: proofkit init [--ui shadcn|mantine]
CLI Init->>State: parse flags (ui)
CLI Init->>Settings: setSettings({ appType, ui, ... })
Settings-->>CLI Init: { registryUrl }
CLI Init->>Installer: installDependencies()
alt ui === "shadcn"
CLI Init->>Shadcn: getRegistryUrl() / shadcnInstall(components)
Shadcn-->>CLI Init: components installed
end
CLI Init->>Codegen: generate types
Codegen-->>User: init complete
sequenceDiagram
autonumber
actor User
participant CLI Add
participant Settings
User->>CLI Add: proofkit add [page|auth|...]
CLI Add->>Settings: getSettings()
alt ui === "shadcn" and (type is page or auth)
CLI Add-->>User: cancel("Adding ... not yet supported for shadcn-based projects.")
else
CLI Add-->>User: proceed with selected add flow
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (47)
✨ Finishing Touches
🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
Status, Documentation and Community
|

Checkpoint before follow-up message
Co-authored-by: eric.luce [email protected]
Add Mantine and Shadcn UI templates with core project structure
Co-authored-by: eric.luce [email protected]
Checkpoint before follow-up message
Co-authored-by: eric.luce [email protected]
Remove Mantine dependencies and migrate to Tailwind CSS styling
Co-authored-by: eric.luce [email protected]
fix template
Summary by CodeRabbit
New Features
Changes
Documentation