docs: sync README with current SKILL.md and update GitHub org#38
Conversation
- Fix CLI install URL and all links: GridLtd-ProductDev → NarratorAI-Studio - Update CLI version requirement: v0.1.0+ → v1.0.0+ - Rewrite "What's in SKILL.md" section table to reflect restructured SKILL.md (references/ split, new Agent Rules / Conversation Initiation / Data & Privacy sections) - Correct Important Notes count: 9 → 7 - Update built-in movie count: 93 → ~100
There was a problem hiding this comment.
Pull request overview
Updates the English and Chinese READMEs to reflect the current SKILL.md structure and the GitHub org/repo location for narrator-ai-cli, aligning onboarding instructions with the latest documentation.
Changes:
- Updated CLI install command and most CLI repo links from
GridLtd-ProductDevtoNarratorAI-Studio. - Updated the CLI minimum version requirement from
v0.1.0+tov1.0.0+. - Refreshed the “What’s in SKILL.md” section tables and resource counts to match the restructured
SKILL.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| README.md | Updates install/link targets, version requirement, and SKILL.md section table/resource counts. |
| README_CN.md | Mirrors the README.md doc sync in Chinese (install/link targets, version requirement, SKILL.md table/resource counts). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…md only SKILL.md references references/ subdirectory (resources.md, workflows.md, magic-video.md, operations.md) — copying only SKILL.md leaves the skill broken. Switch all platforms to git clone; update WorkBuddy/QClaw to upload all files including references/; replace update tip with git pull.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Tested Platforms: replace 'Upload SKILL.md' / 'Point to SKILL.md' with git clone (all CLI-based platforms) or 'SKILL.md + references/' (WorkBuddy/QClaw) - SKILL.md table: use English labels for Two Workflow Paths row
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…les path Cursor uses .cursor/rules/ not .skills/ — group Windsurf/Claude Code together and give Cursor its own snippet. Update Tested Platforms table to match.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…table Both sections exist in SKILL.md but were absent from the README overview table. Also remove the redundant references/ footer row — Reference Index already covers it in the correct position per SKILL.md section order.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…tructure git clone requires parent directory to exist; add mkdir -p for all platforms. WorkBuddy/QClaw: explicitly warn that references/ must be kept as a subfolder alongside SKILL.md — flattening breaks relative path references.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lxjmaster
left a comment
lxjmaster
left a comment
There was a problem hiding this comment.
AI Code Review
Verdict: approve
Model: sonnet | Cost: $0.24 | Mode: session-managed | Config: 517f281a
Issues not placed inline (1)
1. [SUGGESTION] README.md L1
Impact: Linked issue #33 (GridLtd-Labs/agent-pr-review: raise unit test coverage from 31% to 70%+) belongs to a different repository and tracks test coverage for the PR-review system itself, not for this skill repo. None of its acceptance criteria (overall coverage ≥ 70%, quality_gate.py ≥ 90%, state_manager.py ≥ 90%) can be satisfied by a documentation PR in narrator-ai-cli-skill. Impact: issue remains open/unresolved if it was intentionally linked here.
Suggestion: Remove the cross-repo issue link from this PR. If the intent was to track test coverage work, open or link the issue on the correct repository (narrator-ai-cli-skill) or in the agent-pr-review PR where the work actually lives.
Generated by agent-pr-review
Summary
GridLtd-ProductDev→NarratorAI-Studiov0.1.0+→v1.0.0+Changes apply to both
README.md(EN) andREADME_CN.md(CN).Test plan
NarratorAI-Studio/narrator-ai-cliNarratorAI-StudioSKILL.mdstructure