@hed-hog/operations 0.0.329 → 0.0.331

package/hedhog/frontend/app/_components/project-form-screen.tsx.ejs

@@ -1,5 +1,11 @@
 'use client';
 
+import { PersonFormSheet } from '@/app/(app)/(libraries)/contact/person/_components/person-form-sheet';
+import type {
+  ContactTypeOption,
+  DocumentTypeOption,
+  Person,
+} from '@/app/(app)/(libraries)/contact/person/_components/person-types';
 import { EmptyState, Page } from '@/components/entity-list';
 import { Avatar, AvatarFallback, AvatarImage } from '@/components/ui/avatar';
 import { Button } from '@/components/ui/button';
@@ -43,6 +49,12 @@ import {
   SheetHeader,
   SheetTitle,
 } from '@/components/ui/sheet';
+import {
+  Tabs,
+  TabsContent,
+  TabsList,
+  TabsTrigger,
+} from '@/components/ui/tabs';
 import { Textarea } from '@/components/ui/textarea';
 import {
   Tooltip,
@@ -66,7 +78,7 @@ import {
 import { useTranslations } from 'next-intl';
 import Link from 'next/link';
 import { useRouter } from 'next/navigation';
-import { useEffect, useMemo, useState } from 'react';
+import { useEffect, useMemo, useRef, useState } from 'react';
 import { useForm } from 'react-hook-form';
 import { z } from 'zod';
 import { PersonPicker } from '../../contact/_components/person-picker';
@@ -80,7 +92,6 @@ import type {
   OperationsCollaborator,
   OperationsContract,
   OperationsProjectDetails,
-  OperationsProjectRole,
 } from '../_lib/types';
 import { formatEnumLabel } from '../_lib/utils/format';
 import {
@@ -91,14 +102,13 @@ import {
 import { ContractFormScreen } from './contract-form-screen';
 import { DepartmentPicker } from './department-picker';
 import { OperationsHeader } from './operations-header';
+import { ProjectFileAttachments } from './project-file-attachments';
 
 const OPTION_PAGE_SIZE = 12;
 
 type TeamAssignmentState = {
   collaboratorId: number;
   selected: boolean;
-  projectRoleId: string;
-  roleLabel: string;
   weeklyHours: string;
   allocationPercent: string;
   status: string;
@@ -111,6 +121,7 @@ type ProjectFormState = {
   managerCollaboratorId: string;
   code: string;
   name: string;
+  clientPersonId: string;
   clientName: string;
   summary: string;
   status: string;
@@ -173,6 +184,7 @@ function buildEmptyForm(
     managerCollaboratorId: 'none',
     code: '',
     name: '',
+    clientPersonId: '',
     clientName: '',
     summary: '',
     status: 'planning',
@@ -189,8 +201,6 @@ function buildEmptyForm(
     teamAssignments: collaborators.map((collaborator) => ({
       collaboratorId: collaborator.id,
       selected: false,
-      projectRoleId: 'none',
-      roleLabel: '',
       weeklyHours: '',
       allocationPercent: '',
       status: 'active',
@@ -232,6 +242,9 @@ function toFormState(
       : 'none',
     code: project.code ?? '',
     name: project.name ?? '',
+    clientPersonId: project.clientPersonId
+      ? String(project.clientPersonId)
+      : '',
     clientName: project.clientName ?? '',
     summary: project.summary ?? '',
     status: project.status ?? 'planning',
@@ -260,10 +273,6 @@ function toFormState(
       return {
         collaboratorId: collaborator.id,
         selected: Boolean(assignment),
-        projectRoleId: assignment?.projectRoleId
-          ? String(assignment.projectRoleId)
-          : 'none',
-        roleLabel: assignment?.roleLabel ?? '',
         weeklyHours:
           assignment?.weeklyHours !== null &&
           assignment?.weeklyHours !== undefined
@@ -640,6 +649,9 @@ export function ProjectFormScreen({
   const access = useOperationsAccess();
   const router = useRouter();
   const [assignmentSearch, setAssignmentSearch] = useState('');
+  const personSheetModeRef = useRef<'create' | 'edit'>('edit');
+  const [personSheetOpen, setPersonSheetOpen] = useState(false);
+  const [personToEdit, setPersonToEdit] = useState<Person | null>(null);
   const isSheetMode = Boolean(onCancel);
   const isCreateMode = !projectId;
   const [codeAutoMode, setCodeAutoMode] = useState(isCreateMode);
@@ -651,6 +663,7 @@ export function ProjectFormScreen({
         managerCollaboratorId: z.string(),
         code: z.string().trim().min(1, t('messages.requiredFields')),
         name: z.string().trim().min(1, t('messages.requiredFields')),
+        clientPersonId: z.string(),
         clientName: z.string().trim().min(1, t('messages.requiredFields')),
         summary: z.string(),
         status: z.string(),
@@ -678,8 +691,6 @@ export function ProjectFormScreen({
             z.object({
               collaboratorId: z.number(),
               selected: z.boolean(),
-              projectRoleId: z.string(),
-              roleLabel: z.string(),
               weeklyHours: z.string(),
               allocationPercent: z.string(),
               status: z.string(),
@@ -688,14 +699,8 @@ export function ProjectFormScreen({
             })
           )
           .superRefine((assignments, ctx) => {
-            assignments.forEach((assignment, index) => {
-              if (assignment.selected && !assignment.roleLabel.trim()) {
-                ctx.addIssue({
-                  code: z.ZodIssueCode.custom,
-                  message: t('messages.roleRequired'),
-                  path: [index, 'roleLabel'],
-                });
-              }
+            assignments.forEach((_assignment, _index) => {
+              // role validation removed
             });
           }),
       }),
@@ -750,18 +755,32 @@ export function ProjectFormScreen({
       fetchOperations<OperationsContract[]>(request, '/operations/contracts'),
   });
 
-  const { data: projectRoles = [], refetch: refetchProjectRoles } = useQuery<
-    OperationsProjectRole[]
-  >({
-    queryKey: ['operations-project-form-project-roles', currentLocaleCode],
-    enabled: access.isDirector,
-    staleTime: 0,
-    refetchOnMount: 'always',
-    queryFn: () =>
-      fetchOperations<OperationsProjectRole[]>(
-        request,
-        '/operations/project-roles'
-      ),
+  const { data: contactTypes = [] } = useQuery<ContactTypeOption[]>({
+    queryKey: ['contact-person-contact-types', currentLocaleCode],
+    enabled: personSheetOpen,
+    queryFn: async () => {
+      const response = await request<{ data: ContactTypeOption[] }>({
+        url: '/person-contact-type?pageSize=100',
+        method: 'GET',
+      });
+
+      return response.data.data || [];
+    },
+    placeholderData: (previous) => previous ?? [],
+  });
+
+  const { data: documentTypes = [] } = useQuery<DocumentTypeOption[]>({
+    queryKey: ['contact-person-document-types', currentLocaleCode],
+    enabled: personSheetOpen,
+    queryFn: async () => {
+      const response = await request<{ data: DocumentTypeOption[] }>({
+        url: '/person-document-type?pageSize=100',
+        method: 'GET',
+      });
+
+      return response.data.data || [];
+    },
+    placeholderData: (previous) => previous ?? [],
   });
 
   const { data: project, isLoading: isLoadingProject } =
@@ -869,39 +888,6 @@ export function ProjectFormScreen({
     [availableCollaborators]
   );
 
-  const projectRoleOptions = useMemo(
-    () =>
-      projectRoles.map((role) => ({
-        id: role.id,
-        name: role.name,
-        code: role.code ?? null,
-        description: role.description ?? null,
-      })),
-    [projectRoles]
-  );
-
-  const createProjectRole = async (projectRoleName: string) => {
-    try {
-      const createdRole = await mutateOperations<OperationsProjectRole>(
-        request,
-        '/operations/project-roles',
-        'POST',
-        {
-          name: projectRoleName,
-        }
-      );
-
-      await refetchProjectRoles();
-      return createdRole;
-    } catch (error) {
-      showToastHandler?.(
-        'error',
-        getOperationsErrorMessage(error, t('messages.projectRoleSaveError'))
-      );
-      return null;
-    }
-  };
-
   const selectedAssignmentsCount = useMemo(
     () =>
       form.teamAssignments.filter((assignment) => assignment.selected).length,
@@ -965,6 +951,9 @@ export function ProjectFormScreen({
         values.managerCollaboratorId === 'none'
           ? null
           : parseNumberInput(values.managerCollaboratorId),
+      clientPersonId: values.clientPersonId
+        ? parseNumberInput(values.clientPersonId)
+        : null,
       code: values.code.trim(),
       name: values.name.trim(),
       clientName: trimToNull(values.clientName),
@@ -984,11 +973,6 @@ export function ProjectFormScreen({
         .filter((assignment) => assignment.selected)
         .map((assignment) => ({
           collaboratorId: assignment.collaboratorId,
-          projectRoleId:
-            assignment.projectRoleId === 'none'
-              ? null
-              : parseNumberInput(assignment.projectRoleId),
-          roleLabel: trimToNull(assignment.roleLabel),
           weeklyHours: parseNumberInput(assignment.weeklyHours),
           allocationPercent: parseNumberInput(assignment.allocationPercent),
           status: assignment.status,
@@ -1038,6 +1022,44 @@ export function ProjectFormScreen({
     showToastHandler?.('error', t('messages.requiredFields'));
   };
 
+  const resolvePersonDisplayName = (person?: Person | null) => {
+    const tradeName = person?.trade_name?.trim();
+    if (tradeName) {
+      return tradeName;
+    }
+
+    return person?.name?.trim() || '';
+  };
+
+  const handleOpenPersonEditSheet = async (personId?: number | null) => {
+    const targetPersonId =
+      personId && personId > 0
+        ? personId
+        : parseNumberInput(formMethods.getValues('clientPersonId'));
+
+    if (!targetPersonId) {
+      return;
+    }
+
+    try {
+      const response = await request<Person>({
+        url: `/person/${targetPersonId}`,
+        method: 'GET',
+      });
+      personSheetModeRef.current = 'edit';
+      setPersonToEdit(response.data);
+      setPersonSheetOpen(true);
+    } catch {
+      showToastHandler?.('error', t('messages.updateError'));
+    }
+  };
+
+  const handleOpenPersonCreateSheet = () => {
+    personSheetModeRef.current = 'create';
+    setPersonToEdit(null);
+    setPersonSheetOpen(true);
+  };
+
   const noAccessState = (
     <EmptyState
       icon={<FolderKanban className="size-12" />}
@@ -1148,12 +1170,26 @@ export function ProjectFormScreen({
                       <PersonPicker
                         label=""
                         entityLabel={t('fields.clientName')}
-                        value={null}
+                        value={
+                          formMethods.watch('clientPersonId')
+                            ? Number(formMethods.watch('clientPersonId'))
+                            : null
+                        }
                         initialSelectedLabel={field.value}
                         selectPlaceholder={t('placeholders.clientName')}
-                        onChange={(_, personName) =>
-                          field.onChange(personName ?? '')
+                        onChange={(personId, personName) => {
+                          field.onChange(personName ?? '');
+                          formMethods.setValue(
+                            'clientPersonId',
+                            personId ? String(personId) : ''
+                          );
+                        }}
+                        showEditButton
+                        editAriaLabel={commonT('actions.editPersonCrm')}
+                        onEditSelection={(personId) =>
+                          void handleOpenPersonEditSheet(personId)
                         }
+                        onCreateNew={() => handleOpenPersonCreateSheet()}
                         personTypeFilter="all"
                         createType="company"
                       />
@@ -1279,9 +1315,6 @@ export function ProjectFormScreen({
                       <SelectItem value="active">
                         {t('options.statuses.active')}
                       </SelectItem>
-                      <SelectItem value="at_risk">
-                        {t('options.statuses.at_risk')}
-                      </SelectItem>
                       <SelectItem value="paused">
                         {t('options.statuses.paused')}
                       </SelectItem>
@@ -1338,321 +1371,316 @@ export function ProjectFormScreen({
 
         {!isCreateMode ? (
           <>
-            <section className="space-y-3">
-              <div className="space-y-0.5">
-                <h3 className="text-xs font-semibold uppercase tracking-wide text-muted-foreground">
+            <Tabs defaultValue="financials" className="space-y-3">
+              <TabsList className="grid w-full grid-cols-3">
+                <TabsTrigger value="financials">
                   {t('sections.financials')}
-                </h3>
-                <p className="text-[11px] text-muted-foreground/80">
-                  {t('sections.financialsDescription')}
-                </p>
-              </div>
-              <div className="grid min-w-0 gap-3 md:grid-cols-2 xl:grid-cols-4">
-                <div className="min-w-0 space-y-2">
-                  <FieldLabel label={commonT('labels.budget')} />
-                  <InputMoney
-                    value={
-                      form.budgetAmount === '' ? '' : Number(form.budgetAmount)
-                    }
-                    onValueChange={(value) =>
-                      setForm((current) => ({
-                        ...current,
-                        budgetAmount: value !== null ? String(value) : '',
-                      }))
-                    }
-                  />
-                </div>
-                <div className="min-w-0 space-y-2">
-                  <FieldLabel
-                    label={commonT('labels.monthlyHourCap')}
-                    hint={t('hints.monthlyHourCap')}
-                  />
-                  <Input
-                    type="text"
-                    inputMode="numeric"
-                    placeholder={t('placeholders.monthlyHourCap')}
-                    value={form.monthlyHourCap}
-                    onChange={(event) => {
-                      const raw = event.target.value.replace(/[^0-9]/g, '');
-                      setForm((current) => ({
-                        ...current,
-                        monthlyHourCap: raw,
-                      }));
-                    }}
-                  />
-                </div>
-                <div className="min-w-0 space-y-2">
-                  <FieldLabel label={commonT('labels.billingModel')} />
-                  <Select
-                    value={form.billingModel}
-                    onValueChange={(value) =>
-                      setForm((current) => ({
-                        ...current,
-                        billingModel: value,
-                      }))
-                    }
-                  >
-                    <SelectTrigger className="w-full">
-                      <SelectValue />
-                    </SelectTrigger>
-                    <SelectContent>
-                      <SelectItem value="time_and_material">
-                        {t('options.billingModels.time_and_material')}
-                      </SelectItem>
-                      <SelectItem value="monthly_retainer">
-                        {t('options.billingModels.monthly_retainer')}
-                      </SelectItem>
-                      <SelectItem value="fixed_price">
-                        {t('options.billingModels.fixed_price')}
-                      </SelectItem>
-                    </SelectContent>
-                  </Select>
+                </TabsTrigger>
+                <TabsTrigger value="team">{t('sections.team')}</TabsTrigger>
+                <TabsTrigger value="files">{t('sections.files')}</TabsTrigger>
+              </TabsList>
+
+              <TabsContent value="financials" className="space-y-3">
+                <div className="space-y-0.5">
+                  <h3 className="text-xs font-semibold uppercase tracking-wide text-muted-foreground">
+                    {t('sections.financials')}
+                  </h3>
+                  <p className="text-[11px] text-muted-foreground/80">
+                    {t('sections.financialsDescription')}
+                  </p>
                 </div>
-                <div className="min-w-0 space-y-2">
-                  <FieldLabel
-                    label={commonT('labels.contract')}
-                    hint={t('hints.contract')}
-                  />
-                  <ContractSelectWithCreate
-                    label=""
-                    value={form.contractId}
-                    contracts={availableContracts}
-                    selectPlaceholder={commonT('labels.notAssigned')}
-                    searchPlaceholder={t('placeholders.contractSearch')}
-                    onChange={(value) =>
-                      setForm((current) => ({ ...current, contractId: value }))
-                    }
-                    onCreated={async (contract) => {
-                      await refetchContracts();
-                      setForm((current) => ({
-                        ...current,
-                        contractId: contract?.id
-                          ? String(contract.id)
-                          : current.contractId,
-                        billingModel:
-                          contract?.billingModel ?? current.billingModel,
-                        monthlyHourCap:
-                          contract?.monthlyHourCap !== null &&
-                          contract?.monthlyHourCap !== undefined
-                            ? String(contract.monthlyHourCap)
-                            : current.monthlyHourCap,
-                      }));
-                    }}
-                    initialValues={{
-                      code: form.code ? `PRJ-${form.code}` : '',
-                      name: form.name ? `${form.name} Service Agreement` : '',
-                      clientName: form.clientName,
-                      contractCategory: 'client',
-                      contractType: 'service_agreement',
-                      signatureStatus: 'not_started',
-                      billingModel: form.billingModel,
-                      budgetAmount: form.budgetAmount,
-                      monthlyHourCap: form.monthlyHourCap,
-                      startDate: form.startDate,
-                      endDate: form.endDate,
-                      description: form.summary,
-                      contentHtml: '',
-                    }}
-                  />
+                <div className="grid min-w-0 gap-3 md:grid-cols-2 xl:grid-cols-4">
+                  <div className="min-w-0 space-y-2">
+                    <FieldLabel label={commonT('labels.budget')} />
+                    <InputMoney
+                      value={
+                        form.budgetAmount === ''
+                          ? ''
+                          : Number(form.budgetAmount)
+                      }
+                      onValueChange={(value) =>
+                        setForm((current) => ({
+                          ...current,
+                          budgetAmount: value !== null ? String(value) : '',
+                        }))
+                      }
+                    />
+                  </div>
+                  <div className="min-w-0 space-y-2">
+                    <FieldLabel
+                      label={commonT('labels.monthlyHourCap')}
+                      hint={t('hints.monthlyHourCap')}
+                    />
+                    <Input
+                      type="text"
+                      inputMode="numeric"
+                      placeholder={t('placeholders.monthlyHourCap')}
+                      value={form.monthlyHourCap}
+                      onChange={(event) => {
+                        const raw = event.target.value.replace(/[^0-9]/g, '');
+                        setForm((current) => ({
+                          ...current,
+                          monthlyHourCap: raw,
+                        }));
+                      }}
+                    />
+                  </div>
+                  <div className="min-w-0 space-y-2">
+                    <FieldLabel label={commonT('labels.billingModel')} />
+                    <Select
+                      value={form.billingModel}
+                      onValueChange={(value) =>
+                        setForm((current) => ({
+                          ...current,
+                          billingModel: value,
+                        }))
+                      }
+                    >
+                      <SelectTrigger className="w-full">
+                        <SelectValue />
+                      </SelectTrigger>
+                      <SelectContent>
+                        <SelectItem value="time_and_material">
+                          {t('options.billingModels.time_and_material')}
+                        </SelectItem>
+                        <SelectItem value="monthly_retainer">
+                          {t('options.billingModels.monthly_retainer')}
+                        </SelectItem>
+                        <SelectItem value="fixed_price">
+                          {t('options.billingModels.fixed_price')}
+                        </SelectItem>
+                      </SelectContent>
+                    </Select>
+                  </div>
+                  <div className="min-w-0 space-y-2">
+                    <FieldLabel
+                      label={commonT('labels.contract')}
+                      hint={t('hints.contract')}
+                    />
+                    <ContractSelectWithCreate
+                      label=""
+                      value={form.contractId}
+                      contracts={availableContracts}
+                      selectPlaceholder={commonT('labels.notAssigned')}
+                      searchPlaceholder={t('placeholders.contractSearch')}
+                      onChange={(value) =>
+                        setForm((current) => ({
+                          ...current,
+                          contractId: value,
+                        }))
+                      }
+                      onCreated={async (contract) => {
+                        await refetchContracts();
+                        setForm((current) => ({
+                          ...current,
+                          contractId: contract?.id
+                            ? String(contract.id)
+                            : current.contractId,
+                          billingModel:
+                            contract?.billingModel ?? current.billingModel,
+                          monthlyHourCap:
+                            contract?.monthlyHourCap !== null &&
+                            contract?.monthlyHourCap !== undefined
+                              ? String(contract.monthlyHourCap)
+                              : current.monthlyHourCap,
+                        }));
+                      }}
+                      initialValues={{
+                        code: form.code ? `PRJ-${form.code}` : '',
+                        name: form.name
+                          ? `${form.name} Service Agreement`
+                          : '',
+                        clientName: form.clientName,
+                        contractCategory: 'client',
+                        contractType: 'service_agreement',
+                        signatureStatus: 'not_started',
+                        billingModel: form.billingModel,
+                        budgetAmount: form.budgetAmount,
+                        monthlyHourCap: form.monthlyHourCap,
+                        startDate: form.startDate,
+                        endDate: form.endDate,
+                        description: form.summary,
+                        contentHtml: '',
+                      }}
+                    />
+                  </div>
                 </div>
-              </div>
-            </section>
-
-            <section className="space-y-3">
-              <div className="space-y-0.5">
-                <h3 className="text-xs font-semibold uppercase tracking-wide text-muted-foreground">
-                  {t('sections.team')}
-                </h3>
-                <p className="text-[11px] text-muted-foreground/80">
-                  {t('sections.teamDescription', {
-                    count: selectedAssignmentsCount,
-                  })}
-                </p>
-              </div>
-              <div className="space-y-3">
-                <div className="relative">
-                  <Search className="pointer-events-none absolute top-1/2 left-3 h-4 w-4 -translate-y-1/2 text-muted-foreground" />
-                  <Input
-                    className="pl-9"
-                    value={assignmentSearch}
-                    placeholder={t('placeholders.assignmentSearch')}
-                    onChange={(event) =>
-                      setAssignmentSearch(event.target.value)
-                    }
-                  />
+              </TabsContent>
+
+              <TabsContent value="team" className="space-y-3">
+                <div className="space-y-0.5">
+                  <h3 className="text-xs font-semibold uppercase tracking-wide text-muted-foreground">
+                    {t('sections.team')}
+                  </h3>
+                  <p className="text-[11px] text-muted-foreground/80">
+                    {t('sections.teamDescription', {
+                      count: selectedAssignmentsCount,
+                    })}
+                  </p>
                 </div>
+                <div className="space-y-3">
+                  <div className="relative">
+                    <Search className="pointer-events-none absolute top-1/2 left-3 h-4 w-4 -translate-y-1/2 text-muted-foreground" />
+                    <Input
+                      className="pl-9"
+                      value={assignmentSearch}
+                      placeholder={t('placeholders.assignmentSearch')}
+                      onChange={(event) =>
+                        setAssignmentSearch(event.target.value)
+                      }
+                    />
+                  </div>
 
-                <div className="space-y-2">
-                  {filteredAssignments.map((assignment) => {
-                    const collaborator = availableCollaborators.find(
-                      (item) => item.id === assignment.collaboratorId
-                    );
-                    const assignmentIndex = form.teamAssignments.findIndex(
-                      (item) =>
-                        item.collaboratorId === assignment.collaboratorId
-                    );
-                    const roleError =
-                      assignmentIndex >= 0
-                        ? formMethods.formState.errors.teamAssignments?.[
-                            assignmentIndex
-                          ]?.roleLabel
-                        : undefined;
-
-                    if (!collaborator) {
-                      return null;
-                    }
+                  <div className="space-y-2">
+                    {filteredAssignments.map((assignment) => {
+                      const collaborator = availableCollaborators.find(
+                        (item) => item.id === assignment.collaboratorId
+                      );
 
-                    return (
-                      <div
-                        key={assignment.collaboratorId}
-                        className="grid min-w-0 gap-2 rounded-lg border px-3 py-2 xl:grid-cols-[minmax(0,1.25fr)_minmax(0,1.2fr)_110px_110px]"
-                      >
-                        <label className="flex cursor-pointer items-start gap-3 py-1">
-                          <Checkbox
-                            checked={assignment.selected}
-                            onCheckedChange={(checked) =>
-                              updateAssignment(assignment.collaboratorId, {
-                                selected: checked === true,
-                              })
-                            }
-                          />
-                          <div className="min-w-0">
-                            <div className="truncate font-medium">
-                              {collaborator.displayName}
-                            </div>
-                            <div className="truncate text-xs text-muted-foreground">
-                              {[
-                                collaborator.department,
-                                collaborator.title,
-                                collaborator.code,
-                              ]
-                                .filter(Boolean)
-                                .join(' • ') || commonT('labels.notAvailable')}
-                            </div>
-                          </div>
-                        </label>
-                        <div className="space-y-1">
-                          <DepartmentPicker
-                            label=""
-                            value={assignment.roleLabel}
-                            options={projectRoleOptions}
-                            disabled={!assignment.selected}
-                            selectPlaceholder={t('placeholders.roleLabel')}
-                            createDescription={t('fields.roleLabel')}
-                            createPlaceholder={t(
-                              'placeholders.roleLabelCreate'
-                            )}
-                            onChange={(role) =>
-                              updateAssignment(assignment.collaboratorId, {
-                                projectRoleId: role.id
-                                  ? String(role.id)
-                                  : 'none',
-                                roleLabel: role.name,
-                              })
-                            }
-                            onCreate={createProjectRole}
-                          />
-                          {roleError?.message ? (
-                            <p className="text-sm text-destructive">
-                              {String(roleError.message)}
-                            </p>
-                          ) : null}
-                        </div>
-                        <Input
-                          className="h-9 mt-2 self-start"
-                          type="number"
-                          min="0"
-                          step="0.5"
-                          placeholder={t('fields.weeklyHours')}
-                          value={assignment.weeklyHours}
-                          disabled={!assignment.selected}
-                          onChange={(event) => {
-                            const hours = event.target.value;
-                            const updates: Partial<TeamAssignmentState> = {
-                              weeklyHours: hours,
-                            };
-                            const cap = collaborator.weeklyCapacityHours;
-                            if (cap && hours && Number(hours) > 0) {
-                              const pct = Math.round(
-                                (Number(hours) / cap) * 100
-                              );
-                              updates.allocationPercent = String(
-                                Math.min(pct, 100)
-                              );
-                            }
-                            updateAssignment(
-                              assignment.collaboratorId,
-                              updates
-                            );
-                          }}
-                        />
-                        <Input
-                          className="h-9 mt-2 self-start"
-                          type="text"
-                          inputMode="decimal"
-                          placeholder={t('fields.allocationPercent')}
-                          value={assignment.allocationPercent}
-                          disabled={!assignment.selected}
-                          onChange={(event) => {
-                            const pct = normalizePercentInput(
-                              event.target.value
-                            );
-                            const updates: Partial<TeamAssignmentState> = {
-                              allocationPercent: pct,
-                            };
-                            const cap = collaborator.weeklyCapacityHours;
-                            if (cap && pct && Number(pct) > 0) {
-                              const hours = Math.round(
-                                (Number(pct) / 100) * cap
-                              );
-                              updates.weeklyHours = String(hours);
-                            }
-                            updateAssignment(
-                              assignment.collaboratorId,
-                              updates
-                            );
-                          }}
-                        />
-                        <div className="flex items-center gap-2 xl:col-span-4">
-                          <div className="flex-1">
-                            <label className="text-xs text-muted-foreground">
-                              {t('fields.assignmentStartDate')}
-                            </label>
-                            <Input
-                              className="h-9"
-                              type="date"
-                              value={assignment.startDate}
-                              disabled={!assignment.selected}
-                              onChange={(event) =>
+                      if (!collaborator) {
+                        return null;
+                      }
+
+                      return (
+                        <div
+                          key={assignment.collaboratorId}
+                          className="grid min-w-0 gap-2 rounded-lg border px-3 py-2 xl:grid-cols-[minmax(0,2fr)_110px_110px]"
+                        >
+                          <label className="flex cursor-pointer items-start gap-3 py-1">
+                            <Checkbox
+                              checked={assignment.selected}
+                              onCheckedChange={(checked) =>
                                 updateAssignment(assignment.collaboratorId, {
-                                  startDate: event.target.value,
+                                  selected: checked === true,
                                 })
                               }
                             />
-                          </div>
-                          <div className="flex-1">
-                            <label className="text-xs text-muted-foreground">
-                              {t('fields.assignmentEndDate')}
-                            </label>
-                            <Input
-                              className="h-9"
-                              type="date"
-                              value={assignment.endDate}
-                              disabled={!assignment.selected}
-                              onChange={(event) =>
-                                updateAssignment(assignment.collaboratorId, {
-                                  endDate: event.target.value,
-                                })
+                            <div className="min-w-0">
+                              <div className="truncate font-medium">
+                                {collaborator.displayName}
+                              </div>
+                              <div className="truncate text-xs text-muted-foreground">
+                                {[
+                                  collaborator.department,
+                                  collaborator.title,
+                                  collaborator.code,
+                                ]
+                                  .filter(Boolean)
+                                  .join(' • ') ||
+                                  commonT('labels.notAvailable')}
+                              </div>
+                            </div>
+                          </label>
+                          <Input
+                            className="h-9 mt-2 self-start"
+                            type="number"
+                            min="0"
+                            step="0.5"
+                            placeholder={t('fields.weeklyHours')}
+                            value={assignment.weeklyHours}
+                            disabled={!assignment.selected}
+                            onChange={(event) => {
+                              const hours = event.target.value;
+                              const updates: Partial<TeamAssignmentState> = {
+                                weeklyHours: hours,
+                              };
+                              const cap = collaborator.weeklyCapacityHours;
+                              if (cap && hours && Number(hours) > 0) {
+                                const pct = Math.round(
+                                  (Number(hours) / cap) * 100
+                                );
+                                updates.allocationPercent = String(
+                                  Math.min(pct, 100)
+                                );
                               }
-                            />
+                              updateAssignment(
+                                assignment.collaboratorId,
+                                updates
+                              );
+                            }}
+                          />
+                          <Input
+                            className="h-9 mt-2 self-start"
+                            type="text"
+                            inputMode="decimal"
+                            placeholder={t('fields.allocationPercent')}
+                            value={assignment.allocationPercent}
+                            disabled={!assignment.selected}
+                            onChange={(event) => {
+                              const pct = normalizePercentInput(
+                                event.target.value
+                              );
+                              const updates: Partial<TeamAssignmentState> = {
+                                allocationPercent: pct,
+                              };
+                              const cap = collaborator.weeklyCapacityHours;
+                              if (cap && pct && Number(pct) > 0) {
+                                const hours = Math.round(
+                                  (Number(pct) / 100) * cap
+                                );
+                                updates.weeklyHours = String(hours);
+                              }
+                              updateAssignment(
+                                assignment.collaboratorId,
+                                updates
+                              );
+                            }}
+                          />
+                          <div className="flex items-center gap-2 xl:col-span-4">
+                            <div className="flex-1">
+                              <label className="text-xs text-muted-foreground">
+                                {t('fields.assignmentStartDate')}
+                              </label>
+                              <Input
+                                className="h-9"
+                                type="date"
+                                value={assignment.startDate}
+                                disabled={!assignment.selected}
+                                onChange={(event) =>
+                                  updateAssignment(assignment.collaboratorId, {
+                                    startDate: event.target.value,
+                                  })
+                                }
+                              />
+                            </div>
+                            <div className="flex-1">
+                              <label className="text-xs text-muted-foreground">
+                                {t('fields.assignmentEndDate')}
+                              </label>
+                              <Input
+                                className="h-9"
+                                type="date"
+                                value={assignment.endDate}
+                                disabled={!assignment.selected}
+                                onChange={(event) =>
+                                  updateAssignment(assignment.collaboratorId, {
+                                    endDate: event.target.value,
+                                  })
+                                }
+                              />
+                            </div>
                           </div>
                         </div>
-                      </div>
-                    );
-                  })}
+                      );
+                    })}
+                  </div>
                 </div>
-              </div>
-            </section>
+              </TabsContent>
+
+              <TabsContent value="files" className="space-y-3">
+                <div className="space-y-0.5">
+                  <h3 className="text-xs font-semibold uppercase tracking-wide text-muted-foreground">
+                    {t('sections.files')}
+                  </h3>
+                  <p className="text-[11px] text-muted-foreground/80">
+                    {t('sections.filesDescription')}
+                  </p>
+                </div>
+                {projectId ? (
+                  <ProjectFileAttachments projectId={projectId} />
+                ) : null}
+              </TabsContent>
+            </Tabs>
           </>
         ) : (
           <section className="space-y-3 rounded-lg border border-dashed bg-muted/20 px-4 py-4">
@@ -1684,6 +1712,43 @@ export function ProjectFormScreen({
     </div>
   ) : null;
 
+  const personSheet = (
+    <PersonFormSheet
+      open={personSheetOpen}
+      person={personToEdit}
+      contactTypes={contactTypes}
+      documentTypes={documentTypes}
+      onOpenChange={(nextOpen) => {
+        setPersonSheetOpen(nextOpen);
+        if (!nextOpen) {
+          setPersonToEdit(null);
+        }
+      }}
+      onSuccess={(person) => {
+        const personLabel = resolvePersonDisplayName(person);
+
+        if (personSheetModeRef.current === 'create' && person?.id) {
+          formMethods.setValue('clientPersonId', String(person.id), {
+            shouldDirty: true,
+            shouldTouch: true,
+            shouldValidate: true,
+          });
+        }
+
+        if (personLabel) {
+          formMethods.setValue('clientName', personLabel, {
+            shouldDirty: true,
+            shouldTouch: true,
+            shouldValidate: true,
+          });
+        }
+
+        personSheetModeRef.current = 'edit';
+      }}
+      allowedTypes={['company']}
+    />
+  );
+
   if (isSheetMode) {
     return (
       <div className="mt-6 space-y-4 pb-6">
@@ -1704,6 +1769,8 @@ export function ProjectFormScreen({
           submitLabel={commonT('actions.save')}
           submitSize="lg"
         />
+
+        {personSheet}
       </div>
     );
   }
@@ -1747,6 +1814,8 @@ export function ProjectFormScreen({
         {loadingState}
         {contractStatusState}
       </div>
+
+      {personSheet}
     </Page>
   );
 }