@we-scrum/cli 1.0.4 → 1.0.5

package/src/mcp-server/tools/get-next-task.ts

@@ -1,150 +1,26 @@
-import { WeScrumHelper } from '@helpers';
-import { _compact } from '@my-devkit/core';
-import { Operation, ProgressStatus } from '@we-scrum/enums';
-import {
-    ProjectStorySectionModelEnumerationChange,
-    ProjectStorySectionModelObjectChange,
-    ProjectStorySectionModelRelationChange,
-    ProjectStorySectionModelRouteChange,
-    ProjectStorySectionModelTask,
-} from '@we-scrum/models';
+import { renderTemplate, WeScrumHelper } from '@helpers';
+import { ProgressStatus } from '@we-scrum/enums';
+import { StorySectionHelper, stringifyAnalysisDslChange } from '@we-scrum/utils';
 
-export async function getNextTask(identificationNumber: string): Promise<Record<string, string>> {
-    const nextTask = await WeScrumHelper.getNextChange(identificationNumber);
+export async function getNextTask(identificationNumber: string): Promise<string> {
+    const nextChange = await WeScrumHelper.getNextChange(identificationNumber);
 
-    if (!nextTask) {
-        return { message: `All tasks for story "${identificationNumber}" are complete.` };
+    if (!nextChange) {
+        return `## Story ${identificationNumber}\n\nAll tasks are complete.`;
     }
 
-    const isTodo = nextTask.progress.status === ProgressStatus.ToDo;
-    const generalInstructions = _compact([
-        'Implement the task described in the fields below.',
-        'Refer to `implementationGuidelines` for coding conventions and project-specific instructions.',
-        isTodo ? 'Before making any changes, run `start_task` to mark the task as in progress.' : null,
-        'If anything is unclear, stop and ask before making any changes.',
-        'Once all changes are done, commit your code and run `complete_task`.',
-        'After completing a task, immediately run `get_next_task` again and continue until all tasks are complete.',
-    ]).join('\n');
+    const analysisChange = StorySectionHelper.mapSingleChange(nextChange);
+    const isTodo = nextChange.progress.status === ProgressStatus.ToDo;
+    const developmentPolicy = await WeScrumHelper.getChangeDevelopmentPolicy(nextChange);
 
-    const developmentPolicy = await WeScrumHelper.getChangeDevelopmentPolicy(nextTask);
-    const implementationGuidelines = developmentPolicy?.codeGuidelines ?? 'No guidelines found for this task... :-(';
+    const { guidelinesPath, unitTestsPath } = WeScrumHelper.getDevelopmentPolicyPaths(developmentPolicy);
 
-    const unitTestInstructions = 'Run get_unit_test_guidelines to get instructions to generate unit tests for this task';
-    const unitTests = developmentPolicy?.areUnitTestsMandatory ? unitTestInstructions : 'No unit test needed.';
-
-    if (nextTask instanceof ProjectStorySectionModelEnumerationChange) {
-        const propertyChanges = nextTask.propertyChanges
-            .filter((pc) => !!pc.operation)
-            .map((pc) => `${pc.operation} property ${pc.propertyName} (= '${pc.propertyValue}')`)
-            .sort()
-            .join('\n');
-
-        return {
-            generalInstructions,
-            taskType: `${nextTask.operation} enumeration`,
-            enumerationName: nextTask.enumerationName,
-            propertyChanges: propertyChanges || 'No change on properties',
-            taskDescription: nextTask.description ?? undefined,
-            implementationGuidelines,
-            unitTests,
-        };
-    }
-    if (nextTask instanceof ProjectStorySectionModelObjectChange) {
-        const propertyMap = new Map(nextTask.propertyChanges.map((pc) => [pc.objectPropertyId, pc]));
-
-        const buildPath = (pc: (typeof nextTask.propertyChanges)[0]): string => {
-            const name = pc.isArray ? `${pc.propertyName}[]` : pc.propertyName;
-            if (!pc.objectPropertyParentId) return name;
-            const parent = propertyMap.get(pc.objectPropertyParentId);
-            return parent ? `${buildPath(parent)}.${name}` : name;
-        };
-
-        const changedProperties = nextTask.propertyChanges.filter((pc) => !!pc.operation);
-        const parentIdsWithChangedChildren = new Set(changedProperties.map((pc) => pc.objectPropertyParentId).filter(Boolean));
-
-        const propertyChanges = changedProperties
-            .filter((pc) => !(pc.operation === Operation.Create && parentIdsWithChangedChildren.has(pc.objectPropertyId)))
-            .map((pc) => {
-                const enumInfo = pc.enumerationName ? ` (enum: ${pc.enumerationName})` : '';
-                return `${pc.operation} property ${buildPath(pc)}: ${pc.propertyType}${enumInfo}`;
-            })
-            .sort()
-            .join('\n');
-
-        return {
-            generalInstructions,
-            taskType: `${nextTask.operation} object`,
-            objectName: nextTask.objectName,
-            renamedFrom: nextTask.initialObjectName
-                ? `Object renamed from "${nextTask.initialObjectName}" to "${nextTask.objectName}"`
-                : undefined,
-            propertyChanges: propertyChanges || 'No change on properties',
-            taskDescription: nextTask.description ?? undefined,
-            implementationGuidelines,
-            unitTests,
-        };
-    }
-    if (nextTask instanceof ProjectStorySectionModelRouteChange) {
-        const queryParamChanges = nextTask.queryParamChanges
-            .filter((qp) => !!qp.operation)
-            .map((qp) => {
-                const enumInfo = qp.enumerationName ? ` (enum: ${qp.enumerationName})` : '';
-                return `${qp.operation} query param ${qp.routeQueryParamName}: ${qp.routeQueryParamType}${enumInfo}`;
-            })
-            .sort()
-            .join('\n');
-
-        return {
-            generalInstructions,
-            taskType: `${nextTask.operation} route`,
-            route: `${nextTask.routeMethod} ${nextTask.routePath}`,
-            routeObject: nextTask.routeObjectName ?? undefined,
-            pathChangedFrom:
-                nextTask.initialRoutePath && nextTask.initialRoutePath !== nextTask.routePath ? nextTask.initialRoutePath : undefined,
-            methodChangedFrom:
-                nextTask.initialRouteMethod && nextTask.initialRouteMethod !== nextTask.routeMethod
-                    ? nextTask.initialRouteMethod
-                    : undefined,
-            objectChangedFrom:
-                nextTask.initialRouteObjectName && nextTask.initialRouteObjectName !== nextTask.routeObjectName
-                    ? nextTask.initialRouteObjectName
-                    : undefined,
-            permissionChangedFrom:
-                nextTask.initialRoutePermission && nextTask.initialRoutePermission !== nextTask.routePermission
-                    ? nextTask.initialRoutePermission
-                    : undefined,
-            queryParamChanges: queryParamChanges || 'No change on query params',
-            taskDescription: nextTask.description ?? undefined,
-            implementationGuidelines,
-            unitTests,
-        };
-    }
-    if (nextTask instanceof ProjectStorySectionModelRelationChange) {
-        const relations = nextTask.relations
-            .filter((r) => !!r.operation)
-            .map((r) => `${r.operation} [${r.objects.map((o) => o.objectName).join(' ⇒ ')}]`)
-            .sort()
-            .join('\n');
-
-        return {
-            generalInstructions,
-            taskType: 'relation change',
-            handlerName: nextTask.handlerName,
-            boundedContext: nextTask.boundedContext,
-            subscriptionName: nextTask.subscriptionName ?? undefined,
-            relations: relations || 'No relation changes',
-            taskDescription: nextTask.description ?? undefined,
-            implementationGuidelines,
-            unitTests,
-        };
-    }
-    if (nextTask instanceof ProjectStorySectionModelTask) {
-        return {
-            generalInstructions,
-            taskType: nextTask.taskTypeName,
-            taskDescription: nextTask.description ?? undefined,
-            implementationGuidelines,
-            unitTests,
-        };
-    }
+    return renderTemplate('get-next-task', {
+        taskDetails: stringifyAnalysisDslChange(analysisChange),
+        guidelinesPath,
+        unitTestsPath: developmentPolicy?.areUnitTestsMandatory ? unitTestsPath : null,
+        isTodo,
+        identificationNumber,
+        taskId: analysisChange.id,
+    });
 }

package/src/mcp-server/tools/index.ts

@@ -1,4 +1 @@
-export * from './complete-task';
 export * from './get-next-task';
-export * from './get-unit-test-guidelines';
-export * from './start-task';

package/templates/develop-story-skill.hbs

@@ -0,0 +1,40 @@
+---
+name: develop-story
+description: >
+  Develop a we-scrum user story by iterating through its tasks one by one.
+  Use this skill whenever the user says "develop story XXXXX", "/develop-story XXXXX",
+  or asks to implement or work on a specific story number. The skill synchronizes
+  development policies, then fetches and implements each task in sequence using
+  the we-scrum MCP server and CLI.
+compatibility: "Requires: we-scrum MCP server, we-scrum CLI (we-scrum)"
+---
+
+# Develop Story Skill
+
+Iterates through a we-scrum story's tasks, implementing each one in sequence until
+all tasks are complete.
+
+---
+
+## Step 0 — Resolve the story identification number
+
+If the user provided a story identification number (e.g. `/develop-story 00240`), use it.
+Otherwise, ask: _"Which story would you like to develop? Please provide the identification number."_
+Do not proceed until a number is provided.
+
+---
+
+## Step 1 — Synchronize development policies
+
+Pull the latest development guidelines into the project before starting any work:
+
+```bash
+we-scrum synchronize-development-policies
+```
+
+---
+
+## Step 2 — Fetch and implement tasks
+
+Call the `get_next_task` MCP tool with the story identification number and follow
+the instructions it returns. Repeat until all tasks are complete.

package/templates/get-next-task-description.hbs

@@ -0,0 +1,3 @@
+Get the next pending task for a story as a Markdown document.
+Returns the task description, location of development guidelines, unit test instructions,
+and the exact CLI commands to run (we-scrum start-task / complete-task, with taskId pre-filled).

package/templates/get-next-task.hbs

@@ -0,0 +1,21 @@
+{{{taskDetails}}}
+
+## Development guidelines
+{{#if guidelinesPath}}Refer to `{{{guidelinesPath}}}` for coding conventions and project-specific instructions.{{else}}_No development guidelines file configured for this task._{{/if}}
+
+## Unit tests
+{{#if unitTestsPath}}Unit tests are required. Refer to `{{{unitTestsPath}}}` for test writing guidelines.{{else}}No unit tests required.{{/if}}
+
+## Next steps
+{{#if isTodo}}
+1. Run `we-scrum start-task --identificationNumber "{{identificationNumber}}" --taskId "{{taskId}}"` before making any changes.
+2. Implement the task described above.
+3. Commit your changes.
+4. Run `we-scrum complete-task --identificationNumber "{{identificationNumber}}" --taskId "{{taskId}}"` when done.
+5. Call `get_next_task` to continue with the next task.
+{{else}}
+1. Implement the task described above.
+2. Commit your changes.
+3. Run `we-scrum complete-task --identificationNumber "{{identificationNumber}}" --taskId "{{taskId}}"` when done.
+4. Call `get_next_task` to continue with the next task.
+{{/if}}

package/tsconfig.json

@@ -14,6 +14,7 @@
         "resolveJsonModule": true,
         "paths": {
             "@helpers": ["helpers"]
-        }
+        },
+        "forceConsistentCasingInFileNames": true
     }
 }

package/src/mcp-server/tools/complete-task.ts

@@ -1,6 +0,0 @@
-import { WeScrumHelper } from '@helpers';
-
-export async function completeTask(identificationNumber: string): Promise<Record<string, string>> {
-    await WeScrumHelper.completeTask(identificationNumber);
-    return { message: 'Task completed successfully.' };
-}

package/src/mcp-server/tools/get-unit-test-guidelines.ts

@@ -1,15 +0,0 @@
-import { WeScrumHelper } from '@helpers';
-
-export async function getUnitTestGuidelines(identificationNumber: string): Promise<Record<string, string>> {
-    const nextTask = await WeScrumHelper.getNextChange(identificationNumber);
-
-    if (!nextTask) {
-        return { message: `All tasks for story "${identificationNumber}" are complete.` };
-    }
-
-    const developmentPolicy = await WeScrumHelper.getChangeDevelopmentPolicy(nextTask);
-
-    return {
-        unitTestGuidelines: developmentPolicy?.unitTestsGuidelines ?? 'No unit test guidelines found for this task.',
-    };
-}

package/src/mcp-server/tools/start-task.ts

@@ -1,6 +0,0 @@
-import { WeScrumHelper } from '@helpers';
-
-export async function startTask(identificationNumber: string): Promise<Record<string, string>> {
-    await WeScrumHelper.startTask(identificationNumber);
-    return { message: 'Task started successfully. You can now make changes to the codebase.' };
-}