@dynamicworks/br-openspec 2.0.0 → 2.1.0

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/parsers/requirement-blocks.js

@@ -1,7 +1,7 @@
 export function normalizeRequirementName(name) {
     return name.trim();
 }
-const REQUIREMENT_HEADER_REGEX = /^###\s*Requirement:\s*(.+)\s*$/;
+const REQUIREMENT_HEADER_REGEX = /^###\s*Requirement:\s*(.+)\s*$/i;
 /**
  * Extracts the Requirements section from a spec file and parses requirement blocks.
  */
@@ -37,7 +37,7 @@ export function extractRequirementsSection(content) {
     let cursor = 0;
     let preambleLines = [];
     // Collect preamble lines until first requirement header
-    while (cursor < sectionBodyLines.length && !/^###\s+Requirement:/.test(sectionBodyLines[cursor])) {
+    while (cursor < sectionBodyLines.length && !REQUIREMENT_HEADER_REGEX.test(sectionBodyLines[cursor])) {
         preambleLines.push(sectionBodyLines[cursor]);
         cursor++;
     }
@@ -54,7 +54,7 @@ export function extractRequirementsSection(content) {
         cursor++;
         // Gather lines until next requirement header or end of section
         const bodyLines = [headerLineCandidate];
-        while (cursor < sectionBodyLines.length && !/^###\s+Requirement:/.test(sectionBodyLines[cursor]) && !/^##\s+/.test(sectionBodyLines[cursor])) {
+        while (cursor < sectionBodyLines.length && !REQUIREMENT_HEADER_REGEX.test(sectionBodyLines[cursor]) && !/^##\s+/.test(sectionBodyLines[cursor])) {
             bodyLines.push(sectionBodyLines[cursor]);
             cursor++;
         }
@@ -136,7 +136,7 @@ function parseRequirementBlocksFromSection(sectionBody) {
     let i = 0;
     while (i < lines.length) {
         // Seek next requirement header
-        while (i < lines.length && !/^###\s+Requirement:/.test(lines[i]))
+        while (i < lines.length && !REQUIREMENT_HEADER_REGEX.test(lines[i]))
             i++;
         if (i >= lines.length)
             break;
@@ -149,7 +149,7 @@ function parseRequirementBlocksFromSection(sectionBody) {
         const name = normalizeRequirementName(m[1]);
         const buf = [headerLine];
         i++;
-        while (i < lines.length && !/^###\s+Requirement:/.test(lines[i]) && !/^##\s+/.test(lines[i])) {
+        while (i < lines.length && !REQUIREMENT_HEADER_REGEX.test(lines[i]) && !/^##\s+/.test(lines[i])) {
             buf.push(lines[i]);
             i++;
         }

package/dist/core/parsers/spec-structure.js

@@ -1,7 +1,7 @@
 const REQUIREMENTS_SECTION_HEADER = /^##\s+Requirements\s*$/i;
 const TOP_LEVEL_SECTION_HEADER = /^##\s+/;
 const DELTA_HEADER = /^##\s+(ADDED|MODIFIED|REMOVED|RENAMED)\s+Requirements\s*$/i;
-const REQUIREMENT_HEADER = /^###\s+Requirement:\s*(.+)\s*$/;
+const REQUIREMENT_HEADER = /^###\s+Requirement:\s*(.+)\s*$/i;
 export function findMainSpecStructureIssues(content) {
     const normalized = content.replace(/\r\n?/g, '\n');
     const stripped = stripFencedCodeBlocksPreservingLines(normalized);

package/dist/core/profile-sync-drift.js

@@ -17,6 +17,7 @@ export const WORKFLOW_TO_SKILL_DIR = {
     'archive': 'openspec-archive-change',
     'bulk-archive': 'openspec-bulk-archive-change',
     'verify': 'openspec-verify-change',
+    'code-review': 'openspec-code-review',
     'onboard': 'openspec-onboard',
     'propose': 'openspec-propose',
 };

package/dist/core/profiles.d.ts

@@ -9,11 +9,11 @@ import type { Profile } from './global-config.js';
  * Core workflows included in the 'core' profile.
  * These provide the streamlined experience for new users.
  */
-export declare const CORE_WORKFLOWS: readonly ["propose", "explore", "apply", "archive"];
+export declare const CORE_WORKFLOWS: readonly ["propose", "explore", "apply", "sync", "archive"];
 /**
  * All available workflows in the system.
  */
-export declare const ALL_WORKFLOWS: readonly ["propose", "explore", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard"];
+export declare const ALL_WORKFLOWS: readonly ["propose", "explore", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "code-review", "onboard"];
 export type WorkflowId = (typeof ALL_WORKFLOWS)[number];
 export type CoreWorkflowId = (typeof CORE_WORKFLOWS)[number];
 /**

package/dist/core/profiles.js

@@ -8,7 +8,7 @@
  * Core workflows included in the 'core' profile.
  * These provide the streamlined experience for new users.
  */
-export const CORE_WORKFLOWS = ['propose', 'explore', 'apply', 'archive'];
+export const CORE_WORKFLOWS = ['propose', 'explore', 'apply', 'sync', 'archive'];
 /**
  * All available workflows in the system.
  */
@@ -23,6 +23,7 @@ export const ALL_WORKFLOWS = [
     'archive',
     'bulk-archive',
     'verify',
+    'code-review',
     'onboard',
 ];
 /**

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

@@ -3,7 +3,7 @@
  *
  * Shared utilities for generating skill and command files.
  */
-import { getExploreSkillTemplate, getNewChangeSkillTemplate, getContinueChangeSkillTemplate, getApplyChangeSkillTemplate, getFfChangeSkillTemplate, getSyncSpecsSkillTemplate, getArchiveChangeSkillTemplate, getBulkArchiveChangeSkillTemplate, getVerifyChangeSkillTemplate, getOnboardSkillTemplate, getOpsxProposeSkillTemplate, getOpsxExploreCommandTemplate, getOpsxNewCommandTemplate, getOpsxContinueCommandTemplate, getOpsxApplyCommandTemplate, getOpsxFfCommandTemplate, getOpsxSyncCommandTemplate, getOpsxArchiveCommandTemplate, getOpsxBulkArchiveCommandTemplate, getOpsxVerifyCommandTemplate, getOpsxOnboardCommandTemplate, getOpsxProposeCommandTemplate, } from '../templates/skill-templates.js';
+import { getExploreSkillTemplate, getNewChangeSkillTemplate, getContinueChangeSkillTemplate, getApplyChangeSkillTemplate, getFfChangeSkillTemplate, getSyncSpecsSkillTemplate, getArchiveChangeSkillTemplate, getBulkArchiveChangeSkillTemplate, getVerifyChangeSkillTemplate, getCodeReviewSkillTemplate, getOnboardSkillTemplate, getOpsxProposeSkillTemplate, getOpsxExploreCommandTemplate, getOpsxNewCommandTemplate, getOpsxContinueCommandTemplate, getOpsxApplyCommandTemplate, getOpsxFfCommandTemplate, getOpsxSyncCommandTemplate, getOpsxArchiveCommandTemplate, getOpsxBulkArchiveCommandTemplate, getOpsxVerifyCommandTemplate, getOpsxCodeReviewCommandTemplate, getOpsxOnboardCommandTemplate, getOpsxProposeCommandTemplate, } from '../templates/skill-templates.js';
 /**
  * Gets skill templates with their directory names, optionally filtered by workflow IDs.
  *
@@ -20,6 +20,7 @@ export function getSkillTemplates(workflowFilter) {
         { template: getArchiveChangeSkillTemplate(), dirName: 'openspec-archive-change', workflowId: 'archive' },
         { template: getBulkArchiveChangeSkillTemplate(), dirName: 'openspec-bulk-archive-change', workflowId: 'bulk-archive' },
         { template: getVerifyChangeSkillTemplate(), dirName: 'openspec-verify-change', workflowId: 'verify' },
+        { template: getCodeReviewSkillTemplate(), dirName: 'openspec-code-review', workflowId: 'code-review' },
         { template: getOnboardSkillTemplate(), dirName: 'openspec-onboard', workflowId: 'onboard' },
         { template: getOpsxProposeSkillTemplate(), dirName: 'openspec-propose', workflowId: 'propose' },
     ];
@@ -44,6 +45,7 @@ export function getCommandTemplates(workflowFilter) {
         { template: getOpsxArchiveCommandTemplate(), id: 'archive' },
         { template: getOpsxBulkArchiveCommandTemplate(), id: 'bulk-archive' },
         { template: getOpsxVerifyCommandTemplate(), id: 'verify' },
+        { template: getOpsxCodeReviewCommandTemplate(), id: 'code-review' },
         { template: getOpsxOnboardCommandTemplate(), id: 'onboard' },
         { template: getOpsxProposeCommandTemplate(), id: 'propose' },
     ];

package/dist/core/shared/tool-detection.d.ts

@@ -6,12 +6,12 @@
 /**
  * Names of skill directories created by openspec init.
  */
-export declare const SKILL_NAMES: readonly ["openspec-explore", "openspec-new-change", "openspec-continue-change", "openspec-apply-change", "openspec-ff-change", "openspec-sync-specs", "openspec-archive-change", "openspec-bulk-archive-change", "openspec-verify-change", "openspec-onboard", "openspec-propose"];
+export declare const SKILL_NAMES: readonly ["openspec-explore", "openspec-new-change", "openspec-continue-change", "openspec-apply-change", "openspec-ff-change", "openspec-sync-specs", "openspec-archive-change", "openspec-bulk-archive-change", "openspec-verify-change", "openspec-code-review", "openspec-onboard", "openspec-propose"];
 export type SkillName = (typeof SKILL_NAMES)[number];
 /**
  * IDs of command templates created by openspec init.
  */
-export declare const COMMAND_IDS: readonly ["explore", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard", "propose"];
+export declare const COMMAND_IDS: readonly ["explore", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "code-review", "onboard", "propose"];
 export type CommandId = (typeof COMMAND_IDS)[number];
 /**
  * Status of skill configuration for a tool.

package/dist/core/shared/tool-detection.js

@@ -19,6 +19,7 @@ export const SKILL_NAMES = [
     'openspec-archive-change',
     'openspec-bulk-archive-change',
     'openspec-verify-change',
+    'openspec-code-review',
     'openspec-onboard',
     'openspec-propose',
 ];
@@ -35,6 +36,7 @@ export const COMMAND_IDS = [
     'archive',
     'bulk-archive',
     'verify',
+    'code-review',
     'onboard',
     'propose',
 ];

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*$/);
+        const modHeaderMatch = mod.raw.split('\n')[0].match(/^###\s*Requirement:\s*(.+)\s*$/i);
         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,

package/dist/core/templates/skill-templates.d.ts

@@ -13,8 +13,8 @@ export { getSyncSpecsSkillTemplate, getOpsxSyncCommandTemplate } from './workflo
 export { getArchiveChangeSkillTemplate, getOpsxArchiveCommandTemplate } from './workflows/archive-change.js';
 export { getBulkArchiveChangeSkillTemplate, getOpsxBulkArchiveCommandTemplate } from './workflows/bulk-archive-change.js';
 export { getVerifyChangeSkillTemplate, getOpsxVerifyCommandTemplate } from './workflows/verify-change.js';
+export { getCodeReviewSkillTemplate, getOpsxCodeReviewCommandTemplate } from './workflows/code-review.js';
 export { getOnboardSkillTemplate, getOpsxOnboardCommandTemplate } from './workflows/onboard.js';
 export { getOpsxProposeSkillTemplate, getOpsxProposeCommandTemplate } from './workflows/propose.js';
 export { getFeedbackSkillTemplate } from './workflows/feedback.js';
-export { getUpstreamSyncSkillTemplate, getOpsxUpstreamSyncCommandTemplate } from './workflows/upstream-sync.js';
 //# sourceMappingURL=skill-templates.d.ts.map

package/dist/core/templates/skill-templates.js

@@ -12,8 +12,8 @@ export { getSyncSpecsSkillTemplate, getOpsxSyncCommandTemplate } from './workflo
 export { getArchiveChangeSkillTemplate, getOpsxArchiveCommandTemplate } from './workflows/archive-change.js';
 export { getBulkArchiveChangeSkillTemplate, getOpsxBulkArchiveCommandTemplate } from './workflows/bulk-archive-change.js';
 export { getVerifyChangeSkillTemplate, getOpsxVerifyCommandTemplate } from './workflows/verify-change.js';
+export { getCodeReviewSkillTemplate, getOpsxCodeReviewCommandTemplate } from './workflows/code-review.js';
 export { getOnboardSkillTemplate, getOpsxOnboardCommandTemplate } from './workflows/onboard.js';
 export { getOpsxProposeSkillTemplate, getOpsxProposeCommandTemplate } from './workflows/propose.js';
 export { getFeedbackSkillTemplate } from './workflows/feedback.js';
-export { getUpstreamSyncSkillTemplate, getOpsxUpstreamSyncCommandTemplate } from './workflows/upstream-sync.js';
 //# sourceMappingURL=skill-templates.js.map

package/dist/core/templates/workflows/code-review.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Skill Template Workflow Modules
+ *
+ * This file is generated by splitting the legacy monolithic
+ * templates file into workflow-focused modules.
+ */
+import type { SkillTemplate, CommandTemplate } from '../types.js';
+export declare function getCodeReviewSkillTemplate(): SkillTemplate;
+export declare function getOpsxCodeReviewCommandTemplate(): CommandTemplate;
+//# sourceMappingURL=code-review.d.ts.map

package/dist/core/templates/workflows/code-review.js

@@ -0,0 +1,21 @@
+import { CODE_REVIEW_TEMPLATE_MESSAGES } from '../../../messages/index.js';
+export function getCodeReviewSkillTemplate() {
+    return {
+        name: 'openspec-code-review',
+        description: CODE_REVIEW_TEMPLATE_MESSAGES.skillDescription,
+        instructions: CODE_REVIEW_TEMPLATE_MESSAGES.instructions,
+        license: 'MIT',
+        compatibility: CODE_REVIEW_TEMPLATE_MESSAGES.skillCompatibility,
+        metadata: { author: 'openspec', version: '1.0' },
+    };
+}
+export function getOpsxCodeReviewCommandTemplate() {
+    return {
+        name: 'OPSX: Code Review',
+        description: CODE_REVIEW_TEMPLATE_MESSAGES.opsxDescription,
+        category: 'Workflow',
+        tags: ['workflow', 'review', 'code-review', 'experimental'],
+        content: CODE_REVIEW_TEMPLATE_MESSAGES.instructions,
+    };
+}
+//# sourceMappingURL=code-review.js.map

package/dist/core/templates/workflows/sync-specs.js

@@ -75,7 +75,7 @@ Esta é uma operação **dirigida por agente** — você lerá os delta specs e
 ## ADDED Requirements
 
 ### Requirement: New Feature
-O sistema DEVERÁ fazer algo novo.
+The system SHALL do something new.
 
 #### Scenario: Basic case
 - **WHEN** user does X
@@ -213,7 +213,7 @@ Esta é uma operação **dirigida por agente** — você lerá os delta specs e
 ## ADDED Requirements
 
 ### Requirement: New Feature
-O sistema DEVERÁ fazer algo novo.
+The system SHALL do something new.
 
 #### Scenario: Basic case
 - **WHEN** user does X

package/dist/core/tools-manager.js

@@ -4,7 +4,7 @@
  * Encapsulates adding and removing IDE/Code Agent OpenSpec configuration files.
  * Shared by `openspec init` (via InitCommand) and `openspec tools`.
  */
-import { TOOLS_MESSAGES } from '../messages/index.js';
+import { TOOLS_MESSAGES, TOOLS_MANAGER_MESSAGES } from '../messages/index.js';
 import path from 'path';
 import * as fs from 'fs';
 import { createRequire } from 'module';
@@ -29,6 +29,7 @@ const WORKFLOW_TO_SKILL_DIR = {
     archive: 'openspec-archive-change',
     'bulk-archive': 'openspec-bulk-archive-change',
     verify: 'openspec-verify-change',
+    'code-review': 'openspec-code-review',
     onboard: 'openspec-onboard',
     propose: 'openspec-propose',
 };
@@ -100,7 +101,7 @@ export async function removeOpenSpecCommandFiles(projectPath, toolId) {
  */
 export async function addTool(projectPath, tool) {
     if (!tool.skillsDir) {
-        throw new Error(`Tool '${tool.value}' does not support skill generation.`);
+        throw new Error(TOOLS_MANAGER_MESSAGES.toolDoesNotSupportSkills(tool.value));
     }
     const globalConfig = getGlobalConfig();
     const profile = globalConfig.profile ?? 'core';

package/dist/core/update.d.ts

@@ -38,6 +38,12 @@ export declare class UpdateCommand {
      * Displays a note about extra workflows installed that aren't in the current profile.
      */
     private displayExtraWorkflowsNote;
+    /**
+     * Sugere voltar ao perfil core quando um perfil personalizado ainda
+     * corresponde ao conjunto core antigo (anterior ao sync). Mantém os perfis
+     * personalizados sob controle do usuário; não os altera.
+     */
+    private displayOldCoreCustomProfileNote;
     /**
      * Removes skill directories for workflows when delivery changed to commands-only.
      * Returns the number of directories removed.