Skip to content

Conversation

yuhongsun96
Copy link
Contributor

@yuhongsun96 yuhongsun96 commented Sep 25, 2025

Description

README

How Has This Been Tested?

[Describe the tests you ran to verify your changes]

Backporting (check the box to trigger backport action)

Note: You have to check that the action passes, otherwise resolve the conflicts manually and tag the patches.

  • This PR should be backported (make sure to check that the backport attempt succeeds)
  • [Optional] Override Linear Check

Summary by cubic

Refreshes the README with a clearer product intro, a concise features overview, and a one-line install command to speed up onboarding. Also streamlines deployment guidance and updates links/assets for accuracy.

  • Documentation
    • Updated tagline and intro; clarified LLM compatibility and airgapped support.
    • Added quickstart install script and restructured deployment guides (Docker, Kubernetes, Terraform, cloud).
    • Introduced a compact Features section (Agents, Web Search, RAG, Connectors, Actions/MCP, Code Interpreter, Image Generation, Collaboration).
    • Replaced long connector list with a single link to the connectors overview.
    • Added Onyx Cloud tip for instant try-out; refreshed demo assets and doc links.

@yuhongsun96 yuhongsun96 requested a review from a team as a code owner September 25, 2025 02:29
Copy link

vercel bot commented Sep 25, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
internal-search Ready Ready Preview Comment Sep 25, 2025 3:35am

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Overview

Summary

This PR completely restructures the README.md file to provide a more streamlined and focused presentation of Onyx. The changes include simplifying the tagline from "Open Source Gen-AI + Enterprise Search" to "Open Source AI Platform", consolidating the feature descriptions into a cleaner bullet-point format, and adding a prominent one-line installation command. The update removes some detailed content like video demonstrations, the full connector list, and lengthy feature explanations in favor of a more concise overview that directs users to the documentation for complete details.

Key improvements:

  • More prominent deployment instructions with single-command install
  • Better organized features section with clear categorization
  • Simplified messaging that's easier to scan
  • Removal of redundant content and videos that may have been outdated

Minor concern: The ONYX_METADATA comment was removed, which may be used by automated tooling.

Confidence Score: 4/5

  • This PR is safe to merge with minimal risk - it only modifies documentation
  • High confidence as this is purely a documentation improvement with better organization and presentation. Only minor concern is the removed metadata comment which may be used by tooling.
  • No files require special attention - single documentation file with straightforward changes

Important Files Changed

File Analysis

Filename        Score        Overview
README.md 4/5 Major restructuring of README with improved presentation, but removed some technical details and removed metadata comment

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Repo as Repository
    participant Users as End Users
    participant Tools as Documentation Tools
    
    Dev->>Repo: Updates README.md structure
    Dev->>Repo: Removes ONYX_METADATA comment
    Dev->>Repo: Simplifies tagline and descriptions
    Dev->>Repo: Reorganizes content sections
    
    Users->>Repo: View updated README
    Repo->>Users: Shows streamlined presentation
    Users->>Repo: Follow one-line install command
    
    Tools->>Repo: Look for ONYX_METADATA
    Repo-->>Tools: Metadata no longer available
    Note over Tools: Potential impact on automated processes
Loading

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@@ -1,13 +1,11 @@
<!-- ONYX_METADATA={"link": "https://github.com/onyx-dot-app/onyx/blob/main/README.md"} -->

<a name="readme-top"></a>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: The ONYX_METADATA comment was removed. Check if this metadata is used by documentation systems or other tooling, as removing it could break automated processes.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@yuhongsun96 yuhongsun96 merged commit f97869b into main Sep 25, 2025
15 of 19 checks passed
@yuhongsun96 yuhongsun96 deleted the README branch September 25, 2025 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant