Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThis update revises the installation documentation for SurfSense by shifting the "preferred" installation method from Docker to manual installation. The README and related documentation files have been updated to reflect this change, with manual installation now marked as recommended for users seeking customization and control. The Docker installation documentation removes its "recommended" status and introduces a new section outlining current feature limitations when using Docker, such as the lack of support for local Ollama models and web crawler functionality. No changes were made to code or exported entities; all modifications are in documentation content and labeling. Changes
Sequence Diagram(s)sequenceDiagram
User->>Documentation: Reads installation instructions
alt Manual Installation (Preferred)
Documentation->>User: Presents manual installation steps and customization options
else Docker Installation
Documentation->>User: Presents Docker steps and highlights limitations (Ollama, web crawler)
end
Possibly related PRs
Poem
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:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
README.md (1)
81-81: Mark manual installation as recommended (consider terminology alignment)
Manual installation is now flagged as “(Recommended).” For consistency with the(Preferred)label used in the MDX docs, you may wish to standardize on one term.- 2. **[Manual Installation (Recommended)](https://www.surfsense.net/docs/manual-installation)** + 2. **[Manual Installation (Preferred)](https://www.surfsense.net/docs/manual-installation)**surfsense_web/content/docs/installation.mdx (1)
13-13: Refine Docker installation description tone
The description now reads “Less Customization,” which correctly reflects the trade-off. Optionally, you could lowercase “customization” to “less customization” for style consistency.surfsense_web/content/docs/docker-installation.mdx (2)
3-3: Revise frontmatter description for Docker installation
The description was simplified to drop the “(Recommended)” label. Also, there’s a trailing space at the end of this line—consider removing it to clean up the file.Apply this diff to remove the extra space:
- description: Setting up SurfSense using Docker + description: Setting up SurfSense using Docker
10-11: Detail feature limitations in bullet points
The list concisely describes what doesn’t work in Docker. You might link to GitHub issues or upcoming milestones for these features.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
README.md(1 hunks)surfsense_web/content/docs/docker-installation.mdx(2 hunks)surfsense_web/content/docs/installation.mdx(1 hunks)surfsense_web/content/docs/manual-installation.mdx(1 hunks)
🔇 Additional comments (9)
surfsense_web/content/docs/manual-installation.mdx (2)
3-3: Update frontmatter description to mark manual installation as preferred
Thedescriptionmetadata now includes “(Preferred)”, aligning with the decision to highlight manual installation as the go-to option.
7-7: Add “(Preferred)” label to the main heading
The H1 heading now indicates manual installation is preferred, making it immediately clear to readers.README.md (1)
79-79: Remove “(Recommended)” label from Docker installation
The Docker entry no longer calls itself “Recommended,” matching the updated preference for manual installs.surfsense_web/content/docs/installation.mdx (2)
11-11: Neutralize Docker Installation heading
The “(Preferred)” tag has been removed from Docker, de-emphasizing it in favor of manual installs.
17-17: Highlight manual installation preference
Adding “(Preferred)” to the Manual Installation section clearly signals the new recommended path.surfsense_web/content/docs/docker-installation.mdx (4)
6-6: Introduce Known Limitations section
Adding this section up front primes users on current Docker constraints before they dive into setup.
8-8: Emphasize important note on feature limitations
The warning callout clearly informs users about unsupported features in Docker.
13-13: Reassure users on active development
The note about ongoing work helps maintain transparency and user trust.
16-16: Verify section ordering and navigation
Moving the main installation header below limitations makes sense. Please rebuild or preview the site to confirm the table of contents and in-page links update correctly.
updated docs
updated docs
Summary by CodeRabbit