fix: wrap raw video URLs in HTML video tags; remove orphaned summary tag#40
Closed
farizanjum wants to merge 8 commits into
Closed
fix: wrap raw video URLs in HTML video tags; remove orphaned summary tag#40farizanjum wants to merge 8 commits into
farizanjum wants to merge 8 commits into
Conversation
farizanjum
force-pushed
the
fix/video-tags-and-readme-cleanup
branch
2 times, most recently
from
83f5f81 to
2df2a65
Compare
There was a problem hiding this comment.
Pull request overview
This PR aims to standardize how installation demo videos are embedded across the root README and skill READMEs by replacing raw video URLs with HTML <video> tags, and to clean up README rendering issues (notably an orphaned <summary> tag per the PR description). It also updates the README update tooling to generate the new embed format going forward.
Changes:
- Replace raw video URLs with HTML
<video>embeds in README content. - Add a Manus AI installation section across skill READMEs and update
scripts/update_skill_readmes.pyto generate it. - Add several scripts under
scripts/used during the update process.
Reviewed changes
Copilot reviewed 65 out of 65 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Embeds installation videos and adds a Manus AI installation option (but still contains an orphaned <summary> per current file content). |
| scripts/update_skill_readmes.py | Updates the standard install snippet template to include <video> embeds and a Manus AI section. |
| scripts/add_manus_all_skills.py | One-off helper script to inject Manus AI instructions across skill READMEs. |
| scripts/check_git.py | Local helper script (currently hard-coded to a personal Windows path). |
| scripts/check_problem_skills.py | Helper script for inspecting install section presence in a set of skills. |
| scripts/check_problem_skills2.py | Helper script for printing full install sections for certain skills. |
| scripts/cleanup_manus_formatting.py | Helper script to normalize Manus section spacing/formatting in skill READMEs. |
| scripts/fix_video_tags.py | Local helper script to wrap video URLs (currently hard-coded to a personal Windows path). |
| scripts/git_commit_push.py | Local helper script to create/commit/push a branch (currently hard-coded to a personal Windows path). |
| scripts/git_ops.py | Local helper script to stash/pull/pop (currently hard-coded to a personal Windows path). |
| scripts/resolve_conflict.py | Local helper script to remove a specific conflict-marker block (currently hard-coded to a personal Windows path). |
| scripts/scan_videos.py | Local helper script to scan READMEs for video URLs (currently hard-coded to a personal Windows path). |
| scripts/stage.py | Local helper script to stage README (currently hard-coded to a personal Windows path). |
| scripts/verify.py | Local helper script for conflict-marker detection and git status (currently hard-coded to a personal Windows path). |
| skills/app-store-review-arbitrage/README.md | Updates video embed/installation content to the standardized format. |
| skills/blog-cover-image-cli/README.md | Updates video embed/installation content to the standardized format. |
| skills/brand-alchemy/README.md | Updates video embed/installation content to the standardized format. |
| skills/claude-md-generator/README.md | Updates video embed/installation content to the standardized format. |
| skills/cold-email-verifier/README.md | Updates video embed/installation content to the standardized format. |
| skills/company-radar/README.md | Updates video embed/installation content to the standardized format. |
| skills/competitor-pr-finder/README.md | Updates video embed/installation content to the standardized format. |
| skills/cook-the-blog/README.md | Updates video embed/installation content to the standardized format. |
| skills/dependency-update-bot/README.md | Updates video embed/installation content to the standardized format. |
| skills/docs-from-code/README.md | Updates video embed/installation content to the standardized format. |
| skills/domain-expired-opportunity-finder/README.md | Updates video embed/installation content to the standardized format. |
| skills/dx-roaster/README.md | Updates video embed/installation content to the standardized format. |
| skills/email-newsletter/README.md | Updates video embed/installation content to the standardized format. |
| skills/explain-this-pr/README.md | Updates video embed/installation content to the standardized format. |
| skills/gh-issue-to-demand-signal/README.md | Updates video embed/installation content to the standardized format. |
| skills/github-discussion-to-devrel-content/README.md | Updates video embed/installation content to the standardized format. |
| skills/google-trends-api-skills/README.md | Updates video embed/installation content to the standardized format. |
| skills/graphic-case-study/README.md | Updates video embed/installation content to the standardized format. |
| skills/graphic-chart/README.md | Updates video embed/installation content to the standardized format. |
| skills/graphic-ebook/README.md | Updates video embed/installation content to the standardized format. |
| skills/graphic-gif/README.md | Updates video embed/installation content to the standardized format. |
| skills/graphic-slide-deck/README.md | Updates video embed/installation content to the standardized format. |
| skills/hackernews-intel/README.md | Updates video embed/installation content to the standardized format. |
| skills/human-tone/README.md | Updates video embed/installation content to the standardized format. |
| skills/hyperframes-product-launch-video/README.md | Updates video embed/installation content to the standardized format. |
| skills/kill-the-standup/README.md | Updates video embed/installation content to the standardized format. |
| skills/linkedin-job-post-to-buyer-pain-map/README.md | Updates video embed/installation content to the standardized format. |
| skills/linkedin-post-generator/README.md | Updates video embed/installation content to the standardized format. |
| skills/llms-txt-generator/README.md | Updates video embed/installation content to the standardized format. |
| skills/map-your-market/README.md | Updates video embed/installation content to the standardized format. |
| skills/meeting-brief-generator/README.md | Updates video embed/installation content to the standardized format. |
| skills/meta-ads-skill/README.md | Updates video embed/installation content to the standardized format. |
| skills/meta-tribeV2-skill/README.md | Updates video embed/installation content to the standardized format. |
| skills/newsletter-digest/README.md | Updates video embed/installation content to the standardized format. |
| skills/noise-to-linkedin-carousel/README.md | Updates video embed/installation content to the standardized format. |
| skills/noise2blog/README.md | Updates video embed/installation content to the standardized format. |
| skills/npm-downloads-to-leads/README.md | Updates video embed/installation content to the standardized format. |
| skills/oss-launch-kit/README.md | Updates video embed/installation content to the standardized format. |
| skills/outreach-sequence-builder/README.md | Updates video embed/installation content to the standardized format. |
| skills/position-me/README.md | Updates video embed/installation content to the standardized format. |
| skills/pr-description-writer/README.md | Updates video embed/installation content to the standardized format. |
| skills/pricing-finder/README.md | Updates video embed/installation content to the standardized format. |
| skills/pricing-page-psychology-audit/README.md | Updates video embed/installation content to the standardized format. |
| skills/product-update-logger/README.md | Updates video embed/installation content to the standardized format. |
| skills/producthunt-launch-kit/README.md | Updates video embed/installation content to the standardized format. |
| skills/reddit-icp-monitor/README.md | Updates video embed/installation content to the standardized format. |
| skills/reddit-post-engine/README.md | Updates video embed/installation content to the standardized format. |
| skills/schema-markup-generator/README.md | Updates video embed/installation content to the standardized format. |
| skills/sdk-adoption-tracker/README.md | Updates video embed/installation content to the standardized format. |
| skills/show-hn-writer/README.md | Updates video embed/installation content to the standardized format. |
| skills/store-listing-optimizer/README.md | Updates video embed/installation content to the standardized format (but currently labels Manus section as “Step 3” without Steps 1/2). |
| skills/tweet-thread-from-blog/README.md | Updates video embed/installation content to the standardized format. |
| skills/twitter-GTM-find-skill/README.md | Updates video embed/installation content to the standardized format. |
| skills/vc-curated-match/README.md | Updates video embed/installation content to the standardized format. |
| skills/vc-finder/README.md | Updates video embed/installation content to the standardized format. |
| skills/vid-motion-graphics/README.md | Updates video embed/installation content to the standardized format. |
| skills/vid-product-launch/README.md | Updates video embed/installation content to the standardized format. |
| skills/vid-sizzle-reel/README.md | Updates video embed/installation content to the standardized format. |
| skills/where-your-customer-lives/README.md | Adds Manus installation section (but currently labels it “Step 3” despite no Steps 1/2 in this README). |
| skills/yc-intent-radar-skill/README.md | Updates video embed/installation content to the standardized format. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
Greptile SummaryThis PR wraps raw video URLs in
Confidence Score: 3/5Not safe to merge without restoring deleted content in at least three skill READMEs and removing the duplicate install section in pricing-page-psychology-audit. The bulk-update approach accidentally deleted entire documentation bodies from linkedin-job-post-to-buyer-pain-map (losing ~100 lines: all usage, scoring, and integration sections), competitor-pr-finder (losing What It Does / Output / Requirements), and meta-tribeV2-skill (losing the capabilities list). A second ## Install block was also left behind in pricing-page-psychology-audit. The video-tag wrapping and summary-tag removal that form the core of this PR are correct, but the content losses make several skill pages meaningfully worse for end users. skills/linkedin-job-post-to-buyer-pain-map/README.md, skills/competitor-pr-finder/README.md, skills/meta-tribeV2-skill/README.md, and skills/pricing-page-psychology-audit/README.md all need attention before merging. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[update_skill_readmes.py runs] --> B{has_install_section?\nChecks for '### Option D: Manus AI'}
B -- Yes --> C[Skip — already updated]
B -- No --> D[inject_install_section]
D --> E[Find first '## ' heading in file]
E --> F{File has existing ## Install section?}
F -- No --> G[Prepend new Options A–D block\nbefore first heading ✅]
F -- Yes, without Option D --> H[Prepend BEFORE existing install\n→ Duplicate ## Install sections ⚠️]
F -- Yes, with Option D --> C
subgraph Changed Files
I[README.md\nvideo tags + Option D added ✅]
J[60+ skill READMEs\nvideo tags + Options C/D added]
K[linkedin-job-post-to-buyer-pain-map\n~100 lines of docs deleted ❌]
L[pricing-page-psychology-audit\nduplicate ## Install section ❌]
M[competitor-pr-finder\nWhat It Does / Output / Requirements deleted ❌]
end
Reviews (4): Last reviewed commit: "cleanup: remove temp helper scripts" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
…ort to all skill READMEs
…aned summary in details tag
farizanjum
force-pushed
the
fix/video-tags-and-readme-cleanup
branch
from
2df2a65 to
2bc6446
Compare
…to app-store-review-arbitrage, fix has_install_section check
Contributor
Author
|
what i mess-up i have done! let's close it. |
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
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
Fixes to clean up video tags, remove broken HTML, and ensure all 60 skills have consistent install sections including Manus AI.
1. Remove orphaned summary tag from update-readme.ts template
The root README had a bare
Details
. The auto-generation script (update-readme.ts) kept re-adding it. Removed it from the script template so it stays gone.2. Add Option D: Manus AI to all 60 skills
app-store-review-arbitrage (the 60th skill) was missing the Manus AI install section entirely. Added full install section with video and install link. All other 59 skills already had it.
3. Fix has_install_section check in update_skill_readmes.py
The duplicate-detection regex was checking for
## Installimmediately followed by a code block, but the actual format has sub-headings between them. Changed to check for### Option D: Manus AIinstead, which is more reliable.4. Standardize video embeds
Wrapped raw video URLs in
5. Synced with origin/main
Applied all fixes consistently across all 60 skills.
Files Changed