@startsimpli/ui 0.4.72 → 0.4.73

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@startsimpli/ui",
-  "version": "0.4.72",
+  "version": "0.4.73",
   "description": "Shared UI components package for StartSimpli applications",
   "main": "./src/index.ts",
   "types": "./src/index.ts",
@@ -42,8 +42,8 @@
     "next": "^14.0.0 || ^15.0.0 || ^16.0.0",
     "react": "^18.0.0 || ^19.0.0",
     "react-dom": "^18.0.0 || ^19.0.0",
-    "@startsimpli/auth": "^0.4.31",
-    "@startsimpli/api": "^0.5.45"
+    "@startsimpli/api": "^0.5.45",
+    "@startsimpli/auth": "^0.4.31"
   },
   "peerDependenciesMeta": {
     "@startsimpli/api": {

package/src/components/collection/DraftReviewWorkspace.tsx

@@ -27,6 +27,7 @@ import {
   gradeWords,
   wordTargets,
   sourceOverlapCheck,
+  pickChanges,
   type DraftGroup,
   type DraftSource,
   type CheckStatus,
@@ -93,14 +94,7 @@ export function DraftReviewWorkspace({ client }: DraftReviewWorkspaceProps) {
   async function pick(group: DraftGroup, cand: EntityRecord) {
     setBusyKey(group.key);
     try {
-      await Promise.all(
-        group.candidates.map((c) =>
-          writeEntity(c, {
-            chosen: String(c.id) === String(cand.id),
-            status: String(c.id) === String(cand.id) ? 'selected' : field(c, 'status') === 'ready' ? 'ready' : '',
-          }),
-        ),
-      );
+      await Promise.all(group.candidates.map((c) => writeEntity(c, pickChanges(c.id, cand.id))));
       await qc.invalidateQueries({ queryKey: ['entities', 'draft'] });
     } catch (err) {
       notify.error(err instanceof Error ? err.message : 'Could not save the pick.');

package/src/components/collection/__tests__/drafts.test.ts

@@ -1,4 +1,4 @@
-import { parseSources, normUrl, groupKeyOf, groupDrafts, candidateTitle, gradeWords, wordTargets, sourceOverlapCheck } from '../drafts';
+import { parseSources, normUrl, groupKeyOf, groupDrafts, candidateTitle, gradeWords, wordTargets, sourceOverlapCheck, pickChanges } from '../drafts';
 import type { EntityRecord } from '../types';
 
 function rec(id: string, name: string, data: Record<string, unknown>): EntityRecord {
@@ -121,3 +121,16 @@ describe('sourceOverlapCheck (768w.18.16.2 wiring)', () => {
     expect(check!.status).toBe('fail');
   });
 });
+
+describe('pickChanges (startsim-yvd3 — pick must not write an invalid status)', () => {
+  it('marks the picked candidate chosen and writes NO status key', () => {
+    const picked = pickChanges('a', 'a');
+    expect(picked).toEqual({ chosen: true });
+    expect('status' in picked).toBe(false); // status enum is the tenant's; picking must not touch it
+    expect(pickChanges('b', 'a')).toEqual({ chosen: false });
+  });
+  it('compares ids as strings (handles number vs string ids)', () => {
+    expect(pickChanges(7 as unknown as string, '7')).toEqual({ chosen: true });
+    expect(pickChanges(7 as unknown as string, 8 as unknown as string)).toEqual({ chosen: false });
+  });
+});

package/src/components/collection/drafts.ts

@@ -160,6 +160,17 @@ export function gradeWords(count: number, [lo, hi]: [number, number]): CheckStat
   return 'ok';
 }
 
+/**
+ * The record change a Pick applies to ONE candidate: mark it chosen (true for the
+ * picked candidate, false for the rest). Picking is a selection flag — it must NOT
+ * write a lifecycle `status`. The tenant constrains draft `status` to an enum, so
+ * writing 'selected' (or '') 400s the whole pick (startsim-yvd3); the "picked"
+ * badge already keys off `chosen` via isChosen(). Pure, so it's unit-tested.
+ */
+export function pickChanges(candidateId: EntityRecord['id'], pickedId: EntityRecord['id']): { chosen: boolean } {
+  return { chosen: String(candidateId) === String(pickedId) };
+}
+
 /**
  * The verbatim-overlap guardrail (768w.18.16.2) for a single draft. Runs the
  * shared `source-overlap` check over the draft's blog + linkedin against the