Skip to content

Add the ProfitCtl product video to the website - #70

Merged
hudsonaikins merged 5 commits into
mainfrom
codex/profitctl-video-distribution
Oct 8, 2026
Merged

hudsonaikins merged 5 commits into
mainfrom
codex/profitctl-video-distribution

Conversation

@hudsonaikins

@hudsonaikins hudsonaikins commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Prospective users can watch a 30-second ProfitCtl comparison between the homepage introduction and the existing interactive terminal, then try the demo or install the CLI. The reviewed MP4 and a lightweight poster are bundled with the website, and the hero links to the stable #watch section.

The player starts only on request, supports keyboard pause and seeking across browsers, handles failed loads with retry, and keeps native controls available without JavaScript. A written summary labels the actual ProfitCtl output as illustrative scenarios.

Validation

  • go test ./... and the required schema, semantic, CLI workflow, operational, and security validators passed through the pinned product-validation runner on 0e75093.
  • Website build, typecheck, interactive example, metadata, and internal-link checks passed.
  • Chromium and WebKit passed desktop, tablet, narrow mobile, light/dark themes, reduced motion, keyboard controls, error/retry, and no-JavaScript fallback checks. Neither engine requested the MP4 before play. The existing terminal completed its comparison in both engines.
  • Packaged MP4 matches the reviewed source: SHA-256 55e6d297c0894ee105d77d0adcd1cd1e35bd08e38f2c4e90a8f00f3230720fbc; 4,662,112 bytes; 30 seconds; 1920×1080; silent H.264.

Review Readiness

  • Codex reviewed the scoped diff and resolved the browser issues found during validation.
  • All six hosted checks passed on the pushed head: dead-code, Go verification, product validation, installer smoke, website verification, and security scan.
  • Merge and production deployment have explicit human approval.

No matching AgentShift issue was found for this distribution work. PCTL-27 covers a separate Condere demo and was left unchanged.

Release Impact

  • No CLI release or install behavior changes.

Notes

Publish this website and verify https://profitctl.com/#watch and both media URLs before merging the separate README update and IntelIP homepage PR. The README is intentionally unchanged in this PR. Live publication and live playback have not yet been verified. No paid generation or external model calls were used for this integration.

@makeplane

makeplane Bot commented Oct 8, 2026

Copy link
Copy Markdown

Linked to Plane Work Item(s)

References

This comment was auto-generated by Plane

@hudsonaikins
hudsonaikins marked this pull request as ready for review October 8, 2026 00:45
@hudsonaikins
hudsonaikins merged commit 5df31b5 into main Oct 8, 2026
6 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T00:47:36.121963Z 0e75093 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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.

1 participant