@aaronsb/jira-cloud-mcp 0.5.5 → 0.5.6
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -4,11 +4,12 @@ import { analysisNextSteps } from '../utils/next-steps.js';
|
|
|
4
4
|
import { normalizeArgs } from '../utils/normalize-args.js';
|
|
5
5
|
const ALL_METRICS = ['points', 'time', 'schedule', 'cycle', 'distribution'];
|
|
6
6
|
const VALID_GROUP_BY = ['project', 'assignee', 'priority', 'issuetype'];
|
|
7
|
-
const
|
|
7
|
+
const MAX_ISSUES_HARD = 500; // absolute ceiling for detail metrics — beyond this, context explodes
|
|
8
|
+
const MAX_ISSUES_DEFAULT = 100;
|
|
8
9
|
const CUBE_SAMPLE_PCT = 0.2; // 20% of total issues
|
|
9
10
|
const CUBE_SAMPLE_MIN = 50; // floor — enough for rare dimension values
|
|
10
11
|
const CUBE_SAMPLE_MAX = 500; // ceiling — proven fast with lean search
|
|
11
|
-
const
|
|
12
|
+
const DEFAULT_GROUP_LIMIT = 20;
|
|
12
13
|
const MAX_COUNT_QUERIES = 150; // ADR-206 budget: max count API calls per execution
|
|
13
14
|
const STANDARD_MEASURES = 6; // total, open, overdue, high+, created_7d, resolved_7d
|
|
14
15
|
// ── Helpers ────────────────────────────────────────────────────────────
|
|
@@ -58,11 +59,14 @@ function countBy(items, keyFn) {
|
|
|
58
59
|
function sumBy(items, valueFn) {
|
|
59
60
|
return items.reduce((sum, item) => sum + (valueFn(item) || 0), 0);
|
|
60
61
|
}
|
|
61
|
-
function mapToString(map, separator = ' | ') {
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
62
|
+
function mapToString(map, separator = ' | ', limit) {
|
|
63
|
+
const sorted = [...map.entries()].sort((a, b) => b[1] - a[1]);
|
|
64
|
+
const capped = limit ? sorted.slice(0, limit) : sorted;
|
|
65
|
+
const result = capped.map(([k, v]) => `${k}: ${v}`).join(separator);
|
|
66
|
+
if (limit && sorted.length > limit) {
|
|
67
|
+
return `${result} | (+${sorted.length - limit} more — use groupLimit to see all)`;
|
|
68
|
+
}
|
|
69
|
+
return result;
|
|
66
70
|
}
|
|
67
71
|
function median(values) {
|
|
68
72
|
if (values.length === 0)
|
|
@@ -255,16 +259,16 @@ export function renderCycle(issues, now) {
|
|
|
255
259
|
}
|
|
256
260
|
return lines.join('\n');
|
|
257
261
|
}
|
|
258
|
-
export function renderDistribution(issues) {
|
|
262
|
+
export function renderDistribution(issues, groupLimit = DEFAULT_GROUP_LIMIT) {
|
|
259
263
|
const lines = ['## Distribution', ''];
|
|
260
264
|
const byStatus = countBy(issues, i => i.status);
|
|
261
|
-
lines.push(`**By status:** ${mapToString(byStatus)}`);
|
|
265
|
+
lines.push(`**By status:** ${mapToString(byStatus, ' | ', groupLimit)}`);
|
|
262
266
|
const byAssignee = countBy(issues, i => i.assignee || 'Unassigned');
|
|
263
|
-
lines.push(`**By assignee:** ${mapToString(byAssignee)}`);
|
|
267
|
+
lines.push(`**By assignee:** ${mapToString(byAssignee, ' | ', groupLimit)}`);
|
|
264
268
|
const byPriority = countBy(issues, i => i.priority || 'None');
|
|
265
|
-
lines.push(`**By priority:** ${mapToString(byPriority)}`);
|
|
269
|
+
lines.push(`**By priority:** ${mapToString(byPriority, ' | ', groupLimit)}`);
|
|
266
270
|
const byType = countBy(issues, i => i.issueType || 'Unknown');
|
|
267
|
-
lines.push(`**By type:** ${mapToString(byType)}`);
|
|
271
|
+
lines.push(`**By type:** ${mapToString(byType, ' | ', groupLimit)}`);
|
|
268
272
|
return lines.join('\n');
|
|
269
273
|
}
|
|
270
274
|
/** Extract project keys from JQL like "project in (AA, GC, GD)" or "project = AA" */
|
|
@@ -408,7 +412,7 @@ function maxGroupsForBudget(implicitCount) {
|
|
|
408
412
|
const queriesPerGroup = STANDARD_MEASURES + implicitCount;
|
|
409
413
|
return Math.floor(MAX_COUNT_QUERIES / queriesPerGroup);
|
|
410
414
|
}
|
|
411
|
-
async function handleSummary(jiraClient, jql, groupBy, compute) {
|
|
415
|
+
async function handleSummary(jiraClient, jql, groupBy, compute, groupLimit = DEFAULT_GROUP_LIMIT) {
|
|
412
416
|
const lines = [];
|
|
413
417
|
lines.push(`# Summary: ${jql}`);
|
|
414
418
|
lines.push(`As of ${formatDateShort(new Date())} — counts are exact (no sampling cap)`);
|
|
@@ -416,21 +420,26 @@ async function handleSummary(jiraClient, jql, groupBy, compute) {
|
|
|
416
420
|
const implicitDefs = buildImplicitMeasures(jiraClient.customFieldIds);
|
|
417
421
|
const neededImplicits = compute ? detectImplicitMeasures(compute, implicitDefs) : [];
|
|
418
422
|
const groupBudget = maxGroupsForBudget(neededImplicits.length);
|
|
423
|
+
// Effective group cap: user's preference vs API query budget (whichever is smaller)
|
|
424
|
+
const effectiveGroupCap = Math.min(groupLimit, groupBudget);
|
|
419
425
|
if (groupBy === 'project') {
|
|
420
426
|
let keys = extractProjectKeys(jql);
|
|
421
427
|
if (keys.length === 0) {
|
|
422
428
|
throw new McpError(ErrorCode.InvalidParams, 'groupBy "project" requires project keys in JQL (e.g., project in (AA, GC))');
|
|
423
429
|
}
|
|
424
|
-
const capped = keys.length >
|
|
430
|
+
const capped = keys.length > effectiveGroupCap;
|
|
425
431
|
if (capped)
|
|
426
|
-
keys = keys.slice(0,
|
|
432
|
+
keys = keys.slice(0, effectiveGroupCap);
|
|
427
433
|
const remaining = removeProjectClause(jql);
|
|
428
434
|
const rows = await batchParallel(keys.map(k => () => buildCountRow(jiraClient, k, remaining ? `project = ${k} AND (${remaining})` : `project = ${k}`, neededImplicits, implicitDefs)), ROW_BATCH_SIZE);
|
|
429
435
|
rows.sort((a, b) => b.unresolved - a.unresolved);
|
|
430
436
|
lines.push('');
|
|
431
437
|
lines.push(renderSummaryTable(rows, compute));
|
|
432
438
|
if (capped) {
|
|
433
|
-
|
|
439
|
+
const reason = effectiveGroupCap < groupBudget
|
|
440
|
+
? `groupLimit=${effectiveGroupCap} — increase groupLimit to see more`
|
|
441
|
+
: `${MAX_COUNT_QUERIES}-query budget`;
|
|
442
|
+
lines.push(`*Capped at ${effectiveGroupCap} groups (${reason})*`);
|
|
434
443
|
}
|
|
435
444
|
}
|
|
436
445
|
else if (groupBy) {
|
|
@@ -439,23 +448,29 @@ async function handleSummary(jiraClient, jql, groupBy, compute) {
|
|
|
439
448
|
if (issues.length === 0) {
|
|
440
449
|
throw new McpError(ErrorCode.InvalidParams, `No issues matched JQL — cannot discover ${groupBy} values`);
|
|
441
450
|
}
|
|
442
|
-
const dims = extractDimensions(issues);
|
|
451
|
+
const dims = extractDimensions(issues, groupLimit);
|
|
443
452
|
const dim = dims.find(d => d.name === groupBy);
|
|
444
453
|
if (!dim || dim.values.length === 0) {
|
|
445
454
|
throw new McpError(ErrorCode.InvalidParams, `No ${groupBy} values found in sampled issues`);
|
|
446
455
|
}
|
|
447
|
-
// Cap groups to
|
|
448
|
-
const cappedValues = dim.values.slice(0,
|
|
456
|
+
// Cap groups to effective limit
|
|
457
|
+
const cappedValues = dim.values.slice(0, effectiveGroupCap);
|
|
449
458
|
const jqlClause = groupByJqlClause(groupBy, cappedValues);
|
|
450
459
|
const rows = await batchParallel(cappedValues.map((value, idx) => () => buildCountRow(jiraClient, value, `(${jql}) AND ${jqlClause[idx]}`, neededImplicits, implicitDefs)), ROW_BATCH_SIZE);
|
|
451
460
|
rows.sort((a, b) => b.unresolved - a.unresolved);
|
|
452
461
|
lines.push('');
|
|
453
462
|
lines.push(renderSummaryTable(rows, compute));
|
|
454
463
|
if (dim.count > cappedValues.length) {
|
|
455
|
-
const
|
|
456
|
-
|
|
457
|
-
|
|
458
|
-
|
|
464
|
+
const reasons = [];
|
|
465
|
+
if (cappedValues.length < dim.values.length) {
|
|
466
|
+
reasons.push(effectiveGroupCap < groupBudget
|
|
467
|
+
? `groupLimit=${effectiveGroupCap} — increase groupLimit to see more`
|
|
468
|
+
: `${MAX_COUNT_QUERIES}-query budget`);
|
|
469
|
+
}
|
|
470
|
+
else {
|
|
471
|
+
reasons.push(`from ${issues.length}-issue sample`);
|
|
472
|
+
}
|
|
473
|
+
lines.push(`*Showing top ${cappedValues.length} of ${dim.count} ${groupBy} values (${reasons.join(', ')})*`);
|
|
459
474
|
}
|
|
460
475
|
}
|
|
461
476
|
else {
|
|
@@ -472,7 +487,7 @@ function detectImplicitMeasures(compute, implicitMeasureDefs) {
|
|
|
472
487
|
return Object.keys(implicitMeasureDefs).filter(name => refs.has(name));
|
|
473
488
|
}
|
|
474
489
|
/** Extract distinct dimension values from sampled issues */
|
|
475
|
-
export function extractDimensions(issues) {
|
|
490
|
+
export function extractDimensions(issues, groupLimit = DEFAULT_GROUP_LIMIT) {
|
|
476
491
|
const dims = [
|
|
477
492
|
{ name: 'project', extractor: i => i.key.split('-')[0] },
|
|
478
493
|
{ name: 'status', extractor: i => i.status },
|
|
@@ -486,10 +501,10 @@ export function extractDimensions(issues) {
|
|
|
486
501
|
const val = extractor(issue);
|
|
487
502
|
counts.set(val, (counts.get(val) || 0) + 1);
|
|
488
503
|
}
|
|
489
|
-
// Sort by count descending, cap at
|
|
504
|
+
// Sort by count descending, cap at groupLimit
|
|
490
505
|
const sorted = [...counts.entries()]
|
|
491
506
|
.sort((a, b) => b[1] - a[1])
|
|
492
|
-
.slice(0,
|
|
507
|
+
.slice(0, groupLimit);
|
|
493
508
|
return {
|
|
494
509
|
name,
|
|
495
510
|
values: sorted.map(([v]) => v),
|
|
@@ -521,7 +536,7 @@ export function renderCubeSetup(jql, sampleSize, dimensions) {
|
|
|
521
536
|
lines.push('');
|
|
522
537
|
lines.push(`## Suggested Cubes (budget: ${MAX_COUNT_QUERIES} queries)`);
|
|
523
538
|
for (const dim of dimensions) {
|
|
524
|
-
const groups = Math.min(dim.count,
|
|
539
|
+
const groups = Math.min(dim.count, DEFAULT_GROUP_LIMIT);
|
|
525
540
|
const queries = groups * STANDARD_MEASURES;
|
|
526
541
|
const estSeconds = Math.max(1, Math.round(queries / 12)); // ~12 parallel queries/sec
|
|
527
542
|
const withinBudget = queries <= MAX_COUNT_QUERIES;
|
|
@@ -596,6 +611,9 @@ export async function handleAnalysisRequest(jiraClient, request) {
|
|
|
596
611
|
if (args.compute && Array.isArray(args.compute) && args.compute.length > 0) {
|
|
597
612
|
compute = parseComputeList(args.compute);
|
|
598
613
|
}
|
|
614
|
+
// Parse groupLimit — soft cap on group/dimension values (no hard cap for summary metrics)
|
|
615
|
+
const rawGroupLimit = Number(args.groupLimit);
|
|
616
|
+
const groupLimit = rawGroupLimit > 0 ? rawGroupLimit : DEFAULT_GROUP_LIMIT;
|
|
599
617
|
// Cube setup — discover dimensions from sample, no issue fetching
|
|
600
618
|
if (hasCubeSetup) {
|
|
601
619
|
const cubeText = await handleCubeSetup(jiraClient, jql);
|
|
@@ -609,7 +627,7 @@ export async function handleAnalysisRequest(jiraClient, request) {
|
|
|
609
627
|
}
|
|
610
628
|
// If only summary requested, skip issue fetching entirely
|
|
611
629
|
if (hasSummary && fetchMetrics.length === 0) {
|
|
612
|
-
const summaryText = await handleSummary(jiraClient, jql, groupBy, compute);
|
|
630
|
+
const summaryText = await handleSummary(jiraClient, jql, groupBy, compute, groupLimit);
|
|
613
631
|
const nextSteps = analysisNextSteps(jql, []);
|
|
614
632
|
return {
|
|
615
633
|
content: [{
|
|
@@ -618,8 +636,7 @@ export async function handleAnalysisRequest(jiraClient, request) {
|
|
|
618
636
|
}],
|
|
619
637
|
};
|
|
620
638
|
}
|
|
621
|
-
const
|
|
622
|
-
const maxResults = Math.min(Number(args.maxResults) || DEFAULT_MAX, MAX_ISSUES);
|
|
639
|
+
const maxResults = Math.min(Number(args.maxResults) || MAX_ISSUES_DEFAULT, MAX_ISSUES_HARD);
|
|
623
640
|
// Fetch issues using cursor-based pagination (50 per page, Jira enhanced search API)
|
|
624
641
|
const allIssues = [];
|
|
625
642
|
const seen = new Set();
|
|
@@ -658,7 +675,7 @@ export async function handleAnalysisRequest(jiraClient, request) {
|
|
|
658
675
|
const lines = [];
|
|
659
676
|
// Summary first if requested alongside other metrics
|
|
660
677
|
if (hasSummary) {
|
|
661
|
-
const summaryText = await handleSummary(jiraClient, jql, groupBy, compute);
|
|
678
|
+
const summaryText = await handleSummary(jiraClient, jql, groupBy, compute, groupLimit);
|
|
662
679
|
lines.push(summaryText);
|
|
663
680
|
}
|
|
664
681
|
// Header for fetch-based metrics
|
|
@@ -668,7 +685,7 @@ export async function handleAnalysisRequest(jiraClient, request) {
|
|
|
668
685
|
lines.push(`# Detail: ${jql}`);
|
|
669
686
|
lines.push(`Analyzed ${allIssues.length} issues (as of ${formatDateShort(now)})`);
|
|
670
687
|
if (truncated) {
|
|
671
|
-
lines.push(`*Results capped at ${maxResults} issues —
|
|
688
|
+
lines.push(`*Results capped at ${maxResults} issues — distributions below are approximate. For exact counts, use metrics: ["summary"] with groupBy instead.*`);
|
|
672
689
|
}
|
|
673
690
|
// Render requested metrics
|
|
674
691
|
const renderers = {
|
|
@@ -676,7 +693,7 @@ export async function handleAnalysisRequest(jiraClient, request) {
|
|
|
676
693
|
time: () => renderTime(allIssues),
|
|
677
694
|
schedule: () => renderSchedule(allIssues, now),
|
|
678
695
|
cycle: () => renderCycle(allIssues, now),
|
|
679
|
-
distribution: () => renderDistribution(allIssues),
|
|
696
|
+
distribution: () => renderDistribution(allIssues, groupLimit),
|
|
680
697
|
};
|
|
681
698
|
for (const metric of fetchMetrics) {
|
|
682
699
|
lines.push('');
|
|
@@ -326,7 +326,7 @@ export const toolSchemas = {
|
|
|
326
326
|
},
|
|
327
327
|
analyze_jira_issues: {
|
|
328
328
|
name: 'analyze_jira_issues',
|
|
329
|
-
description: 'Compute project metrics over issues selected by JQL.
|
|
329
|
+
description: 'Compute project metrics over issues selected by JQL. For counting and breakdown questions ("how many by status/assignee/priority"), use metrics: ["summary"] with groupBy — this gives exact counts with no issue cap. Use detail metrics (points, time, schedule, cycle, distribution) only when you need per-issue analysis; these are capped at maxResults issues. Always prefer this tool over manage_jira_filter or manage_jira_project for quantitative questions. Read jira://analysis/recipes for composition patterns.',
|
|
330
330
|
inputSchema: {
|
|
331
331
|
type: 'object',
|
|
332
332
|
properties: {
|
|
@@ -340,12 +340,12 @@ export const toolSchemas = {
|
|
|
340
340
|
type: 'string',
|
|
341
341
|
enum: ['summary', 'points', 'time', 'schedule', 'cycle', 'distribution', 'cube_setup'],
|
|
342
342
|
},
|
|
343
|
-
description: 'Which metric groups to compute. summary = exact
|
|
343
|
+
description: 'Which metric groups to compute. summary = exact counts via count API (no issue cap, fastest) — use with groupBy for "how many by assignee/status/priority" questions. distribution = approximate counts from fetched issues (capped by maxResults — use summary + groupBy instead when you need exact counts). cube_setup = discover dimensions before cube queries. points = earned value/SPI. time = effort estimates. schedule = overdue/risk. cycle = lead time/throughput. Default: all detail metrics. For counting/breakdown questions, always prefer summary + groupBy over distribution.',
|
|
344
344
|
},
|
|
345
345
|
groupBy: {
|
|
346
346
|
type: 'string',
|
|
347
347
|
enum: ['project', 'assignee', 'priority', 'issuetype'],
|
|
348
|
-
description: 'Split
|
|
348
|
+
description: 'Split counts by this dimension — produces a breakdown table. Use with metrics: ["summary"] for exact counts. This is the correct approach for "how many issues per assignee/priority/type" questions. "project" produces a per-project comparison.',
|
|
349
349
|
},
|
|
350
350
|
compute: {
|
|
351
351
|
type: 'array',
|
|
@@ -353,6 +353,11 @@ export const toolSchemas = {
|
|
|
353
353
|
description: 'Computed columns for cube execute. Each entry: "name = expr". Arithmetic (+,-,*,/), comparisons (>,<,>=,<=,==,!=). Column refs: total, open, overdue, high, created_7d, resolved_7d. Implicit measures resolved lazily: bugs, unassigned, no_due_date, no_estimate, no_start_date, no_labels, blocked, stale (untouched 60d+), stale_status (stuck in status 30d+), backlog_rot (undated+unassigned+untouched 60d+). Max 5 expressions. Example: ["bug_pct = bugs / total * 100", "rot_pct = backlog_rot / open * 100"].',
|
|
354
354
|
maxItems: 5,
|
|
355
355
|
},
|
|
356
|
+
groupLimit: {
|
|
357
|
+
type: 'integer',
|
|
358
|
+
description: 'Max groups/dimension values to show (default 20). Applies to summary groupBy rows and distribution breakdowns. No hard cap for summary metrics — increase freely for full visibility. For detail metrics the issue fetch is separately capped by maxResults.',
|
|
359
|
+
default: 20,
|
|
360
|
+
},
|
|
356
361
|
maxResults: {
|
|
357
362
|
type: 'integer',
|
|
358
363
|
description: 'Max issues to fetch for detail metrics (default 100, max 500). Does not apply to summary (which uses count API).',
|
|
@@ -132,7 +132,7 @@ export function analysisNextSteps(jql, issueKeys, truncated = false) {
|
|
|
132
132
|
}
|
|
133
133
|
steps.push({ description: 'Discover dimensions for cube analysis', tool: 'analyze_jira_issues', example: { jql, metrics: ['cube_setup'] } }, { description: 'Add computed columns', tool: 'analyze_jira_issues', example: { jql, metrics: ['summary'], groupBy: 'project', compute: ['bug_pct = bugs / total * 100'] } }, { description: 'Narrow the analysis with refined JQL', tool: 'analyze_jira_issues', example: { jql: `${jql} AND priority = High` } }, { description: 'View the full issue list', tool: 'manage_jira_filter', example: { operation: 'execute_jql', jql } });
|
|
134
134
|
if (truncated) {
|
|
135
|
-
steps.push({ description: '
|
|
135
|
+
steps.push({ description: 'Distribution counts above are approximate (issue cap hit). For exact breakdowns use summary + groupBy', tool: 'analyze_jira_issues', example: { jql, metrics: ['summary'], groupBy: 'assignee' } }, { description: 'Or narrow JQL for precise detail metrics', tool: 'analyze_jira_issues', example: { jql: `${jql} AND assignee = currentUser()`, metrics: ['cycle'] } });
|
|
136
136
|
}
|
|
137
137
|
return formatSteps(steps) + '\n- Read `jira://analysis/recipes` for data cube patterns and compute DSL examples';
|
|
138
138
|
}
|
package/package.json
CHANGED