Skip to content

feat(discord): include channel name in dynamic context#1452

Closed
Alix-007 wants to merge 1 commit intosipeed:mainfrom
Alix-007:pr/issue-1451-discord-chat-name
Closed

feat(discord): include channel name in dynamic context#1452
Alix-007 wants to merge 1 commit intosipeed:mainfrom
Alix-007:pr/issue-1451-discord-chat-name

Conversation

@Alix-007
Copy link
Copy Markdown
Contributor

Summary

  • include the Discord channel name in inbound metadata when it is available from state or API lookup
  • thread the optional chat name through agent process options into the dynamic system context
  • add regression tests for Discord channel-name resolution and dynamic context rendering

Closes #1451.

Testing

  • go test ./pkg/agent -run TestBuildMessages_IncludesChatNameInDynamicContext -count=1
  • go test ./pkg/agent -run TestTargetReasoningChannelID_AllChannels -count=1
  • go test ./pkg/channels/discord -run TestResolveChannelName_FromState -count=1

@Alix-007
Copy link
Copy Markdown
Contributor Author

Closing this to keep the queue focused on higher-value bugfixes.

After reprioritizing the remaining open PRs, this one lands in the low-value bucket for now:

  • the linked issue is labeled priority: low
  • this is an enhancement rather than a bugfix
  • the branch is now conflicting with main

If Discord context shaping becomes a higher-priority need later, it is better to reopen this as a fresh, narrower PR on top of current main.

@Alix-007 Alix-007 closed this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: agent domain: channel go Pull requests that update go code type: enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Include Discord channel name in agent dynamic context

1 participant