Compare commits

..
2 Commits
Author SHA1 Message Date
tremble7681andclaude-flow 100a29f8c7 fix(repairs): ростер механиков по пути repairs/mechanics-roster
prefixUrl=/api/v1/mechanic, эндпоинт под роутером /v1/mechanic/repairs →
путь обязан включать repairs/ (иначе 404, пикер механиков не грузился). Код-ревью #1.

Co-Authored-By: claude-flow <ruv@ruv.net>
2026-06-10 20:07:52 +03:00
tremble7681andclaude-flow f7ccc7ce77 feat(repairs): блок «Механики» — несколько механиков на ремонт, делёж 1/N
MechanicsField (чипы + пикер ростера) в Create/Edit; mechanic_ids в payload;
RepairDetail показывает работавших + долю 1/N. Ростер: GET /v1/mechanic/mechanics-roster.

Co-Authored-By: claude-flow <ruv@ruv.net>
2026-06-10 19:39:36 +03:00
6 changed files with 270 additions and 4 deletions
@@ -78,6 +78,22 @@ export interface CreateRepairPayload {
parts: CreatePartPayload[]; parts: CreatePartPayload[];
photos: CreatePhotoPayload[]; photos: CreatePhotoPayload[];
oil_change: CreateOilChangePayload | null; oil_change: CreateOilChangePayload | null;
// Работавшие механики (делёж 1/N). Если не передан — backend сидит
// одного создателя. Порядок = sort_order.
mechanic_ids?: number[];
}
// ── Ростер механиков (пикер «Механики») ────────────────────────────────────
export interface MechanicRosterItem {
id: number;
full_name: string;
}
export async function getMechanicsRoster(): Promise<MechanicRosterItem[]> {
// prefixUrl = /api/v1/mechanic, эндпоинт под роутером /v1/mechanic/repairs →
// путь обязан включать сегмент repairs/ (как и остальные вызовы файла).
return api.get("repairs/mechanics-roster").json<MechanicRosterItem[]>();
} }
export interface CreateRepairResponse { export interface CreateRepairResponse {
@@ -79,6 +79,12 @@ export interface RepairDetailOil {
sticker_orig_url: string | null; sticker_orig_url: string | null;
} }
export interface RepairDetailMechanic {
id: number;
mechanic_id: number;
name: string;
}
export interface RepairDetailResponse { export interface RepairDetailResponse {
id: number; id: number;
created_at: string; created_at: string;
@@ -93,6 +99,9 @@ export interface RepairDetailResponse {
photos: RepairDetailPhoto[]; photos: RepairDetailPhoto[];
oil_change: RepairDetailOil | null; oil_change: RepairDetailOil | null;
total_sum: number; total_sum: number;
// Работавшие механики (делёж 1/N total_sum). Может отсутствовать на
// старых ответах backend — фронт делает fallback на одного создателя.
mechanics?: RepairDetailMechanic[];
edited_in_crm: boolean; edited_in_crm: boolean;
can_edit: boolean; can_edit: boolean;
edit_until: string | null; edit_until: string | null;
@@ -127,6 +136,8 @@ export interface PatchRepairPayload {
works: PatchWork[]; works: PatchWork[];
parts: PatchPart[]; parts: PatchPart[];
oil_change: PatchOilChange | null; oil_change: PatchOilChange | null;
// Работавшие механики. Опционально — если не передан, набор не меняется.
mechanic_ids?: number[];
} }
export async function patchRepair(id: number, body: PatchRepairPayload): Promise<RepairDetailResponse> { export async function patchRepair(id: number, body: PatchRepairPayload): Promise<RepairDetailResponse> {
@@ -0,0 +1,168 @@
/**
* Блок «Механики» формы ремонта — чипы работавших механиков + кнопка
* «Добавить механика» (пикер из ростера) + удаление чипа.
*
* Делёж оплаты — поровну (1/N): доля каждого механика = total_sum / N.
* Здесь показывается информативно (сумма на чел.), если передан totalSum.
*
* Состояние выбора живёт в родителе (как у WorksPickerModal): сюда
* приходит value + onChange. Ростер компонент тянет сам при первом
* открытии пикера (лениво) и кэширует на время жизни.
*/
import { useState } from "react";
import { Plus, X } from "lucide-react";
import { Button } from "@/components/ui/button";
import { Input } from "@/components/ui/input";
import { Label } from "@/components/ui/label";
import { fmtMoney } from "@/lib/repairsFeed";
import { getMechanicsRoster, type MechanicRosterItem } from "@/api/repairsCreate";
export interface SelectedMechanic {
mechanic_id: number;
name: string;
}
interface Props {
value: SelectedMechanic[];
onChange: (next: SelectedMechanic[]) => void;
// Если задан и механиков >0 — показываем долю (1/N · totalSum).
totalSum?: number;
disabled?: boolean;
}
export default function MechanicsField({ value, onChange, totalSum, disabled }: Props) {
const [pickerOpen, setPickerOpen] = useState(false);
const [roster, setRoster] = useState<MechanicRosterItem[] | null>(null);
const [rosterError, setRosterError] = useState<string | null>(null);
const [q, setQ] = useState("");
const openPicker = async () => {
setPickerOpen(true);
if (roster) return;
try {
const list = await getMechanicsRoster();
setRoster(list);
setRosterError(null);
} catch (e) {
setRosterError((e as Error).message || "Не удалось загрузить список механиков");
}
};
const addMechanic = (m: MechanicRosterItem) => {
// Дедуп: один механик один раз (UNIQUE на бэке).
if (!value.some((v) => v.mechanic_id === m.id)) {
onChange([...value, { mechanic_id: m.id, name: m.full_name }]);
}
setPickerOpen(false);
setQ("");
};
const removeMechanic = (mechanicId: number) =>
onChange(value.filter((v) => v.mechanic_id !== mechanicId));
const share =
totalSum != null && value.length > 0 ? Math.round(totalSum / value.length) : null;
const filtered = (roster ?? []).filter((m) =>
q.trim() ? m.full_name.toLowerCase().includes(q.trim().toLowerCase()) : true,
);
return (
<section>
<div className="flex items-baseline justify-between">
<Label className="text-sm">Механики</Label>
{!disabled && (
<Button variant="outline" size="sm" onClick={openPicker}>
<Plus className="h-3 w-3 mr-1" />
Добавить механика
</Button>
)}
</div>
{value.length === 0 ? (
<div className="text-xs text-muted-foreground mt-2">Минимум один механик</div>
) : (
<div className="mt-2 flex flex-wrap gap-2">
{value.map((m) => (
<span
key={m.mechanic_id}
className="inline-flex items-center gap-1.5 rounded-full border bg-muted/50 pl-3 pr-1.5 py-1 text-sm"
>
<span className="leading-none">{m.name}</span>
{!disabled && (
<button
type="button"
onClick={() => removeMechanic(m.mechanic_id)}
className="rounded-full p-0.5 hover:bg-accent text-muted-foreground"
aria-label={`Убрать ${m.name}`}
>
<X className="h-3.5 w-3.5" />
</button>
)}
</span>
))}
</div>
)}
{share != null && value.length > 1 && (
<div className="mt-2 text-xs text-muted-foreground">
Делёж поровну: {fmtMoney(share)} на каждого ({value.length} мех.)
</div>
)}
{/* Пикер механиков из ростера */}
{pickerOpen && (
<div className="fixed inset-0 z-50 bg-background/95 flex flex-col">
<header className="border-b p-3 flex items-center gap-2">
<button
type="button"
onClick={() => {
setPickerOpen(false);
setQ("");
}}
className="rounded-full px-3 py-2 text-sm hover:bg-accent"
>
Закрыть
</button>
<Input
autoFocus
placeholder="Поиск механика…"
value={q}
onChange={(e) => setQ(e.target.value)}
/>
</header>
<div className="flex-1 overflow-auto">
{rosterError && (
<div className="text-sm text-destructive bg-destructive/10 m-3 rounded-md p-3">
{rosterError}
</div>
)}
{roster === null && !rosterError && (
<div className="text-center text-muted-foreground p-8 text-sm">Загрузка</div>
)}
{filtered.map((m) => {
const already = value.some((v) => v.mechanic_id === m.id);
return (
<button
type="button"
key={m.id}
disabled={already}
onClick={() => addMechanic(m)}
className="w-full text-left p-3 border-b hover:bg-accent/40 flex items-center justify-between gap-3 disabled:opacity-50"
>
<span className="text-sm">{m.full_name}</span>
{already && <span className="text-xs text-muted-foreground">уже добавлен</span>}
</button>
);
})}
{roster !== null && !rosterError && filtered.length === 0 && (
<div className="text-center text-muted-foreground p-8 text-sm">
Ничего не найдено
</div>
)}
</div>
</div>
)}
</section>
);
}
@@ -23,6 +23,8 @@ import { Input } from "@/components/ui/input";
import { Label } from "@/components/ui/label"; import { Label } from "@/components/ui/label";
import { repairCanSubmit, repairSubmitWhyDisabled } from "@/lib/repairForm"; import { repairCanSubmit, repairSubmitWhyDisabled } from "@/lib/repairForm";
import WorksPickerModal from "@/components/repairs/WorksPickerModal"; import WorksPickerModal from "@/components/repairs/WorksPickerModal";
import MechanicsField, { type SelectedMechanic } from "@/components/repairs/MechanicsField";
import { getMe } from "@/api/me";
import { import {
createRepair, createRepair,
getCarContext, getCarContext,
@@ -111,6 +113,9 @@ export default function CreateRepairPage() {
const [photos, setPhotos] = useState<PhotoState[]>([]); const [photos, setPhotos] = useState<PhotoState[]>([]);
const [works, setWorks] = useState<WorkRow[]>([]); const [works, setWorks] = useState<WorkRow[]>([]);
const [parts, setParts] = useState<PartRow[]>([]); const [parts, setParts] = useState<PartRow[]>([]);
// Работавшие механики (делёж 1/N). По умолчанию — текущий механик
// (создатель), подтягивается из /me. Можно убрать/заменить.
const [mechanics, setMechanics] = useState<SelectedMechanic[]>([]);
const [oilChange, setOilChange] = useState<OilChangeState>({ const [oilChange, setOilChange] = useState<OilChangeState>({
enabled: false, enabled: false,
next_in_km: 10000, next_in_km: 10000,
@@ -153,6 +158,16 @@ export default function CreateRepairPage() {
// eslint-disable-next-line react-hooks/exhaustive-deps // eslint-disable-next-line react-hooks/exhaustive-deps
}, [carId, isExternal]); }, [carId, isExternal]);
// Дефолтный механик-создатель: подтягиваем себя из /me. Если /me упал —
// оставляем список пустым (backend всё равно засидит создателя на create).
useEffect(() => {
getMe()
.then((me) => setMechanics([{ mechanic_id: me.id, name: me.name }]))
.catch(() => {
/* noop — список останется пустым, backend засидит создателя */
});
}, []);
// У24 упрощено: пробег ввода в форме теперь один — главный «Пробег (км)» // У24 упрощено: пробег ввода в форме теперь один — главный «Пробег (км)»
// в шапке. На submit подставляется и в oil_change.mileage_at_change, чтобы // в шапке. На submit подставляется и в oil_change.mileage_at_change, чтобы
// не заставлять механика вводить одно и то же дважды. // не заставлять механика вводить одно и то же дважды.
@@ -322,6 +337,11 @@ export default function CreateRepairPage() {
...(isExternal ...(isExternal
? { external_car: { plate: externalState!.plate, make: externalState!.make } } ? { external_car: { plate: externalState!.plate, make: externalState!.make } }
: { car_id: carId }), : { car_id: carId }),
// Передаём список работавших механиков; если пуст — backend засидит
// создателя сам (см. seed_creator_mechanic).
...(mechanics.length > 0
? { mechanic_ids: mechanics.map((m) => m.mechanic_id) }
: {}),
mileage, mileage,
works: works.map((w) => ({ works: works.map((w) => ({
work_catalog_id: w.work_catalog_id, work_catalog_id: w.work_catalog_id,
@@ -573,6 +593,9 @@ export default function CreateRepairPage() {
)} )}
</section> </section>
{/* Механики (делёж 1/N) */}
<MechanicsField value={mechanics} onChange={setMechanics} totalSum={totalSum} />
{/* Запчасти */} {/* Запчасти */}
<section> <section>
<Label className="text-sm">Запчасти и материалы</Label> <Label className="text-sm">Запчасти и материалы</Label>
@@ -18,6 +18,7 @@ import {
} from "@/api/repairsFeed"; } from "@/api/repairsFeed";
import { searchWorks, suggestParts, type MechWork } from "@/api/repairsCreate"; import { searchWorks, suggestParts, type MechWork } from "@/api/repairsCreate";
import WorksPickerModal from "@/components/repairs/WorksPickerModal"; import WorksPickerModal from "@/components/repairs/WorksPickerModal";
import MechanicsField, { type SelectedMechanic } from "@/components/repairs/MechanicsField";
interface WorkRow { interface WorkRow {
rowKey: string; rowKey: string;
@@ -61,6 +62,8 @@ export default function EditRepairPage() {
const [works, setWorks] = useState<WorkRow[]>([]); const [works, setWorks] = useState<WorkRow[]>([]);
const [parts, setParts] = useState<PartRow[]>([]); const [parts, setParts] = useState<PartRow[]>([]);
const [oil, setOil] = useState<OilState>({ enabled: false, next_in_km: null }); const [oil, setOil] = useState<OilState>({ enabled: false, next_in_km: null });
// Работавшие механики (делёж 1/N).
const [mechanics, setMechanics] = useState<SelectedMechanic[]>([]);
const [worksPicker, setWorksPicker] = useState<{ open: boolean; q: string; results: MechWork[] }>( const [worksPicker, setWorksPicker] = useState<{ open: boolean; q: string; results: MechWork[] }>(
{ open: false, q: "", results: [] } { open: false, q: "", results: [] }
@@ -92,6 +95,14 @@ export default function EditRepairPage() {
next_in_km: d.oil_change.next_in_km, next_in_km: d.oil_change.next_in_km,
}); });
} }
// Механики: из списка работавших; fallback — создатель (старые ответы).
if (d.mechanics && d.mechanics.length > 0) {
setMechanics(d.mechanics.map((m) => ({ mechanic_id: m.mechanic_id, name: m.name })));
} else if (d.mechanic_name) {
// Нет id создателя в payload — оставляем имя без id-сопоставления;
// если механик ничего не трогает, mechanic_ids не отправим.
setMechanics([]);
}
if (!d.can_edit) { if (!d.can_edit) {
setError("Окно правки закрыто. Изменения отправить нельзя."); setError("Окно правки закрыто. Изменения отправить нельзя.");
} }
@@ -157,6 +168,12 @@ export default function EditRepairPage() {
setError("Минимум одна работа в ремонте"); setError("Минимум одна работа в ремонте");
return; return;
} }
// Механики были загружены, но пользователь убрал всех — backend отклонит
// пустой список, перехватываем заранее с понятным текстом.
if ((detail.mechanics?.length ?? 0) > 0 && mechanics.length === 0) {
setError("Минимум один механик в ремонте");
return;
}
setSaving(true); setSaving(true);
try { try {
@@ -176,6 +193,11 @@ export default function EditRepairPage() {
// Один источник истины — главный пробег ремонта. // Один источник истины — главный пробег ремонта.
? { enabled: true, mileage_at_change: mileage, next_in_km: oil.next_in_km } ? { enabled: true, mileage_at_change: mileage, next_in_km: oil.next_in_km }
: { enabled: false, mileage_at_change: null, next_in_km: null }, : { enabled: false, mileage_at_change: null, next_in_km: null },
// Список механиков отправляем, только если он непуст (иначе бэк
// оставит набор как есть; пустой бы вызвал ValueError).
...(mechanics.length > 0
? { mechanic_ids: mechanics.map((m) => m.mechanic_id) }
: {}),
}); });
navigate(`/repairs/${detail.id}`, { replace: true }); navigate(`/repairs/${detail.id}`, { replace: true });
} catch (e) { } catch (e) {
@@ -305,6 +327,9 @@ export default function EditRepairPage() {
</div> </div>
</section> </section>
{/* Механики (делёж 1/N) */}
<MechanicsField value={mechanics} onChange={setMechanics} totalSum={totalSum} />
{/* Запчасти */} {/* Запчасти */}
<section> <section>
<div className="flex items-baseline justify-between"> <div className="flex items-baseline justify-between">
@@ -131,10 +131,33 @@ export default function RepairDetailPage() {
{data.driver_name} {data.driver_name}
</div> </div>
)} )}
{data.mechanics && data.mechanics.length > 0 ? (
<div>
<span className="text-muted-foreground">
{data.mechanics.length > 1 ? "Механики: " : "Механик: "}
</span>
<span className="inline-flex flex-wrap gap-1.5 align-top">
{data.mechanics.map((m) => (
<span
key={m.id}
className="inline-flex items-center rounded-full border bg-muted/50 px-2.5 py-0.5 text-xs"
>
{m.name}
</span>
))}
</span>
{data.mechanics.length > 1 && (
<div className="text-xs text-muted-foreground mt-1">
Делёж поровну: {fmt(data.total_sum / data.mechanics.length)} на каждого
</div>
)}
</div>
) : (
<div> <div>
<span className="text-muted-foreground">Механик: </span> <span className="text-muted-foreground">Механик: </span>
{data.mechanic_name || "—"} {data.mechanic_name || "—"}
</div> </div>
)}
{data.mileage != null && ( {data.mileage != null && (
<div> <div>
<span className="text-muted-foreground">Пробег: </span> <span className="text-muted-foreground">Пробег: </span>