From cb36ec88406af71d799f53f803acf1810dfbbb50 Mon Sep 17 00:00:00 2001
From: Andrey Epifancev <andrey.epifancev@ssi-solutions.ru>
Date: Sun, 18 Oct 2026 22:52:29 +0400
Subject: [PATCH 1/2] =?UTF-8?q?=D0=9D=D0=B0=D1=80=D1=83=D1=88=D0=B5=D0=BD?=
 =?UTF-8?q?=D0=B8=D1=8F=20=D0=BF=D1=80=D0=BE=D0=B2=D0=B5=D1=80=D0=BE=D0=BA?=
 =?UTF-8?q?:=20expected/actual=20=D0=B8=20=D0=B8=D0=BD=D0=B4=D0=B5=D0=BA?=
 =?UTF-8?q?=D1=81=20=D0=BA=D0=BE=D0=BB=D0=BE=D0=BD=D0=BA=D0=B8=20=D0=BC?=
 =?UTF-8?q?=D0=BE=D0=B3=D1=83=D1=82=20=D0=B1=D1=8B=D1=82=D1=8C=20null=20(#?=
 =?UTF-8?q?45)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Типы FailedCheckOut по API ядра (P3-30, P3-60); подпись «нет строки цели», «Без колонки» для -1/null; общий ключ списка.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/types/api.ts                              |  8 ++--
 src/widgets/DocumentReview/ChecksPanel.tsx    |  9 ++--
 src/widgets/DocumentReview/OsvDocument.tsx    |  5 ++-
 .../DocumentReview/StatementReview.tsx        |  5 ++-
 src/widgets/DocumentReview/checkText.test.ts  | 41 +++++++++++++++++++
 src/widgets/DocumentReview/checkText.ts       | 25 +++++++++++
 6 files changed, 80 insertions(+), 13 deletions(-)
 create mode 100644 src/widgets/DocumentReview/checkText.test.ts
 create mode 100644 src/widgets/DocumentReview/checkText.ts

diff --git a/src/types/api.ts b/src/types/api.ts
index e068d0c..05b62ea 100644
--- a/src/types/api.ts
+++ b/src/types/api.ts
@@ -228,10 +228,12 @@ export interface UnresolvedLineOut {
 	line_code_text: string | null;
 }
 export interface FailedCheckOut {
-	column_index: number;
+	/** -1 в GET …/statement и null в теле 409 — нарушение без колонки (P3-60) */
+	column_index: number | null;
 	relation: string;
-	expected: string;
-	actual: string;
+	/** null — у связи нет строки цели (P3-30) */
+	expected: string | null;
+	actual: string | null;
 	codes: string[];
 	line_ids: string[];
 }
diff --git a/src/widgets/DocumentReview/ChecksPanel.tsx b/src/widgets/DocumentReview/ChecksPanel.tsx
index 4955519..d626ec9 100644
--- a/src/widgets/DocumentReview/ChecksPanel.tsx
+++ b/src/widgets/DocumentReview/ChecksPanel.tsx
@@ -1,10 +1,7 @@
 import { Alert, Button, Input, Modal, Space, Tooltip, Typography } from 'antd';
 import { useState } from 'react';
 import type { ChecksOut, FailedCheckOut } from '../../types/api';
-import { formatAmount } from '../../utils/money';
-
-export const failureText = (f: FailedCheckOut) =>
-	`Колонка ${f.column_index + 1}: ${f.relation} — ожидалось ${formatAmount(f.expected)}, получено ${formatAmount(f.actual)}`;
+import { expectedActualText, failureKey, failureText } from './checkText';
 
 type Props = {
 	checks: ChecksOut;
@@ -20,7 +17,7 @@ type Props = {
 const failureList = (list: FailedCheckOut[], onShow?: (key: string) => void) => (
 	<div style={{ maxHeight: 120, overflowY: 'auto' }}>
 		{list.map((f) => (
-			<div key={`${f.column_index}-${f.relation}`}>
+			<div key={failureKey(f)}>
 				{failureText(f)}
 				{onShow && (f.line_ids ?? []).length > 0 && (
 					<Button type='link' size='small' onClick={() => onShow(f.line_ids[0])}>
@@ -72,7 +69,7 @@ export function ChecksPanel({ checks, editable, dirty, onAccept, onShow }: Props
 					type='warning'
 					showIcon
 					title='Межформенная проверка ББ↔ОФР'
-					description={`${checks.cross_form.relation}: ожидалось ${formatAmount(checks.cross_form.expected)}, получено ${formatAmount(checks.cross_form.actual)}`}
+					description={`${checks.cross_form.relation}: ${expectedActualText(checks.cross_form)}`}
 				/>
 			)}
 			{checks.accepted && (
diff --git a/src/widgets/DocumentReview/OsvDocument.tsx b/src/widgets/DocumentReview/OsvDocument.tsx
index e957432..e162784 100644
--- a/src/widgets/DocumentReview/OsvDocument.tsx
+++ b/src/widgets/DocumentReview/OsvDocument.tsx
@@ -8,6 +8,7 @@ import type { DocumentOut, EntryOut, FailedCheckOut } from '../../types/api';
 import { formatAmount } from '../../utils/money';
 import { IssuesPanel } from './IssuesPanel';
 import { confirmButtonStyle } from './ReviewCard';
+import { expectedActualText, failureKey } from './checkText';
 import { confirmErrorInfo } from './confirmError';
 import { AMOUNT_KEYS, type AmountKey, type OsvRow, flattenEntries } from './osvRows';
 import { osvBodyHeight } from './osvTableSize';
@@ -183,8 +184,8 @@ export function OsvDocument({ document, onChanged }: { document: DocumentOut; on
 						problem.failures.length > 0 ? (
 							<ul style={{ margin: 0, paddingLeft: 18 }}>
 								{problem.failures.map((f) => (
-									<li key={`${f.column_index}-${f.relation}`}>
-										{f.relation}: ожидалось {formatAmount(f.expected)}, получено {formatAmount(f.actual)}
+									<li key={failureKey(f)}>
+										{f.relation}: {expectedActualText(f)}
 									</li>
 								))}
 							</ul>
diff --git a/src/widgets/DocumentReview/StatementReview.tsx b/src/widgets/DocumentReview/StatementReview.tsx
index bc61dad..5ac0ffd 100644
--- a/src/widgets/DocumentReview/StatementReview.tsx
+++ b/src/widgets/DocumentReview/StatementReview.tsx
@@ -6,12 +6,13 @@ import { ApiError } from '../../api/client';
 import { acceptChecks, confirmDocument, getStatement, patchStatement } from '../../services/api/documents';
 import type { DocumentOut, FailedCheckOut, StatementOut, UnresolvedLineOut } from '../../types/api';
 import { parseAmountInput } from '../../utils/money';
-import { ChecksPanel, failureText } from './ChecksPanel';
+import { ChecksPanel } from './ChecksPanel';
 import { IssuesPanel } from './IssuesPanel';
 import { ReviewCard, confirmButtonStyle } from './ReviewCard';
 import { SourcePreview } from './SourcePreview';
 import { StatementHeaderBlock } from './StatementHeaderBlock';
 import { StatementTable } from './StatementTable';
+import { failureKey, failureText } from './checkText';
 import { codeOptions as buildCodeOptions, lineErrorTarget } from './codeSuggestions';
 import { unresolvedHint, unresolvedLines } from './confirmBlockers';
 import { confirmErrorInfo } from './confirmError';
@@ -205,7 +206,7 @@ export function StatementReview({ document, onChanged }: { document: DocumentOut
 
 	let confirmDescription: ReactNode;
 	if (confirmFailures.length > 0) {
-		confirmDescription = confirmFailures.map((f) => <div key={`${f.column_index}-${f.relation}`}>{failureText(f)}</div>);
+		confirmDescription = confirmFailures.map((f) => <div key={failureKey(f)}>{failureText(f)}</div>);
 	} else if (confirmLines.length > 0) {
 		confirmDescription = (
 			<Button type='link' style={{ paddingInline: 0 }} onClick={() => focusRow(confirmLines[0].line_id)}>
diff --git a/src/widgets/DocumentReview/checkText.test.ts b/src/widgets/DocumentReview/checkText.test.ts
new file mode 100644
index 0000000..43e6e2d
--- /dev/null
+++ b/src/widgets/DocumentReview/checkText.test.ts
@@ -0,0 +1,41 @@
+import { describe, expect, it } from 'vitest';
+import type { FailedCheckOut } from '../../types/api';
+import { failureKey, failureText, hasColumn } from './checkText';
+
+const base: FailedCheckOut = { column_index: 0, relation: '1600 = 1700', expected: '10.00', actual: '9.00', codes: [], line_ids: [] };
+
+describe('hasColumn', () => {
+	it('0 и больше — колонка есть; -1 и null — нарушение без колонки', () => {
+		expect(hasColumn(0)).toBe(true);
+		expect(hasColumn(3)).toBe(true);
+		expect(hasColumn(-1)).toBe(false);
+		expect(hasColumn(null)).toBe(false);
+		expect(hasColumn(undefined)).toBe(false);
+	});
+});
+
+describe('failureText', () => {
+	it('обычное нарушение с номером колонки', () => {
+		expect(failureText(base)).toBe('Колонка 1: 1600 = 1700 — ожидалось 10,00, получено 9,00');
+	});
+	it('expected = null: «нет строки цели», без NaN', () => {
+		const text = failureText({ ...base, expected: null });
+		expect(text).toContain('ожидалось нет строки цели');
+		expect(text).not.toContain('NaN');
+	});
+	it('actual = null показывается прочерком', () => {
+		expect(failureText({ ...base, actual: null })).toContain('получено —');
+	});
+	it('индекс -1 и null: без номера колонки', () => {
+		expect(failureText({ ...base, column_index: -1 })).toMatch(/^Без колонки: /);
+		expect(failureText({ ...base, column_index: null })).toMatch(/^Без колонки: /);
+	});
+});
+
+describe('failureKey', () => {
+	it('для нарушений без колонки ключ не содержит -1/null', () => {
+		expect(failureKey({ ...base, column_index: null })).toBe('none-1600 = 1700');
+		expect(failureKey({ ...base, column_index: -1 })).toBe('none-1600 = 1700');
+		expect(failureKey(base)).toBe('0-1600 = 1700');
+	});
+});
diff --git a/src/widgets/DocumentReview/checkText.ts b/src/widgets/DocumentReview/checkText.ts
new file mode 100644
index 0000000..fabfac8
--- /dev/null
+++ b/src/widgets/DocumentReview/checkText.ts
@@ -0,0 +1,25 @@
+import type { FailedCheckOut } from '../../types/api';
+import { formatAmount } from '../../utils/money';
+
+/** Подпись пустой цели: expected = null — у связи нет строки-цели (P3-30). */
+const NO_TARGET = 'нет строки цели';
+
+/**
+ * Есть ли у нарушения колонка. Ядро кодирует «колонки нет» двумя способами:
+ * -1 в ответе GET …/statement и null в теле 409 CHECKS_NOT_PASSED (P3-60).
+ */
+export const hasColumn = (index: number | null | undefined): index is number => typeof index === 'number' && index >= 0;
+
+export const failureKey = (f: Pick<FailedCheckOut, 'column_index' | 'relation'>): string =>
+	`${hasColumn(f.column_index) ? f.column_index : 'none'}-${f.relation}`;
+
+const expectedText = (v: string | null): string => (v === null ? NO_TARGET : formatAmount(v));
+
+/** «ожидалось …, получено …» для любых expected/actual, в том числе null. */
+export const expectedActualText = (f: Pick<FailedCheckOut, 'expected' | 'actual'>): string =>
+	`ожидалось ${expectedText(f.expected)}, получено ${formatAmount(f.actual)}`;
+
+export const failureText = (f: FailedCheckOut): string => {
+	const where = hasColumn(f.column_index) ? `Колонка ${f.column_index + 1}` : 'Без колонки';
+	return `${where}: ${f.relation} — ${expectedActualText(f)}`;
+};
-- 
2.47.2


From 4fb477d5c2e2b5b199d663e4afc6ee89c5465dd3 Mon Sep 17 00:00:00 2001
From: Andrey Epifancev <andrey.epifancev@ssi-solutions.ru>
Date: Sun, 18 Oct 2026 22:55:42 +0400
Subject: [PATCH 2/2] =?UTF-8?q?=D0=9F=D0=BE=D0=B4=D0=BF=D0=B8=D1=81=D1=8C?=
 =?UTF-8?q?=20=D0=BF=D1=83=D1=81=D1=82=D0=BE=D0=B9=20=D1=86=D0=B5=D0=BB?=
 =?UTF-8?q?=D0=B8=20=C2=AB=D1=86=D0=B5=D0=BB=D1=8C=20=D0=BD=D0=B5=20=D0=B7?=
 =?UTF-8?q?=D0=B0=D0=BF=D0=BE=D0=BB=D0=BD=D0=B5=D0=BD=D0=B0=C2=BB,=20?=
 =?UTF-8?q?=D1=82=D0=B5=D1=81=D1=82=D1=8B=20null/null=20(=D1=80=D0=B5?=
 =?UTF-8?q?=D0=B2=D1=8C=D1=8E=20I-1,=20M-1,=20M-2,=20M-4)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/types/api.ts                             |  3 ++-
 src/widgets/DocumentReview/checkText.test.ts |  7 +++++--
 src/widgets/DocumentReview/checkText.ts      | 10 +++++-----
 3 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/types/api.ts b/src/types/api.ts
index 05b62ea..83d2d5d 100644
--- a/src/types/api.ts
+++ b/src/types/api.ts
@@ -231,8 +231,9 @@ export interface FailedCheckOut {
 	/** -1 в GET …/statement и null в теле 409 — нарушение без колонки (P3-60) */
 	column_index: number | null;
 	relation: string;
-	/** null — у связи нет строки цели (P3-30) */
+	/** null — цель соотношения не заполнена при непустых членах (P3-30) */
 	expected: string | null;
+	/** допускает null по OpenAPI ядра (P3-30) */
 	actual: string | null;
 	codes: string[];
 	line_ids: string[];
diff --git a/src/widgets/DocumentReview/checkText.test.ts b/src/widgets/DocumentReview/checkText.test.ts
index 43e6e2d..0a67bf1 100644
--- a/src/widgets/DocumentReview/checkText.test.ts
+++ b/src/widgets/DocumentReview/checkText.test.ts
@@ -18,11 +18,14 @@ describe('failureText', () => {
 	it('обычное нарушение с номером колонки', () => {
 		expect(failureText(base)).toBe('Колонка 1: 1600 = 1700 — ожидалось 10,00, получено 9,00');
 	});
-	it('expected = null: «нет строки цели», без NaN', () => {
+	it('expected = null: «цель не заполнена»', () => {
 		const text = failureText({ ...base, expected: null });
-		expect(text).toContain('ожидалось нет строки цели');
+		expect(text).toBe('Колонка 1: 1600 = 1700 — цель не заполнена, получено 9,00');
 		expect(text).not.toContain('NaN');
 	});
+	it('expected и actual оба null', () => {
+		expect(failureText({ ...base, expected: null, actual: null })).toContain('цель не заполнена, получено —');
+	});
 	it('actual = null показывается прочерком', () => {
 		expect(failureText({ ...base, actual: null })).toContain('получено —');
 	});
diff --git a/src/widgets/DocumentReview/checkText.ts b/src/widgets/DocumentReview/checkText.ts
index fabfac8..20457f1 100644
--- a/src/widgets/DocumentReview/checkText.ts
+++ b/src/widgets/DocumentReview/checkText.ts
@@ -1,8 +1,8 @@
 import type { FailedCheckOut } from '../../types/api';
 import { formatAmount } from '../../utils/money';
 
-/** Подпись пустой цели: expected = null — у связи нет строки-цели (P3-30). */
-const NO_TARGET = 'нет строки цели';
+/** Подпись пустой цели: expected = null — цель соотношения не заполнена при непустых членах (P3-30). */
+const NO_TARGET = 'цель не заполнена';
 
 /**
  * Есть ли у нарушения колонка. Ядро кодирует «колонки нет» двумя способами:
@@ -13,11 +13,11 @@ export const hasColumn = (index: number | null | undefined): index is number =>
 export const failureKey = (f: Pick<FailedCheckOut, 'column_index' | 'relation'>): string =>
 	`${hasColumn(f.column_index) ? f.column_index : 'none'}-${f.relation}`;
 
-const expectedText = (v: string | null): string => (v === null ? NO_TARGET : formatAmount(v));
-
 /** «ожидалось …, получено …» для любых expected/actual, в том числе null. */
 export const expectedActualText = (f: Pick<FailedCheckOut, 'expected' | 'actual'>): string =>
-	`ожидалось ${expectedText(f.expected)}, получено ${formatAmount(f.actual)}`;
+	f.expected === null
+		? `${NO_TARGET}, получено ${formatAmount(f.actual)}`
+		: `ожидалось ${formatAmount(f.expected)}, получено ${formatAmount(f.actual)}`;
 
 export const failureText = (f: FailedCheckOut): string => {
 	const where = hasColumn(f.column_index) ? `Колонка ${f.column_index + 1}` : 'Без колонки';
-- 
2.47.2