chore(gws): sync skills with upstream gws CLI v0.22.5#68
Conversation
|
Warning Review limit reached
Next review available in: 51 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (27)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR bumps the gws plugin manifest version from 2.0.0 to 2.1.0 and replaces the README's "Updating Skills" section with a "Syncing Skills with Upstream" section documenting the process for syncing skills from the upstream googleworkspace/cli repository. ChangesVersion Bump and Sync Documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@gws/README.md`:
- Around line 62-63: The version-bump guidance is inconsistent between the
README and the gws sync skill. Update the policy in both gws/README.md and
.claude/skills/gws-sync.md so they match, using the same bump rule for
content-only updates and sync behavior. Refer to the plugin version guidance
around .claude-plugin/plugin.json and the sync/version instructions in the
gws-sync skill to keep the documentation consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 396ffc1f-f628-436d-ac4c-09e5ad850913
⛔ Files ignored due to path filters (27)
gws/skills/gws-calendar-agenda/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-calendar-insert/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-calendar/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-docs-write/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-docs/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-drive-upload/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-drive/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail-forward/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail-reply-all/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail-reply/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail-send/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail-triage/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail-watch/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-gmail/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-meet/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-shared/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-sheets-append/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-sheets-read/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-sheets/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-slides/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-tasks/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-workflow-email-to-task/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-workflow-file-announce/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-workflow-meeting-prep/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-workflow-standup-report/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-workflow-weekly-digest/SKILL.mdis excluded by!gws/skills/**gws/skills/gws-workflow/SKILL.mdis excluded by!gws/skills/**
📒 Files selected for processing (2)
gws/.claude-plugin/plugin.jsongws/README.md
| 4. Bump the plugin version in `.claude-plugin/plugin.json` (minor for | ||
| content-only updates). |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '\n## Files\n'
git ls-files gws/README.md .claude/skills/gws-sync.md gws/.claude-plugin/plugin.json
printf '\n## README excerpt\n'
sed -n '50,80p' gws/README.md
printf '\n## Sync skill excerpt\n'
sed -n '1,220p' .claude/skills/gws-sync.md
printf '\n## Plugin manifest excerpt\n'
sed -n '1,120p' gws/.claude-plugin/plugin.jsonRepository: cblecker/claude-plugins
Length of output: 4896
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '\n## Search for version-bump policy references\n'
rg -n --hidden --glob '!**/.git/**' \
-e 'patch version' \
-e 'minor for content-only updates' \
-e 'bump the plugin version' \
-e 'versioning' \
-e 'semantic version' \
gws .claude
printf '\n## Plugin manifest-related docs\n'
rg -n --hidden --glob '!**/.git/**' 'plugin.json|metadata.version|claude plugin validate' gws .claudeRepository: cblecker/claude-plugins
Length of output: 703
Align the version-bump rule between the README and sync skill. The README says content-only updates use a minor bump, but .claude/skills/gws-sync.md says to bump the patch version after syncing. Pick one policy and apply it consistently in both docs.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@gws/README.md` around lines 62 - 63, The version-bump guidance is
inconsistent between the README and the gws sync skill. Update the policy in
both gws/README.md and .claude/skills/gws-sync.md so they match, using the same
bump rule for content-only updates and sync behavior. Refer to the plugin
version guidance around .claude-plugin/plugin.json and the sync/version
instructions in the gws-sync skill to keep the documentation consistent.
Source: Linked repositories
There was a problem hiding this comment.
Pull request overview
This is a documentation maintenance PR that syncs the gws plugin's 28 skill files with upstream googleworkspace/cli v0.22.5. It normalizes each skill's frontmatter to the upstream format (relocating version under metadata, converting inline YAML arrays to block style), drops local CLAUDE_EFFORT customizations that duplicate harness behavior, picks up newly documented CLI flags/resources, and refreshes the plugin README with a documented upstream-sync process. There are no code or hook changes—only Markdown/JSON content.
I verified the key correctness concerns: the newly added +read link in gws-gmail/SKILL.md resolves to the existing gws-gmail-read/ skill; no stray version: 1.0.0 or CLAUDE_EFFORT references remain (the only CLAUDE_EFFORT mention is intentional guidance in the README); the removed /gws-sync command has no lingering definition; and the "28 of ~74 skills" claim matches the 28 skill directories present. Frontmatter block-array formatting is consistent with the already-synced gws-gmail-read skill.
Changes:
- Normalize frontmatter across 27 skill files (
metadata.version: 0.22.5, block-stylebins) and removeCLAUDE_EFFORTscaling tips / thegws-shared"Effort Level" section. - Document newly available upstream flags/resources (
--meet,+read,--range, Gmail--from/--attach/--draft/--no-original-attachments, MeetsmartNotes). - Rewrite the README sync section and bump the plugin version to 2.1.0.
Reviewed changes
Copilot reviewed 29 out of 29 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| gws/.claude-plugin/plugin.json | Minor version bump 2.0.0 → 2.1.0 for the content sync |
| gws/README.md | Replaces /gws-sync reference with a documented upstream sync process/guidelines |
| gws/skills/gws-shared/SKILL.md | Frontmatter normalized; removes the "Effort Level" (CLAUDE_EFFORT) section |
| gws/skills/gws-gmail/SKILL.md | Adds +read helper row (link verified to existing skill); frontmatter normalized |
| gws/skills/gws-gmail-send/SKILL.md | Documents --from/--attach/--draft, updated tips (RFC 5322/MIME) |
| gws/skills/gws-gmail-reply/SKILL.md | Documents --attach/--draft, refreshed --html/--cc descriptions and tips |
| gws/skills/gws-gmail-reply-all/SKILL.md | Documents --attach/--draft, reordered flags, refreshed tips |
| gws/skills/gws-gmail-forward/SKILL.md | Documents --no-original-attachments/--attach/--draft, reordered --body, refreshed tips |
| gws/skills/gws-calendar-insert/SKILL.md | Documents --meet flag and updates tips |
| gws/skills/gws-sheets-append/SKILL.md | Documents --range flag with example and tip |
| gws/skills/gws-meet/SKILL.md | Adds smartNotes resource; frontmatter normalized |
| gws/skills/gws-drive/SKILL.md | Converts relative doc links to absolute URLs; frontmatter normalized |
| gws/skills/gws-workflow-weekly-digest, -standup-report, -meeting-prep/SKILL.md | Removes CLAUDE_EFFORT verbosity tip; frontmatter normalized |
| gws/skills/gws-workflow, -file-announce, -email-to-task, gws-tasks, gws-slides, gws-sheets, gws-sheets-read, gws-gmail-watch, gws-gmail-triage, gws-drive-upload, gws-docs, gws-docs-write, gws-calendar, gws-calendar-agenda/SKILL.md | Frontmatter-only normalization (version relocation, block arrays, trailing-newline trim) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sync all 28 skill files with upstream googleworkspace/cli to pick up new flags (--meet, --attach, --draft, --from, --range, smartNotes) and normalize frontmatter to upstream format (metadata.version, block YAML arrays). Drop local CLAUDE_EFFORT customizations that duplicate harness behavior. Also remove defunct /gws-sync reference from README and add upstream sync process documentation. Assisted-by: Claude:claude-opus-4-6
cbf8687 to
7661c14
Compare
Summary
googleworkspace/cliv0.22.5metadata.version: 0.22.5, block YAML arrays)CLAUDE_EFFORTcustomizations (redundant with harness behavior)/gws-syncreference from READMENew upstream features picked up
gws-calendar-insert--meetflag for Google Meet linksgws-gmail+readhelper command in tablegws-gmail-forward--no-original-attachments,--attach,--draftflagsgws-gmail-reply--attach,--draftflagsgws-gmail-reply-all--attach,--draftflagsgws-gmail-send--from,--attach,--draftflagsgws-meetsmartNotesresourcegws-sheets-append--rangeflag for targeting specific tabsTest plan
claude plugin validate .— marketplace validation passesclaude plugin validate ./gws— plugin validation passesnpx markdownlint-cli2— 0 errorsuvx skillsaw --strict— 0 errors, 0 warnings, grade Aversion: 1.0.0remaining in any skill fileCLAUDE_EFFORTreferences remaining in any skill fileSummary by CodeRabbit
Documentation
Chores