pinokiod 5.1.34 → 5.1.36

package/kernel/git.js

@@ -84,6 +84,17 @@ class Git {
     const offM = pad2(abs % 60)
     return `${y}-${m}-${day}T${hh}:${mm}:${ss}${sign}${offH}:${offM}`
   }
+  normalizeRepoPath(rawPath) {
+    if (typeof rawPath !== "string") return "."
+    let value = rawPath.trim()
+    if (!value) return "."
+    value = value.replace(/\\/g, "/").replace(/\/{2,}/g, "/")
+    if (value === "." || value === "./") return "."
+    if (value.startsWith("./")) {
+      value = value.slice(2)
+    }
+    return value || "."
+  }
   upsertCommitMeta(repoUrlNorm, sha, meta) {
     if (!repoUrlNorm || typeof repoUrlNorm !== "string") return false
     if (!this.isCommitSha(sha)) return false
@@ -231,7 +242,7 @@ class Git {
     const repos = []
     for (const repo of rawRepos) {
       if (!repo) continue
-      const pathVal = typeof repo.path === "string" && repo.path.length > 0 ? repo.path : "."
+      const pathVal = this.normalizeRepoPath(repo.path)
       let remote = typeof repo.remote === "string" && repo.remote.length > 0 ? repo.remote : null
       if (!remote) remote = typeof repo.repo === "string" && repo.repo.length > 0 ? repo.repo : null
       const commit = typeof repo.commit === "string" && repo.commit.length > 0 ? repo.commit : null
@@ -559,6 +570,40 @@ class Git {
     await this.saveManifest()
     return true
   }
+  async deleteCheckpoint(remoteKey, checkpointId) {
+    if (!remoteKey || checkpointId == null) return { ok: false, error: "invalid" }
+    const apps = this.apps()
+    const entry = apps[remoteKey]
+    if (!entry || !Array.isArray(entry.checkpoints)) return { ok: false, error: "not_found" }
+    const idStr = String(checkpointId)
+    const idx = entry.checkpoints.findIndex((c) => c && String(c.id) === idStr)
+    if (idx < 0) return { ok: false, error: "not_found" }
+    const record = entry.checkpoints[idx]
+    entry.checkpoints.splice(idx, 1)
+    await this.saveManifest()
+    const hash = record && record.hash ? String(record.hash) : null
+    let fileDeleted = false
+    if (hash) {
+      let stillUsed = false
+      for (const entry of Object.values(apps)) {
+        if (!entry || !Array.isArray(entry.checkpoints)) continue
+        if (entry.checkpoints.some((c) => c && String(c.hash) === hash)) {
+          stillUsed = true
+          break
+        }
+      }
+      if (!stillUsed) {
+        const filePath = this.checkpointFilePath(hash)
+        if (filePath) {
+          try {
+            await fs.promises.rm(filePath, { force: true })
+            fileDeleted = true
+          } catch (_) {}
+        }
+      }
+    }
+    return { ok: true, hash, fileDeleted }
+  }
   async logCheckpointRestore(event) {
     const logEntry = {
       ts: Date.now(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pinokiod",
-  "version": "5.1.34",
+  "version": "5.1.36",
   "description": "",
   "main": "index.js",
   "scripts": {

package/server/index.js

@@ -5205,6 +5205,38 @@ class Server {
       }
       res.json({ ok: true, redirect: `/p/${encodeURIComponent(folder)}` })
     }))
+    this.app.post("/checkpoints/delete", ex(async (req, res) => {
+      const remoteRaw = typeof req.body.remote === 'string'
+        ? req.body.remote.trim()
+        : (typeof req.query.remote === 'string' ? req.query.remote.trim() : '')
+      const remoteKeyRaw = typeof req.body.remoteKey === 'string'
+        ? req.body.remoteKey.trim()
+        : (typeof req.query.remoteKey === 'string' ? req.query.remoteKey.trim() : '')
+      const snapshotRaw = Object.prototype.hasOwnProperty.call(req.body || {}, "snapshotId")
+        ? req.body.snapshotId
+        : (Object.prototype.hasOwnProperty.call(req.query || {}, "snapshotId") ? req.query.snapshotId : "")
+      const snapshotId = snapshotRaw === 'latest'
+        ? ''
+        : (snapshotRaw == null ? '' : String(snapshotRaw))
+      const remoteKey = remoteKeyRaw || (remoteRaw ? this.kernel.git.normalizeRemote(remoteRaw) : '')
+      if (!snapshotId || !remoteKey) {
+        res.status(400).json({ ok: false, error: "Missing parameters" })
+        return
+      }
+      const result = await this.kernel.git.deleteCheckpoint(remoteKey, snapshotId)
+      if (!result || !result.ok) {
+        res.status(404).json({ ok: false, error: "Snapshot not found" })
+        return
+      }
+      res.json({
+        ok: true,
+        deleted: {
+          id: snapshotId,
+          hash: result.hash || null,
+          fileDeleted: !!result.fileDeleted
+        }
+      })
+    }))
     this.app.get("/checkpoints/restore/:workspace/:snapshotId", ex(async (req, res) => {
       const workspace = typeof req.params.workspace === 'string' ? req.params.workspace : ''
       const snapshotId = req.params.snapshotId
@@ -8652,6 +8684,56 @@ class Server {
     }))
 
 
+    this.app.get("/info/apps", ex(async (req, res) => {
+      const apps = []
+      try {
+        const apipath = this.kernel.path("api")
+        const entries = await fs.promises.readdir(apipath, { withFileTypes: true })
+        for (const entry of entries) {
+          let type
+          try {
+            type = await Util.file_type(apipath, entry)
+          } catch (typeErr) {
+            console.warn('Failed to inspect api entry', entry.name, typeErr)
+            continue
+          }
+          if (!type || !type.directory) {
+            continue
+          }
+          try {
+            const meta = await this.kernel.api.meta(entry.name)
+            apps.push({
+              name: entry.name,
+              title: meta && meta.title ? meta.title : entry.name,
+              description: meta && meta.description ? meta.description : '',
+              icon: meta && meta.icon ? meta.icon : "/pinokio-black.png"
+            })
+          } catch (metaError) {
+            console.warn('Failed to load app metadata', entry.name, metaError)
+            apps.push({
+              name: entry.name,
+              title: entry.name,
+              description: '',
+              icon: "/pinokio-black.png"
+            })
+          }
+        }
+      } catch (enumerationError) {
+        console.warn('Failed to enumerate api apps for url dropdown', enumerationError)
+      }
+
+      apps.sort((a, b) => {
+        const at = (a.title || a.name || '').toLowerCase()
+        const bt = (b.title || b.name || '').toLowerCase()
+        if (at < bt) return -1
+        if (at > bt) return 1
+        return (a.name || '').localeCompare(b.name || '')
+      })
+
+      res.json({ apps })
+    }))
+
+
     this.app.get("/info/procs", ex(async (req, res) => {
       await this.kernel.processes.refresh()
 

package/server/public/nav.js

@@ -25,8 +25,25 @@ document.addEventListener("DOMContentLoaded", () => {
     return;
   }
 
+  const homeIcon = homeLink ? homeLink.querySelector("img.icon") : null;
+  const ensureHomeExpandIcon = () => {
+    if (!homeLink || !homeIcon) {
+      return null;
+    }
+    let icon = homeLink.querySelector(".home-expand-icon");
+    if (!icon) {
+      icon = document.createElement("i");
+      icon.className = "fa-solid fa-expand home-expand-icon";
+      icon.setAttribute("aria-hidden", "true");
+      homeLink.appendChild(icon);
+    }
+    return icon;
+  };
+  ensureHomeExpandIcon();
+
   // Helper functions used during initial restore must be defined before use
-  const MIN_MARGIN = 8;
+  const MIN_MARGIN = 0;
+  const LEGACY_MARGIN = 8;
 
   function clampPosition(left, top, sizeOverride) {
     const rect = header.getBoundingClientRect();
@@ -168,19 +185,24 @@ document.addEventListener("DOMContentLoaded", () => {
   // Restore persisted or respect DOM state on load (per path, per session)
   const persisted = readPersisted();
   const restoreFromStorage = !!(persisted && persisted.minimized);
+  const hasStoredPosition = !!(persisted && Number.isFinite(persisted.left) && Number.isFinite(persisted.top));
+  const isLegacyDefault = hasStoredPosition
+    && Math.abs(persisted.left - LEGACY_MARGIN) < 0.5
+    && Math.abs(persisted.top - LEGACY_MARGIN) < 0.5;
+  const useStoredPosition = restoreFromStorage && hasStoredPosition && !isLegacyDefault;
   const domIsMinimized = header.classList.contains("minimized");
   if (restoreFromStorage || domIsMinimized) {
     header.classList.add("minimized");
     // Use minimized size for clamping/positioning
     const size = measureRect((clone) => { clone.classList.add("minimized"); });
-    const fallbackLeft = Math.max(MIN_MARGIN, window.innerWidth - size.width - MIN_MARGIN);
-    const fallbackTop = Math.max(MIN_MARGIN, window.innerHeight - size.height - MIN_MARGIN);
-    const left = restoreFromStorage && Number.isFinite(persisted.left) ? persisted.left : fallbackLeft;
-    const top = restoreFromStorage && Number.isFinite(persisted.top) ? persisted.top : fallbackTop;
+    const fallbackLeft = MIN_MARGIN;
+    const fallbackTop = MIN_MARGIN;
+    const left = useStoredPosition ? persisted.left : fallbackLeft;
+    const top = useStoredPosition ? persisted.top : fallbackTop;
     const clamped = clampPosition(left, top, size);
     state.lastLeft = clamped.left;
     state.lastTop = clamped.top;
-    state.hasCustomPosition = restoreFromStorage;
+    state.hasCustomPosition = useStoredPosition;
     state.minimized = true;
     // Apply immediately and once after layout settles
     applyPosition(clamped.left, clamped.top);
@@ -234,8 +256,8 @@ document.addEventListener("DOMContentLoaded", () => {
       clone.classList.add("minimized");
     });
 
-    const defaultLeft = Math.max(MIN_MARGIN, window.innerWidth - minimizedSize.width - MIN_MARGIN);
-    const defaultTop = Math.max(MIN_MARGIN, window.innerHeight - minimizedSize.height - MIN_MARGIN);
+    const defaultLeft = MIN_MARGIN;
+    const defaultTop = MIN_MARGIN;
     const targetLeft = state.hasCustomPosition ? state.lastLeft : defaultLeft;
     const targetTop = state.hasCustomPosition ? state.lastTop : defaultTop;
 

package/server/public/style.css

@@ -2978,7 +2978,6 @@ header.navheader.minimized {
   height: auto;
   max-height: none;
   padding: 4px 8px;
-  border-radius: 12px;
   background: var(--light-nav-bg);
   box-shadow: 0 8px 18px rgba(0, 0, 0, 0.14), 0 2px 6px rgba(0, 0, 0, 0.08);
   display: inline-flex;
@@ -3052,9 +3051,23 @@ header.navheader.minimized .home {
   padding: 0;
 }
 header.navheader.minimized .home .icon {
+  display: none;
   width: 24px;
   height: 24px;
 }
+header.navheader .home .home-expand-icon {
+  display: none;
+}
+header.navheader.minimized .home .home-expand-icon {
+  display: inline-flex;
+  width: 24px;
+  height: 24px;
+  align-items: center;
+  justify-content: center;
+  font-size: 16px;
+  line-height: 1;
+  color: inherit;
+}
 
 
 header.navheader.transitioning {

package/server/public/tab-link-popover.css

@@ -132,3 +132,37 @@ body.dark .tab-link-popover .tab-link-popover-footer:hover,
 body.dark .tab-link-popover .tab-link-popover-footer:focus-visible {
   background: rgba(147, 197, 253, 0.35);
 }
+.appcanvas > aside .menu-container .tab-link-popover-host {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+}
+.appcanvas > aside .menu-container .tab-link-popover-trigger {
+  display: inline-flex;
+  align-items: center;
+  justify-content: center;
+  width: 18px;
+  height: 18px;
+  border-radius: 6px;
+  flex: 0 0 auto;
+  color: inherit;
+  opacity: 0.55;
+  cursor: pointer;
+  transition: opacity 0.15s ease, background 0.15s ease, color 0.15s ease;
+}
+.appcanvas > aside .menu-container .tab-link-popover-trigger i {
+  font-size: 14px;
+}
+.appcanvas > aside .menu-container .frame-link:hover .tab-link-popover-trigger,
+.appcanvas > aside .menu-container .frame-link:focus-within .tab-link-popover-trigger {
+  opacity: 0.85;
+}
+.appcanvas > aside .menu-container .tab-link-popover-trigger:hover,
+.appcanvas > aside .menu-container .tab-link-popover-trigger:focus-visible {
+  background: rgba(15, 23, 42, 0.08);
+  opacity: 1;
+}
+body.dark .appcanvas > aside .menu-container .tab-link-popover-trigger:hover,
+body.dark .appcanvas > aside .menu-container .tab-link-popover-trigger:focus-visible {
+  background: rgba(148, 163, 184, 0.2);
+}

package/server/public/tab-link-popover.js

@@ -10,6 +10,40 @@
   let tabLinkRouterHttpsActive = null
   let tabLinkPeerInfoPromise = null
   let tabLinkPeerInfoExpiry = 0
+  const TAB_LINK_TRIGGER_CLASS = "tab-link-popover-trigger"
+  const TAB_LINK_TRIGGER_HOST_CLASS = "tab-link-popover-host"
+
+  const shouldAttachTabLinkTrigger = (link) => {
+    if (!link || !link.classList || !link.classList.contains("frame-link")) {
+      return false
+    }
+    if (!link.hasAttribute("href")) {
+      return false
+    }
+    const href = link.getAttribute("href")
+    return typeof href === "string" && href.trim().length > 0
+  }
+
+  const createTabLinkTrigger = () => {
+    const trigger = document.createElement("span")
+    trigger.className = TAB_LINK_TRIGGER_CLASS
+    trigger.setAttribute("role", "button")
+    trigger.setAttribute("tabindex", "0")
+    trigger.setAttribute("aria-label", "Open in browser")
+    trigger.innerHTML = '<i class="fa-solid fa-arrow-up-right-from-square"></i>'
+    return trigger
+  }
+
+  const ensureTabLinkTrigger = (link) => {
+    if (!shouldAttachTabLinkTrigger(link)) {
+      return
+    }
+    if (link.querySelector(`.${TAB_LINK_TRIGGER_CLASS}`)) {
+      return
+    }
+    link.classList.add(TAB_LINK_TRIGGER_HOST_CLASS)
+    link.appendChild(createTabLinkTrigger())
+  }
 
   const ensureTabLinkPopoverEl = () => {
     if (!tabLinkPopoverEl) {
@@ -38,7 +72,12 @@
           if (targetMode === "_self") {
             window.location.assign(url)
           } else {
-            window.open(url, "_blank", "browser")
+            const agent = document.body ? document.body.getAttribute("data-agent") : null
+            if (agent === "electron") {
+              window.open(url, "_blank", "browser")
+            } else {
+              window.open(url, "_blank")
+            }
 //            fetch("/go", {
 //              method: "POST",
 //              headers: {
@@ -1611,27 +1650,80 @@
     if (!container) {
       return
     }
+    if (container.dataset.tabLinkPopoverReady === "1") {
+      return
+    }
+    container.dataset.tabLinkPopoverReady = "1"
 
-    container.addEventListener("mouseover", (event) => {
-      const link = event.target.closest(".frame-link")
-      if (!link || !container.contains(link)) {
+    const ensureTriggers = (root) => {
+      if (!root || !root.querySelectorAll) {
         return
       }
-      renderTabLinkPopover(link, { requireAlternate: false })
+      root.querySelectorAll(".frame-link").forEach((link) => {
+        ensureTabLinkTrigger(link)
+      })
+    }
+
+    ensureTriggers(container)
+
+    const observer = new MutationObserver((mutations) => {
+      mutations.forEach((mutation) => {
+        mutation.addedNodes.forEach((node) => {
+          if (!node || node.nodeType !== 1) {
+            return
+          }
+          if (node.classList && node.classList.contains("frame-link")) {
+            ensureTabLinkTrigger(node)
+          }
+          if (node.querySelectorAll) {
+            node.querySelectorAll(".frame-link").forEach((link) => {
+              ensureTabLinkTrigger(link)
+            })
+          }
+        })
+      })
     })
+    observer.observe(container, { childList: true, subtree: true })
 
-    container.addEventListener("mouseout", (event) => {
-      const origin = event.target.closest(".frame-link")
-      if (!origin || !container.contains(origin)) {
+    const togglePopoverForLink = (link) => {
+      if (!link) {
         return
       }
-      const related = event.relatedTarget
       const popover = tabLinkPopoverEl || document.getElementById(TAB_LINK_POPOVER_ID)
-      if (related && (origin.contains(related) || (popover && popover.contains(related)))) {
+      if (tabLinkActiveLink === link && popover && popover.classList.contains("visible")) {
+        hideTabLinkPopover({ immediate: true })
         return
       }
-      hideTabLinkPopover()
-    })
+      renderTabLinkPopover(link, { requireAlternate: false })
+    }
+
+    const handleTriggerClick = (event) => {
+      const trigger = event.target.closest(`.${TAB_LINK_TRIGGER_CLASS}`)
+      if (!trigger || !container.contains(trigger)) {
+        return
+      }
+      event.preventDefault()
+      event.stopPropagation()
+      const link = trigger.closest(".frame-link")
+      togglePopoverForLink(link)
+    }
+
+    const handleTriggerKeydown = (event) => {
+      if (event.key !== "Enter" && event.key !== " ") {
+        return
+      }
+      const trigger = event.target.closest(`.${TAB_LINK_TRIGGER_CLASS}`)
+      if (!trigger || !container.contains(trigger)) {
+        return
+      }
+      event.preventDefault()
+      event.stopPropagation()
+      const link = trigger.closest(".frame-link")
+      togglePopoverForLink(link)
+    }
+
+    container.addEventListener("click", handleTriggerClick, true)
+    container.addEventListener("keydown", handleTriggerKeydown, true)
   }
 
   const handleGlobalPointer = (event) => {