Fix contracts page overflow, remove contract link from orders, add directive name (#65)

* feat: soryu-co/soryu - makima: Add frontend pick-up-orders button and API integration

* WIP: heartbeat checkpoint

* feat: soryu-co/soryu - makima: Remove contract link from orders and add directive name to order metadata (frontend)

* fix: contracts page overflow - use contained scrolling layout

Changed the contracts page to use contained scrolling matching the
orders/directives pages, preventing the page from growing beyond
viewport height.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: resolve completion_task_id FK violation and duplicate button

The completion_task_id column has an FK to tasks(id), but
claim_directive_for_completion was being called with a placeholder UUID
that did not exist in the tasks table, causing FK constraint violations.

Fix: Create the task FIRST via create_task_for_owner, then use the real
task.id when calling claim_directive_for_completion. Applied in all three
locations: phase_completion Part 1 (idle directives), Part 3 (verification
tasks), and trigger_completion_task (manual PR creation).

Also removes a duplicate "Pick Up Orders" button in DirectiveDetail.tsx.

* fix: restore Order type changes lost during rebase conflict resolution

Re-apply changes from the orders-refactor commit that were dropped when
resolving rebase conflicts with --ours:
- Replace contractId with directiveName in Order interface
- Make directiveId required in CreateOrderRequest
- Remove contractId from UpdateOrderRequest
- Change listOrders parameter from contractId to search
- Remove linkOrderToContract function
- Simplify convertOrderToStep to single argument

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

1 files changed, 3 insertions, 11 deletions

diff --git a/makima/frontend/src/hooks/useOrders.ts b/makima/frontend/src/hooks/useOrders.ts
index 2dd20bb..9380080 100644
--- a/makima/frontend/src/hooks/useOrders.ts
+++ b/makima/frontend/src/hooks/useOrders.ts
@@ -12,7 +12,6 @@ import {
   updateOrder,
   deleteOrder,
   linkOrderToDirective,
-  linkOrderToContract,
   convertOrderToStep,
 } from "../lib/api";
 
@@ -101,16 +100,9 @@ export function useOrder(id: string | undefined) {
     return o;
   }, [id]);
 
-  const linkContract = useCallback(async (contractId: string) => {
+  const convertToStep = useCallback(async () => {
     if (!id) return;
-    const o = await linkOrderToContract(id, contractId);
-    setOrder(o);
-    return o;
-  }, [id]);
-
-  const convertToStep = useCallback(async (directiveId: string) => {
-    if (!id) return;
-    const step = await convertOrderToStep(id, directiveId);
+    const step = await convertOrderToStep(id);
     await refresh();
     return step;
   }, [id, refresh]);
@@ -118,6 +110,6 @@ export function useOrder(id: string | undefined) {
   return {
     order, loading, error, refresh,
     update, remove,
-    linkDirective, linkContract, convertToStep,
+    linkDirective, convertToStep,
   };
 }