Closes#51
The root cause: all 10 harnesses used bare `open("w") + json.dump()` for
session saves. `open("w")` truncates the file before any lock can be
acquired, so concurrent writes silently corrupt or lose data.
Fix: add `_locked_save_json()` to each session.py that opens with "r+"
(no truncation), acquires fcntl.LOCK_EX, then truncates inside the lock.
Falls back gracefully on Windows where fcntl is unavailable.
Also:
- Update HARNESS.md with session file locking guidance (per maintainer
request on PR #52)
- Add concurrent write tests (4 threads × 50 writes) to verify the fix
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move skills/SKILL.md from the harness root into
cli_anything/<software>/skills/SKILL.md so it is installed alongside the
package via pip. Add package_data to all 11 setup.py files. Make
ReplSkin auto-detect the skill file relative to its own __file__
location, removing the need for an explicit skill_path argument.
Update HARNESS.md, cli-anything.md, plugin README, and main README.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Standardize skill file naming from <software>_SKILL.md to SKILL.md across
all 11 harnesses. Update skill_generator.py, HARNESS.md, repl_skin.py,
cli-anything.md, and plugin README.md to reference the new path. Add
SKILL.md generation section to main README.md and mark roadmap item done.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
## Changes
### 1. Display skill path in CLI header (repl_skin.py)
- Added `skill_path` parameter to ReplSkin class
- Banner now displays skill path when provided: `◇ Skill: skills/<software>_SKILL.md`
- This enables AI agents to discover where to read the skill documentation
### 2. Generate SKILL.md files for all 11 existing CLI harnesses
- anygen/agent-harness/skills/anygen_SKILL.md
- audacity/agent-harness/skills/audacity_SKILL.md
- blender/agent-harness/skills/blender_SKILL.md
- drawio/agent-harness/skills/drawio_SKILL.md
- gimp/agent-harness/skills/gimp_SKILL.md
- inkscape/agent-harness/skills/inkscape_SKILL.md
- kdenlive/agent-harness/skills/kdenlive_SKILL.md
- libreoffice/agent-harness/skills/libreoffice_SKILL.md
- obs-studio/agent-harness/skills/obs_studio_SKILL.md
- shotcut/agent-harness/skills/shotcut_SKILL.md
- zoom/agent-harness/skills/zoom_SKILL.md
### 3. Update HARNESS.md with skill path guidance
- Updated Phase 3 (REPL section) to show skill_path parameter usage
- Added "Display Skill Path in CLI Banner" section in Phase 6.5
### 4. Fix skill_generator.py syntax error
- Fixed list comprehension indentation on line 80-81
Addresses maintainer feedback from PR #47 comment:
- "displaying the skill path on the cli header so that agents can know where to read it"
- "generate skill.md files for all existing CLIs we have in the repo"
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit resolves all 8 Copilot review comments:
Template fixes (SKILL.md.template):
- Fix#8: Use YAML block scalars for frontmatter to safely handle quotes/newlines
- Fix#2: Add conditional wrapper for command groups section to prevent empty headers
Generator fixes (skill_generator.py):
- Fix#6: Remove unused imports (os, json) and unused decorator_name variable
- Fix#5: Add cli_anything_dir.exists() check before iterdir() with descriptive error
- Fix#7: Add _format_display_name() helper for proper software name formatting
- Fix#4: Change output_path default to use harness_path instead of CWD
- Fix#3: Update regex patterns to handle multi-line decorators and docstrings
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add skill_generator.py module for extracting CLI metadata
- Add templates/SKILL.md.template with Jinja2 placeholders
- Update HARNESS.md with Phase 6.5: SKILL.md Generation section
- Update cli-anything.md command to include skill generation step
- Update README.md to move SKILL.md generation to Features section
- SKILL.md follows skill-creator methodology with YAML frontmatter
- Generated files enable AI agents to discover and use CLIs
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add --depth option to command signature
- Document options and update examples
- Clarify version is now shown for all tools
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Previously --depth 2 only scanned exactly 2 levels deep, missing
tools at depth 1. Now it scans all depths from 0 to N inclusive.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add --depth option to limit how many directory levels to search
- depth=0 scans only current directory, depth=1 scans one level deep, etc.
- Default is unlimited depth (recursive ** glob)
- Helps control scan time in large directory structures
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The setup.py file is in agent-harness/, not cli_anything/. Fixed the
path calculation to correctly extract version info.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add new command to list all available CLI-Anything tools.
Supports both installed and generated CLIs with table/JSON output.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
