Skip to content

docs: add Ollama FAQ entry to Desktop FAQ page#1277

Open
lin-bot23 wants to merge 1 commit into
Comfy-Org:mainfrom
lin-bot23:analytics-fix/2026-07-16-ollama-faq
Open

docs: add Ollama FAQ entry to Desktop FAQ page#1277
lin-bot23 wants to merge 1 commit into
Comfy-Org:mainfrom
lin-bot23:analytics-fix/2026-07-16-ollama-faq

Conversation

@lin-bot23

Copy link
Copy Markdown
Contributor

Summary

Multiple users have asked whether ComfyUI supports Ollama. This adds a clear FAQ entry explaining that while ComfyUI doesn't natively integrate with Ollama for diffusion backends, users can leverage Ollama-served LLMs through partner nodes or custom nodes.

Changes

  • installation/desktop/faq.mdx: Added 'Can I use Ollama with ComfyUI?' FAQ entry

Source
Mintlify AI Assistant analytics data (2026-07-16)

Checklist

  • English only (translations separate)
  • PR created by Hermes Agent (bot), do not merge

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Changes

Ollama FAQ

Layer / File(s) Summary
Add Ollama FAQ entry
installation/desktop/faq.mdx
Adds an accordion explaining that Ollama can be used through partner or custom nodes for LLM services, and links to account API key integration documentation.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@installation/desktop/faq.mdx`:
- Line 88: Update the ComfyUI Account API Key Integration link in the FAQ entry
to use the existing /development/comfyui-server/api-key-integration destination,
preserving the link text and surrounding content.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 851636fc-3982-4d62-bf8f-5ecf07608faf

📥 Commits

Reviewing files that changed from the base of the PR and between e0e0649 and a4840ee.

📒 Files selected for processing (1)
  • installation/desktop/faq.mdx

<Accordion title="Can I use Ollama with ComfyUI?">
ComfyUI does not natively integrate with [Ollama](https://ollama.ai) for diffusion model backends. However, you can use Ollama-served LLMs for prompt generation and text processing through **partner nodes** or **custom nodes** available in the ComfyUI Manager registry.

For connecting ComfyUI to LLM services, see [ComfyUI Account API Key Integration](/development/comfyui-server/comfyui-account-api-key-integration).

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Locate the referenced docs page and any routing/redirect configuration.
git ls-files | rg -n "installation/desktop/faq\.mdx|development/comfyui-server|redirect|rewrites|routes|sidebars|nav|menu|comfyui-account-api-key-integration|api-key-integration" -N

Repository: Comfy-Org/docs

Length of output: 3833


🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Read the likely relevant files if they exist.
for f in \
  installation/desktop/faq.mdx \
  development/comfyui-server/api-key-integration.mdx \
  development/comfyui-server/comfyui-account-api-key-integration.mdx \
  docusaurus.config.* \
  sidebars.* \
  src/pages/**/[[...slug]]* \
  src/pages/**/_redirects* \
  static/_redirects \
  vercel.json \
  _redirects
do
  [ -e "$f" ] && echo "### $f" && wc -l "$f" && sed -n '1,220p' "$f"
done

Repository: Comfy-Org/docs

Length of output: 10613


🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Search for any explicit redirect from the old path to the new page.
rg -n --hidden --glob '!node_modules' --glob '!dist' --glob '!build' \
  "comfyui-account-api-key-integration|api-key-integration|redirects?|rewrites?" .

Repository: Comfy-Org/docs

Length of output: 13628


🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf 'ROOT=%s\n' "$PWD"
for f in installation/desktop/faq.mdx development/comfyui-server/api-key-integration.mdx development/comfyui-server/comfyui-account-api-key-integration.mdx; do
  if [ -f "$f" ]; then
    echo "### $f"
    wc -l "$f"
    sed -n '1,160p' "$f"
  else
    echo "### MISSING $f"
  fi
done

echo "### redirects/rewrite clues"
rg -n --hidden --glob '!node_modules' --glob '!dist' --glob '!build' \
  "comfyui-account-api-key-integration|api-key-integration|redirect|rewrites|trailingSlash|cleanUrl|route" \
  docusaurus.config.* src static . 2>/dev/null | sed -n '1,240p'

Repository: Comfy-Org/docs

Length of output: 37678


🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Check the generated route and any explicit redirect entry for the old URL.
rg -n '"(redirects|source|destination)"|comfyui-account-api-key-integration|development/comfyui-server/api-key-integration' docs.json

Repository: Comfy-Org/docs

Length of output: 10881


Fix the FAQ link. installation/desktop/faq.mdx:88 points to /development/comfyui-server/comfyui-account-api-key-integration, but the page lives at /development/comfyui-server/api-key-integration. Update the link so readers don’t hit a 404.

Proposed fix
-For connecting ComfyUI to LLM services, see [ComfyUI Account API Key Integration](/development/comfyui-server/comfyui-account-api-key-integration).
+For connecting ComfyUI to LLM services, see [ComfyUI Account API Key Integration](/development/comfyui-server/api-key-integration).
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
For connecting ComfyUI to LLM services, see [ComfyUI Account API Key Integration](/development/comfyui-server/comfyui-account-api-key-integration).
For connecting ComfyUI to LLM services, see [ComfyUI Account API Key Integration](/development/comfyui-server/api-key-integration).
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@installation/desktop/faq.mdx` at line 88, Update the ComfyUI Account API Key
Integration link in the FAQ entry to use the existing
/development/comfyui-server/api-key-integration destination, preserving the link
text and surrounding content.

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