bridgerator 1.1.9 → 1.1.11

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "bridgerator",
-  "version": "1.1.9",
+  "version": "1.1.11",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "bridgerator",
-      "version": "1.1.9",
+      "version": "1.1.11",
       "license": "MIT",
       "dependencies": {
         "@modelcontextprotocol/sdk": "^1.29.0",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bridgerator",
-  "version": "1.1.9",
+  "version": "1.1.11",
   "description": "Connect ChatGPT to your Windows PC: files, search and terminal through an outbound-only bridge with folder limits. No inbound ports, no admin rights.",
   "license": "MIT",
   "type": "module",

package/src/bridge.mjs

@@ -251,6 +251,8 @@ export async function startEngine(cfg, log = console.log, { ask: askArg, recycle
 
   // bridgerator_setup: applied at once; after Apply on the PC only with askSettings on (setup-tool.mjs).
   const say = (t, isError) => ({ content: [{ type: 'text', text: t }], ...(isError ? { isError: true } : {}) });
+  // The person sees every settings change not made in the window (notify.settingsChanged, on by default; owner 26.10).
+  const toldSettings = (by, lines) => { if (lines.length && cfg.notify?.settingsChanged !== false) showNotice({ kind: 'settings_changed', by, what: lines }, log); };
   async function setup(args) {
     const v = validateSetup(args);
     if (v.error) return say(`Not applied: ${v.error}`, true);
@@ -261,7 +263,7 @@ export async function startEngine(cfg, log = console.log, { ask: askArg, recycle
       delete v.value.preset;
     }
     // Owner 21.10: applied at once; Apply on the PC only when the person turned on "Ask before AI changes settings".
-    if (cfg.askSettings !== true) { await applyValues(v.value, 'bridgerator_setup'); return say('Applied on the PC.'); }
+    if (cfg.askSettings !== true) { const d = diffSetup(cfg, cfg.confirmIni, v.value).short; await applyValues(v.value, 'bridgerator_setup'); toldSettings('ai', d ? [d] : []); return say('Applied on the PC.'); }
     // A diff that fits the notification gets Apply there; a longer one is reviewed in the window.
     const diff = diffSetup(cfg, cfg.confirmIni, v.value);
     let r;
@@ -293,12 +295,13 @@ export async function startEngine(cfg, log = console.log, { ask: askArg, recycle
     const risky = v.changes.filter(ask1), safe = v.changes.filter((c) => !ask1(c));
     for (const c of safe) await rpc('setSetting', { ...c, via: 'bridgerator_settings' });
     const done = safe.map((c) => describe(cur, c));
+    toldSettings('ai', done);
     if (!risky.length) return say(`Applied: ${done.join('; ')}`);
     const lines = risky.map((c) => describe(cur, c)), short = lines.join(' · ');
     const r = short.length <= 180
       ? await gate.askPerson('settings', { head: 'ChatGPT wants to change Bridgerator settings', text: short, buttons: [['allow', 'Apply', ''], ['deny', 'Cancel', 'deny']], secs: 120, auto: 'auto-cancel', ...popup() })
       : await gate.askPerson('settings', { review: true, lines });
-    if (r === 'allow') { for (const c of risky) await rpc('setSetting', { ...c, via: 'bridgerator_settings' }); return say(`Applied${done.length ? `: ${done.join('; ')};` : ':'} after the person's OK: ${lines.join('; ')}`); }
+    if (r === 'allow') { for (const c of risky) await rpc('setSetting', { ...c, via: 'bridgerator_settings' }); toldSettings('ai', lines); return say(`Applied${done.length ? `: ${done.join('; ')};` : ':'} after the person's OK: ${lines.join('; ')}`); }
     return { ...say(`${done.length ? `Applied: ${done.join('; ')}. ` : ''}Not applied (the person at the PC ${r === 'cooldown' ? 'declined a few minutes ago' : 'cancelled or didn\'t answer'}): ${lines.join('; ')}`), notConfirmed: true };
   }
 
@@ -504,7 +507,7 @@ export async function startEngine(cfg, log = console.log, { ask: askArg, recycle
   const bannerNow = () => { const b = planBanner(out.plan); return { ...b, visible: bannerVisible(b, cfg.planBannerDismissed) }; };
   out.banner = bannerNow;
   out.onPlanMsg = (m) => { out.plan = readPlan(m); out.onPlan?.(); };
-  const BOOL_NOTIFY = ['sound', 'offline', 'queueDone', 'queueStalled', 'limit'];
+  const BOOL_NOTIFY = ['sound', 'offline', 'queueDone', 'queueStalled', 'settingsChanged', 'limit'];
 
   // Requests of the Bridgerator window (tray helper). Made by the person at the PC: no extra prompt.
   async function rpc(method, p = {}, paths = []) {
@@ -654,21 +657,27 @@ export async function startEngine(cfg, log = console.log, { ask: askArg, recycle
     const den = list(j.deniedDirectories), denNow = cfg.deniedDirectories || [];
     const denRemoved = denNow.filter((x) => !den.some((y) => y.toLowerCase() === x.toLowerCase()));
     if (!changes.length && JSON.stringify(den) === JSON.stringify(denNow)) return;
-    const risky = changes.filter((c) => widens(cur, c)), safe = changes.filter((c) => !widens(cur, c));
+    // Owner 26.10: a file edit applies at once like an AI change; Apply on the PC only with askSettings on.
+    const ask1 = (c) => cfg.askSettings === true && widens(cur, c);
+    const risky = changes.filter(ask1), safe = changes.filter((c) => !ask1(c));
+    const denAsk = cfg.askSettings === true && denRemoved.length > 0;
     const back = {};
     const top = (key) => (key === 'terminal' ? 'disabledTools' : key.split('.')[0]);
     for (const c of safe) { try { await rpc('setSetting', { ...c, via: 'settings file' }); } catch (e) { log(`[bridgerator] settings file: ${c.key} not applied (${e.message})`); back[top(c.key)] = cfg[top(c.key)]; } }
-    if (!denRemoved.length) { cfg.deniedDirectories = den; policy = makePolicy(); }
-    const lines = [...risky.map((c) => describe(cur, c)), ...(denRemoved.length ? [`Excluded folders: ${denRemoved.map((x) => `−${x}`).join(', ')}`] : [])];
+    if (!denAsk) { cfg.deniedDirectories = den; policy = makePolicy(); }
+    const denLine = denRemoved.length ? [`Excluded folders: ${denRemoved.map((x) => `−${x}`).join(', ')}`] : [];
+    toldSettings('file', [...safe.filter((c) => !(top(c.key) in back)).map((c) => describe(cur, c)), ...(denAsk ? [] : denLine)]);
+    const lines = [...risky.map((c) => describe(cur, c)), ...(denAsk ? denLine : [])];
     if (lines.length) {
       const r = await gate.askPerson('file', { head: 'Bridgerator settings file changed', text: clip(`Apply: ${lines.join(' · ')}?`, '', 180), buttons: [['allow', 'Apply', ''], ['deny', 'Keep', 'deny']], secs: 120, auto: 'auto-keep', ...popup() });
       if (r === 'allow') {
         for (const c of risky) { try { await rpc('setSetting', { ...c, via: 'settings file' }); } catch (e) { log(`[bridgerator] settings file: ${c.key} not applied (${e.message})`); back[top(c.key)] = cfg[top(c.key)]; } }
-        if (denRemoved.length) { cfg.deniedDirectories = den; policy = makePolicy(); }
+        if (denAsk) { cfg.deniedDirectories = den; policy = makePolicy(); }
+        toldSettings('file', lines);
         log(`[bridgerator] settings file change applied on the PC: ${lines.join('; ')}`);
       } else {
         for (const c of risky) back[top(c.key)] = cfg[top(c.key)];
-        if (denRemoved.length) back.deniedDirectories = denNow;
+        if (denAsk) back.deniedDirectories = denNow;
         log(`[bridgerator] settings file change kept out (${r}): ${lines.join('; ')}`);
       }
     }

package/src/cli.mjs

@@ -337,7 +337,9 @@ const defaultDenied = () => [];
 /** The setup summary's "excluded" line: only what really applies (sensitive folders only when that switch is on). */
 export const excludedLine = (cfg) => [...(cfg.deniedDirectories || []), `built-in: Bridgerator's own folders${cfg.protectSensitive ? ', AppData, credentials (.ssh, .aws, …), Startup, PowerShell profiles' : ''}`].join(', ');
 /** 'ru' when the Windows display language or regional format is Russian, else 'en'. */
-export function osLanguage(locales = [Intl.DateTimeFormat().resolvedOptions().locale, process.env.LANG]) {
+// Node's Intl gives only the regional format: the display language comes from Windows, as in install.ps1.
+const uiCulture = () => { try { return process.platform === 'win32' ? ps('[Globalization.CultureInfo]::CurrentUICulture.Name').trim() : ''; } catch { return ''; } };
+export function osLanguage(locales = [Intl.DateTimeFormat().resolvedOptions().locale, process.env.LANG, uiCulture()]) {
   return locales.some((l) => /^ru([-_.]|$)/i.test(String(l || ''))) ? 'ru' : 'en';
 }
 const keptDenied = (list) => (Array.isArray(list) && !(list.length === 1 && process.env.OneDrive && list[0].toLowerCase() === process.env.OneDrive.toLowerCase()) ? list : []);

package/src/notice.mjs

@@ -35,6 +35,12 @@ export function noticeText(n) {
     if (!title) return undefined;
     const more = int(n.more) || 0;
     t = ['ChatGPT may be waiting for you', `No activity on the queue task "${title}"${more ? ` and ${more} more` : ''} for 10 minutes. Open the ChatGPT chat: it may be asking you to allow an action, or show an error such as "model is at capacity" — press Retry there or pick another model.`];
+  } else if (kind === 'settings_changed') {
+    // The bridge's own change lines (describe / diffSetup): paths and labels, no relay text.
+    const what = Array.isArray(n.what) ? n.what.map((w) => String(w).replace(/[\u0000-\u001f\u007f-\u009f]/g, ' ').trim()).filter(Boolean) : [];
+    if (!what.length || !['ai', 'file'].includes(n.by)) return undefined;
+    const all = what.join(' · ');
+    t = ['Bridgerator settings changed', `${n.by === 'ai' ? 'By the AI' : 'In the settings file'}: ${all.length > 200 ? `${all.slice(0, 199)}…` : all}. Turn this notice off in the Bridgerator window.`];
   } else if (kind === 'protection_off') {
     // Internal list only (the bridge's own labels), never relay text.
     const ok = new Set(['confirmations', 'your own patterns', ...Object.values(LABEL).map((l) => `confirmations for ${l}`)]);

package/src/queue.mjs

@@ -24,7 +24,7 @@ export const QUEUE_TOOL_DEFS = [
   { name: 'queue_list', description: `List the task queue the person keeps on this PC (a checklist of things to do): counts and items (id, title, status, note, worker, updatedAt, from, resultPath), optionally only one status.`, inputSchema: { type: 'object', additionalProperties: false, properties: { status: { type: 'string', enum: STATUSES } } } },
   { name: 'queue_next', description: `Take the next item of the task queue: returns the first "todo" item and marks it "doing" (nothing when the queue is empty). An item left "doing" for 30 minutes without an update (another chat stopped) is returned first, with resumed: true. Several chats can work through one queue at once; each call gets a different item. Pass worker = a short name for this chat (e.g. its title) so the person sees who is on it. ${RULE}`, inputSchema: { type: 'object', additionalProperties: false, properties: { worker: STR(80), id: { type: 'string', description: 'take this "todo" item instead of the first one' } } } },
   { name: 'queue_update', description: `Set an item's status (doing / done / failed / skipped / todo) with a short result note.`, inputSchema: { type: 'object', additionalProperties: false, required: ['id', 'status'], properties: { id: ID, status: { type: 'string', enum: STATUSES }, note: STR(MAX_NOTE) } } },
-  { name: 'queue_add', description: `Add steps to the task queue (e.g. when planning a bigger job with the person): one { title, details?, paths?, from?, resultPath? } or items: [...] (1–100). Another AI handing work to ChatGPT sets from (its name) and resultPath (a file for the result), then reads the outcome with queue_list.`, inputSchema: { type: 'object', additionalProperties: false, properties: { title: STR(MAX_TITLE), details: STR(MAX_DETAILS), paths: PATHS, ...HANDOFF, items: { type: 'array', minItems: 1, maxItems: 100, items: { type: 'object', additionalProperties: false, required: ['title'], properties: { title: STR(MAX_TITLE), details: STR(MAX_DETAILS), paths: PATHS, ...HANDOFF } } } } } },
+  { name: 'queue_add', description: `Add steps to the task queue (e.g. when planning a bigger job with the person): one { title, details?, paths?, from?, resultPath? } or items: [...] (1–100). Another AI handing work to ChatGPT sets from (its name) and resultPath (a file for the result), then reads the outcome with queue_list. A step whose title matches an open (todo / doing) item is not added again — the reply names that item; allowDuplicate: true adds it anyway.`, inputSchema: { type: 'object', additionalProperties: false, properties: { title: STR(MAX_TITLE), details: STR(MAX_DETAILS), paths: PATHS, ...HANDOFF, allowDuplicate: { type: 'boolean' }, items: { type: 'array', minItems: 1, maxItems: 100, items: { type: 'object', additionalProperties: false, required: ['title'], properties: { title: STR(MAX_TITLE), details: STR(MAX_DETAILS), paths: PATHS, ...HANDOFF } } } } } },
   { name: 'queue_clear', description: 'Remove finished items from the task queue: status done, failed or skipped. status "all" removes every item and needs the person\'s OK on the PC.', inputSchema: { type: 'object', additionalProperties: false, properties: { status: { type: 'string', enum: ['done', 'failed', 'skipped', 'all'] } } } },
   { name: 'queue_edit', description: 'Change the title, details, paths or result file of a queue item.', inputSchema: { type: 'object', additionalProperties: false, required: ['id'], properties: { id: ID, title: STR(MAX_TITLE), details: STR(MAX_DETAILS), paths: PATHS, resultPath: HANDOFF.resultPath } } },
   { name: 'queue_reorder', description: 'Put queue items in a new order: ids first, in this order; the rest keep their order after them.', inputSchema: { type: 'object', additionalProperties: false, required: ['ids'], properties: { ids: { type: 'array', maxItems: MAX_ITEMS, items: ID } } } },
@@ -190,7 +190,18 @@ export function queueTool(q, name, args = {}) {
       case 'queue_list': return ok({ counts: q.counts(), items: q.list(args.status).map(({ id, title, status, note, worker, updatedAt, details, paths, from, resultPath }) => ({ id, title, status, ...(note ? { note } : {}), ...(worker ? { worker } : {}), updatedAt, ...(details ? { details } : {}), ...(paths ? { paths } : {}), ...(from ? { from } : {}), ...(resultPath ? { resultPath } : {}) })) });
       case 'queue_next': { const it = q.next(args); return ok(it ? { item: it } : { item: null, message: 'The queue is empty — tell the person and stop.' }); }
       case 'queue_update': return ok({ item: q.update(args) });
-      case 'queue_add': return args.items ? ok({ items: q.addMany(args.items) }) : ok({ item: q.add(args) });
+      case 'queue_add': {
+        // A second chat given the same request re-added the whole set (E2E 26.10): an open item with the same
+        // title is skipped and named. Only the AI's tool; the window adds what the person types.
+        const key = (t) => (typeof t === 'string' ? t.trim().toLowerCase() : null);
+        const open = new Map(q.list().filter((x) => x.status === 'todo' || x.status === 'doing').map((x) => [key(x.title), x.id]));
+        const want = Array.isArray(args.items) ? args.items : [args], keep = [], skipped = [];
+        for (const x of want) { const d = args.allowDuplicate === true ? undefined : open.get(key(x?.title)); if (d) skipped.push({ title: x.title, duplicate_of: d }); else keep.push(x); }
+        const why = skipped.length ? { skipped, message: 'Already in the queue (open), not added again: work on those items (queue_list / queue_next), or pass allowDuplicate: true to add them anyway.' } : {};
+        if (!keep.length && skipped.length) return ok({ added: 0, ...why });
+        if (args.items === undefined) return ok({ item: q.add(args) });
+        return ok({ items: q.addMany(Array.isArray(args.items) ? keep : args.items), ...why });
+      }
       case 'queue_clear': { if (args.status === 'all') throw new Error('clearing every item needs the person\'s OK'); const gone = q.clear({ status: args.status || 'finished' }); return ok({ removed: gone.length, counts: q.counts() }); }
       case 'queue_edit': return ok({ item: q.edit(args) });
       case 'queue_reorder': return ok({ items: q.reorder(args) });

package/src/settings-tool.mjs

@@ -21,11 +21,11 @@ export const DEFAULTS = {
   preset: 'full', terminal: true, allowedDirectories: [], readOnlyDirectories: [], protectSensitive: false,
   confirm: { enabled: false, ...readSettings('').ask, timeoutSeconds: 120 },   // no confirm.ini = the defaults per category
   recycleBin: true, snapshots: { ...SNAP_DEFAULTS }, fuse: { ...FUSE_DEFAULTS }, keepAwake: true, askSettings: false,
-  theme: 'system', language: 'en', popupStyle: 'toast', notify: { sound: true, offline: true, queueDone: true, queueStalled: true, limit: true },
+  theme: 'system', language: 'en', popupStyle: 'toast', notify: { sound: true, offline: true, queueDone: true, queueStalled: true, settingsChanged: true, limit: true },
   deviceName: '', autostart: true, localHistory: false, telemetryEnabled: true,
   ...EXTRA.defaults(),
 };
-const NOTIFY = ['sound', 'offline', 'queueDone', 'queueStalled', 'limit'];
+const NOTIFY = ['sound', 'offline', 'queueDone', 'queueStalled', 'settingsChanged', 'limit'];
 const dirs = { type: 'array', maxItems: 20, items: { type: 'string' } };
 export const SETTINGS_TOOL_DEF = {
   name: 'bridgerator_settings',

package/ui/app.js

@@ -334,7 +334,7 @@ P.notifications = () => {
   return `<main class="notifications">${bannerHtml()}
 <div class="head"><h1>${esc(t('notify.title'))}</h1><div class="sub">${esc(t('notify.sub'))}</div></div>
 <div class="styles" role="radiogroup" aria-label="${esc(t('aria.style'))}">${style('toast')}${style('popup')}</div>
-<div class="nsplit"><section class="card" aria-label="${esc(t('aria.options'))}">${sw('sound', 'notify.sound')}${sw('offline', 'notify.offline')}${sw('queueDone', 'notify.queueDone')}${sw('queueStalled', 'notify.queueStalled')}${sw('limit', 'notify.limit')}</section>
+<div class="nsplit"><section class="card" aria-label="${esc(t('aria.options'))}">${sw('sound', 'notify.sound')}${sw('offline', 'notify.offline')}${sw('queueDone', 'notify.queueDone')}${sw('queueStalled', 'notify.queueStalled')}${sw('settingsChanged', 'notify.settingsChanged')}${sw('limit', 'notify.limit')}</section>
 <section class="preview" aria-label="${esc(t('notify.preview'))}"><div class="bar-top"><span class="b">${esc(t('notify.preview'))}</span><div class="seg" role="radiogroup" aria-label="${esc(t('aria.previewTheme'))}">${pv('light')}${pv('dark')}</div></div>
 <div class="toast ${previewTheme}"><div class="app">${LOGO16}<span class="grow">Bridgerator</span><span>⋯</span></div><div class="h">${esc(t('notify.pHead'))}</div><div class="c">${esc(t('notify.pCmd'))}</div><div class="m">${esc(t('notify.pMeta', { name: state.name || 'PC' }))}</div>
 <div class="btns"><span>${esc(t('notify.allow'))}</span><span>${esc(t('notify.allow10'))}</span><span>${esc(t('notify.deny'))}</span></div></div>

package/ui/i18n/en.json

@@ -276,6 +276,7 @@
   "sound": "Play a sound",
   "offline": "When the PC goes offline",
   "queueDone": "When a queue task is done",
+  "settingsChanged": "When the AI or the settings file changes Bridgerator settings",
   "queueStalled": "When a queue task stalls (ChatGPT may be waiting for your OK or show an error in the chat)",
   "limit": "Monthly limit warnings",
   "preview": "Preview",

package/ui/i18n/ru.json

@@ -276,6 +276,7 @@
   "sound": "Звук",
   "offline": "Когда ПК уходит из сети",
   "queueDone": "Когда задача из очереди готова",
+  "settingsChanged": "Когда ИИ или файл настроек меняет настройки Bridgerator",
   "queueStalled": "Когда задача из очереди зависла (ChatGPT может ждать вашего «Разрешить» или показать ошибку в чате)",
   "limit": "Предупреждения о лимите",
   "preview": "Пример",