Skip to content

Remove duplicate YouTube covers from README demo sections - #42

Merged
houlianpi merged 1 commit into
mainfrom
codex/readme-demo-trim
Oct 20, 2026
Merged

houlianpi merged 1 commit into
mainfrom
codex/readme-demo-trim

Conversation

@houlianpi

Copy link
Copy Markdown
Collaborator

Summary

The English and Chinese README demo sections displayed the GitHub inline video followed by a second, linked YouTube cover. Remove the duplicate covers while retaining the inline video and all existing YouTube, caption, and provenance text links.

Related issue

No separate issue is required for this narrowly scoped presentation cleanup requested directly by the user. The video remains sourced from #40.

Design document

Direct /spec-driven request. No separate design document was needed for this existing-content cleanup.

SPEC updates

No SPEC delta. Root SPEC.md, Public Product Presentation, already grounds concise current media and equivalent bilingual journeys; it does not require duplicate video covers. No supported workflow, destination, runtime, interface, dependency, or architecture changes.

Changes

  • Delete only the linked YouTube cover paragraph in each README.
  • Update one media-description sentence and the existing documentation contract test.
  • Preserve the three explanatory images, poster asset, Website, and all text links.

Verification

  • .venv/bin/python -m pytest tests/test_distribution_contract.py -q — 20 passed.
  • .venv/bin/ruff check tests/test_distribution_contract.py — passed.
  • .venv/bin/ruff format --check tests/test_distribution_contract.py — passed.
  • git diff --check — passed.
  • Byte comparison confirms each README differs only by the removed cover paragraph.
  • Independent combined SPEC-to-change and change-to-SPEC audit passed with no findings; final diff and file identities matched the reviewed snapshot.

Evidence

The four-file diff is the complete change. No new visual media is generated; the existing GitHub video and destinations are unchanged.

Release or documentation impact

The README demo area is shorter and shows one inline video. No Website deployment, package release, merge, or runtime change is included.

Checklist

Spec-driven development

  • Reviewed root SPEC.md and relevant media contracts.
  • Recorded the no-SPEC-delta decision before implementation.
  • Implementation, tests, and media description agree with existing SPEC.
  • Used current SPEC as the implementation authority.

Quality and safety

  • Focused tests, Ruff, format, and diff checks passed.
  • Related media documentation updated.
  • No credentials, private data, generated logs, or workspace output committed.
  • No new screenshots, videos, logs, or Run artifacts included.
  • No large media added; existing poster and Website remain unchanged.

@houlianpi
houlianpi merged commit 9ef27aa into main Oct 20, 2026
9 checks passed
@houlianpi houlianpi mentioned this pull request Oct 21, 2026
9 tasks done
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