(texera) 01/01: refactor(agent-service): remove unused targetOperatorId from toLogicalPlan (#7430)

[gi...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

github-merge-queue[bot] pushed a commit to branch gh-readonly-queue/main/pr-7430-fb1a5c4255f03e195e4ff36821d29acb71348071
in repository https://gitbox.apache.org/repos/asf/texera.git

commit e03d97108ddd47befc78a87f467f781dd50a56e6
Author: Ghulam Mustafa <13...@users.noreply.github.com>
AuthorDate: Sun Aug 9 16:26:22 2026 +0500

    refactor(agent-service): remove unused targetOperatorId from toLogicalPlan (#7430)
    
    ### What changes were proposed in this PR?
    It Refactors `WorkflowState.toLogicalPlan()` by removing the unused
    `targetOperatorId` parameter.
    
    The parameter was never referenced in the method (it always built the
    whole-graph plan) and the only caller (`texera-agent.ts`) passed no
    arguments to it. The need to fetch a sub-graph of a target is already
    properly served by `getSubDAG(targetOperatorId)`.
    
    ### Any related issues, documentation, discussions?
    Closes #7170
    
    ### How was this PR tested?
    Run the TypeScript compiler checks and existing agent-service unit tests
    to verify no callers or types were broken.
    
    ### Was this PR authored or co-authored using generative AI tooling?
    Generated-by: Gemini 3.1 Pro (High)
---
 agent-service/src/agent/workflow-state.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/agent-service/src/agent/workflow-state.ts b/agent-service/src/agent/workflow-state.ts
index 04ad2b0e4e..d53692f33c 100644
--- a/agent-service/src/agent/workflow-state.ts
+++ b/agent-service/src/agent/workflow-state.ts
@@ -415,7 +415,7 @@ export class WorkflowState {
     this.settings = content.settings ? { ...content.settings } : { ...DEFAULT_WORKFLOW_SETTINGS };
   }
 
-  toLogicalPlan(targetOperatorId?: string): LogicalPlan {
+  toLogicalPlan(): LogicalPlan {
     const enabledOperators = this.getAllEnabledOperators();
 
     const operators: LogicalOperator[] = enabledOperators.map(op => ({