Skip to content

refactor(claws): retire split sageox skills; unified pin → v0.7.2#603

Merged
galexy merged 1 commit into
mainfrom
worktree-openclaw-consolidation
May 12, 2026
Merged

refactor(claws): retire split sageox skills; unified pin → v0.7.2#603
galexy merged 1 commit into
mainfrom
worktree-openclaw-consolidation

Conversation

@galexy
Copy link
Copy Markdown
Contributor

@galexy galexy commented May 12, 2026

Summary

Consolidates the OpenClaw skill family to a single unified sageox skill and retires the two specialized predecessors.

  • Removed claws/openclaw/sageox-distill/ and claws/openclaw/sageox-summary/ (the unified claws/openclaw/sageox/ skill already covers their full capability surface).
  • Rewrote claws/openclaw/README.md for the single-skill story; updated claws/openclaw/PUBLISHING.md slug reservation and the explicit-publish example.
  • Ported forward the upgrader behavior from feat(claws): make openclaw update-ox.sh actually upgrade ox #600 into the unified skill's scripts/update-ox.sh — on drift, re-runs install-ox-curl.sh to converge (was a passive readiness gate); SKILL.md § 2 documents the new contract (exit 0/2/3).
  • Bumped the unified skill's pinned ox release v0.6.3v0.7.2 in scripts/install-ox-curl.sh; all 5 sha256s cross-checked against the actual v0.7.2 checksums.txt published on the release.
  • Fixed .claude/skills/clawhub-skill-lint/SKILL.md example commands that referenced the retired skill paths.

Net: 21 files changed, 119 insertions, 1,964 deletions.

Motivation

The two specialized skills (distill, summary) duplicated install scaffolding (install-ox-curl.sh, update-ox.sh, references/INSTALL.md) and forced consumers to install two skills to get one end-to-end pipeline. The unified sageox skill already exposes both capabilities (plus query, coworkers, glance, catchup, import/export) with a single install path and shared state.

The slug-retirement on ClawHub itself (rename to redirect, or hard-retire) is called out in PUBLISHING.md as a separate registry-side action — out of scope here.

Test plan

  • clawhub sync --root claws/openclaw --dry-run shows only the sageox skill (no orphan refs to the retired slugs)
  • python3 .claude/skills/clawhub-skill-lint/scripts/lint.py claws/openclaw/sageox returns 0
  • bash -n claws/openclaw/sageox/scripts/update-ox.sh (syntax check — verified locally)
  • On a host with ox v0.6.3 pinned via the old state file, running the new update-ox.sh triggers an in-place re-run of install-ox-curl.sh and converges to v0.7.2 with matching sha256

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Refactor

    • Consolidated sageox-distill and sageox-summary skills into a unified sageox skill with simplified installation and usage.
    • Updated prerequisite installation flow with clearer exit-code guidance and enhanced version drift detection.
  • Bug Fixes

    • Updated pinned binary version from v0.6.3 to v0.7.2 with refreshed checksums across all platforms.
  • Documentation

    • Updated publishing runbook, README, and skill documentation to reflect the consolidated skill structure and simplified installation workflow.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ca72a3fb-2307-42e4-b8bc-9f078d6969f9

📥 Commits

Reviewing files that changed from the base of the PR and between 8967a47 and 7507de5.

📒 Files selected for processing (21)
  • .claude/skills/clawhub-skill-lint/SKILL.md
  • claws/openclaw/PUBLISHING.md
  • claws/openclaw/README.md
  • claws/openclaw/sageox-distill/.clawhubignore
  • claws/openclaw/sageox-distill/README.md
  • claws/openclaw/sageox-distill/SKILL.md
  • claws/openclaw/sageox-distill/references/INSTALL.md
  • claws/openclaw/sageox-distill/scripts/install-ox-curl.sh
  • claws/openclaw/sageox-distill/scripts/update-ox.sh
  • claws/openclaw/sageox-summary/.clawhubignore
  • claws/openclaw/sageox-summary/README.md
  • claws/openclaw/sageox-summary/SKILL.md
  • claws/openclaw/sageox-summary/assets/SUMMARIZE.md
  • claws/openclaw/sageox-summary/references/INSTALL.md
  • claws/openclaw/sageox-summary/scripts/install-ox-curl.sh
  • claws/openclaw/sageox-summary/scripts/select-new-entries.sh
  • claws/openclaw/sageox-summary/scripts/update-ox.sh
  • claws/openclaw/sageox-summary/scripts/update-state.sh
  • claws/openclaw/sageox/SKILL.md
  • claws/openclaw/sageox/scripts/install-ox-curl.sh
  • claws/openclaw/sageox/scripts/update-ox.sh
💤 Files with no reviewable changes (15)
  • claws/openclaw/sageox-distill/.clawhubignore
  • claws/openclaw/sageox-summary/.clawhubignore
  • claws/openclaw/sageox-distill/references/INSTALL.md
  • claws/openclaw/sageox-summary/references/INSTALL.md
  • claws/openclaw/sageox-distill/scripts/install-ox-curl.sh
  • claws/openclaw/sageox-distill/README.md
  • claws/openclaw/sageox-summary/scripts/update-state.sh
  • claws/openclaw/sageox-distill/SKILL.md
  • claws/openclaw/sageox-summary/README.md
  • claws/openclaw/sageox-summary/SKILL.md
  • claws/openclaw/sageox-summary/scripts/update-ox.sh
  • claws/openclaw/sageox-summary/assets/SUMMARIZE.md
  • claws/openclaw/sageox-summary/scripts/select-new-entries.sh
  • claws/openclaw/sageox-summary/scripts/install-ox-curl.sh
  • claws/openclaw/sageox-distill/scripts/update-ox.sh

📝 Walkthrough

Walkthrough

This PR consolidates two separate OpenClaw skills (sageox-distill and sageox-summary) into a unified sageox skill, removes the split skill directories and associated scripts, updates the unified skill's ox installation pin from v0.6.3 to v0.7.2 with refreshed checksums, refactors the version-update logic in update-ox.sh, and updates public documentation to reflect the single-skill model.

Changes

Skill Consolidation & Removal

Layer / File(s) Summary
Remove sageox-distill split skill
claws/openclaw/sageox-distill/SKILL.md, claws/openclaw/sageox-distill/README.md, claws/openclaw/sageox-distill/references/INSTALL.md, claws/openclaw/sageox-distill/scripts/install-ox-curl.sh, claws/openclaw/sageox-distill/scripts/update-ox.sh, claws/openclaw/sageox-distill/.clawhubignore
Deleted the separate sageox-distill skill definition, documentation, installation/update scripts, and asset references; folds distill functionality into the unified sageox skill.
Remove sageox-summary split skill
claws/openclaw/sageox-summary/SKILL.md, claws/openclaw/sageox-summary/README.md, claws/openclaw/sageox-summary/assets/SUMMARIZE.md, claws/openclaw/sageox-summary/references/INSTALL.md, claws/openclaw/sageox-summary/scripts/install-ox-curl.sh, claws/openclaw/sageox-summary/scripts/select-new-entries.sh, claws/openclaw/sageox-summary/scripts/update-ox.sh, claws/openclaw/sageox-summary/scripts/update-state.sh, claws/openclaw/sageox-summary/.clawhubignore
Deleted the separate sageox-summary skill definition, documentation, templates, installation/update/state-management scripts, and references; consolidates summary functionality into the unified sageox skill.

Unified sageox Skill Enhancement

Layer / File(s) Summary
Enhanced ox installation prerequisites & documentation
claws/openclaw/sageox/SKILL.md
Expanded the "Installing `ox`" prerequisite section to clarify pinned ox version synchronization via scripts/update-ox.sh and scripts/install-ox-curl.sh, in-place upgrades when drift is detected, and refined exit-code semantics (added exit code 3 for upgrade failure with stderr instructions).
Ox version pin update & checksum refresh
claws/openclaw/sageox/scripts/install-ox-curl.sh
Bumped pinned sageox/ox release from v0.6.3 to v0.7.2 and updated all embedded SHA256 checksums for darwin_amd64, darwin_arm64, linux_amd64, linux_arm64, and freebsd_amd64.
Refactored ox version detection & drift handling
claws/openclaw/sageox/scripts/update-ox.sh
Restructured script to extract pinned OX_INSTALL_REF directly from the sibling install-ox-curl.sh using awk (instead of reading state JSON), execute the pinned ox binary to verify version match, and handle drift by running the install script with improved exit-code reporting (exit 3 on install failure). Added SCRIPT_DIR initialization and refined stdout/stderr/exit-code documentation.

Public Documentation & Guidance

Layer / File(s) Summary
Consolidate skill references in parent README
claws/openclaw/README.md
Updated the "Skills" section to list only the unified sageox skill instead of separate sageox-distill and sageox-summary entries, rewrote installation instructions to use clawhub install sageox, revised Claude/credential documentation to describe unified skill auth via claude CLI, and adjusted PATH troubleshooting language to reference "the skill" instead of "the SageOx skills".
Update publishing runbook with unified skill example & slug consolidation
claws/openclaw/PUBLISHING.md
Added an explicit clawhub skill publish example for the unified sageox skill with version/tags/changelog options, and updated the "Slug reservation" section to reserve only the sageox slug while noting that the older sageox-distill and sageox-summary slugs are folded into sageox and require separate registry-side retirement if needed.
Improve clawhub-skill-lint invocation examples
.claude/skills/clawhub-skill-lint/SKILL.md
Enhanced the "How to invoke" bash code block to add a dedicated single-skill-folder example, retain parent-directory and multi-path patterns, and replace hardcoded example paths with more generic multi-skill usage forms.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • sageox/ox#548: Updates pinned sageox/ox release and embedded SHA256 checksums in install-ox-curl.sh, directly overlapping the version-pin and checksum refresh in this PR.
  • sageox/ox#511: Refactors ox install/update logic; both PRs modify the same scripts (install-ox-curl.sh, update-ox.sh) with coordinated version detection and exit-code handling.
  • sageox/ox#517: Updates SKILL.md and auth/CLI documentation for the consolidated sageox skill, aligning with the skill consolidation and documentation changes in this PR.

Poem

🐰 Two skills merge into one,
The ox runs faster, the work is done—
Distill and summarize now entwined,
A unified hop leaves splits behind!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: consolidating split sageox skills into a unified skill and updating the pinned version from v0.6.3 to v0.7.2. It is concise and specific.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch worktree-openclaw-consolidation

Comment @coderabbitai help to get the list of available commands and usage tips.

@galexy galexy merged commit 30832ea into main May 12, 2026
3 checks passed
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