@mbc-cqrs-serverless/cli 1.0.26 → 1.1.0-beta.0

package/dist/actions/install-skills.action.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.VERSION_FILE_NAME = void 0;
+exports.VERSION_CACHE_TTL_MS = exports.VERSION_CACHE_FILE_NAME = exports.VERSION_FILE_NAME = void 0;
 exports.getSkillsSourcePath = getSkillsSourcePath;
 exports.getPersonalSkillsPath = getPersonalSkillsPath;
 exports.getProjectSkillsPath = getProjectSkillsPath;
@@ -11,7 +11,9 @@ exports.copySkills = copySkills;
 exports.getInstalledVersion = getInstalledVersion;
 exports.getPackageVersion = getPackageVersion;
 exports.writeVersionFile = writeVersionFile;
+exports.getLatestVersionFromRegistry = getLatestVersionFromRegistry;
 exports.default = installSkillsAction;
+const child_process_1 = require("child_process");
 const fs_1 = require("fs");
 const os_1 = __importDefault(require("os"));
 const path_1 = __importDefault(require("path"));
@@ -20,6 +22,18 @@ const ui_1 = require("../ui");
  * Version file name for tracking installed skills version
  */
 exports.VERSION_FILE_NAME = '.mbc-skills-version';
+/**
+ * Cache file name for storing latest version from npm registry
+ */
+exports.VERSION_CACHE_FILE_NAME = '.mbc-version-cache.json';
+/**
+ * Cache TTL in milliseconds (24 hours)
+ */
+exports.VERSION_CACHE_TTL_MS = 24 * 60 * 60 * 1000;
+/**
+ * npm package name for mcp-server
+ */
+const MCP_SERVER_PACKAGE = '@mbc-cqrs-serverless/mcp-server';
 /**
  * Get the path to the mcp-server skills source directory
  */
@@ -130,6 +144,61 @@ function writeVersionFile(destPath, version) {
     const versionFilePath = path_1.default.join(destPath, exports.VERSION_FILE_NAME);
     (0, fs_1.writeFileSync)(versionFilePath, version, 'utf-8');
 }
+/**
+ * Get the latest package version from npm registry with caching
+ * @param destPath - The destination path where cache file is stored
+ * @param forceRefresh - Force refresh from npm registry, ignoring cache
+ * @returns The latest version or null if not available
+ */
+function getLatestVersionFromRegistry(destPath, forceRefresh = false) {
+    const cacheFilePath = path_1.default.join(destPath, exports.VERSION_CACHE_FILE_NAME);
+    // Check cache first (unless force refresh)
+    if (!forceRefresh && (0, fs_1.existsSync)(cacheFilePath)) {
+        try {
+            const cache = JSON.parse((0, fs_1.readFileSync)(cacheFilePath, 'utf-8'));
+            const cacheAge = Date.now() - new Date(cache.checkedAt).getTime();
+            if (cacheAge < exports.VERSION_CACHE_TTL_MS) {
+                // Cache is still valid
+                return cache.version;
+            }
+        }
+        catch {
+            // Cache read failed, continue to fetch from registry
+        }
+    }
+    // Fetch from npm registry
+    try {
+        const version = (0, child_process_1.execSync)(`npm view ${MCP_SERVER_PACKAGE} version`, {
+            encoding: 'utf-8',
+            timeout: 10000,
+            stdio: ['pipe', 'pipe', 'pipe'],
+        }).trim();
+        // Ensure destination directory exists before writing cache
+        if (!(0, fs_1.existsSync)(destPath)) {
+            (0, fs_1.mkdirSync)(destPath, { recursive: true });
+        }
+        // Save to cache
+        const cache = {
+            version,
+            checkedAt: new Date().toISOString(),
+        };
+        (0, fs_1.writeFileSync)(cacheFilePath, JSON.stringify(cache, null, 2), 'utf-8');
+        return version;
+    }
+    catch {
+        // Offline or error - try to use expired cache as fallback
+        if ((0, fs_1.existsSync)(cacheFilePath)) {
+            try {
+                const cache = JSON.parse((0, fs_1.readFileSync)(cacheFilePath, 'utf-8'));
+                return cache.version;
+            }
+            catch {
+                return null;
+            }
+        }
+        return null;
+    }
+}
 /**
  * Install Claude Code skills for MBC CQRS Serverless
  */
@@ -170,23 +239,24 @@ async function installSkillsAction(options, command) {
     // Check mode - compare versions without installing
     if (check) {
         const installedVersion = getInstalledVersion(destPath);
-        const packageVersion = getPackageVersion(sourcePath);
+        // Get latest version from npm registry (with 24h cache)
+        const latestVersion = getLatestVersionFromRegistry(destPath, force);
         if (!installedVersion) {
             ui_1.logger.warn('Skills are not installed.');
-            ui_1.logger.info(`Available version: ${packageVersion || 'unknown'}`);
+            ui_1.logger.info(`Available version: ${latestVersion || 'unknown'}`);
             ui_1.logger.info('Run `mbc install-skills` to install.');
             return;
         }
-        if (!packageVersion) {
-            ui_1.logger.warn('Could not determine package version.');
+        if (!latestVersion) {
+            ui_1.logger.warn('Could not determine latest version. Check your network connection.');
             ui_1.logger.info(`Installed version: ${installedVersion}`);
             return;
         }
-        if (installedVersion === packageVersion) {
+        if (installedVersion === latestVersion) {
             ui_1.logger.success(`Skills are up to date (${installedVersion}).`);
         }
         else {
-            ui_1.logger.warn(`Update available: ${installedVersion} → ${packageVersion}`);
+            ui_1.logger.warn(`Update available: ${installedVersion} → ${latestVersion}`);
             ui_1.logger.info('Run `mbc install-skills --force` to update.');
         }
         return;
@@ -203,8 +273,10 @@ async function installSkillsAction(options, command) {
     // Copy skills
     ui_1.logger.title('install', `Installing skills to ${destPath}`);
     const copiedSkills = copySkills(sourcePath, destPath);
+    // Get version from npm registry (preferred) or fall back to local package.json
+    const latestVersion = getLatestVersionFromRegistry(destPath, true);
+    const packageVersion = latestVersion || getPackageVersion(sourcePath);
     // Write version file
-    const packageVersion = getPackageVersion(sourcePath);
     if (packageVersion) {
         writeVersionFile(destPath, packageVersion);
     }

package/dist/actions/install-skills.action.spec.js

@@ -39,7 +39,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const install_skills_action_1 = __importStar(require("./install-skills.action"));
 jest.mock('fs');
 jest.mock('os');
+jest.mock('child_process');
 const fs_1 = require("fs");
+const child_process_1 = require("child_process");
 const os_1 = __importDefault(require("os"));
 const mockExistsSync = fs_1.existsSync;
 const mockMkdirSync = fs_1.mkdirSync;
@@ -47,6 +49,7 @@ const mockCpSync = fs_1.cpSync;
 const mockReaddirSync = fs_1.readdirSync;
 const mockReadFileSync = fs_1.readFileSync;
 const mockWriteFileSync = fs_1.writeFileSync;
+const mockExecSync = child_process_1.execSync;
 describe('Install Skills Action', () => {
     const mockCommand = {
         name: () => 'install-skills',
@@ -55,6 +58,8 @@ describe('Install Skills Action', () => {
     beforeEach(() => {
         jest.clearAllMocks();
         os_1.default.homedir.mockReturnValue('/home/user');
+        // Default mock for npm registry - returns a test version
+        mockExecSync.mockReturnValue('1.0.25\n');
     });
     describe('getSkillsSourcePath', () => {
         it('should return the path to mcp-server skills directory', () => {
@@ -278,6 +283,81 @@ describe('Install Skills Action', () => {
                 expect(mockWriteFileSync).toHaveBeenCalledWith(expect.stringContaining(install_skills_action_1.VERSION_FILE_NAME), '1.0.25', 'utf-8');
             });
         });
+        describe('getLatestVersionFromRegistry', () => {
+            it('should fetch version from npm registry', () => {
+                mockExistsSync.mockReturnValue(false);
+                mockExecSync.mockReturnValue('1.0.26\n');
+                const version = (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills');
+                expect(version).toBe('1.0.26');
+                expect(mockExecSync).toHaveBeenCalledWith('npm view @mbc-cqrs-serverless/mcp-server version', expect.objectContaining({ encoding: 'utf-8', timeout: 10000 }));
+            });
+            it('should use cached version if cache is valid', () => {
+                const cacheData = {
+                    version: '1.0.25',
+                    checkedAt: new Date().toISOString(), // Fresh cache
+                };
+                mockExistsSync.mockReturnValue(true);
+                mockReadFileSync.mockReturnValue(JSON.stringify(cacheData));
+                const version = (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills');
+                expect(version).toBe('1.0.25');
+                expect(mockExecSync).not.toHaveBeenCalled();
+            });
+            it('should fetch from registry if cache is expired', () => {
+                const cacheData = {
+                    version: '1.0.24',
+                    checkedAt: new Date(Date.now() - 25 * 60 * 60 * 1000).toISOString(), // 25 hours ago
+                };
+                mockExistsSync.mockReturnValue(true);
+                mockReadFileSync.mockReturnValue(JSON.stringify(cacheData));
+                mockExecSync.mockReturnValue('1.0.26\n');
+                const version = (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills');
+                expect(version).toBe('1.0.26');
+                expect(mockExecSync).toHaveBeenCalled();
+            });
+            it('should force refresh when forceRefresh is true', () => {
+                const cacheData = {
+                    version: '1.0.24',
+                    checkedAt: new Date().toISOString(), // Fresh cache
+                };
+                mockExistsSync.mockReturnValue(true);
+                mockReadFileSync.mockReturnValue(JSON.stringify(cacheData));
+                mockExecSync.mockReturnValue('1.0.26\n');
+                const version = (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills', true);
+                expect(version).toBe('1.0.26');
+                expect(mockExecSync).toHaveBeenCalled();
+            });
+            it('should use expired cache as fallback when offline', () => {
+                const cacheData = {
+                    version: '1.0.24',
+                    checkedAt: new Date(Date.now() - 25 * 60 * 60 * 1000).toISOString(),
+                };
+                mockExistsSync.mockReturnValue(true);
+                mockReadFileSync.mockReturnValue(JSON.stringify(cacheData));
+                mockExecSync.mockImplementation(() => {
+                    throw new Error('Network error');
+                });
+                const version = (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills');
+                expect(version).toBe('1.0.24');
+            });
+            it('should return null when offline and no cache exists', () => {
+                mockExistsSync.mockReturnValue(false);
+                mockExecSync.mockImplementation(() => {
+                    throw new Error('Network error');
+                });
+                const version = (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills');
+                expect(version).toBeNull();
+            });
+            it('should save version to cache after fetching', () => {
+                mockExistsSync.mockReturnValue(true);
+                mockExecSync.mockReturnValue('1.0.26\n');
+                // Make cache read fail to trigger fetch
+                mockReadFileSync.mockImplementation(() => {
+                    throw new Error('No cache');
+                });
+                (0, install_skills_action_1.getLatestVersionFromRegistry)('/dest/skills');
+                expect(mockWriteFileSync).toHaveBeenCalledWith(expect.stringContaining(install_skills_action_1.VERSION_CACHE_FILE_NAME), expect.stringContaining('"version": "1.0.26"'), 'utf-8');
+            });
+        });
         describe('Check option', () => {
             beforeEach(() => {
                 mockReaddirSync.mockReturnValue([

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mbc-cqrs-serverless/cli",
-  "version": "1.0.26",
+  "version": "1.1.0-beta.0",
   "description": "a CLI to get started with MBC CQRS serverless framework",
   "keywords": [
     "mbc",
@@ -58,5 +58,5 @@
     "@faker-js/faker": "^8.3.1",
     "copyfiles": "^2.4.1"
   },
-  "gitHead": "0956dc696475fb769dd134aa619679ba5155e646"
+  "gitHead": "b16d8982c02933fd41a455d3e41f9750bbcf668c"
 }