base red: acpmux bg_start passes result_line through bg_start_with_id (cmux-tui compiles again)

Closed
LA
2d ago
+3-1

Summary

Base red on feat-cmux-next: cmux-tui does not compile. crates/acpmux/src/hub/bg_tasks.rs:160: cannot find value result_line in this scope.

Cause: a semantic merge of two PRs. #33347 (06fa0edc371) added a result_line parameter to Hub::bg_start and put it into the task Spec. #33779 and #33982 had moved the body of bg_start into bg_start_with_id, which has no result_line. Each PR compiled alone. Together they do not.

Fix: bg_start_with_id takes result_line and bg_start passes it through. The workflow's machine task (workflow_activity.rs, the only other caller) passes false, so its behavior stays as before #33347.

Testing

Observed / Not run

  • Observed: the compile checks above.
  • Not run: the acpmux test suite (cargo test); the change only threads a flag through one function.

Changelog

none

Proof

No capture: build fix.


Autofix with [code]smith
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.


Summary by cubic

Fixes a compile error in cmux-tui caused by a semantic merge of two PRs: one added a result_line parameter to bg_start, and another moved the body into bg_start_with_id, which lacks that parameter.

bg_start_with_id now takes result_line and bg_start passes it through. The workflow's machine task passes false, so its behavior is unchanged.