优化

tests/loading-completion-state.test.mjs

@@ -35,13 +35,13 @@ test('completion failure exposes a readable stage and retry entry', () => {
 	})
 })
 
-test('retry reuses the original operation id with the latest server version and command facts', () => {
+test('retry reuses the original operation id and request-hashed task version', () => {
 	const attempt = refreshCompletionAttempt(
 		{ operationId: 'complete-task-8-v3', expectedTaskVersion: 3, commandId: 'cmd-old' },
 		{ taskVersion: 7, commandId: 'cmd-latest', commandStatus: 'COMPLETION_FAILED' }
 	)
 	assert.equal(attempt.operationId, 'complete-task-8-v3')
-	assert.equal(attempt.expectedTaskVersion, 7)
+	assert.equal(attempt.expectedTaskVersion, 3)
 	assert.equal(attempt.commandId, 'cmd-latest')
 	assert.throws(() => refreshCompletionAttempt({}, { taskVersion: 7 }), /缺少原完成操作标识/)
 	assert.throws(() => refreshCompletionAttempt({ operationId: 'stable-id' }), /缺少服务端最新任务版本/)
@@ -59,4 +59,3 @@ test('completed facts win over stale processing facts', () => {
 	assert.equal(phase, COMPLETION_PHASE.COMPLETED)
 	assert.equal(completionPresentation(phase).label, '装柜已完成')
 })
-

tests/page-contract.test.mjs

@@ -59,13 +59,14 @@ test('picking task list uses remote loading orders and never falls back to demo
 	assert.match(tasks, /进入并开始拣货/)
 	assert.match(tasks, /AWAITING_REVIEW[\s\S]*?待上传证据\/确认装柜/)
 	assert.match(tasks, /task\.workflowStatus === 'AWAITING_REVIEW' \? '继续装柜取证'/)
-	assert.match(tasks, /\['UNASSIGNED', 'IN_PROGRESS', 'AWAITING_REVIEW'\]\.includes\(task\.workflowStatus\)/)
-	assert.match(tasks, /task\.workflowStatus === 'AWAITING_REVIEW' \|\| \(task\.assignmentVersion !== null/)
+	assert.match(tasks, /\['UNASSIGNED', 'IN_PROGRESS', 'AWAITING_REVIEW', 'COMPLETION_FAILED'\]\.includes\(task\.workflowStatus\)/)
+	assert.match(tasks, /\['UNASSIGNED', 'IN_PROGRESS', 'AWAITING_REVIEW', 'PROCESSING', 'COMPLETION_FAILED'\]\.includes\(task\.workflowStatus\)/)
+	assert.match(tasks, /\['AWAITING_REVIEW', 'COMPLETION_FAILED'\]\.includes\(task\.workflowStatus\) \|\| \(task\.assignmentVersion !== null/)
 	assert.match(tasks, /暂无待处理的拣货任务/)
 	assert.match(tasks, /class="task-label">集装箱号<\/text><text class="task-code">\{\{ task\.currentContainerNo \}\}/)
 	assert.match(tasks, /柜次号 \{\{ task\.cabinetNo \}\}/)
 	assert.match(tasks, /button v-for="task in tasks"[\s\S]*?class="task-card"[\s\S]*?@tap="enterTask\(task\)"/)
-	assert.match(tasks, /records\.filter\(\(task\) => this\.isTaskReady\(task\)\)/)
+	assert.match(tasks, /records\.filter\(\(task\) => this\.isTaskVisible\(task\)\)/)
 	assert.match(tasks, /isTaskReady\(task\)[\s\S]*?task\.recordStatus === 'ACTIVE'[\s\S]*?!task\.mergedIntoContainerBookingId/)
 	assert.match(tasks, /records\.filter\(\(task\) => this\.isMergedTask\(task\)\)\.forEach\(\(task\) => this\.clearMergedTaskDraft\(task\)\)/)
 	assert.match(tasks, /clearMergedTaskDraft\(task\)[\s\S]*?clearPickingDraft\(\{ operatorId, taskId: task\.taskId, loadingPlanId: task\.loadingPlanId, containerBookingId: task\.containerBookingId/)

tests/picking-store.test.mjs

@@ -29,3 +29,11 @@ test('mutable real container number is not part of draft identity or restore val
 	assert.doesNotMatch(taskIdentitySource, /containerNo|currentContainerNo|boxNo/)
 	assert.doesNotMatch(restoreGuard, /containerNo|currentContainerNo|boxNo/)
 })
+
+test('completion retry persists the exact accepted operation id and request version', () => {
+	assert.match(source, /pda-loading-completion/)
+	assert.match(source, /saveLoadingCompletionAttempt/)
+	assert.match(source, /restoreLoadingCompletionAttempt/)
+	assert.match(source, /operationId: String\(attempt\.operationId\)/)
+	assert.match(source, /expectedTaskVersion: Number\(attempt\.expectedTaskVersion\)/)
+})