@things-factory/worklist 10.1.168 → 10.1.169

package/dist-client/my-work-menus.js.map

@@ -1 +1 @@
-{"version":3,"file":"my-work-menus.js","sourceRoot":"","sources":["../client/my-work-menus.ts"],"names":[],"mappings":"AA2BA,MAAM,CAAC,MAAM,aAAa,GAAG;IAC3B,2EAA2E;IAC3E,SAAS,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,cAAc,EAAE,KAAK,EAAE,gBAAgB,EAAE;IAC/F,IAAI,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,iBAAiB,EAAE,KAAK,EAAE,eAAe,EAAE;IACnG,kBAAkB,EAAE;QAClB,IAAI,EAAE,uBAAuB;QAC7B,IAAI,EAAE,UAAU;QAChB,KAAK,EAAE,6BAA6B;QACpC,KAAK,EAAE,0BAA0B;KAClC;IACD,KAAK,EAAE,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,2BAA2B,EAAE,KAAK,EAAE,kBAAkB,EAAE,KAAK,EAAE,gBAAgB,EAAE;IACpH,IAAI,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,iBAAiB,EAAE;IACvE,eAAe,EAAE,EAAE,IAAI,EAAE,oBAAoB,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,0BAA0B,EAAE;IACnG,kBAAkB,EAAE,EAAE,IAAI,EAAE,oBAAoB,EAAE,IAAI,EAAE,gBAAgB,EAAE,KAAK,EAAE,wBAAwB,EAAE;IAC3G,QAAQ,EAAE,EAAE,IAAI,EAAE,eAAe,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,yBAAyB,EAAE,KAAK,EAAE,eAAe,EAAE;IAC3G,OAAO,EAAE,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,eAAe,EAAE,KAAK,EAAE,6BAA6B,EAAE;IAC9F,QAAQ,EAAE,EAAE,IAAI,EAAE,eAAe,EAAE,IAAI,EAAE,sBAAsB,EAAE,KAAK,EAAE,8BAA8B,EAAE;IACxG,mBAAmB,EAAE,EAAE,IAAI,EAAE,wBAAwB,EAAE,IAAI,EAAE,gBAAgB,EAAE,KAAK,EAAE,8BAA8B,EAAE;IACtH,kBAAkB,EAAE,EAAE,IAAI,EAAE,kBAAkB,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,wBAAwB,EAAE;IACpG,YAAY,EAAE,EAAE,IAAI,EAAE,iBAAiB,EAAE,IAAI,EAAE,YAAY,EAAE,KAAK,EAAE,iBAAiB,EAAE;CAC1C,CAAA","sourcesContent":["/**\n * The menu entries for worklist's own screens — **which address, which icon, which label, which count.**\n *\n * Every product puts \"my work\" in its menu, and each used to write these rows itself: the same path,\n * icon, label key and badge name, copied and drifting (one app had a different to-do icon, one had its\n * own English labels). The screens are worklist's, so the facts about them are too. A product picks the\n * entries it shows, in its own order, and keeps its own menu shape:\n *\n * ```ts\n * const MY_WORK: MyWorkMenuName[] = ['todo', 'approval-pending', 'draft', 'done']\n * MY_WORK.map(name => MY_WORK_MENUS[name]).map(({ path, icon, label, badge }) => ({ path, icon, name: i18next.t(label), … }))\n * ```\n *\n * `label` is a translation key that worklist carries in every language. `badge` is a name for\n * `generateActivitySummary`; how a product draws the count stays with the product.\n */\nexport interface MyWorkMenu {\n  /** Address of the worklist screen. */\n  path: string\n  /** Material Symbols name. */\n  icon: string\n  /** Translation key in worklist's translations. */\n  label: string\n  /** `generateActivitySummary` name, for entries that count something. */\n  badge?: 'numberOfToDos' | 'numberOfApprovalPendings' | 'numberOfDrafts' | 'numberOfPicks' | 'numberOfMyWork'\n}\n\nexport const MY_WORK_MENUS = {\n  /** The group row: everything that has come to me. Opens the to-do list. */\n  'my-work': { path: 'todo-list', icon: 'inbox', label: 'menu.my-work', badge: 'numberOfMyWork' },\n  todo: { path: 'todo-list', icon: 'things_to_do', label: 'title.todo list', badge: 'numberOfToDos' },\n  'approval-pending': {\n    path: 'approval-pending-list',\n    icon: 'approval',\n    label: 'title.approval pending list',\n    badge: 'numberOfApprovalPendings'\n  },\n  draft: { path: 'draft-list', icon: 'drive_file_rename_outline', label: 'title.draft list', badge: 'numberOfDrafts' },\n  done: { path: 'done-list', icon: 'task_alt', label: 'title.done list' },\n  'approval-done': { path: 'approval-done-list', icon: 'history', label: 'title.approval done list' },\n  'my-work-calendar': { path: 'done-list-calendar', icon: 'calendar_month', label: 'title.my work calendar' },\n  pickable: { path: 'pickable-list', icon: 'shop', label: 'title.activity shopping', badge: 'numberOfPicks' },\n  starter: { path: 'starter-list', icon: 'rocket_launch', label: 'title.activity starter list' },\n  reporter: { path: 'reporter-list', icon: 'content_paste_search', label: 'title.activity reporter list' },\n  'activity-instance': { path: 'activity-instance-list', icon: 'list_alt_check', label: 'title.activity instance list' },\n  'activity-catalog': { path: 'activity-catalog', icon: 'menu_book', label: 'title.activity catalog' },\n  'work-doors': { path: 'work-doors-page', icon: 'how_to_reg', label: 'menu.work-doors' }\n} as const satisfies Record<string, MyWorkMenu>\n\nexport type MyWorkMenuName = keyof typeof MY_WORK_MENUS\n"]}
+{"version":3,"file":"my-work-menus.js","sourceRoot":"","sources":["../client/my-work-menus.ts"],"names":[],"mappings":"AA2BA,MAAM,CAAC,MAAM,aAAa,GAAG;IAC3B,2EAA2E;IAC3E,SAAS,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,cAAc,EAAE,KAAK,EAAE,gBAAgB,EAAE;IAC/F,IAAI,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,iBAAiB,EAAE,KAAK,EAAE,eAAe,EAAE;IACnG,kBAAkB,EAAE;QAClB,IAAI,EAAE,uBAAuB;QAC7B,IAAI,EAAE,UAAU;QAChB,KAAK,EAAE,6BAA6B;QACpC,KAAK,EAAE,0BAA0B;KAClC;IACD,KAAK,EAAE,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,2BAA2B,EAAE,KAAK,EAAE,kBAAkB,EAAE,KAAK,EAAE,gBAAgB,EAAE;IACpH,eAAe,EAAE,EAAE,IAAI,EAAE,oBAAoB,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,0BAA0B,EAAE;IACnG,kBAAkB,EAAE,EAAE,IAAI,EAAE,oBAAoB,EAAE,IAAI,EAAE,gBAAgB,EAAE,KAAK,EAAE,wBAAwB,EAAE;IAC3G,QAAQ,EAAE,EAAE,IAAI,EAAE,eAAe,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,yBAAyB,EAAE,KAAK,EAAE,eAAe,EAAE;IAC3G,OAAO,EAAE,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,eAAe,EAAE,KAAK,EAAE,6BAA6B,EAAE;IAC9F,QAAQ,EAAE,EAAE,IAAI,EAAE,eAAe,EAAE,IAAI,EAAE,sBAAsB,EAAE,KAAK,EAAE,8BAA8B,EAAE;IACxG,mBAAmB,EAAE,EAAE,IAAI,EAAE,wBAAwB,EAAE,IAAI,EAAE,gBAAgB,EAAE,KAAK,EAAE,8BAA8B,EAAE;IACtH,kBAAkB,EAAE,EAAE,IAAI,EAAE,kBAAkB,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,wBAAwB,EAAE;IACpG,YAAY,EAAE,EAAE,IAAI,EAAE,iBAAiB,EAAE,IAAI,EAAE,YAAY,EAAE,KAAK,EAAE,iBAAiB,EAAE;CAC1C,CAAA","sourcesContent":["/**\n * The menu entries for worklist's own screens — **which address, which icon, which label, which count.**\n *\n * Every product puts \"my work\" in its menu, and each used to write these rows itself: the same path,\n * icon, label key and badge name, copied and drifting (one app had a different to-do icon, one had its\n * own English labels). The screens are worklist's, so the facts about them are too. A product picks the\n * entries it shows, in its own order, and keeps its own menu shape:\n *\n * ```ts\n * const MY_WORK: MyWorkMenuName[] = ['todo', 'approval-pending', 'draft']\n * MY_WORK.map(name => MY_WORK_MENUS[name]).map(({ path, icon, label, badge }) => ({ path, icon, name: i18next.t(label), … }))\n * ```\n *\n * `label` is a translation key that worklist carries in every language. `badge` is a name for\n * `generateActivitySummary`; how a product draws the count stays with the product.\n */\nexport interface MyWorkMenu {\n  /** Address of the worklist screen. */\n  path: string\n  /** Material Symbols name. */\n  icon: string\n  /** Translation key in worklist's translations. */\n  label: string\n  /** `generateActivitySummary` name, for entries that count something. */\n  badge?: 'numberOfToDos' | 'numberOfApprovalPendings' | 'numberOfDrafts' | 'numberOfPicks' | 'numberOfMyWork'\n}\n\nexport const MY_WORK_MENUS = {\n  /** The group row: everything that has come to me. Opens the to-do list. */\n  'my-work': { path: 'todo-list', icon: 'inbox', label: 'menu.my-work', badge: 'numberOfMyWork' },\n  todo: { path: 'todo-list', icon: 'things_to_do', label: 'title.todo list', badge: 'numberOfToDos' },\n  'approval-pending': {\n    path: 'approval-pending-list',\n    icon: 'approval',\n    label: 'title.approval pending list',\n    badge: 'numberOfApprovalPendings'\n  },\n  draft: { path: 'draft-list', icon: 'drive_file_rename_outline', label: 'title.draft list', badge: 'numberOfDrafts' },\n  'approval-done': { path: 'approval-done-list', icon: 'history', label: 'title.approval done list' },\n  'my-work-calendar': { path: 'done-list-calendar', icon: 'calendar_month', label: 'title.my work calendar' },\n  pickable: { path: 'pickable-list', icon: 'shop', label: 'title.activity shopping', badge: 'numberOfPicks' },\n  starter: { path: 'starter-list', icon: 'rocket_launch', label: 'title.activity starter list' },\n  reporter: { path: 'reporter-list', icon: 'content_paste_search', label: 'title.activity reporter list' },\n  'activity-instance': { path: 'activity-instance-list', icon: 'list_alt_check', label: 'title.activity instance list' },\n  'activity-catalog': { path: 'activity-catalog', icon: 'menu_book', label: 'title.activity catalog' },\n  'work-doors': { path: 'work-doors-page', icon: 'how_to_reg', label: 'menu.work-doors' }\n} as const satisfies Record<string, MyWorkMenu>\n\nexport type MyWorkMenuName = keyof typeof MY_WORK_MENUS\n"]}

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

@@ -105,60 +105,10 @@ 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 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.
-     */
+    /** The decision the activity declared, or nothing - drawn by `renderDecision`, the one drawing both screens use. */
     decisionContent(): DirectiveResult<{
-        new (_partInfo: import("lit-html/directive.js").PartInfo): import("lit-html/directives/keyed.js").Keyed<TemplateResult<1>>;
+        new (_partInfo: import("lit-html/directive.js").PartInfo): import("lit-html/directives/keyed.js").Keyed<TemplateResult>;
     }>;
-    /**
-     * 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.
-     *
-     * ── The sentence is built here, not frozen when the request was raised ──
-     * Eighteen doors write a Korean sentence at issue and the issue snapshots it, so it stands in the
-     * language of whoever raised it and an approver reading another language never catches up. Where
-     * a door declares `valueTKey`, the sentence is made at reading time instead, out of the values
-     * the request did freeze.
-     *
-     * **A door that has not moved yet draws exactly what it drew before.** That is what lets plant
-     * move them one at a time instead of eighteen at once, and it covers a door that moved into a
-     * locale nobody has written yet: an empty `defaultValue` means a missing key falls through to the
-     * frozen sentence rather than printing `approval.…` on the card.
-     */
-    private slotText;
-    /** The sentence this slot declares, when it declares one and the locale has it. */
-    private sentenceOf;
-    /** Did a person actually write something here. Whitespace is not words. */
-    private hasWords;
     generatedContent(): DirectiveResult;
     boardContent(): DirectiveResult;
     /**

package/dist-client/pages/activity-approval/activity-approval-page.js

@@ -19,8 +19,9 @@ import { PageView } from '@operato/shell';
 import { DocumentPageStyles, ScrollbarStyles } from '@operato/styles';
 import { provider } from '@operato/board/board-provider.js';
 import { isReferenceStep } from '@things-factory/organization/dist-client';
-import { ActivityFieldsStyles, declaredLabel, fieldValue, inputSpecOf, outputSpecOf, renderGiven } from '../../components/activity-fields.js';
-import { decisionSlots, reversibilityIsMalformed, reversibilityOf, COMPENSABLE, slotOfRole, slotsOfRole } from '../../components/decision-slots.js';
+import { ActivityFieldsStyles, inputSpecOf, outputSpecOf, renderGiven } from '../../components/activity-fields.js';
+import { DecisionContentStyles, DecisionKeysFields, renderDecision } from '../../components/decision-content.js';
+import { decisionSlots, slotOfRole } from '../../components/decision-slots.js';
 const ActivityApprovalFetchResult = `\
 {   id
   judgment
@@ -66,11 +67,7 @@ const ActivityApprovalFetchResult = `\
       activity {         id
         label
         labelTKey
-        decisionKeys {           role
-          inputPath
-          tKey
-          unit
- }
+        ${DecisionKeysFields}
         model {           name
           description
           active
@@ -144,6 +141,7 @@ let ActivityApprovalPage = class ActivityApprovalPage extends localize(i18next)(
         DocumentPageStyles,
         ScrollbarStyles,
         ActivityFieldsStyles,
+        DecisionContentStyles,
         css `
       :host {
         color: var(--ui-ink, #17242c);
@@ -305,126 +303,6 @@ let ActivityApprovalPage = class ActivityApprovalPage extends localize(i18next)(
         color: var(--ui-faint, #8a959c);
         word-break: keep-all;
       }
-
-      /*
-       * The decision, drawn from the activity's declaration.
-       *
-       * The order is the declaration's and the weight descends with it: what is being asked reads
-       * first at title size, then what it is about, then the words a person wrote, then what changes
-       * on approval, and last why it came up at all.
-       */
-      section[decision] .headline {
-        font-size: var(--ui-t-headline, 19px);
-        font-weight: 600;
-        line-height: 1.35;
-        color: var(--ui-ink, #17242c);
-        word-break: keep-all;
-      }
-
-      section[decision] .subject {
-        margin-top: 3px;
-        font-family: var(--ui-mono, ui-monospace, Menlo, monospace);
-        font-size: var(--ui-t-body, 13px);
-        color: var(--ui-muted, #57676f);
-      }
-
-      section[decision] .block {
-        margin-top: 14px;
-      }
-
-      section[decision] .block > .label {
-        margin-bottom: 4px;
-        font-size: var(--ui-t-small, 11.5px);
-        color: var(--ui-muted, #57676f);
-      }
-
-      /* What a person wrote is set apart, because it is the only part nobody computed. */
-      section[decision] .said {
-        padding: 9px 11px;
-        border-left: 3px solid var(--ui-line, #d5dce0);
-        background: var(--ui-line-soft, #f2f5f7);
-        color: var(--ui-ink, #17242c);
-        word-break: break-word;
-      }
-
-      section[decision] .none {
-        color: var(--ui-faint, #8a959c);
-      }
-
-      section[decision] .consequence {
-        margin-top: 8px;
-        padding: 9px 11px;
-        border: 1px solid var(--ui-line, #d5dce0);
-        border-radius: var(--ui-r-small, 6px);
-      }
-
-      section[decision] .consequence > .label {
-        margin-bottom: 3px;
-        font-size: var(--ui-t-small, 11.5px);
-        color: var(--ui-muted, #57676f);
-      }
-
-      /*
-       * Whether approving can be undone reads as the frame around the decision, not as one of its
-       * consequences. Among them it carries the same weight as "four orders get pushed back", and
-       * on an approval that moves equipment it is the heaviest line on the screen.
-       */
-      section[decision] .reversibility {
-        display: flex;
-        align-items: center;
-        gap: 6px;
-
-        margin-top: 12px;
-        padding: 8px 11px;
-        border-radius: var(--ui-r-small, 6px);
-
-        background: var(--ui-line-soft, #f2f5f7);
-        color: var(--ui-muted, #57676f);
-        font-size: var(--ui-t-small, 11.5px);
-      }
-
-      section[decision] .reversibility[irreversible] {
-        background: var(--md-sys-color-error-container, #ffdad6);
-        color: var(--md-sys-color-on-error-container, #410002);
-        font-weight: 600;
-      }
-
-      section[decision] .reversibility md-icon {
-        font-size: 1.15em;
-      }
-
-      /*
-       * What the decision was made from. Set flatter than the consequences — it is the record the
-       * approver checks, not something the approval brings about.
-       */
-      section[decision] .evidence {
-        margin-top: 8px;
-        padding: 7px 0 7px 11px;
-        border-left: 2px solid var(--ui-line, #d5dce0);
-      }
-
-      section[decision] .evidence > .label {
-        margin-bottom: 2px;
-        font-size: var(--ui-t-small, 11.5px);
-        color: var(--ui-muted, #57676f);
-      }
-
-      /*
-       * What is true right now sits apart from what the approval would change. Sharing the box the
-       * consequences use would say the approval causes these readings.
-       */
-      section[decision] .current-state {
-        margin-top: 14px;
-        padding: 9px 11px;
-        border: 1px dashed var(--ui-line, #d5dce0);
-        border-radius: var(--ui-r-small, 6px);
-      }
-
-      section[decision] .current-state > .label {
-        margin-bottom: 3px;
-        font-size: var(--ui-t-small, 11.5px);
-        color: var(--ui-muted, #57676f);
-      }
     `
     ]; }
     /**
@@ -682,179 +560,11 @@ let ActivityApprovalPage = class ActivityApprovalPage extends localize(i18next)(
      * 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 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.
-     */
+    /** The decision the activity declared, or nothing - drawn by `renderDecision`, the one drawing both screens use. */
     decisionContent() {
         const { activityInstance } = this.activityThread || {};
-        const slots = decisionSlots(activityInstance?.activity?.decisionKeys, activityInstance?.input);
-        if (!slots.length) {
-            return nothing;
-        }
-        const headline = slotOfRole(slots, 'headline');
-        const subject = slotOfRole(slots, 'subject');
-        const reason = slotOfRole(slots, 'reason');
-        const consequences = slotsOfRole(slots, 'consequence').filter(slot => slot.found);
-        const evidence = slotsOfRole(slots, 'evidence').filter(slot => slot.found);
-        const currentState = slotOfRole(slots, 'currentState');
-        const reversibility = slotOfRole(slots, 'reversibility');
-        const drawable = [headline, subject, currentState].some(slot => slot?.found) ||
-            !!reason ||
-            consequences.length ||
-            evidence.length ||
-            reversibilityOf(reversibility) !== undefined;
-        /*
-         * Every declared slot missed this payload. Saying so is not this screen's job — falling back to
-         * the model at least shows the values that are there, and the reader is no worse off than before
-         * the declaration existed.
-         */
-        if (!drawable) {
-            return nothing;
-        }
-        return keyed(this.activityApproval.id, html `
-        <section decision>
-          ${headline?.found ? html `<div class="headline">${this.slotText(headline)}</div>` : ''}
-          ${subject?.found ? html `<div class="subject">${this.slotText(subject)}</div>` : ''}
-          ${reason
-            ? html `
-                  <div class="block">
-                    <div class="label">${i18next.t('field.reason')}</div>
-                    <div class="said ${this.hasWords(reason) ? '' : 'none'}">
-                      ${this.hasWords(reason) ? this.slotText(reason) : i18next.t('text.no-reason-given')}
-                    </div>
-                  </div>
-                `
-            : ''}
-          ${this.renderReversibility(reversibility)}
-          ${evidence.length
-            ? html `
-                  <div class="block">
-                    <div class="label">${i18next.t('label.what-it-was-judged-from')}</div>
-                    ${evidence.map(slot => html `
-                        <div class="evidence">
-                          ${declaredLabel(slot) ? html `<div class="label">${declaredLabel(slot)}</div>` : ''}
-                          <div>${this.slotText(slot)}</div>
-                        </div>
-                      `)}
-                  </div>
-                `
-            : ''}
-          ${currentState?.found
-            ? html `
-                  <div class="current-state">
-                    <div class="label">${declaredLabel(currentState) || i18next.t('label.current-state')}</div>
-                    <div>${this.slotText(currentState)}</div>
-                  </div>
-                `
-            : ''}
-          ${consequences.length
-            ? html `
-                  <div class="block">
-                    <div class="label">${i18next.t('label.what-changes')}</div>
-                    ${consequences.map(slot => html `
-                        <div class="consequence">
-                          ${declaredLabel(slot) ? html `<div class="label">${declaredLabel(slot)}</div>` : ''}
-                          <div>${this.slotText(slot)}</div>
-                        </div>
-                      `)}
-                  </div>
-                `
-            : ''}
-        </section>
-      `);
-    }
-    /**
-     * 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.
-     */
-    renderReversibility(slot) {
-        const said = reversibilityOf(slot);
-        if (!said) {
-            if (reversibilityIsMalformed(slot)) {
-                /*
-                 * Declared, and the value is there, but not the shape this role takes — a bare sentence
-                 * where { reversible, caveat } belongs. Nothing is drawn, because this side cannot tell
-                 * which way it goes; the log is so a developer does not have to find it by hand, which is
-                 * how it was found the first time.
-                 */
-                console.error(`[worklist] reversibility at "${slot?.inputPath}" is not { reversible, caveat } — nothing drawn`);
-            }
-            return '';
-        }
-        /*
-         * The module's own sentence when it wrote one. 「버린 자재는 돌아오지 않습니다」 says why, which
-         * the plain wording cannot; the boolean beside it is what decides how heavily this is drawn.
-         */
-        return html `
-      <div class="reversibility" ?irreversible=${!said.reversible}>
-        <md-icon>${said.kind === COMPENSABLE ? 'history' : said.reversible ? 'undo' : 'lock'}</md-icon>
-        <span
-          >${said.caveat || (said.kind === COMPENSABLE
-            ? i18next.t('text.approval-is-compensable', { defaultValue: 'The original decision remains. A subsequent corrective action is required.' })
-            : i18next.t(said.reversible ? 'text.approval-is-reversible' : 'text.approval-is-irreversible'))}</span
-        >
-      </div>
-    `;
-    }
-    /**
-     * 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.
-     *
-     * ── The sentence is built here, not frozen when the request was raised ──
-     * Eighteen doors write a Korean sentence at issue and the issue snapshots it, so it stands in the
-     * language of whoever raised it and an approver reading another language never catches up. Where
-     * a door declares `valueTKey`, the sentence is made at reading time instead, out of the values
-     * the request did freeze.
-     *
-     * **A door that has not moved yet draws exactly what it drew before.** That is what lets plant
-     * move them one at a time instead of eighteen at once, and it covers a door that moved into a
-     * locale nobody has written yet: an empty `defaultValue` means a missing key falls through to the
-     * frozen sentence rather than printing `approval.…` on the card.
-     */
-    slotText(slot) {
-        const said = this.sentenceOf(slot);
-        if (said) {
-            return said;
-        }
-        const scalar = slot.value !== null && slot.value !== undefined && typeof slot.value !== 'object';
-        if (scalar && slot.unit) {
-            return `${slot.value} ${slot.unit}`;
-        }
-        return scalar ? String(slot.value) : fieldValue(slot.value);
-    }
-    /** The sentence this slot declares, when it declares one and the locale has it. */
-    sentenceOf(slot) {
-        if (!slot.valueTKey) {
-            return '';
-        }
-        return String(i18next.t(slot.valueTKey, { ...(slot.params || {}), defaultValue: '' }) || '');
-    }
-    /** Did a person actually write something here. Whitespace is not words. */
-    hasWords(slot) {
-        return typeof slot.value === 'string' ? slot.value.trim().length > 0 : slot.found && slot.value != null;
+        const decision = renderDecision(activityInstance?.activity?.decisionKeys, activityInstance?.input);
+        return decision === nothing ? nothing : keyed(this.activityApproval.id, decision);
     }
     generatedContent() {
         if (!this.activityApproval) {