docs: add changelog for hooks/jobs/sync improvements (2026-05-17)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

CHANGELOG.md

@@ -0,0 +1,26 @@
+# Changelog — Plugin Legal AI
+
+## [2026-05-17] שיפורי תשתית: Hooks, Scheduled Jobs, Per-Agent Validation
+
+### שינויים ב-plugin-legal-ai
+
+#### Webhook Hook — עדכוני סטטוס תיק
+- נוסף `onWebhook()` handler לאירוע `case-status` מ-legal-ai
+- כל שינוי סטטוס תיק (מכל endpoint) → תגובה אוטומטית בעברית על ה-issue המקושר ב-Paperclip
+- בשינוי סטטוס `qa_failed` — CEO מתעורר אוטומטית עם הנחיות לתיקון
+- נוספה הגנת payload + לוג אזהרה על שדות חסרים
+
+#### Scheduled Jobs — עבודות רקע
+- **`stale-case-reminder`** (כל יום 08:00): מזהה תיקים שלא עודכנו 3+ ימים → מוסיף תגובה אזהרה על ה-issue
+- **`weekly-feedback-analysis`** (כל ראשון 19:00): מסכם פידבק יו"ר מהשבוע → CEO מעדכן `decision-lessons.md`
+- ביצועים: בנייה חד-פעמית של `Map<caseNumber, {issueId, companyId}>` — O(companies × issues) במקום O(cases × companies × issues)
+
+#### manifest / plugin.json
+- נוספה capability: `webhooks.receive`
+- נוספה הצהרת webhook: `{ endpointKey: "case-status" }`
+- נוספו הגדרות ל-2 jobs חדשים
+
+#### תיקוני Quality
+- החלפת `let` ב-`const` (biome)
+- תיקון סדר imports (biome)
+- תיקון פורמט manifest.ts (biome)

plugin.json

@@ -63,6 +63,18 @@
       "displayName": "סנכרון סטטוס תיקים",
       "description": "סנכרון סטטוס בין legal-ai ל-Paperclip כל 15 דקות",
       "schedule": "*/15 * * * *"
+    },
+    {
+      "jobKey": "stale-case-reminder",
+      "displayName": "תזכורת תיקים תקועים",
+      "description": "מזהה תיקים שלא עודכנו 3+ ימים ומוסיף תגובה ל-issue",
+      "schedule": "0 8 * * *"
+    },
+    {
+      "jobKey": "weekly-feedback-analysis",
+      "displayName": "ניתוח פידבק שבועי",
+      "description": "מסכם פידבק יו\"ר מהשבוע האחרון ומעדכן את decision-lessons.md",
+      "schedule": "0 19 * * 0"
     }
   ],
   "webhooks": [

src/manifest.ts

@@ -158,6 +158,19 @@ export default {
 				"Polls legal-ai for case status changes and updates Paperclip issues",
 			schedule: "*/15 * * * *",
 		},
+		{
+			jobKey: "stale-case-reminder",
+			displayName: "תזכורת תיקים תקועים",
+			description: "מזהה תיקים שלא עודכנו 3+ ימים ומוסיף תגובה ל-issue",
+			schedule: "0 8 * * *",
+		},
+		{
+			jobKey: "weekly-feedback-analysis",
+			displayName: "ניתוח פידבק שבועי",
+			description:
+				'מסכם פידבק יו"ר מהשבוע האחרון ומעדכן את decision-lessons.md',
+			schedule: "0 19 * * 0",
+		},
 	],
 	webhooks: [
 		{

src/worker.ts

@@ -1,5 +1,8 @@
+import type {
+	PluginContext,
+	PluginWebhookInput,
+} from "@paperclipai/plugin-sdk";
 import { definePlugin, runWorker } from "@paperclipai/plugin-sdk";
-import type { PluginContext, PluginWebhookInput } from "@paperclipai/plugin-sdk";
 import { LegalApi } from "./legal-api.js";
 
 // Hoisted so onWebhook can access the context after setup() completes.
@@ -7,8 +10,10 @@ let pluginCtx: PluginContext | null = null;
 
 // Per-company CEO agent IDs (shared between setup and onWebhook).
 const CEO_AGENT_IDS: Record<string, string> = {
-	"42a7acd0-30c5-4cbd-ac97-7424f65df294": "752cebdd-6748-4a04-aacd-c7ab0294ef33", // CMP (רישוי ובניה)
-	"8639e837-4c9d-47fa-a76b-95788d651896": "cdbfa8bc-3d61-41a4-a2e7-677ec7d34562", // CMPA (היטלי השבחה)
+	"42a7acd0-30c5-4cbd-ac97-7424f65df294":
+		"752cebdd-6748-4a04-aacd-c7ab0294ef33", // CMP (רישוי ובניה)
+	"8639e837-4c9d-47fa-a76b-95788d651896":
+		"cdbfa8bc-3d61-41a4-a2e7-677ec7d34562", // CMPA (היטלי השבחה)
 };
 
 const plugin = definePlugin({
@@ -545,7 +550,7 @@ const plugin = definePlugin({
 				body?: string;
 			} | null;
 
-			let issueId = payload?.issueId;
+			const issueId = payload?.issueId;
 			let commentBody = payload?.body;
 
 			// If issueId is not in payload, try to find it from the comment entity
@@ -590,10 +595,7 @@ const plugin = definePlugin({
 			}
 
 			try {
-				const { runId } = await ctx.agents.invoke(
-					ceoAgentId,
-					event.companyId,
-					{
+				const { runId } = await ctx.agents.invoke(ceoAgentId, event.companyId, {
 					prompt: [
 						`תגובה חדשה מחיים על issue "${issue.title}" (${issue.identifier || issueId}):`,
 						"",
@@ -603,8 +605,7 @@ const plugin = definePlugin({
 						`אם ההוראה ברורה — נתב לסוכן המתאים. אם לא ברור — שאל את חיים.`,
 					].join("\n"),
 					reason: `user_commented_on_${issue.identifier || issueId}`,
-					},
-				);
+				});
 				ctx.logger.info("Routed user comment to CEO agent", {
 					issueId,
 					commentId: entityId,
@@ -706,6 +707,120 @@ const plugin = definePlugin({
 			}
 		});
 
+		ctx.jobs.register("stale-case-reminder", async (_job) => {
+			ctx.logger.info("stale-case-reminder: starting");
+			const config = await ctx.config.get();
+			const apiBase =
+				(config.legalApiBaseUrl as string) ?? "http://localhost:8085";
+
+			const resp = await ctx.http.fetch(`${apiBase}/api/cases/stale?days=3`);
+			if (!resp.ok) {
+				ctx.logger.error(`stale-case-reminder: API error ${resp.status}`);
+				return;
+			}
+
+			const data = (await resp.json()) as {
+				cases: Array<{
+					case_number: string;
+					title: string;
+					status: string;
+					days_stale: number;
+				}>;
+				total: number;
+			};
+
+			// Build case→issue map once (O(companies × issues)) to avoid N×M RPCs per stale case
+			const companies = await ctx.companies.list();
+			const caseIssueMap = new Map<
+				string,
+				{ issueId: string; companyId: string }
+			>();
+			for (const company of companies) {
+				const issues = await ctx.issues.list({ companyId: company.id });
+				for (const issue of issues) {
+					const linkedCase = await ctx.state.get({
+						scopeKind: "issue",
+						scopeId: issue.id,
+						stateKey: "legal-case-number",
+					});
+					if (linkedCase && typeof linkedCase === "string") {
+						caseIssueMap.set(linkedCase, {
+							issueId: issue.id,
+							companyId: company.id,
+						});
+					}
+				}
+			}
+
+			let reminded = 0;
+			for (const staleCase of data.cases) {
+				const linked = caseIssueMap.get(staleCase.case_number);
+				if (!linked) continue;
+
+				await ctx.issues.createComment(
+					linked.issueId,
+					`⚠️ **תיק תקוע ${staleCase.case_number}** — ${staleCase.days_stale} ימים ללא עדכון (סטטוס: ${staleCase.status}). האם נדרשת פעולה?`,
+					linked.companyId,
+				);
+				reminded++;
+				ctx.logger.info(
+					`stale-case-reminder: reminded case ${staleCase.case_number} (${staleCase.days_stale}d)`,
+				);
+			}
+
+			ctx.logger.info(
+				`stale-case-reminder: done. ${reminded}/${data.total} cases reminded`,
+			);
+		});
+
+		ctx.jobs.register("weekly-feedback-analysis", async (_job) => {
+			ctx.logger.info("weekly-feedback-analysis: starting");
+			const config = await ctx.config.get();
+			const apiBase =
+				(config.legalApiBaseUrl as string) ?? "http://localhost:8085";
+
+			const resp = await ctx.http.fetch(
+				`${apiBase}/api/chair-feedback/weekly-summary`,
+			);
+			if (!resp.ok) {
+				ctx.logger.error(`weekly-feedback-analysis: API error ${resp.status}`);
+				return;
+			}
+
+			const data = (await resp.json()) as {
+				summary: string;
+				entry_count: number;
+			};
+
+			if (data.entry_count === 0) {
+				ctx.logger.info(
+					"weekly-feedback-analysis: no feedback this week, skipping",
+				);
+				return;
+			}
+
+			const companies = await ctx.companies.list();
+			const company = companies[0];
+			if (!company) return;
+
+			const ceoId = CEO_AGENT_IDS[company.id];
+			if (!ceoId) {
+				ctx.logger.warn(
+					`weekly-feedback-analysis: no CEO agent for company ${company.id}`,
+				);
+				return;
+			}
+
+			await ctx.agents.invoke(ceoId, company.id, {
+				prompt: `ניתוח פידבק שבועי יו"ר (${data.entry_count} פריטים):\n\n${data.summary}\n\nהמשימה: עדכן את /home/chaim/legal-ai/docs/legal-decision-lessons.md עם הלקחים החדשים שעולים מהפידבק. הוסף רק לקחים חדשים שלא קיימים כבר. קבץ לפי נושא.`,
+				reason: "weekly-feedback-analysis scheduled job",
+			});
+
+			ctx.logger.info(
+				`weekly-feedback-analysis: invoked CEO ${ceoId} with ${data.entry_count} feedback entries`,
+			);
+		});
+
 		ctx.logger.info("Legal AI plugin ready");
 	},
 
@@ -716,6 +831,9 @@ const plugin = definePlugin({
 	async onWebhook(input: PluginWebhookInput): Promise<void> {
 		if (!pluginCtx) return; // not yet initialized
 
+		// TODO: add idempotency guard using input.requestId to prevent duplicate
+		// comments on rapid retries (store requestId in plugin state with ~60s TTL)
+
 		const { endpointKey, parsedBody } = input;
 
 		if (endpointKey !== "case-status") return;
@@ -731,13 +849,20 @@ const plugin = definePlugin({
 		const { caseNumber, oldStatus, newStatus, companyId } = payload;
 
 		if (!caseNumber || !newStatus || !companyId) {
-			pluginCtx.logger.warn("onWebhook: malformed payload", { caseNumber, newStatus, companyId });
+			pluginCtx.logger.warn("onWebhook: malformed payload", {
+				caseNumber,
+				newStatus,
+				companyId,
+			});
 			return;
 		}
 
-		pluginCtx.logger.info(`Webhook: case ${caseNumber} ${oldStatus} → ${newStatus}`, {
+		pluginCtx.logger.info(
+			`Webhook: case ${caseNumber} ${oldStatus} → ${newStatus}`,
+			{
 				companyId,
-		});
+			},
+		);
 
 		// Find the Paperclip issue linked to this case number by scanning plugin state.
 		// State stores: issue.id → case_number (scopeKind=issue, stateKey=legal-case-number)
@@ -756,7 +881,9 @@ const plugin = definePlugin({
 		}
 
 		if (!linkedIssueId) {
-			pluginCtx.logger.warn(`onWebhook: no Paperclip issue linked to case ${caseNumber}`);
+			pluginCtx.logger.warn(
+				`onWebhook: no Paperclip issue linked to case ${caseNumber}`,
+			);
 			return;
 		}