From 735c3c263d9207cd981190952c92eb94b9073d37 Mon Sep 17 00:00:00 2001
From: abose <arun@core.ai>
Date: Thu, 26 Oct 2023 14:43:04 +0530
Subject: [PATCH] chore: move to phcode protocol in tauri instead of tauri
 protocol

---
 src/assets/new-project/assets/js/new-project-more.js   |  2 +-
 src/extensibility/Package.js                           |  2 +-
 src/extensions/default/HTMLCodeHints/HTMLJumpToDef.js  |  2 +-
 .../default/QuickView/ImagePreviewProvider.js          |  2 +-
 src/extensions/default/RemoteFileAdapter/main.js       |  6 +++---
 src/filesystem/FileSystem.js                           |  2 +-
 src/index.html                                         |  9 ++++++++-
 src/utils/ExtensionLoader.js                           |  4 ++--
 src/utils/ExtensionUtils.js                            |  2 +-
 src/view/ThemeManager.js                               |  2 +-
 test/SpecRunner.html                                   | 10 ++++++++--
 11 files changed, 28 insertions(+), 15 deletions(-)

diff --git a/src/assets/new-project/assets/js/new-project-more.js b/src/assets/new-project/assets/js/new-project-more.js
index 969f0fffc9..039d0a46d0 100644
--- a/src/assets/new-project/assets/js/new-project-more.js
+++ b/src/assets/new-project/assets/js/new-project-more.js
@@ -31,7 +31,7 @@ function _getIconURL(iconURL) {
     } else if(iconURL === 'appLogo'){
         return 'images/logo.png';
     } else if(iconURL && (iconURL.startsWith("https://") || iconURL.startsWith("http://")
-        || iconURL.startsWith("tauri://") || iconURL.startsWith("asset://"))){
+        || iconURL.startsWith("phcode://") || iconURL.startsWith("asset://"))){
         return iconURL;
     }
     return 'images/tab-img2.png'; // HTML icon
diff --git a/src/extensibility/Package.js b/src/extensibility/Package.js
index 491c3b06ce..912d066224 100644
--- a/src/extensibility/Package.js
+++ b/src/extensibility/Package.js
@@ -242,7 +242,7 @@ define(function (require, exports, module) {
             return d.promise();
         }
         if (!(parsed.protocol === "http:" || parsed.protocol === "https:"
-            || parsed.protocol === "tauri:" || parsed.protocol === "asset:")) {
+            || parsed.protocol === "phcode:" || parsed.protocol === "asset:")) {
             d.reject(Errors.UNSUPPORTED_PROTOCOL);
             return d.promise();
         }
diff --git a/src/extensions/default/HTMLCodeHints/HTMLJumpToDef.js b/src/extensions/default/HTMLCodeHints/HTMLJumpToDef.js
index 71de6b2b07..ff184ec086 100644
--- a/src/extensions/default/HTMLCodeHints/HTMLJumpToDef.js
+++ b/src/extensions/default/HTMLCodeHints/HTMLJumpToDef.js
@@ -59,7 +59,7 @@ define(function (require, exports, module) {
 
     function _openFile(fileRelativePath, mainDocPath) {
         if(fileRelativePath.startsWith("http://") || fileRelativePath.startsWith("https://")
-            || fileRelativePath.startsWith("tauri://") || fileRelativePath.startsWith("asset://")){
+            || fileRelativePath.startsWith("phcode://") || fileRelativePath.startsWith("asset://")){
             return FileViewController.openAndSelectDocument(fileRelativePath, FileViewController.PROJECT_MANAGER);
         }
         const targetPath = path.resolve(mainDocPath, fileRelativePath);
diff --git a/src/extensions/default/QuickView/ImagePreviewProvider.js b/src/extensions/default/QuickView/ImagePreviewProvider.js
index f4eafaab1a..9856100bd4 100644
--- a/src/extensions/default/QuickView/ImagePreviewProvider.js
+++ b/src/extensions/default/QuickView/ImagePreviewProvider.js
@@ -41,7 +41,7 @@ define(function (require, exports, module) {
         extensionlessImagePreview;           // Whether to try and preview extensionless URLs
 
     // List of protocols which we will support for image preview urls
-    let validProtocols = ["data:", "http:", "https:", "tauri:", "asset:", "ftp:", "file:"];
+    let validProtocols = ["data:", "http:", "https:", "phcode:", "asset:", "ftp:", "file:"];
 
     prefs = PreferencesManager.getExtensionPrefs("quickview");
 
diff --git a/src/extensions/default/RemoteFileAdapter/main.js b/src/extensions/default/RemoteFileAdapter/main.js
index d124daa633..dd5de98406 100644
--- a/src/extensions/default/RemoteFileAdapter/main.js
+++ b/src/extensions/default/RemoteFileAdapter/main.js
@@ -34,7 +34,7 @@ define(function (require, exports, module) {
 
     const HTTP_PROTOCOL = "http:",
         HTTPS_PROTOCOL = "https:",
-        TAURI_PROTOCOL = "tauri:",
+        TAURI_PROTOCOL = "phcode:",
         TAURI_ASSET_PROTOCOL = "asset:";
 
     ExtensionUtils.loadStyleSheet(module, "styles.css");
@@ -48,8 +48,8 @@ define(function (require, exports, module) {
             return "https";
         }
 
-        if (data.fullPath.startsWith("tauri://")) {
-            return "tauri";
+        if (data.fullPath.startsWith("phcode://")) {
+            return "phcode";
         }
 
         if (data.fullPath.startsWith("asset://")) {
diff --git a/src/filesystem/FileSystem.js b/src/filesystem/FileSystem.js
index e928b4f0c6..a686cb0761 100644
--- a/src/filesystem/FileSystem.js
+++ b/src/filesystem/FileSystem.js
@@ -1221,7 +1221,7 @@ define(function (require, exports, module) {
     // attach remote file handlers
     var HTTP_PROTOCOL = "http:",
         HTTPS_PROTOCOL = "https:",
-        TAURI_PROTOCOL = "tauri:",
+        TAURI_PROTOCOL = "phcode:",
         TAURI_ASSET_PROTOCOL = "asset:";
 
     var protocolAdapter = {
diff --git a/src/index.html b/src/index.html
index c615a3b988..99c506c631 100644
--- a/src/index.html
+++ b/src/index.html
@@ -44,6 +44,12 @@
     <!-- start inline javascript and non module bootstrap scripts. you must add module scripts to the
      javascript module section only!!-->
     <script>
+        if(location.href.startsWith("tauri://") || location.href.startsWith('https://tauri.localhost')){
+            const errorMessage = `You should use custom protocol phcode:// instead of tauri protocol ${location.href} .`;
+            alert(errorMessage);
+            console.error(errorMessage);
+            throw new Error(errorMessage);
+        }
         function _isTestWindow() {
             const isTestPhoenixWindow = !!(new window.URLSearchParams(window.location.search || "")).get("testEnvironment");
             const isSpecRunnerWindow = window.location.pathname.endsWith("/SpecRunner.html");
@@ -262,7 +268,8 @@
                     'http://127.0.0.1:8000': true, // phcode dev server
                     'http://127.0.0.1:8001': true, // phcode dev live preview server
                     'http://127.0.0.1:5000': true, // playwright tests
-                    'tauri://localhost': true, // tauri prod app
+                    'phcode://localhost': true, // tauri prod app
+                    'https://phcode.localhost': true, // tauri
                     'https://phcode.live': true, // phcode prod live preview server
                     'https://phcode.dev': true,
                     'https://dev.phcode.dev': true,
diff --git a/src/utils/ExtensionLoader.js b/src/utils/ExtensionLoader.js
index 0c624b5b06..2c7b504f22 100644
--- a/src/utils/ExtensionLoader.js
+++ b/src/utils/ExtensionLoader.js
@@ -202,7 +202,7 @@ define(function (require, exports, module) {
      */
     function _mergeConfig(baseConfig) {
         if(baseConfig.baseUrl.startsWith("http://") || baseConfig.baseUrl.startsWith("https://")
-            || baseConfig.baseUrl.startsWith("tauri://") || baseConfig.baseUrl.startsWith("asset://")) {
+            || baseConfig.baseUrl.startsWith("phcode://") || baseConfig.baseUrl.startsWith("asset://")) {
             return _mergeConfigFromURL(baseConfig);
         }
         throw new Error("Config can only be loaded from an http url, but got" + baseConfig.baseUrl);
@@ -394,7 +394,7 @@ define(function (require, exports, module) {
         var result = new $.Deferred(),
             extensionPath = config.baseUrl + "/" + entryPoint + ".js";
         if(extensionPath.startsWith("http://") || extensionPath.startsWith("https://")
-            || extensionPath.startsWith("tauri://") || extensionPath.startsWith("asset://")) {
+            || extensionPath.startsWith("phcode://") || extensionPath.startsWith("asset://")) {
             return _testExtensionByURL(name, config, entryPoint);
         }
 
diff --git a/src/utils/ExtensionUtils.js b/src/utils/ExtensionUtils.js
index 148d231ee6..7f876e4412 100644
--- a/src/utils/ExtensionUtils.js
+++ b/src/utils/ExtensionUtils.js
@@ -269,7 +269,7 @@ define(function (require, exports, module) {
      */
     function loadMetadata(metadataURL, extensionName) {
         if(metadataURL.startsWith("http://") || metadataURL.startsWith("https://")
-            || metadataURL.startsWith("tauri://") || metadataURL.startsWith("asset://")) {
+            || metadataURL.startsWith("phcode://") || metadataURL.startsWith("asset://")) {
             return _loadExtensionMetadata(metadataURL, extensionName);
         }
         throw new Error(`Cannot load extension metadata for ${extensionName} at path ${metadataURL}`);
diff --git a/src/view/ThemeManager.js b/src/view/ThemeManager.js
index 9f37d8b448..d66133b3c8 100644
--- a/src/view/ThemeManager.js
+++ b/src/view/ThemeManager.js
@@ -445,7 +445,7 @@ define(function (require, exports, module) {
      */
     function loadFile(fileName, options) {
         if(fileName.startsWith("http://") || fileName.startsWith("https://")
-            || fileName.startsWith("tauri://") || fileName.startsWith("asset://")) {
+            || fileName.startsWith("phcode://") || fileName.startsWith("asset://")) {
             if(Phoenix.VFS.getPathForVirtualServingURL(fileName)){
                 fileName = Phoenix.VFS.getPathForVirtualServingURL(fileName);
             } else {
diff --git a/test/SpecRunner.html b/test/SpecRunner.html
index c5ca7937fd..ff44bac93f 100644
--- a/test/SpecRunner.html
+++ b/test/SpecRunner.html
@@ -31,7 +31,12 @@
            frame-src * localhost:* asset: https://asset.localhost ;
            connect-src * localhost:* asset: https://asset.localhost ;">
   <script type="text/javascript">
-    //brackets.app.showDeveloperTools();
+    if(location.href.startsWith("tauri://") || location.href.startsWith('https://tauri.localhost')){
+      const errorMessage = `You should use custom protocol phcode:// instead of tauri protocol ${location.href} .`;
+      alert(errorMessage);
+      console.error(errorMessage);
+      throw new Error(errorMessage);
+    }
     console.warn('Make sure to run this following command before starting tests : npm run build ');
     if(window.__TAURI__) {
       window.testRunnerLogToConsole = function (...args) {
@@ -240,7 +245,8 @@
           'http://127.0.0.1:8000': true, // phcode dev server
           'http://127.0.0.1:8001': true, // phcode dev live preview server
           'http://127.0.0.1:5000': true, // playwright tests
-          'tauri://localhost': true, // tauri prod app
+          'phcode://localhost': true, // tauri prod app
+          'https://phcode.localhost': true, // tauri
           'https://phcode.live': true, // phcode prod live preview server
           'https://phcode.dev': true,
           'https://dev.phcode.dev': true,