fix: cut copy paste files across different tauri windows working and …

…localstorage refactor

.eslintrc.js

@@ -74,6 +74,7 @@ module.exports = {
         "$": false,
         "brackets": false,
         "Phoenix": false,
+        "PhStore": false,
         "iconv": false,
         "Buffer": true,
         "clearTimeout": false,

src/assets/new-project/assets/js/code-editor.js

@@ -122,20 +122,20 @@ function removeProject(fullPath) {
 }
 
 function _showFirstTimeExperience() {
-    let shownBefore = localStorage.getItem('notification.defaultProject.Shown');
+    let shownBefore = PhStore.getItem('notification.defaultProject.Shown');
     if(!shownBefore){
         createNotificationFromTemplate(Strings.DEFAULT_PROJECT_NOTIFICATION,
             "defaultProjectButton", {
                 allowedPlacements: ["left", "right"],
                 autoCloseTimeS: 15,
                 dismissOnClick: true
             });
-        localStorage.setItem('notification.defaultProject.Shown', 'true');
+        PhStore.setItem('notification.defaultProject.Shown', 'true');
     }
 }
 
 function _updateDropdown() {
-    let shouldShowWelcome = localStorage.getItem("new-project.showWelcomeScreen") || 'Y';
+    let shouldShowWelcome = PhStore.getItem("new-project.showWelcomeScreen") || 'Y';
     if(shouldShowWelcome === 'Y') {
         document.getElementById("showWelcomeIndicator").style = "visibility: visible";
     } else {
@@ -158,9 +158,9 @@ function _attachSettingBtnEventListeners() {
     });
 
     document.getElementById("showWelcome").addEventListener('click', (event)=>{
-        let shouldShowWelcome = localStorage.getItem("new-project.showWelcomeScreen") || 'Y';
+        let shouldShowWelcome = PhStore.getItem("new-project.showWelcomeScreen") || 'Y';
         shouldShowWelcome = shouldShowWelcome === 'Y'? 'N' : 'Y';
-        localStorage.setItem("new-project.showWelcomeScreen", shouldShowWelcome);
+        PhStore.setItem("new-project.showWelcomeScreen", shouldShowWelcome);
     });
 
     document.getElementById("showAbout").addEventListener('click', (event)=>{

src/assets/new-project/assets/js/phoenix.js

@@ -28,6 +28,7 @@ const RECENT_PROJECTS_INTERFACE = "Extn.Phoenix.recentProjects";
 const selfFileName = location.href.split('?')[0].split('/').pop();
 window.Strings = window.parent.Strings;
 window.path = window.parent.path;
+window.PhStore = window.parent.PhStore;
 window.parent.ExtensionInterface.waitAndGetExtensionInterface(NEW_PROJECT_EXTENSION_INTERFACE)
     .then(interfaceObj => {
         window.newProjectExtension = interfaceObj;

src/extensibility/Package.js

@@ -367,14 +367,14 @@ define(function (require, exports, module) {
      * function manages state weather an extension is enabled or disabled
      */
     function _toggleDisabledExtension(path, enabled) {
-        let arr = JSON.parse(localStorage.getItem(DISABLED_EXTENSIONS_KEY) || "[]");
+        let arr = JSON.parse(PhStore.getItem(DISABLED_EXTENSIONS_KEY) || "[]");
         const io = arr.indexOf(path);
         if (enabled === true && io !== -1) {
             arr.splice(io, 1);
         } else if (enabled === false && io === -1) {
             arr.push(path);
         }
-        localStorage.setItem(DISABLED_EXTENSIONS_KEY, JSON.stringify(arr));
+        PhStore.setItem(DISABLED_EXTENSIONS_KEY, JSON.stringify(arr));
     }
 
     /**

src/extensions/default/Phoenix/guided-tour.js

@@ -45,8 +45,8 @@ define(function (require, exports, module) {
         TWO_WEEKS_IN_DAYS = 14,
         USAGE_COUNTS_KEY    = "healthDataUsage"; // private to phoenix, set from health data extension
 
-    const userAlreadyDidAction = localStorage.getItem(GUIDED_TOUR_LOCAL_STORAGE_KEY)
-        ? JSON.parse(localStorage.getItem(GUIDED_TOUR_LOCAL_STORAGE_KEY)) : {
+    const userAlreadyDidAction = PhStore.getItem(GUIDED_TOUR_LOCAL_STORAGE_KEY)
+        ? JSON.parse(PhStore.getItem(GUIDED_TOUR_LOCAL_STORAGE_KEY)) : {
             version: 1,
             newProjectShown: false,
             beautifyCodeShown: false,
@@ -85,7 +85,7 @@ define(function (require, exports, module) {
                 let keyboardShortcut = KeyBindingManager.getKeyBindings(Commands.EDIT_BEAUTIFY_CODE);
                 keyboardShortcut = (keyboardShortcut && keyboardShortcut[0]) ? keyboardShortcut[0].displayKey : "-";
                 userAlreadyDidAction.beautifyCodeShown =  true;
-                localStorage.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
+                PhStore.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
                 Metrics.countEvent(Metrics.EVENT_TYPE.UI, "guide", "beautify");
                 currentlyShowingNotification = NotificationUI.createFromTemplate(
                     StringUtils.format(Strings.BEAUTIFY_CODE_NOTIFICATION, keyboardShortcut),
@@ -115,7 +115,7 @@ define(function (require, exports, module) {
                 return;
             }
             userAlreadyDidAction.newProjectShown =  true;
-            localStorage.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
+            PhStore.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
             Metrics.countEvent(Metrics.EVENT_TYPE.UI, "guide", "newProj");
             currentlyShowingNotification = NotificationUI.createFromTemplate(Strings.NEW_PROJECT_NOTIFICATION,
                 "newProject", {
@@ -138,7 +138,7 @@ define(function (require, exports, module) {
             function _showNotification() {
                 // legacy key. cant change without triggering the user base
                 let notificationKey = 'livePreviewPopoutShown', version = "v1";
-                let popoutMessageShown = localStorage.getItem(notificationKey);
+                let popoutMessageShown = PhStore.getItem(notificationKey);
                 if(popoutMessageShown === version){
                     // already shown
                     LiveDevelopment.off(LiveDevelopment.EVENT_LIVE_PREVIEW_CLICKED, _showNotification);
@@ -158,7 +158,7 @@ define(function (require, exports, module) {
                     currentlyShowingNotification.done(()=>{
                         currentlyShowingNotification = null;
                     });
-                    localStorage.setItem(notificationKey, version);
+                    PhStore.setItem(notificationKey, version);
                 }
                 LiveDevelopment.off(LiveDevelopment.EVENT_LIVE_PREVIEW_CLICKED, _showNotification);
             }
@@ -171,7 +171,7 @@ define(function (require, exports, module) {
     function _showLivePreviewNotification() {
         // legacy reasons live preview notification is called new project notification.
         const livePreviewNotificationKey = "newProjectNotificationShown";
-        const livePreviewNotificationShown = localStorage.getItem(livePreviewNotificationKey);
+        const livePreviewNotificationShown = PhStore.getItem(livePreviewNotificationKey);
         if(livePreviewNotificationShown){
             return;
         }
@@ -185,7 +185,7 @@ define(function (require, exports, module) {
                 autoCloseTimeS: 15,
                 dismissOnClick: true}
         );
-        localStorage.setItem(livePreviewNotificationKey, "true");
+        PhStore.setItem(livePreviewNotificationKey, "true");
         currentlyShowingNotification.done(()=>{
             currentlyShowingNotification = null;
         });
@@ -262,7 +262,7 @@ define(function (require, exports, module) {
                 Metrics.countEvent(Metrics.EVENT_TYPE.USER, "notify", "star", 1);
                 _openStarsPopup();
                 userAlreadyDidAction.lastShownGithubStarsDate = Date.now();
-                localStorage.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
+                PhStore.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
             }, GITHUB_STARS_POPUP_TIME);
         }
     }
@@ -278,7 +278,7 @@ define(function (require, exports, module) {
                 Metrics.countEvent(Metrics.EVENT_TYPE.USER, "survey", "generalShown", 1);
                 Dialogs.showModalDialogUsingTemplate(Mustache.render(SurveyTemplate, templateVars));
                 userAlreadyDidAction.generalSurveyShownVersion = surveyVersion;
-                localStorage.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
+                PhStore.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
             }
         }, GENERAL_SURVEY_TIME);
     }
@@ -326,7 +326,7 @@ define(function (require, exports, module) {
                 $content.find("a").click(_openPowerUserSurvey);
                 NotificationUI.createToastFromTemplate(Strings.POWER_USER_POPUP_TITLE, $content);
                 userAlreadyDidAction.lastShownPowerSurveyDate = Date.now();
-                localStorage.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
+                PhStore.setItem(GUIDED_TOUR_LOCAL_STORAGE_KEY, JSON.stringify(userAlreadyDidAction));
             }, POWER_USER_SURVEY_TIME);
         }
     }

src/extensions/default/Phoenix/new-project.js

@@ -96,7 +96,7 @@ define(function (require, exports, module) {
 
     function init() {
         _addMenuEntries();
-        const shouldShowWelcome = localStorage.getItem("new-project.showWelcomeScreen") || 'Y';
+        const shouldShowWelcome = PhStore.getItem("new-project.showWelcomeScreen") || 'Y';
         if(shouldShowWelcome !== 'Y') {
             Metrics.countEvent(Metrics.EVENT_TYPE.NEW_PROJECT, "dialogue", "disabled");
             guidedTour.startTourIfNeeded();

src/extensions/default/Phoenix/serverSync.js

@@ -45,11 +45,24 @@ define(function (require, exports, module) {
     let projectSyncCompleted = false;
     let previewURL;
 
+    function _fixLegacyUserContext() {
+        // In earlier versions, user context data was stored in the browser's localStorage. We have since
+        // transitioned to using PhStore for this purpose. This function helps in migrating the user context
+        // from localStorage to PhStore. This ensures that existing users don't lose access to their published projects.
+        // Note: Published projects are retained for only 30 days. This function is primarily intended to support
+        // users during the transition period and is safe to remove 3 months after the date of this implementation,
+        // as users are informed that their published projects are kept for a 30-day period only.
+        if (!PhStore.getItem(USER_CONTEXT) && localStorage.getItem(USER_CONTEXT)) {
+            PhStore.setItem(USER_CONTEXT, localStorage.getItem(USER_CONTEXT));
+        }
+    }
+
     function _setupUserContext() {
-        userContext = localStorage.getItem(USER_CONTEXT);
+        _fixLegacyUserContext();
+        userContext = PhStore.getItem(USER_CONTEXT);
         if(!userContext){
             userContext = "p-" + Math.round( Math.random()*10000000000000).toString(16);
-            localStorage.setItem(USER_CONTEXT, userContext);
+            PhStore.setItem(USER_CONTEXT, userContext);
         }
     }
 

src/node-loader.js

@@ -592,11 +592,14 @@ function nodeLoader() {
         let commandID = 0, pendingCommands = {};
         const PHNODE_PREFERENCES_KEY = "PhNode.Prefs";
         function setInspectEnabled(enabled) {
+            // cannot use PhStore instead of localStorage here as this is required at boot. Should be fine
+            // as this to use non-persistent local storage(due to safari ITP) here as this is a debug flag.
             const prefs = JSON.parse(localStorage.getItem(PHNODE_PREFERENCES_KEY) || "{}");
             prefs.inspectEnabled = enabled;
             localStorage.setItem(PHNODE_PREFERENCES_KEY, JSON.stringify(prefs));
         }
         function isInspectEnabled() {
+            // cannot use PhStore instead of localStorage here as this is required at boot.
             const prefs = JSON.parse(localStorage.getItem(PHNODE_PREFERENCES_KEY) || "{}");
             return !!prefs.inspectEnabled;
         }

src/project/ProjectManager.js

@@ -44,10 +44,10 @@ define(function (require, exports, module) {
 
     require("utils/Global");
 
-    var _ = require("thirdparty/lodash");
+    const _ = require("thirdparty/lodash");
 
     // Load dependent modules
-    let AppInit             = require("utils/AppInit"),
+    const AppInit             = require("utils/AppInit"),
         Async               = require("utils/Async"),
         PreferencesDialogs  = require("preferences/PreferencesDialogs"),
         PreferencesManager  = require("preferences/PreferencesManager"),
@@ -92,6 +92,8 @@ define(function (require, exports, module) {
 
     EventDispatcher.setLeakThresholdForEvent(EVENT_PROJECT_OPEN, 25);
 
+    const CLIPBOARD_SYNC_KEY = "phoenix.clipboard";
+
     /**
      * @private
      * Filename to use for project settings files.
@@ -1521,22 +1523,12 @@ define(function (require, exports, module) {
     const OPERATION_CUT = 'cut',
         OPERATION_COPY = 'copy';
 
-    function _addTextToSystemClipboard(text) {
-        if (!navigator.clipboard) {
-            console.warn('Browser doesnt support clipboard control. system cut/copy/paste may not work');
-            return;
-        }
-        navigator.clipboard.writeText(text).catch(function(err) {
-            console.error('System clipboard error: Could not copy text: ', err);
-        });
-    }
-
     function _registerPathWithClipboard(path, operation) {
-        _addTextToSystemClipboard(window.path.basename(path));
-        localStorage.setItem("phoenix.clipboard", JSON.stringify({
+        Phoenix.app.copyToClipboard(window.path.basename(path));
+        PhStore.setItem(CLIPBOARD_SYNC_KEY, {
             operation: operation,
             path: path
-        }));
+        });
     }
 
     /**
@@ -1575,8 +1567,8 @@ define(function (require, exports, module) {
         if(fullPath){
             let projectRoot = getProjectRoot().fullPath;
             let relativePath = window.path.relative(projectRoot, fullPath);
-            _addTextToSystemClipboard(relativePath);
-            localStorage.setItem("phoenix.clipboard", JSON.stringify({}));
+            Phoenix.app.copyToClipboard(relativePath);
+            PhStore.setItem(CLIPBOARD_SYNC_KEY, {});
         }
     }
 
@@ -1691,11 +1683,10 @@ define(function (require, exports, module) {
         if(context){
             targetPath = context.fullPath;
         }
-        let clipboard = localStorage.getItem("phoenix.clipboard");
+        const clipboard = PhStore.getItem(CLIPBOARD_SYNC_KEY);
         if(!clipboard){
             return;
         }
-        clipboard = JSON.parse(clipboard);
         switch (clipboard.operation) {
         case OPERATION_CUT: _performCut(clipboard.path, targetPath); break;
         case OPERATION_COPY: _performCopy(clipboard.path, targetPath); break;
@@ -1705,6 +1696,7 @@ define(function (require, exports, module) {
 
     // Initialize variables and listeners that depend on the HTML DOM
     AppInit.htmlReady(function () {
+        PhStore.watchExternalChanges(CLIPBOARD_SYNC_KEY);
         $projectTreeContainer = $("#project-files-container");
         $projectTreeContainer.addClass("jstree jstree-brackets");
         $projectTreeContainer.css("overflow", "auto");