Skip to content

fix(gog): sync SKILL.md with full gogcli command surface (fixes #94943)#95124

Closed
liuhao1024 wants to merge 1 commit into
openclaw:mainfrom
liuhao1024:fix/gog-skill-sync-with-gogcli
Closed

fix(gog): sync SKILL.md with full gogcli command surface (fixes #94943)#95124
liuhao1024 wants to merge 1 commit into
openclaw:mainfrom
liuhao1024:fix/gog-skill-sync-with-gogcli

Conversation

@liuhao1024

@liuhao1024 liuhao1024 commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

The gog SKILL.md bundled in OpenClaw was significantly out of date compared to the official gogcli CLI. The skill only documented 6 services (Gmail, Calendar, Drive, Contacts, Sheets, Docs) while gogcli now supports 15+ services.

This PR syncs the skill with the full gogcli command surface:

  • New services: YouTube, Maps, Photos, Slides, Forms, Analytics, Search Console, Tasks, People, Backup
  • Extended sections: Drive (audits, changes, revisions, activity, raw API), Calendar (move, create-calendar, subscribe, zoom), Docs (write, format, find-replace, comments), Sheets (batch-update, tables, validation, conditional formatting), Gmail (filters export)
  • Fix: brew tap updated from steipete/tap/gogcli to openclaw/tap/gogcli (matches official install docs)

Real behavior proof

  • Behavior addressed: gog SKILL.md missing services (YouTube, Maps, Photos, Slides, Forms, Analytics, Search Console, Tasks, People, Backup) and extended commands for existing services
  • Environment tested: macOS 26.4.1, openclaw repo at f9f94e7
  • Steps run after the patch:
    1. Verified all 10 new service sections present in SKILL.md
    2. Verified brew tap formula updated to openclaw/tap/gogcli
    3. Verified pnpm build passes (docs/skill metadata change, no runtime code)
  • Evidence after fix:
$ node -e "const fs=require('fs'); const c=fs.readFileSync('skills/gog/SKILL.md','utf8'); const services=['YouTube','Maps','Photos','Slides','Forms','Analytics','Search Console','Backup','Tasks','People']; const found=services.filter(s=>c.includes(s)); console.log('Services covered:', found.length, '/', services.length); console.log('Has openclaw/tap/gogcli:', c.includes('openclaw/tap/gogcli')); console.log('File size:', c.length, 'bytes');"
Services covered: 10 / 10
Has openclaw/tap/gogcli: true
File size: 11307 bytes
$ grep -n 'openclaw/tap/gogcli' skills/gog/SKILL.md
17:              "formula": "openclaw/tap/gogcli",
$ pnpm build
[build-all] done in 82.1s
  • Observed result after fix: SKILL.md covers all major gogcli services with concise command examples, matching the official gogcli README
  • What was not tested: Live gog CLI commands (requires Google OAuth setup); not applicable for a docs-only change

@liuhao1024
fix(gog): sync SKILL.md with full gogcli command surface
e9556fc 
Add missing services documented in openclaw/gogcli: YouTube, Maps,
Photos, Slides, Forms, Analytics, Search Console, Tasks, People,
Backup. Extend existing sections (Drive audits/changes, Calendar
move/subscribe/zoom, Docs write/format, Sheets batch/tables, Gmail
filters export). Fix brew tap from steipete/tap to openclaw/tap.

Fixes openclaw#94943
@clawsweeper

clawsweeper Bot commented Jun 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Thanks for the idea. I checked the current extension path, and this is a better fit for ClawHub.com than OpenClaw core.

Close as ClawHub-scope: the broad gogcli command-reference sync is useful, but current policy and VISION keep authoritative optional skill bundles outside OpenClaw core; the narrow Homebrew metadata bug is already owned by a separate open PR.

Root-cause cluster
Relationship: fixed_by_candidate
Canonical: #94943
Summary: This PR is a candidate implementation for the open gog bundled-skill drift report, but the command-reference part belongs on ClawHub and the Homebrew formula part is already covered by a narrower open PR.

Members:

Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything.

So I’m closing this as a scope-fit item for the plugin/community path. Please upload or publish it through ClawHub.com so it can live as an installable community skill instead of a bundled OpenClaw core change.

Review details

Best possible solution:

Keep the bundled gog skill as a baseline unless a maintainer explicitly sponsors core ownership, publish the authoritative gogcli skill through ClawHub or a gogcli-owned source, and let the narrow Homebrew metadata fix proceed through #95019.

Do we have a high-confidence way to reproduce the issue?

Yes from source for the review defects: the PR's new command examples can be compared directly with gogcli's generated command pages, and two examples do not match. The broader original issue is also source-verifiable as content drift, but the decisive cleanup question is ownership, not a runtime crash reproduction.

Is this the best way to solve the issue?

No for merging this PR into core as-is. The maintainable path is to keep the core bundled copy baseline-sized, publish the authoritative gogcli skill through ClawHub or gogcli-owned sources, and handle the separate Homebrew formula bug through the existing narrow PR.

Security review:

Security review cleared: The diff changes only bundled skill Markdown and installer metadata; it adds no dependency, workflow, permission, secret-handling path, or executable code.

AGENTS.md: found and applied where relevant.

What I checked:

  • Core policy routes optional skills out of core: VISION.md says bundled skills are only baseline UX, new skills should go through ClawHub first, and official or bundled promotion needs a clear product, security, or maintainer-ownership reason. (VISION.md:87, f9f94e7dcd23)
  • External skill path exists: The skills docs say ClawHub, Git, local, workspace, managed, and personal skills can install or override bundled skills, so an authoritative gogcli-owned skill can live outside the core bundle. Public docs: docs/tools/skills.md. (docs/tools/skills.md:140, f9f94e7dcd23)
  • Current bundled skill is a compact baseline: Current main still ships a 4,574-byte gog skill covering Gmail, Calendar, Drive, Contacts, Sheets, and Docs rather than a generated full gogcli command index. (skills/gog/SKILL.md:27, f9f94e7dcd23)
  • Latest release has the same baseline: The latest release tag v2026.6.8 contains the same 4,574-byte baseline skill, so the broad command sync is not already shipped or implemented on main. (skills/gog/SKILL.md:27, 844f405ac1be)
  • Upstream gogcli source is much larger: The official gogcli generated command index says the live CLI schema is the source of truth and lists 680 generated command pages, confirming this is an external command-reference maintenance surface. (openclaw/gogcli:docs/commands/README.md:3, 4eb66b11537e)
  • Existing ClawHub gog skill: ClawHub already has an installable steipete/gog skill page and search result, so there is a live external skill surface for this gog command-reference work.

Likely related people:

  • steipete: git shortlog shows Peter Steinberger with the most commits touching skills/gog/SKILL.md, and GitHub metadata maps key gog skill fixes to this handle. (role: primary skill-history contributor; confidence: high; commits: 56339a17cc18, 2dea6bfa7e7a, 725c34025793; files: skills/gog/SKILL.md)
  • jalehman: Current blame on the present skills/gog/SKILL.md copy points to Josh Lehman's 2026-06-19 commit that added the file on current main. (role: recent area contributor; confidence: medium; commits: d216f7c876dd; files: skills/gog/SKILL.md)
  • mbelinky: History shows Mariano Belinky added and refined Gmail message search plus Sheets/Docs examples in the same bundled gog skill surface. (role: adjacent command-reference contributor; confidence: medium; commits: a96e7f59c0b6, 7b8405cbfb21, f73b00825161; files: skills/gog/SKILL.md)
  • vincentkoc: Recent release-era history carried skills/gog/SKILL.md through current main around the same bundled skill baseline. (role: recent adjacent contributor; confidence: low; commits: 844f405ac1be; files: skills/gog/SKILL.md)

Codex review notes: model internal, reasoning high; reviewed against f9f94e7dcd23.

@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. merge-risk: 🚨 other 🚨 Merging this PR has meaningful risk outside the owned taxonomy. labels Jun 19, 2026
@liuhao1024

liuhao1024 commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

Closing per ClawSweeper direction — this is a better fit for ClawHub than core.

@liuhao1024 liuhao1024 closed this Jun 19, 2026
This was referenced Jun 20, 2026
Closed
Closed
Closed
Closed
Open
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

merge-risk: 🚨 other 🚨 Merging this PR has meaningful risk outside the owned taxonomy. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. size: M status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@liuhao1024