Test print out API key

Author: nodwells

.github/workflows/summarise_upcoming_events.yml

@@ -37,7 +37,7 @@ jobs:
       - name: Run summary script
         run: |
           cd tools/llm_meetup_summary
-          python summarise_events_with_llms.py --channel events
+          python llm_event_summary.py --channel events
         env:
           OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
           SLACK_BOT_WEBHOOK: ${{ secrets.SLACK_BOT_WEBHOOK }}

…up_summary/summarise_events_with_llms.py …/llm_meetup_summary/llm_event_summary.pytools/llm_meetup_summary/summarise_events_with_llms.py renamed to tools/llm_meetup_summary/llm_event_summary.py tools/llm_meetup_summary/summarise_events_with_llms.py renamed to tools/llm_meetup_summary/llm_event_summary.py

@@ -11,7 +11,7 @@
 dotenv.load_dotenv()
 
 OPENAI_API_KEY = os.getenv('OPENAI_API_KEY')
-if OPENAI_API_KEY is None:
+if not OPENAI_API_KEY:
     raise KeyError("OPENAI_API_KEY is not set in environment variables")
 
 # Get current folder
@@ -124,6 +124,10 @@ def _get_llm_summary(formatted_events):
     with open(os.path.join(examples_dir, "current_prompt.md"), 'w') as txt:
         txt.write(prompt) # save for reference
 
+    OPENAI_API_KEY = os.getenv('OPENAI_API_KEY')
+    if not OPENAI_API_KEY:
+        raise KeyError("OPENAI_API_KEY is not set in environment variables (summary func)")
+    
     print("First 10 characters of key", OPENAI_API_KEY[:10] if OPENAI_API_KEY else "None")
     openai.api_key = OPENAI_API_KEY
 
@@ -153,7 +157,7 @@ def _post_to_slack(message, slack_webhook_url):
     response.raise_for_status()
     logger.info("Message posted to Slack successfully")
 
-def summarise_events_with_llm(events_file=EVENTS_FILE):
+def load_events_and_summarise(events_file=EVENTS_FILE):
     try:
         events = _load_events(events_file)
         future_events = _filter_future_events(events)
@@ -202,7 +206,7 @@ def summarise_events_with_llm(events_file=EVENTS_FILE):
         raise KeyError("SLACK_BOT_TEST_WEBHOOK or SLACK_BOT_WEBHOOK must be set in environment variables")
 
     try:
-        summary = summarise_events_with_llm(args.events_file)
+        summary = load_events_and_summarise(args.events_file)
         _post_to_slack(summary, slack_webhook_url=SLACK_WEBHOOK)
     except Exception as e:
         logger.error(f"Failed to summarize and post events: {e}", exc_info=True)

tools/llm_meetup_summary/tests/tests.py

@@ -1,6 +1,6 @@
 import pytest
 
-from llm_meetup_summary.summarise_events_with_llms import summarize_meetup_events_with_llms
+from tools.llm_meetup_summary.llm_event_summary import summarize_meetup_events_with_llms
 
 import pytest
 from unittest.mock import patch, mock_open, MagicMock
@@ -11,7 +11,7 @@
 # Add parent directory to path for absolute imports
 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 
-from llm_meetup_summary.summarise_events_with_llms import (
+from tools.llm_meetup_summary.llm_event_summary import (
     _get_llm_summary,
     _format_for_slack,
     _validate_event,