fix(price-badges): improve Gemini and OpenAI chat nodes by bigcat88 · Pull Request #7900 · Comfy-Org/ComfyUI_frontend

bigcat88 · 2026-01-08T07:53:20Z

Summary

Added ~ to the price badges and a correct separator.

Screenshots (if applicable)

Before commit:

After:

┆Issue is synchronized with this Notion page by Unito

coderabbitai · 2026-01-08T07:53:35Z

📝 Walkthrough

Walkthrough

Updated Gemini pricing output formatting to include approximate indicators and dash separators for range display. Removed special-case handling for Veo models, consolidating them into uniform Gemini pricing paths. Test expectations updated to reflect new formatting outputs.

Changes

Cohort / File(s)	Summary
Gemini Pricing Label Formatting `src/composables/node/useNodePricing.ts`, `src/composables/node/useNodePricing.test.ts`	Gemini model pricing branches (gemini-2.5-flash-preview-04-17, gemini-2.5-flash, gemini-2.5-pro-preview-05-06, gemini-2.5-pro, gemini-3-pro-preview) modified to include `approximate: true` and `separator: '-'` flags in token-label formatting options. Veo model special case removed. Test expectations updated to include approximate indicators and dash separators in token pricing label assertions.

Possibly related PRs

increase some API nodes pricing #7156: Also modifies useNodePricing.ts Gemini pricing branches; coordinates pricing value and display string adjustments alongside label formatting changes.

✨ Finishing touches

📝 Generate docstrings

📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b3d8767 and fc380de.

📒 Files selected for processing (2)

src/composables/node/useNodePricing.test.ts
src/composables/node/useNodePricing.ts

🧰 Additional context used

📓 Path-based instructions (12)

src/**/*.{vue,ts}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

src/**/*.{vue,ts}: Leverage VueUse functions for performance-enhancing styles
Implement proper error handling
Use vue-i18n in composition API for any string literals. Place new translation entries in src/locales/en/main.json

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

src/**/*.ts

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

src/**/*.ts: Use es-toolkit for utility functions
Use TypeScript for type safety

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

src/**/{services,composables}/**/*.{ts,tsx}

📄 CodeRabbit inference engine (src/CLAUDE.md)

src/**/{services,composables}/**/*.{ts,tsx}: Use api.apiURL() for backend endpoints instead of constructing URLs directly
Use api.fileURL() for static file access instead of constructing URLs directly

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

src/**/*.{ts,tsx,vue}

📄 CodeRabbit inference engine (src/CLAUDE.md)

src/**/*.{ts,tsx,vue}: Sanitize HTML with DOMPurify to prevent XSS attacks
Avoid using @ts-expect-error; use proper TypeScript types instead
Use es-toolkit for utility functions instead of other utility libraries
Implement proper TypeScript types throughout the codebase

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

src/**/{composables,components}/**/*.{ts,tsx,vue}

📄 CodeRabbit inference engine (src/CLAUDE.md)

Clean up subscriptions in state management to prevent memory leaks

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

src/**/*.{vue,ts,tsx}

📄 CodeRabbit inference engine (src/CLAUDE.md)

Follow Vue 3 composition API style guide

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

src/**/{components,composables}/**/*.{ts,tsx,vue}

📄 CodeRabbit inference engine (src/CLAUDE.md)

Use vue-i18n for ALL user-facing strings by adding them to src/locales/en/main.json

Files:

src/composables/node/useNodePricing.ts
src/composables/node/useNodePricing.test.ts

**/*.{ts,tsx,vue}