feat: complete in-person claims without final sign

src/claim-request-management/claim-request-management.v2.controller.ts

@@ -322,7 +322,8 @@ export class ClaimRequestManagementV2Controller {
   }
 
   /**
-   * V2: Owner signature — priced-line gate (mixed factors) or final accept/reject.
+   * V2–V5: owner signature used only as the priced-line gate for mixed-factor claims.
+   * The final accept/reject phase is retained for legacy rows only.
    */
   @Put("request/:claimRequestId/owner-insurer-approval/sign")
   @ApiParam({
@@ -332,12 +333,12 @@ export class ClaimRequestManagementV2Controller {
   })
   @ApiConsumes("multipart/form-data")
   @ApiOperation({
-    summary: "Sign priced lines or final claim pricing (owner)",
+    summary: "Sign priced lines before factor uploads (owner; final phase is legacy only)",
     description:
       "Multipart: `sign`, `agree`, `branchId`. Requires `ClaimCaseStatus` **`INSURER_REVIEW_AWAITING_OWNER_SIGN`**, **`INSURER_REVIEW_MIXED_FACTORS_PENDING`**, or legacy **`WAITING_FOR_INSURER_APPROVAL`**, and `workflow.currentStep=INSURER_REVIEW` (not during owner factor upload or `EXPERT_COST_EVALUATION`).\n\n" +
       "**Phase A — Mixed reply, priced lines only:** `claimStatus=NEEDS_REVISION`, no `evaluation.ownerPricedPartsApproval` yet. `agree=true` records that signature and moves to `OWNER_UPLOAD_FACTOR_DOCUMENTS` for factor uploads; `agree=false` rejects the whole case (`REJECTED`).\n\n" +
-      "**Phase B — Final:** `claimStatus=APPROVED`, no `evaluation.ownerInsurerApproval` yet. `agree=true` → `COMPLETED`; `agree=false` → `REJECTED`.\n\n" +
-      "Response may include `phase`: `PRICED_PARTS_FOR_FACTORS` or `FINAL_APPROVAL` for UI state.",
+      "**Phase B — Legacy final phase only:** pre-existing rows with `claimStatus=APPROVED` may still be accepted or rejected through this endpoint. New V2–V5 claims complete after expert work (and V5 FileMaker approval) without a final owner signature.\n\n" +
+      "Response may include `phase`: `PRICED_PARTS_FOR_FACTORS` or legacy `FINAL_APPROVAL` for UI state.",
   })
   @ApiBody({
     description: "Signature file, agreement, and branch",

src/claim-request-management/dto/claim-details-v2.dto.ts

@@ -102,7 +102,7 @@ export class ClaimDetailsV2ResponseDto {
 
   @ApiProperty({
     description:
-      "ClaimCaseStatus; see also `ownerGuidance` for UX. Post-expert: INSURER_REVIEW_AWAITING_OWNER_SIGN | INSURER_REVIEW_MIXED_FACTORS_PENDING | OWNER_REPAIR_FACTOR_UPLOAD_PENDING | EXPERT_VALIDATING_REPAIR_FACTORS; legacy WAITING_FOR_INSURER_APPROVAL may still appear.",
+      "ClaimCaseStatus; see also `ownerGuidance` for UX. New V2–V5 priced-only claims are COMPLETED after expert work; factor claims use INSURER_REVIEW_MIXED_FACTORS_PENDING | OWNER_REPAIR_FACTOR_UPLOAD_PENDING | EXPERT_VALIDATING_REPAIR_FACTORS. V5 then uses WAITING_FOR_FILE_MAKER_APPROVAL. Legacy WAITING_FOR_INSURER_APPROVAL or INSURER_REVIEW_AWAITING_OWNER_SIGN may still appear.",
     example: "OWNER_REPAIR_FACTOR_UPLOAD_PENDING",
   })
   status: string;

src/claim-request-management/dto/my-claims-v2.dto.ts

@@ -12,8 +12,8 @@ export class ClaimListItemV2Dto {
 
   @ApiProperty({
     description:
-      "ClaimCaseStatus. Post-expert owner phase includes: INSURER_REVIEW_AWAITING_OWNER_SIGN (priced lines only → final owner sign); INSURER_REVIEW_MIXED_FACTORS_PENDING (priced + factor lines); OWNER_REPAIR_FACTOR_UPLOAD_PENDING (all lines factor-needed); EXPERT_VALIDATING_REPAIR_FACTORS (all factors uploaded, expert validating). Legacy DB rows may still use WAITING_FOR_INSURER_APPROVAL for those flows.",
-    example: "INSURER_REVIEW_AWAITING_OWNER_SIGN",
+      "ClaimCaseStatus. New V2–V5 priced-only claims become COMPLETED after expert work. Factor claims use INSURER_REVIEW_MIXED_FACTORS_PENDING (priced-line acceptance before factor uploads), OWNER_REPAIR_FACTOR_UPLOAD_PENDING, and EXPERT_VALIDATING_REPAIR_FACTORS. V5 then waits at WAITING_FOR_FILE_MAKER_APPROVAL. Legacy DB rows may still use WAITING_FOR_INSURER_APPROVAL or INSURER_REVIEW_AWAITING_OWNER_SIGN.",
+    example: "COMPLETED",
   })
   status: string;
 

src/claim-request-management/expert-initiated-claim.mirror.controller.ts

@@ -527,7 +527,7 @@ Returns status of each item (uploaded/captured or not).
     );
   }
 
-  // ─── Owner signature on expert pricing ───────────────────────────────────────
+  // ─── Mixed-factor priced-line signature ──────────────────────────────────────
 
   @Put("claim-sign/:claimRequestId")
   @ApiParam({ name: "claimRequestId" })
@@ -553,11 +553,12 @@ Returns status of each item (uploaded/captured or not).
   })
   @ApiOperation({
     summary:
-      "Owner signature on expert pricing (Flow 3 — expert acts on behalf of user)",
+      "Priced-line acceptance before factor uploads (Flow 3 — expert acts on behalf of user)",
     description:
-      "Field expert submits the damaged party's signature during the final approval stage. " +
-      "Delegates to the same service method as the user sign endpoint; the expert's " +
-      "identity is resolved to the claim owner via `resolveClaimEffectiveUserId`.",
+      "For mixed priced/factor claims only, the field expert records the damaged party's " +
+      "acceptance of priced lines before factor uploads. V2–V5 no longer require a final " +
+      "owner signature: they complete after expert work. The expert's identity is resolved " +
+      "to the claim owner via `resolveClaimEffectiveUserId`.",
   })
   @UseInterceptors(
     FileInterceptor("sign", {