@pwddd/skills-scanner 1.0.0-beta.6 → 1.0.0-beta.8

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "skills-scanner",
   "name": "Skills Scanner",
   "description": "Security scanner for OpenClaw Skills to detect potential threats",
-  "version": "1.0.0-beta.6",
+  "version": "1.0.0-beta.8",
   "author": "pwddd",
   "skills": ["./skills"],
   "uiHints": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pwddd/skills-scanner",
-  "version": "1.0.0-beta.6",
+  "version": "1.0.0-beta.8",
   "description": "OpenClaw Skills security scanner plugin - detect malicious code, data exfiltration, and prompt injection",
   "type": "module",
   "main": "./index.ts",

package/src/commands.ts

@@ -232,22 +232,12 @@ export function createCommandHandlers(
         ].join("\n"),
       };
     } else if (action === "unregister" || action === "remove") {
+      await cleanupCronJob({
+        logger,
+        config: apiConfig,
+      });
       return {
-        text: [
-          "✅ *删除定时任务*",
-          "",
-          "定时任务会在插件卸载时自动清理。",
-          "",
-          "如需手动删除，请使用 CLI 命令：",
-          "",
-          "```bash",
-          "# 1. 查找任务 ID",
-          "openclaw cron list | grep skills-scanner",
-          "",
-          "# 2. 删除任务",
-          "openclaw cron remove skills-scanner:weekly-report",
-          "```",
-        ].join("\n"),
+        text: "✅ 定时任务删除完成，请运行 `openclaw cron list` 查看结果",
       };
     } else if (action === "trigger" || action === "run") {
       return {

package/src/cron-manager.ts

@@ -100,34 +100,25 @@ export async function ensureCronJob(options: CronManagerOptions): Promise<void>
       }
     }
 
-    // If multiple jobs exist with the same name, remove duplicates
-    if (existingJobs.length > 1) {
-      logger.warn(`[skills-scanner] ⚠️  Found ${existingJobs.length} duplicate jobs, cleaning up...`);
+    // If any jobs exist, remove ALL of them and create a fresh one
+    if (existingJobs.length > 0) {
+      logger.info(`[skills-scanner] Found ${existingJobs.length} existing job(s), cleaning up...`);
 
-      // Keep the first one, remove the rest
-      for (let i = 1; i < existingJobs.length; i++) {
-        const jobId = existingJobs[i].id;
+      // Remove all existing jobs
+      for (const job of existingJobs) {
+        const jobId = job.id;
         try {
           execSync(`${openclawCmd} cron remove ${jobId}`, {
             encoding: "utf-8",
             timeout: 5000,
           });
-          logger.info(`[skills-scanner] ✅ Removed duplicate job: ${jobId}`);
+          logger.info(`[skills-scanner] ✅ Removed job: ${jobId}`);
         } catch (removeErr: any) {
-          logger.warn(`[skills-scanner] ⚠️  Failed to remove duplicate job ${jobId}: ${removeErr.message}`);
+          logger.warn(`[skills-scanner] ⚠️  Failed to remove job ${jobId}: ${removeErr.message}`);
         }
       }
     }
 
-    // Check if we have an existing job (after cleanup)
-    const existingJob = existingJobs.length > 0 ? existingJobs[0] : null;
-
-    if (existingJob) {
-      const jobId = existingJob.id;
-      logger.info(`[skills-scanner] ✅ Job already exists: ${jobId}`);
-      return;
-    }
-
     logger.info("[skills-scanner] 📝 Creating cron job via CLI...");
 
     // Create cron job with --announce and --channel last