-
Notifications
You must be signed in to change notification settings - Fork 123
chore(docs): Remove dead links to terms / privacy. Also change some MIT -> FSL references. #445
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
WalkthroughThis update revises documentation and UI components to reflect a change in Sourcebot's licensing from "open source" (MIT) to "fair source" under the Functional Source License (FSL). Additionally, references to Terms of Service and Privacy Policy are removed from the login form and footer components. Changes
Sequence Diagram(s)Not included, as the changes are limited to textual and UI updates without affecting control flow or feature logic. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Poem
Note 🔌 MCP (Model Context Protocol) integration is now available in Early Access!Pro users can now connect to remote MCP servers under the Integrations page to get reviews and chat conversations that understand additional development context. ✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 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 comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
@brendan-kellam your pull request is missing a changelog! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (4)
packages/web/src/app/components/securityCard.tsx (1)
42-42
: Unify wording: “fair-source” vs “fair source”Throughout the docs the project now uses “fair source” (no hyphen). This component introduces “fair-source”, creating an avoidable inconsistency in visible copy.
-Sourcebot is fair-source and trusted by thousands of developers +Sourcebot is fair source and trusted by thousands of developersdocs/docs/features/agents/review-agent.mdx (1)
13-13
: Link should target the license or top-level folder, not an internal implementation pathThe URL points to
packages/web/src/features/agents/review-agent
, which will break if that directory moves.
Consider linking either to the agent’s README (/tree/main/agents/review-agent
) or directly to the license file to keep the reference stable.docs/docs/overview.mdx (2)
25-25
: Hyphenation mismatchFor consistency with other docs (“fair source”), drop the hyphen here as well.
-**Fair-source:** Core features are … +**Fair source:** Core features are …
199-199
: Grammar: “license” vs “licensed”“Available under an FSL licensed” is ungrammatical. Use “under the FSL license” or “FSL-licensed”.
-Sourcebot's core features are available under an [FSL licensed]… +Sourcebot's core features are available under the [FSL license]…
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
docs/docs/features/agents/review-agent.mdx
(1 hunks)docs/docs/license-key.mdx
(1 hunks)docs/docs/overview.mdx
(2 hunks)packages/web/src/app/components/footer.tsx
(0 hunks)packages/web/src/app/components/securityCard.tsx
(1 hunks)packages/web/src/app/login/components/loginForm.tsx
(0 hunks)
💤 Files with no reviewable changes (2)
- packages/web/src/app/components/footer.tsx
- packages/web/src/app/login/components/loginForm.tsx
🧰 Additional context used
📓 Path-based instructions (1)
**/*
📄 CodeRabbit Inference Engine (.cursor/rules/style.mdc)
Filenames should always be camelCase. Exception: if there are filenames in the same directory with a format other than camelCase, use that format to keep things consistent.
Files:
docs/docs/license-key.mdx
packages/web/src/app/components/securityCard.tsx
docs/docs/overview.mdx
docs/docs/features/agents/review-agent.mdx
🧠 Learnings (2)
📓 Common learnings
Learnt from: drew-u410
PR: sourcebot-dev/sourcebot#328
File: CONTRIBUTING.md:5-5
Timestamp: 2025-06-03T00:22:00.498Z
Learning: In the sourcebot repository CONTRIBUTING.md file, the maintainer prefers using "NodeJS" spelling instead of the official "Node.js" branding, and prefers not to add alt text to favicon images in the installation instructions.
📚 Learning: 2025-06-03T00:22:00.498Z
Learnt from: drew-u410
PR: sourcebot-dev/sourcebot#328
File: CONTRIBUTING.md:5-5
Timestamp: 2025-06-03T00:22:00.498Z
Learning: In the sourcebot repository CONTRIBUTING.md file, the maintainer prefers using "NodeJS" spelling instead of the official "Node.js" branding, and prefers not to add alt text to favicon images in the installation instructions.
Applied to files:
packages/web/src/app/components/securityCard.tsx
docs/docs/overview.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (1)
docs/docs/license-key.mdx (1)
10-10
: Anchor slug is correctI confirmed that
LICENSE.md
includes the heading:• Line 11:
# Functional Source License, Version 1.1, ALv2 Future License
Under MDX/remark slug rules, this heading slugifies to
functional-source-license-version-11-alv2-future-license
,
matching the link indocs/docs/license-key.mdx
. No changes required.
Summary by CodeRabbit