automatic issues

Author: stuchain

.github/workflows/release.yml

@@ -216,6 +216,8 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           GITHUB_REPOSITORY: ${{ github.repository }}
 
+      # Test tags (e.g. v1.0.3-test1.1) create a draft release: hidden from public
+      # Releases list; only visible to maintainers at GitHub Releases when logged in.
       - name: Create GitHub Release
         uses: softprops/action-gh-release@v1
         with:
@@ -227,8 +229,8 @@ jobs:
             THIRD_PARTY_LICENSES.txt
             SHA256SUMS
           body_path: RELEASE_NOTES.md
-          draft: false
-          prerelease: false
+          draft: ${{ contains(github.ref_name, '-test') }}
+          prerelease: ${{ contains(github.ref_name, '-test') }}
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
@@ -239,6 +241,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Fetch Existing Appcast Feeds
+        if: ${{ !contains(github.ref_name, '-test') }}
         run: |
           echo "=== Fetching Existing Appcast Feeds ==="
           
@@ -266,6 +269,7 @@ jobs:
           fi
       
       - name: Generate Appcast Feeds
+        if: ${{ !contains(github.ref_name, '-test') }}
         run: |
           echo "=== Generating Appcast Feeds ==="
           VERSION="${{ github.ref_name }}"
@@ -330,6 +334,7 @@ jobs:
           echo "Appcast generation complete"
       
       - name: Validate Appcast Feeds
+        if: ${{ !contains(github.ref_name, '-test') }}
         run: |
           echo "=== Validating Appcast Feeds ==="
           if [ -f "updates/macos/stable/appcast.xml" ] && [ -f "updates/windows/stable/appcast.xml" ]; then
@@ -342,6 +347,7 @@ jobs:
           fi
       
       - name: Publish Appcast Feeds to GitHub Pages
+        if: ${{ !contains(github.ref_name, '-test') }}
         run: |
           echo "=== Publishing Appcast Feeds ==="
           

.gitignore

@@ -333,3 +333,4 @@ build/version_info.txt
 *.sqlite
 bp_cache.sqlite*
 
+scripts/delete_all_releases.py

requirements-build.txt

@@ -48,6 +48,9 @@ selenium==4.39.0
 # Excel export support (for GUI export functionality)
 openpyxl==3.1.5
 
+# Error reporting (optional: set SENTRY_DSN to enable)
+sentry-sdk>=2.0.0,<3
+
 # Build tool (PyInstaller for packaging)
 pyinstaller==6.17.0
 

requirements.txt

@@ -40,6 +40,9 @@ selenium==4.39.0
 # Excel export support (for GUI export functionality)
 openpyxl==3.1.5
 
+# Error reporting (optional: set SENTRY_DSN to enable)
+sentry-sdk>=2.0.0,<3
+
 # Testing dependencies
 pytest==9.0.2
 pytest-qt==4.5.0

src/cuepoint/ui/dialogs/first_run_error_reporting_dialog.py

@@ -0,0 +1,64 @@
+#!/usr/bin/env python3
+# -*- coding: utf-8 -*-
+
+"""
+First-run error reporting prompt.
+
+Shown once when the user runs the app for the first time (or after a fresh
+install). Asks whether to allow sending error reports to help fix bugs.
+"""
+
+from typing import Optional
+
+from PySide6.QtCore import QSettings, QTimer
+from PySide6.QtWidgets import QApplication, QMessageBox, QWidget
+
+
+_FIRST_RUN_PROMPT_SHOWN_KEY = "error_reporting/first_run_prompt_shown"
+
+
+def show_if_first_run(parent: Optional[QWidget] = None) -> None:
+    """Show the error-reporting consent prompt only on first run.
+
+    If the user has never been asked, shows a dialog. Saves their choice
+    and sets a flag so we do not ask again.
+
+    Args:
+        parent: Optional parent widget for the dialog.
+    """
+    settings = QSettings()
+    if settings.value(_FIRST_RUN_PROMPT_SHOWN_KEY, False, type=bool):
+        return
+
+    msg = QMessageBox(parent)
+    msg.setWindowTitle("Help improve CuePoint")
+    msg.setIcon(QMessageBox.Question)
+    msg.setText("Send error reports to help fix bugs?")
+    msg.setInformativeText(
+        "When crashes or errors happen, we can send a report to the developers. "
+        "No personal data or file contents are included. "
+        "You can change this later in Settings → Privacy."
+    )
+    msg.setStandardButtons(QMessageBox.Yes | QMessageBox.No)
+    msg.setDefaultButton(QMessageBox.Yes)
+
+    def center_on_screen() -> None:
+        screen = QApplication.primaryScreen().availableGeometry()
+        geom = msg.frameGeometry()
+        geom.moveCenter(screen.center())
+        msg.move(geom.topLeft())
+
+    QTimer.singleShot(0, center_on_screen)
+    choice = msg.exec()
+
+    try:
+        from cuepoint.utils.error_reporting_prefs import ErrorReportingPrefs
+
+        prefs = ErrorReportingPrefs()
+        allowed = choice == QMessageBox.Yes
+        prefs.set_enabled(allowed)
+        prefs.set_consented(allowed)
+    except Exception:
+        pass
+
+    settings.setValue(_FIRST_RUN_PROMPT_SHOWN_KEY, True)

src/cuepoint/ui/main_window.py

@@ -935,7 +935,7 @@ def show_tool_selection_page(self) -> None:
         if self.tool_selection_page:
             self.setCentralWidget(self.tool_selection_page)
             self.current_page = "tool_selection"
-            self.menuBar().hide()
+            self.menuBar().show()
 
     def show_main_interface(self) -> None:
         """Show the main interface (existing tabs)"""
@@ -1233,6 +1233,11 @@ def create_menu_bar(self) -> None:
         report_issue_action.triggered.connect(self.on_report_issue)
         diagnostics_menu.addAction(report_issue_action)
 
+        test_sentry_action = QAction("Send &test event to Sentry", self)
+        test_sentry_action.setToolTip("Send a test error to Sentry to verify reporting is working")
+        test_sentry_action.triggered.connect(self._on_test_sentry_report)
+        diagnostics_menu.addAction(test_sentry_action)
+
         help_menu.addSeparator()
 
         # About (includes changelog)
@@ -1882,6 +1887,36 @@ def on_report_issue(self) -> None:
                 self, "Report Issue", f"Could not open issue reporter:\n{e}"
             )
 
+    def _on_test_sentry_report(self) -> None:
+        """Send a test event to Sentry so the user can verify reporting works."""
+        try:
+            from cuepoint.utils.error_reporting_prefs import ErrorReportingPrefs
+
+            prefs = ErrorReportingPrefs()
+            if not (prefs.is_enabled() and prefs.has_user_consented()):
+                QMessageBox.warning(
+                    self,
+                    "Sentry test",
+                    "Error reporting is off.\n\n"
+                    "Enable it in Settings → Privacy → \"Send error reports to help fix bugs\", "
+                    "then try again.",
+                    QMessageBox.Ok,
+                )
+                return
+        except Exception:
+            pass
+        import logging
+
+        logger = logging.getLogger(__name__)
+        logger.error("CuePoint Sentry test event (user-triggered)")
+        QMessageBox.information(
+            self,
+            "Sentry test",
+            "A test event was sent to Sentry.\n\n"
+            "Check your Sentry project (Issues) in a few seconds.",
+            QMessageBox.Ok,
+        )
+
     def _open_folder(self, folder_path) -> None:
         """Open a folder in the OS file manager."""
         try:

src/cuepoint/ui/widgets/privacy_settings.py

@@ -62,6 +62,14 @@ def _init_ui(self) -> None:
         )
         group_layout.addWidget(self.chk_telemetry)
 
+        # Error reporting (Sentry) – consent to send crash/error reports
+        self.chk_error_reporting = QCheckBox("Send error reports to help fix bugs")
+        self.chk_error_reporting.setToolTip(
+            "When enabled, crash reports and error details are sent to the developers "
+            "so we can fix issues. No personal data or file contents are included."
+        )
+        group_layout.addWidget(self.chk_error_reporting)
+
         self.chk_clear_cache = QCheckBox("Clear cache on exit")
         self.chk_clear_logs = QCheckBox("Clear logs on exit")
 
@@ -80,6 +88,7 @@ def _init_ui(self) -> None:
         layout.addWidget(group)
 
         self.chk_telemetry.toggled.connect(self._on_telemetry_changed)
+        self.chk_error_reporting.toggled.connect(self._on_error_reporting_changed)
         self.chk_clear_cache.toggled.connect(self._on_changed)
         self.chk_clear_logs.toggled.connect(self._on_changed)
         self.btn_manage.clicked.connect(self.open_privacy_dialog_requested.emit)
@@ -93,6 +102,15 @@ def _load(self) -> None:
                 "telemetry.enabled", False
             )
             self.chk_telemetry.setChecked(bool(enabled))
+        try:
+            from cuepoint.utils.error_reporting_prefs import ErrorReportingPrefs
+
+            er_prefs = ErrorReportingPrefs()
+            self.chk_error_reporting.setChecked(
+                er_prefs.is_enabled() and er_prefs.has_user_consented()
+            )
+        except Exception:
+            self.chk_error_reporting.setChecked(False)
 
     def _on_telemetry_changed(self, checked: bool) -> None:
         if self._config_controller:
@@ -106,6 +124,16 @@ def _on_telemetry_changed(self, checked: bool) -> None:
                 except Exception:
                     pass
 
+    def _on_error_reporting_changed(self, checked: bool) -> None:
+        try:
+            from cuepoint.utils.error_reporting_prefs import ErrorReportingPrefs
+
+            er_prefs = ErrorReportingPrefs()
+            er_prefs.set_enabled(bool(checked))
+            er_prefs.set_consented(bool(checked))
+        except Exception:
+            pass
+
     def _on_changed(self) -> None:
         self._privacy.set_clear_cache_on_exit(self.chk_clear_cache.isChecked())
         self._privacy.set_clear_logs_on_exit(self.chk_clear_logs.isChecked())
@@ -114,6 +142,7 @@ def get_snapshot(self) -> tuple:
         """Return a comparable snapshot for change detection."""
         return (
             self.chk_telemetry.isChecked(),
+            self.chk_error_reporting.isChecked(),
             self.chk_clear_cache.isChecked(),
             self.chk_clear_logs.isChecked(),
         )

src/cuepoint/update/update_checker.py

@@ -515,12 +515,12 @@ def _find_latest_update(self, items: List[Dict]) -> Optional[Dict]:
                     continue
 
                 # Both on same track (both test or both non-test) - allow if newer
-                # Prefer items with checksum (SHA256); allow items with only EdDSA signature
-                # so we don't skip all appcast items (many feeds use sparkle:edSignature only).
+                # Design 4.27, 4.109: require checksum; skip item if missing (no update offered).
                 if not item.get("checksum"):
                     logger.debug(
-                        f"Item {version} has no SHA256 checksum (may have EdDSA only); still offering as update"
+                        f"Skipping item {version}: no checksum (SHA256 required)"
                     )
+                    continue
                 logger.info(
                     f"Found newer version: {version} (current: {self.current_version})"
                 )

src/cuepoint/utils/crash_handler.py

@@ -99,6 +99,14 @@ def _handle_exception(self, exc_type, exc_value, exc_traceback):
                 exception, traceback_str, crash_log, crash_report_path
             )
 
+        # Report to Sentry (if DSN set and user consented)
+        try:
+            import sentry_sdk
+
+            sentry_sdk.capture_exception(exception)
+        except Exception as e:
+            logger.debug("Sentry capture failed: %s", e)
+
         # Report to GitHub Issues (Step 11.2)
         try:
             from cuepoint.utils.error_reporter import report_error
@@ -207,6 +215,14 @@ def thread_exception_handler(args):
             crash_report_path = crash_log.with_suffix(".json")
             CrashReport.save_report(crash_report, crash_report_path)
 
+            # Report to Sentry (if DSN set and user consented)
+            try:
+                import sentry_sdk
+
+                sentry_sdk.capture_exception(exception)
+            except Exception as e:
+                logger.debug("Sentry capture failed: %s", e)
+
             # Report to GitHub Issues (Step 11.2)
             try:
                 from cuepoint.utils.error_reporter import report_error