Skip to content

Simplify the transformation skill#7

Open
carlevison wants to merge 1 commit into
mainfrom
simplify_transformation_skill
Open

Simplify the transformation skill#7
carlevison wants to merge 1 commit into
mainfrom
simplify_transformation_skill

Conversation

@carlevison
Copy link
Copy Markdown
Contributor

Simplified cloudinary-transformations into a compact agent playbook that treats Cloudinary docs as the source of truth for exact syntax, current limitations, and costs.

Key changes:

  • Reduced the skill from large duplicated reference content to a concise SKILL.md.
  • Replaced eight bulky reference files with three focused playbooks:
    - agent-playbook.md
    - debugging-playbook.md
    - cost-and-caching.md
  • Preserved core agent guidance: requirement gathering, defaults, validation, debugging order, common footguns, and cost/caching heuristics.
  • Added guidance to present transformation parameters in SDK-style alphabetical order within components.
  • Validated the skill with quick_validate.py.

Copy link
Copy Markdown
Collaborator

@njb90 njb90 left a comment

Choose a reason for hiding this comment

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

One comment but otherwise it is definitely an improvement.

- Close conditionals with `if_end`.
- Prefer named transformations when the same complex chain is reused across assets.

## Footguns To Check
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not sure this is a term we should use.

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