mcoda 0.1.2 → 0.1.4

package/packages/core/src/services/execution/QaFollowupService.ts

@@ -1,289 +0,0 @@
-import {
-  EpicInsert,
-  StoryInsert,
-  TaskCommentInsert,
-  TaskDependencyInsert,
-  TaskInsert,
-  TaskRow,
-  WorkspaceRepository,
-} from '@mcoda/db';
-import { createTaskKeyGenerator } from '../planning/KeyHelpers.js';
-import path from 'node:path';
-import fs from 'node:fs/promises';
-import { PathHelper } from '@mcoda/shared';
-
-export interface FollowupSuggestion {
-  title: string;
-  description?: string;
-  type?: string;
-  priority?: number;
-  storyPoints?: number;
-  tags?: string[];
-  epicKeyHint?: string;
-  storyKeyHint?: string;
-  relatedTaskKey?: string;
-  components?: string[];
-  docLinks?: string[];
-  testName?: string;
-  evidenceUrl?: string;
-  artifacts?: string[];
-}
-
-const BUG_EPIC_KEY = 'EPIC-BUGS';
-const BUG_STORY_KEY = 'US-BUGS';
-const FOLLOWUP_DESCRIPTION_TEMPLATE = (
-  summary: string,
-  actual: string,
-  expected: string,
-  envLines: string[],
-  steps?: string[],
-): string => {
-  const lines = [
-    '# Summary',
-    summary || 'Summarize the problem discovered during QA.',
-    '',
-    '# Steps to Reproduce',
-    ...(steps && steps.length
-      ? steps.map((step, idx) => `${idx + 1}. ${step}`)
-      : ['1. Provide minimal reproducible steps based on failing test or QA notes.']),
-    '',
-    '# Expected',
-    expected || 'Describe the expected behavior per acceptance criteria / OpenAPI.',
-    '',
-    '# Actual',
-    actual || 'Describe the observed behavior.',
-    '',
-    '# Environment',
-    ...envLines.map((line) => `- ${line}`),
-  ];
-  return lines.join('\n');
-};
-
-export class QaFollowupService {
-  constructor(private workspaceRepo: WorkspaceRepository, private workspaceRoot: string) {}
-
-  private get cachePath(): string {
-    return path.join(this.workspaceRoot, '.mcoda', 'qa-containers.json');
-  }
-
-  private async readCache(): Promise<Record<string, { epicId: string; storyId: string; epicKey: string; storyKey: string }>> {
-    try {
-      const raw = await fs.readFile(this.cachePath, 'utf8');
-      return JSON.parse(raw) as Record<string, { epicId: string; storyId: string; epicKey: string; storyKey: string }>;
-    } catch {
-      return {};
-    }
-  }
-
-  private async writeCache(data: Record<string, { epicId: string; storyId: string; epicKey: string; storyKey: string }>): Promise<void> {
-    await PathHelper.ensureDir(path.dirname(this.cachePath));
-    await fs.writeFile(this.cachePath, JSON.stringify(data, null, 2), 'utf8');
-  }
-
-  private async ensureBugContainer(projectId: string): Promise<{
-    epicId: string;
-    storyId: string;
-    epicKey: string;
-    storyKey: string;
-  }> {
-    const cache = await this.readCache();
-    if (cache[projectId]) {
-      return cache[projectId];
-    }
-
-    let epic = await this.workspaceRepo.getEpicByKey(projectId, BUG_EPIC_KEY);
-    if (!epic) {
-      const epicInsert: EpicInsert = {
-        projectId,
-        key: BUG_EPIC_KEY,
-        title: 'Generic Bugs / Issues',
-        description: 'Container epic for QA-found bugs and follow-up items.',
-        storyPointsTotal: null,
-        priority: 999,
-        metadata: { tags: ['generic', 'qa-intake'] },
-      };
-      const [created] = await this.workspaceRepo.insertEpics([epicInsert], true);
-      epic = created;
-    }
-    let story = await this.workspaceRepo.getStoryByKey(epic.id, BUG_STORY_KEY);
-    if (!story) {
-      const storyInsert: StoryInsert = {
-        projectId,
-        epicId: epic.id,
-        key: BUG_STORY_KEY,
-        title: 'Generic Bugs / Issues',
-        description: 'Default story container for unmapped QA follow-ups.',
-        acceptanceCriteria: undefined,
-        storyPointsTotal: null,
-        priority: 999,
-        metadata: { tags: ['generic', 'qa-intake'] },
-      };
-      const [created] = await this.workspaceRepo.insertStories([storyInsert], true);
-      story = created;
-    }
-    const resolved = { epicId: epic.id, storyId: story.id, epicKey: epic.key, storyKey: story.key };
-    cache[projectId] = resolved;
-    await this.writeCache(cache);
-    return resolved;
-  }
-
-  async createFollowupTask(
-    sourceTask: TaskRow & { storyKey?: string; epicKey?: string },
-    suggestion: FollowupSuggestion,
-  ): Promise<{ task: TaskInsert & { id: string }; dependency?: TaskDependencyInsert; comment?: TaskCommentInsert }> {
-    const projectId = sourceTask.projectId;
-    const resolved = await this.resolveTargetContainer(sourceTask, suggestion);
-    const storyKeyBase =
-      suggestion.storyKeyHint ?? resolved.storyKey ?? sourceTask.storyKey ?? sourceTask.key.split('-t')[0] ?? 'task';
-    const existingKeys = await this.workspaceRepo.listTaskKeys(resolved.storyId);
-    const keyGen = createTaskKeyGenerator(storyKeyBase, existingKeys);
-    const followupKey = keyGen();
-    const now = new Date().toISOString();
-    const metadata: Record<string, unknown> = {
-      tags: ['qa-found', 'auto-created', 'ready-for-ai-dev', 'source=qa', ...(suggestion.tags ?? [])],
-      source_task: sourceTask.key,
-      ...(suggestion.components ? { components: suggestion.components } : {}),
-      ...(suggestion.docLinks ? { doc_links: suggestion.docLinks } : {}),
-      ...(suggestion.testName ? { failing_test: suggestion.testName } : {}),
-      ...(suggestion.evidenceUrl ? { evidence_url: suggestion.evidenceUrl } : {}),
-    };
-    const envInfo = [
-      `Task: ${sourceTask.key}`,
-      `Epic: ${sourceTask.epicKey ?? resolved.epicKey ?? sourceTask.epicId}`,
-      `Story: ${sourceTask.storyKey ?? resolved.storyKey ?? sourceTask.userStoryId}`,
-      `Branch/Commit: ${sourceTask.vcsBranch ?? 'n/a'} / ${sourceTask.vcsLastCommitSha ?? 'n/a'}`,
-      `Components: ${Array.isArray(suggestion.components) && suggestion.components.length ? suggestion.components.join(', ') : Array.isArray((sourceTask.metadata as any)?.components) ? (sourceTask.metadata as any).components.join(', ') : 'n/a'}`,
-      `Tests: ${suggestion.testName ?? (Array.isArray((sourceTask.metadata as any)?.tests) ? (sourceTask.metadata as any).tests.join(', ') : 'n/a')}`,
-      `Docs: ${suggestion.docLinks?.join(', ') ?? (Array.isArray((sourceTask.metadata as any)?.doc_links) ? (sourceTask.metadata as any).doc_links.join(', ') : 'n/a')}`,
-      suggestion.evidenceUrl ? `Evidence: ${suggestion.evidenceUrl}` : '',
-      suggestion.artifacts && suggestion.artifacts.length ? `Artifacts: ${suggestion.artifacts.join(', ')}` : '',
-    ].filter(Boolean);
-    const description = FOLLOWUP_DESCRIPTION_TEMPLATE(
-      suggestion.title ?? `Follow-up for ${sourceTask.key}`,
-      suggestion.description ?? 'Actual behavior not provided.',
-      'Expected behavior per acceptance criteria/OpenAPI.',
-      envInfo,
-    );
-    const taskInsert: TaskInsert = {
-      projectId,
-      epicId: resolved.epicId,
-      userStoryId: resolved.storyId,
-      key: followupKey,
-      title: suggestion.title || `Follow-up for ${sourceTask.key}`,
-      description,
-      type: suggestion.type ?? 'bug',
-      status: 'not_started',
-      storyPoints: suggestion.storyPoints ?? 1,
-      priority: suggestion.priority ?? 99,
-      metadata,
-    };
-    const [createdTask] = await this.workspaceRepo.insertTasks([taskInsert], true);
-    const dependency: TaskDependencyInsert = {
-      taskId: sourceTask.id,
-      dependsOnTaskId: createdTask.id,
-      relationType: 'blocks',
-    };
-    await this.workspaceRepo.insertTaskDependencies([dependency], true);
-    const sourceComment: TaskCommentInsert = {
-      taskId: sourceTask.id,
-      sourceCommand: 'qa-tasks',
-      authorType: 'agent',
-      category: 'qa_followup',
-      body: `Created follow-up task ${createdTask.key} for QA findings.`,
-      metadata: { followupTaskKey: createdTask.key },
-      createdAt: now,
-    };
-    await this.workspaceRepo.createTaskComment(sourceComment);
-    const followupComment: TaskCommentInsert = {
-      taskId: createdTask.id,
-      sourceCommand: 'qa-tasks',
-      authorType: 'agent',
-      category: 'qa_origin',
-      body: `Created automatically from QA of ${sourceTask.key}.`,
-      metadata: { sourceTaskKey: sourceTask.key },
-      createdAt: now,
-    };
-    await this.workspaceRepo.createTaskComment(followupComment);
-    return { task: { ...taskInsert, id: createdTask.id }, dependency, comment: sourceComment };
-  }
-
-  private async resolveTargetContainer(
-    sourceTask: TaskRow & { storyKey?: string; epicKey?: string },
-    suggestion: FollowupSuggestion,
-  ): Promise<{ epicId: string; storyId: string; storyKey?: string; epicKey?: string }> {
-    const projectId = sourceTask.projectId;
-    if (suggestion.relatedTaskKey) {
-      const related = await this.workspaceRepo.getTaskByKey(suggestion.relatedTaskKey);
-      if (related) {
-        return {
-          epicId: related.epicId,
-          storyId: related.userStoryId,
-          storyKey: related.key.split('-t')[0] ?? related.key,
-        };
-      }
-    }
-    if (suggestion.storyKeyHint) {
-      const story = await this.workspaceRepo.getStoryByProjectAndKey(projectId, suggestion.storyKeyHint);
-      if (story) {
-        return { epicId: story.epicId, storyId: story.id, storyKey: story.key };
-      }
-    }
-    if (Array.isArray(suggestion.docLinks)) {
-      for (const link of suggestion.docLinks) {
-        const storyKeyCandidate = link.match(/US-[A-Za-z0-9]+/i)?.[0];
-        if (storyKeyCandidate) {
-          const story = await this.workspaceRepo.getStoryByProjectAndKey(projectId, storyKeyCandidate.toUpperCase());
-          if (story) return { epicId: story.epicId, storyId: story.id, storyKey: story.key };
-        }
-        const epicKeyCandidate = link.match(/EPIC-[A-Za-z0-9]+/i)?.[0];
-        if (epicKeyCandidate) {
-          const epic = await this.workspaceRepo.getEpicByKey(projectId, epicKeyCandidate.toUpperCase());
-          if (epic) {
-            return {
-              epicId: epic.id,
-              epicKey: epic.key,
-              storyId: sourceTask.userStoryId ?? epic.id,
-              storyKey: sourceTask.storyKey ?? epic.key,
-            };
-          }
-        }
-      }
-    }
-    const branch = sourceTask.vcsBranch ?? '';
-    const storyMatch = branch.match(/US-[A-Za-z0-9]+/i);
-    if (storyMatch) {
-      const story = await this.workspaceRepo.getStoryByProjectAndKey(projectId, storyMatch[0].toUpperCase());
-      if (story) {
-        return { epicId: story.epicId, storyId: story.id, storyKey: story.key };
-      }
-    }
-    const epicMatch = branch.match(/EPIC-[A-Za-z0-9]+/i);
-    if (epicMatch) {
-      const epic = await this.workspaceRepo.getEpicByKey(projectId, epicMatch[0].toUpperCase());
-      if (epic) {
-        return {
-          epicId: epic.id,
-          epicKey: epic.key,
-          storyId: sourceTask.userStoryId ?? epic.id,
-          storyKey: sourceTask.storyKey ?? suggestion.storyKeyHint ?? epic.key,
-        };
-      }
-    }
-    if (suggestion.epicKeyHint) {
-      const epic = await this.workspaceRepo.getEpicByKey(projectId, suggestion.epicKeyHint);
-      if (epic) {
-        return {
-          epicId: epic.id,
-          epicKey: epic.key,
-          storyId: sourceTask.userStoryId ?? epic.id,
-          storyKey: sourceTask.storyKey ?? suggestion.storyKeyHint ?? epic.key,
-        };
-      }
-    }
-    if (sourceTask.epicId && sourceTask.userStoryId) {
-      return { epicId: sourceTask.epicId, storyId: sourceTask.userStoryId, storyKey: sourceTask.storyKey, epicKey: sourceTask.epicKey };
-    }
-    const container = await this.ensureBugContainer(projectId);
-    return { epicId: container.epicId, storyId: container.storyId, storyKey: container.storyKey, epicKey: container.epicKey };
-  }
-}

package/packages/core/src/services/execution/QaProfileService.ts

@@ -1,160 +0,0 @@
-import path from 'node:path';
-import { TaskRow } from '@mcoda/db';
-import { PathHelper } from '@mcoda/shared';
-import { QaProfile } from '@mcoda/shared/qa/QaProfile.js';
-import fs from 'node:fs/promises';
-
-export interface QaProfileResolutionOptions {
-  profileName?: string;
-  level?: string;
-  defaultLevel?: string;
-}
-
-export class QaProfileService {
-  private cache?: QaProfile[];
-  private routingCache?: {
-    defaultProfile?: string;
-    levels?: Record<string, string>;
-    taskTypes?: Record<string, string>;
-    tags?: Record<string, string>;
-  };
-
-  constructor(private workspaceRoot: string) {}
-
-  private get profilePath(): string {
-    return path.join(this.workspaceRoot, '.mcoda', 'qa-profiles.json');
-  }
-
-  private get workspaceConfigPath(): string {
-    return path.join(this.workspaceRoot, '.mcoda', 'config.json');
-  }
-
-  private async getConfiguredDefaultProfileName(): Promise<string | undefined> {
-    try {
-      const raw = await fs.readFile(this.workspaceConfigPath, 'utf8');
-      const parsed = JSON.parse(raw);
-      return parsed?.qa?.defaultProfile ?? parsed?.qa?.default_profile ?? parsed?.qaDefaultProfile;
-    } catch {
-      return undefined;
-    }
-  }
-
-  private async getRoutingConfig(): Promise<{
-    defaultProfile?: string;
-    levels?: Record<string, string>;
-    taskTypes?: Record<string, string>;
-    tags?: Record<string, string>;
-  }> {
-    if (this.routingCache) return this.routingCache;
-    try {
-      const raw = await fs.readFile(this.workspaceConfigPath, 'utf8');
-      const parsed = JSON.parse(raw);
-      const qa = parsed?.qa ?? {};
-      this.routingCache = {
-        defaultProfile: qa.defaultProfile ?? qa.default_profile ?? qa.default,
-        levels: qa.routing?.levels ?? qa.routing?.level ?? qa.levelRouting ?? undefined,
-        taskTypes: qa.routing?.taskTypes ?? qa.routing?.types ?? qa.typeRouting ?? undefined,
-        tags: qa.routing?.tags ?? undefined,
-      };
-      return this.routingCache;
-    } catch {
-      this.routingCache = {};
-      return this.routingCache;
-    }
-  }
-
-  async loadProfiles(): Promise<QaProfile[]> {
-    if (this.cache) return this.cache;
-    await PathHelper.ensureDir(path.join(this.workspaceRoot, '.mcoda'));
-    try {
-      const raw = await fs.readFile(this.profilePath, 'utf8');
-      const parsed = JSON.parse(raw);
-      if (Array.isArray(parsed)) {
-        this.cache = parsed as QaProfile[];
-        return this.cache;
-      }
-      if (Array.isArray((parsed as any)?.profiles)) {
-        this.cache = (parsed as any).profiles as QaProfile[];
-        return this.cache;
-      }
-      this.cache = [];
-      return this.cache;
-    } catch (error: any) {
-      if (error?.code === 'ENOENT') {
-        this.cache = [];
-        return this.cache;
-      }
-      throw error;
-    }
-  }
-
-  async resolveProfileForTask(task: TaskRow & { metadata?: any }, options: QaProfileResolutionOptions = {}): Promise<QaProfile | undefined> {
-    const profiles = await this.loadProfiles();
-    if (!profiles.length) return undefined;
-    const envProfile = process.env.MCODA_QA_PROFILE;
-    const routing = await this.getRoutingConfig();
-    const configuredDefault = options.profileName ?? envProfile ?? (await this.getConfiguredDefaultProfileName()) ?? routing.defaultProfile;
-    const pickByName = (name: string | undefined): QaProfile | undefined => {
-      if (!name) return undefined;
-      const match = profiles.find((p) => p.name === name);
-      if (!match) {
-        throw new Error(`QA profile not found: ${name}`);
-      }
-      return match;
-    };
-    const explicit = pickByName(configuredDefault);
-    if (explicit) {
-      return explicit;
-    }
-    const taskTags: string[] = Array.isArray((task.metadata as any)?.tags)
-      ? ((task.metadata as any).tags as string[]).map((t) => t.toLowerCase())
-      : [];
-    let candidates = profiles.filter((profile) => {
-      const typeMatch =
-        !profile.matcher?.task_types ||
-        profile.matcher.task_types.length === 0 ||
-        (task.type ? profile.matcher.task_types.includes(task.type) : false);
-      const tagMatch =
-        !profile.matcher?.tags ||
-        profile.matcher.tags.length === 0 ||
-        profile.matcher.tags.some((tag) => taskTags.includes(tag.toLowerCase()));
-      return typeMatch && tagMatch;
-    });
-    if (routing) {
-      const levelRoute = options.level && routing.levels ? routing.levels[options.level] : undefined;
-      const typeRoute = task.type && routing.taskTypes ? routing.taskTypes[task.type] : undefined;
-      const tagRoute =
-        routing.tags && taskTags.length
-          ? taskTags.map((tag) => routing.tags?.[tag]).find((name): name is string => Boolean(name))
-          : undefined;
-      const routedName = levelRoute ?? typeRoute ?? tagRoute;
-      const routed = pickByName(routedName);
-      if (routed) return routed;
-    }
-    const targetLevel = options.level ?? options.defaultLevel;
-    if (targetLevel) {
-      const levelMatches = candidates.filter((p) => (p.level ?? (p as any).qa_level) === targetLevel);
-      if (levelMatches.length === 1) return levelMatches[0];
-      if (levelMatches.length > 1) candidates = levelMatches;
-    }
-    if (candidates.length === 1) return candidates[0];
-    if (candidates.length > 1) {
-      const defaults = candidates.filter((p) => p.default);
-      if (defaults.length === 1) return defaults[0];
-      if (defaults.length > 1) {
-        throw new Error('Multiple default QA profiles configured; specify --profile.');
-      }
-      throw new Error(
-        `Multiple QA profiles match task (${task.key}); please specify --profile. Candidates: ${candidates
-          .map((p) => p.name)
-          .join(', ')}`,
-      );
-    }
-    const defaults = profiles.filter((p) => p.default);
-    if (defaults.length === 1) return defaults[0];
-    if (defaults.length > 1) {
-      throw new Error('Multiple default QA profiles configured; please specify --profile.');
-    }
-    return undefined;
-  }
-}