fix pubilsh job posting

src/components/ai-assistant/capabilities/admin.js

@@ -967,9 +967,14 @@ const positionsNeedingAttention = (f) => {
  *
  * A draft is a real record with real values on it and no way for anyone to
  * apply to it, which makes it the one thing on this page that is genuinely
- * unfinished. Each one is offered with the control that finishes it: the same
- * authoring form, told which draft it is continuing, with everything already
- * entered still in place.
+ * unfinished. Each one is named, with what it already holds.
+ *
+ * It reports, it does not reopen. This used to attach "Continue to save" to
+ * every draft, routing to the authoring form — so a question asked in the panel
+ * could end with the reader back inside Create Position, and a position that
+ * had been saved minutes earlier was presented as still needing saving. The
+ * draft is on the Positions list marked Draft and its own card opens the form;
+ * the panel says where it is rather than taking anyone there.
  *
  * Read from `postings` by status — no list of drafts is kept anywhere, and a
  * position stops appearing here the moment it is published.
@@ -991,9 +996,8 @@ const positionDrafts = (f) => {
         p.location,
         'Not published yet, so nobody can apply and no screening runs.',
       ].filter(Boolean).join(' · '),
-      action: { label: 'Continue to save', to: `/admin/positions/new?draft=${encodeURIComponent(p.id)}` },
     }))),
-    note('Continuing reopens the same form with everything already entered — the vetting weights included. Publishing it there is what makes it an active position.')
+    note('Each one is on the Positions list marked Draft. Opening its card reopens the form with everything already entered — the vetting weights included — and publishing it there is what makes it an active position.')
   );
 };
 

src/components/ai-assistant/useAssistant.js

@@ -110,6 +110,50 @@ async function streamDocument({ document, signal, onFrame, onDone }) {
   else if (latest.length) onDone(latest);
 }
 
+/**
+ * The Create Position form, as an address.
+ *
+ * Named once because two things have to agree about it: nothing stored may
+ * offer a way back into the authoring form after a position has been saved.
+ */
+const AUTHORING_ROUTE = '/admin/positions/new';
+
+/**
+ * A stored thread, with any completed-then-reopen action stripped out.
+ *
+ * A reply is persisted as the blocks it rendered, so a thread written before
+ * this was fixed still carries the old "Continue to save" — a live control
+ * routing back into Create Position, offered on a position that was saved long
+ * ago. Removing it in code was not enough; the ones already written down had to
+ * stop working too.
+ *
+ * Deliberately narrow: only actions pointing at the authoring form are dropped,
+ * and only from stored threads. Every other block, chip and link is left as it
+ * was written, so a thread reads exactly as it did apart from the one control
+ * that should never have been there.
+ */
+function withoutAuthoringActions(messages = []) {
+  const leadsToForm = (target) => String(target || '').includes(AUTHORING_ROUTE);
+
+  return messages.map((message) => {
+    if (message.role !== 'assistant') return message;
+
+    const followUp = message.followUp?.filter((chip) => !leadsToForm(chip.route));
+    const blocks = message.blocks?.map((block) => (
+      block?.type === 'insights'
+        ? {
+          ...block,
+          items: block.items.map(({ action, ...item }) => (
+            leadsToForm(action?.to) ? item : { ...item, ...(action ? { action } : null) }
+          )),
+        }
+        : block
+    ));
+
+    return { ...message, ...(followUp ? { followUp } : null), ...(blocks ? { blocks } : null) };
+  });
+}
+
 /**
  * Conversation state for one page context.
  *
@@ -174,7 +218,7 @@ export function useConversation({
   React.useEffect(() => {
     try {
       const raw = sessionStorage.getItem(storageKey);
-      setMessages(raw ? JSON.parse(raw) : []);
+      setMessages(raw ? withoutAuthoringActions(JSON.parse(raw)) : []);
     } catch {
       setMessages([]);
     }
@@ -491,12 +535,17 @@ export function useConversation({
     setError(null);
     setFlow(null);
 
-    messagesRef.current = record.messages;
-    setMessages(record.messages);
+    /* Same rule as loading a stored thread: a conversation kept in History was
+       written before this was fixed too, and reopening it must not hand back a
+       route into the authoring form. */
+    const messages = withoutAuthoringActions(record.messages);
+
+    messagesRef.current = messages;
+    setMessages(messages);
     conversationRef.current = record.id;
 
     try {
-      sessionStorage.setItem(storageKey, JSON.stringify(record.messages));
+      sessionStorage.setItem(storageKey, JSON.stringify(messages));
       sessionStorage.setItem(idKey, record.id);
     } catch {
       // The reopened thread stays in memory only.