acdbot: enable autopilot for one-offs

Author: wolovim

.github/ACDbot/call_series_config.yml

@@ -25,6 +25,14 @@ default_autopilot_settings:
   display_zoom_link_in_invite: true
   external_meeting_link: false
 
+# Autopilot settings for one-off calls
+one_off_autopilot_settings:
+  duration: 60
+  occurrence_rate: "custom"
+  need_youtube_streams: false
+  display_zoom_link_in_invite: true
+  external_meeting_link: false
+
 call_series:
   # Core Protocol Calls
   acdc:

.github/ACDbot/modules/call_series_config.py

@@ -159,3 +159,14 @@ def get_default_autopilot_settings() -> dict:
     """
     config = _load_config()
     return config.get("default_autopilot_settings", {})
+
+
+def get_one_off_autopilot_settings() -> dict:
+    """
+    Get the autopilot settings for one-off calls.
+
+    Returns:
+        Dict with one-off autopilot settings
+    """
+    config = _load_config()
+    return config.get("one_off_autopilot_settings", {})

.github/ACDbot/scripts/handle_protocol_call.py

@@ -21,7 +21,7 @@
 from modules.mapping_manager import MappingManager
 from modules.datetime_utils import generate_savvytime_link, format_datetime_for_discourse, format_datetime_for_stream_display
 from modules.logging_config import get_logger, log_success, log_resource_status, log_api_call, should_log_debug
-from modules.call_series_config import get_autopilot_defaults, has_autopilot_support, get_default_autopilot_settings
+from modules.call_series_config import get_autopilot_defaults, has_autopilot_support, get_default_autopilot_settings, get_one_off_autopilot_settings
 
 
 class ProtocolCallHandler:
@@ -112,20 +112,11 @@ def _apply_autopilot_defaults(self, form_data: Dict, issue) -> Dict:
 
         # Check if this is a one-off call
         if call_series and call_series.startswith("one-off"):
-            self.logger.info("Autopilot mode enabled but call is one-off - using user-provided values")
-            # Post informational comment
-            try:
-                comment_text = """ℹ️ **Autopilot Note**
-
-Autopilot mode was enabled, but this is a one-time call. One-time calls don't have predefined defaults, so your provided configuration values have been used instead."""
-
-                issue.create_comment(comment_text)
-            except Exception as e:
-                self.logger.warning(f"Failed to post autopilot note: {e}")
-            return form_data
-
-        # Get autopilot defaults for this series, or use system defaults
-        defaults = get_autopilot_defaults(call_series)
+            self.logger.info("Autopilot mode enabled for one-off call - applying one-off defaults")
+            defaults = get_one_off_autopilot_settings()
+        else:
+            # Get autopilot defaults for this series, or use system defaults
+            defaults = get_autopilot_defaults(call_series)
 
         if not defaults:
             self.logger.info(f"No specific autopilot config for '{call_series}', using defaults")

.github/ACDbot/tests/unit/test_protocol_call_handler.py

@@ -557,25 +557,23 @@ def test_apply_autopilot_defaults_disabled(self):
         self.assertFalse(result["need_youtube_streams"])
 
     def test_apply_autopilot_defaults_one_off_call(self):
-        """Test that autopilot posts notice for one-off calls."""
+        """Test that autopilot applies one-off defaults for one-off calls."""
         mock_issue = unittest.mock.MagicMock()
 
         form_data = {
             "call_series": "one-off-123",
             "autopilot_mode": True,
-            "duration": 60,
+            "duration": 90,
         }
 
         result = self.handler._apply_autopilot_defaults(form_data, mock_issue)
 
-        # Values should remain unchanged
+        # One-off defaults should be applied
         self.assertEqual(result["duration"], 60)
-
-        # Should post informational comment
-        mock_issue.create_comment.assert_called_once()
-        comment_text = mock_issue.create_comment.call_args[0][0]
-        self.assertIn("Autopilot Note", comment_text)
-        self.assertIn("one-time call", comment_text)
+        self.assertEqual(result["occurrence_rate"], "custom")
+        self.assertFalse(result["need_youtube_streams"])
+        self.assertTrue(result["display_zoom_link_in_invite"])
+        self.assertFalse(result["skip_zoom_creation"])
 
     def test_apply_autopilot_defaults_no_defaults_configured(self):
         """Test autopilot with series that has no defaults uses system defaults."""