Set push_run_results_to_xcom kwargs correctly for invocation mode subprocess and Watcher mode

[pankajastro]

This PR fixes incorrect handling of the push_run_results_to_xcom keyword argument in subprocess invocation, Watcher execution modes. The issue likely stems from a method signature mismatch between the build_and_run method in the base class and its override in the local

Summary of changes:

• Align method signatures between base and local classes
• Ensure push_run_results_to_xcom is passed correctly in all invocation modes

related: #1968
related: https://github.com/astronomer/oss-integrations-private/issues/243

[Copilot]

Pull Request Overview

This PR ensures that the push_run_results_to_xcom parameter is correctly passed through the operator execution chain for subprocess invocation mode and Watcher mode. The changes add explicit handling of this parameter by extracting it from kwargs and passing it to the run_command method.

Key changes:

• Modified build_and_run_cmd method signature to accept **kwargs and extract push_run_results_to_xcom
• Updated test expectations to include push_run_results_to_xcom: False in expected kwargs
• Ensured the parameter flows from execute method through to run_command

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
cosmos/operators/base.py	Added **kwargs parameter to build_and_run_cmd signature and forwarded to implementation
cosmos/operators/local.py	Extracted push_run_results_to_xcom from kwargs and explicitly passed it to run_command
tests/operators/test_local.py	Updated test expectations to include push_run_results_to_xcom: False for all operator test cases

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.77%. Comparing base (9953f96) to head (7a076aa).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2067   +/-   ##
=======================================
  Coverage   97.77%   97.77%           
=======================================
  Files          91       91           
  Lines        5830     5830           
=======================================
  Hits         5700     5700           
  Misses        130      130           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pankajkoti]

Do we first need to merge #2070 into this branch(because I see the merge request is into this PR's branch from that PR), rebase and then merge this PR?

[pankajastro]

Do we first need to merge #2070 into this branch(because I see the merge request is into this PR's branch from that PR), rebase and then merge this PR?

I'll merge this one first — #2070 will handle switching the branch to main automatically.