Skip to content

Conversation

@RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Nov 27, 2025

Description

Followup to #8867


Summary by cubic

Unifies working directory resolution for terminal commands and adds a final fallback to the system temp directory to prevent errors when no workspace is available. Applies the same cwd logic to both streaming and non-streaming execution paths.

  • Bug Fixes
    • Resolve cwd from file:/ workspace URIs when present.
    • Fallback order: HOME/USERPROFILE → process.cwd → os.tmpdir.

Written for commit 814cc20. Summary will update automatically on new commits.

@RomneyDa RomneyDa requested a review from a team as a code owner November 27, 2025 10:29
@RomneyDa RomneyDa requested review from tingwai and removed request for a team November 27, 2025 10:29
@continue
Copy link
Contributor

continue bot commented Nov 27, 2025

Keep this PR in a mergeable state →

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 27, 2025
@github-actions
Copy link

github-actions bot commented Nov 27, 2025

✅ Review Complete

Code Review Summary

⚠️ Continue configuration error. Please verify that the assistant exists in Continue Hub.


@continue
Copy link
Contributor

continue bot commented Nov 27, 2025

Reviewed PR #8908 for documentation updates.

No documentation changes needed - this is an internal refactoring that:

  • Unifies working directory resolution logic
  • Improves fallback handling for edge cases
  • Doesn't change any user-facing behavior, APIs, or configuration options

The existing documentation about runTerminalCommand remains accurate.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 1, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Dec 1, 2025
@RomneyDa RomneyDa merged commit cd55d39 into main Dec 1, 2025
56 of 60 checks passed
@RomneyDa RomneyDa deleted the dallin/cwd-fix branch December 1, 2025 22:55
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Dec 1, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Dec 1, 2025
@sestinj
Copy link
Contributor

sestinj commented Dec 1, 2025

🎉 This PR is included in version 1.33.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants