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 <[email protected]> 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 => ({
