docs(skill): Add planning, anti-slop and preview-first rules to the start of the fframes-video skill

[dmtrKovalenko]

Add two rules to the start of the fframes-video skill, so that an agent reads them before it installs anything. The rules tell the agent to plan the brief, the sound and the motion design before it writes code, and to avoid the default choices that make a video look generated. The loop section now tells the agent to run the native preview instead of rendering the video again after each change on the GPU backend, and the "Finish" checklist checks every strip against the "No slop" list.

---

Where	Change
SKILL.md, after the intro	New section "Two rules before anything else": "Plan before you write code" and "No slop"
SKILL.md, "3. The loop"	The preview paragraph starts with "Preview, do not render" for the GPU backend
SKILL.md, "7. Finish"	New step 3: no item from the "No slop" list is left in any strip

Summary by CodeRabbit

• Documentation
  • Updated video-production guidance to emphasize defining the brief and sound, planning distinctive motion, and avoiding generic or static design choices.
  • For GPU projects on machines with an available GPU, guidance now recommends previewing changes rather than rendering after each one.
  • Added a final check for remaining “No slop” items and adjusted the order of the audio-level check.

[cloudflare-workers-and-pages]

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://docs-skill-plan-and-slop-rules-fframes.dmitriy-86d.workers.dev (commit 6519b13)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status	Deployment URL	Commit	Updated (UTC)	See this deployment's details
Build: Success ✅ Deployment: Success ✅ View logs ↗	https://1373ed1a-fframes.dmitriy-86d.workers.dev	6519b13	2026-10-16T03:37:20.091Z	Visit the dashboard ↗
Build: Success ✅ Deployment: Success ✅ View logs ↗	https://b463457d-fframes.dmitriy-86d.workers.dev	94d92c5	2026-10-16T03:36:15.431Z	Visit the dashboard ↗
Build: Success ✅ Deployment: Success ✅ View logs ↗	https://a2b8183e-fframes.dmitriy-86d.workers.dev	17ff76c	2026-10-16T03:34:58.152Z	Visit the dashboard ↗
Build: Success ✅ Deployment: Success ✅ View logs ↗	https://ae6004b3-fframes.dmitriy-86d.workers.dev	c77a522	2026-10-16T03:26:38.262Z	Visit the dashboard ↗

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4e85b875-1823-4247-b6dd-3c390cd5aeb2

📥 Commits

Reviewing files that changed from the base of the PR and between cc99e97 and c77a522.

📒 Files selected for processing: 1

• skills/fframes-video/SKILL.md

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 6 remain after this review.

📝 Walkthrough

The video skill adds guidance for planning the audience, platform, message, tone, assets, sound, and motion. It also adds a checklist against generic visual and copy choices and a corresponding finish-checklist item.

Changes: Video skill guidance

Layer / File(s)	Summary
Planning guidance and finish checks skills/fframes-video/SKILL.md	Adds planning guidance and a “No slop” checklist. The finish checklist now checks strips against that list, and renumbers the audio-level and GPU-preview steps.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to c77a5

This change adds planning and anti-generic-output guidance to the video skill and a matching finish-checklist item. It does not alter runtime behavior, and no merge-blocking risk was found.

Pre-merge checks: 5

Check name	Status	Explanation
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly identifies the documentation change and its main additions: planning, anti-slop guidance, and preview-first rules for the fframes-video skill.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}