Optimize PathBuf usage in handle_build to avoid unnecessary clone

[coderabbitai]

Description

In src/runner.rs, the handle_build function performs an unnecessary PathBuf clone when handling the emitted path (line 118):

(path.clone(), None)

Optimization Opportunity

The tuple type (PathBuf, Option<NamedTempFile>) could be replaced with (Cow<Path>, Option<NamedTempFile>) to avoid the allocation when borrowing the emitted path. The Some(path) branch would borrow the path without cloning, while the temp-file branch would own the path.

Context

This optimization was identified during the review of PR #82 and discussed in this comment thread.

Issue: #88

Requested by

@leynos

[leynos]

@coderabbitai Is this a duplicate of #84?

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Fetch issue #84 details
gh issue view 84 --json title,body,state,url

Length of output: 1495

---

@leynos Yes, this appears to be a duplicate of #84. Both issues address the same optimization in src/runner.rs::handle_build:

• Same problem: Avoiding unnecessary PathBuf clone when handling the emitted path
• Same solution: Using Cow<Path> to express mixed ownership (borrowed for emitted path, owned for temp file)
• Same origin: Both stemmed from the review of PR Refactor CLI build handling into helper functions #82

Issue #84 provides more implementation details about the acceptance criteria and specific steps, while #88 focuses on the tuple type change. They're describing the same optimization opportunity.

Would you like to close one of them as a duplicate?

[leynos]

Duplicate of #84