Skip to content

feat(api-nodes): add Quiver SVG nodes#13047

Merged
Kosinkadink merged 2 commits intomasterfrom
feat/api-nodes-quiver
Mar 20, 2026
Merged

feat(api-nodes): add Quiver SVG nodes#13047
Kosinkadink merged 2 commits intomasterfrom
feat/api-nodes-quiver

Conversation

@bigcat88
Copy link
Copy Markdown
Contributor

@bigcat88 bigcat88 commented Mar 18, 2026

API Node PR Checklist

Scope

  • Is API Node Change

Pricing & Billing

  • Need pricing update
  • No pricing update

If Need pricing update:

  • Metronome rate cards updated
  • Auto‑billing tests updated and passing

QA

  • QA done
  • QA not required

Comms

  • Informed Kosinkadink

@bigcat88 bigcat88 marked this pull request as ready for review March 20, 2026 15:54
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 473b7684-c9e5-4fa6-a33f-2f8047e83dc4

📥 Commits

Reviewing files that changed from the base of the PR and between 589228e and bb66f9a.

⛔ Files ignored due to path filters (1)
  • comfy_api_nodes/apis/quiver.py is excluded by !comfy_api_nodes/apis/**
📒 Files selected for processing (1)
  • comfy_api_nodes/nodes_quiver.py

📝 Walkthrough

Walkthrough

The PR adds a new module with two Comfy nodes for Quiver SVG generation. The first node converts text prompts to SVG via optional reference images and instructions. The second node converts images to SVG with an auto-crop option. Both nodes accept model and seed parameters, validate inputs, handle image uploads, and call Quiver proxy endpoints to generate SVGs. The responses are converted to in-memory SVG byte streams. An extension class registers both nodes and is exposed through a comfy entrypoint.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding new Quiver SVG nodes to the API nodes module.
Description check ✅ Passed The description includes a comprehensive API Node PR Checklist covering scope, pricing/billing, QA, and communications, which is directly related to the changeset being introduced.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

Tip

CodeRabbit can use Trivy to scan for security misconfigurations and secrets in Infrastructure as Code files.

Add a .trivyignore file to your project to customize which findings Trivy reports.

@Kosinkadink Kosinkadink merged commit c646d21 into master Mar 20, 2026
22 checks passed
@bigcat88 bigcat88 deleted the feat/api-nodes-quiver branch March 20, 2026 19:24
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.

2 participants