ruff fixes

Author: FredHappyface

cli2gui/application/application.py

@@ -6,14 +6,14 @@
 import sys
 from typing import Any
 
-from cli2gui import types
+from cli2gui import models
 from cli2gui.application.application2args import argFormat
 from cli2gui.gui import helpers
 from cli2gui.gui.dearpygui_wrapper import DearPyGuiWrapper
 from cli2gui.gui.pysimplegui_wrapper import PySimpleGUIWrapper
 
 
-def run(buildSpec: types.FullBuildSpec) -> Any:
+def run(buildSpec: models.FullBuildSpec) -> Any:
 	"""Establish the main entry point.
 
 	Args:

cli2gui/application/application2args.py

@@ -7,7 +7,7 @@
 from pathlib import Path
 from typing import Any
 
-from cli2gui.types import SEP, ParserType
+from cli2gui.models import SEP, ParserType
 
 
 def processValue(key: str, value: str) -> tuple[str, Any]:
@@ -44,44 +44,44 @@ def processValue(key: str, value: str) -> tuple[str, Any]:
 def argparseFormat(values: dict[str, Any]) -> argparse.Namespace:
 	"""Format args for argparse."""
 	args = {}
-	for key in values:
-		cleankey, value = processValue(key, values[key])
+	for key, _value in values.items():
+		cleankey, value = processValue(key, _value)
 		args[cleankey] = value
 	return argparse.Namespace(**args)
 
 
 def optparseFormat(values: dict[str, Any]) -> tuple[optparse.Values, list[str]]:
 	"""Format args for optparse."""
 	args = {}
-	for key in values:
-		cleankey, value = processValue(key, values[key])
+	for key, _value in values.items():
+		cleankey, value = processValue(key, _value)
 		args[cleankey] = value
 	return (optparse.Values(args), [])
 
 
 def getoptFormat(values: dict[str, Any]) -> tuple[list[Any], list[Any]]:
 	"""Format args for getopt."""
-	return ([processValue(key, values[key]) for key in values if values[key]], [])
+	return ([processValue(key, _value) for key, _value in values.items() if _value], [])
 
 
 def docoptFormat(values: dict[str, Any]) -> dict[str, Any]:
 	"""Format args for docopt."""
 	import docopt
 
 	args = {}
-	for key in values:
-		cleankey, value = processValue(key, values[key])
+	for key, _value in values.items():
+		cleankey, value = processValue(key, _value)
 		args[cleankey] = value
 	return docopt.Dict(args)
 
 
 def clickFormat(values: dict[str, Any]) -> list[Any]:
 	"""Format args for click."""
 	args = []
-	for key in values:
-		val = str(values[key])
+	for key, _value in values.items():
+		val = str(_value)
 		if not callable(key) and len(val) > 0:
-			cleankey, value = processValue(key, values[key])
+			cleankey, value = processValue(key, _value)
 			args.extend([cleankey, value])
 	return args
 

cli2gui/decorators.py

@@ -13,14 +13,14 @@
 from typing import Any, Iterable
 
 from cli2gui.application import application
+from cli2gui.models import BuildSpec, FullBuildSpec, GUIType, ParserType
 from cli2gui.tojson import (
 	argparse2json,
 	click2json,
 	docopt2json,
 	getopt2json,
 	optparse2json,
 )
-from cli2gui.types import BuildSpec, FullBuildSpec, GUIType, ParserType
 
 DO_COMMAND = "--cli2gui"
 DO_NOT_COMMAND = "--disable-cli2gui"

cli2gui/gui/__init__.py

@@ -0,0 +1,2 @@
+"""Package containing GUI classes and functionality, responsible for driving the various
+GUI backends supported by cli2gui."""

cli2gui/gui/abstract_gui.py

@@ -1,20 +1,26 @@
+"""Abstract base class for GUI wrappers."""
+
 from __future__ import annotations
 
 from abc import ABC, abstractmethod
+from typing import Any, Callable
 
-from cli2gui import types
+from cli2gui import models
 
 
 class AbstractGUI(ABC):
 	"""Abstract base class for GUI wrappers."""
 
 	@abstractmethod
 	def __init__(self) -> None:
-		pass
+		"""Abstract base class for GUI wrappers."""
 
 	@abstractmethod
 	def main(
-		self, buildSpec: types.FullBuildSpec, menu: list[str], quit_callback, run_callback
+		self,
+		buildSpec: models.FullBuildSpec,
+		quit_callback: Callable[[], None],
+		run_callback: Callable[[dict[str, Any]], None],
 	) -> None:
 		"""Abstract method for the main function."""
 		raise NotImplementedError

cli2gui/gui/dearpygui_wrapper.py

@@ -1,3 +1,5 @@
+"""Wrapper class for Dear PyGui."""
+
 from __future__ import annotations
 
 from pathlib import Path
@@ -7,7 +9,7 @@
 
 from cli2gui.gui import helpers
 from cli2gui.gui.abstract_gui import AbstractGUI
-from cli2gui.types import SEP, FullBuildSpec, Group, Item, ItemType
+from cli2gui.models import SEP, FullBuildSpec, Group, Item, ItemType
 
 THISDIR = Path(__file__).resolve().parent
 
@@ -25,6 +27,11 @@ class DearPyGuiWrapper(AbstractGUI):
 	"""Wrapper class for Dear PyGui."""
 
 	def __init__(self, base24Theme: list[str]) -> None:
+		"""Dearpygui wrapper class.
+
+		:param list[str] base24Theme: list representing a base24 theme. Containing 24 elements
+		(of hex strings like "#e7e7e9")
+		"""
 		self.base24Theme = base24Theme
 		super().__init__()
 

cli2gui/gui/helpers.py

@@ -1,3 +1,6 @@
+"""Basic helper package containing commonly used functions, such as isDarkMode,
+get_base24_theme etc."""
+
 from __future__ import annotations
 
 from pathlib import Path

cli2gui/gui/pysimplegui_wrapper.py

@@ -1,3 +1,5 @@
+"""Wrapper class for PySimpleGUI."""
+
 from __future__ import annotations
 
 import io
@@ -8,15 +10,19 @@
 
 from cli2gui.gui import helpers
 from cli2gui.gui.abstract_gui import AbstractGUI
-from cli2gui.types import SEP, FullBuildSpec, Group, Item, ItemType
-
-# ruff: noqa: ANN401
+from cli2gui.models import SEP, FullBuildSpec, Group, Item, ItemType
 
 
 class PySimpleGUIWrapper(AbstractGUI):
 	"""Wrapper class for PySimpleGUI."""
 
 	def __init__(self, base24Theme: list[str], psg_lib: str) -> None:
+		"""PySimpleGUI wrapper class.
+
+		:param list[str] base24Theme: list representing a base24 theme. Containing 24 elements
+		(of hex strings like "#e7e7e9")
+		:param str psg_lib: string representing the pysimplegui lib to use
+		"""
 		super().__init__()
 
 		if psg_lib == "psg":
@@ -138,14 +144,14 @@ def _fileBrowser(
 		additional_properties: dict | None = None,
 	) -> list[Any]:
 		"""Return a fileBrowser button and field."""
-		additional_properties = additional_properties or {}
+		prop = additional_properties or {}
 
 		height = self.sizes["input_size"][1]
 		width = self.sizes["input_size"][0]
 
 		key = f"{key}{SEP}{_type}"
 		if _type in [ItemType.FileWrite, ItemType.File]:
-			key += f";{additional_properties.get('file_mode')};{additional_properties.get('file_encoding')}"
+			key += f";{prop.get('file_mode')};{prop.get('file_encoding')}"
 
 		browser = self.sg.FileBrowse(
 			key="@@" + key,
@@ -394,15 +400,10 @@ def createLayout(
 	) -> list[list[Any]]:
 		"""Create the pysimplegui layout from the build spec.
 
-		Args:
-		----
-			buildSpec (FullBuildSpec): build spec containing widget
-			self (self): class to build self
-
+		:param FullBuildSpec buildSpec: build spec containing widget
+		:param str | list[str] menu: menu definition. containing menu keys
 
-		Returns:
-		-------
-			list[list[Any]]: list of self (layout list)
+		:return list[list[Any]]: list of self (layout list)
 
 		"""
 		argConstruct = []

cli2gui/types.py renamed to cli2gui/models.py

@@ -98,13 +98,6 @@ class FullBuildSpec:
 class ParserType(str, Enum):
 	"""Supported parser types.
 
-	OPTPARSE = "optparse"
-	ARGPARSE = "argparse"
-	DEPHELL_ARGPARSE = "dephell_argparse"
-	DOCOPT = "docopt"
-	GETOPT = "getopt"
-	CLICK = "click"
-	CUSTOM = "input()"  # this seems like a pretty poor pattern to use
 	"""
 
 	OPTPARSE = "optparse"
@@ -120,13 +113,12 @@ class ParserType(str, Enum):
 class GUIType(str, Enum):
 	"""Supported gui types.
 
-	DEFAULT = "pysimplegui"
-	WEB = "pysimpleguiweb"
-	QT = "pysimpleguiqt"
-	FSG = "freesimplegui"
 	"""
 
-	DEFAULT = "pysimplegui"
+	PSG = "pysimplegui"
 	WEB = "pysimpleguiweb"
 	QT = "pysimpleguiqt"
 	FSG = "freesimplegui"
+	FSGWEB = "freesimpleguiweb"
+	FSGQT = "freesimpleguiqt"
+	DPG = "dearpygui"

cli2gui/tojson/__init__.py

@@ -0,0 +1,2 @@
+"""Package containing transforms for each supported parser/cli library, responsible for
+taking each parser representation (and BuildSpec) and generating a FullBuildSpec."""