openvibers/openvibe.bot Commits

openvibe-shared v4.228.0 (was v4.227.0) (#427)

OP OpenVibers committed 10 h ago · a7b1967f26 · 6 files, +18 −18

* openvibe-shared v4.228.0 (was v4.227.0)

Opened by scripts/pin-bump.js in OpenVibers/OpenVibe.Shared after the v4.228.0 release.

* Home: the pick block is Shared's showcase.pick(), tabs on a phone; the old chips only jumped, so every pane showed

Changes

ddiff --git a/README.md b/README.md
iindex 3ef49ec..7932ce6 100644
---- a/README.md
++++ b/README.md
@@@ -155,7 +155,7 @@ Sitemap entries use the release commit time for these code-backed pages.
  - OpenVibe.Network: user and service tokens, node tokens, pairing when the authority is `network`, and identity resolution (a `@username` to a subject, and subjects to names, for the panel's People card).
  - OpenRestream: the WHIP ingest base (`BOT_WHIP_BASE`) devices publish to.
  - OpenVibe.Billing: job usage readings go to `billing.usage.record` (`BOT_BILLING_URL`; Bot mints its own Network service token for audience `openvibe.billing` with `billing.usage.record`, and `BOT_BILLING_TOKEN` overrides it); unset, they wait in `run_usage_outbox`.
--- `openvibe-contracts` v0.417.0, `openvibe-sdk` v0.161.0 and `openvibe-shared` v4.227.0 (package.json).
++- `openvibe-contracts` v0.417.0, `openvibe-sdk` v0.161.0 and `openvibe-shared` v4.228.0 (package.json).
  - Account export and deletion (ADR-033): Network grants `events.subscription.manage` (openvibe.events) for the two
    subscriptions created at boot, then, last and once the release is live, `network.account.export.contribute` and
    `network.account.deletion.confirm` (openvibe.network).
@@@ -247,5 +247,5 @@ OpenVibe.Sites no longer generates `deploy/nginx/openvibe.bot.conf` (sites.json
  <!-- versions:start -->
  - openvibe-contracts: v0.417.0
  - openvibe-sdk: v0.161.0
--- openvibe-shared: v4.227.0
++- openvibe-shared: v4.228.0
  <!-- versions:end -->
ddiff --git a/STATUS.json b/STATUS.json
iindex 98f9d5f..86625bd 100644
---- a/STATUS.json
++++ b/STATUS.json
@@@ -7,11 +7,11 @@
    "domain": "openvibe.bot (www -> apex; /device and /control WebSockets; /install)",
    "contracts": "openvibe-contracts v0.417.0",
    "sdk": "openvibe-sdk v0.161.0",
--  "shared": "openvibe-shared v4.227.0",
++  "shared": "openvibe-shared v4.228.0",
    "packages": [
      "openvibe-contracts v0.417.0",
      "openvibe-sdk v0.161.0",
--    "openvibe-shared v4.227.0"
++    "openvibe-shared v4.228.0"
    ],
    "capabilities": [
      "bot.robot.read",
ddiff --git a/package-lock.json b/package-lock.json
iindex 7d65d44..7e3e94b 100644
---- a/package-lock.json
++++ b/package-lock.json
@@@ -15,7 +15,7 @@
          "iovalkey": "^0.4.0",
          "openvibe-contracts": "https://codeload.github.com/OpenVibers/OpenVibe.Contracts/tar.gz/refs/tags/v0.417.0",
          "openvibe-sdk": "https://codeload.github.com/OpenVibers/OpenVibe.SDK/tar.gz/refs/tags/v0.161.0",
--        "openvibe-shared": "https://codeload.github.com/OpenVibers/OpenVibe.Shared/tar.gz/refs/tags/v4.227.0",
++        "openvibe-shared": "https://codeload.github.com/OpenVibers/OpenVibe.Shared/tar.gz/refs/tags/v4.228.0",
          "pg": "^8.23.0",
          "qrcode-generator": "^2.0.4",
          "ws": "^8.18.0"
@@@ -1245,9 +1245,9 @@
        }
      },
      "node_modules/openvibe-shared": {
--      "version": "4.227.0",
--      "resolved": "https://codeload.github.com/OpenVibers/OpenVibe.Shared/tar.gz/refs/tags/v4.227.0",
--      "integrity": "sha512-/PqSqCC/vw/iwvkhDipLyixPyrtTcMpPhZ8UQ35fGDfAEJt43du2MYmZetu/TUysz+iJ9/Z7+VCFsFI1XzuF3Q==",
++      "version": "4.228.0",
++      "resolved": "https://codeload.github.com/OpenVibers/OpenVibe.Shared/tar.gz/refs/tags/v4.228.0",
++      "integrity": "sha512-HR0Hac/K+qKmwMf2QZWCFxW+uVja36nyAtxVbNFgPGUDFdmWhmb6RMoP/CkU0+GSCPsC50k6aTS+oPx2czDgGg==",
        "license": "MIT",
        "dependencies": {
          "clean-css": "^5.3.3",
ddiff --git a/package.json b/package.json
iindex 77800f1..c96b17d 100644
---- a/package.json
++++ b/package.json
@@@ -27,7 +27,7 @@
      "iovalkey": "^0.4.0",
      "openvibe-contracts": "https://codeload.github.com/OpenVibers/OpenVibe.Contracts/tar.gz/refs/tags/v0.417.0",
      "openvibe-sdk": "https://codeload.github.com/OpenVibers/OpenVibe.SDK/tar.gz/refs/tags/v0.161.0",
--    "openvibe-shared": "https://codeload.github.com/OpenVibers/OpenVibe.Shared/tar.gz/refs/tags/v4.227.0",
++    "openvibe-shared": "https://codeload.github.com/OpenVibers/OpenVibe.Shared/tar.gz/refs/tags/v4.228.0",
      "pg": "^8.23.0",
      "qrcode-generator": "^2.0.4",
      "ws": "^8.18.0"
ddiff --git a/server/web/short.js b/server/web/short.js
iindex e1a9341..bc27b7f 100644
---- a/server/web/short.js
++++ b/server/web/short.js
@@@ -1,4 +1,5 @@
  'use strict';
++const { pick } = require('openvibe-shared/showcase');
  /**
   * The home on a phone (the owner's short-pages rule, 2026-10-30): one or two screens, app-like.
   *
@@@ -18,12 +19,11 @@ const esc = (s) => String(s == null ? '' : s).replace(/&/g, '&amp;').replace(/</
   *
   *   pickBlock({ key, label, head, panes })   panes: [{ id, label, html }]
   */
--function pickBlock({ key, label, head = '', headId = '', bodyClass = '', panes }) {
--    if (!panes || !panes.length) return '';
--    const chips = panes.map((p) => `<a class="pick-chip" href="#pp-${esc(key)}-${esc(p.id)}">${esc(p.label)}</a>`).join('');
--    const body = panes.map((p) => `<div class="pick-pane" id="pp-${esc(key)}-${esc(p.id)}">${p.html}</div>`).join('');
--    const heading = `<h2${headId ? ` id="${esc(headId)}"` : ''}>${esc(head || label)}</h2>`;
--    return `<div class="pick" id="pick-${esc(key)}"><div class="pick-bar"><div class="pick-head">${heading}</div><nav class="pick-row" data-ovk-scroll aria-label="${esc(label)}">${chips}</nav></div><div class="pick-body${bodyClass ? ` ${bodyClass}` : ''}">${body}</div></div>`;
++// 4.228.0: Shared's showcase.pick() draws the block (tabs on a phone, every pane in the page, a link to a pane opens
++// it); this copy only jumped to a pane with chips, so a phone showed them all (the owner, 11-12). The ids are this
++// site's own, so its links keep working.
++function pickBlock({ key, label, head = '', headId = '', bodyClass = '', panes, checked = '' }) {
++    return pick({ key, label, head, headId, bodyClass, checked: checked ? `pp-${key}-${checked}` : '', panes: (panes || []).map((p) => ({ id: `pp-${key}-${p.id}`, label: p.label, html: p.html })) });
  }
  
  /**
ddiff --git a/test/home-short.test.js b/test/home-short.test.js
iindex e34613e..6aac33c 100644
---- a/test/home-short.test.js
++++ b/test/home-short.test.js
@@@ -18,16 +18,16 @@ const { boot, check, done } = require('./helpers/app');
  
          // The ways to connect: one pick block, a chip link per way, every way in the page.
          assert.ok(html.includes('<div class="pick" id="pick-connect">'), 'the ways to connect are one pick block');
--        assert.match(html, /<nav class="pick-row" data-ovk-scroll aria-label="The ways to connect">/);
++        assert.match(html, /<div class=\"ov-parts ov-parts--front\" data-parts=\"\d+\">/, 'its parts are tabs on a phone');
          const paneIds = [...html.matchAll(/<div class="pick-pane" id="(pp-connect-[a-z-]+)">/g)].map((m) => m[1]);
--        const chips = [...html.matchAll(/<a class="pick-chip" href="#(pp-connect-[a-z-]+)">([^<]+)<\/a>/g)];
++        const chips = [...html.matchAll(/<label for="ovp-[a-z0-9]+-(pp-connect-[a-z-]+)">([^<]+)(?: <span class="ov-parts-n">[^<]*<\/span>)?<\/label>/g)];
          assert.ok(chips.length >= 3, 'a chip per way to connect');
          assert.deepStrictEqual(chips.map((m) => m[1]), paneIds, 'one chip per pane, in the same order');
          // Every way stays in the page for the crawlers.
          for (const label of ['Simulated', 'A phone', 'On-board']) assert.ok(html.includes(`<strong>${label}</strong>`), `${label} is in the page`);
          // The chips are links, not radios, and their targets exist in the page.
          const pick = html.slice(html.indexOf('<div class="pick" id="pick-connect">'), html.indexOf('</section>', html.indexOf('<div class="pick" id="pick-connect">')));
--        assert.ok(!/type="radio"/.test(pick), 'no radio input inside the pick block');
++        assert.ok(/<input type=\"radio\" class=\"ov-parts-r\"[^>]* checked>/.test(pick), 'the first part is open on a phone (Shared pick(): tabs, 11-12)');
          assert.ok(!/pickRule|pk-connect/.test(html), 'no inline rule that shows one picked pane');
          for (const m of chips) assert.ok(html.includes(`id="${m[1]}"`), `the pane ${m[1]} exists`);