@jxsuite/studio 10.2.0 → 10.4.0

package/src/panels/tab-strip.ts

@@ -83,6 +83,7 @@ import { openMenu } from "../surfaces/menu";
 import type { MenuHandle, MenuRowProjection } from "../surfaces/menu";
 import { saveFile } from "../files/file-ops";
 import { collabReadOnly } from "../collab/collab-session";
+import { imageEditorDirty } from "../media/image-editor";
 import { collabState } from "../collab/collab-state";
 import { elementAtPoint, rectOf } from "../utils/geometry";
 import { resolveRegion } from "../ui/regions";
@@ -255,6 +256,8 @@ export function mount(host: HTMLElement) {
         // A tab's workspace follows its editor (`project.json` is Styles' in its stylebook only).
         void tab.session.ui.canvasMode;
         void tab.doc.dirty;
+        // An image edit is the tab's unsaved work too (§9.5), held beside the document, not in it.
+        void imageEditorDirty(tab);
         void tab.documentPath;
         void tab.pinned;
         void tab.preview;
@@ -689,7 +692,7 @@ function chipView(
   const pill = entryDraftPill(tab);
   return [
     {
-      dirty: tab.doc.dirty,
+      dirty: tab.doc.dirty || imageEditorDirty(tab),
       draft: pill?.draft ?? false,
       index,
       key: id,
@@ -1336,11 +1339,17 @@ export function tabLabel(tab: Tab): string {
  * buffer rather than resyncing over a half-typed heading), and that text exists nowhere else.
  * {@link tabBufferUnsaved} is that residue, and it is the user's own typing by construction.
  *
+ * **And an image tab's unsaved work is not in its document at all.** A media tab's document is a
+ * stub; an Image editor edit (§9.5) is a recipe held beside it, so it is a third source of dirt,
+ * and it takes the residue's side of the peer-count gate for the residue's reason: a recipe has
+ * never been published, mirrored or written, so no peer focused on the path holds a copy of it.
+ *
  * @param {Tab} tab
  */
 export function shouldWarnOnClose(tab: Tab): boolean {
   const residue = tabBufferUnsaved(tab);
-  if (!tab.doc.dirty && !residue) {
+  const image = imageEditorDirty(tab);
+  if (!tab.doc.dirty && !residue && !image) {
     return false;
   }
   const state = collabState(tab);
@@ -1361,7 +1370,7 @@ export function shouldWarnOnClose(tab: Tab): boolean {
      lock holder is by definition a peer focused on this path, so the busier the room got, the
      quieter the close became. Asked before the peer count rather than after, because the peer count
      has no jurisdiction over text the room has never been told about. */
-  if (residue) {
+  if (residue || image) {
     return true;
   }
   const peersHere = state.peers.filter((p) => p.state.focusedPath === tab.documentPath);

package/src/studio.ts

@@ -212,6 +212,7 @@ import { setProjectAdopter } from "./services/project-adoption";
 import { setImportHandoff } from "./services/import-seed";
 import { watchConnection } from "./services/connection";
 import { commitTabBuffers, tabBufferUnsaved } from "./services/monaco-buffer";
+import { imageEditorDirty } from "./media/image-editor";
 import { clearTyping } from "./tabs/typing";
 import * as leftPanelMod from "./panels/left-panel";
 import * as tabStrip from "./panels/tab-strip";
@@ -605,12 +606,19 @@ setInsertZoneClickHandler(runInsertZoneAction);
 setFileDropHandler(applyFileDrop);
 // Every upload surface routes through uploadAssets(); refresh the three caches that list project
 // Files afterwards. Injected here because all three modules import from media-upload.
-setMediaChangedHandler(async (dir) => {
+setMediaChangedHandler(async (dir, overwritten) => {
   invalidateMediaCache();
   invalidateLibrary();
   invalidateSiteIndex();
   await loadDirectory(dir);
   renderLeftPanel();
+  /* Bytes replaced IN PLACE change no listing, so nothing above redraws what was already drawn from
+     them. Every pane draws again, and its asset resolver now carries the path's repaint key. */
+  if (overwritten !== undefined) {
+    for (const pane of workspace.panes) {
+      renderCanvas(pane.id);
+    }
+  }
 });
 // The in-iframe "/" trigger drives the parent-realm slash menu across the bridge.
 setCanvasSlashHandler(canvasSlashHandler);
@@ -677,7 +685,7 @@ document.addEventListener(
  */
 export function hasUnsavedTabs(): boolean {
   for (const tab of workspace.tabs.values()) {
-    if (tab.doc.dirty || tabBufferUnsaved(tab)) {
+    if (tab.doc.dirty || tabBufferUnsaved(tab) || imageEditorDirty(tab)) {
       return true;
     }
   }
@@ -1119,6 +1127,9 @@ if (_projectParam) {
           /* Through `loadDirectory` (inside `loadProjectDirectories`), not `platform.listDirectory`:
              the loader is what reads the `.gitignore` chain governing each directory. */
           await loadProjectDirectories();
+          /* The other two ways into a project subscribe, and this one did not, so a project opened
+             by its URL never heard about a file changing on disk. */
+          ensureFsSync();
           void maybePromptJxsuiteUpdate(siteCtx.sitePath);
         }