authser.py

@@ -1,19 +1,43 @@
 import sys
+import subprocess
 import re
-import colorama
-import requests
 from PyQt5.QtWidgets import (QApplication, QDialog, QLabel, QVBoxLayout, 
                           QPushButton, QLineEdit, QMessageBox)
-from PyQt5.QtCore import QThread, pyqtSignal, Qt, QUrl, QObject, QTimer
+from PyQt5.QtCore import QThread, pyqtSignal, Qt, QUrl, QObject
 from PyQt5.QtGui import QDesktopServices
-from picomc.logging import logger
 
-# Constants
+class AuthenticationParser:
-URL_DEVICE_AUTH = "https://login.microsoftonline.com/consumers/oauth2/v2.0/devicecode"
+    @staticmethod
-URL_TOKEN = "https://login.microsoftonline.com/consumers/oauth2/v2.0/token"
+    def clean_ansi(text):
-CLIENT_ID = "c52aed44-3b4d-4215-99c5-824033d2bc0f"
+        ansi_clean = re.compile(r'\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])')
-SCOPE = "XboxLive.signin offline_access"
+        printable_clean = re.compile(r'[^\x20-\x7E\n]')
-GRANT_TYPE = "urn:ietf:params:oauth:grant-type:device_code"
+        text = ansi_clean.sub('', text)
+        text = printable_clean.sub('', text)
+        return text.strip()
+
+    @staticmethod
+    def is_auth_error(output):
+        cleaned_output = AuthenticationParser.clean_ansi(output)
+        return "AADSTS70016" in cleaned_output and "not yet been authorized" in cleaned_output
+
+    @staticmethod
+    def parse_auth_output(output):
+        cleaned_output = AuthenticationParser.clean_ansi(output)
+        if AuthenticationParser.is_auth_error(cleaned_output):
+            return None
+
+        pattern = r"https://[^\s]+"
+        code_pattern = r"code\s+([A-Z0-9]+)"
+        
+        url_match = re.search(pattern, cleaned_output)
+        code_match = re.search(code_pattern, cleaned_output, re.IGNORECASE)
+        
+        if url_match and code_match:
+            return {
+                'url': url_match.group(0),
+                'code': code_match.group(1)
+            }
+        return None
 
 class AuthDialog(QDialog):
     def __init__(self, url, code, parent=None, error_mode=False):
@@ -82,81 +106,66 @@ class AuthenticationThread(QThread):
     error_occurred = pyqtSignal(str)
     auth_error_detected = pyqtSignal(str)
     finished = pyqtSignal()
-    access_token_received = pyqtSignal(str, str)
     
     def __init__(self, account):
         super().__init__()
         self.account = account
-        self.device_code = None
+        self.process = None
         self.is_running = True
+        self.current_output = ""
+        self.waiting_for_auth = False
 
     def run(self):
         try:
-            self.authenticate(self.account)
+            command = f'picomc account authenticate {self.account}'
+            
+            self.process = subprocess.Popen(
+                command,
+                shell=True,
+                stdout=subprocess.PIPE,
+                stderr=subprocess.STDOUT,
+                stdin=subprocess.PIPE,
+                text=True,
+                bufsize=1,
+                universal_newlines=True
+            )
+            
+            self.current_output = ""
+            while self.is_running and self.process.poll() is None:
+                line = self.process.stdout.readline()
+                if line:
+                    self.current_output += line
+                    
+                    if not self.waiting_for_auth:
+                        parsed_data = AuthenticationParser.parse_auth_output(self.current_output)
+                        if parsed_data:
+                            self.auth_data_received.emit(parsed_data)
+                            self.waiting_for_auth = True
+                            self.current_output = ""
+                    elif AuthenticationParser.is_auth_error(self.current_output):
+                        self.auth_error_detected.emit(self.current_output)
+                        self.waiting_for_auth = False
+                        self.current_output = ""
+            
+            self.process.wait()
+            self.finished.emit()
+            
         except Exception as e:
             self.error_occurred.emit(str(e))
             self.finished.emit()
 
-    def authenticate(self, account):
-        try:
-            data = {"client_id": CLIENT_ID, "scope": SCOPE}
-
-            # Request device code
-            resp = requests.post(URL_DEVICE_AUTH, data)
-            resp.raise_for_status()
-
-            j = resp.json()
-            self.device_code = j["device_code"]
-            user_code = j["user_code"]
-            link = j["verification_uri"]
-
-            # Format message with colorama
-            msg = j["message"]
-            msg = msg.replace(
-                user_code, colorama.Fore.RED + user_code + colorama.Fore.RESET
-            ).replace(link, colorama.Style.BRIGHT + link + colorama.Style.NORMAL)
-
-            # Emit auth data received signal
-            self.auth_data_received.emit({'url': link, 'code': user_code})
-
-        except requests.exceptions.RequestException as e:
-            logger.error(f"Request failed: {e}")
-            self.error_occurred.emit(str(e))
-            self.finished.emit()
-    
-    def poll_for_token(self):
-        try:
-            data = {"code": self.device_code, "grant_type": GRANT_TYPE, "client_id": CLIENT_ID}
-            resp = requests.post(URL_TOKEN, data)
-            if resp.status_code == 400:
-                j = resp.json()
-                logger.debug(j)
-                if j["error"] == "authorization_pending":
-                    logger.warning(j["error_description"])
-                    self.auth_error_detected.emit(j["error_description"])
-                    return
-                else:
-                    raise Exception(j["error_description"])
-            resp.raise_for_status()
-            j = resp.json()
-            access_token = j["access_token"]
-            refresh_token = j["refresh_token"]
-            logger.debug("OAuth device code flow successful")
-            self.access_token_received.emit(access_token, refresh_token)
-            self.finished.emit()
-        except requests.exceptions.RequestException as e:
-            logger.error(f"Request failed: {e}")
-            self.error_occurred.emit(str(e))
-            self.finished.emit()
-    
     def send_enter(self):
-        self.poll_for_token()
+        if self.process and self.process.poll() is None:
+            self.process.stdin.write("\n")
+            self.process.stdin.flush()
 
     def stop(self):
         self.is_running = False
+        if self.process:
+            self.process.terminate()
 
-class MinecraftAuthenticator(QObject):
+class MinecraftAuthenticator(QObject):  # Changed to inherit from QObject
-    auth_finished = pyqtSignal(bool)
+    auth_finished = pyqtSignal(bool)  # Add signal for completion
 
     def __init__(self, parent=None):
         super().__init__(parent)
@@ -166,12 +175,15 @@ class MinecraftAuthenticator(QObject):
         self.success = False
         
     def authenticate(self, username):
+        """
+        Start the authentication process for the given username
+        Returns immediately, authentication result will be emitted via auth_finished signal
+        """
         self.success = False
         self.auth_thread = AuthenticationThread(username)
         self.auth_thread.auth_data_received.connect(self.show_auth_dialog)
         self.auth_thread.auth_error_detected.connect(self.handle_auth_error)
         self.auth_thread.error_occurred.connect(self.show_error)
-        self.auth_thread.access_token_received.connect(self.on_access_token_received)
         self.auth_thread.finished.connect(self.on_authentication_finished)
         self.auth_thread.start()
 
@@ -205,11 +217,6 @@ class MinecraftAuthenticator(QObject):
         self.success = False
         self.auth_finished.emit(False)
 
-    def on_access_token_received(self, access_token, refresh_token):
-        QMessageBox.information(None, "Success", "Authentication successful!")
-        self.success = True
-        self.auth_finished.emit(True)
-
     def on_authentication_finished(self):
         if self.auth_dialog is not None:
             self.auth_dialog.close()
@@ -219,8 +226,8 @@ class MinecraftAuthenticator(QObject):
             self.auth_thread.stop()
             self.auth_thread = None
             
-        if not self.success:
+        self.success = True
-            self.auth_finished.emit(False)
+        self.auth_finished.emit(True)
 
     def cleanup(self):
         if self.auth_dialog is not None:
@@ -233,7 +240,5 @@ class MinecraftAuthenticator(QObject):
 
 # Example usage
 if __name__ == '__main__':
-    app = QApplication(sys.argv)
     authenticator = MinecraftAuthenticator()
     authenticator.authenticate("TestUser")
-    sys.exit(app.exec_())

modulecli.py

@@ -1,30 +0,0 @@
-import click
-from picomc.cli.main import picomc_cli
-from io import StringIO
-import sys
-
-def run_command(command="picomc"):
-    # Redirect stdout and stderr to capture the command output
-    old_stdout, old_stderr = sys.stdout, sys.stderr
-    sys.stdout = mystdout = StringIO()
-    sys.stderr = mystderr = StringIO()
-    
-    try:
-        picomc_cli.main(args=command.split())
-    except SystemExit as e:
-        if e.code != 0:
-            print(f"Command exited with code {e.code}", file=sys.stderr)
-    except Exception as e:
-        print(f"Unexpected error: {e}", file=sys.stderr)
-    finally:
-        # Restore stdout and stderr
-        sys.stdout = old_stdout
-        sys.stderr = old_stderr
-
-    output = mystdout.getvalue().strip()
-    error = mystderr.getvalue().strip()
-    
-    if not output:
-        return f"Error: No output from command. Stderr: {error}"
-    
-    return output

picodulce.py

@@ -13,14 +13,13 @@ import time
 
 from authser import MinecraftAuthenticator
 from healthcheck import HealthCheck
-import modulecli
 
 from PyQt5.QtWidgets import QApplication, QComboBox, QWidget, QInputDialog, QVBoxLayout, QListWidget, QPushButton, QMessageBox, QDialog, QHBoxLayout, QLabel, QLineEdit, QCheckBox, QTabWidget, QFrame, QSpacerItem, QSizePolicy, QMainWindow, QGridLayout, QTextEdit, QListWidget, QListWidgetItem, QMenu
 from PyQt5.QtGui import QFont, QIcon, QColor, QPalette, QMovie, QPixmap, QDesktopServices, QBrush
 from PyQt5.QtCore import Qt, QObject, pyqtSignal, QThread, QUrl, QMetaObject, Q_ARG, QByteArray, QSize
 from datetime import datetime
 
-logging.basicConfig(level=logging.ERROR, format='%(levelname)s - %(message)s')
+logging.basicConfig(level=logging.ERROR, format='%(asctime)s - %(levelname)s - %(message)s')
 
 class PicomcVersionSelector(QWidget):
     def __init__(self):
@@ -127,13 +126,11 @@ class PicomcVersionSelector(QWidget):
         try:
             self.config_path = "config.json"
             print("Running picomc instance create default command...")
-            
+            # Run the command using subprocess
-            # Run the command using modulecli
+            result = subprocess.run(["picomc", "instance", "create", "default"], check=True, capture_output=True, text=True)
-            command = "instance create default"
-            result = modulecli.run_command(command)
             
             # Print the output of the command
-            print("Command output:", result)
+            print("Command output:", result.stdout)
             
             # Change the value of IsFirstLaunch to False
             self.config["IsFirstLaunch"] = False
@@ -144,9 +141,9 @@ class PicomcVersionSelector(QWidget):
                 json.dump(self.config, f, indent=4)
             print("Configuration saved to", self.config_path)
 
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
             print("An error occurred while creating the instance.")
-            print("Error output:", str(e))
+            print("Error output:", e.stderr)
 
     def resize_event(self, event):
         if hasattr(self, 'movie_label'):
@@ -397,8 +394,8 @@ class PicomcVersionSelector(QWidget):
                 discord_rcp_checkbox.isChecked(),
                 check_updates_checkbox.isChecked(),
                 theme_background_checkbox.isChecked(),
-                self.selected_theme,
+                self.selected_theme,  # Pass the selected theme here
-                bleeding_edge_checkbox.isChecked() 
+                bleeding_edge_checkbox.isChecked()  # Pass the bleeding edge setting here
             )
         )
 
@@ -662,6 +659,20 @@ class PicomcVersionSelector(QWidget):
         )
         self.__init__()
 
+    def get_palette(self, palette_type):
+        """Retrieve the corresponding palette based on the palette type."""
+        palettes = {
+            "Dark": self.create_dark_palette,
+            "Obsidian": self.create_obsidian_palette,
+            "Redstone": self.create_redstone_palette,
+            "Alpha": self.create_alpha_palette,
+            "Strawberry": self.create_strawberry_palette,
+            "Native": self.create_native_palette,
+            "Christmas": self.create_christmas_palette,
+        }
+        # Default to dark palette if the type is not specified or invalid
+        return palettes.get(palette_type, self.create_dark_palette)()
+
     def get_system_info(self):
         # Get system information
         java_version = subprocess.getoutput("java -version 2>&1 | head -n 1")
@@ -701,16 +712,16 @@ class PicomcVersionSelector(QWidget):
 
     def open_game_directory(self):
         try:
-            # Run the command using modulecli
+            # Run the command and capture the output
-            command = "instance dir"
+            result = subprocess.run(['picomc', 'instance', 'dir'], capture_output=True, text=True, check=True)
-            result = modulecli.run_command(command)
+            game_directory = result.stdout.strip()
-            game_directory = result.strip()
 
             # Open the directory in the system's file explorer
             QDesktopServices.openUrl(QUrl.fromLocalFile(game_directory))
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
             print(f"Error running picomc command: {e}")
 
+
     def populate_installed_versions(self):
         config_path = "config.json"
         
@@ -731,17 +742,20 @@ class PicomcVersionSelector(QWidget):
 
         # Run the command and capture the output
         try:
-            command = "version list"
+            process = subprocess.Popen(['picomc', 'version', 'list'], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-            output = modulecli.run_command(command)
+            output, error = process.communicate()
             
-            if not output:
+            if process.returncode != 0:
-                raise Exception("Failed to get output from modulecli")
+                raise subprocess.CalledProcessError(process.returncode, process.args, output=output, stderr=error)
-        except Exception as e:
+        except FileNotFoundError:
-            logging.error("Error running 'picomc': %s", e)
+            logging.error("'picomc' command not found. Please ensure it's installed and in your PATH.")
+            return
+        except subprocess.CalledProcessError as e:
+            logging.error("Error running 'picomc': %s", e.stderr)
             return
 
         # Parse the output and replace '[local]' with a space
-        versions = [version.replace('[local]', ' ').strip() for version in output.splitlines() if version.strip()]
+        versions = [version.replace('[local]', ' ').strip() for version in output.splitlines()]
 
         # Get the last played version from the config
         last_played = self.config.get("LastPlayed", "")
@@ -758,22 +772,28 @@ class PicomcVersionSelector(QWidget):
     def populate_installed_versions_normal_order(self):
         # Run the 'picomc instance create default' command at the start
         try:
-            command = "instance create default"
+            process = subprocess.Popen(['picomc', 'instance', 'create', 'default'], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-            output = modulecli.run_command(command)
+            output, error = process.communicate()
-            if not output:
+            if process.returncode != 0:
-                raise Exception("Failed to get output from modulecli for 'instance create default'")
+                raise subprocess.CalledProcessError(process.returncode, process.args, error)
-        except Exception as e:
+        except FileNotFoundError:
-            logging.error("Error creating default instance: %s", str(e))
+            logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
+            return
+        except subprocess.CalledProcessError as e:
+            logging.error("Error creating default instance: %s", e.stderr)
             return
 
         # Run the 'picomc version list' command and get the output
         try:
-            command = "version list"
+            process = subprocess.Popen(['picomc', 'version', 'list'], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-            output = modulecli.run_command(command)
+            output, error = process.communicate()
-            if not output:
+            if process.returncode != 0:
-                raise Exception("Failed to get output from modulecli for 'version list'")
+                raise subprocess.CalledProcessError(process.returncode, process.args, error)
-        except Exception as e:
+        except FileNotFoundError:
-            logging.error("Error: %s", str(e))
+            logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
+            return
+        except subprocess.CalledProcessError as e:
+            logging.error("Error: %s", e.stderr)
             return
 
         # Parse the output and replace '[local]' with a space
@@ -800,7 +820,7 @@ class PicomcVersionSelector(QWidget):
 
         # Check if there are any accounts
         try:
-            account_list_output = modulecli.run_command("account list").strip()
+            account_list_output = subprocess.check_output(["picomc", "account", "list"]).decode("utf-8").strip()
             if not account_list_output:
                 QMessageBox.warning(self, "No Account Available", "Please create an account first.")
                 return
@@ -809,25 +829,18 @@ class PicomcVersionSelector(QWidget):
             if '*' not in account_list_output:
                 QMessageBox.warning(self, "No Account Selected", "Please select an account.")
                 return
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
             error_message = f"Error fetching accounts: {str(e)}"
             logging.error(error_message)
             QMessageBox.critical(self, "Error", error_message)
             return
 
         selected_instance = self.installed_version_combo.currentText()
-        logging.info(f"Selected instance from dropdown: {selected_instance}")
+        logging.info(f"Selected instance: {selected_instance}")
-
-        # Verify the selected instance value before starting the game
-        if not selected_instance:
-            logging.error("No instance selected.")
-            QMessageBox.warning(self, "No Instance Selected", "Please select an instance.")
-            return
 
         play_thread = threading.Thread(target=self.run_game, args=(selected_instance,))
         play_thread.start()
 
-
     def run_game(self, selected_instance):
         try:
             # Set current_state to the selected instance
@@ -842,14 +855,10 @@ class PicomcVersionSelector(QWidget):
             update_thread = threading.Thread(target=self.update_last_played, args=(selected_instance,))
             update_thread.start()
 
-            # Run the game using the modulecli module
+            # Run the game subprocess with the instance_value from config.json
-            command = f"instance launch --version-override {selected_instance} {instance_value}"
+            subprocess.run(['picomc', 'instance', 'launch', '--version-override', selected_instance, instance_value], check=True)
-            output = modulecli.run_command(command)
 
-            if not output:
+        except subprocess.CalledProcessError as e:
-                raise Exception("Failed to get output from modulecli")
-
-        except Exception as e:
             error_message = f"Error playing {selected_instance}: {e}"
             logging.error(error_message)
             # Use QMetaObject.invokeMethod to call showError safely
@@ -861,7 +870,6 @@ class PicomcVersionSelector(QWidget):
             # Reset current_state to "menu" after the game closes
             self.current_state = "menu"
             
-            
     def update_last_played(self, selected_instance):
         config_path = "config.json"
         self.config["LastPlayed"] = selected_instance
@@ -959,16 +967,16 @@ class PicomcVersionSelector(QWidget):
             return
 
         try:
-            command = f"account create {username}"
+            command = ['picomc', 'account', 'create', username]
             if is_microsoft:
-                command += " --ms"
+                command.append('--ms')
 
-            modulecli.run_command(command)
+            subprocess.run(command, check=True)
             QMessageBox.information(dialog, "Success", f"Account '{username}' created successfully!")
             self.populate_accounts_for_all_dialogs()
             dialog.accept()
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
-            error_message = f"Error creating account: {str(e)}"
+            error_message = f"Error creating account: {e.stderr.decode()}"
             logging.error(error_message)
             QMessageBox.critical(dialog, "Error", error_message)
 
@@ -1021,20 +1029,22 @@ class PicomcVersionSelector(QWidget):
         confirm_dialog = QMessageBox.question(dialog, "Confirm Removal", confirm_message, QMessageBox.Yes | QMessageBox.No, QMessageBox.No)
         if confirm_dialog == QMessageBox.Yes:
             try:
-                command = f"account remove {username}"
+                subprocess.run(['picomc', 'account', 'remove', username], check=True)
-                modulecli.run_command(command)
                 QMessageBox.information(dialog, "Success", f"Account '{username}' removed successfully!")
                 self.populate_accounts_for_all_dialogs()
-            except Exception as e:
+            except subprocess.CalledProcessError as e:
-                error_message = f"Error removing account: {str(e)}"
+                error_message = f"Error removing account: {e.stderr.decode()}"
                 logging.error(error_message)
                 QMessageBox.critical(dialog, "Error", error_message)
 
+
     def populate_accounts(self, account_combo):
         # Populate the account dropdown
         try:
-            command = "account list"
+            process = subprocess.Popen(['picomc', 'account', 'list'], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-            output = modulecli.run_command(command)
+            output, error = process.communicate()
+            if process.returncode != 0:
+                raise subprocess.CalledProcessError(process.returncode, process.args, error)
 
             # Process accounts, keeping the one with "*" at the top
             accounts = output.splitlines()
@@ -1059,8 +1069,10 @@ class PicomcVersionSelector(QWidget):
             for account in normal_accounts:
                 account_combo.addItem(account)
 
-        except Exception as e:
+        except FileNotFoundError:
-            logging.error(f"Error: {str(e)}")
+            logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
+        except subprocess.CalledProcessError as e:
+            logging.error(f"Error: {e.stderr}")
 
     def populate_accounts_for_all_dialogs(self):
         # Update account dropdowns in all open dialogs
@@ -1077,16 +1089,14 @@ class PicomcVersionSelector(QWidget):
             return
 
         try:
-            command = f"account setdefault {account_name}"
+            subprocess.run(['picomc', 'account', 'setdefault', account_name], check=True)
-            modulecli.run_command(command)
             QMessageBox.information(self, "Success", f"Account '{account_name}' set as default!")
             self.populate_accounts_for_all_dialogs()
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
-            error_message = f"Error setting default account '{account_name}': {str(e)}"
+            error_message = f"Error setting default account '{account_name}': {e.stderr.decode()}"
             logging.error(error_message)
             QMessageBox.critical(self, "Error", error_message)
 
-
     def show_about_dialog(self):
         # Load the version number from version.json
         try:
@@ -1316,11 +1326,10 @@ class DownloadThread(QThread):
 
     def run(self):
         try:
-            command = f"version prepare {self.version}"
+            subprocess.run(['picomc', 'version', 'prepare', self.version], check=True)
-            modulecli.run_command(command)
             self.completed.emit(True, f"Version {self.version} prepared successfully!")
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
-            error_message = f"Error preparing {self.version}: {str(e)}"
+            error_message = f"Error preparing {self.version}: {e.stderr.decode()}"
             self.completed.emit(False, error_message)
 
 class ModLoaderAndVersionMenu(QDialog):
@@ -1337,11 +1346,11 @@ class ModLoaderAndVersionMenu(QDialog):
         # Create tabs
         install_mod_tab = QWidget()
         download_version_tab = QWidget()
-        instances_tab = QWidget()
+        instances_tab = QWidget()  # New tab for instances
 
         tab_widget.addTab(download_version_tab, "Download Version")
         tab_widget.addTab(install_mod_tab, "Install Mod Loader")
-        tab_widget.addTab(instances_tab, "Instances")
+        tab_widget.addTab(instances_tab, "Instances")  # Add the new tab
 
         # Add content to "Install Mod Loader" tab
         self.setup_install_mod_loader_tab(install_mod_tab)
@@ -1393,8 +1402,11 @@ class ModLoaderAndVersionMenu(QDialog):
         if instance_name:
             try:
                 # Run the "picomc instance create" command
-                command = f"instance create {instance_name}"
+                process = subprocess.Popen(['picomc', 'instance', 'create', instance_name], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-                modulecli.run_command(command)
+                output, error = process.communicate()
+
+                if process.returncode != 0:
+                    raise subprocess.CalledProcessError(process.returncode, process.args, error)
 
                 # Notify the user that the instance was created
                 QMessageBox.information(self, "Instance Created", f"Instance '{instance_name}' has been created successfully.")
@@ -1405,9 +1417,11 @@ class ModLoaderAndVersionMenu(QDialog):
                 # Optionally select the newly created instance
                 self.on_instance_selected(self.instances_list_widget.item(self.instances_list_widget.count() - 1))
 
-            except Exception as e:
+            except FileNotFoundError:
-                logging.error("Error creating instance: %s", str(e))
+                logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
-                QMessageBox.critical(self, "Error", f"Failed to create instance: {str(e)}")
+            except subprocess.CalledProcessError as e:
+                logging.error("Error creating instance: %s", e.stderr)
+                QMessageBox.critical(self, "Error", f"Failed to create instance: {e.stderr}")
         else:
             QMessageBox.warning(self, "Invalid Input", "Please enter a valid instance name.")
 
@@ -1418,8 +1432,14 @@ class ModLoaderAndVersionMenu(QDialog):
 
         try:
             # Run the "picomc instance rename" command
-            command = f"instance rename {old_instance_name} {new_instance_name}"
+            process = subprocess.Popen(
-            modulecli.run_command(command)
+                ['picomc', 'instance', 'rename', old_instance_name, new_instance_name],
+                stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True
+            )
+            output, error = process.communicate()
+
+            if process.returncode != 0:
+                raise subprocess.CalledProcessError(process.returncode, process.args, error)
 
             QMessageBox.information(self, "Instance Renamed", f"Instance '{old_instance_name}' has been renamed to '{new_instance_name}' successfully.")
 
@@ -1431,9 +1451,11 @@ class ModLoaderAndVersionMenu(QDialog):
             if matching_items:
                 self.instances_list_widget.setCurrentItem(matching_items[0])
 
-        except Exception as e:
+        except FileNotFoundError:
-            logging.error("Error renaming instance: %s", str(e))
+            logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
-            QMessageBox.critical(self, "Error", f"Failed to rename instance: {str(e)}")
+        except subprocess.CalledProcessError as e:
+            logging.error("Error renaming instance: %s", e.stderr)
+            QMessageBox.critical(self, "Error", f"Failed to rename instance: {e.stderr}")
 
     def delete_instance(self, instance_name):
         if instance_name == "default":
@@ -1448,8 +1470,11 @@ class ModLoaderAndVersionMenu(QDialog):
         if confirm_delete == QMessageBox.Yes:
             try:
                 # Run the "picomc instance delete" command
-                command = f"instance delete {instance_name}"
+                process = subprocess.Popen(['picomc', 'instance', 'delete', instance_name], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-                modulecli.run_command(command)
+                output, error = process.communicate()
+
+                if process.returncode != 0:
+                    raise subprocess.CalledProcessError(process.returncode, process.args, error)
 
                 # Notify the user that the instance was deleted
                 QMessageBox.information(self, "Instance Deleted", f"Instance '{instance_name}' has been deleted successfully.")
@@ -1457,15 +1482,18 @@ class ModLoaderAndVersionMenu(QDialog):
                 # Reload the instances list
                 self.load_instances()
 
-            except Exception as e:
+            except FileNotFoundError:
-                logging.error("Error deleting instance: %s", str(e))
+                logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
-                QMessageBox.critical(self, "Error", f"Failed to delete instance: {str(e)}")
+            except subprocess.CalledProcessError as e:
+                logging.error("Error deleting instance: %s", e.stderr)
+                QMessageBox.critical(self, "Error", f"Failed to delete instance: {e.stderr}")
 
     def load_instances(self):
         try:
-            # Run the "picomc instance list" command
+            process = subprocess.Popen(['picomc', 'instance', 'list'], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-            command = "instance list"
+            output, error = process.communicate()
-            output = modulecli.run_command(command)
+            if process.returncode != 0:
+                raise subprocess.CalledProcessError(process.returncode, process.args, error)
 
             # Parse the output and add each instance to the list widget
             instances = output.splitlines()
@@ -1475,9 +1503,10 @@ class ModLoaderAndVersionMenu(QDialog):
                 self.instances_list_widget.addItem(item)
                 self.add_instance_buttons(item, instance)
 
-        except Exception as e:
+        except FileNotFoundError:
-            logging.error("Error fetching instances: %s", str(e))
+            logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
-
+        except subprocess.CalledProcessError as e:
+            logging.error("Error fetching instances: %s", e.stderr)
 
     def add_instance_buttons(self, list_item, instance_name):
         widget = QWidget()
@@ -1633,19 +1662,21 @@ class ModLoaderAndVersionMenu(QDialog):
                 options.append('--beta')
             if options:
                 try:
-                    command = 'version list ' + ' '.join(options)
+                    process = subprocess.Popen(['picomc', 'version', 'list'] + options, stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-                    output = modulecli.run_command(command)
+                    output, error = process.communicate()
-                    if "Error" in output:
+                    if process.returncode != 0:
-                        logging.error(output)
+                        raise subprocess.CalledProcessError(process.returncode, process.args, error)
+                except FileNotFoundError:
+                    logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
+                    return
+                except subprocess.CalledProcessError as e:
+                    logging.error("Error: %s", e.stderr)
                     return
 
                 # Parse the output and replace '[local]' with a space
                 versions = output.splitlines()
                 versions = [version.replace('[local]', ' ').strip() for version in versions]
                 self.version_combo.addItems(versions)
-                except Exception as e:
-                    logging.error("Unexpected error: %s", e)
-                    return
             # Update the download button state whenever versions are updated
             self.update_download_button_state()
 
@@ -1702,10 +1733,15 @@ class ModLoaderAndVersionMenu(QDialog):
 
     def populate_available_releases(self, version_combo, install_forge, install_fabric):
         try:
-            command = "version list --release"
+            process = subprocess.Popen(['picomc', 'version', 'list', '--release'], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-            output = modulecli.run_command(command)
+            output, error = process.communicate()
-        except Exception as e:
+            if process.returncode != 0:
-            logging.error("Error: %s", str(e))
+                raise subprocess.CalledProcessError(process.returncode, process.args, error)
+        except FileNotFoundError:
+            logging.error("'picomc' command not found. Please make sure it's installed and in your PATH.")
+            return
+        except subprocess.CalledProcessError as e:
+            logging.error("Error: %s", e.stderr)
             return
 
         if install_fabric:
@@ -1735,13 +1771,12 @@ class ModLoaderAndVersionMenu(QDialog):
 
         try:
             if mod_loader == 'forge':
-                command = f"mod loader forge install --game {version}"
+                subprocess.run(['picomc', 'mod', 'loader', 'forge', 'install', '--game', version], check=True)
             elif mod_loader == 'fabric':
-                command = f"mod loader fabric install {version}"
+                subprocess.run(['picomc', 'mod', 'loader', 'fabric', 'install', version], check=True)
-            modulecli.run_command(command)
             QMessageBox.information(self, "Success", f"{mod_loader.capitalize()} installed successfully for version {version}!")
-        except Exception as e:
+        except subprocess.CalledProcessError as e:
-            error_message = f"Error installing {mod_loader} for version {version}: {str(e)}"
+            error_message = f"Error installing {mod_loader} for version {version}: {e.stderr.decode()}"
             QMessageBox.critical(self, "Error", error_message)
             logging.error(error_message)
 

version.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.13",
+  "version": "0.12.1",
   "links": [
     "https://raw.githubusercontent.com/nixietab/picodulce/main/version.json",
     "https://raw.githubusercontent.com/nixietab/picodulce/main/picodulce.py",
@@ -8,8 +8,7 @@
     "https://raw.githubusercontent.com/nixietab/picodulce/main/marroc.py",
     "https://raw.githubusercontent.com/nixietab/picodulce/main/holiday.ico",
     "https://raw.githubusercontent.com/nixietab/picodulce/main/authser.py",
-    "https://raw.githubusercontent.com/nixietab/picodulce/main/healthcheck.py",
+    "https://raw.githubusercontent.com/nixietab/picodulce/main/healthcheck.py"
-    "https://raw.githubusercontent.com/nixietab/picodulce/main/modulecli.py"
   ],
-  "versionBleeding": "0.13-194"
+  "versionBleeding": "0.12.1-188"
 }