Add 'reuseSplitView' support to createOpenOrDeleteNoteCallbackUrl() and add openNoteInSplitViewIfNotOpenAlready()

Author: jgclark

helpers/HTMLView.js

@@ -709,6 +709,12 @@ export function replaceMarkdownLinkWithHTMLLink(str: string): string {
 
 export async function sendToHTMLWindow(windowId: string, actionType: string, data: any = {}, updateInfo: string = ''): Promise<any> {
   try {
+    // If HTMLView API isn't available (older platforms / builds), quietly skip sending
+    if (typeof HTMLView === 'undefined' || typeof HTMLView.runJavaScript !== 'function') {
+      logWarn('sendToHTMLWindow', `HTMLView API is not available on this platform/build; skipping message "${actionType}" to HTML window`)
+      return
+    }
+
     const windowExists = isHTMLWindowOpen(windowId)
     if (!windowExists) logWarn(`sendToHTMLWindow`, `Window ${windowId} does not exist; setting NPWindowID = undefined`)
     const windowIdToSend = windowExists ? windowId : undefined // for iphone/ipad you have to send undefined

helpers/NPVersions.js

@@ -36,6 +36,7 @@ export function usersVersionHas(feature: string): boolean {
     availableReminderLists: '3.20.0', // Dec 2025, macOS build 1469
     showInMainWindow: '3.20.0', // Dec 2025, macOS build 1469
     showInMainWindowOniOS: '3.20.1', // Jan 2026, iOS build 1380
+    reuseSplitView: '3.20.1', // Jan 2026, macOS build 1479ish
   }
 
   // Check if the user's version meets the requirement for the requested feature

helpers/NPWindows.js

@@ -8,6 +8,7 @@ import { clo, logDebug, logError, logInfo, logWarn } from '@helpers/dev'
 import { usersVersionHas } from '@helpers/NPVersions'
 import { caseInsensitiveMatch, caseInsensitiveStartsWith } from '@helpers/search'
 import { inputIntegerBounded } from '@helpers/userInput'
+import { createOpenOrDeleteNoteCallbackUrl } from '@helpers/general'
 
 // ----------------------------------------------------------------------------
 // Types
@@ -647,6 +648,33 @@ export async function openNoteInNewSplitIfNeeded(filename: string): Promise<bool
   return !!res
 }
 
+/**
+ * Open a note in a split view using x-callback-url, but only if it is not already open in any Editor window.
+ * Uses the 'reuseSplitView' openType so that a single split view is reused where possible.
+ * Note: This is in place of `await   Editor.openNoteByFilename(note.filename, true, 0, 0, false, false)` which doesn't have reuseSplitView option. (Yet.)
+ * @author @jgclark
+ * @param {string} filename - filename of the note to open
+ * @returns {boolean} true if a new split view was opened, false if the note was already open
+ */
+export function openNoteInSplitViewIfNotOpenAlready(filename: string, callingFunctionName?: string): boolean {
+  try {
+    if (noteOpenInEditor(filename)) {
+      logDebug('openNoteInSplitViewIfNotOpenAlready', `(for ${callingFunctionName ?? '?'}) Note '${filename}' is already open in an Editor window. Skipping.`)
+      return false
+    }
+
+    const splitOpenType = usersVersionHas('reuseSplitView') ? 'reuseSplitView' : 'splitView'
+    const callbackUrl = createOpenOrDeleteNoteCallbackUrl(filename, 'filename', null, splitOpenType, false)
+    logDebug('openNoteInSplitViewIfNotOpenAlready', `(for ${callingFunctionName ?? '?'}) splitOpenType: ${splitOpenType} openNote in Editor callbackUrl: ${callbackUrl}`)
+    NotePlan.openURL(callbackUrl)
+    logDebug('openNoteInSplitViewIfNotOpenAlready', `(for ${callingFunctionName ?? '?'}) after x-callback call to openNote`)
+    return true
+  } catch (error) {
+    logError('openNoteInSplitViewIfNotOpenAlready', `openNoteInSplitViewIfNotOpenAlready: Error: ${error.message}`)
+    return false
+  }
+}
+
 /**
  * Open a calendar note in a split editor, and (optionally) move insertion point to 'cursorPointIn'
  * @author @jgclark

helpers/general.js

@@ -277,7 +277,7 @@ export function returnNoteLink(noteTitle: string, heading: string | null = ''):
  * @param {string} titleOrFilename - title of the note or the filename
  * @param {string} paramType - 'title' | 'filename' | 'date' (default is 'title')
  * @param {string | null} heading - heading inside of note (optional)
- * @param {string} openType - 'subWindow' | 'splitView' | 'useExistingSubWindow' (default: null)
+ * @param {string} openType - 'subWindow' | 'splitView' | 'reuseSplitView' | 'useExistingSubWindow' (default: null)
  * @param {boolean} isDeleteNote - whether this is actually a deleteNote
  * @param {string} blockID - the blockID if this is a line link (includes the ^) -- only works with title (not filename)
  * @returns {string} the x-callback-url string
@@ -288,7 +288,7 @@ export function createOpenOrDeleteNoteCallbackUrl(
   titleOrFilename: string,
   paramType: 'title' | 'filename' | 'date' = 'title',
   heading: string | null = '',
-  openType: 'subWindow' | 'splitView' | 'useExistingSubWindow' | null = null,
+  openType: 'subWindow' | 'splitView' | 'reuseSplitView' | 'useExistingSubWindow' | null = null,
   isDeleteNote: boolean = false,
   blockID: string = '',
 ): string {
@@ -300,7 +300,10 @@ export function createOpenOrDeleteNoteCallbackUrl(
   const head = heading && heading.length ? encodePlusParens(heading.replace('#', '')) : ''
   // console.log(`createOpenOrDeleteNoteCallbackUrl: ${xcb}${titleOrFilename}${head ? `&heading=${head}` : ''}`)
   const encodedTitleOrFilename = encodePlusParens(titleOrFilename)
-  const openAs = openType && ['subWindow', 'splitView', 'useExistingSubWindow'].includes(openType) ? `&${openType}=yes` : ''
+  let openAs = openType && ['subWindow', 'splitView', 'reuseSplitView', 'useExistingSubWindow'].includes(openType) ? `&${openType}=yes` : ''
+  if (openType === 'reuseSplitView') {
+    openAs += '&splitView=yes' // special case for reuseSplitView, which needs both
+  }
   let retVal = ''
   if (isLineLink) {
     retVal = `${xcb}${encodedTitleOrFilename}${encodeURIComponent(blockID)}`