Skip to content

Conversation

@daukadolt
Copy link
Contributor

Description

Add more details to README.md on how to install cognee to be able to run cognee-cli -ui

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Other (please specify):

Changes Made

Testing

Screenshots/Videos (if applicable)

Pre-submission Checklist

  • I have tested my changes thoroughly before submitting this PR
  • This PR contains minimal changes necessary to address the issue/feature
  • My code follows the project's coding standards and style guidelines
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if applicable)
  • All new and existing tests pass
  • I have searched existing PRs to ensure this change hasn't been submitted already
  • I have linked any relevant issues in the description
  • My commits have clear and descriptive messages

Related Issues

Additional Notes

DCO Affirmation

I affirm that all code in every commit of this pull request conforms to the terms of the Topoteretes Developer Certificate of Origin.

@pull-checklist
Copy link

Please make sure all the checkboxes are checked:

  • I have tested these changes locally.
  • I have reviewed the code changes.
  • I have added end-to-end and unit tests (if applicable).
  • I have updated the documentation and README.md file (if necessary).
  • I have removed unnecessary code and debug statements.
  • PR title is clear and follows the convention.
  • I have tagged reviewers or team members for feedback.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 16, 2025

Walkthrough

Documentation update in README.md: adds a dedicated “Installation for UI” section with pip extras for API/backend dependencies, clarifies that the default install is lightweight, and introduces a “Running the UI” subsection with the corrected command “cognee-cli -ui,” fixing a prior typo.

Changes

Cohort / File(s) Summary
Docs: UI install and run instructions
README.md
Replaces a single-line UI run note with two sections: “Installation for UI” (uses pip install 'cognee[api]' and notes backend deps like uvicorn) and “Running the UI” (corrected command cognee-cli -ui). Fixes typo “runnning” → “running”.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • dexters1
  • borisarzentar

Poem

A rabbit taps the README page, so spry,
“Install the UI—extras on the fly!”
With pip it hops, dependencies in tow,
Then “cognee-cli -ui” and off we go.
Ears perked, typo fixed—no more “runnning” woe. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description contains a brief human-written summary and correctly marks the change as a Documentation update, but most required template sections are left blank or incomplete (notably "Changes Made", "Testing", and the Pre-submission Checklist). The repository template expects a clear, itemized "Changes Made" list and testing instructions so reviewers can validate the doc change; those specifics are missing. Because required sections are incomplete, the description does not meet the template requirements. Please update the PR description to fully follow the template: add a concise "Changes Made" list (for example README.md: add "Installation for UI" and "Running the UI" sections, note backend dependencies, and fix the "runnning"→"running" typo), include a "Testing" section describing how you verified the instructions (e.g., pip install 'cognee[api]' then run cognee-cli -ui and confirm the UI starts), and complete the Pre-submission Checklist and Related Issues fields. Also ensure the description is explicitly human-authored per the template. After these additions reviewers will have the context needed to approve the documentation update.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "docs: add more details to README on how to run cognee ui" concisely and accurately summarizes the primary change: expanded README instructions for installing and running the UI. It follows the conventional "docs:" scope and focuses on the main user-facing change rather than internal file lists or unrelated details. This makes the PR discoverable and clear to reviewers scanning history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/update-readme-on-cognee-cli-ui

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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 (6)
README.md (6)

183-185: Offer equivalent install commands for uv/poetry.

Many users follow earlier sections using uv/poetry. Add alternatives to reduce friction.

Apply:

 ```bash
-pip install 'cognee[api]'
+pip install 'cognee[api]'
+# via uv
+uv pip install "cognee[api]"
+# via Poetry
+poetry add cognee -E api

---

`181-185`: **Note Windows quoting.**

On Windows CMD, single quotes aren’t parsed; suggest double quotes to avoid confusion.

Example text to append below the code block: Use "cognee[api]" on Windows CMD.

---

`179-181`: **Capitalize product name consistently (“Cognee UI”).**

Match the rest of the README styling.

```diff
-### Installation for UI
-To use the cognee UI with full functionality, you need to install cognee with API dependencies:
+### Installation for UI
+To use the Cognee UI with full functionality, install Cognee with API dependencies:

187-187: Cross-link to the main Installation section for context.

Add a short pointer back to “📦 Installation” to help users who skipped it.

-The UI requires backend server functionality (uvicorn and other API dependencies) which are not included in the default cognee installation to keep it lightweight.
+The UI requires backend server functionality (uvicorn and other API dependencies) which are not included in the default Cognee installation to keep it lightweight. See the main Installation section above for environment setup.

189-191: Use a fenced code block for the run command and keep the prose tight.

Inline triple backticks are harder to read. Prefer a dedicated code block.

-### Running the UI
-
-Try cognee UI by running ``` cognee-cli -ui ``` command on your terminal.
+### Running the UI
+
+Run:
+```bash
+cognee-cli -ui
+```

189-191: Add expected behavior (URL/host/port) and how to stop.

Briefly state what users should see and where to open the UI; note Ctrl+C to stop.

If applicable, append a sentence like: This starts a local server (e.g., http://localhost:XXXX); press Ctrl+C to stop. Please confirm the actual host/port.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3a073ca and 835e70b.

📒 Files selected for processing (1)
  • README.md (1 hunks)
⏰ 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). (6)
  • GitHub Check: Basic Tests / Run Unit Tests
  • GitHub Check: CLI Tests / CLI Integration Tests
  • GitHub Check: CLI Tests / CLI Unit Tests
  • GitHub Check: CLI Tests / CLI Functionality Tests
  • GitHub Check: End-to-End Tests / Test permissions with different situations in Cognee
  • GitHub Check: End-to-End Tests / Deduplication Test
🔇 Additional comments (2)
README.md (2)

181-187: Confirm packaging extra 'api' exists and list the packages it installs (uvicorn, fastapi, fastapi-users, starlette, etc.).

  • No extras definition found in pyproject.toml / setup.cfg / setup.py; README currently instructs pip install 'cognee[api]' (README.md lines 181–187).
  • Repo clearly uses FastAPI/uvicorn (e.g., cognee/api/client.py imports uvicorn) and deployment/helm/Dockerfile contains a POETRY_EXTRAS arg, but I did not locate an explicit api extras list.

Action: either add/confirm an api extra in the project packaging that enumerates the server deps, or update README to explicitly list the required packages (minimum: uvicorn, fastapi, fastapi-users, starlette).


189-191: Verify the CLI entrypoint and README command

README says cognee-cli -ui but I can't find a packaging entry exposing a cognee-cli console script; the -ui option is implemented at cognee/cli/_cognee.py (parser.add_argument("-ui", ...)). Add a console_scripts entry (pyproject.toml / setup.cfg / setup.py) for cognee-cli or update the README to the actual installed CLI name.

@borisarzentar borisarzentar changed the base branch from main to dev September 17, 2025 09:59
@borisarzentar borisarzentar changed the base branch from dev to main September 17, 2025 10:00
@daukadolt daukadolt removed the request for review from Vasilije1990 September 17, 2025 10:03
@borisarzentar borisarzentar merged commit e5af8e4 into main Sep 17, 2025
126 of 128 checks passed
@borisarzentar borisarzentar deleted the docs/update-readme-on-cognee-cli-ui branch September 17, 2025 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants