@codewalla_india/openspec 1.3.1 → 1.3.2

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

@@ -3,7 +3,12 @@ import * as fs from 'fs';
 import { AI_TOOLS } from './config.js';
 import { ALL_WORKFLOWS } from './profiles.js';
 import { CommandAdapterRegistry } from './command-generation/index.js';
-import { COMMAND_IDS, getConfiguredTools } from './shared/index.js';
+import { getConfiguredTools } from './shared/index.js';
+import { shouldGenerateCommandsForTool, shouldGenerateSkillsForTool, shouldReconcileCommandFilesForTool, shouldRemoveSkillsForTool, } from './command-surface.js';
+import { readSharedSkillTarget } from './shared-skill-target.js';
+import { FileSystemUtils } from '../utils/file-system.js';
+import { isLegacyCodexSkillEquivalentToCurrent } from './shared/skill-content-equivalence.js';
+import { hasGlobalSkillTarget, resolveToolSkillsDir, toolSupportsSkills, } from './shared/skill-paths.js';
 /**
  * Maps workflow IDs to their skill directory names.
  */
@@ -11,8 +16,8 @@ export const WORKFLOW_TO_SKILL_DIR = {
     'explore': 'openspec-explore',
     'new': 'openspec-new-change',
     'continue': 'openspec-continue-change',
-    'modify': 'openspec-modify-change',
     'apply': 'openspec-apply-change',
+    'update': 'openspec-update-change',
     'ff': 'openspec-ff-change',
     'sync': 'openspec-sync-specs',
     'archive': 'openspec-archive-change',
@@ -24,48 +29,11 @@ export const WORKFLOW_TO_SKILL_DIR = {
 function toKnownWorkflows(workflows) {
     return workflows.filter((workflow) => ALL_WORKFLOWS.includes(workflow));
 }
-/**
- * Checks whether a tool has at least one generated OpenSpec command file.
- */
-export function toolHasAnyConfiguredCommand(projectPath, toolId) {
-    const adapter = CommandAdapterRegistry.get(toolId);
-    if (!adapter)
-        return false;
-    for (const commandId of COMMAND_IDS) {
-        const cmdPath = adapter.getFilePath(commandId);
-        const fullPath = path.isAbsolute(cmdPath) ? cmdPath : path.join(projectPath, cmdPath);
-        if (fs.existsSync(fullPath)) {
-            return true;
-        }
-    }
-    return false;
-}
-/**
- * Returns tools with at least one generated command file on disk.
- */
-export function getCommandConfiguredTools(projectPath) {
-    return AI_TOOLS
-        .filter((tool) => {
-        if (!tool.skillsDir)
-            return false;
-        const toolDir = path.join(projectPath, tool.skillsDir);
-        try {
-            return fs.statSync(toolDir).isDirectory();
-        }
-        catch {
-            return false;
-        }
-    })
-        .map((tool) => tool.value)
-        .filter((toolId) => toolHasAnyConfiguredCommand(projectPath, toolId));
-}
 /**
  * Returns tools that are configured via either skills or commands.
  */
 export function getConfiguredToolsForProfileSync(projectPath) {
-    const skillConfigured = getConfiguredTools(projectPath);
-    const commandConfigured = getCommandConfiguredTools(projectPath);
-    return [...new Set([...skillConfigured, ...commandConfigured])];
+    return getConfiguredTools(projectPath);
 }
 /**
  * Detects if a single tool has profile/delivery drift against the desired state.
@@ -77,14 +45,45 @@ export function getConfiguredToolsForProfileSync(projectPath) {
  */
 export function hasToolProfileOrDeliveryDrift(projectPath, toolId, desiredWorkflows, delivery) {
     const tool = AI_TOOLS.find((t) => t.value === toolId);
-    if (!tool?.skillsDir)
+    if (!tool || !toolSupportsSkills(tool))
         return false;
     const knownDesiredWorkflows = toKnownWorkflows(desiredWorkflows);
     const desiredWorkflowSet = new Set(knownDesiredWorkflows);
-    const skillsDir = path.join(projectPath, tool.skillsDir, 'skills');
+    const skillsDir = resolveToolSkillsDir(projectPath, tool);
     const adapter = CommandAdapterRegistry.get(toolId);
-    const shouldGenerateSkills = delivery !== 'commands';
-    const shouldGenerateCommands = delivery !== 'skills';
+    const shouldGenerateSkills = shouldGenerateSkillsForTool(toolId, delivery);
+    const shouldGenerateCommands = shouldGenerateCommandsForTool(toolId, delivery);
+    const sharedTarget = tool.skillsDir
+        ? readSharedSkillTarget(projectPath, tool.skillsDir)
+        : undefined;
+    for (const root of tool.legacySkillsDirs ?? []) {
+        for (const workflow of knownDesiredWorkflows) {
+            const dirName = WORKFLOW_TO_SKILL_DIR[workflow];
+            const legacySkill = path.join(projectPath, root, 'skills', dirName, 'SKILL.md');
+            if (!fs.existsSync(legacySkill))
+                continue;
+            const currentSkill = path.join(skillsDir, dirName, 'SKILL.md');
+            if (!fs.existsSync(currentSkill) || sharedTarget !== toolId) {
+                return true;
+            }
+            try {
+                if (FileSystemUtils.canonicalizeExistingPath(legacySkill) ===
+                    FileSystemUtils.canonicalizeExistingPath(currentSkill)) {
+                    continue;
+                }
+                // Equivalent generated copies are actionable: migration can safely
+                // remove the redundant legacy file even when version, line endings,
+                // or supported invocation syntax changed. Materially divergent copies
+                // stay in place without forcing an update on every run.
+                if (isLegacyCodexSkillEquivalentToCurrent(fs.readFileSync(legacySkill, 'utf-8'), fs.readFileSync(currentSkill, 'utf-8'))) {
+                    return true;
+                }
+            }
+            catch {
+                return true;
+            }
+        }
+    }
     if (shouldGenerateSkills) {
         for (const workflow of knownDesiredWorkflows) {
             const dirName = WORKFLOW_TO_SKILL_DIR[workflow];
@@ -104,7 +103,7 @@ export function hasToolProfileOrDeliveryDrift(projectPath, toolId, desiredWorkfl
             }
         }
     }
-    else {
+    else if (shouldRemoveSkillsForTool(toolId, delivery) && !hasGlobalSkillTarget(tool)) {
         for (const workflow of ALL_WORKFLOWS) {
             const dirName = WORKFLOW_TO_SKILL_DIR[workflow];
             const skillDir = path.join(skillsDir, dirName);
@@ -132,7 +131,7 @@ export function hasToolProfileOrDeliveryDrift(projectPath, toolId, desiredWorkfl
             }
         }
     }
-    else if (!shouldGenerateCommands && adapter) {
+    else if (shouldReconcileCommandFilesForTool(toolId, delivery) && adapter) {
         for (const workflow of ALL_WORKFLOWS) {
             const cmdPath = adapter.getFilePath(workflow);
             const fullPath = path.isAbsolute(cmdPath) ? cmdPath : path.join(projectPath, cmdPath);
@@ -152,10 +151,10 @@ export function getToolsNeedingProfileSync(projectPath, desiredWorkflows, delive
 }
 function getInstalledWorkflowsForTool(projectPath, toolId, options) {
     const tool = AI_TOOLS.find((t) => t.value === toolId);
-    if (!tool?.skillsDir)
+    if (!tool || !toolSupportsSkills(tool))
         return [];
     const installed = new Set();
-    const skillsDir = path.join(projectPath, tool.skillsDir, 'skills');
+    const skillsDir = resolveToolSkillsDir(projectPath, tool);
     if (options.includeSkills) {
         for (const workflow of ALL_WORKFLOWS) {
             const dirName = WORKFLOW_TO_SKILL_DIR[workflow];
@@ -188,9 +187,9 @@ export function hasProjectConfigDrift(projectPath, desiredWorkflows, delivery) {
         return true;
     }
     const desiredSet = new Set(toKnownWorkflows(desiredWorkflows));
-    const includeSkills = delivery !== 'commands';
-    const includeCommands = delivery !== 'skills';
     for (const toolId of configuredTools) {
+        const includeSkills = shouldGenerateSkillsForTool(toolId, delivery);
+        const includeCommands = shouldGenerateCommandsForTool(toolId, delivery);
         const installed = getInstalledWorkflowsForTool(projectPath, toolId, { includeSkills, includeCommands });
         if (installed.some((workflow) => !desiredSet.has(workflow))) {
             return true;

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", "modify", "apply", "sync", "archive"];
+export declare const CORE_WORKFLOWS: readonly ["propose", "explore", "apply", "update", "sync", "archive"];
 /**
  * All available workflows in the system.
  */
-export declare const ALL_WORKFLOWS: readonly ["propose", "explore", "modify", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard"];
+export declare const ALL_WORKFLOWS: readonly ["propose", "explore", "new", "continue", "apply", "update", "ff", "sync", "archive", "bulk-archive", "verify", "onboard"];
 export type WorkflowId = (typeof ALL_WORKFLOWS)[number];
 export type CoreWorkflowId = (typeof CORE_WORKFLOWS)[number];
 /**

package/dist/core/profiles.js

@@ -8,17 +8,17 @@
  * Core workflows included in the 'core' profile.
  * These provide the streamlined experience for new users.
  */
-export const CORE_WORKFLOWS = ['propose', 'explore', 'modify', 'apply', 'sync', 'archive'];
+export const CORE_WORKFLOWS = ['propose', 'explore', 'apply', 'update', 'sync', 'archive'];
 /**
  * All available workflows in the system.
  */
 export const ALL_WORKFLOWS = [
     'propose',
     'explore',
-    'modify',
     'new',
     'continue',
     'apply',
+    'update',
     'ff',
     'sync',
     'archive',

package/dist/core/project-config.d.ts

@@ -1,4 +1,10 @@
 import { z } from 'zod';
+export declare const OPERATION_IDS: readonly ["apply", "archive"];
+export type OperationId = (typeof OPERATION_IDS)[number];
+export interface OperationConfig {
+    guidance?: string[];
+}
+export type OperationsConfig = Partial<Record<OperationId, OperationConfig>>;
 /**
  * Zod schema for project configuration.
  *
@@ -16,12 +22,17 @@ export declare const ProjectConfigSchema: z.ZodObject<{
     schema: z.ZodString;
     context: z.ZodOptional<z.ZodString>;
     rules: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodArray<z.ZodString>>>;
+    operations: z.ZodOptional<z.ZodObject<{
+        apply: z.ZodOptional<z.ZodObject<{
+            guidance: z.ZodOptional<z.ZodArray<z.ZodString>>;
+        }, z.core.$strip>>;
+        archive: z.ZodOptional<z.ZodObject<{
+            guidance: z.ZodOptional<z.ZodArray<z.ZodString>>;
+        }, z.core.$strip>>;
+    }, z.core.$strip>>;
     store: z.ZodOptional<z.ZodString>;
-    comprehension: z.ZodOptional<z.ZodObject<{
-        enabled: z.ZodOptional<z.ZodBoolean>;
-        threshold_percent: z.ZodOptional<z.ZodNumber>;
-        min_questions: z.ZodOptional<z.ZodNumber>;
-        max_questions: z.ZodOptional<z.ZodNumber>;
+    githubCopilot: z.ZodOptional<z.ZodObject<{
+        cloudAgent: z.ZodOptional<z.ZodBoolean>;
     }, z.core.$strip>>;
 }, z.core.$strip>;
 /** Normalized in-memory shape of a referenced store declaration. */
@@ -32,14 +43,12 @@ export interface DeclarationEntry {
 }
 export type ProjectConfig = z.infer<typeof ProjectConfigSchema> & {
     references?: DeclarationEntry[];
-    /** Normalized comprehension settings (camelCase in memory). */
-    comprehension?: {
-        enabled?: boolean;
-        thresholdPercent?: number;
-        minQuestions?: number;
-        maxQuestions?: number;
-    };
 };
+export interface OperationInputs {
+    context?: string;
+    operationGuidance?: string[];
+}
+export declare function loadOperationInputs(projectConfig: ProjectConfig | null, operationId: OperationId): OperationInputs;
 export declare const MAX_CONTEXT_SIZE: number;
 /**
  * Read and parse openspec/config.yaml from project root.
@@ -62,16 +71,16 @@ export declare const MAX_CONTEXT_SIZE: number;
  */
 export declare function readProjectConfig(projectRoot: string): ProjectConfig | null;
 /**
- * Validate artifact IDs in rules against a schema's artifacts.
- * Called during instruction loading (when schema is known).
- * Returns warnings for unknown artifact IDs.
+ * Validate artifact IDs in rules against the artifacts of every available
+ * schema. The `rules:` map is global, but each change can use a different
+ * schema, so a key is only unknown when it matches no artifact in ANY schema.
+ * Returns warnings for keys that are unknown everywhere.
  *
  * @param rules - The rules object from config
- * @param validArtifactIds - Set of valid artifact IDs from the schema
- * @param schemaName - Name of the schema for error messages
+ * @param validArtifactIds - Set of valid artifact IDs across all schemas
  * @returns Array of warning messages for unknown artifact IDs
  */
-export declare function validateConfigRules(rules: Record<string, string[]>, validArtifactIds: Set<string>, schemaName: string): string[];
+export declare function validateConfigRules(rules: Record<string, string[]>, validArtifactIds: Set<string>): string[];
 /**
  * Suggest valid schema names when user provides invalid schema.
  * Uses fuzzy matching to find similar names.

package/dist/core/project-config.js

@@ -2,6 +2,10 @@ import { existsSync, readFileSync, statSync } from 'fs';
 import path from 'path';
 import { parse as parseYaml } from 'yaml';
 import { z } from 'zod';
+export const OPERATION_IDS = ['apply', 'archive'];
+const OperationConfigSchema = z.object({
+    guidance: z.array(z.string()).optional(),
+});
 /**
  * Zod schema for project configuration.
  *
@@ -34,6 +38,14 @@ export const ProjectConfigSchema = z.object({
     )
         .optional()
         .describe('Per-artifact rules, keyed by artifact ID'),
+    // Optional: per-operation advisory guidance, kept separate from artifact rules.
+    operations: z
+        .object({
+        apply: OperationConfigSchema.optional(),
+        archive: OperationConfigSchema.optional(),
+    })
+        .optional()
+        .describe('Per-operation advisory guidance'),
     // Note: the `references` field (id strings or {id, remote} maps) is
     // deliberately absent here — readProjectConfig parses and normalizes
     // it by hand (see DeclarationEntry below); a schema entry nothing
@@ -45,17 +57,70 @@ export const ProjectConfigSchema = z.object({
         .string()
         .optional()
         .describe('Store id used as the OpenSpec root when no local planning shape exists'),
-    // Optional: apply comprehension quiz gate (default enabled when absent)
-    comprehension: z
+    // Optional: GitHub Copilot integration preferences. `cloudAgent` is the
+    // opt-in for generating the Copilot cloud coding-agent files (a GitHub
+    // Actions workflow + agent file); absent means "not yet decided".
+    githubCopilot: z
         .object({
-        enabled: z.boolean().optional(),
-        threshold_percent: z.number().int().min(0).max(100).optional(),
-        min_questions: z.number().int().positive().optional(),
-        max_questions: z.number().int().positive().optional(),
+        cloudAgent: z.boolean().optional(),
     })
         .optional()
-        .describe('Comprehension quiz gate before apply'),
+        .describe('GitHub Copilot integration preferences'),
 });
+export function loadOperationInputs(projectConfig, operationId) {
+    const context = projectConfig?.context !== undefined && projectConfig.context.trim().length > 0
+        ? projectConfig.context
+        : undefined;
+    const guidance = projectConfig?.operations?.[operationId]?.guidance;
+    const operationGuidance = guidance && guidance.length > 0 ? guidance : undefined;
+    return {
+        ...(context !== undefined ? { context } : {}),
+        ...(operationGuidance !== undefined ? { operationGuidance } : {}),
+    };
+}
+function parseOperations(raw) {
+    if (raw === undefined) {
+        return undefined;
+    }
+    if (!raw || typeof raw !== 'object' || Array.isArray(raw)) {
+        console.warn(`Invalid 'operations' field in config (must be object)`);
+        return undefined;
+    }
+    const supported = new Set(OPERATION_IDS);
+    const operations = {};
+    for (const [operationId, value] of Object.entries(raw)) {
+        if (!supported.has(operationId)) {
+            console.warn(`Unknown operation ID '${operationId}' in config. Supported operation IDs: ${OPERATION_IDS.join(', ')}`);
+            continue;
+        }
+        const typedOperationId = operationId;
+        if (!value || typeof value !== 'object' || Array.isArray(value)) {
+            console.warn(`Invalid 'operations.${operationId}' field in config (must be object), ignoring this operation`);
+            continue;
+        }
+        const operation = value;
+        const unknownFields = Object.keys(operation).filter((field) => field !== 'guidance');
+        if (unknownFields.length > 0) {
+            console.warn(`Unknown field(s) in 'operations.${operationId}': ${unknownFields.join(', ')}. Supported fields: guidance`);
+        }
+        if (operation.guidance === undefined) {
+            continue;
+        }
+        const guidanceResult = z.array(z.string()).safeParse(operation.guidance);
+        if (!guidanceResult.success) {
+            console.warn(`Guidance for operation '${operationId}' must be an array of strings, ignoring this operation's guidance`);
+            continue;
+        }
+        const guidance = guidanceResult.data.filter((entry) => entry.length > 0);
+        if (guidance.length < guidanceResult.data.length) {
+            console.warn(`Some guidance for operation '${operationId}' are empty strings, ignoring them`);
+        }
+        if (guidance.length > 0) {
+            operations[typedOperationId] = { guidance };
+        }
+    }
+    return Object.keys(operations).length > 0 ? operations : undefined;
+}
 /**
  * Parser for `references:` declarations: string entries or
  * {id, remote} maps, normalized to DeclarationEntry[]. Dedup keys on
@@ -179,7 +244,11 @@ export function readProjectConfig(projectRoot) {
             const rulesField = z.record(z.string(), z.array(z.string()));
             // First check if it's an object structure (guard against null since typeof null === 'object')
             if (typeof raw.rules === 'object' && raw.rules !== null && !Array.isArray(raw.rules)) {
-                const parsedRules = {};
+                // Artifact ids are intentionally not restricted to the built-in naming
+                // convention, so keys such as "constructor" remain valid for custom
+                // schemas. A null-prototype map preserves those keys as data without
+                // letting "__proto__" mutate the lookup object's prototype.
+                const parsedRules = Object.create(null);
                 let hasValidRules = false;
                 for (const [artifactId, rules] of Object.entries(raw.rules)) {
                     const rulesArrayResult = z.array(z.string()).safeParse(rules);
@@ -206,6 +275,10 @@ export function readProjectConfig(projectRoot) {
                 console.warn(`Invalid 'rules' field in config (must be object)`);
             }
         }
+        const operations = parseOperations(raw.operations);
+        if (operations) {
+            config.operations = operations;
+        }
         const references = parseDeclarationList(raw.references);
         if (references) {
             config.references = references;
@@ -221,27 +294,21 @@ export function readProjectConfig(projectRoot) {
                 console.warn(`Warning: ignoring invalid store: field in ${configPathForWarnings(projectRoot)} (must be a single store id string).`);
             }
         }
-        if (raw.comprehension !== undefined) {
-            const comprehensionField = z.object({
-                enabled: z.boolean().optional(),
-                threshold_percent: z.number().int().min(0).max(100).optional(),
-                min_questions: z.number().int().positive().optional(),
-                max_questions: z.number().int().positive().optional(),
-            });
-            const comprehensionResult = comprehensionField.safeParse(raw.comprehension);
-            if (comprehensionResult.success) {
-                const c = comprehensionResult.data;
-                config.comprehension = {
-                    ...(c.enabled !== undefined ? { enabled: c.enabled } : {}),
-                    ...(c.threshold_percent !== undefined
-                        ? { thresholdPercent: c.threshold_percent }
-                        : {}),
-                    ...(c.min_questions !== undefined ? { minQuestions: c.min_questions } : {}),
-                    ...(c.max_questions !== undefined ? { maxQuestions: c.max_questions } : {}),
-                };
+        // Parse githubCopilot preferences (only cloudAgent is recognized today).
+        if (raw.githubCopilot !== undefined) {
+            if (typeof raw.githubCopilot === 'object' &&
+                raw.githubCopilot !== null &&
+                !Array.isArray(raw.githubCopilot)) {
+                const cloudAgent = raw.githubCopilot.cloudAgent;
+                if (typeof cloudAgent === 'boolean') {
+                    config.githubCopilot = { cloudAgent };
+                }
+                else if (cloudAgent !== undefined) {
+                    console.warn(`Invalid 'githubCopilot.cloudAgent' field in config (must be a boolean)`);
+                }
             }
             else {
-                console.warn(`Invalid 'comprehension' field in config (must be an object)`);
+                console.warn(`Invalid 'githubCopilot' field in config (must be an object)`);
             }
         }
         // Return partial config even if some fields failed
@@ -256,22 +323,22 @@ function configPathForWarnings(projectRoot) {
     return resolveConfigFilePath(projectRoot) ?? path.join(projectRoot, 'openspec', 'config.yaml');
 }
 /**
- * Validate artifact IDs in rules against a schema's artifacts.
- * Called during instruction loading (when schema is known).
- * Returns warnings for unknown artifact IDs.
+ * Validate artifact IDs in rules against the artifacts of every available
+ * schema. The `rules:` map is global, but each change can use a different
+ * schema, so a key is only unknown when it matches no artifact in ANY schema.
+ * Returns warnings for keys that are unknown everywhere.
  *
  * @param rules - The rules object from config
- * @param validArtifactIds - Set of valid artifact IDs from the schema
- * @param schemaName - Name of the schema for error messages
+ * @param validArtifactIds - Set of valid artifact IDs across all schemas
  * @returns Array of warning messages for unknown artifact IDs
  */
-export function validateConfigRules(rules, validArtifactIds, schemaName) {
+export function validateConfigRules(rules, validArtifactIds) {
     const warnings = [];
     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}`);
+                `It matches no artifact in any available schema. Known artifact IDs: ${validIds}`);
         }
     }
     return warnings;

package/dist/core/references.js

@@ -53,6 +53,46 @@ function registerFix(id, remote) {
     }
     return `Get a checkout from a teammate and run: openspec store register <path> --id ${id}`;
 }
+const WHITESPACE = /\s/;
+/**
+ * Drop a CommonMark closing sequence (`## Purpose ##`). The closing run only
+ * counts when whitespace precedes it, so `Purpose###` keeps its hashes.
+ * Scans from the end so the cost stays linear in the title length.
+ */
+function stripClosingSequence(title) {
+    let end = title.length;
+    while (end > 0 && WHITESPACE.test(title[end - 1])) {
+        end--;
+    }
+    const hashEnd = end;
+    while (end > 0 && title[end - 1] === '#') {
+        end--;
+    }
+    const noClosingRun = end === hashEnd;
+    const missingLeadingSpace = end === 0 || !WHITESPACE.test(title[end - 1]);
+    if (noClosingRun || missingLeadingSpace) {
+        return title.trim();
+    }
+    return title.slice(0, end).trim();
+}
+/**
+ * Heading title, or null when the line is not an ATX heading. Hand-rolled
+ * rather than a regex so a title padded with whitespace cannot backtrack.
+ */
+function parseHeadingTitle(line) {
+    let level = 0;
+    while (level < 6 && level < line.length && line[level] === '#') {
+        level++;
+    }
+    if (level === 0 || level >= line.length || !WHITESPACE.test(line[level])) {
+        return null;
+    }
+    let start = level;
+    while (start < line.length && WHITESPACE.test(line[start])) {
+        start++;
+    }
+    return stripClosingSequence(line.slice(start));
+}
 /**
  * Tolerant first-Purpose-line extraction. parseSpec() throws on specs
  * without Purpose/Requirements sections; the index must never fail on an
@@ -79,12 +119,11 @@ export function extractFirstPurposeLine(markdown) {
         if (fenceMarker !== null) {
             continue;
         }
-        const heading = line.match(/^(#{1,6})\s+(.*)$/);
-        if (heading) {
+        const title = parseHeadingTitle(line);
+        if (title !== null) {
             if (inPurpose) {
                 return '';
             }
-            const title = heading[2].replace(/\s+#+\s*$/, '').trim();
             inPurpose = title.toLowerCase() === 'purpose';
             continue;
         }
@@ -152,7 +191,6 @@ export function renderReferencedStoresSection(entries) {
  * let hostile content forge instruction lines (slice 6.1 hardening).
  */
 export function sanitizeInline(value, maxLength = 300) {
-    // eslint-disable-next-line no-control-regex
     const flattened = value.replace(/[\u0000-\u001f\u007f]+/g, ' ').trim();
     return flattened.length > maxLength ? `${flattened.slice(0, maxLength)}…` : flattened;
 }

package/dist/core/relationship-health.d.ts

@@ -26,6 +26,10 @@ export interface RelationshipHealth {
             remote?: string;
         };
         origin_url?: string;
+        drift?: {
+            ahead: number;
+            behind: number;
+        };
         status: StoreDiagnostic[];
     } | null;
     references: ReferenceIndexEntry[];
@@ -42,6 +46,10 @@ export interface InspectRelationshipsInput {
         metadataValid: boolean;
         canonicalRemote?: string;
         originUrl?: string;
+        drift?: {
+            ahead: number;
+            behind: number;
+        };
     };
     referenceEntries: ReferenceIndexEntry[];
     registryUnreadable: boolean;

package/dist/core/relationship-health.js

@@ -37,6 +37,18 @@ export function inspectRelationships(input) {
             input.storeFacts.canonicalRemote !== input.storeFacts.originUrl) {
             storeStatus.push(makeStoreDiagnostic('info', 'store_remote_divergence', `The store.yaml remote (${sanitizeInline(input.storeFacts.canonicalRemote, 200)}) differs from the checkout's origin (${sanitizeInline(input.storeFacts.originUrl, 200)}).`, { target: 'store.metadata' }));
         }
+        // Checkout behind its upstream tracking ref: a read-only staleness
+        // signal, not a version pin — OpenSpec never syncs stores, so this
+        // compares against the local upstream ref, not the live remote.
+        // Behind means teammates on newer commits may resolve different specs.
+        // Ahead-only is normal (OpenSpec never pushes stores), so it stays quiet.
+        const drift = input.storeFacts.drift;
+        if (drift && drift.behind > 0) {
+            const behindCommits = `${drift.behind} commit${drift.behind === 1 ? '' : 's'}`;
+            storeStatus.push(makeStoreDiagnostic('info', 'store_checkout_drift', drift.ahead > 0
+                ? `This store checkout has diverged from its upstream tracking branch (${drift.behind} behind, ${drift.ahead} ahead); teammates on newer commits may resolve different specs.`
+                : `This store checkout is ${behindCommits} behind its upstream tracking branch; teammates on newer commits may resolve different specs.`, { target: 'store.git' }));
+        }
         store = {
             id: input.storeFacts.id,
             metadata: {
@@ -47,6 +59,7 @@ export function inspectRelationships(input) {
                     : {}),
             },
             ...(input.storeFacts.originUrl ? { origin_url: input.storeFacts.originUrl } : {}),
+            ...(drift ? { drift } : {}),
             status: storeStatus,
         };
     }

package/dist/core/root-selection.d.ts

@@ -7,8 +7,11 @@
  * - `--store <id>` selects a registered store's root.
  * - Without `--store`, the nearest ancestor containing `openspec/` wins.
  *   Leftover workspace view state is never considered a root here.
- * - With no nearest root, registered stores produce a selection hint error;
- *   otherwise commands may treat the current directory as an implicit root.
+ * - With no nearest root, a global `defaultStore` (if set) is the last
+ *   machine-level fallback before the selection hint error.
+ * - With no nearest root and no default, registered stores produce a
+ *   selection hint error; otherwise commands may treat the current
+ *   directory as an implicit root.
  *
  * Diagnostic codes reuse the store taxonomy where an error passes
  * through unchanged (`invalid_store_id`, metadata parse failures);
@@ -19,7 +22,7 @@
  * `no_openspec_root`).
  */
 import { type PlanningHome } from './planning-home.js';
-export type OpenSpecRootSource = 'store' | 'declared' | 'nearest' | 'implicit';
+export type OpenSpecRootSource = 'store' | 'declared' | 'global_default' | 'nearest' | 'implicit';
 export interface StoreSelectorOptions {
     store?: string;
     storePath?: string;
@@ -84,9 +87,9 @@ export interface RootOutput {
 }
 export declare function toRootOutput(root: ResolvedOpenSpecRoot): RootOutput;
 /**
- * A store-selected root — explicit `--store` or the declared fallback.
- * Cross-root behavior (absolute paths, --store hints, suppressed
- * noun-form suggestions) keys on this, never on `source` directly.
+ * A store-selected root — explicit `--store`, a declared pointer, or the
+ * global default. Cross-root behavior (absolute paths, --store hints,
+ * suppressed noun-form suggestions) keys on this, never on `source` directly.
  */
 export declare function isStoreSelectedRoot(root: ResolvedOpenSpecRoot): root is ResolvedOpenSpecRoot & {
     storeId: string;