Skip to content

Remove unused docs and images, merge duplicate installers - #206

Merged
titanwings merged 1 commit into
dot-skillfrom
cleanup/unused-files-and-installers
Oct 12, 2026
Merged

titanwings merged 1 commit into
dot-skillfrom
cleanup/unused-files-and-installers

Conversation

@titanwings

Copy link
Copy Markdown
Owner

Requested by Tianyi · project thread

Summary

Deletes files that nothing in the repo references, and merges five near-identical installer scripts into the two --host installers.

Changes

  • Delete unreferenced docs: docs/PRD.md (original colleague.skill PRD), docs/SKILL_TYPE_ABSTRACTION_DESIGN.md and _ZH.md (design notes now implemented in tools/skill_presets.py), docs/lang/README_EN.md (stale copy of the root README.md, linked from no language nav).
  • Delete unreferenced images: docs/social-preview.png, docs/social-preview-distilly-v5/v6/v7.png (only v10 is used), and docs/assets/wechat-group-qr-*.png (8 QR codes).
  • Merge install_openclaw_skill.py, install_hermes_skill.py and install_codex_skill.py into tools/install_distilly.py --host {openclaw,hermes,codex}, using the same default paths as before.
  • Remove install_openclaw_generated_skill.py and install_codex_generated_skill.py, which were thin wrappers. install_generated_skill.py --host already covers them. skill_writer.py --install-{openclaw,codex}-skill now calls the shared installer directly, and the manifest installers map points at install_generated_skill.py --host ....
  • install_claude_generated_skill.py stays because it has its own Windows command-shim logic.
  • INSTALL.md / INSTALL_EN.md and tests are updated to the new commands. The Hermes installer test file is renamed to tests/test_install_distilly.py and gains a per-host default path test.

Motivation

Repo cleanup. Every deleted file was checked with git grep against SKILL.md, tools, tests, package.json, bin and README links.

Testing

  • python -m unittest discover -s tests -p 'test_*.py' passed (110 tests)
  • python -m compileall tools/ passed
  • Manually tested: install_distilly.py --host {openclaw,hermes,codex} --force into a temp HOME; skill_writer.py --action create --install-codex-skill --install-openclaw-skill installs into both hosts; install_generated_skill.py --host codex; node bin/distilly.mjs --check-package and npm pack --dry-run are valid. ruff check tools/ reports fewer findings than on the base.

Checklist

  • Docs updated if behavior or usage changed (INSTALL.md / INSTALL_EN.md)
  • No secrets, tokens, or personal data committed
  • Tests updated

🤖 Generated with Claude Code

https://claude.ai/code/session_01QNauB8VvkxewzmhA2ZVT25


Generated by Claude Code

Delete files nothing references: the old PRD, the skill-type abstraction
design notes, the duplicate docs/lang/README_EN.md, four old social preview
images and eight WeChat group QR codes.

Merge install_{openclaw,hermes,codex}_skill.py into install_distilly.py
--host, and drop install_{openclaw,codex}_generated_skill.py in favor of
install_generated_skill.py --host. Docs, skill_writer, the manifest
installer map and tests now point at the merged scripts.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QNauB8VvkxewzmhA2ZVT25
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.

2 participants