@clawplays/ospec-cli 0.3.7 → 0.3.8

package/dist/services/ProjectService.js

@@ -4545,6 +4545,7 @@ class ProjectService {
 
 
 
+        await this.rebaseMovedChangeMarkdownLinks(resolvedFeaturePath, archivePath);
         await this.rebuildIndex(projectRoot);
 
 
@@ -4593,6 +4594,51 @@ class ProjectService {
 
 
 
+    async rebaseMovedChangeMarkdownLinks(previousChangePath, nextChangePath) {
+        const previousRoot = path_1.default.resolve(previousChangePath);
+        const nextRoot = path_1.default.resolve(nextChangePath);
+        const markdownFiles = [
+            constants_1.FILE_NAMES.PROPOSAL,
+            constants_1.FILE_NAMES.TASKS,
+            constants_1.FILE_NAMES.VERIFICATION,
+            constants_1.FILE_NAMES.REVIEW,
+        ];
+        for (const fileName of markdownFiles) {
+            const nextFilePath = path_1.default.join(nextRoot, fileName);
+            if (!(await this.fileService.exists(nextFilePath))) {
+                continue;
+            }
+            const previousFilePath = path_1.default.join(previousRoot, fileName);
+            const originalContent = await this.fileService.readFile(nextFilePath);
+            const previousDir = path_1.default.dirname(previousFilePath);
+            const nextDir = path_1.default.dirname(nextFilePath);
+            const rewrittenContent = originalContent.replace(/\[([^\]]+)\]\(([^)]+)\)/g, (match, label, rawHref) => {
+                const href = String(rawHref || '').trim();
+                if (!this.isRelativeMarkdownHref(href)) {
+                    return match;
+                }
+                const previousTargetPath = path_1.default.resolve(previousDir, href);
+                if (this.isPathWithin(previousTargetPath, previousRoot)) {
+                    return match;
+                }
+                const rebasedHref = path_1.default.relative(nextDir, previousTargetPath).replace(/\\/g, '/');
+                return `[${label}](${rebasedHref || '.'})`;
+            });
+            if (rewrittenContent !== originalContent) {
+                await this.fileService.writeFile(nextFilePath, rewrittenContent);
+            }
+        }
+    }
+    isRelativeMarkdownHref(href) {
+        return Boolean(href) &&
+            !href.startsWith('#') &&
+            !href.startsWith('//') &&
+            !/^[a-z][a-z0-9+.-]*:/i.test(href);
+    }
+    isPathWithin(targetPath, parentPath) {
+        const relativePath = path_1.default.relative(parentPath, targetPath);
+        return relativePath === '' || (!relativePath.startsWith('..') && !path_1.default.isAbsolute(relativePath));
+    }
     async getFeatureProjectContext(rootDir, affects = []) {
 
 
@@ -5275,7 +5321,8 @@ class ProjectService {
     }
     async syncConfigDocumentLanguage(rootDir, config, documentLanguage) {
         const normalized = this.normalizeDocumentLanguage(documentLanguage);
-        if (!normalized || config?.documentLanguage === normalized) {
+        const configured = this.normalizeDocumentLanguage(config?.documentLanguage);
+        if (!normalized || configured) {
             return false;
         }
         config.documentLanguage = normalized;
@@ -5284,13 +5331,28 @@ class ProjectService {
     }
 
     detectDocumentLanguageFromTexts(contents) {
-        for (const content of contents) {
+        const detectionCounts = new Map();
+        const firstSeenOrder = new Map();
+        for (const [index, content] of contents.entries()) {
             const detected = this.detectDocumentLanguageFromText(content);
             if (detected) {
-                return detected;
+                detectionCounts.set(detected, (detectionCounts.get(detected) || 0) + 1);
+                if (!firstSeenOrder.has(detected)) {
+                    firstSeenOrder.set(detected, index);
+                }
             }
         }
-        return undefined;
+        if (detectionCounts.size === 0) {
+            return undefined;
+        }
+        return Array.from(detectionCounts.entries())
+            .sort((left, right) => {
+            if (right[1] !== left[1]) {
+                return right[1] - left[1];
+            }
+            return (firstSeenOrder.get(left[0]) ?? Number.MAX_SAFE_INTEGER) -
+                (firstSeenOrder.get(right[0]) ?? Number.MAX_SAFE_INTEGER);
+        })[0][0];
     }
     detectDocumentLanguageFromText(content) {
         if (typeof content !== 'string' || content.trim().length === 0) {
@@ -5299,28 +5361,28 @@ class ProjectService {
         if (/[\u0600-\u06FF]/.test(content)) {
             return 'ar';
         }
-        if (/[ぁ-ゟ゠-ヿ]/.test(content)) {
-            return 'ja-JP';
-        }
-        if (this.isLikelyJapaneseKanjiContent(content)) {
+        if (this.hasJapaneseKana(content)) {
             return 'ja-JP';
         }
-        if (/[一-龥]/.test(content)) {
-            return 'zh-CN';
+        if (this.hasCjkIdeographs(content)) {
+            return this.isLikelyJapaneseKanjiContent(content) ? 'ja-JP' : 'zh-CN';
         }
         if (/[A-Za-z]/.test(content)) {
             return 'en-US';
         }
         return undefined;
     }
+    hasJapaneseKana(content) {
+        return /[\u3040-\u30FF]/.test(content);
+    }
+    hasCjkIdeographs(content) {
+        return /[\u3400-\u9FFF]/.test(content);
+    }
     isLikelyJapaneseKanjiContent(content) {
-        if (!/[一-龥]/.test(content)) {
+        if (!this.hasCjkIdeographs(content)) {
             return false;
         }
-        if (/[々〆ヵヶ「」『』]/.test(content)) {
-            return true;
-        }
-        return /(一覧|詳細|設定|権限|検索|構成|変更|確認|対応|連携|承認|申請|手順|履歴|機能|実装|設計|運用|画面|帳票|組織|拠点|区分|種別|完了|開始|終了|表示|取得|追加|削除|更新|登録)/.test(content);
+        return /[\u3005\u3006\u300C-\u300F\u30F5\u30F6]/.test(content);
     }
 
 
@@ -11034,197 +11096,103 @@ ${formatSuggestion()}
 
 
         const content = await this.fileService.readFile(filePath);
-
-
-
-
-
-
-
-        const parsed = (0, gray_matter_1.default)(content);
-
-
-
-
-
-
-
-        const optionalSteps = Array.isArray(parsed.data.optional_steps) ? parsed.data.optional_steps : [];
-
-
-
-
-
-
-
-        const missing = activatedSteps.filter(step => !optionalSteps.includes(step));
-
-
-
-
-
-
-
-        const checklistComplete = !/- \[ \]/.test(content);
-
-
-
-
-
-
-
+        const hasFrontmatter = /^---\r?\n[\s\S]*?\r?\n---(?:\r?\n|$)/.test(content);
+        let parsed = null;
+        let parseError = null;
+        if (hasFrontmatter) {
+            try {
+                parsed = (0, gray_matter_1.default)(content);
+            }
+            catch (error) {
+                parseError = error;
+            }
+        }
+        const data = parsed?.data ?? {};
+        const optionalStepsFieldValid = Array.isArray(data.optional_steps);
+        const optionalSteps = optionalStepsFieldValid ? data.optional_steps : [];
+        const createdFieldValid = (typeof data.created === 'string' && data.created.trim().length > 0) ||
+            (data.created instanceof Date && !Number.isNaN(data.created.getTime()));
+        const missingRequiredFields = [];
+        if (typeof data.feature !== 'string' || data.feature.trim().length === 0) {
+            missingRequiredFields.push('feature');
+        }
+        if (!createdFieldValid) {
+            missingRequiredFields.push('created');
+        }
+        if (!optionalStepsFieldValid) {
+            missingRequiredFields.push('optional_steps');
+        }
+        const missing = optionalStepsFieldValid
+            ? activatedSteps.filter(step => !optionalSteps.includes(step))
+            : [...activatedSteps];
+        const checklistItems = parsed?.content.match(/^\s*-\s+\[(?: |x|X)\]\s+.+$/gm) ?? [];
+        const uncheckedItems = parsed?.content.match(/^\s*-\s+\[ \]\s+.+$/gm) ?? [];
+        const checklistStructureValid = checklistItems.length > 0;
+        const checklistComplete = hasFrontmatter &&
+            parseError === null &&
+            missingRequiredFields.length === 0 &&
+            checklistStructureValid &&
+            uncheckedItems.length === 0;
+        let frontmatterMessage = `${name} frontmatter parsed successfully`;
+        if (!hasFrontmatter) {
+            frontmatterMessage = `${name} is missing a valid frontmatter block`;
+        }
+        else if (parseError) {
+            frontmatterMessage = `${name} frontmatter cannot be parsed: ${parseError.message}`;
+        }
+        let requiredFieldsMessage = `${name} has all required frontmatter fields`;
+        if (!hasFrontmatter || parseError) {
+            requiredFieldsMessage = `Cannot validate required fields in ${name} because frontmatter is invalid`;
+        }
+        else if (missingRequiredFields.length > 0) {
+            requiredFieldsMessage = `Missing or invalid required fields in ${name}: ${missingRequiredFields.join(', ')}`;
+        }
+        let optionalStepsMessage = `All activated optional steps are present in ${name}`;
+        if (!optionalStepsFieldValid) {
+            optionalStepsMessage = `${name} frontmatter field optional_steps must be an array`;
+        }
+        else if (missing.length > 0) {
+            optionalStepsMessage = `Missing optional steps in ${name}: ${missing.join(', ')}`;
+        }
+        let checklistStatus = 'pass';
+        let checklistMessage = `${name} checklist is complete`;
+        if (!hasFrontmatter || parseError) {
+            checklistStatus = 'fail';
+            checklistMessage = `${name} checklist cannot be validated because frontmatter is invalid`;
+        }
+        else if (!checklistStructureValid) {
+            checklistStatus = 'fail';
+            checklistMessage = `${name} must contain at least one Markdown checklist item`;
+        }
+        else if (uncheckedItems.length > 0) {
+            checklistStatus = 'warn';
+            checklistMessage = `${name} still has unchecked items`;
+        }
         return {
-
-
-
-
-
-
-
             optionalSteps,
-
-
-
-
-
-
-
             checklistComplete,
-
-
-
-
-
-
-
             checks: [
-
-
-
-
-
-
-
                 {
-
-
-
-
-
-
-
+                    name: `${name}.frontmatter`,
+                    status: hasFrontmatter && parseError === null ? 'pass' : 'fail',
+                    message: frontmatterMessage,
+                },
+                {
+                    name: `${name}.required_fields`,
+                    status: hasFrontmatter && parseError === null && missingRequiredFields.length === 0 ? 'pass' : 'fail',
+                    message: requiredFieldsMessage,
+                },
+                {
                     name: `${name}.optional_steps`,
-
-
-
-
-
-
-
-                    status: missing.length === 0 ? 'pass' : 'fail',
-
-
-
-
-
-
-
-                    message: missing.length === 0
-
-
-
-
-
-
-
-                        ? `All activated optional steps are present in ${name}`
-
-
-
-
-
-
-
-                        : `Missing optional steps in ${name}: ${missing.join(', ')}`,
-
-
-
-
-
-
-
+                    status: optionalStepsFieldValid && missing.length === 0 ? 'pass' : 'fail',
+                    message: optionalStepsMessage,
                 },
-
-
-
-
-
-
-
                 {
-
-
-
-
-
-
-
                     name: `${name}.checklist`,
-
-
-
-
-
-
-
-                    status: checklistComplete ? 'pass' : 'warn',
-
-
-
-
-
-
-
-                    message: checklistComplete
-
-
-
-
-
-
-
-                        ? `${name} checklist is complete`
-
-
-
-
-
-
-
-                        : `${name} still has unchecked items`,
-
-
-
-
-
-
-
+                    status: checklistStatus,
+                    message: checklistMessage,
                 },
-
-
-
-
-
-
-
             ],
-
-
-
-
-
-
-
         };
 
 
@@ -11250,229 +11218,112 @@ ${formatSuggestion()}
 
 
         const content = await this.fileService.readFile(filePath);
-
-
-
-
-
-
-
-        const parsed = (0, gray_matter_1.default)(content);
-
-
-
-
-
-
-
-        const optionalSteps = Array.isArray(parsed.data.optional_steps) ? parsed.data.optional_steps : [];
-
-
-
-
-
-
-
-        const passedOptionalSteps = Array.isArray(parsed.data.passed_optional_steps)
-
-
-
-
-
-
-
-            ? parsed.data.passed_optional_steps
-
-
-
-
-
-
-
-            : [];
-
-
-
-
-
-
-
-        const missing = activatedSteps.filter(step => !optionalSteps.includes(step));
-
-
-
-
-
-
-
-        const checklistComplete = !/- \[ \]/.test(content);
-
-
-
-
-
-
-
+        const hasFrontmatter = /^---\r?\n[\s\S]*?\r?\n---(?:\r?\n|$)/.test(content);
+        let parsed = null;
+        let parseError = null;
+        if (hasFrontmatter) {
+            try {
+                parsed = (0, gray_matter_1.default)(content);
+            }
+            catch (error) {
+                parseError = error;
+            }
+        }
+        const data = parsed?.data ?? {};
+        const optionalStepsFieldValid = Array.isArray(data.optional_steps);
+        const passedOptionalStepsFieldValid = Array.isArray(data.passed_optional_steps);
+        const optionalSteps = optionalStepsFieldValid ? data.optional_steps : [];
+        const passedOptionalSteps = passedOptionalStepsFieldValid ? data.passed_optional_steps : [];
+        const createdFieldValid = (typeof data.created === 'string' && data.created.trim().length > 0) ||
+            (data.created instanceof Date && !Number.isNaN(data.created.getTime()));
+        const missingRequiredFields = [];
+        if (typeof data.feature !== 'string' || data.feature.trim().length === 0) {
+            missingRequiredFields.push('feature');
+        }
+        if (!createdFieldValid) {
+            missingRequiredFields.push('created');
+        }
+        if (typeof data.status !== 'string' || data.status.trim().length === 0) {
+            missingRequiredFields.push('status');
+        }
+        if (!optionalStepsFieldValid) {
+            missingRequiredFields.push('optional_steps');
+        }
+        if (!passedOptionalStepsFieldValid) {
+            missingRequiredFields.push('passed_optional_steps');
+        }
+        const missing = optionalStepsFieldValid
+            ? activatedSteps.filter(step => !optionalSteps.includes(step))
+            : [...activatedSteps];
+        const checklistItems = parsed?.content.match(/^\s*-\s+\[(?: |x|X)\]\s+.+$/gm) ?? [];
+        const uncheckedItems = parsed?.content.match(/^\s*-\s+\[ \]\s+.+$/gm) ?? [];
+        const checklistStructureValid = checklistItems.length > 0;
+        const checklistComplete = hasFrontmatter &&
+            parseError === null &&
+            missingRequiredFields.length === 0 &&
+            checklistStructureValid &&
+            uncheckedItems.length === 0;
+        let frontmatterMessage = 'verification.md frontmatter parsed successfully';
+        if (!hasFrontmatter) {
+            frontmatterMessage = 'verification.md is missing a valid frontmatter block';
+        }
+        else if (parseError) {
+            frontmatterMessage = `verification.md frontmatter cannot be parsed: ${parseError.message}`;
+        }
+        let requiredFieldsMessage = 'verification.md has all required frontmatter fields';
+        if (!hasFrontmatter || parseError) {
+            requiredFieldsMessage = 'Cannot validate required fields in verification.md because frontmatter is invalid';
+        }
+        else if (missingRequiredFields.length > 0) {
+            requiredFieldsMessage = `Missing or invalid required fields in verification.md: ${missingRequiredFields.join(', ')}`;
+        }
+        let optionalStepsMessage = 'All activated optional steps are present in verification.md';
+        if (!optionalStepsFieldValid) {
+            optionalStepsMessage = 'verification.md frontmatter field optional_steps must be an array';
+        }
+        else if (missing.length > 0) {
+            optionalStepsMessage = `Missing optional steps in verification.md: ${missing.join(', ')}`;
+        }
+        let checklistStatus = 'pass';
+        let checklistMessage = 'verification.md checklist is complete';
+        if (!hasFrontmatter || parseError) {
+            checklistStatus = 'fail';
+            checklistMessage = 'verification.md checklist cannot be validated because frontmatter is invalid';
+        }
+        else if (!checklistStructureValid) {
+            checklistStatus = 'fail';
+            checklistMessage = 'verification.md must contain at least one Markdown checklist item';
+        }
+        else if (uncheckedItems.length > 0) {
+            checklistStatus = 'warn';
+            checklistMessage = 'verification.md still has unchecked items';
+        }
         return {
-
-
-
-
-
-
-
             optionalSteps,
-
-
-
-
-
-
-
             passedOptionalSteps,
-
-
-
-
-
-
-
             checklistComplete,
-
-
-
-
-
-
-
             checks: [
-
-
-
-
-
-
-
                 {
-
-
-
-
-
-
-
+                    name: 'verification.md.frontmatter',
+                    status: hasFrontmatter && parseError === null ? 'pass' : 'fail',
+                    message: frontmatterMessage,
+                },
+                {
+                    name: 'verification.md.required_fields',
+                    status: hasFrontmatter && parseError === null && missingRequiredFields.length === 0 ? 'pass' : 'fail',
+                    message: requiredFieldsMessage,
+                },
+                {
                     name: 'verification.md.optional_steps',
-
-
-
-
-
-
-
-                    status: missing.length === 0 ? 'pass' : 'fail',
-
-
-
-
-
-
-
-                    message: missing.length === 0
-
-
-
-
-
-
-
-                        ? 'All activated optional steps are present in verification.md'
-
-
-
-
-
-
-
-                        : `Missing optional steps in verification.md: ${missing.join(', ')}`,
-
-
-
-
-
-
-
+                    status: optionalStepsFieldValid && missing.length === 0 ? 'pass' : 'fail',
+                    message: optionalStepsMessage,
                 },
-
-
-
-
-
-
-
                 {
-
-
-
-
-
-
-
                     name: 'verification.md.checklist',
-
-
-
-
-
-
-
-                    status: checklistComplete ? 'pass' : 'warn',
-
-
-
-
-
-
-
-                    message: checklistComplete
-
-
-
-
-
-
-
-                        ? 'verification.md checklist is complete'
-
-
-
-
-
-
-
-                        : 'verification.md still has unchecked items',
-
-
-
-
-
-
-
+                    status: checklistStatus,
+                    message: checklistMessage,
                 },
-
-
-
-
-
-
-
             ],
-
-
-
-
-
-
-
         };