Skip to content

fix: load ForgeFlow in native child sessions - #112

Merged
BakerSean168 merged 1 commit into
mainfrom
fix/required-child-extension
Oct 3, 2026
Merged

BakerSean168 merged 1 commit into
mainfrom
fix/required-child-extension

Conversation

@BakerSean168

Copy link
Copy Markdown
Owner

Summary

  • register ForgeFlow itself through pi-subagents' required-child-extension registry for every Pi session
  • make forgeflow/* virtual models available in native foreground, detached, nested, and recovery children
  • dispose the required-child registration on session shutdown
  • avoid hard-coding ForgeFlow installation paths in Pi operator profiles
  • document why local foreground children need this explicit registration

Root cause

pi-subagents foreground children intentionally do not load the parent's ambient extensions. After #110, the parent could resolve forgeflow/reviewer, but a reviewer child failed with Model "forgeflow/reviewer:high" not found because the ForgeFlow extension that registers virtual models was absent from the child.

Verification

  • npm run check: 47/47 passed
  • npm pack --dry-run: passed
  • git diff --check: passed
  • lifecycle test proves the required-child registration owns the parent session and is released at shutdown

@BakerSean168
BakerSean168 merged commit 864669d into main Oct 3, 2026
1 check passed
@BakerSean168
BakerSean168 deleted the fix/required-child-extension branch October 3, 2026 02:45
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.

1 participant