agentirc-cli 4.2.1__tar.gz → 4.3.1__tar.gz

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/.github/workflows/pages.yml

@@ -5,11 +5,6 @@ on:
     branches: ["main"]
   workflow_dispatch:
 
-permissions:
-  contents: read
-  pages: write
-  id-token: write
-
 concurrency:
   group: "pages"
   cancel-in-progress: false
@@ -17,6 +12,9 @@ concurrency:
 jobs:
   build:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pages: write
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -46,6 +44,9 @@ jobs:
       url: ${{ steps.deployment.outputs.page_url }}
     runs-on: ubuntu-latest
     needs: build
+    permissions:
+      pages: write
+      id-token: write
     steps:
       - name: Deploy to GitHub Pages
         id: deployment

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/.github/workflows/publish.yml

@@ -12,12 +12,11 @@ on:
       - "pyproject.toml"
       - "culture/**"
 
-permissions:
-  contents: read
-
 jobs:
   test:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     steps:
       - uses: actions/checkout@v4
 

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/.github/workflows/security-checks.yml

@@ -9,13 +9,12 @@ on:
     - cron: '0 0 * * 0'  # Weekly on Sunday at midnight
   workflow_dispatch:
 
-permissions:
-  contents: read
-
 jobs:
   security-scans:
     name: Security Scans
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     steps:
       - uses: actions/checkout@v4
 

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/.github/workflows/tests.yml

@@ -4,12 +4,11 @@ on:
   pull_request:
     branches: [main]
 
-permissions:
-  contents: read
-
 jobs:
   test:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     steps:
       - uses: actions/checkout@v4
 

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/CHANGELOG.md

@@ -4,6 +4,26 @@ All notable changes to this project will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/).
 
+## [4.3.1] - 2026-04-07
+
+
+### Fixed
+
+- Remove unnecessary list() wrapping on already-iterable values (SonarCloud S7504/S7494)
+
+## [4.3.0] - 2026-04-07
+
+
+### Added
+
+- agent delete command to remove agents from config entirely
+- agent create now overwrites archived agents, enabling harness/model migration
+
+
+### Fixed
+
+- agent create no longer blocks when the matching nick is archived
+
 ## [4.2.1] - 2026-04-07
 
 

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentirc-cli
-Version: 4.2.1
+Version: 4.3.1
 Summary: Legacy alias for culture — install culture instead
 Project-URL: Homepage, https://github.com/OriNachum/culture
 Author: Ori Nachum

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/bots/bot_manager.py

@@ -86,7 +86,7 @@ class BotManager:
 
     def list_bots(self, owner: str | None = None) -> list[Bot]:
         """List bots, optionally filtered by owner."""
-        bots = list(self.bots.values())
+        bots = self.bots.values()
         if owner:
             bots = [b for b in bots if b.config.owner == owner]
         return sorted(bots, key=lambda b: b.name)

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/cli/agent.py

@@ -16,6 +16,7 @@ from culture.clients.claude.config import (
     archive_agent,
     load_config,
     load_config_or_default,
+    remove_agent,
     sanitize_agent_name,
     unarchive_agent,
 )
@@ -158,11 +159,16 @@ def register(subparsers: argparse._SubParsersAction) -> None:
     unarchive_parser.add_argument("nick", help="Agent nick to unarchive")
     unarchive_parser.add_argument("--config", default=DEFAULT_CONFIG, help=_CONFIG_HELP)
 
+    # -- delete ---------------------------------------------------------------
+    delete_parser = agent_sub.add_parser("delete", help="Remove an agent from config entirely")
+    delete_parser.add_argument("nick", help="Agent nick to delete")
+    delete_parser.add_argument("--config", default=DEFAULT_CONFIG, help=_CONFIG_HELP)
+
 
 def dispatch(args: argparse.Namespace) -> None:
     if not args.agent_command:
         print(
-            "Usage: culture agent {create|join|start|stop|status|rename|assign|sleep|wake|learn|message|read|archive|unarchive}",
+            "Usage: culture agent {create|join|start|stop|status|rename|assign|sleep|wake|learn|message|read|archive|unarchive|delete}",
             file=sys.stderr,
         )
         sys.exit(1)
@@ -182,6 +188,7 @@ def dispatch(args: argparse.Namespace) -> None:
         "read": _cmd_read,
         "archive": _cmd_archive,
         "unarchive": _cmd_unarchive,
+        "delete": _cmd_delete,
     }
     handler = handlers.get(args.agent_command)
     if handler:
@@ -264,6 +271,10 @@ def _cmd_create(args: argparse.Namespace) -> None:
 
     for existing in config.agents:
         if existing.nick == full_nick:
+            if existing.archived:
+                print(f"Replacing archived agent '{full_nick}'")
+                remove_agent(args.config, full_nick)
+                break
             channels = existing.channels if isinstance(existing.channels, list) else []
             print(f"Agent '{full_nick}' already exists in config", file=sys.stderr)
             print(f"  Directory: {existing.directory}", file=sys.stderr)
@@ -437,10 +448,10 @@ async def _run_single_agent(config: DaemonConfig, agent: AgentConfig) -> None:
                 agent="acp",
                 acp_command=getattr(agent, "acp_command", None) or ["opencode", "acp"],
                 directory=agent.directory,
-                channels=list(agent.channels),
+                channels=agent.channels,
                 model=agent.model,
                 system_prompt=agent.system_prompt,
-                tags=list(agent.tags),
+                tags=agent.tags,
             )
         else:
             acp_agent = agent
@@ -831,3 +842,20 @@ def _cmd_unarchive(args: argparse.Namespace) -> None:
 
     print(f"Agent unarchived: {args.nick}")
     print(f"\nStart with: culture agent start {args.nick}")
+
+
+def _cmd_delete(args: argparse.Namespace) -> None:
+    """Remove an agent from config entirely."""
+    config = load_config_or_default(args.config)
+    agent = config.get_agent(args.nick)
+    if not agent:
+        print(f"Agent '{args.nick}' not found in config", file=sys.stderr)
+        sys.exit(1)
+
+    # Stop the agent if it's running
+    pid = read_pid(f"agent-{args.nick}")
+    if pid and is_process_alive(pid):
+        stop_agent(args.nick)
+
+    remove_agent(args.config, args.nick)
+    print(f"Agent deleted: {args.nick}")

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/cli/mesh.py

@@ -324,7 +324,7 @@ def _generate_agent_configs(mesh, server_name: str) -> None:
                     nick=full_nick,
                     agent=a.type,
                     directory=workdir,
-                    channels=list(a.channels),
+                    channels=a.channels,
                 )
             )
 

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/cli/server.py

@@ -230,7 +230,7 @@ def _server_start(args: argparse.Namespace) -> None:
         print(f"Server '{args.name}' is already running (PID {existing})")
         sys.exit(1)
 
-    links = list(args.link)
+    links = args.link
     if getattr(args, "mesh_config", None):
         links = resolve_links_from_mesh(args.mesh_config)
 

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/clients/acp/config.py

@@ -248,3 +248,30 @@ def rename_agent(
             return
 
     raise ValueError(f"agent {old_nick!r} not found in config")
+
+
+def remove_agent(
+    path: str | Path,
+    nick: str,
+) -> None:
+    """Remove an agent from config entirely.
+
+    Operates on raw YAML to preserve backend-specific fields on other
+    agents that the typed schema would strip.
+    Raises ValueError if the agent is not found.
+    """
+    path = Path(path)
+    if not path.exists():
+        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)
+            return
+    raise ValueError(f"agent {nick!r} not found in config")

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/clients/claude/config.py

@@ -303,6 +303,33 @@ def unarchive_agent(
     raise ValueError(f"agent {nick!r} not found in config")
 
 
+def remove_agent(
+    path: str | Path,
+    nick: str,
+) -> None:
+    """Remove an agent from config entirely.
+
+    Operates on raw YAML to preserve backend-specific fields (e.g.
+    acp_command) on other agents that the typed schema would strip.
+    Raises ValueError if the agent is not found.
+    """
+    path = Path(path)
+    if not path.exists():
+        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)
+            return
+    raise ValueError(f"agent {nick!r} not found in config")
+
+
 def archive_server(
     path: str | Path,
     reason: str = "",

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/clients/codex/config.py

@@ -248,3 +248,30 @@ def rename_agent(
             return
 
     raise ValueError(f"agent {old_nick!r} not found in config")
+
+
+def remove_agent(
+    path: str | Path,
+    nick: str,
+) -> None:
+    """Remove an agent from config entirely.
+
+    Operates on raw YAML to preserve backend-specific fields on other
+    agents that the typed schema would strip.
+    Raises ValueError if the agent is not found.
+    """
+    path = Path(path)
+    if not path.exists():
+        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)
+            return
+    raise ValueError(f"agent {nick!r} not found in config")

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/clients/copilot/config.py

@@ -248,3 +248,30 @@ def rename_agent(
             return
 
     raise ValueError(f"agent {old_nick!r} not found in config")
+
+
+def remove_agent(
+    path: str | Path,
+    nick: str,
+) -> None:
+    """Remove an agent from config entirely.
+
+    Operates on raw YAML to preserve backend-specific fields on other
+    agents that the typed schema would strip.
+    Raises ValueError if the agent is not found.
+    """
+    path = Path(path)
+    if not path.exists():
+        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)
+            return
+    raise ValueError(f"agent {nick!r} not found in config")

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/culture/console/client.py

@@ -199,7 +199,7 @@ class ConsoleIRCClient:
         finally:
             self._pending.pop("323", None)
 
-        channels = list(self._collect_buffers.pop(key, []))
+        channels = self._collect_buffers.pop(key, [])
         return sorted(channels)
 
     async def who(self, target: str) -> list[dict]:
@@ -221,7 +221,7 @@ class ConsoleIRCClient:
         finally:
             self._pending.pop(f"315:{target}", None)
 
-        entries = list(self._collect_buffers.pop(key, []))
+        entries = self._collect_buffers.pop(key, [])
         return entries
 
     async def history(self, channel: str, limit: int = 50) -> list[dict]:
@@ -243,7 +243,7 @@ class ConsoleIRCClient:
         finally:
             self._pending.pop(f"HISTORYEND:{channel}", None)
 
-        entries = list(self._collect_buffers.pop(key, []))
+        entries = self._collect_buffers.pop(key, [])
         return entries
 
     # ------------------------------------------------------------------

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/docs/superpowers/specs/2026-04-07-entity-archiving-design.md

@@ -238,6 +238,58 @@ per-file (no list to search through).
 - `test_start_refuses_archived_server` -- verify error message
 - `test_start_all_skips_archived` -- verify `--all` only starts active agents
 
+## Agent Deletion and Create-Overwrite
+
+### Problem
+
+Once an agent is archived, there's no way to recreate it with a different
+backend or model. `culture agent create` blocks with "already exists in config"
+because the archived entry is still present. Users must manually edit
+`agents.yaml` to change an agent's harness.
+
+### Delete Command
+
+```
+culture agent delete <nick> [--config PATH]
+```
+
+Removes an agent from config entirely. Stops the agent if running.
+
+### Create Overwrites Archived Agents
+
+`culture agent create` now detects when the matching agent is archived and
+automatically replaces it:
+
+```
+$ culture agent archive spark-daria
+Agent archived: spark-daria
+
+$ culture agent create --server spark --nick daria --agent acp
+Replacing archived agent 'spark-daria'
+Agent created: spark-daria
+  ...
+```
+
+Non-archived agents still block `create` with the existing error. This is the
+primary mechanism for moving an agent to a different harness or model.
+
+### Config Layer
+
+```python
+def remove_agent(path, nick):
+    """Remove an agent from config entirely. Raises ValueError if not found."""
+```
+
+Added to all backend config files (`claude`, `acp`, `codex`, `copilot`) and
+`packages/agent-harness/config.py`. Uses raw YAML manipulation to preserve
+backend-specific fields (e.g. `acp_command`) on other agents.
+
+### Protocol Details
+
+No new protocol changes. Delete and create-overwrite are CLI/config operations
+only, consistent with the existing archiving scope (CLI-only, no federation
+concern).
+
 ## Verification
 
 1. Create test agents: `culture agent create --nick test1 && culture agent create --nick test2`

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/packages/agent-harness/config.py

@@ -246,3 +246,30 @@ def rename_agent(
             return
 
     raise ValueError(f"agent {old_nick!r} not found in config")
+
+
+def remove_agent(
+    path: str | Path,
+    nick: str,
+) -> None:
+    """Remove an agent from config entirely.
+
+    Operates on raw YAML to preserve backend-specific fields on other
+    agents that the typed schema would strip.
+    Raises ValueError if the agent is not found.
+    """
+    path = Path(path)
+    if not path.exists():
+        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)
+            return
+    raise ValueError(f"agent {nick!r} not found in config")

{agentirc_cli-4.2.1 → agentirc_cli-4.3.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agentirc-cli"
-version = "4.2.1"
+version = "4.3.1"
 description = "Legacy alias for culture — install culture instead"
 readme = "README.md"
 license = "MIT"