npm - getobsrv - Versions diffs - 0.10.0 → 0.12.0 - Mend

getobsrv 0.10.0 → 0.12.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (16) hide show

package/README.md +34 -0
package/out/main/cli.js +1 -1
package/out/main/index.js +842 -192
package/out/main/{targetSource-CbJwfugi.js → targetSource-DlsHc-Ri.js} +50 -1
package/out/mcp/server.js +21 -1
package/out/preload/app.js +25 -16
package/out/renderer/assets/{index-CZUlXhEb.css → index-DvwDBeho.css} +284 -14
package/out/renderer/assets/{index-CQVE5hF3.js → index-gYDbIgqJ.js} +655 -157
package/out/renderer/index.html +2 -2
package/out/shared/control.js +13 -1
package/out/shared/history.js +90 -0
package/out/shared/ipcPayloads.js +41 -2
package/out/shared/presets.js +12 -1
package/out/shared/tabList.js +46 -0
package/package.json +1 -1
package/skills/obsrv-screens/SKILL.md +7 -0

package/out/renderer/index.html CHANGED Viewed

@@ -4,8 +4,8 @@
     <meta charset="UTF-8" />
     <meta http-equiv="Content-Security-Policy" content="default-src 'self'; style-src 'self' 'unsafe-inline'; img-src 'self' blob: data:" />
     <title>Obsrv</title>
-    <script type="module" crossorigin src="./assets/index-CQVE5hF3.js"></script>
-    <link rel="stylesheet" crossorigin href="./assets/index-CZUlXhEb.css">
+    <script type="module" crossorigin src="./assets/index-gYDbIgqJ.js"></script>
+    <link rel="stylesheet" crossorigin href="./assets/index-DvwDBeho.css">
   </head>
   <body>
     <div id="root"></div>

package/out/shared/control.js CHANGED Viewed

@@ -233,5 +233,17 @@ function parseControlStatus(raw) {
     const panes = raw.panes ?? 'both';
     if (panes !== 'both' && panes !== 'target')
         return null;
-    return { version, url, presetId, profileId, viewMode, panes, mode };
+    // The same skew, one release later. An app that predates tabs has exactly
+    // one session — it is the tab at index 0, and it has no id to name — so the
+    // defaults describe that app truthfully rather than papering over it. An
+    // agent polling one sees an unchanging `tabId`, which is correct: there is
+    // no other tab for the user to switch to. Returning null instead would take
+    // out drive and live snap wholesale against every app older than tabs.
+    const tabId = raw.tabId ?? '';
+    if (typeof tabId !== 'string')
+        return null;
+    const tabIndex = raw.tabIndex ?? 0;
+    if (typeof tabIndex !== 'number' || !Number.isInteger(tabIndex) || tabIndex < 0)
+        return null;
+    return { version, url, presetId, profileId, viewMode, panes, mode, tabId, tabIndex };
 }

package/out/shared/history.js ADDED Viewed

@@ -0,0 +1,90 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.isStamp = exports.isCount = exports.isRecord = exports.MAX_URL_LENGTH = exports.HISTORY_SUGGESTIONS = exports.HISTORY_MAX = void 0;
+exports.isStorableUrl = isStorableUrl;
+exports.byRank = byRank;
+exports.recordVisit = recordVisit;
+exports.matchHistory = matchHistory;
+const url_1 = require("./url");
+/**
+ * The file is read and written whole, so it needs a bound. 500 is far beyond
+ * what this app's usage produces — a handful of dev and staging addresses
+ * returned to over and over — and small enough to parse instantly.
+ */
+exports.HISTORY_MAX = 500;
+/** Rows the URL bar offers. Past six, a longer list is a worse tool than a better query. */
+exports.HISTORY_SUGGESTIONS = 6;
+/**
+ * The other half of the bound: 500 entries of unbounded length is not a
+ * bounded file. Far longer than any address this tool is pointed at, and
+ * short enough that the whole file stays trivial to parse.
+ */
+exports.MAX_URL_LENGTH = 2048;
+const isRecord = (v) => typeof v === 'object' && v !== null;
+exports.isRecord = isRecord;
+const isCount = (v) => typeof v === 'number' && Number.isInteger(v) && v >= 1;
+exports.isCount = isCount;
+const isStamp = (v) => typeof v === 'number' && Number.isFinite(v) && v >= 0;
+exports.isStamp = isStamp;
+/**
+ * Is this an address worth remembering? Only the schemes the app can load
+ * back (`shared/url.ts`), so `about:blank` — which the native pane commits on
+ * every launch — never becomes a suggestion, and a scheme the URL bar would
+ * refuse can never be offered by it.
+ */
+function isStorableUrl(v) {
+    if (typeof v !== 'string' || v === '' || v.length > exports.MAX_URL_LENGTH)
+        return false;
+    try {
+        return url_1.ALLOWED_URL_SCHEMES.includes(new URL(v).protocol);
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Most recent first, visit count breaking ties — so the address used
+ * constantly stays at the top without a stale favourite outranking what was
+ * on screen ten minutes ago. URL breaks the remaining tie only to keep the
+ * order stable across reloads of the same file.
+ */
+function byRank(a, b) {
+    if (a.lastVisit !== b.lastVisit)
+        return b.lastVisit - a.lastVisit;
+    if (a.visits !== b.visits)
+        return b.visits - a.visits;
+    return a.url < b.url ? -1 : a.url > b.url ? 1 : 0;
+}
+/**
+ * The list after a committed navigation to `url`. Returns `entries`
+ * *unchanged* — the same reference — when the URL is not one we store, which
+ * is what lets the caller write the file only when something actually
+ * changed.
+ *
+ * Eviction is by rank, so the least recently visited entry is the one that
+ * goes; a busy day cannot push out the address visited an hour ago in favour
+ * of one visited last year.
+ */
+function recordVisit(entries, url, now) {
+    if (!isStorableUrl(url) || !(0, exports.isStamp)(now))
+        return entries;
+    const previous = entries.find(e => e.url === url);
+    const next = {
+        url,
+        visits: previous ? previous.visits + 1 : 1,
+        lastVisit: now,
+    };
+    return [next, ...entries.filter(e => e.url !== url)].sort(byRank).slice(0, exports.HISTORY_MAX);
+}
+/**
+ * The rows to offer for what has been typed: case-insensitive substring
+ * against the whole URL, in rank order. An empty query matches everything,
+ * so pressing Down in an empty field offers the most recent addresses.
+ */
+function matchHistory(entries, query, limit = exports.HISTORY_SUGGESTIONS) {
+    const needle = query.trim().toLowerCase();
+    return entries
+        .filter(e => e.url.toLowerCase().includes(needle))
+        .sort(byRank)
+        .slice(0, Math.max(0, limit));
+}

package/out/shared/ipcPayloads.js CHANGED Viewed

@@ -6,10 +6,12 @@ exports.parseInputEvent = parseInputEvent;
 exports.parseDeviceScaleFactor = parseDeviceScaleFactor;
 exports.parseSettings = parseSettings;
 exports.parseMode = parseMode;
+exports.parseTabId = parseTabId;
 exports.parseUiState = parseUiState;
 exports.parseScrollPos = parseScrollPos;
 exports.parseScrollRequest = parseScrollRequest;
 exports.parseScrollReport = parseScrollReport;
+const presets_1 = require("./presets");
 const types_1 = require("./types");
 /**
  * Parsers for everything the renderer sends main over IPC. Each returns a
@@ -99,7 +101,7 @@ function parseDeviceScaleFactor(raw) {
     return raw;
 }
 /**
- * Copies exactly the five known keys; the numbers must be finite and
+ * Copies exactly the known keys; the numbers must be finite and
  * positive. A missing `agentControl` means false and a missing `updateCheck`
  * means true (the pre-feature wire shapes); any non-boolean value is refused,
  * never coerced.
@@ -121,13 +123,43 @@ function parseSettings(raw) {
     const lastUpdateCheck = raw.lastUpdateCheck ?? 0;
     if (!isFiniteNumber(lastUpdateCheck) || lastUpdateCheck < 0)
         return null;
-    return { hostDiagonalInches, hostNits, agentControl, updateCheck, lastUpdateCheck };
+    const recordHistory = raw.recordHistory ?? true;
+    if (typeof recordHistory !== 'boolean')
+        return null;
+    // A missing `split` is the pre-feature wire shape and means the default.
+    // Out of band it is refused rather than clamped: `loadSettings` forgives a
+    // hand-edited file because it must, but the renderer clamps before it
+    // sends, so a bad ratio arriving here is a bug worth surfacing.
+    const split = raw.split ?? presets_1.DEFAULT_SETTINGS.split;
+    if (!isFiniteNumber(split) || split < presets_1.SPLIT_MIN || split > presets_1.SPLIT_MAX)
+        return null;
+    // Same shape for the tab cap: absent is the pre-tabs wire shape and means
+    // the default, and an out-of-band or fractional count is refused rather
+    // than clamped — the Settings input is a bounded integer field, so one
+    // arriving here is a bug rather than a hand-edited file.
+    const maxTabs = raw.maxTabs ?? presets_1.DEFAULT_SETTINGS.maxTabs;
+    if (!isFiniteNumber(maxTabs) || !Number.isInteger(maxTabs) || maxTabs < presets_1.MAX_TABS_MIN || maxTabs > presets_1.MAX_TABS_MAX)
+        return null;
+    return { hostDiagonalInches, hostNits, agentControl, updateCheck, lastUpdateCheck, recordHistory, split, maxTabs };
 }
 function parseMode(raw) {
     return raw === 'url' || raw === 'image' ? raw : null;
 }
 /** Longest preset/profile id the UI-state mirror will store. */
 const MAX_UI_ID = 64;
+/**
+ * Longest tab id main will act on. Ids are minted main-side (`tab-N`), so a
+ * renderer message naming one is only ever echoing what it was told; the bound
+ * is there because the renderer is still the one sending it. An id no session
+ * carries resolves to nothing in the manager and the command is dropped, so
+ * shape is all this has to check.
+ */
+const MAX_TAB_ID = 64;
+function parseTabId(raw) {
+    if (typeof raw !== 'string' || raw.length === 0 || raw.length > MAX_TAB_ID)
+        return null;
+    return raw;
+}
 /**
  * The renderer's UI-state report (`IPC.uiState`), mirrored main-side so the
  * agent-control server can answer `status` without a renderer round-trip.
@@ -145,6 +177,12 @@ function parseUiState(raw) {
     if (!isRecord(raw))
         return null;
     const { presetId, profileId, viewMode, mode } = raw;
+    // Required, unlike `panes` below: there is no sane default for "which tab
+    // this describes", and guessing would reintroduce exactly the misattribution
+    // the field exists to stop.
+    const tabId = parseTabId(raw.tabId);
+    if (tabId === null)
+        return null;
     if (typeof presetId !== 'string' || presetId.length === 0 || presetId.length > MAX_UI_ID)
         return null;
     if (typeof profileId !== 'string' || profileId.length === 0 || profileId.length > MAX_UI_ID)
@@ -160,6 +198,7 @@ function parseUiState(raw) {
     if (panes !== 'both' && panes !== 'target')
         return null;
     return {
+        tabId,
         presetId,
         profileId,
         viewMode,

package/out/shared/presets.js CHANGED Viewed

@@ -1,15 +1,26 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.PANEL_PROFILES = exports.SCREEN_PRESETS = exports.DEFAULT_SETTINGS = exports.MAX_VIEWPORT = void 0;
+exports.PANEL_PROFILES = exports.SCREEN_PRESETS = exports.DEFAULT_SETTINGS = exports.MAX_TABS_MAX = exports.MAX_TABS_MIN = exports.SPLIT_MAX = exports.SPLIT_MIN = exports.MAX_VIEWPORT = void 0;
 exports.findPreset = findPreset;
 exports.findProfile = findProfile;
 exports.MAX_VIEWPORT = 4096;
+/** The loose sanity band for `Settings.split`; the clamp that matters is the
+ *  drag's 240px-a-side floor, which only the renderer knows the pixels for. */
+exports.SPLIT_MIN = 0.1;
+exports.SPLIT_MAX = 0.9;
+/** The band `Settings.maxTabs` is held to. Below two there is nothing to tab
+ *  between; above thirty-two the process count is a problem on any machine. */
+exports.MAX_TABS_MIN = 2;
+exports.MAX_TABS_MAX = 32;
 exports.DEFAULT_SETTINGS = {
     hostDiagonalInches: 27,
     hostNits: 500,
     agentControl: false,
     updateCheck: true,
     lastUpdateCheck: 0,
+    recordHistory: true,
+    split: 0.5,
+    maxTabs: 12,
 };
 exports.SCREEN_PRESETS = [
     // Laptops — ordered largest to smallest panel, then the denser 1080p outlier.

package/out/shared/tabList.js ADDED Viewed

@@ -0,0 +1,46 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.closeTab = closeTab;
+exports.canAddTab = canAddTab;
+exports.tabTitle = tabTitle;
+/**
+ * Removes a tab and says which one should take focus. Closing the active tab
+ * moves right, because the tab that took its screen position is the one the
+ * eye is already on; at the end of the strip there is nothing to the right, so
+ * it moves left.
+ */
+function closeTab(tabs, closeId, activeId) {
+    const index = tabs.findIndex(t => t.id === closeId);
+    if (index === -1)
+        return { tabs, activeId };
+    const next = tabs.filter(t => t.id !== closeId);
+    if (next.length === 0)
+        return { tabs: next, activeId: null };
+    if (closeId !== activeId)
+        return { tabs: next, activeId };
+    const neighbour = next[Math.min(index, next.length - 1)];
+    return { tabs: next, activeId: neighbour.id };
+}
+function canAddTab(count, max) {
+    return count < max;
+}
+/**
+ * What the strip shows. The port is kept deliberately: two local dev servers
+ * differ only by it, and a host-only label would render them identically.
+ */
+function tabTitle(url, pageTitle) {
+    if (pageTitle.trim() !== '')
+        return pageTitle;
+    // `about:blank` is where every session starts and what an unused tab still
+    // holds, so it is the empty tab spelled in Chromium's words rather than an
+    // address anyone typed. It has no host, and falling through to the raw
+    // string labels a fresh tab `about:blank` — which the screenshot caught.
+    if (url.trim() === '' || url.trim() === 'about:blank')
+        return 'New tab';
+    try {
+        return new URL(url).host || url;
+    }
+    catch {
+        return url;
+    }
+}

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "getobsrv",
-  "version": "0.10.0",
+  "version": "0.12.0",
   "description": "See your site the way 1x screens see it",
   "main": "./out/main/index.js",
   "bin": {

package/skills/obsrv-screens/SKILL.md CHANGED Viewed

@@ -61,6 +61,13 @@ when the PNG is taken. Reaching for `obsrv_snap` after a scroll works only
 when the app is already on that exact URL (it answers `navigated: false`);
 snapping a different URL is a fresh load and lands back at the top.
+The app can hold several sessions open as tabs, and both tools act on the one
+in **front** — resolved per command, so the user can move it under you. Each
+result names the tab (`tabId`, `tabIndex`); if you drive over several calls and
+the state has to hold, check that `tabId` did not change rather than assuming
+it. You cannot name another tab, and you cannot open, close or switch tabs —
+ask the user. An empty `tabId` is an app older than tabs, which has only one.
 ## The loop that catches real regressions
 1. Snap the dev URL across `--matrix laptop-768,android-65,1080p-24`, plus a