agentirc-cli 0.10.6__tar.gz → 0.10.7__tar.gz

agentirc_cli-0.10.7/.claude/skills/pr-review/SKILL.md

@@ -0,0 +1,184 @@
+---
+name: pr-review
+description: >
+  Full PR workflow for agentirc: branch, commit, push, create PR, wait for
+  automated reviewers, fetch comments, fix or pushback, reply, resolve threads.
+  Use when: creating PRs, handling review feedback, or the user says
+  "create PR", "review comments", "address feedback", or "resolve threads".
+---
+
+# PR Review Workflow
+
+Complete pull request lifecycle for the agentirc project. Follow every step
+in order.
+
+## Step 1 — Branch
+
+If you are on `main`, create a feature branch first:
+
+```bash
+git checkout -b <branch-name>
+```
+
+Branch naming conventions:
+
+| Type | Pattern | Example |
+|------|---------|---------|
+| Bug fix | `fix/<short-desc>` | `fix/server-not-running-crash` |
+| Feature | `feat/<short-desc>` | `feat/webhook-alerts` |
+| Docs | `docs/<short-desc>` | `docs/protocol-extensions` |
+
+## Step 2 — Make changes, commit, push
+
+1. Edit code
+2. Run tests: `uv run pytest tests/ -x -q`
+3. Bump the version (required before PR):
+
+   ```bash
+   echo '{"fixed":["..."]}' | python3 ~/.claude/skills/version-bump/scripts/bump.py patch
+   ```
+
+4. Stage and commit:
+
+   ```bash
+   git add <files>
+   git commit -m "$(cat <<'EOF'
+   Commit message here.
+
+   Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
+   EOF
+   )"
+   ```
+
+5. Push:
+
+   ```bash
+   git push -u origin <branch-name>
+   ```
+
+## Step 3 — Create PR
+
+```bash
+gh pr create --title "Short title" --body "$(cat <<'EOF'
+## Summary
+- Bullet points describing changes
+
+## Test plan
+- [ ] Test items
+
+- Claude
+EOF
+)"
+```
+
+## Step 4 — Wait for reviewers
+
+Automated reviewers (Qodo, Copilot) need time to post comments.
+
+**Wait 5 minutes** after creating the PR before checking for comments:
+
+```bash
+sleep 300
+```
+
+## Step 5 — Poll for comments
+
+After the initial wait, poll every 60 seconds until comments appear:
+
+```bash
+bash ~/.claude/skills/pr-review/scripts/pr-comments.sh <PR_NUMBER>
+```
+
+If no comments yet, wait and retry:
+
+```bash
+sleep 60
+bash ~/.claude/skills/pr-review/scripts/pr-comments.sh <PR_NUMBER>
+```
+
+Continue until at least one unresolved comment exists, or 3 consecutive polls
+return zero comments (reviewers are done / not configured).
+
+## Step 6 — Triage each comment
+
+For every review comment, decide:
+
+- **FIX** — valid concern, make the code change
+- **PUSHBACK** — disagree, explain why in the reply
+
+Guidelines:
+
+- Exception type mismatches (e.g., breaking reconnect loops) are always valid
+- Test requests are always valid — add them
+- Style nits are usually valid — fix them
+- Architecture opinions may warrant pushback if they conflict with project
+  conventions (check `CLAUDE.md` and `docs/`)
+
+## Step 7 — Fix code and push
+
+1. Make all code fixes
+2. Run tests: `uv run pytest tests/ -x -q`
+3. Commit with a descriptive message
+4. Push: `git push`
+
+## Step 8 — Reply and resolve threads
+
+Use batch mode to reply to all comments at once:
+
+```bash
+bash ~/.claude/skills/pr-review/scripts/pr-batch.sh --resolve <PR_NUMBER> <<'EOF'
+{"comment_id": 123, "body": "Fixed -- changed X to Y.\n\n- Claude"}
+{"comment_id": 456, "body": "Intentional -- this follows the pattern in Z because...\n\n- Claude"}
+EOF
+```
+
+Or reply to a single comment:
+
+```bash
+bash ~/.claude/skills/pr-review/scripts/pr-reply.sh --resolve <PR_NUMBER> <COMMENT_ID> "Fixed -- updated.\n\n- Claude"
+```
+
+**Important:**
+
+- Always sign replies with `\n\n- Claude`
+- Always use `--resolve` to resolve the thread after replying
+- Every comment must get a reply — no silent fixes
+
+## Step 9 — Wait for merge
+
+**Never merge the PR yourself.** The PR is merged manually on the GitHub site.
+
+Report completion back to the IRC channel:
+
+```bash
+# Using the IRC skill
+AGENTIRC_NICK=<your-nick> python3 -m agentirc.clients.claude.skill.irc_client \
+  send "#general" "PR #<N> — all review threads addressed and resolved. Ready for merge."
+```
+
+## Script reference
+
+| Script | Purpose |
+|--------|---------|
+| `pr-comments.sh <PR>` | Fetch all review comments |
+| `pr-reply.sh [--resolve] <PR> <ID> "body"` | Reply to one comment |
+| `pr-batch.sh [--resolve] <PR> < jsonl` | Batch reply from JSONL stdin |
+
+All scripts auto-detect `owner/repo` from the current git remote.
+
+## Quick reference — full flow
+
+```text
+git checkout -b fix/my-fix
+# ... make changes ...
+uv run pytest tests/ -x -q
+echo '{"fixed":["desc"]}' | python3 ~/.claude/skills/version-bump/scripts/bump.py patch
+git add <files> && git commit -m "message"
+git push -u origin fix/my-fix
+gh pr create --title "..." --body "..."
+sleep 300
+bash ~/.claude/skills/pr-review/scripts/pr-comments.sh <PR>
+# ... fix issues, commit, push ...
+bash ~/.claude/skills/pr-review/scripts/pr-batch.sh --resolve <PR> <<< '{"comment_id":N,"body":"Fixed\n\n- Claude"}'
+# Wait for manual merge — never merge yourself
+```

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/CHANGELOG.md

@@ -4,6 +4,15 @@ All notable changes to this project will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/).
 
+## [0.10.7] - 2026-03-28
+
+
+### Fixed
+
+- Fix crash with cryptic asyncio Event loop is closed errors when starting agent without IRC server running
+- Add server-running pre-check in CLI before starting agent daemon
+- Wrap IRC transport connect in try/except for clear error on connection failure
+
 ## [0.10.6] - 2026-03-28
 
 

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentirc-cli
-Version: 0.10.6
+Version: 0.10.7
 Summary: IRC protocol chatrooms for AI agents (and humans allowed)
 Project-URL: Homepage, https://github.com/OriNachum/agentirc
 Author: Ori Nachum

agentirc_cli-0.10.7/agentirc/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.10.7"

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/agentirc/cli.py

@@ -433,6 +433,26 @@ def _cmd_start(args: argparse.Namespace) -> None:
         print("No agents configured", file=sys.stderr)
         sys.exit(1)
 
+    # Best-effort check that the IRC server is reachable before starting agent(s)
+    import socket as _socket
+
+    server_name = config.server.name
+    host, port = config.server.host, config.server.port
+    try:
+        with _socket.create_connection((host, port), timeout=2):
+            pass
+    except (ConnectionRefusedError, OSError):
+        # TCP probe failed — add PID hint if available
+        hint = ""
+        server_pid = read_pid(f"server-{server_name}")
+        if not server_pid or not is_process_alive(server_pid):
+            hint = f"\nStart it with: agentirc server start --name {server_name}"
+        print(
+            f"Error: cannot connect to IRC server at {host}:{port}.{hint}",
+            file=sys.stderr,
+        )
+        sys.exit(1)
+
     if len(agents) == 1:
         # Run in foreground (single agent)
         agent = agents[0]

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/agentirc/clients/claude/irc_transport.py

@@ -35,7 +35,13 @@ class IRCTransport:
         await self._do_connect()
 
     async def _do_connect(self) -> None:
-        self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        try:
+            self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        except (ConnectionRefusedError, OSError) as exc:
+            raise ConnectionError(
+                f"Cannot connect to IRC server at {self.host}:{self.port} "
+                f"- is the server running?"
+            ) from exc
         await self._send_raw(f"NICK {self.nick}")
         await self._send_raw(f"USER {self.user} 0 * :{self.user}")
         self._read_task = asyncio.create_task(self._read_loop())

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/agentirc/clients/codex/irc_transport.py

@@ -35,7 +35,13 @@ class IRCTransport:
         await self._do_connect()
 
     async def _do_connect(self) -> None:
-        self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        try:
+            self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        except (ConnectionRefusedError, OSError) as exc:
+            raise ConnectionError(
+                f"Cannot connect to IRC server at {self.host}:{self.port} "
+                f"- is the server running?"
+            ) from exc
         await self._send_raw(f"NICK {self.nick}")
         await self._send_raw(f"USER {self.user} 0 * :{self.user}")
         self._read_task = asyncio.create_task(self._read_loop())

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/agentirc/clients/copilot/irc_transport.py

@@ -35,7 +35,13 @@ class IRCTransport:
         await self._do_connect()
 
     async def _do_connect(self) -> None:
-        self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        try:
+            self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        except (ConnectionRefusedError, OSError) as exc:
+            raise ConnectionError(
+                f"Cannot connect to IRC server at {self.host}:{self.port} "
+                f"- is the server running?"
+            ) from exc
         await self._send_raw(f"NICK {self.nick}")
         await self._send_raw(f"USER {self.user} 0 * :{self.user}")
         self._read_task = asyncio.create_task(self._read_loop())

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/agentirc/clients/opencode/irc_transport.py

@@ -35,7 +35,13 @@ class IRCTransport:
         await self._do_connect()
 
     async def _do_connect(self) -> None:
-        self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        try:
+            self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        except (ConnectionRefusedError, OSError) as exc:
+            raise ConnectionError(
+                f"Cannot connect to IRC server at {self.host}:{self.port} "
+                f"- is the server running?"
+            ) from exc
         await self._send_raw(f"NICK {self.nick}")
         await self._send_raw(f"USER {self.user} 0 * :{self.user}")
         self._read_task = asyncio.create_task(self._read_loop())

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/packages/agent-harness/irc_transport.py

@@ -36,7 +36,13 @@ class IRCTransport:
         await self._do_connect()
 
     async def _do_connect(self) -> None:
-        self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        try:
+            self._reader, self._writer = await asyncio.open_connection(self.host, self.port)
+        except (ConnectionRefusedError, OSError) as exc:
+            raise ConnectionError(
+                f"Cannot connect to IRC server at {self.host}:{self.port} "
+                f"- is the server running?"
+            ) from exc
         await self._send_raw(f"NICK {self.nick}")
         await self._send_raw(f"USER {self.user} 0 * :{self.user}")
         self._read_task = asyncio.create_task(self._read_loop())

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agentirc-cli"
-version = "0.10.6"
+version = "0.10.7"
 description = "IRC protocol chatrooms for AI agents (and humans allowed)"
 readme = "README.md"
 license = "MIT"

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/tests/test_irc_transport.py

@@ -1,5 +1,6 @@
 import asyncio
 import pytest
+import re
 from agentirc.clients.claude.irc_transport import IRCTransport
 from agentirc.clients.claude.message_buffer import MessageBuffer
 
@@ -89,3 +90,50 @@ async def test_send_join_part(server):
     await asyncio.sleep(0.2)
     assert "#new" not in server.channels
     await transport.disconnect()
+
+
+@pytest.mark.asyncio
+async def test_connect_raises_on_refused():
+    """Connecting to an unreachable server raises ConnectionError with a clear message."""
+    buf = MessageBuffer()
+    transport = IRCTransport(
+        host="127.0.0.1", port=1,  # nothing listens here
+        nick="testserv-bot", user="bot", channels=["#general"], buffer=buf,
+    )
+    with pytest.raises(ConnectionError, match=re.escape("127.0.0.1:1")):
+        await transport.connect()
+
+
+@pytest.mark.asyncio
+async def test_reconnect_retries_after_connection_error(server):
+    """The reconnect loop retries after ConnectionError instead of crashing."""
+    buf = MessageBuffer()
+    # Use a port where nothing listens so the first _do_connect fails
+    transport = IRCTransport(
+        host="127.0.0.1", port=1,
+        nick="testserv-bot", user="bot", channels=["#general"], buffer=buf,
+    )
+    transport._should_run = True
+    transport._reconnecting = False
+
+    # Patch _do_connect to fail once with ConnectionError, then succeed
+    call_count = 0
+    original_do_connect = transport._do_connect
+
+    async def patched_do_connect():
+        nonlocal call_count
+        call_count += 1
+        if call_count == 1:
+            raise ConnectionError("simulated failure")
+        # On retry, connect to the real server
+        transport.host = "127.0.0.1"
+        transport.port = server.config.port
+        await original_do_connect()
+
+    transport._do_connect = patched_do_connect
+
+    # _reconnect should retry and eventually succeed
+    await asyncio.wait_for(transport._reconnect(), timeout=5.0)
+    assert call_count >= 2
+    assert transport.connected or not transport._reconnecting
+    await transport.disconnect()

{agentirc_cli-0.10.6 → agentirc_cli-0.10.7}/uv.lock

@@ -4,7 +4,7 @@ requires-python = ">=3.12"
 
 [[package]]
 name = "agentirc-cli"
-version = "0.10.6"
+version = "0.10.7"
 source = { editable = "." }
 dependencies = [
     { name = "anthropic" },

agentirc_cli-0.10.6/agentirc/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.10.6"