@ikenga/pkg-sales 0.3.0 → 0.4.0

package/dist/app.js

@@ -126,6 +126,12 @@ function App() {
   const [bridgeReady, setBridgeReady] = useState(false);
   const [bridgeError, setBridgeError] = useState(null);
   const [activeFeature, setActiveFeature] = useState(null);
+  // Operator identity (hostContext.operator — see @ikenga/contract's
+  // host-context.ts). OPTIONAL: absent means unknown operator, standalone
+  // dev never sets it. Threaded down as a plain id/label pair, same shape
+  // as activeFeature, so the view fails safe instead of assuming a default.
+  const [operatorId, setOperatorId] = useState(null);
+  const [operatorLabel, setOperatorLabel] = useState(null);
 
   useEffect(() => {
     if (isStandalone()) {
@@ -138,11 +144,17 @@ function App() {
       onContextChange: (ctx) => {
         const af = ctx?.royaltiSuite?.activeFeature;
         if (typeof af === 'string') setActiveFeature(af);
+        const op = ctx?.operator;
+        setOperatorId(typeof op?.id === 'string' ? op.id : null);
+        setOperatorLabel(typeof op?.displayName === 'string' ? op.displayName : (typeof op?.id === 'string' ? op.id : null));
       },
     })
       .then((ctx) => {
         const af = ctx?.royaltiSuite?.activeFeature;
         if (typeof af === 'string') setActiveFeature(af);
+        const op = ctx?.operator;
+        setOperatorId(typeof op?.id === 'string' ? op.id : null);
+        setOperatorLabel(typeof op?.displayName === 'string' ? op.displayName : (typeof op?.id === 'string' ? op.id : null));
         setBridgeReady(true);
       })
       .catch((e) => setBridgeError(e.message ?? String(e)));
@@ -155,7 +167,7 @@ function App() {
     return html`<div style=${{ padding: '2rem', color: 'var(--fg-muted)' }}>Connecting…</div>`;
   }
 
-  return html`<${QueryClientProvider} client=${queryClient}><${SalesView} activeFeature=${activeFeature} /></${QueryClientProvider}>`;
+  return html`<${QueryClientProvider} client=${queryClient}><${SalesView} activeFeature=${activeFeature} operatorId=${operatorId} operatorLabel=${operatorLabel} /></${QueryClientProvider}>`;
 }
 
 createRoot(document.getElementById('root')).render(html`<${App} />`);

package/dist/features/sales/sales-view.js

@@ -39,6 +39,9 @@ import { buildCreateBrief, dispatchCreate } from '../../lib/create-dispatch.js';
 import { dispatchItemAction } from '../../lib/dispatch.js';
 // facet-wire recipe: sidebar filter facets (f:*) narrow the pipeline list/kanban.
 import { applyFacet } from '../../lib/facet-filter.js';
+// operator-identity recipe: hostContext.operator threaded down from app.js —
+// "mine" predicates/fallbacks fail safe (empty/unclaimed) when unknown.
+import { isMine } from '../../lib/operator.js';
 
 // ─── Stage enum ───────────────────────────────────────────────────────────────
 // Per the R-04 Pipeline-stages convention (06-skill-action-contract.md §Pipeline-stages).
@@ -139,7 +142,7 @@ async function fetchOpenDeals() {
   }
 }
 
-async function fetchWonDeals() {
+async function fetchWonDeals(operatorId) {
   if (isStandalone()) return WON_DEALS_FIXTURE;
   try {
     const rows = await hostDbQuery(
@@ -153,7 +156,7 @@ async function fetchWonDeals() {
     return rows.map((r) => ({
       ...r,
       title: r.title ?? r.company,
-      owner: r.owner ?? r.assigned_to ?? 'nedjamez',
+      owner: r.owner ?? r.assigned_to ?? operatorId ?? null,
       closed: r.last_contact ? r.last_contact.substring(0, 10) : '—',
       value: typeof r.value === 'string' ? parseFloat(r.value) || 0 : (r.value ?? 0),
     }));
@@ -227,13 +230,17 @@ function handleAction(deal) {
 
 const SALES_RESET_FACET = 'f:open-pipeline';
 
-const FACET_PREDICATES = {
-  'f:my-deals':     (d) => d.owner === 'nedjamez' || d.assigned_to === 'nedjamez',
-  'f:closing-soon': (d) => d.next_action_mode === 'approve',
-  'f:agent-run':    (d) => d.owner === 'sales-agent' || d.assigned_to === 'sales-agent',
-  ...Object.fromEntries(STAGES.map((s) => [`f:stage:${s}`, (d) => d.stage === s])),
-  // 'f:open-pipeline' intentionally ABSENT → SALES_RESET_FACET returns every deal.
-};
+/** operatorId-parameterized so 'f:my-deals' fails safe (matches nothing) when
+ *  the operator is unknown — see lib/operator.js. */
+function salesFacetPredicates(operatorId) {
+  return {
+    'f:my-deals':     (d) => isMine(d.owner, operatorId) || isMine(d.assigned_to, operatorId),
+    'f:closing-soon': (d) => d.next_action_mode === 'approve',
+    'f:agent-run':    (d) => d.owner === 'sales-agent' || d.assigned_to === 'sales-agent',
+    ...Object.fromEntries(STAGES.map((s) => [`f:stage:${s}`, (d) => d.stage === s])),
+    // 'f:open-pipeline' intentionally ABSENT → SALES_RESET_FACET returns every deal.
+  };
+}
 
 // ─── Menu builder ─────────────────────────────────────────────────────────────
 
@@ -242,10 +249,11 @@ const FACET_PREDICATES = {
  *   activeView: 0 = Pipeline | 1 = Forecast | 2 = Won
  *   pipeMode: 'list' | 'kanban' (only relevant when activeView === 0)
  *   deals: the open deals array (for counts)
+ *   operatorId: current known operator id (null when unknown — see lib/operator.js)
  */
-function buildSalesMenu(activeView, pipeMode, deals, activeFacet) {
+function buildSalesMenu(activeView, pipeMode, deals, activeFacet, operatorId) {
   const openCount = deals.length;
-  const myDeals = deals.filter((d) => d.owner === 'nedjamez' || d.assigned_to === 'nedjamez');
+  const myDeals = deals.filter((d) => isMine(d.owner, operatorId) || isMine(d.assigned_to, operatorId));
   const closingSoon = deals.filter((d) => d.next_action_mode === 'approve');
   const agentRun = deals.filter((d) => (d.owner === 'sales-agent' || d.assigned_to === 'sales-agent'));
   // A facet only highlights on the Pipeline view (facets dim/inert off it).
@@ -787,7 +795,7 @@ function ErrorState({ error, onRetry }) {
 
 // ─── SalesView root ───────────────────────────────────────────────────────────
 
-export function SalesView({ activeFeature }) {
+export function SalesView({ activeFeature, operatorId }) {
   // View state: 0=Pipeline | 1=Forecast | 2=Won
   const [activeView, setActiveView] = useState(() => {
     const p = new URLSearchParams(window.location.search).get('view');
@@ -803,7 +811,11 @@ export function SalesView({ activeFeature }) {
 
   // ── Data queries ────────────────────────────────────────────────────────────
   const openDealsQ = useQuery({ queryKey: QK.openDeals, queryFn: fetchOpenDeals });
-  const wonDealsQ = useQuery({ queryKey: QK.wonDeals, queryFn: fetchWonDeals, enabled: activeView === 2 });
+  const wonDealsQ = useQuery({
+    queryKey: QK.wonDeals,
+    queryFn: () => fetchWonDeals(operatorId),
+    enabled: activeView === 2,
+  });
 
   const activitiesQ = useQuery({
     queryKey: QK.activities(selectedDeal?.id ?? null),
@@ -818,8 +830,8 @@ export function SalesView({ activeFeature }) {
   // nothing shows the empty pane, not a stale full list). Badges stay live
   // because buildSalesMenu counts the FULL `deals`, not this slice.
   const visibleDeals = useMemo(
-    () => applyFacet(deals, activeFacet, FACET_PREDICATES, SALES_RESET_FACET),
-    [deals, activeFacet],
+    () => applyFacet(deals, activeFacet, salesFacetPredicates(operatorId), SALES_RESET_FACET),
+    [deals, activeFacet, operatorId],
   );
 
   // ── db-updated refresh ──────────────────────────────────────────────────────
@@ -859,9 +871,9 @@ export function SalesView({ activeFeature }) {
   // ── setMenu publish ─────────────────────────────────────────────────────────
   useEffect(() => {
     if (isStandalone()) return;
-    const items = buildSalesMenu(activeView, pipeMode, deals, activeFacet);
+    const items = buildSalesMenu(activeView, pipeMode, deals, activeFacet, operatorId);
     setMenu(items).catch(() => {/* ignore */});
-  }, [activeView, pipeMode, deals, activeFacet]);
+  }, [activeView, pipeMode, deals, activeFacet, operatorId]);
 
   // ── Stage change mutation (kanban drag) ─────────────────────────────────────
   const stageChange = useMutation({

package/dist/lib/operator.js

@@ -0,0 +1,39 @@
+// operator.js — operator-identity helpers shared by every no-build domain app
+// pkg (sales/research/content/strategy). SOURCE OF TRUTH — vendored
+// byte-identically into each pkg's dist/lib/operator.js via vendorRuntime()
+// (add 'operator' to that pkg's build.mjs `files` list). Never hand-edit a
+// vendored copy; edit here and re-run the pkg's `scripts/build.mjs`.
+//
+// hostContext.operator is OPTIONAL (see @ikenga/contract's host-context.ts —
+// IkengaHostContextExtensions.operator: OperatorIdentity | undefined): absent
+// means an UNKNOWN operator, not a default one. Every helper below fails safe
+// on a null/undefined operatorId — "mine" matches nothing, "other owner"
+// defaults to true (never mislabels an unclaimed row as the human), and
+// initials degrade to a neutral glyph rather than guessing an identity.
+//
+// This module is host/bridge-agnostic on purpose (pure functions only, no
+// import from bridge.js): callers thread the plain `operatorId` string (or
+// null) down from app.js's `connectBridge()` ctx.operator.id, the same way
+// `activeFeature` is already threaded from ctx.royaltiSuite.activeFeature.
+
+/** True only when `value` names the CURRENT known operator. An unknown
+ *  operator (`operatorId` null/undefined) never matches — a "mine" filter or
+ *  badge count degrades to empty, never to "everyone" or "no one knows". */
+export function isMine(value, operatorId) {
+  return operatorId != null && value === operatorId;
+}
+
+/** True when `value` names an owner other than the current operator — the
+ *  complement used for "agent-tracked" grouping and is-agent avatar styling.
+ *  An unknown operator makes every named owner read as "other" (we can never
+ *  be sure an unclaimed value is the human), so this defaults to true rather
+ *  than false when `operatorId` is null. */
+export function isOtherOwner(value, operatorId) {
+  return value != null && value !== operatorId;
+}
+
+/** Single-glyph label for an avatar/initial badge. Falls back to '?' rather
+ *  than guessing when the label is missing or empty. */
+export function initialOf(label) {
+  return typeof label === 'string' && label.length > 0 ? label[0].toUpperCase() : '?';
+}

package/manifest.json

@@ -1,7 +1,7 @@
 {
   "id": "com.ikenga.sales",
   "name": "Sales",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "ikenga_api": "1",
   "kind": "embedded",
   "author": { "name": "Royalti", "key": "royalti" },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ikenga/pkg-sales",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Sales — Pipeline, Forecast, Won over the production sales schema. Multi-file iframe pkg; deterministic CSS vendoring build.",
   "license": "Apache-2.0",
   "repository": {