You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Oct 21, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Oct 21, 2026. It is now read-only.
Chore: Run the Minimal Pre-Release Agent-Ready Smoke Test #2931
changed the title [-]Run the Minimal Pre-Release Agent-Ready Smoke Test[/-][+]Chore: Run the Minimal Pre-Release Agent-Ready Smoke Test[/+]on Sep 22, 2026
The exact primary prompts and expected success criteria are not included here, so please add them (or paste a versioned test matrix) for the Next.js, plain-HTML, and Vue runs. That will make this a repeatable release gate rather than an ad-hoc check.
Planning: this is a GA release-gate task and should remain under videojs/video.js#9440; P1 is appropriate because a failed smoke should block the coordinated launch stack. Coordinate its execution with the release-ready states of #2848, #2883, and videojs/skills#1. No duplicate identified.
I ran each installation guide's AI Quickstart prompt in clean apps with two real coding agents, headless, each in an isolated config.
App
Guide prompt
Claude Code
Codex
Next.js (create-next-app)
React
✅
✅
Plain HTML page (no package.json)
HTML
✅ plays, but see finding 1
✅ plays, but see finding 1
Vite (vanilla TS)
HTML
✅
✅
Vite + Vue
Vue
✅
✅
How it ran
Session 1: the guide's exact AI Quickstart prompt.
Session 2: a fresh session in the same config, asked to "Add a Video.js video player to this app's main page that plays a demo video", mirroring the skill's own "start a new session" follow-up.
Pre-release sources: the CLI from pkg.pr.new/@videojs/cli@c40afc2 (main), @videojs/* packages from the same preview, the skill from the feat/videojs-skill branch of feat: add videojs agent skill skills#1, and docs from main.videojs.org. The CDN path used the published 10.0.0-rc.2 CDN, because jsDelivr only serves released versions.
Verification: I rebuilt every app (npm run build passes in all 8) and loaded the production builds in a browser. Every player loads the demo video and plays (readyState 4, currentTime advancing) with no errors.
What worked in every run
Each agent ran agents skills, installed the skill with its own client, and said a new session was needed.
In session 2 the skill was loaded. For Codex, its session lists videojs:videojs.
Agents planned with agents init using explicit flags, and installed the version the plan named.
Findings (non-blocking follow-ups)
Plain HTML page becomes an npm project. Asked to add a player to a static page, both agents scaffolded Vite into it (package.json, src/) instead of using CDN scripts. Codex used CDN first, then switched when asked again.
Cause: agents init defaults to --method packaged. It detects "an index.html with no package.json", but the packaged plan for an existing page needs a bundler, so agents create a new app instead.
Suggested fix: when the CLI detects a plain page, default to --method cdn and list it as defaulted with its source.
Codex treats the Quickstart prompt as a build request. "Then use the Video.js skill for this project" led Codex to add a player in session 1, before anyone asked for one. Claude Code only installed the skill.
Suggested wording: "…when you work on video or audio in this project."
Not covered
The exact released commands (npx @videojs/cli, videojs/skills) can't be exercised until the release is out and videojs/skills#1 merges. A short repeat after release would cover them.
Activity
[-]Run the Minimal Pre-Release Agent-Ready Smoke Test[/-][+]Chore: Run the Minimal Pre-Release Agent-Ready Smoke Test[/+]github-actions commentedon Sep 22, 2026
The exact primary prompts and expected success criteria are not included here, so please add them (or paste a versioned test matrix) for the Next.js, plain-HTML, and Vue runs. That will make this a repeatable release gate rather than an ad-hoc check.
Planning: this is a GA release-gate task and should remain under videojs/video.js#9440; P1 is appropriate because a failed smoke should block the coordinated launch stack. Coordinate its execution with the release-ready states of #2848, #2883, and videojs/skills#1. No duplicate identified.
mihar-22 commentedon Sep 25, 2026
Smoke test results: pass, no release blockers
I ran each installation guide's AI Quickstart prompt in clean apps with two real coding agents, headless, each in an isolated config.
How it ran
pkg.pr.new/@videojs/cli@c40afc2(main),@videojs/*packages from the same preview, the skill from thefeat/videojs-skillbranch of feat: add videojs agent skill skills#1, and docs frommain.videojs.org. The CDN path used the published10.0.0-rc.2CDN, because jsDelivr only serves released versions.npm run buildpasses in all 8) and loaded the production builds in a browser. Every player loads the demo video and plays (readyState4,currentTimeadvancing) with no errors.What worked in every run
agents skills, installed the skill with its own client, and said a new session was needed.videojs:videojs.agents initusing explicit flags, and installed the version the plan named.Findings (non-blocking follow-ups)
package.json,src/) instead of using CDN scripts. Codex used CDN first, then switched when asked again.agents initdefaults to--method packaged. It detects "anindex.htmlwith nopackage.json", but the packaged plan for an existing page needs a bundler, so agents create a new app instead.--method cdnand list it as defaulted with its source.Not covered
The exact released commands (
npx @videojs/cli,videojs/skills) can't be exercised until the release is out and videojs/skills#1 merges. A short repeat after release would cover them.