pinokiod 8.0.106 → 8.0.107

package/kernel/vault/index.js

@@ -3407,7 +3407,6 @@ class Vault {
         status: PUBLIC_FILE_STATUS[row.status] || null
       }, this.locationForPath(row.path, row.source_id))),
       total: Number(result.total) || 0,
-      deduplicatable: Number(result.duplicateTotal) || 0,
       next_cursor: result.nextCursor || null
     }
   }

package/kernel/vault/registry_core.js

@@ -5691,18 +5691,9 @@ class RegistryCore {
       WHERE ${identityWhere}
         AND ${authorized.where.join(" AND ")}
     `).get(...identityValues, ...authorized.values).count) || 0
-    // Only paths still storing their own copy can free anything.
-    const duplicateTotal = Number(this.database.prepare(`
-      SELECT COUNT(*) AS count
-      FROM files child
-      WHERE ${identityWhere}
-        AND child.status = 'duplicate'
-        AND ${authorized.where.join(" AND ")}
-    `).get(...identityValues, ...authorized.values).count) || 0
     return {
       rows,
       total,
-      duplicateTotal,
       nextCursor: hasMore && last
         ? this.encodeCursor({
             sort: "file-locations",

package/package.json

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

package/server/public/vault.js

@@ -658,14 +658,11 @@ const confirmLeave = (message, confirmLabel) => new Promise((resolve) => {
 const openGlobalWorkspace = () => {
   window.parent.location.assign("/vault")
 }
-// Opens a new tab rather than navigating: this workspace can be embedded in an
-// app page, where navigating would either nest a second copy of the interface
-// or drop the user out of the app they were looking at.
 // A copy that belongs to another app opens that app's own workspace, where it
-// can actually be acted on. Anything else — caches, external folders — has no
-// app page, so it opens the global one. Either way the user is leaving this
-// page, so it is confirmed first and the parent window is replaced rather than
-// nesting a second copy of the interface inside this frame.
+// can actually be acted on; caches and external folders have no app page, so
+// they open the global one. Accepting replaces the parent window, because this
+// workspace can be embedded in an app page and navigating in place would nest a
+// second copy of the interface inside that frame.
 const openContentGroup = async (group, kind, label) => {
   // The label and kind travel with the row: an app workspace cannot look up
   // sources belonging to other apps, so a lookup here would name every one of
@@ -2968,9 +2965,6 @@ const loadPagedChildren = async (cache, key, buildUrl, append) => {
     const items = Array.isArray(result.items) ? result.items : []
     current.items = append ? current.items.concat(items) : items
     current.total = Math.max(0, Number(result.total) || 0)
-    // Location lists carry how many copies still cost space; group children
-    // do not, and read as zero.
-    current.deduplicatable = Math.max(0, Number(result.deduplicatable) || 0)
     current.nextCursor = result.next_cursor || null
     current.loaded = true
   } catch (error) {

package/test/vault-ui.test.js

@@ -825,7 +825,6 @@ describe("Save Space interface", () => {
             relative_path: `models/${name}.bin`
           })),
           total: 3,
-          deduplicatable: 1,
           next_cursor: null
         }
       }