iobroker.alias-workbench 0.9.11 → 0.9.12

package/README.md

@@ -25,6 +25,18 @@ good is decided by things you **cannot see**:
 A tool that only links datapoints helps with none of that. This adapter shows
 all four while you are building, not hours later in the log.
 
+### Why an admin tab, not a settings page
+
+The workbench is an **admin tab** (`adminTab` in `io-package.json`) and runs
+without an adapter process (`mode: none`). That is deliberate: it is an
+interactive editor — an object tree on the left, the draft with live values on
+the right, a dry run before anything is written, a source swap, a template
+sheet. None of that fits into a jsonConfig page, which is built for settings.
+The instance's actual settings do live in a normal jsonConfig page
+(`adminUI.config: "json"`, four switches, translated into all eleven
+languages). Without a process the adapter uses no memory while nobody has the
+tab open.
+
 ## What it does
 
 ![The workbench with a device picked](https://raw.githubusercontent.com/RicardoHipp/ioBroker.alias-workbench/main/docs/01-device.png)
@@ -311,6 +323,18 @@ nothing on purpose. Fifty bare `LEVEL` channels are not a device.
 When a template is rejected, the picker says why: *"1.LEVEL missing"*, not
 *"does not fit"*. A reason you cannot act on is not a reason.
 
+**About the role `value.power.consumption`.** Four templates (`messpunkt`,
+`tasmota-lampe`, `tasmota-mehrfach`, `tasmota-steckdose`) give the row
+`CONSUMPTION` the role `value.power.consumption`, which is struck through in
+the ioBroker [state roles list](https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/stateroles.md).
+It stays on purpose: `@iobroker/type-detector` 6.0.x recognises `CONSUMPTION`
+only with exactly this role (`typePatterns.js`, pattern `consumption`,
+`defaultRole: 'value.power.consumption'`). With `value.energy.consumed` the
+point would drop out of the detected device, and vis, Alexa and the device
+manager would no longer show any consumption. Each of the four rows carries
+this reason in a field `_rolle`. The role will change once the type-detector
+does.
+
 ### Role knowledge beside the rows
 
 A template can carry `weiterePunkte`: roles for datapoints that get no row of
@@ -541,12 +565,16 @@ served as they are; editing one and reloading the tab is the whole cycle. On a
 running installation the admin caches adapter files, so `iobroker upload
 alias-workbench` after copying is what makes the browser see the change.
 
-Translations live in `admin/i18n/`. A new language is a new file there and an
-entry in `SPRACHEN` in `admin/js/sprache.js` — nothing else. Watch for strings
+There are two dictionaries. `admin/i18n/` belongs to the **settings page**
+(jsonConfig with `"i18n": true`, all eleven languages, loaded by the admin
+itself). `admin/sprachen/` belongs to the **tab** (German and English, loaded by
+`admin/js/sprache.js`; any other language falls back to English). A new tab
+language is a new file in `admin/sprachen/` and an entry in `SPRACHEN` in
+`admin/js/sprache.js` — nothing else. Watch for strings
 written straight into `admin/tab.html`: they are replaced at runtime, and one
-that nobody wired up stays German forever without anyone noticing.
+that nobody wired up stays English in every language without anyone noticing.
 
-The tab is **29 ES modules** under `admin/js/`, loaded by the browser as
+The tab is **30 ES modules** under `admin/js/`, loaded by the browser as
 modules — there is no build step for them, and styling lives in one file,
 `admin/css/werkbank.css`. Because a module boundary can be crossed wrongly in
 ways the browser only notices while running, `npm test` checks before every
@@ -564,6 +592,26 @@ imported, and that both language files carry the same keys.
 
 ## Changelog
 
+### 0.9.12 (2026-09-18)
+* The small **type badges in the tree** (DEV, CHA, FOL …) now all have the same width. A node without an object of its own shows just a dot there, and its badge was narrower — its name fell out of line.
+* **“Move …” did nothing on some aliases.** Creating alias points by hand in the admin often yields an alias without its own channel object — the node shows in the tree but is not an object itself. The workbench displayed the “Move …” button there, but clicking it opened no dialog and said nothing. Such an alias is now moved exactly like one with a channel; a missing channel is not created along the way.
+* **No German left in the English interface.** Three places were hard-wired German: the type badge in the detail row (“Zahl”), the “Datenpunkt” field next to “writes to” and the screen-reader label of the tick boxes (“SET anlegen”). The fixed default texts in `tab.html` are now English — before, 44 German texts were only translated at runtime. The two browser-console messages are English now as well.
+* The tab's **menu entry** now has the adapter's name in all eleven admin languages. Before, it existed in German and English only, and every other language showed “Alias Workbench”.
+* **Why `value.power.consumption`** — four shipped templates give consumption this role, which the ioBroker roles list marks as deprecated. That is deliberate: the type-detector 6.0.x recognises consumption only with exactly this role. The reason is now written into every affected template (field `_rolle`) and into the README. The README also explains why the workbench is an admin tab rather than a settings page.
+* **The object count in the header grew the longer the tab stayed open.** Only devices, channels, folders and states are meant to be counted — and only those are loaded. When following changes, however, the workbench took in every changed object type: after switching the system language, `system.config` sat in its store and the count read 3146 instead of 3145, likewise after every saved instance setting or adapter update. Following changes now applies the same rule as loading.
+* **Values from another node did not update at a source.** When you select a source that already has an alias, the list also shows that alias’s rows — and they often read from somewhere else. The electricity meter shows three rows “from Solar.Netz”: import, smoothed import and house consumption. Their value was fetched once on selection and then froze; it stayed at 428 W for hours while the system reported 22 W. Only the selected node was subscribed; the rows’ sources were only added in “Edit alias” mode. The workbench now subscribes to everything it displays — including a source you point a row to by hand in the detail view. The header accordingly shows two subscriptions instead of one on the electricity meter.
+* The legend above the list claimed **“The light pattern does not apply right now: SET requires boolean, number is set. That is why no row has a place.”** — on a device whose SET row carries `switch.light`/`boolean` and is perfectly fine. The culprit was `0_AES_KEY`: hm-rpc gives it the role `state` and the type `number`, and `state` is allowed on the light pattern’s SET place. The check asks per row “your role would fit a place, your type does not” — it never looks at who ended up holding the place, and the message then printed the PLACE instead of the row it was about. The sentence was wrong at the end too: five rows did have a place. The type reason now only applies when the named place really did stay empty; otherwise the ordinary legend is back, and every placeless row is tinted instead of just one. The case the message was built for — SET itself in the wrong type, place stays empty — is unchanged.
+* Rows carried **“differs from the template”** where no template was in force at all — and flagged the better value while doing so. What was really compared was the raw draft, that is the bare role of the source point: on a switch actuator, after “— no template —”, `SET` showed the badge and below it “The template wants: **switch**”, while the alias holds `switch.light`, which even fits the `light` pattern. Same on `UNREACH`, where `indicator.maintenance.unreach` counted as the deviation and `indicator.unreach` as the target. And it was not only the dismissed case: on a device for which a template is **never** found, the same badge appeared. The rule now: with no template in force — dismissed or never found — no template value is recorded, so the badge, the detail line, the `←` button and the **template** filter button all drop out. Rows keep what the alias holds; that is the stock rule and has nothing to do with templates. With a template everything stays as it was — a real deviation is still shown.
+* With **“— no template —”** the **output** picker stayed on screen if you came from the multi-output template — coming via any other template it was gone. Two routes, two answers on the same device. Worse: using the leftover picker brought the dismissed template **silently back** (measured: after switching to `Licht_Esstisch` the field read “HomeMatic switch actuator with several outputs” again). An output does not belong to the device but to the template’s `mehrfach` block — without it there is no placeholder and so nothing to choose; of 17 bundled templates exactly two carry such a block. The raw draft no longer inherits it, and the channels go through “one sub-folder per channel” instead. **And everything is ticked now.** The raw draft used to tick whatever had a place in the detected pattern; on two **identical** channels the detector picked one of them — the second — so `Licht_Esstisch` stood ticked and `Licht_Bar` empty. Which one it picks is chance, and guessing is not the job of a draft without a template: dropping the template means picking the points yourself.
+* The expander in the template box read **“Why this template +”** and always wanted the right-hand edge. With a long name and two badges — “HomeMatic roller shutter actuator”, chosen by hand, forced — it no longer found room there and took **a line of its own**; the box grew without anything having been added. It is now a **question mark**, with the sentence in the tooltip. What was left out stays visible as a short badge (`4 WITHOUT SOURCE`) — nobody should have to go looking for the fact that points are missing — and the sentence behind it now reads “4 left out because the source is missing” instead of “Left out because the source is missing: 4”. The header no longer wraps at all: room is made by the template name, which is shortened and shown in full in its tooltip; the id stays put, because `hm-rollladen…` is no longer an id. Expanded, the question mark is filled.
+* **A push-button is not simply any write-only point.** The workbench left the read source empty on every row whose source carried `read: false`. But HomeMatic uses that flag for two different things, counted across every device in the house: for a button press (`PRESS_SHORT` 100 points, `RAMP_STOP` 3, `STOP` 1, `OLD_LEVEL` 3 — all with `native.TYPE: ACTION`) **and** for a setting that stays put (`ON_TIME` 20 points, `RAMP_TIME` 3 — `TYPE: FLOAT`). That is why the living-room dimmer showed “never changes” on `ON_TIME` with an empty “reads from” field — on a value in seconds that one does set at the source now and then. Measured: an alias put on `1.ON_TIME`, the source set to 30 directly, and the alias showed 30. So `read: false` does not block reading. The read source is now left empty only where a button press is really meant: `native.TYPE: ACTION`, or a role starting with `button`. `OLD_LEVEL` is the case where the role is not enough — it is called `value.dimmer`.
+* **“— no template —”** is now in the template picker. Once the workbench had detected a template it could only be swapped for another one, never dropped: the first entry of the list was a mere heading with an empty value, and the branch bailed out on an empty value. Anyone wanting to create all points of a device without a template’s defaults had no way to get there. The choice now rebuilds the draft without any template — ticked is whatever has a place in the detected pattern — and target, folder, name, room and function stay as they were.
+* The mark on a row whose source only writes read **“reports nothing”**. That sounded like a fault — as if the device were mute. What is meant is different: the value stays where it was written and never reports back by itself. It now reads **“never changes”**, and the same in the “target does not read” check.
+* **“One sub-folder per channel”** — the button next to “create as one device” serves a different purpose, and nothing said so: it **mirrors the folder structure of the source**. A container for the device, one channel each below it, the states below those. **The ticks now decide what appears**: one sub-folder for every channel that contributes at least one ticked row, holding exactly those rows. Before, the path built a fresh draft per channel and ticked whatever that channel’s pattern offered — things appeared that nobody had chosen (on the test device the maintenance channel got three points instead of the two ticked ones), and channels with nothing chosen were created all the same. Edits made by hand on a row — role, type, unit, formula, caption — now travel with it. **Information channels count too**: they used to be dropped, so the button said “3 of them” while six channels stood in the tree next to it — and clicking `general` on its own has long given a draft (“Information ✓ 7/14 filled”) that could be created. **The maintenance channel gets its own folder** instead of being copied into every output: `UNREACH` exists only in channel 0 of the dimmer, so that is where it belongs; whoever wants it with the light takes “create as one device”. The copying had produced nonsense — 206 raw data points of a lawn mower in each of four drafts, the three meter phases of a Shelly SHEM-3 counted as “shared” instead of as channels of their own. **Virtual channels drop out by themselves**: on a HomeMatic dimmer nothing is ticked in the two `VIRTUAL_DIMMER` channels, so nothing is created there. **States directly on the device** stay in the container, which is then created as a channel instead of a folder. **And the hint is right again**: it computed `targetFolder + channel` while writing uses `targetFolder + device + channel` — on the dishwasher it promised `alias.0.Commands` and created `alias.0.Dishwasher.Commands`, on the living-room dimmer it read `alias.0.1`, `alias.0.2`, `alias.0.3`. It now names the id, the number of ticked points and the detected type. The label says so too: “one sub-folder per channel, N of them …” instead of “one device per channel”
+* On a **multi-output device** the draft adopted the ticks of a foreign alias. The workbench asked "is there already an alias for this device?" and answered with the one that reads **most often** from it. On Ricardo's power strip that is the consumption alias with six points from `tele.SENSOR` — it beat `RF1000` with two points from `stat.POWER1`. Output 1 then showed ELECTRIC_POWER and CONSUMPTION ticked, SET unticked, and the pattern field reported **"✕ SET missing"** on a device with a working SET. The majority rule is right for a single-output device and the wrong question for a multi-output one. The right question is: **does this alias belong to the output currently selected?** That is not guessed from names but asked of the template — which of its paths carry the placeholder? Exactly those tell the outputs apart (`stat.POWER%N%`, `cmnd.POWER%N%`); everything else (`tele.SENSOR`, `tele.LWT`) is shared. This also removes the trap from I31: `tele.INFO2` does carry a two, but no placeholder. For output 1 the bare `cmnd.POWER` counts as well — with Tasmota, POWER without a number is a second name for POWER1, measured at the device: a query to `cmnd/POWER` is answered with `{"POWER1":"ON"}`, and a topic `stat/POWER` does not exist. **The bug only hit the first glance**: coming through the tree brought the wrong stock, switching the output asked nothing at all and the template defaults won — the same output looked different depending on how you got there. All three paths now ask the same question. Single-output devices never enter the new branch by construction; without the new argument `aliasFuer` behaves line for line as before.
+* **The target line now follows the same alias.** On a multi-output device it always showed the output number — `alias.0.Bastelzimmer.Steckdosenleiste.POWER1`, an ID that does not exist — and reported "will be created", while `RF1000` stood right next to it. Clicking would have put a second alias beside the existing ones. The source said: "the workbench cannot know what the individual output is called." That held as long as it could only ask about the device. The target line now points at `…Steckdosenleiste.RF1000` with the badge "already exists", and "update alias" hits it. Where no alias exists for the output yet — the garden valve island — the proposal from the output number stays, and so does "will be created".
+* The **SetOption59** note appeared on every Tasmota device, including where it has no effect. It claimed an alias would show "the old value for minutes after switching" — but on the power strip not a single row reads from `tele/STATE`; they all read from `stat.POWERn`, which the device sends immediately on every command (measured: 1.07 s). "For minutes" also assumed the Tasmota default of 300 seconds; on that strip it is ten (measured: `tele/STATE` every ten seconds). The block now only appears when a row really reads from `tele/…` **and** writes to `cmnd/…`, names those rows, and reads the interval from `TelePeriod`; if it is not in the telegram the text says "on the telemetry cycle" instead of inventing a number. The same applies to the chip in the header. And the check "feedback is immediate" no longer reports "all fine" where there was nothing to check, but **"no row affected — all read from stat/…"** with a grey dot instead of a tick. Card and check ask the same function and can no longer drift apart.
+
 ### 0.9.11 (2026-09-16)
 * After saving, the next click in the tree asked **“unsaved changes”** — about an alias that had just been written. The “something unwritten is here” mark was only ever set, never cleared; it went away solely when the draft itself was dropped, i.e. on delete and on move. It now falls after every successful write — and only then: if something went wrong, the change really is not in yet, and the question stays justified. **There is a second effect that had been missing for a long time**: while the mark stood, the workbench deliberately did not rebuild the draft. After a write the old one therefore stayed, instead of rebuilding from the freshly written alias.
 * A second press of **“Suggest free slots”** bent back a row you had set yourself. On the air conditioner: guess nine slots, set the role of the SPEED row to `level.speed` by hand, discard — and the next suggestion put the guessed `level.mode.fan` back. The pattern slot looked free, so the same row was guessed a second time; it then carried **both** marks, “changed” and “guessed”, which exclude each other — what was just guessed cannot have been changed before. Guessing now leaves any row marked “changed” alone. The slot stays free, and the note below says why. Putting a second row next to it was the alternative — the same source point would then hold two slots, and the eighth check would rightly report it.

package/admin/css/werkbank.css

@@ -199,10 +199,15 @@
   .tree .node.sel .chip { background:rgba(255,255,255,.22); color:var(--accent-ink); border-color:transparent; }
   .tree .node.sel .nm { color:var(--accent-ink); }
   .tree .node.sel > span { color:var(--accent-ink) !important; }
+  /* Feste Breite: sonst bestimmt der Text die Breite, und ein Knoten ohne
+     eigenes Objekt (nur „·“) rueckte seinen Namen aus der Flucht der
+     anderen (Ricardo, 18.09.2026). Auch DEV/CHA/FOL/STA sind so gleich
+     breit. */
   .tree .kind {
     font-size:8px; padding:0 4px; border-radius:4px; background:var(--surface-3);
     color:var(--ink-3); text-transform:uppercase; letter-spacing:.04em;
     font-family:var(--sans); font-weight:700; flex:0 0 auto;
+    width:26px; box-sizing:border-box; text-align:center;
   }
   .tree .idstueck { flex:0 0 auto; }
   /* Der Name weicht zurueck: gedaempft, und er ist es, der gekuerzt wird,
@@ -840,15 +845,28 @@
   /* Der Aufklapper steht in der Kopfzeile ganz rechts. `margin-left:auto`
      schiebt ihn dorthin, ohne dass die Kopfzeile etwas davon wissen muss. */
   .suggestbox .hd .wkopf {
-    display:inline-flex; align-items:center; gap:6px; cursor:pointer;
-    margin-left:auto; padding:2px 7px; border-radius:var(--r-sm);
+    display:inline-flex; align-items:center; gap:5px; cursor:pointer;
+    margin-left:auto; padding:2px 5px; border-radius:var(--r-sm);
     font-size:11.5px; font-weight:400; color:var(--ink-3);
     user-select:none; transition:background .12s, color .12s;
-    white-space:nowrap;
+    white-space:nowrap; flex:0 0 auto;
+  }
+  /* Statt „Warum diese Vorlage +" ein Fragezeichen mit Tooltip: die
+     Kopfzeile ist eng, der Satz ist lang, und gelesen wurde er ohnehin
+     nur einmal (Ricardo, 17.09.2026). Aufgeklappt wird das Zeichen
+     gefuellt - sonst saehe man dem Kopf nicht an, dass unten etwas steht. */
+  .suggestbox .hd .wkopf .fragez {
+    display:inline-flex; align-items:center; justify-content:center;
+    width:16px; height:16px; flex:0 0 auto; border-radius:50%;
+    border:1px solid var(--line-strong); color:var(--ink-2);
+    font-size:11px; font-weight:700; line-height:1;
+  }
+  .suggestbox .hd .wkopf.auf .fragez {
+    background:var(--accent); border-color:var(--accent); color:#fff;
   }
-  .suggestbox .hd .wkopf .klappz { color:var(--ink-2); font-size:10px; }
   .suggestbox .hd .wkopf:hover { color:var(--ink); background:var(--surface-2); }
-  .suggestbox .hd .wkopf:hover .klappz { color:var(--accent); }
+  .suggestbox .hd .wkopf:hover .fragez { color:var(--accent); border-color:var(--accent); }
+  .suggestbox .hd .wkopf.auf:hover .fragez { color:#fff; }
   .warum .wleib { margin:0; }
   .warum .wleib ul { margin:0; padding-left:16px; }
   .warum .wleib li { margin:1px 0; }
@@ -857,7 +875,25 @@
     text-transform:uppercase; font-weight:700; color:var(--ink-3);
   }
 
-  .suggestbox .hd { display:flex; align-items:center; flex-wrap:wrap; }
+  /* Die Kopfzeile bleibt EINE Zeile. Sie umbrechen zu lassen half nicht:
+     der Hinweis rechts hat `margin-left:auto`, fand bei langem Namen und
+     zwei Marken keinen Platz mehr und nahm sich eine ganze Zeile fuer
+     sich allein. Jetzt weicht stattdessen der Vorlagenname zurueck - er
+     steht vollstaendig im Tooltip und im Auswahlfeld darunter. */
+  .suggestbox .hd { display:flex; align-items:center; flex-wrap:nowrap; overflow:hidden; }
+  .suggestbox .hd > * { flex:0 0 auto; }
+  .suggestbox .hd .tname,
+  .suggestbox .hd .tid {
+    flex:0 1 auto; min-width:0; overflow:hidden;
+    text-overflow:ellipsis; white-space:nowrap;
+  }
+  .suggestbox .hd .tid {
+    font-family:var(--mono); font-size:10.5px; font-weight:400; color:var(--ink-3);
+    /* Die Kennung weicht gar nicht: sie ist kurz, und abgeschnitten sagt
+       sie nichts mehr - `hm-rollladen…` ist keine Kennung. Platz macht der
+       Name, der bis auf null zurueckgehen darf. */
+    flex-shrink:0;
+  }
 
   .suggestbox { flex-direction:column; align-items:stretch; gap:9px; }
   .suggestbox .hd { width:100%; }

package/admin/js/detail.js

@@ -8,7 +8,7 @@ import { wertVon, jsonFelder, feldAusFormel, feldFormel, feldWert, jsonVon, ziel
 import { musterVon, erkenneEntwurf, platzFuerRolle, typenFuerRolle } from './erkennung.js';
 import { musterName } from './musternamen.js';
 import { rollenFeld } from './rollenwahl.js';
-import { opt , quellenAuswahl, vorlagenAbweichung, bestandsAbweichung } from './entwurf.js';
+import { opt , quellenAuswahl, vorlagenAbweichung, bestandsAbweichung, aboErgaenzen } from './entwurf.js';
 import { zeichneErgebnis, entwurfAngefasst } from './ergebnis.js';
 
 import { rateBehalten } from './vorschlagen.js';
@@ -26,7 +26,13 @@ export function detailZeile(e, s, idx) {
 
   /* Wer die Zeile anfasst, hat entschieden - die Vermutung ist damit
      keine mehr und faellt beim Zuruecknehmen nicht mehr weg. */
-  function neu() { s.geaendert = true; rateBehalten(s); entwurfAngefasst(); zeichneErgebnis(); }
+  /* Eine von Hand umgestellte Quelle wird mit abonniert - sonst stuende
+     ihr Wert nach dem ersten Abholen still (J9b). */
+  function neu() {
+    s.geaendert = true; rateBehalten(s); entwurfAngefasst();
+    aboErgaenzen([s.srcR, s.srcW]);
+    zeichneErgebnis();
+  }
 
   /* ---- Die Auswahl der Datenpunkte ---------------------------------
 
@@ -358,7 +364,7 @@ export function detailZeile(e, s, idx) {
     neu();
   });
   var lW = el('label', 'fld');
-  lW.appendChild(el('span', null, 'Datenpunkt'));
+  lW.appendChild(el('span', null, tr('detail.datapoint')));
   lW.appendChild(selW);
   wf.appendChild(lW);
   /* Der Hinweis gilt nur, wenn die beiden Quellen wirklich
@@ -543,7 +549,7 @@ export function detailZeile(e, s, idx) {
     var t = typeof a.val;
     var passt = !s.typ || s.typ === t || s.typ === 'mixed';
     box.appendChild(el('span', 'chip ' + (passt ? 'ok' : 'warn'),
-      (passt ? '✓ ' : '≠ ') + (t === 'number' ? 'Zahl' : (t === 'boolean' ? 'boolean' : t))));
+      (passt ? '✓ ' : '≠ ') + (t === 'number' ? tr('detail.typeNumber') : t)));
     if (a.gewandelt) {
       box.appendChild(document.createTextNode('  '));
       box.appendChild(el('span', 'chip mut', tr('detail.converted')));