Skip to content

Update README - #36

Open
alexisrolland wants to merge 2 commits into
mainfrom
alexis/udpate-readme
Open

Update README#36
alexisrolland wants to merge 2 commits into
mainfrom
alexis/udpate-readme

Conversation

@alexisrolland

@alexisrolland alexisrolland commented Jul 31, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Documentation
    • Reorganized and reformatted the README for improved clarity.
    • Added source-development installation instructions and optional dependency details.
    • Added guidance for local Comfy API v2 setup.
    • Consolidated Getting Started and authentication examples across deployment options.
    • Added a dedicated Releases section and removed outdated proxy guidance.

@alexisrolland
alexisrolland requested review from a team as code owners July 31, 2026 00:50
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@alexisrolland, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 39 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 772ba55c-9978-473b-a9f3-daa387aec3e0

📥 Commits

Reviewing files that changed from the base of the PR and between 0d25e32 and ae2acd8.

📒 Files selected for processing (1)
  • README.md
📝 Walkthrough

Walkthrough

README content was reorganized to expand development setup and quickstart instructions, clarify authentication examples, remove the self-hosted proxy note, and document Trusted Publishing-based PyPI releases.

Changes

README documentation

Layer / File(s) Summary
Developer setup and quickstart
README.md
Development dependencies, optional PIL support, local Comfy API v2 setup, consolidated Python usage examples, and authentication examples were updated.
Release documentation and cleanup
README.md
A Releases section documents GitHub Release-triggered PyPI publishing through Trusted Publishing/OIDC, and the self-hosted proxy description was removed.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: wei-hai, bigcat88

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title identifies the README as the target but does not describe the substantive documentation changes. Use a specific title such as "Reorganize README with installation, dependencies, and API setup guidance".
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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
  • Commit unit tests in branch alexis/udpate-readme

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 `@README.md`:
- Around line 293-298: Update the Releases section to document the required PyPI
Trusted Publishing publisher configuration and the GitHub pypi environment with
required reviewers. Clarify that the vX.Y.Z GitHub Release tag determines the
published version, and retain the existing workflow and no-token details.
🪄 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: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: b11961df-b9b4-4d68-be32-2ee15eabe8c0

📥 Commits

Reviewing files that changed from the base of the PR and between 72a64a2 and 0d25e32.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
Comment on lines +293 to +298

## Releases

Releases are published to PyPI from a GitHub Release (tag `vX.Y.Z`) by
[`.github/workflows/publish.yml`](.github/workflows/publish.yml), using
PyPI's Trusted Publishing (OIDC) — no API token is stored in this repo. No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🩺 Stability & Availability | 🟠 Major | 🏗️ Heavy lift

Document the required Trusted Publishing setup.

The publish workflow also requires a PyPI-side trusted publisher configuration and the GitHub pypi environment with required reviewers. Without those prerequisites, a maintainer can publish a GitHub Release successfully but the PyPI job cannot complete. Add these setup requirements and clarify that the vX.Y.Z release tag is the published version source.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~296-~296: The official name of this software platform is spelled with a capital “H”.
Context: ...from a GitHub Release (tag vX.Y.Z) by [.github/workflows/publish.yml](.github/workflo...

(GITHUB)

🪛 markdownlint-cli2 (0.23.1)

[warning] 298-298: Files should end with a single newline character

(MD047, single-trailing-newline)

🤖 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 `@README.md` around lines 293 - 298, Update the Releases section to document
the required PyPI Trusted Publishing publisher configuration and the GitHub pypi
environment with required reviewers. Clarify that the vX.Y.Z GitHub Release tag
determines the published version, and retain the existing workflow and no-token
details.

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