fix(cli): refresh cached skill registry on add

Author: codeaholicguy

packages/cli/src/__tests__/lib/SkillManager.test.ts

@@ -284,6 +284,55 @@ describe("SkillManager", () => {
       expect(mockedGitUtil.cloneRepository).not.toHaveBeenCalled();
     });
 
+    it("should pull cached registry before installing skill", async () => {
+      const repoPath = path.join(os.homedir(), ".ai-devkit", "skills", mockRegistryId);
+
+      (mockedFs.pathExists as any).mockImplementation((checkPath: string) => {
+        if (checkPath === repoPath) {
+          return Promise.resolve(true);
+        }
+        if (checkPath.includes(`${path.sep}skills${path.sep}${mockSkillName}`)) {
+          return Promise.resolve(true);
+        }
+        if (checkPath.endsWith(`${path.sep}SKILL.md`)) {
+          return Promise.resolve(true);
+        }
+        return Promise.resolve(true);
+      });
+
+      mockedGitUtil.isGitRepository.mockResolvedValue(true);
+      mockedGitUtil.pullRepository.mockResolvedValue(undefined);
+
+      await skillManager.addSkill(mockRegistryId, mockSkillName);
+
+      expect(mockedGitUtil.cloneRepository).not.toHaveBeenCalled();
+      expect(mockedGitUtil.pullRepository).toHaveBeenCalledWith(repoPath);
+    });
+
+    it("should skip pull when cached registry is not a git repository", async () => {
+      const repoPath = path.join(os.homedir(), ".ai-devkit", "skills", mockRegistryId);
+
+      (mockedFs.pathExists as any).mockImplementation((checkPath: string) => {
+        if (checkPath === repoPath) {
+          return Promise.resolve(true);
+        }
+        if (checkPath.includes(`${path.sep}skills${path.sep}${mockSkillName}`)) {
+          return Promise.resolve(true);
+        }
+        if (checkPath.endsWith(`${path.sep}SKILL.md`)) {
+          return Promise.resolve(true);
+        }
+        return Promise.resolve(true);
+      });
+
+      mockedGitUtil.isGitRepository.mockResolvedValue(false);
+
+      await skillManager.addSkill(mockRegistryId, mockSkillName);
+
+      expect(mockedGitUtil.pullRepository).not.toHaveBeenCalled();
+      expect(mockedGitUtil.cloneRepository).not.toHaveBeenCalled();
+    });
+
     it("should throw error if skill not found in repository", async () => {
       (mockedFs.pathExists as any).mockResolvedValue(false);
 

packages/cli/src/lib/SkillManager.ts

@@ -405,6 +405,14 @@ export class SkillManager {
     const repoPath = path.join(SKILL_CACHE_DIR, registryId);
 
     if (await fs.pathExists(repoPath)) {
+      if (await isGitRepository(repoPath)) {
+        const spinner = ui.spinner(`Updating cached repository ${registryId}...`);
+        spinner.start();
+        await pullRepository(repoPath);
+        spinner.succeed(`Cached repository ${registryId} updated`);
+      } else {
+        ui.warning(`Cached registry ${registryId} is not a git repository, using as-is.`);
+      }
       ui.text('  → Using cached repository');
       return repoPath;
     }
@@ -704,4 +712,4 @@ export class SkillManager {
       return nameMatch || descMatch;
     });
   }
-}
+}