-
Notifications
You must be signed in to change notification settings - Fork 0
Clarify Claude reply routing in paired mode #36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
8383ae1
37fbb0e
9727516
6f7d014
86b857b
af73268
bd2b1b4
2d6ee98
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -94,7 +94,7 @@ const bridgeGuidance = (agent: Agent): string => { | |||||||
| return [ | ||||||||
| "Paired mode:", | ||||||||
| `You are in a persistent Claude/Codex pair. Use the MCP tool "send_to_agent" when you want ${peer} to act, review, or answer.`, | ||||||||
| 'Do not ask the human to relay messages between agents. Use "bridge_status" if you need the current bridge state.', | ||||||||
| 'Do not ask the human to relay messages between agents or answer the human on the other agent\'s behalf. Use "bridge_status" if you need the current bridge state.', | ||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This line combines multiple instructions. For better clarity for the agent, consider splitting it into separate lines, each with a single instruction. 'Do not ask the human to relay messages between agents.',
'Do not answer the human on the other agent\'s behalf.',
'Use "bridge_status" if you need the current bridge state.', |
||||||||
| 'If "bridge_status" shows pending messages addressed to you, call "receive_messages" to read them.', | ||||||||
| ].join("\n"); | ||||||||
| }; | ||||||||
|
|
@@ -148,7 +148,9 @@ const forwardBridgePrompt = (source: Agent, message: string): string => | |||||||
| [ | ||||||||
| `Message from ${capitalize(source)} via the loop bridge:`, | ||||||||
| message.trim(), | ||||||||
| "Treat this as direct agent-to-agent coordination. Reply with send_to_agent only when you have something useful for the other agent to act on. Do not acknowledge receipt without new information.", | ||||||||
| "Treat this as direct agent-to-agent coordination. Do not reply to the human.", | ||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For better clarity and to ensure each instruction is distinct for the agent, consider splitting this line into two separate instructions. This aligns with the goal of breaking down complex instructions into simpler, more direct ones.
Suggested change
|
||||||||
| 'Reply to the other agent with "send_to_agent" only when you have something useful for them to act on.', | ||||||||
| "Do not acknowledge receipt without new information.", | ||||||||
| ].join("\n\n"); | ||||||||
|
|
||||||||
| const updateIds = (state: PairedState): void => { | ||||||||
|
|
||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line contains two distinct instructions. To improve clarity for the agent, it would be better to split them into separate lines.