CI fixes

Author: FaserF

.github/scripts/check_relevance.py

@@ -1,16 +1,7 @@
 import subprocess
 import sys
-
-def get_last_tag():
-    try:
-        result = subprocess.run(
-            ["git", "tag", "--sort=-creatordate"],
-            capture_output=True, text=True, check=True
-        )
-        tags = result.stdout.strip().split('\n')
-        return tags[0] if tags and tags[0] else None
-    except:
-        return None
+import os
+from git_utils import get_last_tag
 
 def has_relevant_changes(since_tag):
     if not since_tag:
@@ -37,17 +28,22 @@ def has_relevant_changes(since_tag):
                 print(f"Relevant change detected in: {f}")
                 return True
         return False
-    except:
-        return True # Default to true if something fails
+    except subprocess.CalledProcessError:
+        return True # Default to true if git diff fails
 
 def main():
     last_tag = get_last_tag()
-    if has_relevant_changes(last_tag):
-        print("relevance=true")
-        sys.exit(0)
-    else:
-        print("relevance=false")
-        sys.exit(0)
+    relevance = "true" if has_relevant_changes(last_tag) else "false"
+
+    print(f"relevance={relevance}")
+
+    # Also write to GITHUB_OUTPUT if present
+    output_file = os.getenv("GITHUB_OUTPUT")
+    if output_file:
+        with open(output_file, "a") as f:
+            f.write(f"relevance={relevance}\n")
+
+    sys.exit(0)
 
 if __name__ == "__main__":
     main()

.github/scripts/generate_changelog.py

@@ -1,22 +1,6 @@
-import subprocess
-import os
-import re
 import argparse
+from git_utils import get_last_tag
 
-def get_last_tag():
-    """Returns the most recent git tag."""
-    try:
-        # Get all tags sorted by date desc
-        result = subprocess.run(
-            ["git", "tag", "--sort=-creatordate"],
-            capture_output=True, text=True, check=True
-        )
-        tags = result.stdout.strip().split('\n')
-        if not tags or tags[0] == "":
-            return None
-        return tags[0]
-    except subprocess.CalledProcessError:
-        return None
 
 def get_commits(since_tag=None):
     """Returns list of commit messages since the given tag (or all if None)."""
@@ -63,7 +47,7 @@ def parse_commits(commits):
             categories["Styling"].append(commit)
         elif lower_commit.startswith("docs"):
             categories["Documentation"].append(commit)
-        elif lower_commit.startswith(tuple(["chore", "refactor", "style", "test", "ci", "build"])):
+        elif lower_commit.startswith(tuple(["chore", "refactor", "test", "ci", "build"])):
             categories["Maintenance"].append(commit)
         else:
             # Filter out merge commits from "Other" list if we want clean output

.github/scripts/git_utils.py

@@ -0,0 +1,16 @@
+import subprocess
+
+def get_last_tag():
+    """Returns the most recent git tag."""
+    try:
+        # Get all tags sorted by date desc
+        result = subprocess.run(
+            ["git", "tag", "--sort=-creatordate"],
+            capture_output=True, text=True, check=True
+        )
+        tags = result.stdout.strip().split('\n')
+        if not tags or tags[0] == "":
+            return None
+        return tags[0]
+    except subprocess.CalledProcessError:
+        return None

.github/scripts/release_summary.py

@@ -1,5 +1,4 @@
 import sys
-import os
 
 def main():
     if len(sys.argv) < 3:

.github/workflows/nightly.yml .github/workflows/weekly.yml.github/workflows/nightly.yml renamed to .github/workflows/weekly.yml

@@ -1,4 +1,4 @@
-name: Nightly Release
+name: Weekly Release
 
 on:
   schedule:

.gitignore

@@ -113,3 +113,4 @@ test_flet_controls.py
 test_tabs.py
 test_tabs_2.py
 pytest_out.txt
+mock_script.ps1

src/generate_addons.py

@@ -38,13 +38,13 @@ def ask(self, query):
         if "hi" in q or "hallo" in q:
             return "Hallo! Ich bin der interner SwitchCraft AI Helper. Wie kann ich dir heute beim Paketieren helfen?"
         if "who are you" in q or "wer bist du" in q:
-            return "I am the built-in SwitchCraft AI Assistant. I can help with silent switches, MSI properties, and Intune deployments."
+            return "Ich bin der integrierte SwitchCraft AI Assistent. Ich kann dir bei Silent-Switches, MSI-Properties und Intune-Deployments helfen."
         if "silent" in q or "switches" in q:
             sw = self.ctx.get("install_silent", "/S")
             return f"Für dieses Paket wurden folgende Silent-Switches erkannt: `{sw}`. Du kannst diese im Packaging Wizard noch anpassen."
         if "intune" in q:
             return "Um Apps nach Intune hochzuladen, stelle sicher, dass du die Graph API Credentials in den Einstellungen hinterlegt hast."
-        return f"Ich habe deine Frage zu '{query}' verstanden, kann aber ohne aktive Gemini/OpenAI Verbindung keine tiefergehende Analyse durchführen. Nutze den Packaging Wizard für automatische Erkennungen!"
+        return f"Ich habe deine Frage zu '{query}' verstanden, kann aber ohne aktive Verbindung zu Gemini oder OpenAI keine tiefergehende Analyse durchführen. Nutze den Packaging Wizard für automatische Erkennungen!"
 """
     create_addon("AI Assistant", "ai", {"service.py": ai_service_code})
 

src/switchcraft/assets/lang/de.json

@@ -35,6 +35,8 @@
     "analyzing": "Analysiere",
     "analysis_complete": "Analyse abgeschlossen",
     "ai_helper": "KI Helfer",
+    "ai_stub_greeting": "Hallo! Ich bin dein lokaler KI-Assistent. Ich kann dir bei Paketierungsfragen helfen, auch wenn du keinen API-Key konfiguriert hast.",
+    "ai_stub_welcome": "Hallo! Ich bin der lokale SwitchCraft KI-Helfer. Ich laufe derzeit im eingeschränkten Modus, da das KI-Addon nicht installiert ist, aber ich kann dir trotzdem einige grundlegende Tipps geben!",
     "error": "Fehler",
     "file_not_found": "Datei nicht gefunden.",
     "unknown_installer": "Installer-Typ konnte nicht erkannt werden.",

src/switchcraft/assets/lang/en.json

@@ -35,6 +35,8 @@
     "analyzing": "Analyzing",
     "analysis_complete": "Analysis Complete",
     "ai_helper": "AI Helper",
+    "ai_stub_greeting": "Hello! I am your local AI assistant. I can help you with packaging questions even when you don't have an API key configured.",
+    "ai_stub_welcome": "Hello! I am the local SwitchCraft AI helper. I'm currently running in limited mode because the AI addon is not installed, but I can still give you some basic tips!",
     "error": "Error",
     "file_not_found": "File not found.",
     "unknown_installer": "Could not identify installer type.",

src/switchcraft/gui_modern/utils/view_utils.py

@@ -7,19 +7,67 @@ class ViewMixin:
     """Mixin for common view functionality."""
 
     def _show_snack(self, msg, color="GREEN"):
-        """Show a snackbar message on the page."""
+        """Show a snackbar message on the page using modern API."""
         try:
-            # Check if self has app_page or page
             page = getattr(self, "app_page", getattr(self, "page", None))
             if not page:
-                logger.debug("Cannot show snackbar: No page reference found")
                 return
 
             page.snack_bar = ft.SnackBar(ft.Text(msg), bgcolor=color)
-            if hasattr(page, "open"):
-                page.open(page.snack_bar)
-            else:
-                page.snack_bar.open = True
-                page.update()
+            # Use newer API for showing snackbar
+            page.open(page.snack_bar)
         except Exception as e:
             logger.debug(f"Failed to show snackbar: {e}")
+
+    def _open_path(self, path):
+        """Cross-platform path opener (Folder or File)."""
+        import os
+        import platform
+        import subprocess
+        import webbrowser
+
+        try:
+            if not os.path.exists(path):
+                self._show_snack(f"Path does not exist: {path}", "RED")
+                return
+
+            system = platform.system()
+            if system == "Windows":
+                os.startfile(path)
+            elif system == "Darwin": # macOS
+                subprocess.run(["open", path], check=True)
+            else: # Linux
+                subprocess.run(["xdg-open", path], check=True)
+        except Exception as e:
+            logger.debug(f"Default opener failed, falling back to webbrowser: {e}")
+            try:
+                webbrowser.open(f"file:///{path}")
+            except Exception as e2:
+                self._show_snack(f"Failed to open path: {path}", "RED")
+                logger.error(f"Failed to open path: {e2}")
+    def _close_dialog(self, dialog=None):
+        """Close a dialog on the page."""
+        try:
+            page = getattr(self, "app_page", getattr(self, "page", None))
+            if not page:
+                return
+
+            if dialog:
+                dialog.open = False
+                dialog.update()
+
+            # Fallback for older Flet or to ensure it's removed from overlay
+            if hasattr(page, "close"):
+                try:
+                    page.close(dialog)
+                except:
+                    pass
+            elif hasattr(page, "close_dialog"):
+                try:
+                    page.close_dialog()
+                except:
+                    pass
+
+            page.update()
+        except Exception as e:
+            logger.debug(f"Failed to close dialog: {e}")