Skip to content

feat: add ClawHub publishing workflow#8

Merged
ningzimu merged 3 commits into
mainfrom
codex/add-clawhub-publish-workflow
May 9, 2026
Merged

feat: add ClawHub publishing workflow#8
ningzimu merged 3 commits into
mainfrom
codex/add-clawhub-publish-workflow

Conversation

@ningzimu
Copy link
Copy Markdown
Owner

@ningzimu ningzimu commented May 9, 2026

Summary

  • Add a GitHub Actions workflow to publish the codex-ppt skill to ClawHub with CLAWHUB_TOKEN.
  • Support manual workflow dispatch and v* tag publishing, including automatic SKILL.md version synchronization before upload.
  • Add ClawHub/OpenClaw skill metadata and a .clawhubignore file to keep local runtime artifacts out of releases.
  • Remove the unused --api-key-stdin runtime option after ClawHub's scanner flagged the stdin variable as suspicious in a manual 0.1.0 publish test.

Manual ClawHub test

  • Published codex-ppt@0.1.0 manually with clawhub 0.12.3.
  • Found relative publish paths fail locally with "Path must be a folder"; the workflow now publishes via an absolute skill path.
  • Verified uploaded files excluded .venv, pycache, and .DS_Store.

Validation

  • ruby -e "require 'yaml'; YAML.load_file('.github/workflows/publish-clawhub-skill.yml'); puts 'workflow yaml ok'"
  • python3 -m py_compile skills/codex-ppt/scripts/codex_ppt_runtime.py skills/codex-ppt/scripts/image_gen.py skills/codex-ppt/scripts/assemble_ppt.py skills/codex-ppt/scripts/remove_chroma_key.py
  • git diff --check

@ningzimu ningzimu marked this pull request as ready for review May 9, 2026 14:58
@ningzimu ningzimu merged commit ad3d8cc into main May 9, 2026
2 checks passed
@ningzimu ningzimu deleted the codex/add-clawhub-publish-workflow branch May 9, 2026 14:58
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 378318249f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/codex-ppt/SKILL.md
Comment on lines +10 to +11
config:
- ~/.codex-ppt-skill/.env
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Remove optional .env from required config metadata

Declaring ~/.codex-ppt-skill/.env under metadata.openclaw.requires.config marks that file as a hard runtime requirement, but this skill is designed to work without local API fallback config (built-in image generation path, and OPENAI_API_KEY is explicitly optional in envVars). In OpenClaw/ClawHub requirement checks, this can surface as an unmet prerequisite for fresh installs and block or warn users before they can use the skill in no-API scenarios; this should be moved to optional metadata or omitted from requires.

Useful? React with 👍 / 👎.

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