punchline-mcp 1.2.0 → 1.4.0

package/README.md

@@ -74,7 +74,7 @@ the server twice under two names with two different keys, e.g. `punchline` (proj
 | `create_ticket` | project key | Create a ticket. `reporter_email` is required. Supports idempotent retries via `idempotency_key`, sent to the API as the `idempotencyKey` body field. Optional `custom_fields` (sent as `customFields`) is an array of `{ name, value }` pairs; `name` must match a custom-field definition configured on the project or the request is rejected with 400. |
 | `list_tickets` | project key | List tickets visible to the key (own tickets, or the whole project with `read:all`). Paginated. |
 | `get_ticket` | project key | Fetch one ticket by display id (e.g. `ACM-7`). |
-| `update_ticket` | project key | Change a ticket's workflow status, its priority, and/or customer-facing public summary. Setting `priority` records a **triage ruling** — confirming the current value counts as one — which a project enforcing triage requires before a ticket can close. Priority is applied before status, so one call can triage and close. A ticket created through this API arrives untriaged: the priority sent at creation is the reporter's proposal, not a ruling. *(1.2.0+)* |
+| `update_ticket` | project key | Change a ticket's workflow status, its priority, its labels, and/or customer-facing public summary. Setting `priority` records a **triage ruling** — confirming the current value counts as one — which a project enforcing triage requires before a ticket can close. Priority is applied before status, so one call can triage and close. A ticket created through this API arrives untriaged: the priority sent at creation is the reporter's proposal, not a ruling. *(1.2.0+)* `labels` is a **replace-set** applied before everything else: omit it to leave the labels alone, send the full list to change them, send `[]` to clear them — unknown names are created, exactly as at `create_ticket`; a blank name is refused by both tools rather than silently dropped, and surrounding whitespace is stripped server-side. *(1.3.0+)* `title` and `description` may also be patched: `title` is trimmed and rejected if blank, `description` **replaces only the reporter-authored body**, preserving the `**Reported by** … via …` provenance header the ticket was created with — send `""` to clear the body and keep just the header. Both are applied before `labels`, `priority` and `status`. *(1.4.0+)* |
 | `add_communication` | project key | Post a customer-facing communication (external comment) on a ticket. |
 | `link_tickets` | project key | Link two tickets. `relation` is one of `BLOCKS`, `BLOCKED_BY`, `DUPLICATES`, `DUPLICATE_OF`, `RELATES_TO`, read from the perspective of the ticket in `display_id` — the same vocabulary `get_ticket` returns in `linkedTickets`, so a relation round-trips. ⚠️ `close_as_duplicate: true` also **closes the `display_id` ticket and emails its reporter**; it requires `relation=DUPLICATES` and is not undone by `unlink_tickets`. |
 | `unlink_tickets` | project key | Remove a link. `relation` must match how the link reads from the ticket in `display_id`. |

package/dist/tools/tickets.js

@@ -10,6 +10,13 @@ export const TICKET_TOOL_NAMES = ['create_ticket', 'list_tickets', 'get_ticket',
  */
 export const CROSS_PROJECT_LINK_NOTE = ' Links to tickets in other projects are omitted from the response — a ticket may carry links'
     + ' that are not shown here.';
+/**
+ * A label name, on create and on update alike. Blank is refused rather than dropped: the API
+ * refuses it too, and a name the client silently discarded would leave the caller believing it
+ * had labelled the ticket. Declared once so the two tools cannot drift apart.
+ */
+const LABEL_NAME = z.string().max(100)
+    .refine((n) => n.trim().length > 0, { message: 'Label names must not be blank' });
 export const TICKETS_PATH = '/api/v1/external/tickets';
 export function ticketPath(displayId) {
     return `${TICKETS_PATH}/${encodeURIComponent(displayId)}`;
@@ -27,7 +34,7 @@ export function registerTicketTools(server, client) {
             description: z.string().max(50000).optional(),
             type: z.enum(TICKET_TYPES).optional().describe('Defaults to BUG'),
             priority: z.enum(TICKET_PRIORITIES).optional().describe('Defaults to MEDIUM'),
-            labels: z.array(z.string().max(100)).max(100).optional().describe('Label names; unknown labels are created'),
+            labels: z.array(LABEL_NAME).max(100).optional().describe('Label names; unknown labels are created'),
             reporter_name: z.string().max(500).optional(),
             page_url: z.string().max(2048).optional(),
             user_agent: z.string().max(512).optional(),
@@ -87,7 +94,7 @@ export function registerTicketTools(server, client) {
         },
     }, tool(async (a) => client.request('GET', ticketPath(a.display_id))));
     server.registerTool('update_ticket', {
-        description: 'Update a ticket: change its workflow status, its priority, and/or the customer-facing public summary. At least one field is required; an empty public_summary clears it. Setting priority records a triage ruling — confirming the existing value counts — which is what a project requiring triage before close demands. Priority is applied before status, so one call can triage and close.'
+        description: 'Update a ticket: change its workflow status, its priority, its labels, and/or the customer-facing public summary. At least one field is required; an empty public_summary clears it. Setting priority records a triage ruling — confirming the existing value counts — which is what a project requiring triage before close demands. Priority is applied before status, so one call can triage and close. Labels are applied first, and are a replace-set: omit the field to leave them alone, pass the full list to change them, pass [] to clear them. Title and description may also be patched; a description patch replaces only the reporter-authored body and keeps the "Reported by" header the ticket was created with.'
             + CROSS_PROJECT_LINK_NOTE,
         inputSchema: {
             display_id: z.string().min(1),
@@ -95,13 +102,30 @@ export function registerTicketTools(server, client) {
             public_summary: z.string().max(10000).optional(),
             priority: z.enum(TICKET_PRIORITIES).optional()
                 .describe('Records a triage ruling. Required before closing in a project that enforces triage.'),
+            labels: z.array(LABEL_NAME).max(100).optional()
+                .describe('Replaces the ticket\'s labels with exactly these names; unknown labels are created.'
+                + ' Omit to leave them untouched, [] to clear them.'),
+            title: z.string().min(1).max(500).optional()
+                .describe('Replaces the ticket title. Must not be blank.'),
+            description: z.string().max(50000).optional()
+                .describe('Replaces the reporter-authored body of the description. The'
+                + ' "Reported by … via …" header a ticket was created with is preserved;'
+                + ' pass "" to clear the body and keep only that header.'),
         },
     }, tool(async (a) => {
-        if (a.status === undefined && a.public_summary === undefined && a.priority === undefined) {
-            throw new Error('Provide status, priority and/or public_summary');
+        if (a.status === undefined && a.public_summary === undefined
+            && a.priority === undefined && a.labels === undefined
+            && a.title === undefined && a.description === undefined) {
+            throw new Error('Provide status, priority, labels, title, description and/or public_summary');
         }
+        // JSON.stringify drops undefined keys, which is what keeps "omitted" distinct from "[]" on
+        // the wire — the API reads an absent labels key as "leave them alone" (RUBE-46).
         return client.request('PATCH', ticketPath(a.display_id), {
-            body: { status: a.status, publicSummary: a.public_summary, priority: a.priority },
+            body: {
+                status: a.status, publicSummary: a.public_summary,
+                priority: a.priority, labels: a.labels,
+                title: a.title, description: a.description,
+            },
         });
     }));
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "punchline-mcp",
-  "version": "1.2.0",
+  "version": "1.4.0",
   "description": "MCP stdio server exposing the Punchline external API as tools",
   "type": "module",
   "license": "MIT",