koishi-plugin-githubsth 1.0.2-test1 → 1.0.2

package/lib/commands/subscribe.d.ts

@@ -1,2 +1,2 @@
 import { Context } from 'koishi';
-export declare function apply(ctx: Context): void;
+export declare function apply(ctx: Context, config: any): void;

package/lib/commands/subscribe.js

@@ -1,11 +1,37 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.apply = apply;
-function apply(ctx) {
+function apply(ctx, config) {
     const logger = ctx.logger('githubsth');
     const repoRegex = /^[\w-]+\/[\w-\.]+$/;
+    const validEvents = [
+        'push', 'issues', 'issue_comment', 'pull_request',
+        'pull_request_review', 'star', 'fork', 'release',
+        'discussion', 'workflow_run'
+    ];
+    const defaultConfigEvents = config.defaultEvents || ['push', 'issues', 'issue_comment', 'pull_request', 'pull_request_review', 'release', 'star', 'fork'];
     ctx.command('githubsth.subscribe <repo> [events:text]', '订阅 GitHub 仓库')
         .alias('gh.sub')
+        .usage(`
+订阅 GitHub 仓库通知。
+如果不指定事件，默认订阅: ${defaultConfigEvents.join(', ')}
+
+可选事件:
+- push: 代码推送
+- issues: Issue 创建/关闭/重开
+- issue_comment: Issue 评论
+- pull_request: PR 创建/关闭/重开
+- pull_request_review: PR 审查
+- star: 标星
+- fork: 仓库 Fork
+- release: 发布新版本
+- discussion: 讨论区更新
+- workflow_run: Workflow 运行
+
+示例:
+gh.sub koishijs/koishi
+gh.sub koishijs/koishi push,issues,star
+`)
         .action(async ({ session }, repo, eventsStr) => {
         if (!repo)
             return '请指定仓库名称 (owner/repo)。';
@@ -19,15 +45,46 @@ function apply(ctx) {
             return '该仓库不在信任列表中，无法订阅。请联系管理员添加。';
         }
         // Parse events
-        const events = eventsStr ? eventsStr.split(',').map(e => e.trim()) : ['push', 'issues', 'pull_request']; // Default events
+        let events;
+        if (eventsStr) {
+            // Split by comma, Chinese comma, or whitespace
+            events = eventsStr.split(/[,，\s]+/).map(e => e.trim()).filter(Boolean);
+            // Normalize events (kebab-case to snake_case)
+            events = events.map(e => e.replace(/-/g, '_'));
+            // Validate events
+            const invalidEvents = events.filter(e => !validEvents.includes(e) && e !== '*');
+            if (invalidEvents.length > 0) {
+                return `无效的事件类型: ${invalidEvents.join(', ')}。\n可选事件: ${validEvents.join(', ')}`;
+            }
+        }
+        else {
+            // Default events
+            events = [...defaultConfigEvents];
+        }
         try {
-            await ctx.database.create('github_subscription', {
+            // Check if subscription exists
+            const existing = await ctx.database.get('github_subscription', {
                 repo,
                 channelId: session.channelId,
                 platform: session.platform || 'unknown',
-                events,
             });
-            return `已订阅 ${repo} 的 ${events.join(', ')} 事件。`;
+            if (existing.length > 0) {
+                // Update existing subscription
+                await ctx.database.set('github_subscription', { id: existing[0].id }, {
+                    events,
+                });
+                return `已更新 ${repo} 的订阅，当前监听事件: ${events.join(', ')}。`;
+            }
+            else {
+                // Create new subscription
+                await ctx.database.create('github_subscription', {
+                    repo,
+                    channelId: session.channelId,
+                    platform: session.platform || 'unknown',
+                    events,
+                });
+                return `已订阅 ${repo} 的 ${events.join(', ')} 事件。`;
+            }
         }
         catch (e) {
             logger.warn(e);

package/lib/config.d.ts

@@ -9,6 +9,8 @@ export interface Config {
     defaultOwner?: string;
     defaultRepo?: string;
     debug: boolean;
+    logUnhandledEvents: boolean;
+    defaultEvents: string[];
     rules?: Rule[];
 }
 export declare const Config: Schema<Config>;

package/lib/config.js

@@ -6,6 +6,10 @@ exports.Config = koishi_1.Schema.object({
     defaultOwner: koishi_1.Schema.string().description('默认仓库拥有者'),
     defaultRepo: koishi_1.Schema.string().description('默认仓库名称'),
     debug: koishi_1.Schema.boolean().default(false).description('启用调试模式，输出详细日志'),
+    logUnhandledEvents: koishi_1.Schema.boolean().default(false).description('是否记录未处理的 Webhook 事件 (Unknown events)'),
+    defaultEvents: koishi_1.Schema.array(koishi_1.Schema.string())
+        .default(['push', 'issues', 'issue_comment', 'pull_request', 'pull_request_review', 'release', 'star', 'fork'])
+        .description('默认订阅事件列表 (当不指定事件时使用)'),
     rules: koishi_1.Schema.array(koishi_1.Schema.object({
         repo: koishi_1.Schema.string().required(),
         channelId: koishi_1.Schema.string().required(),

package/lib/services/formatter.js

@@ -8,39 +8,40 @@ class Formatter extends koishi_1.Service {
         ctx.logger('githubsth').info('Formatter service initialized');
     }
     formatPush(payload) {
-        const { repository, pusher, commits, compare } = payload;
+        const { repository, pusher, commits, compare, ref } = payload;
         if (!commits || commits.length === 0)
             return null;
         const commitLines = commits.map((c) => {
-            const shortHash = c.id.substring(0, 7);
-            const message = c.message.split('\n')[0];
-            return `[${shortHash}] ${message} - ${c.author.name}`;
+            const shortHash = c.id?.substring(0, 7) || '???????';
+            const message = c.message?.split('\n')[0] || 'No message';
+            const author = c.author?.name || 'Unknown';
+            return `[${shortHash}] ${message} - ${author}`;
         }).join('\n');
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} 收到新的推送\n` }),
-            (0, koishi_1.h)('text', { content: `提交者: ${pusher.name}\n` }),
-            (0, koishi_1.h)('text', { content: `分支: ${payload.ref.replace('refs/heads/', '')}\n` }),
-            (0, koishi_1.h)('text', { content: `详情: ${compare}\n` }),
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} 收到新的推送\n` }),
+            (0, koishi_1.h)('text', { content: `提交者: ${pusher?.name || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `分支: ${ref ? ref.replace('refs/heads/', '') : 'unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `详情: ${compare || ''}\n` }),
             (0, koishi_1.h)('text', { content: commitLines })
         ]);
     }
     formatIssue(payload) {
         const { action, issue, repository, sender } = payload;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} Issue ${action}\n` }),
-            (0, koishi_1.h)('text', { content: `标题: #${issue.number} ${issue.title}\n` }),
-            (0, koishi_1.h)('text', { content: `发起人: ${sender.login}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${issue.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} Issue ${action || 'updated'}\n` }),
+            (0, koishi_1.h)('text', { content: `标题: #${issue?.number || '?'} ${issue?.title || 'No Title'}\n` }),
+            (0, koishi_1.h)('text', { content: `发起人: ${sender?.login || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${issue?.html_url || ''}` })
         ]);
     }
     formatPullRequest(payload) {
         const { action, pull_request, repository, sender } = payload;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} PR ${action}\n` }),
-            (0, koishi_1.h)('text', { content: `标题: #${pull_request.number} ${pull_request.title}\n` }),
-            (0, koishi_1.h)('text', { content: `发起人: ${sender.login}\n` }),
-            (0, koishi_1.h)('text', { content: `状态: ${pull_request.state}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${pull_request.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} PR ${action || 'updated'}\n` }),
+            (0, koishi_1.h)('text', { content: `标题: #${pull_request?.number || '?'} ${pull_request?.title || 'No Title'}\n` }),
+            (0, koishi_1.h)('text', { content: `发起人: ${sender?.login || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `状态: ${pull_request?.state || 'unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${pull_request?.html_url || ''}` })
         ]);
     }
     formatStar(payload) {
@@ -48,15 +49,15 @@ class Formatter extends koishi_1.Service {
         if (action !== 'created')
             return null;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${sender.login} Star 了仓库 ${repository.full_name} 🌟\n` }),
-            (0, koishi_1.h)('text', { content: `当前 Star 数: ${repository.stargazers_count}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${sender?.login || 'Unknown'} Star 了仓库 ${repository?.full_name || 'Unknown Repo'} 🌟\n` }),
+            (0, koishi_1.h)('text', { content: `当前 Star 数: ${repository?.stargazers_count !== undefined ? repository.stargazers_count : '?'}` })
         ]);
     }
     formatFork(payload) {
         const { forkee, repository, sender } = payload;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${sender.login} Fork 了仓库 ${repository.full_name}\n` }),
-            (0, koishi_1.h)('text', { content: `新仓库: ${forkee.full_name}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${sender?.login || 'Unknown'} Fork 了仓库 ${repository?.full_name || 'Unknown Repo'}\n` }),
+            (0, koishi_1.h)('text', { content: `新仓库: ${forkee?.full_name || 'Unknown'}` })
         ]);
     }
     formatRelease(payload) {
@@ -64,43 +65,45 @@ class Formatter extends koishi_1.Service {
         if (action !== 'published')
             return null;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} 发布了新版本 ${release.tag_name} 🎉\n` }),
-            (0, koishi_1.h)('text', { content: `标题: ${release.name}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${release.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} 发布了新版本 ${release?.tag_name || 'unknown'} 🎉\n` }),
+            (0, koishi_1.h)('text', { content: `标题: ${release?.name || 'No Title'}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${release?.html_url || ''}` })
         ]);
     }
     formatDiscussion(payload) {
         const { action, discussion, repository, sender } = payload;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} Discussion ${action}\n` }),
-            (0, koishi_1.h)('text', { content: `标题: #${discussion.number} ${discussion.title}\n` }),
-            (0, koishi_1.h)('text', { content: `发起人: ${sender.login}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${discussion.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} Discussion ${action || 'updated'}\n` }),
+            (0, koishi_1.h)('text', { content: `标题: #${discussion?.number || '?'} ${discussion?.title || 'No Title'}\n` }),
+            (0, koishi_1.h)('text', { content: `发起人: ${sender?.login || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${discussion?.html_url || ''}` })
         ]);
     }
     formatWorkflowRun(payload) {
         const { action, workflow_run, repository } = payload;
         if (action !== 'completed')
             return null;
-        const statusIcon = workflow_run.conclusion === 'success' ? '✅' : '❌';
+        const statusIcon = workflow_run?.conclusion === 'success' ? '✅' : '❌';
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} 工作流运行完成 ${statusIcon}\n` }),
-            (0, koishi_1.h)('text', { content: `工作流: ${workflow_run.name}\n` }),
-            (0, koishi_1.h)('text', { content: `结果: ${workflow_run.conclusion}\n` }),
-            (0, koishi_1.h)('text', { content: `分支: ${workflow_run.head_branch}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${workflow_run.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} 工作流运行完成 ${statusIcon}\n` }),
+            (0, koishi_1.h)('text', { content: `工作流: ${workflow_run?.name || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `结果: ${workflow_run?.conclusion || 'unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `分支: ${workflow_run?.head_branch || 'unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${workflow_run?.html_url || ''}` })
         ]);
     }
     formatIssueComment(payload) {
         const { action, issue, comment, repository, sender } = payload;
         if (action !== 'created')
             return null;
+        const body = comment?.body || '';
+        const shortBody = body.length > 100 ? body.substring(0, 100) + '...' : body;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} Issue 收到新评论\n` }),
-            (0, koishi_1.h)('text', { content: `标题: #${issue.number} ${issue.title}\n` }),
-            (0, koishi_1.h)('text', { content: `评论人: ${sender.login}\n` }),
-            (0, koishi_1.h)('text', { content: `内容: ${comment.body.substring(0, 100)}${comment.body.length > 100 ? '...' : ''}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${comment.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} Issue 收到新评论\n` }),
+            (0, koishi_1.h)('text', { content: `标题: #${issue?.number || '?'} ${issue?.title || 'No Title'}\n` }),
+            (0, koishi_1.h)('text', { content: `评论人: ${sender?.login || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `内容: ${shortBody}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${comment?.html_url || ''}` })
         ]);
     }
     formatPullRequestReview(payload) {
@@ -108,11 +111,11 @@ class Formatter extends koishi_1.Service {
         if (action !== 'submitted')
             return null;
         return (0, koishi_1.h)('message', [
-            (0, koishi_1.h)('text', { content: `[GitHub] ${repository.full_name} PR 收到 Review\n` }),
-            (0, koishi_1.h)('text', { content: `标题: #${pull_request.number} ${pull_request.title}\n` }),
-            (0, koishi_1.h)('text', { content: `Reviewer: ${sender.login}\n` }),
-            (0, koishi_1.h)('text', { content: `状态: ${review.state}\n` }),
-            (0, koishi_1.h)('text', { content: `链接: ${review.html_url}` })
+            (0, koishi_1.h)('text', { content: `[GitHub] ${repository?.full_name || 'Unknown Repo'} PR 收到 Review\n` }),
+            (0, koishi_1.h)('text', { content: `标题: #${pull_request?.number || '?'} ${pull_request?.title || 'No Title'}\n` }),
+            (0, koishi_1.h)('text', { content: `Reviewer: ${sender?.login || 'Unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `状态: ${review?.state || 'unknown'}\n` }),
+            (0, koishi_1.h)('text', { content: `链接: ${review?.html_url || ''}` })
         ]);
     }
 }

package/lib/services/notifier.d.ts

@@ -10,5 +10,6 @@ export declare class Notifier extends Service {
     constructor(ctx: Context, config: Config);
     private registerListeners;
     private handleEvent;
+    private patchPayloadForEvent;
     private sendMessage;
 }

package/lib/services/notifier.js

@@ -34,31 +34,40 @@ class Notifier extends koishi_1.Service {
                 if (this.config.debug) {
                     this.ctx.logger('githubsth').info('Found payload in session, attempting to handle');
                 }
+                // Check if payload is wrapped (adapter-github structure)
+                const realPayload = payload.payload || payload;
                 // Infer event type
                 let eventType = 'unknown';
-                if (payload.issue && payload.comment)
+                // Check inner payload first if it exists
+                if (realPayload.issue && realPayload.comment)
                     eventType = 'issue_comment';
-                else if (payload.issue)
+                else if (realPayload.issue)
                     eventType = 'issues';
-                else if (payload.pull_request && payload.review)
+                else if (realPayload.pull_request && realPayload.review)
                     eventType = 'pull_request_review';
-                else if (payload.pull_request)
+                else if (realPayload.pull_request)
                     eventType = 'pull_request';
-                else if (payload.commits)
+                else if (realPayload.commits)
                     eventType = 'push';
-                else if (payload.starred_at !== undefined || (payload.action === 'started'))
+                else if (realPayload.starred_at !== undefined || (realPayload.action === 'started'))
                     eventType = 'star'; // star event usually has action 'created' but check payload structure
-                else if (payload.forkee)
+                else if (realPayload.forkee)
                     eventType = 'fork';
-                else if (payload.release)
+                else if (realPayload.release)
                     eventType = 'release';
-                else if (payload.discussion)
+                else if (realPayload.discussion)
                     eventType = 'discussion';
-                else if (payload.workflow_run)
+                else if (realPayload.workflow_run)
                     eventType = 'workflow_run';
+                // Handle raw star event if it has repository info directly
+                else if (realPayload.repository && (realPayload.action === 'created' || realPayload.action === 'started'))
+                    eventType = 'star';
                 if (eventType !== 'unknown') {
                     this.handleEvent(eventType, payload);
                 }
+                else if (this.config.logUnhandledEvents) {
+                    this.ctx.logger('githubsth').info(`Unhandled payload structure. Keys: ${Object.keys(realPayload).join(', ')}`);
+                }
             }
         });
     }
@@ -69,6 +78,14 @@ class Notifier extends koishi_1.Service {
         // Check if payload is nested in an 'event' object (common in some adapter versions)
         // or if the event data is directly in payload
         const realPayload = payload.payload || payload;
+        // Extract sender from wrapper if available (adapter-github often puts it in 'actor')
+        if (payload.actor && !realPayload.sender) {
+            realPayload.sender = payload.actor;
+        }
+        // Extract repository from wrapper if available
+        if (payload.repository && !realPayload.repository) {
+            realPayload.repository = payload.repository;
+        }
         let repoName = realPayload.repository?.full_name;
         // Try to fallback if repoName is missing
         if (!repoName && realPayload.issue?.repository_url) {
@@ -80,17 +97,57 @@ class Notifier extends koishi_1.Service {
         if (!repoName && realPayload.pull_request?.base?.repo?.full_name) {
             repoName = realPayload.pull_request.base.repo.full_name;
         }
+        // Special handling for 'star' event (which might be 'watch' event with action 'started')
+        // The payload might be missing repository info in the main object but have it in the original session payload
+        if (!repoName && event === 'star') {
+            // Sometimes the repository info is at the root of the payload, not inside 'payload' property
+            if (payload.repository?.full_name) {
+                repoName = payload.repository.full_name;
+            }
+        }
         if (!repoName) {
             if (this.config.debug) {
-                this.ctx.logger('githubsth').warn(`Missing repo info for event: ${event}`);
-                this.ctx.logger('notifier').warn(`Event ${event} missing repository info. Keys: ${Object.keys(realPayload).join(', ')}`);
+                this.ctx.logger('githubsth').warn(`Missing repo info for event: ${event}. Keys: ${Object.keys(realPayload).join(', ')}`);
             }
-            else {
-                // Log at warning level even in production if repo info is missing, as this is a critical failure
-                // But only log keys to avoid leaking sensitive data
+            else if (this.config.logUnhandledEvents) {
+                // Log at warning level if repo info is missing and logUnhandledEvents is on
                 this.ctx.logger('githubsth').warn(`Missing repo info for event: ${event}. Keys: ${Object.keys(realPayload).join(', ')}`);
             }
-            return;
+            // Do not return here, let patching logic handle it with defaultRepo
+        }
+        // Patch realPayload with extracted repo info if missing
+        // This is crucial for formatter to work correctly as it expects repository object
+        if (!realPayload.repository) {
+            realPayload.repository = { full_name: repoName || 'Unknown/Repo' };
+        }
+        else if (!realPayload.repository.full_name) {
+            realPayload.repository.full_name = repoName || 'Unknown/Repo';
+        }
+        // Patch realPayload with sender info if missing (e.g. issues event)
+        if (!realPayload.sender) {
+            if (realPayload.issue?.user) {
+                realPayload.sender = realPayload.issue.user;
+            }
+            else if (realPayload.pull_request?.user) {
+                realPayload.sender = realPayload.pull_request.user;
+            }
+            else if (realPayload.discussion?.user) {
+                realPayload.sender = realPayload.discussion.user;
+            }
+            else if (realPayload.pusher) {
+                realPayload.sender = { login: realPayload.pusher.name || 'Pusher' };
+            }
+            else {
+                // Fallback sender
+                realPayload.sender = { login: 'GitHub' };
+            }
+        }
+        // Comprehensive patching for specific events to prevent formatter crashes
+        try {
+            this.patchPayloadForEvent(event, realPayload, repoName || 'Unknown/Repo');
+        }
+        catch (e) {
+            this.ctx.logger('githubsth').warn(`Failed to patch payload for ${event}:`, e);
         }
         if (this.config.debug) {
             this.ctx.logger('githubsth').info(`Processing event ${event} for ${repoName}`);
@@ -99,6 +156,12 @@ class Notifier extends koishi_1.Service {
         }
         // Get rules from database
         // Try to match both exact name and lowercase name to handle case sensitivity
+        // If repoName is missing (undefined), we can't query rules effectively by repo name
+        // But we might want to support global subscriptions or handle it gracefully
+        if (!repoName) {
+            this.ctx.logger('githubsth').warn('Cannot query rules: repoName is missing');
+            return;
+        }
         const repoNames = [repoName];
         if (repoName !== repoName.toLowerCase()) {
             repoNames.push(repoName.toLowerCase());
@@ -126,6 +189,9 @@ class Notifier extends koishi_1.Service {
                 this.ctx.logger('githubsth').info(`No matching rules for ${repoName} (event: ${event})`);
                 this.ctx.logger('notifier').debug(`No matching rules for ${repoName} (event: ${event})`);
             }
+            else if (this.config.logUnhandledEvents) {
+                this.ctx.logger('githubsth').warn(`No matching rules for ${repoName} (event: ${event})`);
+            }
             return;
         }
         if (this.config.debug) {
@@ -138,50 +204,160 @@ class Notifier extends koishi_1.Service {
             this.ctx.logger('notifier').warn('Formatter service not available');
             return;
         }
+        try {
+            switch (event) {
+                case 'push':
+                    message = this.ctx.githubsthFormatter.formatPush(realPayload);
+                    break;
+                case 'issues':
+                    message = this.ctx.githubsthFormatter.formatIssue(realPayload);
+                    break;
+                case 'pull_request':
+                    message = this.ctx.githubsthFormatter.formatPullRequest(realPayload);
+                    break;
+                case 'star':
+                    message = this.ctx.githubsthFormatter.formatStar(realPayload);
+                    break;
+                case 'fork':
+                    message = this.ctx.githubsthFormatter.formatFork(realPayload);
+                    break;
+                case 'release':
+                    message = this.ctx.githubsthFormatter.formatRelease(realPayload);
+                    break;
+                case 'discussion':
+                    message = this.ctx.githubsthFormatter.formatDiscussion(realPayload);
+                    break;
+                case 'workflow_run':
+                    message = this.ctx.githubsthFormatter.formatWorkflowRun(realPayload);
+                    break;
+                case 'issue_comment':
+                    message = this.ctx.githubsthFormatter.formatIssueComment(realPayload);
+                    break;
+                case 'pull_request_review':
+                    message = this.ctx.githubsthFormatter.formatPullRequestReview(realPayload);
+                    break;
+            }
+        }
+        catch (e) {
+            this.ctx.logger('githubsth').error(`Error formatting event ${event}:`, e);
+            if (this.config.debug) {
+                this.ctx.logger('notifier').error(`Error formatting event ${event}:`, e);
+            }
+            return;
+        }
+        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);
+        }
+    }
+    patchPayloadForEvent(event, payload, repoName) {
+        // Ensure sender exists (handled before, but good for type safety)
+        const defaultUser = { login: 'GitHub', id: 0, avatar_url: '' };
+        if (!payload.sender)
+            payload.sender = defaultUser;
+        // Ensure repository exists (handled before, but good for type safety)
+        const defaultRepo = { full_name: repoName, stargazers_count: 0, html_url: `https://github.com/${repoName}` };
+        if (!payload.repository)
+            payload.repository = defaultRepo;
         switch (event) {
             case 'push':
-                message = this.ctx.githubsthFormatter.formatPush(realPayload);
+                if (!payload.pusher)
+                    payload.pusher = { name: payload.sender.login };
+                if (!payload.commits)
+                    payload.commits = [];
+                if (!payload.ref)
+                    payload.ref = 'refs/heads/unknown';
+                if (!payload.compare)
+                    payload.compare = '';
+                // Ensure author exists in commits
+                if (payload.commits.length > 0) {
+                    payload.commits.forEach((c) => {
+                        if (!c.author)
+                            c.author = { name: 'Unknown' };
+                        if (!c.id)
+                            c.id = '0000000';
+                        if (!c.message)
+                            c.message = 'No message';
+                    });
+                }
                 break;
             case 'issues':
-                message = this.ctx.githubsthFormatter.formatIssue(realPayload);
+                if (!payload.action)
+                    payload.action = 'updated';
+                if (!payload.issue)
+                    payload.issue = { number: 0, title: 'Unknown Issue', html_url: '', user: payload.sender };
+                // Ensure user exists in issue
+                if (!payload.issue.user)
+                    payload.issue.user = payload.sender;
                 break;
             case 'pull_request':
-                message = this.ctx.githubsthFormatter.formatPullRequest(realPayload);
+                if (!payload.action)
+                    payload.action = 'updated';
+                if (!payload.pull_request)
+                    payload.pull_request = { number: 0, title: 'Unknown PR', state: 'unknown', html_url: '', user: payload.sender };
+                if (!payload.pull_request.user)
+                    payload.pull_request.user = payload.sender;
                 break;
             case 'star':
-                message = this.ctx.githubsthFormatter.formatStar(realPayload);
+                if (!payload.action)
+                    payload.action = 'created';
+                if (payload.repository && payload.repository.stargazers_count === undefined) {
+                    payload.repository.stargazers_count = '?';
+                }
                 break;
             case 'fork':
-                message = this.ctx.githubsthFormatter.formatFork(realPayload);
+                if (!payload.forkee)
+                    payload.forkee = { full_name: 'unknown/fork' };
                 break;
             case 'release':
-                message = this.ctx.githubsthFormatter.formatRelease(realPayload);
+                if (!payload.action)
+                    payload.action = 'published';
+                if (!payload.release)
+                    payload.release = { tag_name: 'unknown', name: 'Unknown Release', html_url: '' };
                 break;
             case 'discussion':
-                message = this.ctx.githubsthFormatter.formatDiscussion(realPayload);
+                if (!payload.action)
+                    payload.action = 'updated';
+                if (!payload.discussion)
+                    payload.discussion = { number: 0, title: 'Unknown Discussion', html_url: '', user: payload.sender };
+                if (!payload.discussion.user)
+                    payload.discussion.user = payload.sender;
                 break;
             case 'workflow_run':
-                message = this.ctx.githubsthFormatter.formatWorkflowRun(realPayload);
+                if (!payload.action)
+                    payload.action = 'completed';
+                if (!payload.workflow_run)
+                    payload.workflow_run = { conclusion: 'unknown', name: 'Unknown Workflow', head_branch: 'unknown', html_url: '' };
                 break;
             case 'issue_comment':
-                message = this.ctx.githubsthFormatter.formatIssueComment(realPayload);
+                if (!payload.action)
+                    payload.action = 'created';
+                if (!payload.issue)
+                    payload.issue = { number: 0, title: 'Unknown Issue', html_url: '', user: payload.sender };
+                if (!payload.comment)
+                    payload.comment = { body: '', html_url: '' };
+                if (!payload.issue.user)
+                    payload.issue.user = payload.sender;
                 break;
             case 'pull_request_review':
-                message = this.ctx.githubsthFormatter.formatPullRequestReview(realPayload);
+                if (!payload.action)
+                    payload.action = 'submitted';
+                if (!payload.pull_request)
+                    payload.pull_request = { number: 0, title: 'Unknown PR', html_url: '', user: payload.sender };
+                if (!payload.review)
+                    payload.review = { state: 'unknown', html_url: '' };
+                if (!payload.pull_request.user)
+                    payload.pull_request.user = payload.sender;
                 break;
         }
-        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);
-        }
     }
     async sendMessage(rule, message) {
         // Find suitable bots

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "koishi-plugin-githubsth",
-  "version": "1.0.2-test1",
+  "version": "1.0.2",
   "description": "Github Subscriptions Notifications, push notifications for GitHub subscriptions For koishi",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",