Compare commits
3 Commits
6721c0a4ed
...
fix/604-pa
| Author | SHA1 | Date | |
|---|---|---|---|
| 0a134d0134 | |||
| 602b42ca5e | |||
| 378e5657b5 |
@@ -36,6 +36,10 @@ export class LegalApi {
|
|||||||
return this.request("/api/cases");
|
return this.request("/api/cases");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async getStatusModel(): Promise<StatusModel> {
|
||||||
|
return this.request("/api/status-model");
|
||||||
|
}
|
||||||
|
|
||||||
async getCase(caseNumber: string): Promise<CaseDetails> {
|
async getCase(caseNumber: string): Promise<CaseDetails> {
|
||||||
return this.request(`/api/cases/${encodeURIComponent(caseNumber)}/details`);
|
return this.request(`/api/cases/${encodeURIComponent(caseNumber)}/details`);
|
||||||
}
|
}
|
||||||
@@ -184,6 +188,21 @@ export interface CaseSummary {
|
|||||||
status: string;
|
status: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
export interface StatusModelEntry {
|
||||||
|
key: string;
|
||||||
|
label: string;
|
||||||
|
description: string;
|
||||||
|
phase: string;
|
||||||
|
selectable: boolean;
|
||||||
|
terminal: boolean;
|
||||||
|
on_enter: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface StatusModel {
|
||||||
|
statuses: StatusModelEntry[];
|
||||||
|
phases: Array<{ key: string; label: string }>;
|
||||||
|
}
|
||||||
|
|
||||||
export interface CaseDetails {
|
export interface CaseDetails {
|
||||||
id: string;
|
id: string;
|
||||||
case_number: string;
|
case_number: string;
|
||||||
|
|||||||
326
src/sync-target.test.ts
Normal file
326
src/sync-target.test.ts
Normal file
@@ -0,0 +1,326 @@
|
|||||||
|
/// <reference types="node" />
|
||||||
|
|
||||||
|
import assert from "node:assert/strict";
|
||||||
|
import { test } from "node:test";
|
||||||
|
// Excluded from tsc (tsconfig.json) — never bundled/emitted, run natively via
|
||||||
|
// `node --test`, which requires the literal `.ts` extension (Node's ESM
|
||||||
|
// resolver does not remap `.js` specifiers to `.ts` files at runtime).
|
||||||
|
import type { StatusModelEntry } from "./legal-api.ts";
|
||||||
|
import {
|
||||||
|
labelFor,
|
||||||
|
pickSyncTargetIssue,
|
||||||
|
resolveIssueStatus,
|
||||||
|
type SyncCandidate,
|
||||||
|
} from "./sync-target.ts";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* הכלל הישן (worker.ts:894 לפני #446): כל issue מקושר שסטטוסו שונה מהיעד —
|
||||||
|
* נכתב, בלי סינון parentId ובלי סינון-סטטוס.
|
||||||
|
*/
|
||||||
|
function legacyTargets(
|
||||||
|
candidates: readonly SyncCandidate[],
|
||||||
|
targetStatus: string,
|
||||||
|
): string[] {
|
||||||
|
return candidates.filter((c) => c.status !== targetStatus).map((c) => c.id);
|
||||||
|
}
|
||||||
|
|
||||||
|
test("רגרסיית 8125-09-24 / CMPA-140: 11 ה-issues הממשיים של התיק, אף sub-task done לא נבחר", () => {
|
||||||
|
// נמדד ב-DB החי של Paperclip (2026-08-24) — כל ה-issues המקושרים לתיק
|
||||||
|
// 8125-09-24. CMPA-140 (זה שנדרס בפועל, done→in_progress) הוא **ילד** של
|
||||||
|
// CMPA-139, לא שורש. CMPA-139 עצמו אינו ברשימה בכוונה: נמדד שאין לו
|
||||||
|
// רשומת legal-case-number ב-plugin_state, ולכן הוא לא מגיע כמועמד כלל —
|
||||||
|
// לילד יש הורה שאינו מקושר.
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "CMPA-116", status: "cancelled", parentId: null, companyId: "c1" },
|
||||||
|
{ id: "CMPA-84", status: "cancelled", parentId: null, companyId: "c1" },
|
||||||
|
{ id: "CMPA-86", status: "cancelled", parentId: null, companyId: "c1" },
|
||||||
|
{ id: "CMPA-140", status: "done", parentId: "CMPA-139", companyId: "c1" }, // זה שנדרס בפועל
|
||||||
|
{
|
||||||
|
id: "CMPA-141",
|
||||||
|
status: "cancelled",
|
||||||
|
parentId: "CMPA-139",
|
||||||
|
companyId: "c1",
|
||||||
|
},
|
||||||
|
{ id: "CMPA-85", status: "done", parentId: "CMPA-84", companyId: "c1" },
|
||||||
|
{ id: "CMPA-146", status: "done", parentId: "CMPA-84", companyId: "c1" },
|
||||||
|
{ id: "CMPA-147", status: "done", parentId: "CMPA-84", companyId: "c1" },
|
||||||
|
{ id: "CMPA-87", status: "done", parentId: "CMPA-86", companyId: "c1" },
|
||||||
|
{ id: "CMPA-88", status: "done", parentId: "CMPA-86", companyId: "c1" },
|
||||||
|
{
|
||||||
|
id: "CMPA-89",
|
||||||
|
status: "cancelled",
|
||||||
|
parentId: "CMPA-86",
|
||||||
|
companyId: "c1",
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
|
// סטטוס-התיק היה drafted → in_progress (CASE_STATUS_TO_ISSUE_STATUS ב-worker.ts).
|
||||||
|
const targetStatus = "in_progress";
|
||||||
|
|
||||||
|
// הכלל הישן היה כותב על CMPA-140 — זה הבאג שתועד.
|
||||||
|
assert.ok(legacyTargets(candidates, targetStatus).includes("CMPA-140"));
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
// הכלל החדש אינו כותב על דבר.
|
||||||
|
assert.equal(result.target, null);
|
||||||
|
assert.equal(result.reason, "no_writable_root");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("sub-task יחיד ב-done ומקושר: sub-task לעולם לא נבחר", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "sub-1", status: "done", parentId: "some-root", companyId: "c1" },
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target, null);
|
||||||
|
assert.equal(result.reason, "no_writable_root");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("sub-task יחיד פתוח (in_progress) + שורש cancelled: פתוח אינו מספיק, חייב שורש", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "root-1", status: "cancelled", parentId: null, companyId: "c1" },
|
||||||
|
{ id: "sub-1", status: "in_progress", parentId: "root-1", companyId: "c1" },
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target, null);
|
||||||
|
assert.equal(result.reason, "no_writable_root");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("שורש יחיד ב-todo: נבחר", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "root-1", status: "todo", parentId: null, companyId: "c1" },
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target?.id, "root-1");
|
||||||
|
assert.equal(result.reason, "ok");
|
||||||
|
assert.equal(result.writableRoots, 1);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("שורש יחיד ב-in_review: לא נכתב (הגנת auto-block)", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "root-1", status: "in_review", parentId: null, companyId: "c1" },
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target, null);
|
||||||
|
assert.equal(result.reason, "no_writable_root");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("שורש יחיד ב-blocked: לא נכתב", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "root-1", status: "blocked", parentId: null, companyId: "c1" },
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target, null);
|
||||||
|
assert.equal(result.reason, "no_writable_root");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("שני שורשים ברי-כתיבה: ambiguous, לא נכתב", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "root-1", status: "todo", parentId: null, companyId: "c1" },
|
||||||
|
{ id: "root-2", status: "in_progress", parentId: null, companyId: "c1" },
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target, null);
|
||||||
|
assert.equal(result.reason, "ambiguous_writable_roots");
|
||||||
|
assert.equal(result.writableRoots, 2);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("שורש todo + ילד in_progress: השורש נבחר", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{ id: "root-1", status: "todo", parentId: null, companyId: "c1" },
|
||||||
|
{
|
||||||
|
id: "child-1",
|
||||||
|
status: "in_progress",
|
||||||
|
parentId: "root-1",
|
||||||
|
companyId: "c1",
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target?.id, "root-1");
|
||||||
|
assert.equal(result.reason, "ok");
|
||||||
|
});
|
||||||
|
|
||||||
|
/**
|
||||||
|
* פיקסצ'ר ריאלי של `/api/status-model` (case_status_model.py) — הסדר קובע
|
||||||
|
* מי הוא "todo" (האינדקס הראשון), ו-`terminal: true` קובע מי "done".
|
||||||
|
*/
|
||||||
|
const STATUS_MODEL: StatusModelEntry[] = [
|
||||||
|
{
|
||||||
|
key: "new",
|
||||||
|
label: "חדש",
|
||||||
|
description: "תיק נפתח, טרם הועלו מסמכים",
|
||||||
|
phase: "intake",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "processing",
|
||||||
|
label: "בעיבוד",
|
||||||
|
description: "מסמכים בעיבוד",
|
||||||
|
phase: "intake",
|
||||||
|
selectable: false,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "documents_ready",
|
||||||
|
label: "מסמכים מוכנים",
|
||||||
|
description: "העיבוד הושלם",
|
||||||
|
phase: "intake",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "analyst_verified",
|
||||||
|
label: 'אומת ע"י אנליסט',
|
||||||
|
description: "אנליסט אימת את התוכן",
|
||||||
|
phase: "analysis",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "research_complete",
|
||||||
|
label: "מחקר הושלם",
|
||||||
|
description: "מחקר תקדימים הושלם",
|
||||||
|
phase: "analysis",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "outcome_set",
|
||||||
|
label: "תוצאה נקבעה",
|
||||||
|
description: "תוצאה הוזנה",
|
||||||
|
phase: "analysis",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "direction_approved",
|
||||||
|
label: "כיוון אושר",
|
||||||
|
description: "כיוון הכתיבה אושר",
|
||||||
|
phase: "drafting",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "qa_review",
|
||||||
|
label: "בדיקת איכות",
|
||||||
|
description: "בבדיקת איכות",
|
||||||
|
phase: "drafting",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "drafted",
|
||||||
|
label: "טיוטה מוכנה",
|
||||||
|
description: "טיוטה נכתבה",
|
||||||
|
phase: "drafting",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "exported",
|
||||||
|
label: "יוצא ל-DOCX",
|
||||||
|
description: "יוצא כקובץ DOCX",
|
||||||
|
phase: "review",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "reviewed",
|
||||||
|
label: 'נבדק ע"י דפנה',
|
||||||
|
description: "דפנה הגיהה",
|
||||||
|
phase: "review",
|
||||||
|
selectable: true,
|
||||||
|
terminal: false,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "final",
|
||||||
|
label: "סופי",
|
||||||
|
description: "גרסה סופית",
|
||||||
|
phase: "final",
|
||||||
|
selectable: true,
|
||||||
|
terminal: true,
|
||||||
|
on_enter: null,
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
|
test("resolveIssueStatus: הסטטוס הראשון ברשימה (new) → todo", () => {
|
||||||
|
assert.equal(resolveIssueStatus(STATUS_MODEL, "new"), "todo");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("resolveIssueStatus: סטטוס terminal (final) → done", () => {
|
||||||
|
assert.equal(resolveIssueStatus(STATUS_MODEL, "final"), "done");
|
||||||
|
});
|
||||||
|
|
||||||
|
test("resolveIssueStatus: analyst_verified (חסר במפה הישנה) → in_progress", () => {
|
||||||
|
assert.equal(
|
||||||
|
resolveIssueStatus(STATUS_MODEL, "analyst_verified"),
|
||||||
|
"in_progress",
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("resolveIssueStatus: research_complete → in_progress", () => {
|
||||||
|
assert.equal(
|
||||||
|
resolveIssueStatus(STATUS_MODEL, "research_complete"),
|
||||||
|
"in_progress",
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("resolveIssueStatus: סטטוס לא-מוכר → null (לא בליעה שקטה)", () => {
|
||||||
|
assert.equal(
|
||||||
|
resolveIssueStatus(STATUS_MODEL, "totally_unknown_status"),
|
||||||
|
null,
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("labelFor: מחזיר את התווית מהפיקסצ'ר, ו-null לסטטוס לא-מוכר", () => {
|
||||||
|
assert.equal(labelFor(STATUS_MODEL, "new"), "חדש");
|
||||||
|
assert.equal(labelFor(STATUS_MODEL, "totally_unknown_status"), null);
|
||||||
|
});
|
||||||
|
|
||||||
|
test("רגרסיה: companyId של המועמד-הנבחר שורד את pickSyncTargetIssue (לא companyId חיצוני קבוע)", () => {
|
||||||
|
const candidates: SyncCandidate[] = [
|
||||||
|
{
|
||||||
|
id: "issue-a",
|
||||||
|
status: "cancelled",
|
||||||
|
parentId: null,
|
||||||
|
companyId: "company-a",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: "issue-b",
|
||||||
|
status: "todo",
|
||||||
|
parentId: null,
|
||||||
|
companyId: "company-b",
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
|
const result = pickSyncTargetIssue(candidates);
|
||||||
|
|
||||||
|
assert.equal(result.target?.id, "issue-b");
|
||||||
|
assert.equal(result.target?.companyId, "company-b");
|
||||||
|
});
|
||||||
112
src/sync-target.ts
Normal file
112
src/sync-target.ts
Normal file
@@ -0,0 +1,112 @@
|
|||||||
|
/**
|
||||||
|
* בחירת יעד-הסנכרון של הג'וב `sync-case-status` (issue #446).
|
||||||
|
*
|
||||||
|
* המקור לבאג: הג'וב הישן כתב על **כל** issue מקושר-לתיק, כולל sub-issues
|
||||||
|
* ו-issues בסטטוסים סגורים/בהמתנה-לאדם. זה החזיר CMPA-140 מ-`done` ל-`in_progress`
|
||||||
|
* (8125-09-24) — כי sub-task שהושלם עדיין נמנה כ"מקושר".
|
||||||
|
*
|
||||||
|
* המודול הזה טהור בכוונה — בלי import מה-SDK ובלי side effects — כדי שאפשר
|
||||||
|
* יהיה לייבא אותו ישירות בטסט בלי להריץ את `runWorker(plugin, import.meta.url)`
|
||||||
|
* שקורה בזמן import של worker.ts. ה-import היחיד למטה הוא type-only (`import
|
||||||
|
* type`), כך שהוא לא נדרס בזמן ריצה ולא שובר את הטוהר הזה.
|
||||||
|
*/
|
||||||
|
|
||||||
|
import type { StatusModelEntry } from "./legal-api.ts";
|
||||||
|
|
||||||
|
/** "סגור" — ההגדרה היחידה. מראה של web/paperclip_client.py:561 ב-legal-ai. */
|
||||||
|
export const CLOSED_ISSUE_STATUSES: ReadonlySet<string> = new Set([
|
||||||
|
"done",
|
||||||
|
"cancelled",
|
||||||
|
]);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* מצב בבעלות Paperclip / בהמתנה-לאדם — **לא** הגדרה שנייה של "סגור".
|
||||||
|
* `in_review` הוא מצב-ההמתנה-ליו"ר המכוון: CEO שמשאיר issue ב-`in_progress`
|
||||||
|
* מקבל auto-block מ-Paperclip תוך דקה, ולכן הוא מעביר ל-`in_review`
|
||||||
|
* (legal-ai/docs/paperclip-quirks.md §3). כתיבת-סטטוס אוטומטית עליו גונבת את
|
||||||
|
* ה-issue מתור-הביקורת של היו"ר ומזמינה קרב-סטטוסים.
|
||||||
|
* `blocked` — כתיבה עליו מסתירה חוסם קיים.
|
||||||
|
*/
|
||||||
|
export const NON_WRITABLE_STATUSES: ReadonlySet<string> = new Set([
|
||||||
|
"in_review",
|
||||||
|
"blocked",
|
||||||
|
]);
|
||||||
|
|
||||||
|
export interface SyncCandidate {
|
||||||
|
id: string;
|
||||||
|
status: string;
|
||||||
|
parentId: string | null;
|
||||||
|
companyId: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
export type SyncTargetReason =
|
||||||
|
| "ok"
|
||||||
|
| "no_linked_issues"
|
||||||
|
| "no_writable_root"
|
||||||
|
| "ambiguous_writable_roots";
|
||||||
|
|
||||||
|
export interface SyncTargetResult {
|
||||||
|
target: SyncCandidate | null;
|
||||||
|
reason: SyncTargetReason;
|
||||||
|
/** כמה שורשים ברי-כתיבה נמצאו — לצורך לוג. */
|
||||||
|
writableRoots: number;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function isWritableStatus(status: string): boolean {
|
||||||
|
return (
|
||||||
|
!CLOSED_ISSUE_STATUSES.has(status) && !NON_WRITABLE_STATUSES.has(status)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
export function pickSyncTargetIssue(
|
||||||
|
candidates: readonly SyncCandidate[],
|
||||||
|
): SyncTargetResult {
|
||||||
|
if (candidates.length === 0) {
|
||||||
|
return { target: null, reason: "no_linked_issues", writableRoots: 0 };
|
||||||
|
}
|
||||||
|
|
||||||
|
const roots = candidates.filter(
|
||||||
|
(c) => c.parentId === null && isWritableStatus(c.status),
|
||||||
|
);
|
||||||
|
|
||||||
|
if (roots.length === 1) {
|
||||||
|
return { target: roots[0], reason: "ok", writableRoots: 1 };
|
||||||
|
}
|
||||||
|
if (roots.length === 0) {
|
||||||
|
return { target: null, reason: "no_writable_root", writableRoots: 0 };
|
||||||
|
}
|
||||||
|
return {
|
||||||
|
target: null,
|
||||||
|
reason: "ambiguous_writable_roots",
|
||||||
|
writableRoots: roots.length,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Derive the coarse Paperclip issue status for a legal-ai case status, using
|
||||||
|
* only generic fields already returned by legal-ai's `/api/status-model`
|
||||||
|
* (the canonical `case_status_model.py` registry) — never a hardcoded list of
|
||||||
|
* case-status keys. This is what makes a new case status "just work" without
|
||||||
|
* editing this plugin: `terminal: true` → done; the very first status in the
|
||||||
|
* ordered list → todo (nothing has started yet); everything else → in_progress.
|
||||||
|
* Returns `null` when `caseStatus` is not present in `statuses` at all (a
|
||||||
|
* genuinely unknown status — the caller must log this loudly, never swallow
|
||||||
|
* it silently; see legal-ai issue #604).
|
||||||
|
*/
|
||||||
|
export function resolveIssueStatus(
|
||||||
|
statuses: readonly StatusModelEntry[],
|
||||||
|
caseStatus: string,
|
||||||
|
): "todo" | "in_progress" | "done" | null {
|
||||||
|
const index = statuses.findIndex((s) => s.key === caseStatus);
|
||||||
|
if (index === -1) return null;
|
||||||
|
if (statuses[index].terminal) return "done";
|
||||||
|
return index === 0 ? "todo" : "in_progress";
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Hebrew label for a case status from the canonical model, or null if unknown. */
|
||||||
|
export function labelFor(
|
||||||
|
statuses: readonly StatusModelEntry[],
|
||||||
|
caseStatus: string,
|
||||||
|
): string | null {
|
||||||
|
return statuses.find((s) => s.key === caseStatus)?.label ?? null;
|
||||||
|
}
|
||||||
110
src/worker.ts
110
src/worker.ts
@@ -4,6 +4,12 @@ import type {
|
|||||||
} from "@paperclipai/plugin-sdk";
|
} from "@paperclipai/plugin-sdk";
|
||||||
import { definePlugin, runWorker } from "@paperclipai/plugin-sdk";
|
import { definePlugin, runWorker } from "@paperclipai/plugin-sdk";
|
||||||
import { LegalApi } from "./legal-api.js";
|
import { LegalApi } from "./legal-api.js";
|
||||||
|
import {
|
||||||
|
labelFor,
|
||||||
|
pickSyncTargetIssue,
|
||||||
|
resolveIssueStatus,
|
||||||
|
type SyncCandidate,
|
||||||
|
} from "./sync-target.js";
|
||||||
|
|
||||||
// Hoisted so onWebhook can access the context after setup() completes.
|
// Hoisted so onWebhook can access the context after setup() completes.
|
||||||
let pluginCtx: PluginContext | null = null;
|
let pluginCtx: PluginContext | null = null;
|
||||||
@@ -841,76 +847,86 @@ const plugin = definePlugin({
|
|||||||
const cases = await api.listCases();
|
const cases = await api.listCases();
|
||||||
const companies = await ctx.companies.list();
|
const companies = await ctx.companies.list();
|
||||||
if (!companies.length) return;
|
if (!companies.length) return;
|
||||||
|
const statusModel = await api.getStatusModel();
|
||||||
|
|
||||||
const companyId = companies[0].id;
|
// מעבר אחד על ה-issues של כל חברה: state.get אחד ל-issue (לא לכל
|
||||||
const issues = await ctx.issues.list({ companyId });
|
// צירוף תיק×issue), וקיבוץ למפה case_number → מועמדים. הקיבוץ נדרש
|
||||||
|
// כדי לבחור "השורש היחיד". סורק את **כל** החברות (לא רק הראשונה) —
|
||||||
for (const legalCase of cases) {
|
// legal-ai issue #604.
|
||||||
|
const byCase = new Map<string, SyncCandidate[]>();
|
||||||
|
for (const company of companies) {
|
||||||
|
const issues = await ctx.issues.list({ companyId: company.id });
|
||||||
for (const issue of issues) {
|
for (const issue of issues) {
|
||||||
const linkedCase = await ctx.state.get({
|
const linkedCase = await ctx.state.get({
|
||||||
scopeKind: "issue",
|
scopeKind: "issue",
|
||||||
scopeId: issue.id,
|
scopeId: issue.id,
|
||||||
stateKey: "legal-case-number",
|
stateKey: "legal-case-number",
|
||||||
});
|
});
|
||||||
|
if (typeof linkedCase !== "string" || !linkedCase) continue;
|
||||||
|
const list = byCase.get(linkedCase) ?? [];
|
||||||
|
list.push({
|
||||||
|
id: issue.id,
|
||||||
|
status: issue.status,
|
||||||
|
parentId: issue.parentId,
|
||||||
|
companyId: company.id,
|
||||||
|
});
|
||||||
|
byCase.set(linkedCase, list);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
if (linkedCase === legalCase.case_number) {
|
for (const legalCase of cases) {
|
||||||
// Map 13 legal-ai statuses to Paperclip issue status
|
const targetStatus = resolveIssueStatus(
|
||||||
const statusMap: Record<string, "todo" | "in_progress" | "done"> =
|
statusModel.statuses,
|
||||||
|
legalCase.status,
|
||||||
|
);
|
||||||
|
if (targetStatus === null) {
|
||||||
|
ctx.logger.warn(
|
||||||
|
"sync-case-status: unknown case status — not in /api/status-model (case_status_model.py)",
|
||||||
{
|
{
|
||||||
new: "todo",
|
caseNumber: legalCase.case_number,
|
||||||
uploading: "todo",
|
status: legalCase.status,
|
||||||
processing: "in_progress",
|
},
|
||||||
documents_ready: "in_progress",
|
);
|
||||||
outcome_set: "in_progress",
|
continue;
|
||||||
brainstorming: "in_progress",
|
}
|
||||||
direction_approved: "in_progress",
|
|
||||||
drafting: "in_progress",
|
|
||||||
qa_review: "in_progress",
|
|
||||||
drafted: "in_progress",
|
|
||||||
exported: "in_progress",
|
|
||||||
reviewed: "in_progress",
|
|
||||||
final: "done",
|
|
||||||
};
|
|
||||||
|
|
||||||
const statusLabels: Record<string, string> = {
|
const candidates = byCase.get(legalCase.case_number) ?? [];
|
||||||
new: "תיק חדש",
|
const { target, reason, writableRoots } =
|
||||||
uploading: "העלאת מסמכים",
|
pickSyncTargetIssue(candidates);
|
||||||
processing: "עיבוד מסמכים",
|
|
||||||
documents_ready: "מסמכים מוכנים — הזן תוצאה",
|
|
||||||
outcome_set: "תוצאה הוזנה — נדרש סיעור מוחות",
|
|
||||||
brainstorming: "גיבוש כיוון בתהליך",
|
|
||||||
direction_approved: "כיוון אושר — מוכן לכתיבה",
|
|
||||||
drafting: "כתיבת החלטה בתהליך",
|
|
||||||
qa_review: "בדיקת איכות",
|
|
||||||
drafted: "טיוטה מוכנה — בדוק ושלח לדפנה",
|
|
||||||
exported: "DOCX נוצר — ממתין לדפנה",
|
|
||||||
reviewed: "דפנה הגיהה — העלה גרסה סופית",
|
|
||||||
final: "גרסה סופית — לולאת למידה",
|
|
||||||
};
|
|
||||||
|
|
||||||
const targetStatus = statusMap[legalCase.status];
|
if (!target) {
|
||||||
const label = statusLabels[legalCase.status] || legalCase.status;
|
// אין בליעה שקטה: מדווח למה לא נכתב כלום.
|
||||||
|
ctx.logger.info("sync-case-status: no sync target", {
|
||||||
|
caseNumber: legalCase.case_number,
|
||||||
|
reason,
|
||||||
|
linked: candidates.length,
|
||||||
|
writableRoots,
|
||||||
|
});
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (target.status === targetStatus) continue;
|
||||||
|
|
||||||
|
const label =
|
||||||
|
labelFor(statusModel.statuses, legalCase.status) ??
|
||||||
|
legalCase.status;
|
||||||
|
|
||||||
if (targetStatus && issue.status !== targetStatus) {
|
|
||||||
await ctx.issues.update(
|
await ctx.issues.update(
|
||||||
issue.id,
|
target.id,
|
||||||
{ status: targetStatus },
|
{ status: targetStatus },
|
||||||
companyId,
|
target.companyId,
|
||||||
);
|
);
|
||||||
await ctx.issues.createComment(
|
await ctx.issues.createComment(
|
||||||
issue.id,
|
target.id,
|
||||||
`📋 ${label}`,
|
`📋 ${label}`,
|
||||||
companyId,
|
target.companyId,
|
||||||
);
|
);
|
||||||
ctx.logger.info("Synced issue status", {
|
ctx.logger.info("Synced issue status", {
|
||||||
issueId: issue.id,
|
issueId: target.id,
|
||||||
caseNumber: legalCase.case_number,
|
caseNumber: legalCase.case_number,
|
||||||
newStatus: targetStatus,
|
newStatus: targetStatus,
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
ctx.logger.info("Case status sync completed", {
|
ctx.logger.info("Case status sync completed", {
|
||||||
casesChecked: cases.length,
|
casesChecked: cases.length,
|
||||||
|
|||||||
@@ -12,5 +12,6 @@
|
|||||||
"jsx": "react-jsx",
|
"jsx": "react-jsx",
|
||||||
"lib": ["ES2022", "DOM", "DOM.Iterable"]
|
"lib": ["ES2022", "DOM", "DOM.Iterable"]
|
||||||
},
|
},
|
||||||
"include": ["src"]
|
"include": ["src"],
|
||||||
|
"exclude": ["src/**/*.test.ts"]
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user