Skip to content

fix(talon): default workspace to current directory - #4099

Merged
John Kennedy (jkennedyvz) merged 1 commit into
mainfrom
jk/talon/default-cwd-workspace
Jun 20, 2026
Merged

fix(talon): default workspace to current directory#4099
John Kennedy (jkennedyvz) merged 1 commit into
mainfrom
jk/talon/default-cwd-workspace

Conversation

@jkennedyvz

Copy link
Copy Markdown
Contributor

Talon currently defaults its local execution backend to /workspace, which only exists in the Docker example. Bare-metal runs can fail before the agent command executes because LocalShellBackend receives a missing working directory.

This changes the unset workspace behavior to use the process current working directory, while still preserving explicit DEEPAGENTS_TALON_WORKSPACE and DEEPAGENTS_TALON_OUTBOUND_MEDIA_DIR overrides. The Docker example continues to set DEEPAGENTS_TALON_WORKSPACE=/workspace, so container behavior is unchanged.

Tests run:

  • uv run --group test pytest tests/test_runtime.py tests/channels/test_whatsapp.py
  • make lint
  • make test

@github-actions github-actions Bot added fix A bug fix (PATCH) internal User is a member of the `langchain-ai` GitHub organization size: XS < 50 LOC talon labels Jun 20, 2026

@open-swe open-swe Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in WebView Open SWE trace

@jkennedyvz
John Kennedy (jkennedyvz) merged commit 5e337ae into main Jun 20, 2026
50 checks passed
@jkennedyvz
John Kennedy (jkennedyvz) deleted the jk/talon/default-cwd-workspace branch June 20, 2026 19:02
John Kennedy (jkennedyvz) pushed a commit that referenced this pull request Jun 30, 2026
> [!CAUTION]
> Merging this PR will automatically publish to **PyPI** and create a
**GitHub release**.

For the full release process, see
[`.github/RELEASING.md`](https://github.com/langchain-ai/deepagents/blob/main/.github/RELEASING.md).

---

_Everything below this line will be the GitHub release body._

---


##
[0.0.2](deepagents-talon==0.0.1...deepagents-talon==0.0.2)
(2026-06-30)


### Features

* **talon:** `DEEPAGENTS_TALON_RECURSION_LIMIT` env var
([#4354](#4354))
([82d1eac](82d1eac))
* **talon:** add reaction approval routing
([#4345](#4345))
([3fe8c0c](3fe8c0c))
* **talon:** add Telegram channel adapter, CLI wiring, and offset
persistence
([#4097](#4097))
([7c87cec](7c87cec))
* **talon:** add tool approval env override
([#4349](#4349))
([d26481d](d26481d))
* **talon:** audit reaction approval attempts
([#4348](#4348))
([d7895c4](d7895c4))
* **talon:** ingest Telegram approval reactions
([#4346](#4346))
([437af0b](437af0b))


### Bug Fixes

* **talon:** default workspace to current directory
([#4099](#4099))
([5e337ae](5e337ae))

---

_Everything above this line will be the GitHub release body._

---

> [!NOTE]
> A **New Contributors** section is appended to the GitHub release notes
automatically at publish time (see [Release
Pipeline](https://github.com/langchain-ai/deepagents/blob/main/.github/RELEASING.md#release-pipeline),
step 2).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Marcelo5444 pushed a commit to Marcelo5444/deepagents that referenced this pull request Jul 30, 2026
Talon currently defaults its local execution backend to `/workspace`,
which only exists in the Docker example. Bare-metal runs can fail before
the agent command executes because `LocalShellBackend` receives a
missing working directory.

This changes the unset workspace behavior to use the process current
working directory, while still preserving explicit
`DEEPAGENTS_TALON_WORKSPACE` and `DEEPAGENTS_TALON_OUTBOUND_MEDIA_DIR`
overrides. The Docker example continues to set
`DEEPAGENTS_TALON_WORKSPACE=/workspace`, so container behavior is
unchanged.

Tests run:
- `uv run --group test pytest tests/test_runtime.py
tests/channels/test_whatsapp.py`
- `make lint`
- `make test`

Co-authored-by: Deep Agent <agent@deepagents.dev>
Marcelo5444 pushed a commit to Marcelo5444/deepagents that referenced this pull request Jul 30, 2026
> [!CAUTION]
> Merging this PR will automatically publish to **PyPI** and create a
**GitHub release**.

For the full release process, see
[`.github/RELEASING.md`](https://github.com/langchain-ai/deepagents/blob/main/.github/RELEASING.md).

---

_Everything below this line will be the GitHub release body._

---


##
[0.0.2](langchain-ai/deepagents@deepagents-talon==0.0.1...deepagents-talon==0.0.2)
(2026-06-30)


### Features

* **talon:** `DEEPAGENTS_TALON_RECURSION_LIMIT` env var
([langchain-ai#4354](langchain-ai#4354))
([82d1eac](langchain-ai@82d1eac))
* **talon:** add reaction approval routing
([langchain-ai#4345](langchain-ai#4345))
([3fe8c0c](langchain-ai@3fe8c0c))
* **talon:** add Telegram channel adapter, CLI wiring, and offset
persistence
([langchain-ai#4097](langchain-ai#4097))
([7c87cec](langchain-ai@7c87cec))
* **talon:** add tool approval env override
([langchain-ai#4349](langchain-ai#4349))
([d26481d](langchain-ai@d26481d))
* **talon:** audit reaction approval attempts
([langchain-ai#4348](langchain-ai#4348))
([d7895c4](langchain-ai@d7895c4))
* **talon:** ingest Telegram approval reactions
([langchain-ai#4346](langchain-ai#4346))
([437af0b](langchain-ai@437af0b))


### Bug Fixes

* **talon:** default workspace to current directory
([langchain-ai#4099](langchain-ai#4099))
([5e337ae](langchain-ai@5e337ae))

---

_Everything above this line will be the GitHub release body._

---

> [!NOTE]
> A **New Contributors** section is appended to the GitHub release notes
automatically at publish time (see [Release
Pipeline](https://github.com/langchain-ai/deepagents/blob/main/.github/RELEASING.md#release-pipeline),
step 2).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix A bug fix (PATCH) internal User is a member of the `langchain-ai` GitHub organization size: XS < 50 LOC talon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant