Repository navigation
feat(skills): add asset-management skill for the Assets DAM product - #24
Conversation
Advisory skill covering spaces and workspaces, folders, asset types and metadata, AI tags and alt text, duplicate management, versions, trash and restore, visual markups, image editing, and CMS integration. Registered in the router, README and manifest; cursor rules and codex tree regenerated. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Remove image editing section, restrict user-defined field types to single line text, number, select and group, and add localization and saved views guidance. Regenerated cursor rules and codex tree. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Release 0.2.0: native skills for every agent, Studio, Kickstart, Live Preview protocol
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Drop the generated cursor/codex trees, manifest.json and skills/CLAUDE.md router (removed on main). Register asset-management the new way: README row, docs/skills page and index entry. Add Use when trigger and the Environment Variables security subsection required by the validator.
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
timbenniks-contentstack
left a comment
There was a problem hiding this comment.
Nice work on this. The skill is well structured and the facts I spot-checked (14-day trash window, folder restore options, asset_fields[]) match the docs. A few things before we merge:
Must fix
POST /api/searchsection: as you flagged, it isn't in the public docs. It also has no host, path or auth, so an agent can't use it anyway. Let's drop it until the Assets API owner confirms it's customer-facing.- Field types: the skill lists single line text, number, select and group. About asset modeling says only Single Line Textbox, Number and Group are supported today, with Select planned. If Select has shipped, the docs need updating. Either way, I'd soften "do not suggest other types" to "currently supported types are…" so it doesn't go stale.
Should fix
- Missed docs pages:
docs/overview/introduction.mdstill says "six Contentstack product areas" and has no Assets row.docs/how-it-works/router.mdhas no row forasset-management. - Routing goes one way: this skill sends stack-library questions to
cms-assets, but thecms-assetsdescription ("images and other media files in Contentstack") will also catch DAM questions. Add a line there that sends Assets DAM questions (spaces, workspaces, duplicate sets) toasset-management. - Leftover: step 2 of the Workflow Summary still lists "editing" from the removed section.
- PR description: it still mentions
manifest.json,skills/CLAUDE.md, thecursor/rules/regeneration and a count of 22. Can you update it to match what's in the PR now?
…fix docs and routing - Rewrite the search section against the public Assets API search page (host, path, headers, search_field values, query operators) - Say 'currently supported' for user-defined field types - Add asset-management to docs/overview/introduction.md and docs/how-it-works/router.md; fix product count - Point cms-assets at asset-management for DAM questions - Remove leftover 'editing' from the workflow summary Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The stack asset library does not support user-defined fields. Say so in cms-assets and its docs page, and point asset-management at the Assets API. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
|
Thanks for the review. All points are addressed in 518945d and 0cbde4a, and the PR description is updated. Must fix
Should fix
|
|
Thanks, nearly there. Most of the review is addressed. A few things left: Search section (checked against the public docs)
Minor
Process
|
…vars; bump to 0.3.0 - spaces is a plain array of UIDs; list regional hosts and flag the .io/.com mismatch - Use authtoken/OAuth env vars, not a management token - Match 'currently supported' field-type wording in the docs page - Fix docs/README.md table padding - Bump version to 0.3.0 in all manifests and add a CHANGELOG entry Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
|
Thanks Tim. Everything is addressed in 6d6f041, and the PR now targets Search section
Minor
Process
|
- List the AWS NA host (am-api.contentstack.com), now consistent in the docs - Drop the undocumented operator list and $regex claim; use the documented field-value form - Flag user-defined field paths as something to test against the space - Clarify that the organization UID pairs with the authtoken Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
Summary
Adds an advisory, read-only
asset-managementskill for the standalone Contentstack Assets (DAM) product. It is distinct fromcms-assets, which covers the CMS stack asset library. Each skill now routes to the other where the scope differs.Covers: spaces and workspaces, folders, roles, asset types and metadata (user-defined fields: single line text, number, select, group), filters and saved views, localization, searching and filtering by user-defined fields via the Assets API search endpoint, AI tags and alt text, duplicate management, versions, trash and restore, visual markups, and CMS integration.
Changes
skills/asset-management/SKILL.mddocs/skills/asset-management.md, linked fromdocs/skills/index.mdREADME.md,docs/README.md,docs/overview/introduction.mdanddocs/how-it-works/router.md(24 skills, 7 products)cms-assets(skill and docs page): scoped to the stack asset library. DAM questions and user-defined field read/update requests now route toasset-management.main: the generatedcursor//codex/trees,manifest.jsonandskills/CLAUDE.mdrouter no longer exist, so none are touched here.node scripts/validate.mjspasses.Review notes
$and/$or,search_fieldvalues, headers and paging. It does not list the comparison operators, the$regexrestriction, thespaces[].workspacefield or user-defined field paths. Those come from the engineering team's source review, and the skill tells users to test their filters. Please confirm with the Assets API owner or trim them.spacesuses the documented plain array of UIDs, and the skill lists the regional hosts.CHANGELOG.mdentry. Targetsnext, per review.🤖 Generated with Claude Code