Tracking Issue

Part of RocketChat/Rocket.Chat-Open-API#150

Endpoints migrated:

• livechat/tags (GET)
• livechat/tags/:tagId (GET)

Changes

packages/rest-typings/src/v1/omnichannel.ts

• Fixed text to optional in LivechatTagsListProps type & LivechatTagsListSchema — the existing schema marked text as required, but the actual endpoint and 8+ integration tests treat it as optional. Without this fix, enabling AJV query validation would break all tests that don't pass text.
• Added response validators:
  • GETLivechatTagsSuccessResponse — paginated response with {tags, count, offset, total, success} and additionalProperties: false
  • GETLivechatTagByIdSuccessResponse — single tag response with {_id, name, description, numDepartments, departments, success} and additionalProperties: false
• Deleted manual typings for /v1/livechat/tags and /v1/livechat/tags/:tagId from OmnichannelEndpoints (types are now inferred via ExtractRoutesFromAPI)

apps/meteor/ee/app/livechat-enterprise/server/api/tags.ts

• Converted both legacy API.v1.addRoute() endpoints to chained .get() calls
• Merged them into the existing livechatTagsEndpoints chain alongside already-migrated tags.save and tags.delete
• Added query: isLivechatTagsListProps for input validation on the list endpoint
• Added response validators (200, 401, 403, 404) on both endpoints
• Business logic is completely unchanged

Endpoint chain structure (after migration):

const livechatTagsEndpoints = API.v1
  .get('livechat/tags', ...)         // ← migrated in this PR
  .get('livechat/tags/:tagId', ...)  // ← migrated in this PR
  .post('livechat/tags.save', ...)   // already migrated
  .post('livechat/tags.delete', ...) // already migrated

SDK Check

• ✅ livechat/tags is NOT consumed by @rocket.chat/ddp-client — safe to delete manual typings

Verification

✅ Build

yarn build — 67/67 packages compiled successfully, 0 errors.

✅ Swagger UI

GET /livechat/tags — 200 OK:

✅ Integration Tests

yarn testapi:livechat — 181 passing, 81 pending (EE), 1 pre-existing failure (unrelated 02-appearance.ts test).

0 failures from this PR's changes.

Checklist

• Chained .get() used
• AJV query validation added (isLivechatTagsListProps)
• Response validators added with additionalProperties: false
• Types inferred via ExtractRoutesFromAPI
• Manual typings deleted from rest-typings
• Server boots without MissingRefError
• Swagger UI schemas render correctly (200, 401, 403, 404)
• yarn testapi:livechat passes

Summary by CodeRabbit

• Refactor

  • Restructured Livechat tags API endpoint registration and handler naming.
  • Added explicit response mappings for 200/401/403/404 status codes.

• Improvements

  • Made the tag search text parameter optional to allow unfiltered tag retrieval.
  • Added typed success response schemas and stronger response validation.

• Chores

  • Included a changeset entry marking a minor release for the related packages.