@dynamicworks/br-openspec 1.3.1 → 2.0.1

package/dist/core/parsers/change-parser.js

@@ -1,6 +1,7 @@
 import { MarkdownParser } from './markdown-parser.js';
 import path from 'path';
 import { promises as fs } from 'fs';
+import { CHANGE_PARSER_MESSAGES } from '../../messages/index.js';
 export class ChangeParser extends MarkdownParser {
     changeDir;
     constructor(content, changeDir) {
@@ -12,10 +13,10 @@ export class ChangeParser extends MarkdownParser {
         const why = this.findSection(sections, 'Why')?.content || '';
         const whatChanges = this.findSection(sections, 'What Changes')?.content || '';
         if (!why) {
-            throw new Error('Change must have a Why section');
+            throw new Error(CHANGE_PARSER_MESSAGES.mustHaveWhySection);
         }
         if (!whatChanges) {
-            throw new Error('Change must have a What Changes section');
+            throw new Error(CHANGE_PARSER_MESSAGES.mustHaveWhatChangesSection);
         }
         // Parse deltas from the What Changes section (simple format)
         const simpleDeltas = this.parseDeltas(whatChanges);
@@ -72,7 +73,7 @@ export class ChangeParser extends MarkdownParser {
                 deltas.push({
                     spec: specName,
                     operation: 'ADDED',
-                    description: `Add requirement: ${req.text}`,
+                    description: CHANGE_PARSER_MESSAGES.addRequirement(req.text),
                     // Provide both single and plural forms for compatibility
                     requirement: req,
                     requirements: [req],
@@ -87,7 +88,7 @@ export class ChangeParser extends MarkdownParser {
                 deltas.push({
                     spec: specName,
                     operation: 'MODIFIED',
-                    description: `Modify requirement: ${req.text}`,
+                    description: CHANGE_PARSER_MESSAGES.modifyRequirement(req.text),
                     requirement: req,
                     requirements: [req],
                 });
@@ -101,7 +102,7 @@ export class ChangeParser extends MarkdownParser {
                 deltas.push({
                     spec: specName,
                     operation: 'REMOVED',
-                    description: `Remove requirement: ${req.text}`,
+                    description: CHANGE_PARSER_MESSAGES.removeRequirement(req.text),
                     requirement: req,
                     requirements: [req],
                 });
@@ -115,7 +116,7 @@ export class ChangeParser extends MarkdownParser {
                 deltas.push({
                     spec: specName,
                     operation: 'RENAMED',
-                    description: `Rename requirement from "${rename.from}" to "${rename.to}"`,
+                    description: CHANGE_PARSER_MESSAGES.renameRequirement(rename.from, rename.to),
                     rename,
                 });
             });

package/dist/core/project-config.js

@@ -1,4 +1,4 @@
-import { PROJECT_CONFIG_MESSAGES } from '../messages/index.js';
+import { PROJECT_CONFIG_MESSAGES, PROJECT_CONFIG_SUGGEST_MESSAGES } from '../messages/index.js';
 import { existsSync, readFileSync } from 'fs';
 import path from 'path';
 import { parse as parseYaml } from 'yaml';
@@ -69,7 +69,7 @@ export function readProjectConfig(projectRoot) {
         const content = readFileSync(configPath, 'utf-8');
         const raw = parseYaml(content);
         if (!raw || typeof raw !== 'object') {
-            console.warn(`openspec/config.yaml is not a valid YAML object`);
+            console.warn(PROJECT_CONFIG_SUGGEST_MESSAGES.configNotValidYaml);
             return null;
         }
         const config = {};
@@ -136,7 +136,7 @@ export function readProjectConfig(projectRoot) {
         return Object.keys(config).length > 0 ? config : null;
     }
     catch (error) {
-        console.warn(`Failed to parse openspec/config.yaml:`, error);
+        console.warn(PROJECT_CONFIG_SUGGEST_MESSAGES.configFailedToParse, error);
         return null;
     }
 }
@@ -155,8 +155,7 @@ export function validateConfigRules(rules, validArtifactIds, schemaName) {
     for (const artifactId of Object.keys(rules)) {
         if (!validArtifactIds.has(artifactId)) {
             const validIds = Array.from(validArtifactIds).sort().join(', ');
-            warnings.push(`Unknown artifact ID in rules: "${artifactId}". ` +
-                `Valid IDs for schema "${schemaName}": ${validIds}`);
+            warnings.push(PROJECT_CONFIG_SUGGEST_MESSAGES.unknownArtifactId(artifactId, schemaName, validIds));
         }
     }
     return warnings;
@@ -199,26 +198,26 @@ export function suggestSchemas(invalidSchemaName, availableSchemas) {
         .slice(0, 3);
     const builtIn = availableSchemas.filter((s) => s.isBuiltIn).map((s) => s.name);
     const projectLocal = availableSchemas.filter((s) => !s.isBuiltIn).map((s) => s.name);
-    let message = `Schema '${invalidSchemaName}' not found in openspec/config.yaml\n\n`;
+    let message = PROJECT_CONFIG_SUGGEST_MESSAGES.schemaNotFound(invalidSchemaName);
     if (suggestions.length > 0) {
-        message += `Did you mean one of these?\n`;
+        message += PROJECT_CONFIG_SUGGEST_MESSAGES.didYouMean;
         suggestions.forEach((s) => {
-            const type = s.isBuiltIn ? 'built-in' : 'project-local';
+            const type = PROJECT_CONFIG_SUGGEST_MESSAGES.schemaType(s.isBuiltIn);
             message += `  - ${s.name} (${type})\n`;
         });
         message += '\n';
     }
-    message += `Available schemas:\n`;
+    message += PROJECT_CONFIG_SUGGEST_MESSAGES.availableSchemas;
     if (builtIn.length > 0) {
-        message += `  Built-in: ${builtIn.join(', ')}\n`;
+        message += PROJECT_CONFIG_SUGGEST_MESSAGES.builtInSchemas(builtIn.join(', '));
     }
     if (projectLocal.length > 0) {
-        message += `  Project-local: ${projectLocal.join(', ')}\n`;
+        message += PROJECT_CONFIG_SUGGEST_MESSAGES.projectLocalSchemas(projectLocal.join(', '));
     }
     else {
-        message += `  Project-local: (none found)\n`;
+        message += PROJECT_CONFIG_SUGGEST_MESSAGES.noProjectLocalSchemas;
     }
-    message += `\nFix: Edit openspec/config.yaml and change 'schema: ${invalidSchemaName}' to a valid schema name`;
+    message += PROJECT_CONFIG_SUGGEST_MESSAGES.fixSuggestion(invalidSchemaName);
     return message;
 }
 //# sourceMappingURL=project-config.js.map

package/dist/core/shared/skill-generation.js

@@ -79,18 +79,18 @@ export function generateSkillContent(template, generatedByVersion, transformInst
     const instructions = transformInstructions
         ? transformInstructions(template.instructions)
         : template.instructions;
-    return `---
-name: ${template.name}
-description: ${template.description}
-license: ${template.license || 'MIT'}
-compatibility: ${template.compatibility || 'Requires openspec CLI.'}
-metadata:
-  author: ${template.metadata?.author || 'openspec'}
-  version: "${template.metadata?.version || '1.0'}"
-  generatedBy: "${generatedByVersion}"
----
-
-${instructions}
+    return `---
+name: ${template.name}
+description: ${template.description}
+license: ${template.license || 'MIT'}
+compatibility: ${template.compatibility || 'Requires openspec CLI.'}
+metadata:
+  author: ${template.metadata?.author || 'openspec'}
+  version: "${template.metadata?.version || '1.0'}"
+  generatedBy: "${generatedByVersion}"
+---
+
+${instructions}
 `;
 }
 //# sourceMappingURL=skill-generation.js.map

package/dist/core/specs-apply.js

@@ -10,7 +10,7 @@ import chalk from 'chalk';
 import { extractRequirementsSection, parseDeltaSpec, normalizeRequirementName, } from './parsers/requirement-blocks.js';
 import { findMainSpecStructureIssues } from './parsers/spec-structure.js';
 import { Validator } from './validation/validator.js';
-import { ARCHIVE_MESSAGES } from '../messages/index.js';
+import { ARCHIVE_MESSAGES, SPECS_APPLY_MESSAGES } from '../messages/index.js';
 // -----------------------------------------------------------------------------
 // Public API
 // -----------------------------------------------------------------------------
@@ -69,7 +69,7 @@ export async function buildUpdatedSpec(update, changeName) {
     for (const add of plan.added) {
         const name = normalizeRequirementName(add.name);
         if (addedNames.has(name)) {
-            throw new Error(`${specName} validation failed - duplicate requirement in ADDED for header "### Requirement: ${add.name}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.duplicateInSection(specName, 'ADDED', add.name));
         }
         addedNames.add(name);
     }
@@ -77,7 +77,7 @@ export async function buildUpdatedSpec(update, changeName) {
     for (const mod of plan.modified) {
         const name = normalizeRequirementName(mod.name);
         if (modifiedNames.has(name)) {
-            throw new Error(`${specName} validation failed - duplicate requirement in MODIFIED for header "### Requirement: ${mod.name}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.duplicateInSection(specName, 'MODIFIED', mod.name));
         }
         modifiedNames.add(name);
     }
@@ -85,7 +85,7 @@ export async function buildUpdatedSpec(update, changeName) {
     for (const rem of plan.removed) {
         const name = normalizeRequirementName(rem);
         if (removedNamesSet.has(name)) {
-            throw new Error(`${specName} validation failed - duplicate requirement in REMOVED for header "### Requirement: ${rem}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.duplicateInSection(specName, 'REMOVED', rem));
         }
         removedNamesSet.add(name);
     }
@@ -95,10 +95,10 @@ export async function buildUpdatedSpec(update, changeName) {
         const fromNorm = normalizeRequirementName(from);
         const toNorm = normalizeRequirementName(to);
         if (renamedFromSet.has(fromNorm)) {
-            throw new Error(`${specName} validation failed - duplicate FROM in RENAMED for header "### Requirement: ${from}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.duplicateFromInRenamed(specName, from));
         }
         if (renamedToSet.has(toNorm)) {
-            throw new Error(`${specName} validation failed - duplicate TO in RENAMED for header "### Requirement: ${to}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.duplicateToInRenamed(specName, to));
         }
         renamedFromSet.add(fromNorm);
         renamedToSet.add(toNorm);
@@ -120,21 +120,20 @@ export async function buildUpdatedSpec(update, changeName) {
         const fromNorm = normalizeRequirementName(from);
         const toNorm = normalizeRequirementName(to);
         if (modifiedNames.has(fromNorm)) {
-            throw new Error(`${specName} validation failed - when a rename exists, MODIFIED must reference the NEW header "### Requirement: ${to}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.renamedModifiedMustReferenceNew(specName, to));
         }
         // Detect ADDED colliding with a RENAMED TO
         if (addedNames.has(toNorm)) {
-            throw new Error(`${specName} validation failed - RENAMED TO header collides with ADDED for "### Requirement: ${to}"`);
+            throw new Error(SPECS_APPLY_MESSAGES.renamedToCollidesWithAdded(specName, to));
         }
     }
     if (conflicts.length > 0) {
         const c = conflicts[0];
-        throw new Error(`${specName} validation failed - requirement present in multiple sections (${c.a} and ${c.b}) for header "### Requirement: ${c.name}"`);
+        throw new Error(SPECS_APPLY_MESSAGES.requirementInMultipleSections(specName, c.a, c.b, c.name));
     }
     const hasAnyDelta = plan.added.length + plan.modified.length + plan.removed.length + plan.renamed.length > 0;
     if (!hasAnyDelta) {
-        throw new Error(`Delta parsing found no operations for ${path.basename(path.dirname(update.source))}. ` +
-            `Provide ADDED/MODIFIED/REMOVED/RENAMED sections in change spec.`);
+        throw new Error(SPECS_APPLY_MESSAGES.noDeltaOperations(path.basename(path.dirname(update.source))));
     }
     // Load or create base target content
     let targetContent;
@@ -146,7 +145,7 @@ export async function buildUpdatedSpec(update, changeName) {
         // Target spec does not exist; MODIFIED and RENAMED are not allowed for new specs
         // REMOVED will be ignored with a warning since there's nothing to remove
         if (plan.modified.length > 0 || plan.renamed.length > 0) {
-            throw new Error(`${specName}: target spec does not exist; only ADDED requirements are allowed for new specs. MODIFIED and RENAMED operations require an existing spec.`);
+            throw new Error(SPECS_APPLY_MESSAGES.targetSpecNotExists(specName));
         }
         // Warn about REMOVED requirements being ignored for new specs
         if (plan.removed.length > 0) {
@@ -160,7 +159,7 @@ export async function buildUpdatedSpec(update, changeName) {
         const details = structureIssues
             .map(issue => `line ${issue.line}: ${issue.message}`)
             .join('\n');
-        throw new Error(`${specName}: target spec is structurally invalid and cannot be updated until fixed:\n${details}`);
+        throw new Error(SPECS_APPLY_MESSAGES.targetSpecStructurallyInvalid(specName, details));
     }
     // Extract requirements section and build name->block map
     const parts = extractRequirementsSection(targetContent);
@@ -174,10 +173,10 @@ export async function buildUpdatedSpec(update, changeName) {
         const from = normalizeRequirementName(r.from);
         const to = normalizeRequirementName(r.to);
         if (!nameToBlock.has(from)) {
-            throw new Error(`${specName} RENAMED failed for header "### Requirement: ${r.from}" - source not found`);
+            throw new Error(SPECS_APPLY_MESSAGES.renamedFailedSourceNotFound(specName, r.from));
         }
         if (nameToBlock.has(to)) {
-            throw new Error(`${specName} RENAMED failed for header "### Requirement: ${r.to}" - target already exists`);
+            throw new Error(SPECS_APPLY_MESSAGES.renamedFailedTargetExists(specName, r.to));
         }
         const block = nameToBlock.get(from);
         const newHeader = `### Requirement: ${to}`;
@@ -198,7 +197,7 @@ export async function buildUpdatedSpec(update, changeName) {
             // For new specs, REMOVED requirements are already warned about and ignored
             // For existing specs, missing requirements are an error
             if (!isNewSpec) {
-                throw new Error(`${specName} REMOVED failed for header "### Requirement: ${name}" - not found`);
+                throw new Error(SPECS_APPLY_MESSAGES.removedFailedNotFound(specName, name));
             }
             // Skip removal for new specs (already warned above)
             continue;
@@ -209,12 +208,12 @@ export async function buildUpdatedSpec(update, changeName) {
     for (const mod of plan.modified) {
         const key = normalizeRequirementName(mod.name);
         if (!nameToBlock.has(key)) {
-            throw new Error(`${specName} MODIFIED failed for header "### Requirement: ${mod.name}" - not found`);
+            throw new Error(SPECS_APPLY_MESSAGES.modifiedFailedNotFound(specName, mod.name));
         }
         // Replace block with provided raw (ensure header line matches key)
         const modHeaderMatch = mod.raw.split('\n')[0].match(/^###\s*Requirement:\s*(.+)\s*$/);
         if (!modHeaderMatch || normalizeRequirementName(modHeaderMatch[1]) !== key) {
-            throw new Error(`${specName} MODIFIED failed for header "### Requirement: ${mod.name}" - header mismatch in content`);
+            throw new Error(SPECS_APPLY_MESSAGES.modifiedFailedHeaderMismatch(specName, mod.name));
         }
         nameToBlock.set(key, mod);
     }
@@ -222,7 +221,7 @@ export async function buildUpdatedSpec(update, changeName) {
     for (const add of plan.added) {
         const key = normalizeRequirementName(add.name);
         if (nameToBlock.has(key)) {
-            throw new Error(`${specName} ADDED failed for header "### Requirement: ${add.name}" - already exists`);
+            throw new Error(SPECS_APPLY_MESSAGES.addedFailedAlreadyExists(specName, add.name));
         }
         nameToBlock.set(key, add);
     }
@@ -272,22 +271,22 @@ export async function writeUpdatedSpec(update, rebuilt, counts) {
     await fs.mkdir(targetDir, { recursive: true });
     await fs.writeFile(update.target, rebuilt);
     const specName = path.basename(path.dirname(update.target));
-    console.log(`Applying changes to openspec/specs/${specName}/spec.md:`);
+    console.log(SPECS_APPLY_MESSAGES.applyingChangesTo(specName));
     if (counts.added)
-        console.log(`  + ${counts.added} added`);
+        console.log(SPECS_APPLY_MESSAGES.countAdded(counts.added));
     if (counts.modified)
-        console.log(`  ~ ${counts.modified} modified`);
+        console.log(SPECS_APPLY_MESSAGES.countModified(counts.modified));
     if (counts.removed)
-        console.log(`  - ${counts.removed} removed`);
+        console.log(SPECS_APPLY_MESSAGES.countRemoved(counts.removed));
     if (counts.renamed)
-        console.log(`  → ${counts.renamed} renamed`);
+        console.log(SPECS_APPLY_MESSAGES.countRenamed(counts.renamed));
 }
 /**
  * Build a skeleton spec for new capabilities.
  */
 export function buildSpecSkeleton(specFolderName, changeName) {
     const titleBase = specFolderName;
-    return `# ${titleBase} Specification\n\n## Purpose\nTBD - created by archiving change ${changeName}. Update Purpose after archive.\n\n## Requirements\n`;
+    return `# ${titleBase} Specification\n\n## Purpose\n${SPECS_APPLY_MESSAGES.skeletonPurpose(changeName)}\n\n## Requirements\n`;
 }
 /**
  * Apply all delta specs from a change to main specs.
@@ -304,11 +303,11 @@ export async function applySpecs(projectRoot, changeName, options = {}) {
     try {
         const stat = await fs.stat(changeDir);
         if (!stat.isDirectory()) {
-            throw new Error(`Change '${changeName}' not found.`);
+            throw new Error(SPECS_APPLY_MESSAGES.changeNotFound(changeName));
         }
     }
     catch {
-        throw new Error(`Change '${changeName}' not found.`);
+        throw new Error(SPECS_APPLY_MESSAGES.changeNotFound(changeName));
     }
     // Find specs to update
     const specUpdates = await findSpecUpdates(changeDir, mainSpecsDir);
@@ -337,7 +336,7 @@ export async function applySpecs(projectRoot, changeName, options = {}) {
                     .filter((i) => i.level === 'ERROR')
                     .map((i) => `  ✗ ${i.message}`)
                     .join('\n');
-                throw new Error(`Validation errors in rebuilt spec for ${specName}:\n${errors}`);
+                throw new Error(SPECS_APPLY_MESSAGES.validationErrorsInRebuiltSpec(specName, errors));
             }
         }
     }
@@ -352,27 +351,27 @@ export async function applySpecs(projectRoot, changeName, options = {}) {
             await fs.mkdir(targetDir, { recursive: true });
             await fs.writeFile(p.update.target, p.rebuilt);
             if (!options.silent) {
-                console.log(`Applying changes to openspec/specs/${capability}/spec.md:`);
+                console.log(SPECS_APPLY_MESSAGES.applyingChangesTo(capability));
                 if (p.counts.added)
-                    console.log(`  + ${p.counts.added} added`);
+                    console.log(SPECS_APPLY_MESSAGES.countAdded(p.counts.added));
                 if (p.counts.modified)
-                    console.log(`  ~ ${p.counts.modified} modified`);
+                    console.log(SPECS_APPLY_MESSAGES.countModified(p.counts.modified));
                 if (p.counts.removed)
-                    console.log(`  - ${p.counts.removed} removed`);
+                    console.log(SPECS_APPLY_MESSAGES.countRemoved(p.counts.removed));
                 if (p.counts.renamed)
-                    console.log(`  → ${p.counts.renamed} renamed`);
+                    console.log(SPECS_APPLY_MESSAGES.countRenamed(p.counts.renamed));
             }
         }
         else if (!options.silent) {
-            console.log(`Would apply changes to openspec/specs/${capability}/spec.md:`);
+            console.log(SPECS_APPLY_MESSAGES.wouldApplyChangesTo(capability));
             if (p.counts.added)
-                console.log(`  + ${p.counts.added} added`);
+                console.log(SPECS_APPLY_MESSAGES.countAdded(p.counts.added));
             if (p.counts.modified)
-                console.log(`  ~ ${p.counts.modified} modified`);
+                console.log(SPECS_APPLY_MESSAGES.countModified(p.counts.modified));
             if (p.counts.removed)
-                console.log(`  - ${p.counts.removed} removed`);
+                console.log(SPECS_APPLY_MESSAGES.countRemoved(p.counts.removed));
             if (p.counts.renamed)
-                console.log(`  → ${p.counts.renamed} renamed`);
+                console.log(SPECS_APPLY_MESSAGES.countRenamed(p.counts.renamed));
         }
         capabilities.push({
             capability,