Skip to content

Align image-processing skill with multi-format convert CLI#235

Merged
graysurf merged 1 commit intomainfrom
feat/image-processing-skill-refresh
Apr 1, 2026
Merged

Align image-processing skill with multi-format convert CLI#235
graysurf merged 1 commit intomainfrom
feat/image-processing-skill-refresh

Conversation

@graysurf
Copy link
Copy Markdown
Owner

@graysurf graysurf commented Apr 1, 2026

Align image-processing skill with multi-format convert CLI

Summary

Refresh the image-processing skill, reference docs, README index text, and regression tests so this repo matches the current multi-format image-processing CLI surface introduced upstream. The update removes stale SVG-only and ImageMagick-era guidance, aligns the skill with convert --in, png|webp|jpg outputs, and adds direct verification coverage for runtime behaviors that the skill now documents.

Changes

  • Update skills/tools/media/image-processing/SKILL.md and reference docs to match convert --in, accepted input formats, jpg output behavior, and current JSON/report artifacts.
  • Fix the repo README skill index entry so it no longer describes the old resize/crop/optimize ImageMagick workflow.
  • Expand skills/tools/media/image-processing/tests/test_tools_media_image_processing.py with raster-input, .jpeg, summary.json, --dry-run, removed-subcommand, and .svg output-extension coverage.

Testing

  • scripts/test.sh skills/tools/media/image-processing/tests/test_tools_media_image_processing.py (pass)
  • scripts/check.sh --all (pass)

Risk / Notes

  • This PR intentionally uses the current image-processing 0.6.8 CLI/runtime behavior as the source of truth for skill guidance.
  • deliver-feature-pr preflight flagged cross-domain scope because the change touches both the repo README and skills/tools/...; delivery continued with explicit user-confirmed scope.

- Update the image-processing skill and references for convert --in and png/webp/jpg outputs.
- Add README index text and response-template details for summary and report artifacts.
- Expand regression coverage for raster inputs, dry-run, summary.json, and svg output validation.
@graysurf graysurf marked this pull request as ready for review April 1, 2026 15:23
@graysurf graysurf merged commit fb3132f into main Apr 1, 2026
13 checks passed
@graysurf graysurf deleted the feat/image-processing-skill-refresh branch April 1, 2026 15:23
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