Everything is split into modules and put into classes where inheritance is needed

src/cli.py

@@ -0,0 +1,10 @@
+import argparse
+
+def interface():
+	parser = argparse.ArgumentParser()
+	parser.add_argument("--server", required=True, help="")
+	parser.add_argument("--cpu-temp-critical", type=int, default=80, help="cpu tempature crtitical. default = 80")
+	parser.add_argument("--cpu-temp-warning",  type=int, default=70, help="cpu tempature warning. default = 70")
+	parser.add_argument("--update-rate",       type=int, default=1,  help="how often updates happen in seconds. default = 1")
+
+	return parser.parse_args()

src/cpu.py

@@ -1,26 +1,35 @@
 import subprocess
-import main
-import ntfy
 import time
 import re
 
 from typing import Optional
+from ntfy import Ntfy
 
-last_cpu_check: float = time.time()
+last_cpu_check_unix: float = time.time()
+last_cpu_check:      int   = 60 # Seconds
 
-def temperature() -> Optional[float]:
+class Tempature:
-	sensors_out = subprocess.check_output(["sensors"]).decode()
+	cpu_temp_crtitical_message: str = "🔥 CPU tempature is at critical tempatures!"
-	for line in sensors_out.splitlines():
+	cpu_temp_warning_message:   str = "🌡️ CPU tempature is at a high tempature."
-		if "Tctl" in line:
-			match = re.search(r"(\d+\.\d+)°C", line)
-			if match:
-				return float(match.group(1))
-	return None
 
-def temperature_check():
+	def __init__(self, ntfy_instance: Ntfy, cpu_critical_temp: int, cpu_warning_temp: int):
-	cpu_temp = temperature()
+		self.cpu_critical_temp = cpu_critical_temp
-	if cpu_temp and (time.time() - last_cpu_check) > main.last_check_debounce * 1000:
+		self.cpu_warning_temp = cpu_warning_temp
-		if cpu_temp >= main.cpu_critical_temp:
+		self.ntfy = ntfy_instance
-			ntfy.send(f"🔥 CPU tempature is at critical tempatures! {cpu_temp}")
+
-		elif cpu_temp >= main.cpu_warning_temp:
+	def get(self) -> Optional[float]:
-			ntfy.send(f"🌡️ CPU tempature is at a high tempature. {cpu_temp}")
+		sensors_out = subprocess.check_output(["sensors"]).decode()
+		for line in sensors_out.splitlines():
+			if "Tctl" in line:
+				match = re.search(r"(\d+\.\d+)°C", line)
+				if match:
+					return float(match.group(1))
+		return None
+
+	def ntfy_check(self):
+		cpu_temp = self.get()
+		if cpu_temp and (time.time() - last_cpu_check_unix) > last_cpu_check * 1000:
+			if cpu_temp >= self.cpu_critical_temp:
+				self.ntfy.send(f"{Tempature.cpu_temp_crtitical_message} {cpu_temp}")
+			elif cpu_temp >= self.cpu_warning_temp:
+				self.ntfy.send(f"{Tempature.cpu_temp_warning_message} {cpu_temp}")

src/main.py

@@ -1,34 +1,33 @@
-import argparse
 import time
 
 import package
+import cli
 import cpu
 
-from typing import Optional
+from ntfy import Ntfy
 
-clock_interval_secs: int = 1
+def start(
-last_check_debounce: int = 60 # Seconds
+	cpu_critical_temp: int,
-cpu_critical_temp:   int = 80
+	cpu_warning_temp: int,
-cpu_warning_temp:    int = 70
+	ntfy_server: str,
+	interval: int
+):
+	ntfy = Ntfy(ntfy_server)
+	ntfy_cpu_temp_monitor = cpu.Tempature(ntfy, cpu_critical_temp, cpu_warning_temp)
 
-ntfy_url: Optional[str] = None
+	print(f"Started. {time.time()}")
-_ntfy_configure_prompt = """Please configure an ntfy url before starting.
+	print("Ntfy monitoring software is now listening.")
-\033[4mExamples:\033[0m
-\033[32mpython3 main.py --url=10.0.13.37:42069
-python3 main.py --url=ntfy.domain.com\033[0m"""
 
-def cli_interface() -> bool:
-	...
-
-def start():
-	print(f"Working! {time.time()}")
 	while True:
-		cpu.temperature_check()
+		ntfy_cpu_temp_monitor.ntfy_check()
-		time.sleep(clock_interval_secs)
+		time.sleep(interval)
 
 if __name__ == "__main__":
 	if package.installed_list(["lm-sensors", "ntfy", "curl"]):
-		if cli_interface():
+		cli_args = cli.interface()
-			start()
+		start(
-		else:
+			cli_args.cpu_temp_critical,
-			print(_ntfy_configure_prompt)
+			cli_args.cpu_temp_warning,
+			cli_args.server,
+			cli_args.update_rate,
+		)

src/ntfy.py

@@ -1,16 +1,11 @@
 import subprocess
-import main
-import time
 
-last_ntfy_send: float = time.time()
+class Ntfy:
+	def __init__(self, server: str):
+		self.server = server
 
-def send(message: str):
+	def send(self, message: str):
-	try:
+		try:
-		if main.ntfy_url:
+			subprocess.run(["curl", "-d", f"\"{message}\"", self.server], stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-			global last_ntfy_send
+		except Exception as err:
-			last_ntfy_send = time.time()
+			print(f"Ntfy failed. {err}")
-			subprocess.run(["curl", "-d", f"\"{message}\"", main.ntfy_url], stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-		else:
-			print("Ntfy send: url is not configured.")
-	except Exception as err:
-		print(f"Ntfy failed. {err}")