koishi-plugin-githubsth 1.0.1-beta → 1.0.1-test2

package/lib/commands/admin.js

@@ -3,6 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.apply = apply;
 function apply(ctx) {
     const logger = ctx.logger('githubsth');
+    // Allow . in repo name (e.g. koishi.js) but disallow / inside parts.
+    // owner: [\w-]+ (alphanumeric, underscore, dash)
+    // repo: [\w-\.]+ (alphanumeric, underscore, dash, dot)
     const repoRegex = /^[\w-]+\/[\w-\.]+$/;
     ctx.command('githubsth.trust', '管理信任仓库', { authority: 3 })
         .alias('gh.trust');
@@ -10,9 +13,11 @@ function apply(ctx) {
         .action(async ({ session }, repo) => {
         if (!repo)
             return '请指定仓库名称 (owner/repo)。';
-        if (!repoRegex.test(repo))
+        if (!repoRegex.test(repo)) {
             return '仓库名称格式不正确 (应为 owner/repo)。';
+        }
         try {
+            logger.debug(`Adding trusted repo: ${repo}`);
             // Check existence first to avoid UNIQUE constraint error log from driver
             const existing = await ctx.database.get('github_trusted_repo', { repo });
             if (existing.length > 0) {
@@ -27,18 +32,17 @@ function apply(ctx) {
             return `已添加信任仓库: ${repo}`;
         }
         catch (e) {
+            logger.warn('Failed to add trusted repo:', e);
             if (e.code === 'SQLITE_CONSTRAINT') {
                 return '该仓库已在信任列表中。';
             }
-            logger.warn(e);
-            return '添加失败，请查看日志。';
+            return `添加失败: ${e.message}`;
         }
     });
     ctx.command('githubsth.trust.remove <repo>', '移除信任仓库')
         .action(async ({ session }, repo) => {
         if (!repo)
             return '请指定仓库名称 (owner/repo)。';
-        // No regex check here needed strictly, but good for consistency
         const result = await ctx.database.remove('github_trusted_repo', { repo });
         if (result.matched === 0)
             return '未找到该信任仓库。';

package/lib/commands/index.js

@@ -35,6 +35,19 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.apply = apply;
 const repo = __importStar(require("./repo"));
+const admin = __importStar(require("./admin"));
+const subscribe = __importStar(require("./subscribe"));
 function apply(ctx, config) {
+    console.log('Applying githubsth commands...');
+    // Register parent command to show help
+    ctx.command('githubsth', 'GitHub 推送通知')
+        .action(({ session }) => {
+        session?.execute('help githubsth');
+    });
     ctx.plugin(repo, config);
+    console.log('githubsth.repo loaded');
+    ctx.plugin(admin, config);
+    console.log('githubsth.admin loaded');
+    ctx.plugin(subscribe, config);
+    console.log('githubsth.subscribe loaded');
 }

package/lib/commands/repo.d.ts

@@ -1,3 +1,8 @@
 import { Context } from 'koishi';
 import { Config } from '../config';
+declare module 'koishi' {
+    interface Context {
+        github?: any;
+    }
+}
 export declare function apply(ctx: Context, config: Config): void;

package/lib/commands/repo.js

@@ -10,10 +10,28 @@ function apply(ctx, config) {
         const fullRepo = name || (config.defaultOwner ? `${config.defaultOwner}/${config.defaultRepo}` : name);
         if (!fullRepo)
             return session?.text('.specify_repo');
-        // 在实际实现中，我们将在此处使用 GitHub API。
-        // 由于我们没有 API 客户端的具体设置细节（例如 Octokit 或适配器内部实现），
-        // 我们将返回一个占位符，以确认命令结构正常工作。
-        // TODO: 使用 session.bot 或专用服务实现实际的 GitHub API 调用。
-        return session?.text('.repo_info', [fullRepo.split('/')[0], fullRepo.split('/')[1] || '?', 'Demo Description', '100']);
+        try {
+            let data;
+            if (ctx.github && typeof ctx.github.request === 'function') {
+                // If adapter provides a request method (hypothetical, based on common adapter patterns)
+                data = await ctx.github.request('GET /repos/:owner/:repo', {
+                    owner: fullRepo.split('/')[0],
+                    repo: fullRepo.split('/')[1]
+                });
+            }
+            else {
+                const headers = {
+                    'User-Agent': 'Koishi-Plugin-GithubSth'
+                };
+                data = await ctx.http.get(`https://api.github.com/repos/${fullRepo}`, { headers });
+            }
+            return session?.text('.repo_info', [data.owner.login, data.name, data.description || '无描述', data.stargazers_count]);
+        }
+        catch (e) {
+            if (e.response?.status === 404) {
+                return session?.text('.not_found');
+            }
+            return session?.text('.error', [e.message]);
+        }
     });
 }

package/lib/index.d.ts

@@ -3,6 +3,7 @@ import { Config } from './config';
 export declare const name = "githubsth";
 export declare const inject: {
     required: string[];
+    optional: string[];
 };
 export * from './config';
 export declare function apply(ctx: Context, config: Config): void;

package/lib/index.js

@@ -42,15 +42,14 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.inject = exports.name = void 0;
 exports.apply = apply;
 const commands = __importStar(require("./commands"));
-const admin = __importStar(require("./commands/admin"));
-const subscribe = __importStar(require("./commands/subscribe"));
 const database = __importStar(require("./database"));
 const zh_CN_1 = __importDefault(require("./locales/zh-CN"));
 const notifier_1 = require("./services/notifier");
 const formatter_1 = require("./services/formatter");
 exports.name = 'githubsth';
 exports.inject = {
-    required: ['database', 'github'],
+    required: ['database'],
+    optional: ['github'],
 };
 __exportStar(require("./config"), exports);
 function apply(ctx, config) {
@@ -62,9 +61,13 @@ function apply(ctx, config) {
     ctx.plugin(formatter_1.Formatter);
     ctx.plugin(notifier_1.Notifier, config);
     // 注册命令
-    ctx.plugin(commands, config);
-    ctx.plugin(admin);
-    ctx.plugin(subscribe);
-    // 加载成功日志
-    console.log('githubsth plugin loaded');
+    // admin and subscribe are already loaded in commands/index.ts, remove duplicate loading here
+    try {
+        ctx.plugin(commands, config);
+        // 加载成功日志
+        console.log('githubsth plugin loaded');
+    }
+    catch (e) {
+        console.error('githubsth plugin failed to load:', e);
+    }
 }

package/lib/services/notifier.js

@@ -22,9 +22,23 @@ class Notifier extends koishi_1.Service {
         this.ctx.on('github/pull_request_review', (payload) => this.handleEvent('pull_request_review', payload));
     }
     async handleEvent(event, payload) {
-        const repoName = payload.repository?.full_name;
-        if (!repoName)
+        let repoName = payload.repository?.full_name;
+        // Try to fallback if repoName is missing
+        if (!repoName && payload.issue?.repository_url) {
+            const parts = payload.issue.repository_url.split('/');
+            if (parts.length >= 2) {
+                repoName = `${parts[parts.length - 2]}/${parts[parts.length - 1]}`;
+            }
+        }
+        if (!repoName && payload.pull_request?.base?.repo?.full_name) {
+            repoName = payload.pull_request.base.repo.full_name;
+        }
+        if (!repoName) {
+            if (this.config.debug) {
+                this.ctx.logger('notifier').warn(`Event ${event} missing repository info. Keys: ${Object.keys(payload).join(', ')}`);
+            }
             return;
+        }
         if (this.config.debug) {
             this.ctx.logger('notifier').info(`Received event ${event} for ${repoName}`);
             this.ctx.logger('notifier').debug(JSON.stringify(payload, null, 2));
@@ -45,8 +59,15 @@ class Notifier extends koishi_1.Service {
                 return false;
             return true;
         });
-        if (matchedRules.length === 0)
+        if (matchedRules.length === 0) {
+            if (this.config.debug) {
+                this.ctx.logger('notifier').debug(`No matching rules for ${repoName} (event: ${event})`);
+            }
             return;
+        }
+        if (this.config.debug) {
+            this.ctx.logger('notifier').debug(`Found ${matchedRules.length} matching rules for ${repoName}`);
+        }
         let message = null;
         // Ensure formatter is loaded
         if (!this.ctx.formatter) {
@@ -85,9 +106,16 @@ class Notifier extends koishi_1.Service {
                 message = this.ctx.formatter.formatPullRequestReview(payload);
                 break;
         }
-        if (!message)
+        if (!message) {
+            if (this.config.debug) {
+                this.ctx.logger('notifier').debug(`Formatter returned null for event ${event}`);
+            }
             return;
+        }
         for (const rule of matchedRules) {
+            if (this.config.debug) {
+                this.ctx.logger('notifier').debug(`Sending message to channel ${rule.channelId} (platform: ${rule.platform || 'any'})`);
+            }
             await this.sendMessage(rule, message);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "koishi-plugin-githubsth",
-  "version": "1.0.1-beta",
+  "version": "1.0.1-test2",
   "description": "Github Subscriptions Notifications, push notifications for GitHub subscriptions For koishi",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",
@@ -39,10 +39,11 @@
     },
     "service": {
       "required": [
-        "database",
-        "github"
+        "database"
       ],
-      "optional": []
+      "optional": [
+        "github"
+      ]
     }
   }
 }