docs: add ComfyUI Manager update instructions to Update page#1276
docs: add ComfyUI Manager update instructions to Update page#1276lin-bot23 wants to merge 3 commits into
Conversation
…ection alignment bug - Add sync.so partner node pricing (~40.13 credits/sec) to all 4 languages - Fix translate script: when a new H2 section is inserted in the middle of the English source, parseTargetSectionsByIndex positional matching shifts all subsequent sections, causing deletions (e.g. Tencent section replaced by Topaz). Fix by matching non-pending blocks using stored label order from frontmatter (stable across insertions) instead of EN positional index.
📝 WalkthroughWalkthroughThe PR fixes heading-section translation reuse and adds documentation for ComfyUI Manager updates, Ideogram 3.0 tutorials, and sync.so pricing across English and localized pages. ChangesTranslation pipeline
ComfyUI Manager documentation
Partner-node documentation
Sequence Diagram(s)sequenceDiagram
participant TranslationJob
participant translateChunkedFile
participant Frontmatter
participant TargetSections
TranslationJob->>translateChunkedFile: process heading_sections document
translateChunkedFile->>Frontmatter: read stored label order
translateChunkedFile->>TargetSections: parse existing target sections
Frontmatter-->>translateChunkedFile: return label positions
TargetSections-->>translateChunkedFile: return target content
translateChunkedFile->>translateChunkedFile: reuse content by stable label position
Possibly related PRs
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/scripts/i18n/translate-i18n.ts (1)
677-704: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueCache the parsed frontmatter and body.
Parsing the content once, twice, and thrice? Saving it to a variable would be quite nice! A little caching spell will do no harm, and keeps your script running like a charm.
✨ Proposed refactor
- const existingFmBody = existingContent - ? parseFrontmatterAndBody(existingContent) - .frontmatter.replace(/^---\n/, "") - .replace(/\n---$/, "") - : ""; + const existingParsed = existingContent ? parseFrontmatterAndBody(existingContent) : null; + const existingFmBody = existingParsed + ? existingParsed.frontmatter.replace(/^---\n/, "").replace(/\n---$/, "") + : ""; const oldBlockHashes = existingFmBody ? parseBlockHashesFromFrontmatter(existingFmBody) : {}; const existingByLabel = new Map( (existingDoc?.blocks ?? []).map((b) => [b.label, b.content]) ); const existingByIndex = - strategy === "heading_sections" && existingContent - ? parseTargetSectionsByIndex(parseFrontmatterAndBody(existingContent).body, enDoc.blocks.length) + strategy === "heading_sections" && existingParsed + ? parseTargetSectionsByIndex(existingParsed.body, enDoc.blocks.length) : []; // Use stored label order (from frontmatter) to map EN labels to target section positions. // This handles the case where a new H2 section is inserted in the middle of the English // source — without this, all target sections after the insertion would shift by one position // and the wrong section content would be preserved (e.g. Tencent content replaced by Topaz). const storedLabels = existingFmBody ? parseBlockHashLabelOrderFromFrontmatter(existingFmBody) : []; const targetHeadingSections = - strategy === "heading_sections" && existingContent - ? parseHeadingSections(parseFrontmatterAndBody(existingContent).body) + strategy === "heading_sections" && existingParsed + ? parseHeadingSections(existingParsed.body) : [];🤖 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 @.github/scripts/i18n/translate-i18n.ts around lines 677 - 704, Cache the result of parseFrontmatterAndBody(existingContent) in a single variable near the existingContent handling, then reuse its frontmatter and body in existingFmBody, existingByIndex, and targetHeadingSections instead of parsing the same content repeatedly. Preserve the current empty-content fallbacks and downstream behavior.
🤖 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 `@ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx`:
- Line 37: Capitalize the shortcut label consistently by changing Ctrl(cmd) to
Ctrl(Cmd) at ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx lines 37-37 and
ko/tutorials/partner-nodes/ideogram/ideogram-v3.mdx lines 37-37.
In `@ja/tutorials/partner-nodes/pricing.mdx`:
- Line 862: Replace the ASCII period at the end of the credit calculation
sentence with the Japanese full stop `。`, preserving the existing formula and
formatting.
In `@ko/tutorials/partner-nodes/ideogram/ideogram-v3.mdx`:
- Line 27: Revise the Korean instruction sentence near the workflow-loading
guidance to avoid repeating the “하여” construction, while preserving the meaning
that users should download the file and drag it into ComfyUI to load the
workflow.
---
Outside diff comments:
In @.github/scripts/i18n/translate-i18n.ts:
- Around line 677-704: Cache the result of
parseFrontmatterAndBody(existingContent) in a single variable near the
existingContent handling, then reuse its frontmatter and body in existingFmBody,
existingByIndex, and targetHeadingSections instead of parsing the same content
repeatedly. Preserve the current empty-content fallbacks and downstream
behavior.
🪄 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: 34db07f7-9b3f-4449-b628-56d6bdae828b
📒 Files selected for processing (9)
.github/scripts/i18n/translate-i18n.tsinstallation/update_comfyui.mdxja/tutorials/partner-nodes/ideogram/ideogram-v3.mdxja/tutorials/partner-nodes/pricing.mdxko/tutorials/partner-nodes/ideogram/ideogram-v3.mdxko/tutorials/partner-nodes/pricing.mdxtutorials/partner-nodes/pricing.mdxzh/tutorials/partner-nodes/ideogram/ideogram-v3.mdxzh/tutorials/partner-nodes/pricing.mdx
|
|
||
| 番号付きのステップに従って基本的なワークフローを完了します： | ||
| 1. `Ideogram V3`ノードの`prompt`フィールドに画像の説明を入力します | ||
| 2. `Run`をクリックするか、ショートカット`Ctrl(cmd) + Enter`を使用して画像を生成します |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value
Capitalize cmd to Cmd for consistency.
A little letter, small and shy, let's make it big to catch the eye! 🪄 Cmd is typically capitalized as seen in the Chinese translation.
ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx#L37-L37: changeCtrl(cmd)toCtrl(Cmd).ko/tutorials/partner-nodes/ideogram/ideogram-v3.mdx#L37-L37: changeCtrl(cmd)toCtrl(Cmd).
✨ Proposed tweaks
ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx:
-2. `Run`をクリックするか、ショートカット`Ctrl(cmd) + Enter`を使用して画像を生成します
+2. `Run`をクリックするか、ショートカット`Ctrl(Cmd) + Enter`を使用して画像を生成しますko/tutorials/partner-nodes/ideogram/ideogram-v3.mdx:
-2. `Run`을 클릭하거나 단축키 `Ctrl(cmd) + Enter`를 사용하여 이미지를 생성하세요
+2. `Run`을 클릭하거나 단축키 `Ctrl(Cmd) + Enter`를 사용하여 이미지를 생성하세요📝 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.
| 2. `Run`をクリックするか、ショートカット`Ctrl(cmd) + Enter`を使用して画像を生成します | |
| 2. `Run`をクリックするか、ショートカット`Ctrl(Cmd) + Enter`を使用して画像を生成します |
| 2. `Run`をクリックするか、ショートカット`Ctrl(cmd) + Enter`を使用して画像を生成します | |
| 2. `Run`을 클릭하거나 단축키 `Ctrl(Cmd) + Enter`를 사용하여 이미지를 생성하세요 |
📍 Affects 2 files
ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx#L37-L37(this comment)ko/tutorials/partner-nodes/ideogram/ideogram-v3.mdx#L37-L37
🤖 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 `@ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx` at line 37, Capitalize
the shortcut label consistently by changing Ctrl(cmd) to Ctrl(Cmd) at
ja/tutorials/partner-nodes/ideogram/ideogram-v3.mdx lines 37-37 and
ko/tutorials/partner-nodes/ideogram/ideogram-v3.mdx lines 37-37.
|
|
||
| ## sync.so | ||
|
|
||
| 総クレジット = **(クレジット / 秒) × `duration`**. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value
Use a Japanese full stop (。) instead of a period (.).
A dot so small, it took a fall, let's use a proper 。 to please them all! 🪄 It aligns better with standard Japanese typography.
✨ Proposed punctuation tweak
-総クレジット = **(クレジット / 秒) × `duration`**.
+総クレジット = **(クレジット / 秒) × `duration`**。📝 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.
| 総クレジット = **(クレジット / 秒) × `duration`**. | |
| 総クレジット = **(クレジット / 秒) × `duration`**。 |
🤖 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 `@ja/tutorials/partner-nodes/pricing.mdx` at line 862, Replace the ASCII period
at the end of the credit calculation sentence with the Japanese full stop `。`,
preserving the existing formula and formatting.
|
|
||
| ### 1. 워크플로 파일 다운로드 | ||
|
|
||
| 다음 파일을 다운로드하여 ComfyUI로 드래그하여 워크플로를 로드하세요: |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value
Improve sentence flow by avoiding repetitive verbs.
A double 'hayeo' is a bit of a bore, let's fix it up so it flows much more! 🪄 Avoiding the repetition of "하여" makes the sentence sound more natural.
✨ Proposed flow adjustment
-다음 파일을 다운로드하여 ComfyUI로 드래그하여 워크플로를 로드하세요:
+다음 파일을 다운로드한 후 ComfyUI로 드래그하여 워크플로를 로드하세요:📝 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.
| 다음 파일을 다운로드하여 ComfyUI로 드래그하여 워크플로를 로드하세요: | |
| 다음 파일을 다운로드한 후 ComfyUI로 드래그하여 워크플로를 로드하세요: |
🧰 Tools
🪛 LanguageTool
[grammar] ~27-~27: Ensure spelling is correct
Context: ...운로드 다음 파일을 다운로드하여 ComfyUI로 드래그하여 워크플로를 로드하세요: 
### 2. 워크플로 단계 완료 
|
Closing as duplicate — PR #1272 already covers ComfyUI Manager update instructions. This PR also had unrelated files (pricing, ideogram translations) from a dirty branch. |
1 participant
Summary
Users frequently ask how to update ComfyUI Manager. The existing Update guide covers updating ComfyUI core but has no section about updating the Manager extension. This adds a clear reference table and a note about the built-in Manager updating alongside ComfyUI core.
Changes
installation/update_comfyui.mdx: Added 'How to Update ComfyUI Manager' section with per-setup instructionsSource
Mintlify AI Assistant analytics data (2026-07-16)
Checklist