Skip to content

Conversation

@lifeizhou-ap
Copy link
Collaborator

Summary

This PR is addressing the issue #5288

Root cause:
Recently the subagent code change assumes that the parent recipe/agent always runs in a session mode. If the parent recipe/agent is triggered with --no-session mode, it returns the error.

Change
Allow subrecipe/subagent to be run in the parent --no-session mode, and the subrecipe/subagent also runs in no-session mode.

Note
There could be a refactoring in a separate PR to centralise the agent and session creation as there are multiple places to have the similar code.

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

Copy link
Collaborator

@amed-xyz amed-xyz left a comment

Choose a reason for hiding this comment

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

LGTM

@amed-xyz amed-xyz mentioned this pull request Oct 28, 2025
9 tasks
@amed-xyz amed-xyz merged commit 6b6b71f into main Oct 28, 2025
14 checks passed
@amed-xyz amed-xyz deleted the lifei/fix-no-session-subagent branch October 28, 2025 21:59
michaelneale added a commit that referenced this pull request Oct 29, 2025
* main: (30 commits)
  feat: add goose powered ai culinary innovation studio prompt to library (#5423)
  removing golang/temporal building
  testing tetrate with sonnet (#5428)
  Add Recipes Test Script (#5420)
  Don't die on strange chars (#5415)
  fix: allow subagent to run in parent --no-session mode (#5384)
  docs: analyze tool (#5418)
  fix: gracefully close goosed listening port (#5321)
  move history txt to state dir (#5410)
  Dont exit silently when storing api key fails (#5260)
  Make reply use the API (#5389)
  Fix/icon ii (#5413)
  Enable runtime access to provider name (#5399)
  fix: ensure trailing newline in files created by `text_editor` tool (#5336)
  docs: September 2025 Community All-Stars (#5411)
  make supports_cache_control async to avoid block in place (#5362)
  Send all the logs we output (#5363)
  Recipe variables (#5365)
  Feat/add mermaid chart rendering (#5377)
  Set up Datadog metrics for prompt injection detection (#5385)
  ...
wpfleger96 added a commit that referenced this pull request Oct 29, 2025
* main:
  fix: corrected my name in data (#5414)
  feat: add goose powered ai culinary innovation studio prompt to library (#5423)
  removing golang/temporal building
  testing tetrate with sonnet (#5428)
  Add Recipes Test Script (#5420)
  Don't die on strange chars (#5415)
  fix: allow subagent to run in parent --no-session mode (#5384)
  docs: analyze tool (#5418)
  fix: gracefully close goosed listening port (#5321)
  move history txt to state dir (#5410)
BlairAllan pushed a commit to BlairAllan/goose that referenced this pull request Nov 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants