@polymorphism-tech/morph-spec 7.2.0 → 7.3.0

package/src/lib/validators/content/content-validator.js

@@ -1,286 +1,89 @@
-import { readFileSync, existsSync } from 'fs';
-
-/**
- * Content Validator - Validates structure and content of output files
- *
- * Ensures spec.md, tasks.json, and other outputs have required sections
- * and proper structure before allowing phase transitions.
- */
-
-/**
- * Validate spec.md structure
- * @param {string} specPath - Path to spec.md file
- * @returns {Object} Validation result
- */
-export function validateSpecContent(specPath) {
-  if (!existsSync(specPath)) {
-    return {
-      valid: false,
-      missing: ['File does not exist'],
-      errors: ['Spec file not found at: ' + specPath]
-    };
-  }
-
-  const content = readFileSync(specPath, 'utf8');
-
-  // Required sections (flexible regex handles numbered variants like '## 1. Overview')
-  const requiredSections = [
-    { label: '## Overview',                pattern: /^##\s+(?:\d+\.?\s+)?Overview/i },
-    { label: '## Functional Requirements', pattern: /^##\s+(?:\d+\.?\s+)?Functional Requirements/i },
-    { label: '## Technical',              pattern: /^##\s+(?:\d+\.?\s+)?Technical/i },
-    { label: '## Data Model',             pattern: /^##\s+(?:\d+\.?\s+)?Data Model/i },
-  ];
-
-  const lines = content.split('\n');
-  const missing = requiredSections.filter(s =>
-    !lines.some(line => s.pattern.test(line))
-  );
-
-  // Additional quality checks
-  const errors = [];
-  const warnings = [];
-
-  // Missing sections are warnings, not errors — not every spec needs all sections
-  // (e.g., frontend-only features may not have Data Model, CLI tools may not have Technical Architecture)
-  if (missing.length > 0) {
-    missing.forEach(s => {
-      warnings.push(`Missing recommended section: ${s.label}`);
-    });
-  }
-
-  // Check minimum content length
-  if (content.length < 500) {
-    warnings.push('Spec seems very short (< 500 characters). Consider adding more detail.');
-  }
-
-  // Check for placeholder text that should be replaced
-  const placeholders = ['TODO', 'TBD', 'PLACEHOLDER', '{{'];
-  placeholders.forEach(placeholder => {
-    if (content.includes(placeholder)) {
-      warnings.push(`Spec contains "${placeholder}" - ensure all placeholders are replaced`);
-    }
-  });
-
-  // Check for anti-patterns mentioned in the plan
-  if (content.toLowerCase().includes('manual azure portal')) {
-    errors.push(
-      'Spec mentions manual Azure portal creation. ' +
-      'All infrastructure must be defined in Bicep (Infrastructure as Code).'
-    );
-  }
-
-  if (content.toLowerCase().includes('create resource manually')) {
-    errors.push(
-      'Spec mentions manual resource creation. ' +
-      'Use IaC (Bicep) for all infrastructure resources.'
-    );
-  }
-
-  return {
-    valid: errors.length === 0,
-    missing,
-    errors,
-    warnings,
-    sections: {
-      required: requiredSections.length,
-      found: requiredSections.length - missing.length
-    }
-  };
-}
-
-/**
- * Validate tasks.md structure
- * @param {string} tasksPath - Path to tasks.md file
- * @returns {Object} Validation result
- */
-export function validateTasksContent(tasksPath) {
-  if (!existsSync(tasksPath)) {
-    return {
-      valid: false,
-      errors: ['Tasks file not found at: ' + tasksPath]
-    };
-  }
-
-  const content = readFileSync(tasksPath, 'utf8');
-  const errors = [];
-  const warnings = [];
-
-  // Extract task IDs from headings: ### T001 — title OR ### T001: title
-  const taskRe = /^###\s+(T\d{3})\s*[—–:\-]\s*(.+)$/gm;
-  const checkpointRe = /^###\s+(CHECKPOINT[_-]\d+)\s*[—–:\-]\s*(.+)$/gm;
-
-  const tasks = [...content.matchAll(taskRe)];
-  const checkpoints = [...content.matchAll(checkpointRe)];
-
-  if (tasks.length === 0) {
-    errors.push('No tasks found in tasks.md — expected headings like "### T001 — Title" or "### T001: Title"');
-    return { valid: false, errors, warnings };
-  }
-
-  // Validate no duplicate IDs
-  const ids = tasks.map(m => m[1]);
-  const dupes = ids.filter((id, i) => ids.indexOf(id) !== i);
-  dupes.forEach(id => errors.push(`Duplicate task ID: ${id}`));
-
-  // Check for dependency references (look for "Dependencies: T###")
-  const depRe = /\*\*Dependenc(?:y|ies):\*\*\s*([T\d, ]+)/gi;
-  const allDepRefs = [...content.matchAll(depRe)]
-    .flatMap(m => m[1].split(/[,\s]+/).filter(s => /^T\d{3}$/.test(s)));
-
-  const idSet = new Set(ids);
-  allDepRefs.forEach(depId => {
-    if (!idSet.has(depId)) {
-      warnings.push(`Dependency reference "${depId}" not found in task list`);
-    }
-  });
-
-  return {
-    valid: errors.length === 0,
-    errors,
-    warnings,
-    stats: {
-      totalTasks: tasks.length + checkpoints.length,
-      regularTasks: tasks.length,
-      checkpoints: checkpoints.length,
-    },
-  };
-}
-
-/**
- * Validate contracts file structure (C# contracts)
- * @param {string} contractsPath - Path to contracts file
- * @returns {Object} Validation result
- */
-export function validateContractsContent(contractsPath) {
-  if (!existsSync(contractsPath)) {
-    return {
-      valid: false,
-      errors: ['Contracts file not found at: ' + contractsPath]
-    };
-  }
-
-  const content = readFileSync(contractsPath, 'utf8');
-  const errors = [];
-  const warnings = [];
-
-  // Check for basic C# structure
-  if (!content.includes('namespace')) {
-    errors.push('Contracts file missing namespace declaration');
-  }
-
-  // Check for at least one interface or record
-  const hasInterface = /interface\s+\w+/.test(content);
-  const hasRecord = /record\s+\w+/.test(content);
-  const hasClass = /class\s+\w+/.test(content);
-
-  if (!hasInterface && !hasRecord && !hasClass) {
-    errors.push('Contracts file should define at least one interface, record, or class');
-  }
-
-  // Check for common anti-patterns
-  if (content.includes('// TODO') || content.includes('// PLACEHOLDER')) {
-    warnings.push('Contracts contain TODO/PLACEHOLDER comments - ensure they are completed');
-  }
-
-  return {
-    valid: errors.length === 0,
-    errors,
-    warnings,
-    found: {
-      hasNamespace: content.includes('namespace'),
-      hasInterfaces: hasInterface,
-      hasRecords: hasRecord,
-      hasClasses: hasClass
-    }
-  };
-}
-
-/**
- * Validate proposal.md structure
- * @param {string} proposalPath - Path to proposal.md file
- * @returns {Object} Validation result
- */
-export function validateProposalContent(proposalPath) {
-  if (!existsSync(proposalPath)) {
-    return {
-      valid: false,
-      errors: ['Proposal file not found at: ' + proposalPath]
-    };
-  }
-
-  const content = readFileSync(proposalPath, 'utf8');
-  const errors = [];
-  const warnings = [];
-
-  // Required sections
-  const requiredSections = [
-    '## Problem Statement',
-    '## Proposed Solution'
-  ];
-
-  const missing = requiredSections.filter(section => !content.includes(section));
-  missing.forEach(section => errors.push(`Missing required section: ${section}`));
-
-  // Check for minimum content
-  if (content.length < 300) {
-    warnings.push('Proposal seems very brief (< 300 characters)');
-  }
-
-  return {
-    valid: errors.length === 0,
-    errors,
-    warnings,
-    sections: {
-      required: requiredSections.length,
-      found: requiredSections.length - missing.length
-    }
-  };
-}
-
-/**
- * Validate all outputs for a feature comprehensively
- * @param {Object} featureState - Feature state object
- * @param {string} targetPhase - Target phase for validation
- * @returns {Object} Comprehensive validation result
- */
-export function validateFeatureOutputs(featureState, targetPhase) {
-  const results = {
-    valid: true,
-    validations: []
-  };
-
-  // Validate based on target phase
-  if (targetPhase === 'tasks' || targetPhase === 'implement') {
-    // Spec is required
-    if (featureState.outputs?.spec?.created) {
-      const specResult = validateSpecContent(featureState.outputs.spec.path);
-      results.validations.push({ type: 'spec', ...specResult });
-      if (!specResult.valid) results.valid = false;
-    } else {
-      results.valid = false;
-      results.validations.push({
-        type: 'spec',
-        valid: false,
-        errors: ['Spec file not created']
-      });
-    }
-  }
-
-  if (targetPhase === 'implement') {
-    // Tasks are required
-    if (featureState.outputs?.tasks?.created) {
-      const tasksResult = validateTasksContent(featureState.outputs.tasks.path);
-      results.validations.push({ type: 'tasks', ...tasksResult });
-      if (!tasksResult.valid) results.valid = false;
-    } else {
-      results.valid = false;
-      results.validations.push({
-        type: 'tasks',
-        valid: false,
-        errors: ['Tasks file not created']
-      });
-    }
-  }
-
-  return results;
-}
+import { readFileSync, existsSync } from 'fs';
+
+/**
+ * Content Validator - Validates structure and content of output files
+ *
+ * Ensures spec.md, tasks.json, and other outputs have required sections
+ * and proper structure before allowing phase transitions.
+ */
+
+/**
+ * Validate spec.md structure
+ * @param {string} specPath - Path to spec.md file
+ * @returns {Object} Validation result
+ */
+export function validateSpecContent(specPath) {
+  if (!existsSync(specPath)) {
+    return {
+      valid: false,
+      missing: ['File does not exist'],
+      errors: ['Spec file not found at: ' + specPath]
+    };
+  }
+
+  const content = readFileSync(specPath, 'utf8');
+
+  // Required sections (flexible regex handles numbered variants like '## 1. Overview')
+  const requiredSections = [
+    { label: '## Overview',                pattern: /^##\s+(?:\d+\.?\s+)?Overview/i },
+    { label: '## Functional Requirements', pattern: /^##\s+(?:\d+\.?\s+)?Functional Requirements/i },
+    { label: '## Technical',              pattern: /^##\s+(?:\d+\.?\s+)?Technical/i },
+    { label: '## Data Model',             pattern: /^##\s+(?:\d+\.?\s+)?Data Model/i },
+  ];
+
+  const lines = content.split('\n');
+  const missing = requiredSections.filter(s =>
+    !lines.some(line => s.pattern.test(line))
+  );
+
+  // Additional quality checks
+  const errors = [];
+  const warnings = [];
+
+  // Missing sections are warnings, not errors — not every spec needs all sections
+  // (e.g., frontend-only features may not have Data Model, CLI tools may not have Technical Architecture)
+  if (missing.length > 0) {
+    missing.forEach(s => {
+      warnings.push(`Missing recommended section: ${s.label}`);
+    });
+  }
+
+  // Check minimum content length
+  if (content.length < 500) {
+    warnings.push('Spec seems very short (< 500 characters). Consider adding more detail.');
+  }
+
+  // Check for placeholder text that should be replaced
+  const placeholders = ['TODO', 'TBD', 'PLACEHOLDER', '{{'];
+  placeholders.forEach(placeholder => {
+    if (content.includes(placeholder)) {
+      warnings.push(`Spec contains "${placeholder}" - ensure all placeholders are replaced`);
+    }
+  });
+
+  // Check for anti-patterns mentioned in the plan
+  if (content.toLowerCase().includes('manual azure portal')) {
+    errors.push(
+      'Spec mentions manual Azure portal creation. ' +
+      'All infrastructure must be defined in Bicep (Infrastructure as Code).'
+    );
+  }
+
+  if (content.toLowerCase().includes('create resource manually')) {
+    errors.push(
+      'Spec mentions manual resource creation. ' +
+      'Use IaC (Bicep) for all infrastructure resources.'
+    );
+  }
+
+  return {
+    valid: errors.length === 0,
+    missing,
+    errors,
+    warnings,
+    sections: {
+      required: requiredSections.length,
+      found: requiredSections.length - missing.length
+    }
+  };
+}

package/src/lib/validators/content/index.js

@@ -5,4 +5,3 @@
  */
 
 export * from './content-validator.js';
-export * from './cross-phase-validator.js';

package/framework/templates/feature/tasks.md

@@ -1,199 +0,0 @@
-# Tasks - {{titleCase FEATURE_NAME}}
-
-> Implementation checklist for the feature.
-> Update status as you progress.
-
-| Task ID | Type | Title | Status |
-|---------|------|-------|--------|
-| T001 | {type} | {title} | Pending |
-| T002 | {type} | {title} | Pending |
-| T003 | {type} | {title} | In Progress |
-| T004 | {type} | {title} | Blocked |
-| T005 | {type} | {title} | Done |
-
-**Status Values:** `Pending | In Progress | Blocked | Done`
-
----
-
-## Task Granularity
-
-### Ideal Task
-- 15-60 minutes of implementation
-- 1-5 files modified
-- Testable in isolation
-- Independent commit possible
-
-### Signs of Too Broad (Split)
-**Bad example:** "Implement feature"
-**Split into:**
-- T001: Define contracts and interfaces
-- T002: Implement data layer
-- T003: Implement business logic
-- T004: Create UI components
-- T005: Write tests
-
-### Signs of Too Granular (Merge)
-**Bad example:** "Create field A", "Create field B", "Create field C"
-**Merge into:** "Create data model with all fields"
-
----
-
-## Gates & Reviews
-
-| Task ID | Type | Description | Status |
-|---------|------|-------------|--------|
-| T003 | Review | Review contracts and data model | Pending |
-| T006 | Review | Review implementation | Pending |
-| T009 | Approval | Approve UI before testing | Pending |
-
----
-
-## Phase 1: Contracts & Data Model
-
-### T001: Define contracts and interfaces
-- **Type:** Contract
-- **Priority:** 1
-- **Estimate:** 30min
-- **Assigned:** {agent}
-- **Dependencies:** None
-
-**Description:**
-
-Define service interfaces and contracts.
-
-**Deliverables:**
-- [ ] Create: `{path-to-interface-file}`
-
----
-
-### T002: Create data model
-- **Type:** {{#if (eq STACK "nextjs-neon")}}Database{{else}}Domain{{/if}}
-- **Priority:** 1
-- **Estimate:** 45min
-- **Assigned:** {agent}
-- **Dependencies:** None
-
-**Description:**
-
-Create data model {{#if (eq STACK "nextjs-neon")}}with migrations{{else}}with entities{{/if}}.
-
-**Deliverables:**
-- [ ] Create: `{path-to-model-file}`
-
----
-
-## Phase 2: Implementation
-
-### T003: Implement data layer
-- **Type:** {{#if (eq STACK "nextjs-neon")}}Repository{{else}}Infrastructure{{/if}}
-- **Priority:** 2
-- **Estimate:** 1h
-- **Assigned:** {agent}
-- **Dependencies:** T001, T002
-
-**Description:**
-
-Implement data access layer.
-
-**Deliverables:**
-- [ ] Create: `{path-to-repository-file}`
-
----
-
-### T004: Implement business logic
-- **Type:** Service
-- **Priority:** 2
-- **Estimate:** 1h
-- **Assigned:** {agent}
-- **Dependencies:** T001, T003
-
-**Description:**
-
-Implement service layer with business logic.
-
-**Deliverables:**
-- [ ] Create: `{path-to-service-file}`
-
----
-
-## Phase 3: UI Components
-
-### T005: Create list component
-- **Type:** UI
-- **Priority:** 2
-- **Estimate:** 45min
-- **Assigned:** {agent}
-- **Dependencies:** T004
-
-**Description:**
-
-Create component to list items.
-
-**Deliverables:**
-- [ ] Create: `{path-to-component-file}`
-
----
-
-### T006: Create form component
-- **Type:** UI
-- **Priority:** 2
-- **Estimate:** 1h
-- **Assigned:** {agent}
-- **Dependencies:** T005
-
-**Description:**
-
-Create component for create/edit.
-
-**Deliverables:**
-- [ ] Create: `{path-to-form-file}`
-
----
-
-## Phase 4: Testing
-
-### T007: Write unit tests
-- **Type:** Test
-- **Priority:** 2
-- **Estimate:** 1h
-- **Assigned:** {agent}
-- **Dependencies:** T004
-
-**Description:**
-
-Unit tests for service layer.
-
-**Deliverables:**
-- [ ] Create: `{path-to-test-file}`
-
----
-
-### T008: Write integration tests
-- **Type:** Test
-- **Priority:** 3
-- **Estimate:** 1h
-- **Assigned:** {agent}
-- **Dependencies:** T007
-
-**Description:**
-
-Integration tests for {{#if (eq STACK "nextjs-neon")}}API and database{{else}}repository and API{{/if}}.
-
-**Deliverables:**
-- [ ] Create: `{path-to-integration-test-file}`
-
----
-
-## Summary
-
-| Phase | Tasks | Estimate |
-|-------|-------|----------|
-| Phase 1 | T001-T002 | {X}h |
-| Phase 2 | T003-T004 | {X}h |
-| Phase 3 | T005-T006 | {X}h |
-| Phase 4 | T007-T008 | {X}h |
-| **Total** | **8 tasks** | **{X}h** |
-
----
-
-*Generated by MORPH Framework*