chore: drop integration field from cli-run register payload

The server-side handler now rejects unknown fields and the integration
column has been removed from the schema (it was plumbed end-to-end but
never displayed, filtered, or grouped on). Stop sending it.

Removes the integration parameter from register_cli_run and
setup_streaming, drops the corresponding wiring in socketcli.py, and
prunes the now-pointless test_register_cli_run_omits_integration_when_falsy
case.

Author: barslev

socketsecurity/core/cli_run.py

@@ -23,16 +23,12 @@
 def register_cli_run(
     client: CliClient,
     client_version: str,
-    integration: Optional[str] = None,
 ) -> Optional[str]:
-    payload = {"client_version": client_version}
-    if integration:
-        payload["integration"] = integration
     try:
         resp = client.request(
             path="python-cli-runs",
             method="POST",
-            payload=json.dumps(payload),
+            payload=json.dumps({"client_version": client_version}),
         )
     except APIFailure as e:
         log.debug(f"cli-run register failed (streaming disabled): {e}")

socketsecurity/core/streaming.py

@@ -30,13 +30,11 @@ def setup_streaming(
     cli_logger: logging.Logger,
     sdk_logger: logging.Logger,
     client_version: str,
-    integration: Optional[str],
     enable_debug: bool,
 ) -> Optional[Callable[[], None]]:
     run_id = register_cli_run(
         client,
         client_version=client_version,
-        integration=integration,
     )
     if not run_id:
         cli_logger.debug("server log streaming disabled (register failed)")

socketsecurity/socketcli.py

@@ -104,7 +104,6 @@ def main_code():
             cli_logger=log,
             sdk_logger=socket_logger,
             client_version=config.version,
-            integration=config.integration_type,
             enable_debug=config.enable_debug,
         )
         if teardown:

tests/unit/test_cli_run.py

@@ -16,14 +16,14 @@ def test_register_cli_run_returns_run_id():
     client = Mock(spec=CliClient)
     client.request.return_value = _resp({"run_id": "srv-issued-123"})
 
-    run_id = register_cli_run(client, client_version="1.2.3", integration="github")
+    run_id = register_cli_run(client, client_version="1.2.3")
 
     assert run_id == "srv-issued-123"
     args, kwargs = client.request.call_args
     assert kwargs["path"] == "python-cli-runs"
     assert kwargs["method"] == "POST"
     body = json.loads(kwargs["payload"])
-    assert body == {"client_version": "1.2.3", "integration": "github"}
+    assert body == {"client_version": "1.2.3"}
 
 
 def test_register_cli_run_returns_none_on_api_failure():
@@ -49,16 +49,6 @@ def test_register_cli_run_returns_none_on_bad_json():
     assert register_cli_run(client, client_version="1.0.0") is None
 
 
-def test_register_cli_run_omits_integration_when_falsy():
-    client = Mock(spec=CliClient)
-    client.request.return_value = _resp({"run_id": "x"})
-
-    register_cli_run(client, client_version="1.0.0", integration=None)
-
-    body = json.loads(client.request.call_args.kwargs["payload"])
-    assert body == {"client_version": "1.0.0"}
-
-
 def test_finalize_cli_run_posts_status():
     client = Mock(spec=CliClient)
     finalize_cli_run(client, "run-x", status="failure")

tests/unit/test_streaming.py

@@ -21,7 +21,6 @@ def test_setup_streaming_returns_none_when_register_fails():
             cli_logger=logging.getLogger("t-fail"),
             sdk_logger=logging.getLogger("t-fail-sdk"),
             client_version="1.0",
-            integration=None,
             enable_debug=False,
         )
     assert teardown is None
@@ -45,7 +44,6 @@ def fake_finalize(client, run_id, status="success"):
             cli_logger=cli_logger,
             sdk_logger=sdk_logger,
             client_version="1.0",
-            integration=None,
             enable_debug=False,
         )
         assert teardown is not None
@@ -74,7 +72,6 @@ def fake_finalize(client, run_id, status="success"):
             cli_logger=cli_logger,
             sdk_logger=sdk_logger,
             client_version="1.0",
-            integration=None,
             enable_debug=False,
         )
         teardown()
@@ -100,7 +97,6 @@ def test_setup_streaming_restores_logger_state_on_teardown():
             cli_logger=cli_logger,
             sdk_logger=sdk_logger,
             client_version="1.0",
-            integration=None,
             enable_debug=False,
         )
         # During streaming: levels and propagate are forced