Skip to content

fix(irc): replace Libera.Chat default examples with neutral IRC server guidance - #61194

Closed
kyssta-exe wants to merge 1 commit into
NousResearch:mainfrom
kyssta-exe:fix/61181-irc-default-network
Closed

fix(irc): replace Libera.Chat default examples with neutral IRC server guidance#61194
kyssta-exe wants to merge 1 commit into
NousResearch:mainfrom
kyssta-exe:fix/61181-irc-default-network

Conversation

@kyssta-exe

Copy link
Copy Markdown
Contributor

Description

Fixes #61181.

Libera.Chat has a policy prohibiting LLM-driven autonomous clients. The IRC plugin's documentation, config examples, and interactive setup flow all recommended irc.libera.chat as the default server, causing users to inadvertently violate network policies.

Changes

  • plugin.yaml — changed example from irc.libera.chat to irc.example.net with a note to check docs about public network policies
  • adapter.py — updated docstring example, setup prompt, and added a warning banner during interactive setup (hermes gateway setup)
  • config.py — updated IRC_SERVER env var description
  • website docs — updated both the reference (env vars) and user guide (IRC messaging) pages with warnings and neutral examples
  • tests — updated test fixtures to use irc.example.net instead of irc.libera.chat

Testing

All changes are to documentation strings, example strings, and test fixtures — no runtime logic was modified. The IRC plugin continues to function identically regardless of what server the user configures.

…r guidance (NousResearch#61181)

Libera.Chat has a policy prohibiting LLM-driven autonomous clients.
The IRC plugin's documentation and config examples recommended
Libera.Chat as the default server, causing users to inadvertently
violate network policies.

Changes:
- Replace all irc.libera.chat example references with irc.example.net
- Add prominent warning in the IRC docs about public network policies
- Suggest running a local IRCd (e.g. Ergo via Docker) for production
- Add warning to the interactive hermes gateway setup flow
- Update test fixtures to use irc.example.net instead
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery labels Jul 9, 2026

@tonydwb tonydwb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Summary

Verdict: Approved

Looks Good

  • Replaces Libera.Chat-specific default with neutral IRC server guidance in the IRC adapter
  • Scoped fix, no side effects
  • No security concerns

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused IRC guidance correction. No substantive correctness or completeness problems were found: current main still contains the Libera.Chat recommendations at plugins/platforms/irc/adapter.py:15,563,566, plugins/platforms/irc/plugin.yaml:15, hermes_cli/config.py:4314, and website/docs/user-guide/messaging/irc.md:3,11,28,43; the PR diff updates each user-facing location and adds the requested policy warning.

This is an automated hermes-sweeper review.

@teknium1 teknium1 added the sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users label Jul 10, 2026
@kyssta-exe

Copy link
Copy Markdown
Contributor Author

Stale — no merge activity for 4-6 days. Can resubmit if still needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Recommended IRC configuration uses a network that bans agentic connections

4 participants