agentirc-cli 4.4.0__tar.gz → 4.4.2__tar.gz

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/CHANGELOG.md

@@ -4,6 +4,32 @@ All notable changes to this project will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/).
 
+## [4.4.2] - 2026-04-08
+
+
+### Fixed
+
+- Codex/copilot: preserve HOME for auth tokens instead of isolating (#159)
+- Codex: fix turn sync race condition causing concatenated rapid-mention responses (#165)
+- All backends: sleep scheduler no longer overrides manual pause (#162)
+- All backends: poll loop filters @mention messages to prevent duplicate responses (#160)
+- All backends: turn errors now send feedback to IRC channel (#163)
+- All backends: consecutive turn failure circuit breaker pauses agent after 3 failures (#164)
+- Status query response verified not leaking to IRC channel (#161)
+
+## [4.4.1] - 2026-04-07
+
+
+### Fixed
+
+- Config save operations no longer strip backend-specific fields like acp_command (#150)
+- Agent status detail uses cached description by default, --full for live query; IPC deadline increased to 15s (#152)
+- DMs now activate agents — _detect_and_fire_mention handles direct messages in all backends (#153)
+- ACP agent runner preserves HOME/XDG_CONFIG_HOME for auth tokens; warns on authMethods, fails fast on session creation failure (#154)
+- _coerce_to_acp_agent now copies the icon field (#155)
+- _make_backend_config passes supervisor, poll_interval, sleep_start, sleep_end to non-claude backends (#156)
+- ACP load_config strips unknown fields, matching claude/codex/copilot pattern (#157)
+
 ## [4.4.0] - 2026-04-07
 
 

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/CLAUDE.md

@@ -11,7 +11,7 @@ Design spec: `docs/superpowers/specs/2026-03-19-agentirc-design.md`
 ## Package Management
 
 - **External packages:** Managed in `pyproject.toml`, installed with `uv`
-- **Internal packages:** Written in `packages/` folder. Internal packages are NOT installed as dependencies — they are assimilated into target projects as organic code, placed in the correct folder and location as if written directly in the target project.
+- **Internal packages:** Written in `packages/` folder. Internal packages are NOT installed as dependencies — they are reflected into target projects as native code, placed in the correct folder and location as if written directly in the target project.
 
 ## Assimilai Pattern
 

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentirc-cli
-Version: 4.4.0
+Version: 4.4.2
 Summary: Legacy alias for culture — install culture instead
 Project-URL: Homepage, https://github.com/OriNachum/culture
 Author: Ori Nachum
@@ -60,7 +60,7 @@ Claude Code · Codex · Copilot · ACP (Cline, Kiro, OpenCode, Gemini, ...)
 
 | | |
 |---|---|
-| 🎓 **Organic Lifecycle** | Introduce → Educate → Join → Mentor → Promote. Members develop through real work, not configuration. |
+| 🎓 **Reflective Lifecycle** | Introduce → Educate → Join → Mentor → Promote. Members develop through real work, not configuration. |
 | 🌐 **Connected Worlds** | Link cultures across machines. Members see each other without a central controller. |
 | 🧭 **Mentorship** | A guide watches for drift, spiraling, and stalling — whispers corrections when needed. |
 | 🤝 **Open Membership** | Claude, Codex, Copilot, or any ACP agent. All are welcome. |
@@ -84,7 +84,7 @@ Claude Code · Codex · Copilot · ACP (Cline, Kiro, OpenCode, Gemini, ...)
 | **Spiraling detection** | AI supervisor reads conversation meaning | Retry limits + escalation timeouts | Retry limits + fallback agents |
 | **Observability** | Live web dashboard + any IRC client | Web dashboard + Slack/Discord/webhook alerts | CLI commands (metrics partially mocked) |
 | **Self-organization** | Tag-driven room membership | Orchestrator assigns issues to workers | ML-based routing with learning pipeline |
-| **Philosophy** | Simple, organic, transparent | Parallel coding coordinator — isolation via worktrees | Enterprise-complex (130+ skills, vector DB, Q-learning) |
+| **Philosophy** | Simple, reflective, transparent | Parallel coding coordinator — isolation via worktrees | Enterprise-complex (130+ skills, vector DB, Q-learning) |
 
 ---
 
@@ -138,9 +138,9 @@ Members on any machine see each other in `#general`. @mentions cross boundaries.
 
 ---
 
-## Organic Development
+## Reflective Development
 
-Culture follows the **Organic Development** paradigm — agents develop through real work, not configuration. The lifecycle is continuous, not graduated:
+Culture follows the **Reflective Development** paradigm — agents develop by reflecting on real work, not by configuration. Documentation flows back as context. Code reflects from reference to implementation. The lifecycle is continuous, not graduated:
 
 👋 **Introduce** → 🎓 **Educate** → 🤝 **Join** → 🧭 **Mentor** → ⭐ **Promote**
 

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/README.md

@@ -36,7 +36,7 @@ Claude Code · Codex · Copilot · ACP (Cline, Kiro, OpenCode, Gemini, ...)
 
 | | |
 |---|---|
-| 🎓 **Organic Lifecycle** | Introduce → Educate → Join → Mentor → Promote. Members develop through real work, not configuration. |
+| 🎓 **Reflective Lifecycle** | Introduce → Educate → Join → Mentor → Promote. Members develop through real work, not configuration. |
 | 🌐 **Connected Worlds** | Link cultures across machines. Members see each other without a central controller. |
 | 🧭 **Mentorship** | A guide watches for drift, spiraling, and stalling — whispers corrections when needed. |
 | 🤝 **Open Membership** | Claude, Codex, Copilot, or any ACP agent. All are welcome. |
@@ -60,7 +60,7 @@ Claude Code · Codex · Copilot · ACP (Cline, Kiro, OpenCode, Gemini, ...)
 | **Spiraling detection** | AI supervisor reads conversation meaning | Retry limits + escalation timeouts | Retry limits + fallback agents |
 | **Observability** | Live web dashboard + any IRC client | Web dashboard + Slack/Discord/webhook alerts | CLI commands (metrics partially mocked) |
 | **Self-organization** | Tag-driven room membership | Orchestrator assigns issues to workers | ML-based routing with learning pipeline |
-| **Philosophy** | Simple, organic, transparent | Parallel coding coordinator — isolation via worktrees | Enterprise-complex (130+ skills, vector DB, Q-learning) |
+| **Philosophy** | Simple, reflective, transparent | Parallel coding coordinator — isolation via worktrees | Enterprise-complex (130+ skills, vector DB, Q-learning) |
 
 ---
 
@@ -114,9 +114,9 @@ Members on any machine see each other in `#general`. @mentions cross boundaries.
 
 ---
 
-## Organic Development
+## Reflective Development
 
-Culture follows the **Organic Development** paradigm — agents develop through real work, not configuration. The lifecycle is continuous, not graduated:
+Culture follows the **Reflective Development** paradigm — agents develop by reflecting on real work, not by configuration. Documentation flows back as context. Code reflects from reference to implementation. The lifecycle is continuous, not graduated:
 
 👋 **Introduce** → 🎓 **Educate** → 🤝 **Join** → 🧭 **Mentor** → ⭐ **Promote**
 

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/cli/agent.py

@@ -455,8 +455,12 @@ def _make_backend_config(config: DaemonConfig, backend_daemon_config_cls):
     """Build a backend-specific DaemonConfig from the base config."""
     return backend_daemon_config_cls(
         server=config.server,
+        supervisor=config.supervisor,
         webhooks=config.webhooks,
         buffer_size=config.buffer_size,
+        poll_interval=config.poll_interval,
+        sleep_start=config.sleep_start,
+        sleep_end=config.sleep_end,
         agents=config.agents,
     )
 
@@ -484,6 +488,7 @@ def _coerce_to_acp_agent(agent: AgentConfig):
         model=agent.model,
         system_prompt=agent.system_prompt,
         tags=agent.tags,
+        icon=agent.icon,
     )
 
 

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/cli/shared/display.py

@@ -37,7 +37,8 @@ def print_agent_detail(agent, config_path: str, args: argparse.Namespace) -> Non
     print(f"  PID:        {pid or '-'}")
 
     if status == "running":
-        resp = asyncio.run(ipc_request(agent_socket_path(agent.nick), "status", query=True))
+        query = getattr(args, "full", False)
+        resp = asyncio.run(ipc_request(agent_socket_path(agent.nick), "status", query=query))
         if resp and resp.get("ok"):
             data = resp.get("data", {})
             print(f"  Activity:   {data.get('description', 'nothing')}")

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/cli/shared/ipc.py

@@ -31,7 +31,7 @@ async def ipc_request(socket_path: str, msg_type: str, **kwargs) -> dict | None:
         writer.write(encode_message(req))
         await writer.drain()
         loop = asyncio.get_running_loop()
-        deadline = loop.time() + 3.0
+        deadline = loop.time() + 15.0
         while True:
             remaining = deadline - loop.time()
             if remaining <= 0:

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/clients/acp/agent_runner.py

@@ -70,11 +70,12 @@ class ACPAgentRunner:
         """Start the ACP agent as a subprocess and initialize a session."""
         self._stopping = False
 
-        # Isolate from host config (XDG, etc.)
+        # Isolate data/state dirs to prevent session interference, but
+        # preserve HOME and XDG_CONFIG_HOME so the agent finds auth tokens.
         self._isolated_home = tempfile.mkdtemp(prefix="culture-acp-")
         isolated_env = dict(os.environ)
-        isolated_env["HOME"] = self._isolated_home
-        isolated_env.pop("XDG_CONFIG_HOME", None)
+        isolated_env["XDG_DATA_HOME"] = os.path.join(self._isolated_home, ".local", "share")
+        isolated_env["XDG_STATE_HOME"] = os.path.join(self._isolated_home, ".local", "state")
 
         cmd_label = " ".join(self.acp_command)
         try:
@@ -108,6 +109,18 @@ class ACPAgentRunner:
             )
             logger.info("ACP initialized (%s): %s", cmd_label, resp)
 
+            # Log available auth methods as a hint
+            init_result = resp.get("result", {})
+            auth_methods = init_result.get("authMethods", [])
+            if auth_methods:
+                descriptions = [m.get("description", m.get("id", "unknown")) for m in auth_methods]
+                logger.warning(
+                    "ACP agent (%s) reports auth methods: %s. "
+                    "If prompts fail, configure auth tokens.",
+                    cmd_label,
+                    ", ".join(descriptions),
+                )
+
             # Create a session with model selection
             session_params = {
                 "cwd": self.directory,
@@ -120,6 +133,13 @@ class ACPAgentRunner:
             logger.info("ACP session/new raw response: %s", json.dumps(resp)[:500])
             result = resp.get("result", {})
             self._session_id = result.get("sessionId")
+            if not self._session_id:
+                # Session creation failed — likely auth or model issue
+                error = resp.get("error", {})
+                error_msg = error.get("message", "unknown error")
+                if auth_methods:
+                    error_msg += f". Auth may be required: {', '.join(descriptions)}"
+                raise RuntimeError(f"ACP agent ({cmd_label}) session creation failed: {error_msg}")
             self._running = True
             logger.info("ACP session started (%s): %s", cmd_label, self._session_id)
 

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/clients/acp/config.py

@@ -93,8 +93,10 @@ def load_config(path: str | Path) -> DaemonConfig:
     webhooks = WebhookConfig(**raw.get("webhooks", {}))
 
     agents = []
+    known_agent_fields = {f.name for f in AgentConfig.__dataclass_fields__.values()}
     for agent_raw in raw.get("agents", []):
-        agents.append(AgentConfig(**agent_raw))
+        filtered = {k: v for k, v in agent_raw.items() if k in known_agent_fields}
+        agents.append(AgentConfig(**filtered))
 
     return DaemonConfig(
         server=server,
@@ -157,6 +159,32 @@ def save_config(path: str | Path, config: DaemonConfig) -> None:
         raise
 
 
+def _load_raw_yaml(path: str | Path) -> dict:
+    """Load raw YAML from a config file, returning empty dict if missing."""
+    path = Path(path)
+    if not path.exists():
+        return {}
+    with open(path) as f:
+        return yaml.safe_load(f) or {}
+
+
+def _save_raw_yaml(path: str | Path, raw: dict) -> None:
+    """Write raw YAML dict atomically."""
+    path = Path(path)
+    path.parent.mkdir(parents=True, exist_ok=True)
+    fd, tmp_path = tempfile.mkstemp(dir=str(path.parent), suffix=".yaml.tmp")
+    try:
+        with os.fdopen(fd, "w") as f:
+            yaml.dump(raw, f, default_flow_style=False, sort_keys=False)
+        os.replace(tmp_path, str(path))
+    except BaseException:
+        try:
+            os.unlink(tmp_path)
+        except OSError:
+            pass
+        raise
+
+
 def add_agent_to_config(
     path: str | Path,
     agent: AgentConfig,
@@ -166,20 +194,21 @@ def add_agent_to_config(
 
     If server_name is provided, updates config.server.name.
     Raises ValueError if an agent with the same nick already exists.
+    Operates on raw YAML to preserve backend-specific fields.
     """
-    config = load_config_or_default(path)
-
-    if server_name is not None:
-        config.server.name = server_name
+    raw = _load_raw_yaml(path)
+    agents = raw.setdefault("agents", [])
 
-    # Check for nick collision
-    for existing in config.agents:
-        if existing.nick == agent.nick:
+    for existing in agents:
+        if existing.get("nick") == agent.nick:
             raise ValueError(f"Agent with nick {agent.nick!r} already exists in config")
 
-    config.agents.append(agent)
-    save_config(path, config)
-    return config
+    if server_name is not None:
+        raw.setdefault("server", {})["name"] = server_name
+
+    agents.append(asdict(agent))
+    _save_raw_yaml(path, raw)
+    return load_config(path)
 
 
 def rename_server(
@@ -189,23 +218,26 @@ def rename_server(
     """Rename the server and update all agent nick prefixes.
 
     Returns (old_name, [(old_nick, new_nick), ...]).
+    Operates on raw YAML to preserve backend-specific fields.
     """
-    config = load_config_or_default(path)
-    old_name = config.server.name
+    raw = _load_raw_yaml(path)
+    server = raw.get("server", {})
+    old_name = server.get("name", ServerConnConfig().name)
 
     if old_name == new_name:
         return old_name, []
 
-    # Plan renames and check for collisions before mutating
+    agents = raw.get("agents", [])
     prefix = f"{old_name}-"
     plan: list[tuple[int, str, str]] = []
-    for i, agent in enumerate(config.agents):
-        if agent.nick.startswith(prefix):
-            new_nick = f"{new_name}-{agent.nick[len(prefix):]}"
-            plan.append((i, agent.nick, new_nick))
+    for i, agent_raw in enumerate(agents):
+        nick = agent_raw.get("nick", "")
+        if nick.startswith(prefix):
+            new_nick = f"{new_name}-{nick[len(prefix):]}"
+            plan.append((i, nick, new_nick))
 
     planned_nicks = {new_nick for _, _, new_nick in plan}
-    existing_nicks = {a.nick for a in config.agents} - {old for _, old, _ in plan}
+    existing_nicks = {a.get("nick", "") for a in agents} - {old for _, old, _ in plan}
     collisions = planned_nicks & existing_nicks
     if collisions:
         raise ValueError(
@@ -213,14 +245,15 @@ def rename_server(
             f"duplicate nick(s): {', '.join(sorted(collisions))}"
         )
 
-    config.server.name = new_name
+    server["name"] = new_name
+    raw["server"] = server
 
     renamed: list[tuple[str, str]] = []
     for i, old_nick, new_nick in plan:
-        config.agents[i].nick = new_nick
+        agents[i]["nick"] = new_nick
         renamed.append((old_nick, new_nick))
 
-    save_config(path, config)
+    _save_raw_yaml(path, raw)
     return old_name, renamed
 
 
@@ -232,19 +265,19 @@ def rename_agent(
     """Rename an agent's nick in the config.
 
     Raises ValueError if old_nick is not found or new_nick already exists.
+    Operates on raw YAML to preserve backend-specific fields.
     """
-    config = load_config_or_default(path)
+    raw = _load_raw_yaml(path)
+    agents = raw.get("agents", [])
 
-    # Check new nick doesn't collide
-    for agent in config.agents:
-        if agent.nick == new_nick:
+    for agent_raw in agents:
+        if agent_raw.get("nick") == new_nick:
             raise ValueError(f"Agent with nick {new_nick!r} already exists in config")
 
-    # Find and rename
-    for agent in config.agents:
-        if agent.nick == old_nick:
-            agent.nick = new_nick
-            save_config(path, config)
+    for agent_raw in agents:
+        if agent_raw.get("nick") == old_nick:
+            agent_raw["nick"] = new_nick
+            _save_raw_yaml(path, raw)
             return
 
     raise ValueError(f"Agent {old_nick!r} not found in config")
@@ -260,18 +293,14 @@ def remove_agent(
     agents that the typed schema would strip.
     Raises ValueError if the agent is not found.
     """
-    path = Path(path)
-    if not path.exists():
+    raw = _load_raw_yaml(path)
+    if not raw:
         raise ValueError(f"Agent {nick!r} not found in config")
 
-    with open(path) as f:
-        raw = yaml.safe_load(f) or {}
-
     agents = raw.get("agents", [])
     for i, agent_raw in enumerate(agents):
         if agent_raw.get("nick") == nick:
             agents.pop(i)
-            with open(path, "w") as f:
-                yaml.dump(raw, f, default_flow_style=False, sort_keys=False)
+            _save_raw_yaml(path, raw)
             return
     raise ValueError(f"Agent {nick!r} not found in config")

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/clients/acp/daemon.py

@@ -13,6 +13,7 @@ import asyncio
 import datetime
 import logging
 import os
+import re
 import time
 from collections import deque
 
@@ -37,6 +38,7 @@ _ERR_MISSING_CHANNEL_THREAD_MSG = "Missing 'channel', 'thread', or 'message'"
 MAX_CRASH_COUNT = 3
 CRASH_WINDOW_SECONDS = 300
 CRASH_RESTART_DELAY = 5
+MAX_CONSECUTIVE_TURN_FAILURES = 3
 
 
 class ACPDaemon:
@@ -74,9 +76,11 @@ class ACPDaemon:
         # Crash-recovery state
         self._crash_times: list[float] = []
         self._circuit_open = False
+        self._consecutive_turn_failures: int = 0
 
         # Pause/sleep state
         self._paused: bool = False
+        self._manually_paused: bool = False
         self._last_activation: float | None = None
 
         # Status query state — for asking the agent what it's doing
@@ -270,7 +274,7 @@ class ACPDaemon:
                 if should_sleep and not self._paused:
                     self._paused = True
                     logger.info("Sleep schedule: pausing %s", self.agent.nick)
-                elif not should_sleep and self._paused:
+                elif not should_sleep and self._paused and not self._manually_paused:
                     self._paused = False
                     logger.info("Sleep schedule: resuming %s", self.agent.nick)
             except asyncio.CancelledError:
@@ -300,6 +304,17 @@ class ACPDaemon:
 
     def _send_channel_poll(self, channel) -> None:
         msgs = self._buffer.read(channel)
+        if not msgs:
+            return
+        # Filter out messages that @mention this agent (already handled by _on_mention)
+        nick = self.agent.nick
+        short = nick.split("-", 1)[1] if "-" in nick else None
+        msgs = [
+            m
+            for m in msgs
+            if not re.search(rf"@{re.escape(nick)}\b", m.text)
+            and not (short and re.search(rf"@{re.escape(short)}\b", m.text))
+        ]
         if not msgs:
             return
         lines = "\n".join(f"  <{m.nick}> {m.text}" for m in msgs)
@@ -330,10 +345,35 @@ class ACPDaemon:
     # Agent runner helpers
     # ------------------------------------------------------------------
 
-    def _on_turn_error(self) -> None:
-        """Clean up stale relay target when a prompt fails."""
+    async def _on_turn_error(self) -> None:
+        """Send error feedback to IRC and clean up stale relay target."""
         if self._mention_targets:
-            self._mention_targets.popleft()
+            relay_target = self._mention_targets.popleft()
+            if self._transport and relay_target:
+                await self._transport.send_privmsg(
+                    relay_target,
+                    "Sorry, I encountered an error processing your request.",
+                )
+        self._consecutive_turn_failures += 1
+        if self._consecutive_turn_failures >= MAX_CONSECUTIVE_TURN_FAILURES:
+            self._paused = True
+            self._manually_paused = True
+            logger.error(
+                "Agent %s paused after %d consecutive turn failures",
+                self.agent.nick,
+                self._consecutive_turn_failures,
+            )
+            if self._webhook:
+                await self._webhook.fire(
+                    AlertEvent(
+                        event_type="agent_spiraling",
+                        nick=self.agent.nick,
+                        message=(
+                            f"Agent {self.agent.nick} paused after "
+                            f"{self._consecutive_turn_failures} consecutive turn failures."
+                        ),
+                    )
+                )
 
     async def _start_agent_runner(self) -> None:
         self._agent_runner = ACPAgentRunner(
@@ -475,6 +515,7 @@ class ACPDaemon:
 
     async def _on_agent_message(self, msg: dict) -> None:
         """Relay agent text to IRC and feed to supervisor."""
+        self._consecutive_turn_failures = 0
         await self._relay_response_to_irc(msg)
 
         if self._supervisor:
@@ -621,11 +662,13 @@ class ACPDaemon:
 
     def _ipc_pause(self, req_id: str, msg: dict) -> dict:
         self._paused = True
-        logger.info("Agent %s paused", self.agent.nick)
+        self._manually_paused = True
+        logger.info("Agent %s paused (manual)", self.agent.nick)
         return make_response(req_id, ok=True)
 
     def _ipc_resume(self, req_id: str, msg: dict) -> dict:
         self._paused = False
+        self._manually_paused = False
         logger.info("Agent %s resumed", self.agent.nick)
         # NOTE: Catch-up on missed messages is not yet implemented.
         # IRCTransport does not process HISTORY responses into the buffer.

{agentirc_cli-4.4.0 → agentirc_cli-4.4.2}/culture/clients/acp/irc_transport.py

@@ -213,6 +213,10 @@ class IRCTransport:
         """Check if the message mentions this agent and fire the callback."""
         if not self.on_mention:
             return
+        # DMs always activate (target is the agent's own nick)
+        if target == self.nick:
+            self.on_mention(target, sender, text)
+            return
         short = self.nick.split("-", 1)[1] if "-" in self.nick else None
         if re.search(rf"@{re.escape(self.nick)}\b", text) or (
             short and re.search(rf"@{re.escape(short)}\b", text)