@things-factory/worklist 10.1.4 → 10.1.6

package/dist-client/components/activity-approval-ribon.js

@@ -12,7 +12,22 @@ let ActivityApprovalRibon = class ActivityApprovalRibon extends localize(i18next
     render() {
         const { activityInstance, state } = this.activityApproval?.activityThread || {};
         const { description, assignee, approvalLine, activity } = activityInstance || {};
-        const { startingType } = activity || {};
+        const { startingType, label } = activity || {};
+        /*
+         * What kind of decision this is — the activity's own name, not this instance's.
+         *
+         * Those are different facts and both belong here: 「부적합 처분 승인」 says what kind, and the
+         * headline the activity declares says what this one asks for. The activity's name reached only
+         * the catalog until it became a column on the row, so this screen showed the instance
+         * description alone.
+         *
+         * ── The fallback puts English on a Korean screen ──────────────────────
+         * Activity.description is what a developer wrote for other developers ("The disposition waiting
+         * for approval"), so an activity with no label shows that to the person approving. That is worse
+         * than the instance description and it is the reason to fill label in — it is not a fallback
+         * anybody should be content with.
+         */
+        const kind = label || description || '';
         return html `
       <span class="state">
         <i class="dot"></i>
@@ -34,7 +49,7 @@ let ActivityApprovalRibon = class ActivityApprovalRibon extends localize(i18next
         >info</md-icon
       >
 
-      <span class="desc" title=${description || ''}>${description || ''}</span>
+      <span class="desc" title=${kind}>${kind}</span>
 
 
       ${startingType == 'issue' && assignee

package/dist-client/components/activity-approval-ribon.js.map

@@ -1 +1 @@
-{"version":3,"file":"activity-approval-ribon.js","sourceRoot":"","sources":["../../client/components/activity-approval-ribon.ts"],"names":[],"mappings":";AAAA,OAAO,wDAAwD,CAAA;AAC/D,OAAO,2EAA2E,CAAA;AAClF,OAAO,oDAAoD,CAAA;AAE3D,OAAO,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,KAAK,CAAA;AACtC,OAAO,EAAE,aAAa,EAAE,QAAQ,EAAE,MAAM,mBAAmB,CAAA;AAE3D,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAA;AACjD,OAAO,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAA;AAE3C,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAG5C,IAAM,qBAAqB,GAA3B,MAAM,qBAAsB,SAAQ,QAAQ,CAAC,OAAO,CAAC,CAAC,UAAU,CAAC;aAC/D,WAAM,GAAG,CAAC,gBAAgB,CAAC,AAArB,CAAqB;IAIlC,MAAM;QACJ,MAAM,EAAE,gBAAgB,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC,gBAAgB,EAAE,cAAc,IAAI,EAAE,CAAA;QAC/E,MAAM,EAAE,WAAW,EAAE,QAAQ,EAAE,YAAY,EAAE,QAAQ,EAAE,GAAG,gBAAgB,IAAI,EAAE,CAAA;QAChF,MAAM,EAAE,YAAY,EAAE,GAAG,QAAQ,IAAI,EAAE,CAAA;QAEvC,OAAO,IAAI,CAAA;;;UAGL,OAAO,CAAC,CAAC,CAAC,uBAAuB,GAAG,KAAK,CAAC;;;;;;;gBAOpC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,qBAAqB,CAAC,CAAC;iBACvC,GAAG,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC;mBACxB,CAAC,CAAgB,EAAE,EAAE;YAC9B,IAAI,CAAC,CAAC,GAAG,KAAK,OAAO,IAAI,CAAC,CAAC,GAAG,KAAK,GAAG,EAAE,CAAC;gBACvC,CAAC,CAAC,cAAc,EAAE,CAAA;gBAClB,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAA;YACtB,CAAC;QACH,CAAC;;;;iCAIwB,WAAW,IAAI,EAAE,IAAI,WAAW,IAAI,EAAE;;;QAG/D,YAAY,IAAI,OAAO,IAAI,QAAQ;YACnC,CAAC,CAAC,IAAI,CAAA,6CAA6C,OAAO,CAAC,CAAC,CAAC,gBAAgB,CAAC,UAAU,QAAQ,EAAE,IAAI,SAAS;YAC/G,CAAC,CAAC,EAAE;QACJ,YAAY;YACZ,CAAC,CAAC,IAAI,CAAA;qBACO,YAAY;uBACV,IAAI,CAAC,gBAAgB,CAAC,KAAK;kCAChB;YAC1B,CAAC,CAAC,EAAE;KACP,CAAA;IACH,CAAC;IAED;;;;;;;;;;;;;;;;;OAiBG;IACK,QAAQ,CAAC,KAAa;QAC5B,SAAS,CACP,IAAI,CAAA,yDAAyD,IAAI,CAAC,gBAAgB,UAAU,KAAK,wCAAwC,EACzI,EAAE,KAAK,EAAE,OAAO,CAAC,CAAC,CAAC,qBAAqB,CAAC,EAAE,CAC5C,CAAA;IACH,CAAC;IAED,OAAO,CAAC,OAAO;QACb,IAAI,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC,IAAI,IAAI,CAAC,gBAAgB,EAAE,CAAC;YAC7D,MAAM,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC,gBAAgB,CAAC,cAAc,CAAA;YAEtD,IAAI,CAAC,YAAY,CAAC,OAAO,EAAE,KAAK,CAAC,CAAA;QACnC,CAAC;IACH,CAAC;;AA1E2B;IAA3B,QAAQ,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;;+DAAsB;AAHtC,qBAAqB;IADjC,aAAa,CAAC,yBAAyB,CAAC;GAC5B,qBAAqB,CA8EjC","sourcesContent":["import '@operato/property-editor/ox-properties-dynamic-view.js'\nimport '@things-factory/organization/dist-client/component/approval-line-brief.js'\nimport '../templates/activity-approval-context-template.js'\n\nimport { html, LitElement } from 'lit'\nimport { customElement, property } from 'lit/decorators.js'\n\nimport { i18next, localize } from '@operato/i18n'\nimport { openPopup } from '@operato/layout'\n\nimport { StateRibonStyles } from './state-ribon.js'\n\n@customElement('activity-approval-ribon')\nexport class ActivityApprovalRibon extends localize(i18next)(LitElement) {\n  static styles = [StateRibonStyles]\n\n  @property({ type: Object }) activityApproval: any\n\n  render() {\n    const { activityInstance, state } = this.activityApproval?.activityThread || {}\n    const { description, assignee, approvalLine, activity } = activityInstance || {}\n    const { startingType } = activity || {}\n\n    return html`\n      <span class=\"state\">\n        <i class=\"dot\"></i>\n        ${i18next.t('label.activity-state-' + state)}\n      </span>\n\n      <md-icon\n        class=\"info\"\n        role=\"button\"\n        tabindex=\"0\"\n        title=${String(i18next.t('label.approval-info'))}\n        @click=${() => this.openInfo(state)}\n        @keydown=${(e: KeyboardEvent) => {\n          if (e.key === 'Enter' || e.key === ' ') {\n            e.preventDefault()\n            this.openInfo(state)\n          }\n        }}\n        >info</md-icon\n      >\n\n      <span class=\"desc\" title=${description || ''}>${description || ''}</span>\n\n\n      ${startingType == 'issue' && assignee\n        ? html`<span class=\"who\"><span class=\"who-label\">${i18next.t('field.assignee')}</span>${assignee?.name}</span>`\n        : ''}\n      ${approvalLine\n        ? html`<approval-line-brief\n            .model=${approvalLine}\n            .current=${this.activityApproval.order}\n          ></approval-line-brief>`\n        : ''}\n    `\n  }\n\n  /**\n   * 이 건의 기본 정보 — **팝업으로 연다.**\n   *\n   * ── 왜 aside 가 아닌가 (2026-09-06) ────────────────────────────────────\n   * 전에는 화면 오른쪽에서 나오는 aside 판이었다. 두 가지가 걸린다.\n   *\n   *   자리를 다툰다   화면 오른쪽 세로 판은 operato-twin 의 twin-entity360 이 쓰는 자리다\n   *                  (position: fixed · right: 0 · height: 100vh). 같은 자리에 둘이 선다.\n   *   전역 슬롯이다   VIEWPART_POSITION.ASIDEBAR 은 앱이 하나만 갖는다. 열 개가 넘는 패키지가\n   *                  그 자리를 쓰고 있어(twin 의 AI 협의 도크 · board-ui · help · more-ui …),\n   *                  이 판을 열면 그중 열려 있던 것이 닫힌다.\n   *\n   * 그리고 자리의 뜻이 다르다. 오른쪽 드로어는 **보고 있는 대상을 따라다니는 상세**이고(360 이\n   * 그것이다 — 포커스가 바뀌면 내용이 바뀌고 화면을 옮겨도 남는다), 이 판은 **이 화면의 이 건**에\n   * 대한 것이라 화면을 벗어나면 뜻이 없다. 페이지에 매인 것을 앱 전역 자리에 두지 않는다.\n   *\n   * 내용도 팝업 크기다 — 값 여덟 줄과 접힌 타임라인이라 세로 100vh 를 쓸 일이 없었다.\n   */\n  private openInfo(state: string) {\n    openPopup(\n      html`<activity-approval-context-template .activityApproval=${this.activityApproval} state=${state}></activity-approval-context-template>`,\n      { title: i18next.t('label.approval-info') }\n    )\n  }\n\n  updated(changes) {\n    if (changes.has('activityApproval') && this.activityApproval) {\n      const { state } = this.activityApproval.activityThread\n\n      this.setAttribute('state', state)\n    }\n  }\n}\n"]}
+{"version":3,"file":"activity-approval-ribon.js","sourceRoot":"","sources":["../../client/components/activity-approval-ribon.ts"],"names":[],"mappings":";AAAA,OAAO,wDAAwD,CAAA;AAC/D,OAAO,2EAA2E,CAAA;AAClF,OAAO,oDAAoD,CAAA;AAE3D,OAAO,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,KAAK,CAAA;AACtC,OAAO,EAAE,aAAa,EAAE,QAAQ,EAAE,MAAM,mBAAmB,CAAA;AAE3D,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAA;AACjD,OAAO,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAA;AAE3C,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAG5C,IAAM,qBAAqB,GAA3B,MAAM,qBAAsB,SAAQ,QAAQ,CAAC,OAAO,CAAC,CAAC,UAAU,CAAC;aAC/D,WAAM,GAAG,CAAC,gBAAgB,CAAC,AAArB,CAAqB;IAIlC,MAAM;QACJ,MAAM,EAAE,gBAAgB,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC,gBAAgB,EAAE,cAAc,IAAI,EAAE,CAAA;QAC/E,MAAM,EAAE,WAAW,EAAE,QAAQ,EAAE,YAAY,EAAE,QAAQ,EAAE,GAAG,gBAAgB,IAAI,EAAE,CAAA;QAChF,MAAM,EAAE,YAAY,EAAE,KAAK,EAAE,GAAG,QAAQ,IAAI,EAAE,CAAA;QAE9C;;;;;;;;;;;;;WAaG;QACH,MAAM,IAAI,GAAG,KAAK,IAAI,WAAW,IAAI,EAAE,CAAA;QAEvC,OAAO,IAAI,CAAA;;;UAGL,OAAO,CAAC,CAAC,CAAC,uBAAuB,GAAG,KAAK,CAAC;;;;;;;gBAOpC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,qBAAqB,CAAC,CAAC;iBACvC,GAAG,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC;mBACxB,CAAC,CAAgB,EAAE,EAAE;YAC9B,IAAI,CAAC,CAAC,GAAG,KAAK,OAAO,IAAI,CAAC,CAAC,GAAG,KAAK,GAAG,EAAE,CAAC;gBACvC,CAAC,CAAC,cAAc,EAAE,CAAA;gBAClB,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAA;YACtB,CAAC;QACH,CAAC;;;;iCAIwB,IAAI,IAAI,IAAI;;;QAGrC,YAAY,IAAI,OAAO,IAAI,QAAQ;YACnC,CAAC,CAAC,IAAI,CAAA,6CAA6C,OAAO,CAAC,CAAC,CAAC,gBAAgB,CAAC,UAAU,QAAQ,EAAE,IAAI,SAAS;YAC/G,CAAC,CAAC,EAAE;QACJ,YAAY;YACZ,CAAC,CAAC,IAAI,CAAA;qBACO,YAAY;uBACV,IAAI,CAAC,gBAAgB,CAAC,KAAK;kCAChB;YAC1B,CAAC,CAAC,EAAE;KACP,CAAA;IACH,CAAC;IAED;;;;;;;;;;;;;;;;;OAiBG;IACK,QAAQ,CAAC,KAAa;QAC5B,SAAS,CACP,IAAI,CAAA,yDAAyD,IAAI,CAAC,gBAAgB,UAAU,KAAK,wCAAwC,EACzI,EAAE,KAAK,EAAE,OAAO,CAAC,CAAC,CAAC,qBAAqB,CAAC,EAAE,CAC5C,CAAA;IACH,CAAC;IAED,OAAO,CAAC,OAAO;QACb,IAAI,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC,IAAI,IAAI,CAAC,gBAAgB,EAAE,CAAC;YAC7D,MAAM,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC,gBAAgB,CAAC,cAAc,CAAA;YAEtD,IAAI,CAAC,YAAY,CAAC,OAAO,EAAE,KAAK,CAAC,CAAA;QACnC,CAAC;IACH,CAAC;;AA1F2B;IAA3B,QAAQ,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;;+DAAsB;AAHtC,qBAAqB;IADjC,aAAa,CAAC,yBAAyB,CAAC;GAC5B,qBAAqB,CA8FjC","sourcesContent":["import '@operato/property-editor/ox-properties-dynamic-view.js'\nimport '@things-factory/organization/dist-client/component/approval-line-brief.js'\nimport '../templates/activity-approval-context-template.js'\n\nimport { html, LitElement } from 'lit'\nimport { customElement, property } from 'lit/decorators.js'\n\nimport { i18next, localize } from '@operato/i18n'\nimport { openPopup } from '@operato/layout'\n\nimport { StateRibonStyles } from './state-ribon.js'\n\n@customElement('activity-approval-ribon')\nexport class ActivityApprovalRibon extends localize(i18next)(LitElement) {\n  static styles = [StateRibonStyles]\n\n  @property({ type: Object }) activityApproval: any\n\n  render() {\n    const { activityInstance, state } = this.activityApproval?.activityThread || {}\n    const { description, assignee, approvalLine, activity } = activityInstance || {}\n    const { startingType, label } = activity || {}\n\n    /*\n     * What kind of decision this is — the activity's own name, not this instance's.\n     *\n     * Those are different facts and both belong here: 「부적합 처분 승인」 says what kind, and the\n     * headline the activity declares says what this one asks for. The activity's name reached only\n     * the catalog until it became a column on the row, so this screen showed the instance\n     * description alone.\n     *\n     * ── The fallback puts English on a Korean screen ──────────────────────\n     * Activity.description is what a developer wrote for other developers (\"The disposition waiting\n     * for approval\"), so an activity with no label shows that to the person approving. That is worse\n     * than the instance description and it is the reason to fill label in — it is not a fallback\n     * anybody should be content with.\n     */\n    const kind = label || description || ''\n\n    return html`\n      <span class=\"state\">\n        <i class=\"dot\"></i>\n        ${i18next.t('label.activity-state-' + state)}\n      </span>\n\n      <md-icon\n        class=\"info\"\n        role=\"button\"\n        tabindex=\"0\"\n        title=${String(i18next.t('label.approval-info'))}\n        @click=${() => this.openInfo(state)}\n        @keydown=${(e: KeyboardEvent) => {\n          if (e.key === 'Enter' || e.key === ' ') {\n            e.preventDefault()\n            this.openInfo(state)\n          }\n        }}\n        >info</md-icon\n      >\n\n      <span class=\"desc\" title=${kind}>${kind}</span>\n\n\n      ${startingType == 'issue' && assignee\n        ? html`<span class=\"who\"><span class=\"who-label\">${i18next.t('field.assignee')}</span>${assignee?.name}</span>`\n        : ''}\n      ${approvalLine\n        ? html`<approval-line-brief\n            .model=${approvalLine}\n            .current=${this.activityApproval.order}\n          ></approval-line-brief>`\n        : ''}\n    `\n  }\n\n  /**\n   * 이 건의 기본 정보 — **팝업으로 연다.**\n   *\n   * ── 왜 aside 가 아닌가 (2026-09-06) ────────────────────────────────────\n   * 전에는 화면 오른쪽에서 나오는 aside 판이었다. 두 가지가 걸린다.\n   *\n   *   자리를 다툰다   화면 오른쪽 세로 판은 operato-twin 의 twin-entity360 이 쓰는 자리다\n   *                  (position: fixed · right: 0 · height: 100vh). 같은 자리에 둘이 선다.\n   *   전역 슬롯이다   VIEWPART_POSITION.ASIDEBAR 은 앱이 하나만 갖는다. 열 개가 넘는 패키지가\n   *                  그 자리를 쓰고 있어(twin 의 AI 협의 도크 · board-ui · help · more-ui …),\n   *                  이 판을 열면 그중 열려 있던 것이 닫힌다.\n   *\n   * 그리고 자리의 뜻이 다르다. 오른쪽 드로어는 **보고 있는 대상을 따라다니는 상세**이고(360 이\n   * 그것이다 — 포커스가 바뀌면 내용이 바뀌고 화면을 옮겨도 남는다), 이 판은 **이 화면의 이 건**에\n   * 대한 것이라 화면을 벗어나면 뜻이 없다. 페이지에 매인 것을 앱 전역 자리에 두지 않는다.\n   *\n   * 내용도 팝업 크기다 — 값 여덟 줄과 접힌 타임라인이라 세로 100vh 를 쓸 일이 없었다.\n   */\n  private openInfo(state: string) {\n    openPopup(\n      html`<activity-approval-context-template .activityApproval=${this.activityApproval} state=${state}></activity-approval-context-template>`,\n      { title: i18next.t('label.approval-info') }\n    )\n  }\n\n  updated(changes) {\n    if (changes.has('activityApproval') && this.activityApproval) {\n      const { state } = this.activityApproval.activityThread\n\n      this.setAttribute('state', state)\n    }\n  }\n}\n"]}

package/dist-client/components/decision-slots.d.ts

@@ -0,0 +1,89 @@
+/**
+ * Turns an activity's decisionKeys declaration plus an instance's input into the slots a decision
+ * screen draws.
+ *
+ * ── Nothing is imported ───────────────────────────────────────────────────
+ * No lit, no i18next. The rule is a mapping from a declaration and a payload to an ordered list,
+ * and keeping it free of imports is what lets the test runner hold it: that runner parses
+ * TypeScript by stripping types and reads neither decorators nor enums, so one heavy import takes
+ * a whole test file down with it. Labels and markup are the caller's half.
+ */
+/** The roles that may appear more than once. Every other role names one value. */
+export declare const REPEATABLE_ROLES: readonly ["consequence", "evidence"];
+/** Listed in the order the screen reads them: what is asked, then past, present, future. */
+export declare const DECISION_ROLES: readonly ["headline", "subject", "reversibility", "reason", "evidence", "currentState", "consequence", "policy"];
+export type DecisionKeyDeclaration = {
+    role?: string;
+    inputPath?: string;
+    name?: string;
+    tKey?: string;
+    unit?: string;
+};
+export type DecisionSlot = {
+    role: string;
+    inputPath: string;
+    tKey?: string;
+    unit?: string;
+    /** What the path led to. Undefined both when the path leads nowhere and when the value is unset. */
+    value: unknown;
+    /**
+     * Whether the path led anywhere at all.
+     *
+     * A reason nobody wrote and a reason whose path is wrong look identical in `value`, and the screen
+     * owes different answers: the first is 「사유 없음」 in grey, the second is a declaration that does
+     * not match the payload and must not be drawn as if the person left it blank. Folding them into
+     * one value is the fallback that shifts meaning.
+     */
+    found: boolean;
+};
+/**
+ * Reads a dotted path out of the input.
+ *
+ * Dots only. An array index or a wildcard is not part of the path language — drawing a list is the
+ * slot's job — so 'shift.passes[0]' looks for a key spelled "passes[0]", finds nothing, and reports
+ * found: false. The declaration guard refuses such a path when it arrives, so this is the second
+ * line rather than the first.
+ *
+ * A path that runs into a non-object partway reports found: false rather than throwing. The screen
+ * has to draw something either way, and a thrown error here would take the whole approval down over
+ * one slot.
+ */
+export declare function valueAtPath(input: any, path: string): {
+    value: unknown;
+    found: boolean;
+};
+/**
+ * The slots to draw, in the order they were declared.
+ *
+ * Declaration order is the screen's hierarchy: plant wrote four of these screens by hand and all
+ * four put the headline first, then the subject, the reason, what changes, and why it came up. That
+ * agreement is what makes the order part of the contract rather than a preference, so this function
+ * neither sorts nor groups.
+ *
+ * Every declared key produces a slot, including one whose path leads nowhere. Dropping those would
+ * leave the screen unable to say that a declaration and a payload disagree, and a reader would see
+ * a screen that looks complete.
+ */
+export declare function decisionSlots(decisionKeys: DecisionKeyDeclaration[] | null | undefined, input: any): DecisionSlot[];
+/** The slots for one role, in declared order. Empty when the activity declared none. */
+export declare function slotsOfRole(slots: DecisionSlot[], role: string): DecisionSlot[];
+/**
+ * The single slot for a role that names one value, or undefined.
+ *
+ * Reaching for a second one would be reading a declaration the guard refuses, so this answers the
+ * first and does not quietly merge.
+ */
+export declare function slotOfRole(slots: DecisionSlot[], role: string): DecisionSlot | undefined;
+/**
+ * Whether approving can be undone: true, false, or undefined for "the screen does not know".
+ *
+ * Three answers, not two. A path that led nowhere and a value that is not a boolean both mean the
+ * payload did not say, and the screen must draw nothing then — telling an approver that an action
+ * can be taken back when nobody said so is the one mistake this line cannot make. Reading a
+ * truthy value as reversible would do exactly that, since a non-empty string is truthy.
+ *
+ * This is not rejectionIsFinal. That says whether **rejecting** ends the case for good; this says
+ * whether **approving** can be taken back. The twin actuation carries both, which is why they must
+ * never share a sentence.
+ */
+export declare function reversibilityOf(slot?: DecisionSlot): boolean | undefined;

package/dist-client/components/decision-slots.js

@@ -0,0 +1,111 @@
+/**
+ * Turns an activity's decisionKeys declaration plus an instance's input into the slots a decision
+ * screen draws.
+ *
+ * ── Nothing is imported ───────────────────────────────────────────────────
+ * No lit, no i18next. The rule is a mapping from a declaration and a payload to an ordered list,
+ * and keeping it free of imports is what lets the test runner hold it: that runner parses
+ * TypeScript by stripping types and reads neither decorators nor enums, so one heavy import takes
+ * a whole test file down with it. Labels and markup are the caller's half.
+ */
+/** The roles that may appear more than once. Every other role names one value. */
+export const REPEATABLE_ROLES = ['consequence', 'evidence'];
+/** Listed in the order the screen reads them: what is asked, then past, present, future. */
+export const DECISION_ROLES = [
+    'headline',
+    'subject',
+    'reversibility',
+    'reason',
+    'evidence',
+    'currentState',
+    'consequence',
+    'policy'
+];
+/**
+ * Reads a dotted path out of the input.
+ *
+ * Dots only. An array index or a wildcard is not part of the path language — drawing a list is the
+ * slot's job — so 'shift.passes[0]' looks for a key spelled "passes[0]", finds nothing, and reports
+ * found: false. The declaration guard refuses such a path when it arrives, so this is the second
+ * line rather than the first.
+ *
+ * A path that runs into a non-object partway reports found: false rather than throwing. The screen
+ * has to draw something either way, and a thrown error here would take the whole approval down over
+ * one slot.
+ */
+export function valueAtPath(input, path) {
+    const steps = String(path ?? '')
+        .split('.')
+        .filter(step => step.length > 0);
+    if (!steps.length) {
+        return { value: undefined, found: false };
+    }
+    let at = input;
+    for (const step of steps) {
+        if (at === null || at === undefined || typeof at !== 'object') {
+            return { value: undefined, found: false };
+        }
+        if (!Object.prototype.hasOwnProperty.call(at, step)) {
+            return { value: undefined, found: false };
+        }
+        at = at[step];
+    }
+    return { value: at, found: true };
+}
+/**
+ * The slots to draw, in the order they were declared.
+ *
+ * Declaration order is the screen's hierarchy: plant wrote four of these screens by hand and all
+ * four put the headline first, then the subject, the reason, what changes, and why it came up. That
+ * agreement is what makes the order part of the contract rather than a preference, so this function
+ * neither sorts nor groups.
+ *
+ * Every declared key produces a slot, including one whose path leads nowhere. Dropping those would
+ * leave the screen unable to say that a declaration and a payload disagree, and a reader would see
+ * a screen that looks complete.
+ */
+export function decisionSlots(decisionKeys, input) {
+    if (!decisionKeys?.length) {
+        return [];
+    }
+    return decisionKeys.map(key => {
+        const { value, found } = valueAtPath(input, String(key?.inputPath ?? ''));
+        return {
+            role: String(key?.role ?? ''),
+            inputPath: String(key?.inputPath ?? ''),
+            tKey: key?.tKey,
+            unit: key?.unit,
+            value,
+            found
+        };
+    });
+}
+/** The slots for one role, in declared order. Empty when the activity declared none. */
+export function slotsOfRole(slots, role) {
+    return slots.filter(slot => slot.role === role);
+}
+/**
+ * The single slot for a role that names one value, or undefined.
+ *
+ * Reaching for a second one would be reading a declaration the guard refuses, so this answers the
+ * first and does not quietly merge.
+ */
+export function slotOfRole(slots, role) {
+    return slots.find(slot => slot.role === role);
+}
+/**
+ * Whether approving can be undone: true, false, or undefined for "the screen does not know".
+ *
+ * Three answers, not two. A path that led nowhere and a value that is not a boolean both mean the
+ * payload did not say, and the screen must draw nothing then — telling an approver that an action
+ * can be taken back when nobody said so is the one mistake this line cannot make. Reading a
+ * truthy value as reversible would do exactly that, since a non-empty string is truthy.
+ *
+ * This is not rejectionIsFinal. That says whether **rejecting** ends the case for good; this says
+ * whether **approving** can be taken back. The twin actuation carries both, which is why they must
+ * never share a sentence.
+ */
+export function reversibilityOf(slot) {
+    return typeof slot?.value === 'boolean' ? slot.value : undefined;
+}
+//# sourceMappingURL=decision-slots.js.map

package/dist-client/components/decision-slots.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"decision-slots.js","sourceRoot":"","sources":["../../client/components/decision-slots.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,kFAAkF;AAClF,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,aAAa,EAAE,UAAU,CAAU,CAAA;AAEpE,4FAA4F;AAC5F,MAAM,CAAC,MAAM,cAAc,GAAG;IAC5B,UAAU;IACV,SAAS;IACT,eAAe;IACf,QAAQ;IACR,UAAU;IACV,cAAc;IACd,aAAa;IACb,QAAQ;CACA,CAAA;AA4BV;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,WAAW,CAAC,KAAU,EAAE,IAAY;IAClD,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,IAAI,EAAE,CAAC;SAC7B,KAAK,CAAC,GAAG,CAAC;SACV,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,CAAA;IAElC,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC;QAClB,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,KAAK,EAAE,KAAK,EAAE,CAAA;IAC3C,CAAC;IAED,IAAI,EAAE,GAAQ,KAAK,CAAA;IAEnB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,EAAE,KAAK,IAAI,IAAI,EAAE,KAAK,SAAS,IAAI,OAAO,EAAE,KAAK,QAAQ,EAAE,CAAC;YAC9D,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,KAAK,EAAE,KAAK,EAAE,CAAA;QAC3C,CAAC;QAED,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,cAAc,CAAC,IAAI,CAAC,EAAE,EAAE,IAAI,CAAC,EAAE,CAAC;YACpD,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,KAAK,EAAE,KAAK,EAAE,CAAA;QAC3C,CAAC;QAED,EAAE,GAAG,EAAE,CAAC,IAAI,CAAC,CAAA;IACf,CAAC;IAED,OAAO,EAAE,KAAK,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAA;AACnC,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,aAAa,CAC3B,YAAyD,EACzD,KAAU;IAEV,IAAI,CAAC,YAAY,EAAE,MAAM,EAAE,CAAC;QAC1B,OAAO,EAAE,CAAA;IACX,CAAC;IAED,OAAO,YAAY,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE;QAC5B,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,WAAW,CAAC,KAAK,EAAE,MAAM,CAAC,GAAG,EAAE,SAAS,IAAI,EAAE,CAAC,CAAC,CAAA;QAEzE,OAAO;YACL,IAAI,EAAE,MAAM,CAAC,GAAG,EAAE,IAAI,IAAI,EAAE,CAAC;YAC7B,SAAS,EAAE,MAAM,CAAC,GAAG,EAAE,SAAS,IAAI,EAAE,CAAC;YACvC,IAAI,EAAE,GAAG,EAAE,IAAI;YACf,IAAI,EAAE,GAAG,EAAE,IAAI;YACf,KAAK;YACL,KAAK;SACN,CAAA;IACH,CAAC,CAAC,CAAA;AACJ,CAAC;AAED,wFAAwF;AACxF,MAAM,UAAU,WAAW,CAAC,KAAqB,EAAE,IAAY;IAC7D,OAAO,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,IAAI,KAAK,IAAI,CAAC,CAAA;AACjD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,KAAqB,EAAE,IAAY;IAC5D,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,IAAI,KAAK,IAAI,CAAC,CAAA;AAC/C,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,eAAe,CAAC,IAAmB;IACjD,OAAO,OAAO,IAAI,EAAE,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAA;AAClE,CAAC","sourcesContent":["/**\n * Turns an activity's decisionKeys declaration plus an instance's input into the slots a decision\n * screen draws.\n *\n * ── Nothing is imported ───────────────────────────────────────────────────\n * No lit, no i18next. The rule is a mapping from a declaration and a payload to an ordered list,\n * and keeping it free of imports is what lets the test runner hold it: that runner parses\n * TypeScript by stripping types and reads neither decorators nor enums, so one heavy import takes\n * a whole test file down with it. Labels and markup are the caller's half.\n */\n\n/** The roles that may appear more than once. Every other role names one value. */\nexport const REPEATABLE_ROLES = ['consequence', 'evidence'] as const\n\n/** Listed in the order the screen reads them: what is asked, then past, present, future. */\nexport const DECISION_ROLES = [\n  'headline',\n  'subject',\n  'reversibility',\n  'reason',\n  'evidence',\n  'currentState',\n  'consequence',\n  'policy'\n] as const\n\nexport type DecisionKeyDeclaration = {\n  role?: string\n  inputPath?: string\n  name?: string\n  tKey?: string\n  unit?: string\n}\n\nexport type DecisionSlot = {\n  role: string\n  inputPath: string\n  tKey?: string\n  unit?: string\n  /** What the path led to. Undefined both when the path leads nowhere and when the value is unset. */\n  value: unknown\n  /**\n   * Whether the path led anywhere at all.\n   *\n   * A reason nobody wrote and a reason whose path is wrong look identical in `value`, and the screen\n   * owes different answers: the first is 「사유 없음」 in grey, the second is a declaration that does\n   * not match the payload and must not be drawn as if the person left it blank. Folding them into\n   * one value is the fallback that shifts meaning.\n   */\n  found: boolean\n}\n\n/**\n * Reads a dotted path out of the input.\n *\n * Dots only. An array index or a wildcard is not part of the path language — drawing a list is the\n * slot's job — so 'shift.passes[0]' looks for a key spelled \"passes[0]\", finds nothing, and reports\n * found: false. The declaration guard refuses such a path when it arrives, so this is the second\n * line rather than the first.\n *\n * A path that runs into a non-object partway reports found: false rather than throwing. The screen\n * has to draw something either way, and a thrown error here would take the whole approval down over\n * one slot.\n */\nexport function valueAtPath(input: any, path: string): { value: unknown; found: boolean } {\n  const steps = String(path ?? '')\n    .split('.')\n    .filter(step => step.length > 0)\n\n  if (!steps.length) {\n    return { value: undefined, found: false }\n  }\n\n  let at: any = input\n\n  for (const step of steps) {\n    if (at === null || at === undefined || typeof at !== 'object') {\n      return { value: undefined, found: false }\n    }\n\n    if (!Object.prototype.hasOwnProperty.call(at, step)) {\n      return { value: undefined, found: false }\n    }\n\n    at = at[step]\n  }\n\n  return { value: at, found: true }\n}\n\n/**\n * The slots to draw, in the order they were declared.\n *\n * Declaration order is the screen's hierarchy: plant wrote four of these screens by hand and all\n * four put the headline first, then the subject, the reason, what changes, and why it came up. That\n * agreement is what makes the order part of the contract rather than a preference, so this function\n * neither sorts nor groups.\n *\n * Every declared key produces a slot, including one whose path leads nowhere. Dropping those would\n * leave the screen unable to say that a declaration and a payload disagree, and a reader would see\n * a screen that looks complete.\n */\nexport function decisionSlots(\n  decisionKeys: DecisionKeyDeclaration[] | null | undefined,\n  input: any\n): DecisionSlot[] {\n  if (!decisionKeys?.length) {\n    return []\n  }\n\n  return decisionKeys.map(key => {\n    const { value, found } = valueAtPath(input, String(key?.inputPath ?? ''))\n\n    return {\n      role: String(key?.role ?? ''),\n      inputPath: String(key?.inputPath ?? ''),\n      tKey: key?.tKey,\n      unit: key?.unit,\n      value,\n      found\n    }\n  })\n}\n\n/** The slots for one role, in declared order. Empty when the activity declared none. */\nexport function slotsOfRole(slots: DecisionSlot[], role: string): DecisionSlot[] {\n  return slots.filter(slot => slot.role === role)\n}\n\n/**\n * The single slot for a role that names one value, or undefined.\n *\n * Reaching for a second one would be reading a declaration the guard refuses, so this answers the\n * first and does not quietly merge.\n */\nexport function slotOfRole(slots: DecisionSlot[], role: string): DecisionSlot | undefined {\n  return slots.find(slot => slot.role === role)\n}\n\n/**\n * Whether approving can be undone: true, false, or undefined for \"the screen does not know\".\n *\n * Three answers, not two. A path that led nowhere and a value that is not a boolean both mean the\n * payload did not say, and the screen must draw nothing then — telling an approver that an action\n * can be taken back when nobody said so is the one mistake this line cannot make. Reading a\n * truthy value as reversible would do exactly that, since a non-empty string is truthy.\n *\n * This is not rejectionIsFinal. That says whether **rejecting** ends the case for good; this says\n * whether **approving** can be taken back. The twin actuation carries both, which is why they must\n * never share a sentence.\n */\nexport function reversibilityOf(slot?: DecisionSlot): boolean | undefined {\n  return typeof slot?.value === 'boolean' ? slot.value : undefined\n}\n"]}

package/dist-client/pages/activity-approval/activity-approval-page.d.ts

@@ -1,3 +1,4 @@
+import '@material/web/icon/icon.js';
 import '@operato/property-editor/ox-properties-dynamic-view.js';
 import '@operato/board/ox-board-viewer.js';
 import '../../components/activity-approval-ribon.js';
@@ -30,6 +31,20 @@ export declare class ActivityApprovalPage extends ActivityApprovalPage_base {
     activityThread: any;
     board: any;
     commentInput: HTMLTextAreaElement;
+    /**
+     * Is the reader seated here as a reference — someone who comments and does not decide?
+     *
+     * The rule comes from organization's client types, the same function its own approval-line editor
+     * and list use, so the two wire spellings of the enum are handled in one place. The approval line
+     * answers rather than ActivityApproval.participation, matching what the server refuses on
+     * (isReferenceOrder in approval-line.ts): the line decides who comes next, so letting the row
+     * decide who may judge would leave two authorities with nothing to settle a disagreement.
+     *
+     * The value comes back false while the query is still out. That is the same answer as for an
+     * approver, so the buttons show for a moment on a reference's screen — pressing one is refused by
+     * the server, which is why the server closed first.
+     */
+    get isReference(): boolean;
     get context(): {
         title: any;
         help: string;
@@ -72,8 +87,65 @@ export declare class ActivityApprovalPage extends ActivityApprovalPage_base {
      * approveActivityApproval / rejectActivityApproval 은 id 와 comment 만 보낸다 — 그 값은 어디에도
      * 가지 않는다. 결재자가 적을 수 있는 것은 의견 하나이고, 그것은 화면 아래 칸이 받는다.
      */
+    /**
+     * The decision the activity declared, or nothing when it declared none.
+     *
+     * Returning nothing is how generatedContent knows to fall back to the model. An activity with no
+     * decisionKeys draws exactly what it drew before.
+     *
+     * A slot whose path led nowhere is skipped rather than drawn empty. Older instances of an activity
+     * predate a field the declaration names — plant hit that with its `policy` value — and an empty
+     * bar on the screen would read as "there is no reason" when the truth is that this record has no
+     * such value. What cannot be shown is left out; what is shown is only what the payload actually
+     * carries.
+     *
+     * `reason` is the exception. Missing and blank mean the same thing to whoever is reading — nobody
+     * wrote one — so both draw the grey line rather than disappearing. A reason that silently vanishes
+     * would let an approver assume it was never asked for.
+     */
+    decisionContent(): DirectiveResult<{
+        new (_partInfo: import("lit-html/directive.js").PartInfo): import("lit-html/directives/keyed.js").Keyed<TemplateResult<1>>;
+    }>;
+    /**
+     * Whether approving can be undone, drawn as the frame around the decision.
+     *
+     * Only an explicit true or false draws. Anything else — the path led nowhere, or the value is not
+     * a boolean — draws nothing, because saying "this can be undone" on a screen that does not know is
+     * the one mistake this line must never make.
+     *
+     * This is not rejectionIsFinal. That one says whether **rejecting** ends the case for good; this
+     * says whether **approving** can be taken back. The twin actuation carries both, which is exactly
+     * why they must not share a sentence.
+     */
+    private renderReversibility;
+    /**
+     * One slot's value as something to read.
+     *
+     * A scalar prints with its unit when one was declared. Anything shaped — an object, a list — goes
+     * through fieldValue, the same laying-out the model sections use, so a payload reads the same way
+     * wherever it appears and a four-level object does not become one long line.
+     */
+    private slotText;
+    /** Did a person actually write something here. Whitespace is not words. */
+    private hasWords;
     generatedContent(): DirectiveResult;
     boardContent(): DirectiveResult;
+    /**
+     * The element the activity names, or a plain statement that it could not be loaded.
+     *
+     * document.createElement answers something for any name, so an activity pointing at an element
+     * nobody registered used to produce an empty card. The approver got a screen with nothing on it
+     * and the approve button still worked — plant shipped two such forms in one day and found them by
+     * reading the file list, not by opening the screen.
+     *
+     * The reader is told what to judge by instead. Same shape as the empty generated screen: what is
+     * missing, and where to look — the submission reason and the history in the timeline below.
+     *
+     * ── Waiting for a late definition ─────────────────────────────────────────
+     * An element can be registered after this page renders, and answering "not loaded" forever in
+     * that case would be a screen inventing a defect. whenDefined resolves whenever it arrives and
+     * the page redraws; a name nobody ever registers simply never resolves, and the statement stands.
+     */
     customElementContent(): any;
     oxBoardViewer: any;
     getOutputFromBoard(): any;