@educa-corp/sdd-framework 0.7.2 → 0.7.3

package/bin/index.js

@@ -5,6 +5,21 @@ const path = require('path');
 
 const ROOT = path.join(__dirname, '..');
 
+// NGUỒN ASSET ĐỂ CÀI — luôn là `core/`, không bao giờ là dir ở gốc.
+//
+// `core/` là bản dựng sẵn và là thứ DUY NHẤT được ship (package.json → files:
+// bin/ core/ scripts/ docs/). Các dir `modules/`, `hooks/`, `steps/`, `templates/`,
+// `rules/`, `skills/` ở gốc là NGUỒN BUILD — chúng chỉ tồn tại trong dev checkout.
+//
+// Vì sao khai một hằng số cho việc này (hồi quy 0.7.0 / G59): đợt dọn tập publish bỏ
+// `modules/` + `hooks/` khỏi `files`, nhưng BA chỗ trong file này vẫn đọc
+// `path.join(ROOT, 'modules'|'hooks')`. Trong dev checkout chúng vẫn có nên test và
+// `npm run build` đều xanh; trong tarball thì KHÔNG, và `--module` crash cứng bằng
+// ENOENT giữa `--init` — chết TRƯỚC bước `--hooks`, nên một cờ hỏng làm chết cờ kia và
+// project bị cài dở (thiếu .claude/settings.json).
+// Test `bin/ chỉ đọc dir CÓ trong package.json files` canh việc này từ nay.
+const CORE_DIR = path.join(ROOT, 'core');
+
 const pkg      = JSON.parse(fs.readFileSync(path.join(ROOT, 'package.json'), 'utf8'));
 const VERSION  = pkg.version;
 
@@ -14,6 +29,12 @@ const VERSION  = pkg.version;
 // of this file is evaluated, and `const` does not hoist.
 const MANIFEST_NAME = '.install-manifest.json';
 
+// Asset được YÊU CẦU tường minh (--module, --hooks, module của service) mà bản cài không
+// có. Gom lại thay vì crash giữa `--init`: crash để lại project cài DỞ — đúng ca hồi quy
+// 0.7.0, nơi ENOENT ở bước --module làm bước --hooks không bao giờ chạy. Nhưng cũng KHÔNG
+// im lặng: exit 1 ở cuối, vì người dùng đã xin thứ họ không nhận được.
+const assetErrors = [];
+
 const args = process.argv.slice(2);
 const isMigrateSpecs = args.includes('--migrate-specs');
 const isMigrateBddPf = args.includes('--migrate-bdd-platform');
@@ -301,11 +322,20 @@ if (isInit) {
       console.log(`⚠️  Unknown module: "${moduleName}"`);
       console.log(`   Available: ${AVAILABLE_MODULES.join(', ')}`);
     } else {
-      const srcModuleDir  = path.join(ROOT, 'modules', moduleName);
+      const srcModuleDir  = path.join(CORE_DIR, 'modules', moduleName);
       const destModuleDir = path.join(agentDir, 'modules', moduleName);
-      fs.mkdirSync(destModuleDir, { recursive: true });
-      copyDirRecursive(srcModuleDir, destModuleDir);
-      console.log(`✅ Module "${moduleName}" installed to: .agent/modules/${moduleName}`);
+      // Guard TRƯỚC khi copy. Bản cũ gọi thẳng copyDirRecursive và ENOENT làm chết cả
+      // `--init` giữa đường — bước `--hooks` ngay dưới không bao giờ chạy, project bị cài
+      // dở. Một cờ hỏng không được phép làm chết cờ khác.
+      if (!fs.existsSync(srcModuleDir)) {
+        console.log(`❌ Module "${moduleName}" không có trong bản cài: ${path.relative(ROOT, srcModuleDir)}`);
+        console.log(`   Bản cài thiếu/hỏng — cài lại, hoặc xoá cache npx rồi chạy lại.`);
+        assetErrors.push(`module ${moduleName}`);
+      } else {
+        fs.mkdirSync(destModuleDir, { recursive: true });
+        copyDirRecursive(srcModuleDir, destModuleDir);
+        console.log(`✅ Module "${moduleName}" installed to: .agent/modules/${moduleName}`);
+      }
     }
   }
 
@@ -440,12 +470,18 @@ if (isInit) {
         if (!AVAILABLE_MODULES.includes(svc.module)) {
           console.log(`  ⚠️  Unknown module "${svc.module}" — skipped`);
         } else {
-          const srcMod  = path.join(ROOT, 'modules', svc.module);
+          const srcMod  = path.join(CORE_DIR, 'modules', svc.module);
           const destMod = path.join(svcAgentDir, 'modules', svc.module);
           if (fs.existsSync(srcMod)) {
             fs.mkdirSync(destMod, { recursive: true });
             copyDirRecursive(srcMod, destMod);
             console.log(`  ✅ ${svc.name}/.agent/modules/${svc.module}/`);
+          } else {
+            // Bản cũ bỏ qua ÂM THẦM ở đây: người dùng khai module cho service, không nhận
+            // được gì, và không dòng nào báo. Cùng nguồn lỗi với ca --module ở trên nhưng
+            // hỏng theo hướng im lặng — tệ hơn.
+            console.log(`  ❌ ${svc.name}: module "${svc.module}" không có trong bản cài`);
+            assetErrors.push(`${svc.name}/module ${svc.module}`);
           }
         }
       }
@@ -577,6 +613,12 @@ if (isInit) {
   console.log('To upgrade later:  /update-framework   (trong Claude Code)');
   console.log('                   (or: npx @educa-corp/sdd-framework@latest --init)');
   console.log('');
+  if (assetErrors.length) {
+    console.log('');
+    console.log(`❌ ${assetErrors.length} asset được yêu cầu nhưng bản cài không có: ${assetErrors.join(' · ')}`);
+    console.log('   Phần còn lại của --init đã hoàn tất — nhưng exit 1 vì bạn đã xin thứ không nhận được.');
+    process.exit(1);
+  }
   process.exit(0);
 }
 
@@ -963,7 +1005,7 @@ function installDataGuardHook() {
 
   // 1. Copy hook script
   fs.mkdirSync(hooksDir, { recursive: true });
-  const srcHook  = path.join(ROOT, 'hooks', 'data-guard.js');
+  const srcHook  = path.join(CORE_DIR, 'hooks', 'data-guard.js');
   const destHook = path.join(hooksDir, 'data-guard.js');
   fs.copyFileSync(srcHook, destHook);
   console.log(`  ✅ Hook copied to: .claude/hooks/data-guard.js`);

package/core/FRAMEWORK_VERSION

@@ -1 +1 @@
-0.7.2
+0.7.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@educa-corp/sdd-framework",
-  "version": "0.7.2",
+  "version": "0.7.3",
   "description": "Spec Driven Development workflow framework for Claude Code",
   "bin": {
     "sdd-framework": "./bin/index.js"