Require a short intent line and demo video from external contributors - #7654
Merged
Merged
Conversation
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 4 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
ComputelessComputer
force-pushed
the
docs/pr-demo-video-requirement
branch
from
September 16, 2026 23:47
3bc2265 to
0d3cc47
Compare
There was a problem hiding this comment.
All reported issues were addressed across 3 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Ask contributors for a short screen recording or GIF alongside the Problem/Fix summary, with an N/A escape hatch for non-functional changes.
Cubic already summarizes the code diff, so contributors only need one short Intent line plus the demo, not a two-field write-up.
Update the "Open a pull request" section to match the new template: a short Intent line plus an attached demo video/GIF, and explain why (Cubic already handles code summaries).
Replace the Problem/Fix parsing with a single Intent check (20-400 chars, so it stays short) and add the Demo section check requiring a video/GIF link or N/A plus reason.
Agents that open PRs or write commit messages here should keep the title as intent and the description as a short summary of what changed, matching the human-facing template.
Accept demo links across hosts, exclude code blocks, and require same-line N/A reasons. Correct the Intent prompt and add workflow regression tests.
ComputelessComputer
force-pushed
the
docs/pr-demo-video-requirement
branch
from
September 17, 2026 06:13
55ac90f to
2f6fc35
Compare
Test overly long intent lines and missing Demo headings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Intent: External PRs only explained problem/fix in prose. Cubic already summarizes the code diff, so replace that with one short Intent line plus a required demo video/GIF, so maintainers can see what changed without reading a long write-up.
Demo
N/A, this is a docs and CI-config change to the contribution process itself with no user-facing product behavior.
Verification
Read the rendered diff for all 3 files. Tested the updated CI validation logic against 6 cases (good PR, missing intent, too-short intent, too-verbose intent, missing demo, maintainer bypass) by running the extracted check logic locally with sample PR bodies.
Summary by cubic
Tightens the external contribution flow: the PR template now asks for one short Intent line and a required Demo video/GIF instead of a Problem/Fix write-up, and agents are held to the same convention.
CONTRIBUTING.mddocuments the new template, andAGENTS.mdtells agents to keep PR titles as intent and descriptions short.Written for commit eea8361. Summary will update on new commits.