Merge pull request #376 from microsoft/psl-codeql-fixcodmod

Roopan-Microsoft · web-flow · commit e47b91cedb10 · 2026-03-24T20:11:19.000+05:30
fix: Explicit returns mixed with implicit (fall through) returns
diff --git a/src/backend/sql_agents/helpers/comms_manager.py b/src/backend/sql_agents/helpers/comms_manager.py
@@ -79,6 +79,8 @@ async def select_agent(self, agents, history):
                         ),
                         None,
                     )
+            # No matching case found, so explicitly return None
+            return None
 
     # class for termination strategy
     class ApprovalTerminationStrategy(TerminationStrategy):

Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,8 @@ async def select_agent(self, agents, history):`
`79`	`79`	`),`
`80`	`80`	`None,`
`81`	`81`	`)`
	`82`	`+ # No matching case found, so explicitly return None`
	`83`	`+ return None`
`82`	`84`
`83`	`85`	`# class for termination strategy`
`84`	`86`	`class ApprovalTerminationStrategy(TerminationStrategy):`