punchline-mcp 1.1.0 → 1.3.0

package/README.md

@@ -71,10 +71,10 @@ the server twice under two names with two different keys, e.g. `punchline` (proj
 
 | Tool | Available with | Description |
 |---|---|---|
-| `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. |
+| `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 and/or customer-facing public summary. |
+| `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+)* |
 | `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)}`;
@@ -17,6 +24,9 @@ export function ticketPath(displayId) {
 export function registerTicketTools(server, client) {
     server.registerTool('create_ticket', {
         description: 'Create a ticket in the project bound to the API key. Supports idempotent retries via idempotency_key.'
+            + ' custom_fields lets you stamp name-keyed values that other project tooling (e.g. outbound webhooks) reads back'
+            + ' later; each name must match a custom-field definition already configured on the project — an unknown name is'
+            + ' rejected with a 400 error rather than silently ignored.'
             + CROSS_PROJECT_LINK_NOTE,
         inputSchema: {
             title: z.string().min(1).max(500).describe('Ticket title'),
@@ -24,13 +34,18 @@ 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(),
             app_version: z.string().max(100).optional(),
             idempotency_key: z.string().max(255).optional()
                 .describe('Same key on the same API key returns the original ticket instead of creating a duplicate'),
+            custom_fields: z.array(z.object({
+                name: z.string().min(1).max(255).describe('Must match a custom-field name configured on the project'),
+                value: z.string().max(4096),
+            })).max(50).optional()
+                .describe('Name-keyed custom field values, e.g. [{ name: "rubedyne_user_id", value: "3172" }]. An unknown name is rejected with 400.'),
         },
     }, tool(async (a) => client.request('POST', TICKETS_PATH, {
         body: {
@@ -45,6 +60,7 @@ export function registerTicketTools(server, client) {
             userAgent: a.user_agent,
             appVersion: a.app_version,
             idempotencyKey: a.idempotency_key,
+            customFields: a.custom_fields,
         },
     })));
     server.registerTool('list_tickets', {
@@ -78,19 +94,30 @@ 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 and/or the customer-facing public summary. At least one field is required; an empty public_summary clears it.'
+        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.'
             + CROSS_PROJECT_LINK_NOTE,
         inputSchema: {
             display_id: z.string().min(1),
             status: z.string().max(100).optional(),
             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.'),
         },
     }, tool(async (a) => {
-        if (a.status === undefined && a.public_summary === undefined) {
-            throw new Error('Provide status and/or public_summary');
+        if (a.status === undefined && a.public_summary === undefined
+            && a.priority === undefined && a.labels === undefined) {
+            throw new Error('Provide status, priority, labels 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 },
+            body: {
+                status: a.status, publicSummary: a.public_summary,
+                priority: a.priority, labels: a.labels,
+            },
         });
     }));
 }

package/package.json

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