nav/Dirigent
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: add turn check debug logs + one-time prompt injection #6

Merged
zhi merged 1 commits from fix/turn-check-debug-and-onetime-inject into feat/turn-based-speaking 2026-02-28 19:27:59 +00:00
Conversation 0 Commits 1 Files Changed 1 +52 -5
zhi commented 2026-02-28 19:27:54 +00:00
Collaborator
Copy Link

Changes

Problem 1: Moderator not working

  • Add debug logs for ctx.agentId, resolved accountId, turnOrder length
  • Fallback to ctx.accountId if resolveAccountId fails
  • Add resolveDiscordUserId debug logs for handoff troubleshooting

Problem 2: Prompt injected on every user message

  • Use prependContext (not systemPrompt which overrides base system prompt)
  • Add sessionInjected Set to track which sessions have already injected
  • Only inject once per session (one-time injection)
  • Skip subsequent injections with debug log

Discussion with Orion

  • Confirmed that systemPrompt hook directly overrides base system prompt (no append capability)
  • The one-time prependContext approach is a good compromise until core supports appending to system prompt
## Changes ### Problem 1: Moderator not working - Add debug logs for `ctx.agentId`, resolved accountId, turnOrder length - Fallback to `ctx.accountId` if `resolveAccountId` fails - Add `resolveDiscordUserId` debug logs for handoff troubleshooting ### Problem 2: Prompt injected on every user message - Use `prependContext` (not `systemPrompt` which overrides base system prompt) - Add `sessionInjected` Set to track which sessions have already injected - Only inject once per session (one-time injection) - Skip subsequent injections with debug log ## Discussion with Orion - Confirmed that `systemPrompt` hook directly overrides base system prompt (no append capability) - The one-time `prependContext` approach is a good compromise until core supports appending to system prompt
zhi added 1 commit 2026-02-28 19:27:54 +00:00
fix: add turn check debug logs + one-time prompt injection f03d32e15f 
1. Turn check improvements:
   - Add debug logs for ctx.agentId, resolved accountId, turnOrder length
   - Fallback to ctx.accountId if resolveAccountId fails
   - Add resolveDiscordUserId debug logs for handoff troubleshooting

2. One-time prompt injection:
   - Add sessionInjected Set to track injected sessions
   - Use prependContext (not systemPrompt) but only inject once per session
   - Skip subsequent injections with debug log
zhi merged commit 7fdc7952b7 into feat/turn-based-speaking 2026-02-28 19:27:59 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
orion zhi nav
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nav/Dirigent#6
Delete Branch "fix/turn-check-debug-and-onetime-inject"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?