fix: add penny case to inject-subagent-context.sh #176

Merged

forgejo_admin merged 1 commit from 157-penny-inject-context into main

2026-03-27 21:37:06 +00:00

forgejo_admin commented

2026-03-27 21:34:56 +00:00

Contributor

Summary

Penny agents were getting zero context injection at spawn because inject-subagent-context.sh had no penny case branch. This adds the missing case with context scoped to Penny's currently deployed MCP tools.

Changes

hooks/inject-subagent-context.sh: added penny) case between the dottie and wildcard cases. Context string includes profile pointer (get_note(slug="agent-penny")), available tools (pal-e-docs read-only, Notion full read/write), and constraints (no writes to pal-e-docs, no repo/Forgejo access, no unsanctioned external actions).

Test Plan

echo '{"agent_type":"penny"}' | bash hooks/inject-subagent-context.sh produces valid JSON with correct additionalContext
All existing cases (qa, dev, dottie, unknown) produce unchanged output -- no regression
Spawn a penny agent and confirm context appears in SubagentStart hook output

Review Checklist

No secrets committed
No unnecessary file changes
Commit messages are descriptive
Context string references only currently deployed tools (pal-e-docs, Notion) -- not aspirational ones (Gmail, gcal, LinkedIn)

Closes #157
PR #135 (where this gap was discovered)
Issue #132 (parent penny spawn fix)
Issue #133 (Penny MCP spike -- depends on this)

## Summary Penny agents were getting zero context injection at spawn because `inject-subagent-context.sh` had no `penny` case branch. This adds the missing case with context scoped to Penny's currently deployed MCP tools. ## Changes - `hooks/inject-subagent-context.sh`: added `penny)` case between the `dottie` and wildcard cases. Context string includes profile pointer (`get_note(slug="agent-penny")`), available tools (pal-e-docs read-only, Notion full read/write), and constraints (no writes to pal-e-docs, no repo/Forgejo access, no unsanctioned external actions). ## Test Plan - [x] `echo '{"agent_type":"penny"}' | bash hooks/inject-subagent-context.sh` produces valid JSON with correct additionalContext - [x] All existing cases (qa, dev, dottie, unknown) produce unchanged output -- no regression - [ ] Spawn a penny agent and confirm context appears in SubagentStart hook output ## Review Checklist - [x] No secrets committed - [x] No unnecessary file changes - [x] Commit messages are descriptive - [x] Context string references only currently deployed tools (pal-e-docs, Notion) -- not aspirational ones (Gmail, gcal, LinkedIn) ## Related Notes - Closes #157 - PR #135 (where this gap was discovered) - Issue #132 (parent penny spawn fix) - Issue #133 (Penny MCP spike -- depends on this)

forgejo_admin added 1 commit

2026-03-27 21:34:56 +00:00

fix: add penny case to inject-subagent-context.sh 68a0cd4003

Penny agents were getting zero context injection at spawn because the
case statement had no penny branch. Add penny case with context that
references her pal-e-docs profile, enumerates her currently available
MCP tools (pal-e-docs read-only, Notion read/write), and enforces her
constraints (no repo access, no Forgejo, no unsanctioned external
actions).

Closes #157

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>