Bugfix/(nut tree/nut.js#477)/remove permission check for delay setters (#153)

* (nut-tree/nut.js#477) Removed both setMouseDelay and setKeyboardDelay from permissionCheck.js since these setters do not required any permissions

* (nut-tree/nut.js#477) Update permissionCheck.js to cache permission check results and only ask for permissions on actual function call by wrapping it in a HOF

* (nut-tree/nut.js#477) Re-require native bindings on error

Author: s1hofmann

permissionCheck.js

@@ -1,4 +1,7 @@
-const libnut = require("bindings")("libnut");
+let libnut = require("bindings")("libnut");
+
+let hasScreenRecordingPermission = false;
+let hasAccessibilityPermission = false;
 
 try {
     const permissions = require("@nut-tree/node-mac-permissions");
@@ -9,25 +12,27 @@ try {
     };
 
     const askForAccessibility = (nativeFunction, functionName) => {
-        if (process.platform !== 'darwin') {
+        if (process.platform !== 'darwin' || hasAccessibilityPermission) {
             return nativeFunction;
         }
         const accessibilityStatus = permissions.getAuthStatus("accessibility");
 
         if (accessibilityStatus === 'authorized') {
+            hasAccessibilityPermission = true;
             return nativeFunction;
         } else if (accessibilityStatus === 'not determined' || accessibilityStatus === 'denied') {
             permissions.askForAccessibilityAccess();
             return wrapWithWarning(`##### WARNING! The application running this script tries to access accessibility features to execute ${functionName}! Please grant requested access and visit https://github.com/nut-tree/nut.js#macos for further information. #####`, nativeFunction);
         }
     }
     const askForScreenRecording = (nativeFunction, functionName) => {
-        if (process.platform !== 'darwin') {
+        if (process.platform !== 'darwin' || hasScreenRecordingPermission) {
             return nativeFunction;
         }
         const screenCaptureStatus = permissions.getAuthStatus("screen");
 
         if (screenCaptureStatus === 'authorized') {
+            hasScreenRecordingPermission = true;
             return nativeFunction;
         } else if (screenCaptureStatus === 'not determined' || screenCaptureStatus === 'denied') {
             permissions.askForScreenCaptureAccess();
@@ -42,11 +47,9 @@ try {
         "mouseClick",
         "mouseToggle",
         "scrollMouse",
-        "setMouseDelay",
         "keyTap",
         "keyToggle",
         "typeString",
-        "setKeyboardDelay",
         "getScreenSize",
         "highlight",
         "captureScreen",
@@ -61,14 +64,17 @@ try {
     ];
 
     for (const functionName of accessibilityAccess) {
-        libnut[functionName] = askForAccessibility(libnut[functionName], functionName);
+        const originalFunction = libnut[functionName];
+        libnut[functionName] = (...args) => askForAccessibility(originalFunction, functionName)(...args);
     }
     for (const functionName of screenCaptureAccess) {
-        libnut[functionName] = askForScreenRecording(libnut[functionName], functionName);
+        const originalFunction = libnut[functionName];
+        libnut[functionName] = (...args) => askForScreenRecording(originalFunction, functionName)(...args);
     }
 } catch (e) {
     console.warn(`Encountered error establishing macOS permission checks:`, e.message);
     console.warn(`Returning original module.`);
+    libnut = require("bindings")("libnut");
 } finally {
     module.exports = libnut;
 }