Skip to content

feat(asm): add PATCH and DELETE /tags/{tag_id} to complete the Tags resource#3

Open
dmchaledev wants to merge 1 commit into
mainfrom
claude/elegant-edison-kXvoT
Open

feat(asm): add PATCH and DELETE /tags/{tag_id} to complete the Tags resource#3
dmchaledev wants to merge 1 commit into
mainfrom
claude/elegant-edison-kXvoT

Conversation

@dmchaledev
Copy link
Copy Markdown
Contributor

Problem

The Tags resource was missing two of the four CRUD operations. Users could create and list tags, but there was no way to rename a tag, change its color, or delete it — making tag management effectively read-only after initial creation.

Changes

  • PATCH /tags/{tag_id} (updateTag) — update a tag's name and/or color; returns the updated Tag object
  • DELETE /tags/{tag_id} (deleteTag) — permanently removes the tag and strips it from all assets it was applied to; returns 204 No Content
  • components/parameters/TagId — reusable path parameter (mirrors the existing AssetId pattern)
  • Added description fields to listTags and createTag (were previously empty, which triggers Spectral warnings)

Why this is high-leverage

Tags are used to organize and filter assets across the entire ASM inventory. Without update/delete, any typo or color change requires deleting and re-creating the tag at the application level, and stale tags accumulate with no removal path via the API.

Test plan

  • CI Spectral lint passes on asm/openapi.yaml
  • PATCH /tags/{tag_id} response schema matches Tag component
  • DELETE /tags/{tag_id} returns 204 with no body
  • TagId pattern (^tag_[a-z0-9]{16}$) matches the example in Tag schema (tag_0001aabbccdd0001)

https://claude.ai/code/session_01J5oRgC1dr2enDMdNx8WxhD


Generated by Claude Code

Tags could be created and listed but never renamed, recolored, or deleted.
Adds updateTag and deleteTag operations, a reusable TagId path parameter,
and description fields on listTags and createTag.

https://claude.ai/code/session_01J5oRgC1dr2enDMdNx8WxhD
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