secufusion-mcp 1.2.0 → 1.2.2

package/AGENTS.md

@@ -468,6 +468,7 @@ STEP 1: call run_pre_pr_checks_with_reviewer_agent(work_item_id: <id>)
             P3 MISSING_INDEX     — new query columns have migration index
             P4 KAFKA_SYNC        — no sync work inside @KafkaListener
             P5 EARLY_RETURN      — no early returns (single-exit rule)
+            (Note: If a mechanical check is a false positive, pass the `suppressions` parameter to the tool with `{ check, method, reason }` instead of ignoring it.)
         — Tier 2: AI File Reviewer (only after P1–P5 pass)
             Checks: hardcoded URLs, missing @Transactional(readOnly),
             missing @PreAuthorize, debug statements, exception swallowing,
@@ -727,37 +728,37 @@ Before Phase 0.7 plan presentation:
 **Audience:** Technical and Non-Technical stakeholders on the Azure DevOps board.
 **Format rules:**
 - Layman summary: NO class names, NO method names, NO technical terms. Pure English.
-- Technical deep-dive: BE specific. Exact class names, exact method names, exact file paths. Source from `files-touched.json`.
+- Execution Flow: BE specific. Exact class names, exact method names, exact file paths. Source from `files-touched.json`.
+- Sequential flow ONLY. Do not use before/after framing (e.g., avoid "not X directly," "exactly as before," "hardened after review," etc.). Just state what happens and in what order.
 - Tables must have content in every cell. If unknown → "not recorded".
 - Max 2 sentences in any single prose paragraph.
 - Max 600 words.
 
 ```markdown
 ## Layman Summary
-{layman_summary — 2-3 sentences max}
+{layman_summary — 2-3 sentences max. Sequential flow only.}
 
-## Technical Deep-Dive
+## Execution Flow
 {If auth/permission gating exists:}
-> Both endpoints gated by `{permission}` authority.
+> 1. Request gated by `{permission}` authority.
 
-### Scenarios Handled
-**Happy path:**
-{happy_path from scenarios.json — one sentence}
+{Describe the straight sequence of what happens, in order, step-by-step. E.g.:}
+2. **Trigger:** {How the flow starts}
+3. **Validation:** {What checks occur}
+4. **Processing:** {What data is transformed or queried}
+5. **Result:** {What is returned or persisted}
 
-**Sad paths handled:**
-{for each sad_path in scenarios.json:}
-- {sad_path description} → {how handled}
-
-**Security cases:**
+### Security & Error Handling
 {for each security_case in scenarios.json:}
 - {security concern} → {enforcement}
+{for each sad_path in scenarios.json:}
+- {sad_path description} → {how handled}
 
-### What did NOT change
+### Infrastructure Impact
 | Area | Status |
 |---|---|
 | DB migration | {✅ Not required / ⚠️ Created: V{n}__...sql} |
 | Kafka topics | {✅ No new topics / ⚠️ New topic: {name}} |
-| {other_service}-api | {✅ No changes / ⚠️ Changes required} |
 | API contract | {✅ No breaking changes / ⚠️ See decisions} |
 
 ---
@@ -767,7 +768,8 @@ Before Phase 0.7 plan presentation:
 ### PR Comment Template (`pr-comment.md`)
 **Audience:** Busy Developer Reviewers reading the PR description.
 **Format rules:**
-- Group changes by WHAT changed, not by file name (e.g., "New endpoint: GET /path", "Test coverage: N scenarios").
+- Group changes by the sequence of execution, not by file name.
+- Sequential flow ONLY. No before/after comparisons. Just state what the code does now.
 - ✅ for things NOT changing is equally important.
 - Guardrails section is mandatory.
 - Max 400 words. Condense changes table if longer.
@@ -779,13 +781,14 @@ Before Phase 0.7 plan presentation:
 
 ---
 ## Summary
-{2-3 sentences MAX. What changed and why.}
+{2-3 sentences MAX. What the code does. Sequential flow only.}
 
 ---
-## Changes
-| Area | What changed |
-|---|---|
-| {logical area} | {what changed in plain terms} |
+## Implementation Sequence
+1. {Step 1 of the new flow}
+2. {Step 2 of the new flow}
+3. {Step 3 of the new flow}
+4. {Step 4 of the new flow}
 
 ---
 ## Affected Service
@@ -793,16 +796,13 @@ Before Phase 0.7 plan presentation:
 
 ---
 ## Impact
-- ✅ No functional or behavioral changes {OR describe actual behavioral change}
+- ✅ No functional or behavioral changes {OR describe actual behavioral change sequentially}
 - ✅ No DB migration required {OR: ⚠️ Flyway migration V{n} included}
 - ✅ No Kafka topic changes {OR: ⚠️ New topic: {name}}
-- ✅ No API contract changes {OR: ⚠️ New endpoints: see Changes above}
+- ✅ No API contract changes {OR: ⚠️ New endpoints: see Implementation Sequence above}
 
 ---
-## Scenarios
-**Happy path:** {happy_path one line}
-
-**Sad paths handled:**
+## Error Handling
 - {what fails} → {how handled}
 
 ---

package/index.js

@@ -524,24 +524,32 @@ function generateNewTaskFolder(workItemId, title, tasksRoot) {
     let numericId = workItemId.replace(/\D/g, ''); // Extract just the digits
     if (!numericId)
         numericId = workItemId; // fallback
-    let taskTypeStr = "task";
+    let taskTypeStr = "Task";
     // Try to load the classification to get the true task_type
     const classifPath = path.join(tasksRoot, '..', 'classifications', `${workItemId}.json`);
     const classif = readJson(classifPath);
     if (classif && typeof classif.task_type === 'string') {
         const rawType = classif.task_type.toLowerCase();
         if (rawType === 'user_story')
-            taskTypeStr = 'story';
+            taskTypeStr = 'US';
+        else if (rawType === 'bug')
+            taskTypeStr = 'Bug';
         else
-            taskTypeStr = rawType;
+            taskTypeStr = rawType.charAt(0).toUpperCase() + rawType.slice(1);
     }
     else {
         // Fallback: guess from the workItemId string itself if passed as "BUG-1173"
         const prefixMatch = workItemId.match(/^([a-zA-Z]+)-/);
         if (prefixMatch) {
-            taskTypeStr = prefixMatch[1].toLowerCase();
-            if (taskTypeStr === 'wi')
-                taskTypeStr = 'task'; // Generic WI to task
+            const lowerPrefix = prefixMatch[1].toLowerCase();
+            if (lowerPrefix === 'wi')
+                taskTypeStr = 'Task';
+            else if (lowerPrefix === 'bug')
+                taskTypeStr = 'Bug';
+            else if (lowerPrefix === 'us')
+                taskTypeStr = 'US';
+            else
+                taskTypeStr = lowerPrefix.charAt(0).toUpperCase() + lowerPrefix.slice(1);
         }
     }
     return path.join(tasksRoot, `${taskTypeStr}-${numericId}-${slug}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "secufusion-mcp",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "type": "module",
   "description": "SecuFusion MCP server - developer workflow tooling with guardrails",
   "main": "index.js",