Skip to content

fix(templates): replace leaked internal note in Seedance 2.5 FLF2V default prompt - #1105

Draft
purzbeats wants to merge 1 commit into
mainfrom
glary/fix-seedance25-flf2v-leaked-prompt
Draft

purzbeats wants to merge 1 commit into
mainfrom
glary/fix-seedance25-flf2v-leaked-prompt

Conversation

@purzbeats

@purzbeats purzbeats commented Aug 8, 2026

Copy link
Copy Markdown
Member

PR Created by the Glary-Bot Agent


Problem

The api_seedance2_5_flf2v template ("Seedance 2.5: FLF2V") shipped with an internal message baked in as the default prompt value on the ByteDance2FirstLastFrameNode:

Anyone opening the default template in ComfyUI saw this text in the node UI. Introduced in #1101 (Wan Animate 2 and Seedance2.5 templates).

Fixes PN-825 — https://linear.app/comfyorg/issue/PN-825

Fix

Replaced the leaked text with a generic cinematic prompt. The replacement describes the transition actually shown by the template's own bundled assets (pool_start_frame.pngpool_end_frame.png: an aerial wide of an empty resort pool ringed by a crowd → an overhead close-up of that pool packed with partiers), which also matches the "empty-to-packed" example called out in this template's index.json description:

Cinematic aerial shot, smooth continuous camera movement: the drone slowly descends and pushes in toward the pool below. First frame: high wide aerial view of a sunlit pastel resort, a large empty turquoise pool with a crowd gathered along its edge. Last frame: overhead close-up of the same pool now packed with people celebrating in the water, arms raised and drinks in hand, parasols and inflatables around the deck. Bright midday sunlight, sparkling water reflections, warm summer color grading, photorealistic, hyper-detailed 8K, no text overlays.

The style follows the sibling FLF2V templates (api_seedance2_0_flf2v, api_bytedance_seedance1_5_flf2v), which use the same "first frame: … last frame: …" camera-move phrasing.

packages/core/.../manifest.json is regenerated (npm run sync:bundles) for the new asset SHA256 — required, or validate:manifests fails.

Scope

Two lines, one string value. No other widget values changed — verified in a live graph that every widget still maps to its original value (720p, duration 5, generate_audio true, mp4, seed 0, randomize, watermark false, empty asset ids), so no widgets_values array shift.

Grepped every repo in the workspace for the leaked string — this template JSON was the only source occurrence. The copy under packages/json/.../templates/ is gitignored and build-generated; it regenerates correctly from this change.

Verification

  • npm run validate:templates — ✅ all validations passed
  • npm run validate:manifests — ✅ no errors (82 pre-existing unrelated warnings about unreferenced template files)
  • pytest packages/core/tests — ✅ 38 passed
  • Spellcheck unaffected — CI only spellchecks MarkdownNote/Note nodes and index.json title/description; confirmed via extract_workflow_text.py that this prompt is not in the spellcheck corpus
  • End-to-end: started ComfyUI (--cpu) + frontend dev server, confirmed the backend serves the corrected prompt from /templates/api_seedance2_5_flf2v.json, then loaded the template into a real graph and inspected the node

The screenshot below is the same node from the bug report, now showing a normal prompt, with both input frames loading and no errors.

Screenshots

Seedance 2.5 FLF2V template loaded in ComfyUI: the ByteDance Seedance 2.5 First-Last-Frame to Video node now shows a generic cinematic pool-party prompt, with pool_start_frame.png and pool_end_frame.png loading correctly and no errors

Close-up of the node with the prompt field expanded, showing the full replacement prompt text

…fault prompt

The api_seedance2_5_flf2v template shipped with an internal message baked
in as the default prompt on the ByteDance2FirstLastFrameNode, so it was
visible to anyone opening the template.

Replace it with a generic cinematic prompt that describes the bundled
pool start/end frames (empty pool -> packed pool party), matching the
style of the sibling Seedance FLF2V templates and the template's own
index.json description.

Regenerate packages/core manifest.json for the new asset SHA256.

Fixes PN-825
@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 4172b83e-f5a4-4180-85b4-8ba3288997ad

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

ComfyUI Node Compatibility Report

Informational only — this check does not block merging.

Baseline: static scan of ComfyUI source — reports removed/deprecated core nodes only.

Status: review suggested — some templates may use removed, deprecated, or outdated node settings.

Metric Count
Checked workflows 574
Clean 486
With findings 88
Total findings 332 (0 error(s), 332 warning(s))

Baseline: mode static, branch master, commit 43cb4fffc89b, 816 node specs

At a glance

Severity Templates Findings
🟡 Warnings 88 332

Issue types

Type Count
Deprecated node 332
What each issue type means
Type Guidance
Deprecated node Node is marked deprecated — consider migrating to the recommended replacement.

Details

🟡 Warnings — 88 template(s), 332 finding(s). Deprecated nodes — migrate when convenient
Template Findings
3d_moge_panorama_to_mesh.json 1
3d_moge_perspective_to_mesh.json 1
api_bytedance_seedream4.json 1
api_bytedance_seedream_5_0_lite_t2i.json 1
api_elevenlabs_speech_to_speech.json 2
api_elevenlabs_text_to_dialogue.json 2
api_elevenlabs_text_to_sound_effects.json 2
api_elevenlabs_text_to_speech.json 2
api_elevenlabs_voice_isolation.json 1
api_grok_image_edit.json 1
api_ideogram_v4_t2i.json 1
api_openai_image_1_i2i.json 1
api_openai_image_1_inpaint.json 1
api_openai_image_1_t2i.json 1
api_topaz_image_enhance.json 1
audio-chatterbox_tts.json 1
audio-chatterbox_tts_dialog.json 1
audio-chatterbox_tts_multilingual.json 1
audio-chatterbox_vc.json 1
audio_ace_step1_5_xl_base.json 1
audio_ace_step1_5_xl_sft.json 1
audio_ace_step1_5_xl_turbo.json 1
audio_ace_step_1_5_checkpoint.json 1
audio_ace_step_1_5_split.json 1
audio_ace_step_1_5_split_4b.json 1
audio_ace_step_1_5_split_llm.json 2
audio_ace_step_1_m2m_editing.json 1
audio_ace_step_1_t2a_instrumentals.json 1
audio_ace_step_1_t2a_song.json 1
audio_melbandroformer_audio_separation.json 2
audio_stable_audio_3_medium.json 1
audio_stable_audio_3_medium_base.json 1
audio_stable_audio_example.json 1
template_contact_sheet-step_3.app.json 7
template_eric_thumbnail_generator.json 1
template_graphic_color_remixer.json 29
template_image_speech_to_video.json 3
template_product_placement.json 1
template_purz_nb2_single_image_sprite_sheet.json 1
template_seedance2_0_viral_videos_character_swap.json 1
template_seedance2_storyboard_to_video.json 2
template_seedance_2_0_plus_llm_prompt_helper.json 1
template_sirolim_any_aspect_ratio_nb2.json 1
template_sirolim_image_script_video.json 2
template_sirolim_seamless_loop.json 1
templates-1_input-multiple_styles_prompt.app.json 12
templates-2x2_grid-character_bg_product.json 4
templates-2x2_grid-iso_miniatures.json 4
templates-3D_logo_texture_animation.json 1
templates-3x3_grid_brand_icons.json 9
templates-8x8_grid-pfp.json 65
templates-9grid_social_media-v2.0.json 10
templates-all_in_one-image_edit_models.json 3
templates-car_product.json 7
templates-fashion_shoot_prompt_doodle.json 5
templates-fashion_shoot_vton.json 6
templates-multiple_consistent_shots-nb_pro.json 1
templates-photo_to_product_vid.json 3
templates-poster_product_integration.json 3
templates-poster_to_2x2_mockups-v2.0.json 5
templates-product_ad-v2.0.json 1
templates-product_scene_relight.json 1
templates-split_stack.json 5
templates-sprite_sheet.json 32
templates-stitched_vid_contact_sheet.json 21
templates-textured_logotype-v2.1.json 1
templates_all_in_one_image_edit_models.app.json 4
templates_mjm_airt_machine_api.json 6
templates_mjm_image_to_3d.json 3
templates_mjm_looped_restyler.json 3
templates_product_scene_transformation.json 4
templates_purz_wan22_animate_auto_full_scene.json 1
templates_rob_fashion_shoot_vton-4in1.app.json 4
templates_rob_kling3_0_multishot_llm_product.json 1
utility-audioseparation.json 4
utility-topaz_landscape_upscaler.json 1
utility_moge_depth_estimation.json 1
utility_sirolim_image_controlled_upscale.json 1
utility_topaz_illustration_upscale.json 1
video_ltx2_3_i2v.json 1
video_ltx2_3_ia2v.json 1
video_ltx2_3_ic_lora.json 1
video_ltx2_3_id_lora.json 1
video_ltx2_3_t2v.json 1
video_ltx2_i2v.json 1
video_ltx2_i2v_distilled.json 1
video_ltx2_i2v_lora.json 1
video_wan2_2_14B_animate.json 2

This comment is automatically updated by the ComfyUI Node Compatibility Report workflow.

@glary-bot

glary-bot Bot commented Aug 8, 2026

Copy link
Copy Markdown

No findings to address here — CodeRabbit skipped the review because the PR is a draft, which is intentional on my side (our agent workflow opens PRs as drafts for a human to promote). No code changes were made in response.

Triggering a scan anyway so the change still gets covered before anyone promotes it, since this one is gating the prod template merge.

For reviewer context, the diff is two lines:

  • templates/api_seedance2_5_flf2v.json — the leaked internal note in the ByteDance2FirstLastFrameNode default prompt replaced with a generic cinematic prompt describing the template's own bundled pool_start_frame.pngpool_end_frame.png transition.
  • packages/core/.../manifest.json — regenerated asset SHA256 via npm run sync:bundles (required, or validate:manifests fails).

Already verified: validate:templates ✅, validate:manifests ✅ (no errors), pytest packages/core/tests ✅ 38 passed, plus an end-to-end load of the template in a live ComfyUI + frontend to confirm the node renders a normal prompt and every other widget value is unchanged (no widgets_values array shift).

@coderabbitai review

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

🔤 Spellcheck Results

Check Status
templates/index.json (titles & descriptions) ✅ No issues found
Workflow JSON notes (MarkdownNote / Note nodes) ✅ No issues found

All spellchecks passed! 🎉

@glary-bot

glary-bot Bot commented Aug 8, 2026

Copy link
Copy Markdown

Thanks for the review @synap5e. The approval came through with no comment body or inline comments, so I read it as a straight approve — no changes made, branch is still at the reviewed commit 363b355.

CI on that commit for the record:

  • Spellcheck ✅ both checks passed (as expected — prompt widgets aren't in the spellcheck corpus, only MarkdownNote/Note nodes and index.json)
  • ComfyUI Node Compatibility — "review suggested", but it's informational/non-blocking and api_seedance2_5_flf2v.json isn't among the 88 flagged templates. All 332 findings are pre-existing deprecated-node warnings on other templates, 0 errors.

Left as a draft per our agent workflow — ready for a maintainer to promote and merge whenever. Since this one is gating the prod template merge, no need to wait on me.

@mobeenabdullah

Copy link
Copy Markdown
Contributor

This looks superseded: #1106 landed the same fix on 2026-08-08 and main's flf2v default prompt no longer carries the internal text. That is also where the conflict here comes from, since both PRs rewrite the same widget value.

Suggest closing unless there is something in this version worth keeping over what merged.

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.

4 participants