Updated chatterlang_serve to support api key as a configuration setting

Author: travis-bauer

CHANGELOG.md

@@ -23,6 +23,10 @@
   is displayed with a full traceback, and the prompt continues accepting input. This makes interactive
   pipelines more robust and user-friendly. The error-resilient behavior is enabled by default but can
   be disabled by setting `error_resilient=False`.
+- Updated **chatterlang_serve** to support API key configuration via the configuration system. When
+  `--api-key` is not specified on the command line, it will check for `API_KEY` in the configuration,
+  which automatically checks the `TALKPIPE_API_KEY` environment variable. This makes it easier to
+  deploy chatterlang_serve in Docker and CI/CD environments without exposing API keys in command history.
 
 ## 0.9.3 
 ### Improvements

CLAUDE.md

@@ -16,6 +16,8 @@ ChatterLang is an external DSL and is defined under talkpipe/chatterlang
 Other packages in the project other than util include implementations of pipe sources and segments
 Util contains various generally usable utility functions, expecially for basic data manipulation and configuration.
 talkpipe/README.md contains a longer overview
+After making a change, update CHANGELOG.md
+When making a change, write a unit test that fails without the change, then make the change and test that the test now passes.  Notify the user if it is not reasonable to write the unit test.
 
 ## Important Commands
 python -m build for creating a new release

src/talkpipe/app/chatterlang_serve.py

@@ -1788,19 +1788,24 @@ def go():
                         help='Property of the input json to display in the stream interface as user input.')
 
     args, unknown_args = parser.parse_known_args()
-    
+
     # Parse unknown arguments and add to configuration so they're accessible via $key syntax
     constants = parse_unknown_args(unknown_args)
-    
+
     # Add command-line constants to the configuration
     if constants:
         add_config_values(constants, override=True)
         print(f"Added command-line values to configuration: {list(constants.keys())}")
-    
+
     if args.load_module:
         for module_file in args.load_module:
             load_module_file(fname=module_file, fail_on_missing=False)
 
+    # Get API key from command line, or fall back to configuration (which checks environment variable)
+    api_key = args.api_key
+    if api_key is None:
+        api_key = get_config().get('API_KEY')
+
     script_content = None
     if args.script:
         script_content = load_script(args.script)
@@ -1823,7 +1828,7 @@ def go():
     receiver = ChatterlangServer(
         host=args.host,
         port=args.port,
-        api_key=args.api_key,
+        api_key=api_key,
         require_auth=args.require_auth,
         title=args.title,
         form_config=form_config,

tests/talkpipe/app/test_chatterlang_serve.py

@@ -939,6 +939,50 @@ def test_go_with_constants(self, mock_server_class):
                 override=True
             )
 
+    @patch('talkpipe.app.chatterlang_serve.ChatterlangServer')
+    @patch('talkpipe.app.chatterlang_serve.get_config')
+    @patch('sys.argv', ['script.py'])
+    def test_go_api_key_from_config(self, mock_get_config, mock_server_class):
+        """Test go function loads API key from configuration when not provided on command line."""
+        mock_server = Mock()
+        mock_server_class.return_value = mock_server
+
+        # Mock configuration with API_KEY
+        mock_config = {'API_KEY': 'config-api-key'}
+        mock_get_config.return_value = mock_config
+
+        with patch('talkpipe.app.chatterlang_serve.parse_unknown_args', return_value={}), \
+             patch('talkpipe.app.chatterlang_serve.add_config_values'), \
+             patch('talkpipe.app.chatterlang_serve.load_script', return_value=None):
+
+            go()
+
+            # Verify server was created with API key from config
+            call_args = mock_server_class.call_args[1]
+            assert call_args["api_key"] == "config-api-key"
+
+    @patch('talkpipe.app.chatterlang_serve.ChatterlangServer')
+    @patch('talkpipe.app.chatterlang_serve.get_config')
+    @patch('sys.argv', ['script.py', '--api-key', 'cmdline-api-key'])
+    def test_go_api_key_cmdline_overrides_config(self, mock_get_config, mock_server_class):
+        """Test go function uses command line API key over configuration."""
+        mock_server = Mock()
+        mock_server_class.return_value = mock_server
+
+        # Mock configuration with API_KEY
+        mock_config = {'API_KEY': 'config-api-key'}
+        mock_get_config.return_value = mock_config
+
+        with patch('talkpipe.app.chatterlang_serve.parse_unknown_args', return_value={}), \
+             patch('talkpipe.app.chatterlang_serve.add_config_values'), \
+             patch('talkpipe.app.chatterlang_serve.load_script', return_value=None):
+
+            go()
+
+            # Verify server was created with command line API key (takes precedence)
+            call_args = mock_server_class.call_args[1]
+            assert call_args["api_key"] == "cmdline-api-key"
+
 
 class TestAsyncFunctionality:
     """Test async functionality."""