agent-guild 1.2.0__tar.gz → 1.2.1__tar.gz

{agent_guild-1.2.0 → agent_guild-1.2.1}/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [1.2.1] - 2026-10-22
+
+- The install log says `skipped ... section '<name>' is already in CLAUDE.md or AGENTS.md` instead of the confusing
+  `unchanged`, notes when `.claude/skills` is a symlink and where skills are written, and ends with a hint to merge
+  skipped files that differ by hand.
+- README: a section on adding agent-guild to a project that already has its own agents, skills and rules.
+
 ## [1.2.0] - 2026-10-22
 
 - Follow the updated sdd-feature skill: Phase 0 has a UI track (wireframe) and a system track (`DESIGN.md` with Mermaid

{agent_guild-1.2.0 → agent_guild-1.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: agent-guild
-Version: 1.2.0
+Version: 1.2.1
 Summary: A multi-agent setup for Claude Code (orchestrator + architect, coder, grader, utility subagents) with memory-bank, jevmem and Spec-Driven Development wired in, installed into a project with one command.
 Project-URL: Homepage, https://github.com/illescasDaniel/agent-guild
 Project-URL: Issues, https://github.com/illescasDaniel/agent-guild/issues
@@ -123,6 +123,24 @@ uv run agent-guild /path/to/project --dry-run
 From a checkout, the CLI reads `agents/`, `templates/` and `development-workflow.md` at the repo root; the published
 wheel bundles them.
 
+## Adding it to a project that already has its own setup
+
+The installer never overwrites a file that differs from the shipped one (and skips CLAUDE.md sections that `AGENTS.md`
+or `CLAUDE.md` already has), so it is safe to run on a project with its own agents, skills and rules. After it runs:
+
+- **Skipped files are yours.** Anything the log marks `skipped ... differs` keeps your version. If it is an older or
+  customised copy of a shipped one (the `sdd-feature` skill, `code-grader`, `jev-memory`), diff it against the shipped
+  file and merge what you want by hand. `--force` replaces it instead.
+- **New files can land next to yours.** A skill folder you already have keeps its `SKILL.md`, but the shipped files
+  that are missing (templates, scripts) are added beside it. If `.claude/skills` is a symlink to `.cursor/skills`, the
+  log says so, and the skills are written there.
+- **Cursor reads `.claude/agents/` too.** The agents installed here work in Cursor without copies in `.cursor/agents/`
+  (`.cursor/agents/` wins on a name clash). Cursor ignores `tools`, `disallowedTools` and `effort`, so the read-only
+  agents are not read-only there. If your project has a test that requires every `.claude/agents/*.md` to have a
+  `.cursor/agents/` twin, relax it to require the twin only for the Cursor agents.
+- **Skip what you already have** with `--without memory-bank` / `jevmem` / `sdd`, for example when your own memory-bank
+  skill has a different name.
+
 ## Design choices
 
 - **The orchestrator is the only writer of memory.** Parallel coders writing `memory/` would conflict, and someone has

{agent_guild-1.2.0 → agent_guild-1.2.1}/README.md

@@ -104,6 +104,24 @@ uv run agent-guild /path/to/project --dry-run
 From a checkout, the CLI reads `agents/`, `templates/` and `development-workflow.md` at the repo root; the published
 wheel bundles them.
 
+## Adding it to a project that already has its own setup
+
+The installer never overwrites a file that differs from the shipped one (and skips CLAUDE.md sections that `AGENTS.md`
+or `CLAUDE.md` already has), so it is safe to run on a project with its own agents, skills and rules. After it runs:
+
+- **Skipped files are yours.** Anything the log marks `skipped ... differs` keeps your version. If it is an older or
+  customised copy of a shipped one (the `sdd-feature` skill, `code-grader`, `jev-memory`), diff it against the shipped
+  file and merge what you want by hand. `--force` replaces it instead.
+- **New files can land next to yours.** A skill folder you already have keeps its `SKILL.md`, but the shipped files
+  that are missing (templates, scripts) are added beside it. If `.claude/skills` is a symlink to `.cursor/skills`, the
+  log says so, and the skills are written there.
+- **Cursor reads `.claude/agents/` too.** The agents installed here work in Cursor without copies in `.cursor/agents/`
+  (`.cursor/agents/` wins on a name clash). Cursor ignores `tools`, `disallowedTools` and `effort`, so the read-only
+  agents are not read-only there. If your project has a test that requires every `.claude/agents/*.md` to have a
+  `.cursor/agents/` twin, relax it to require the twin only for the Cursor agents.
+- **Skip what you already have** with `--without memory-bank` / `jevmem` / `sdd`, for example when your own memory-bank
+  skill has a different name.
+
 ## Design choices
 
 - **The orchestrator is the only writer of memory.** Parallel coders writing `memory/` would conflict, and someone has

{agent_guild-1.2.0 → agent_guild-1.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agent-guild"
-version = "1.2.0"
+version = "1.2.1"
 description = "A multi-agent setup for Claude Code (orchestrator + architect, coder, grader, utility subagents) with memory-bank, jevmem and Spec-Driven Development wired in, installed into a project with one command."
 readme = "README.md"
 license = "MIT"

{agent_guild-1.2.0 → agent_guild-1.2.1}/src/agent_guild/cli.py

@@ -10,6 +10,9 @@ from .specialists import SPECIALISTS, detect_languages
 _BUNDLED = Path(__file__).resolve().parent / "_assets"  # present in installed wheels
 DEFAULT_ASSETS = _BUNDLED if _BUNDLED.is_dir() else Path(__file__).resolve().parents[2]  # else: a source checkout
 
+KEPT_HINT = """Files marked "skipped ... differs" keep your version. If a skill or agent of yours is an older or customised
+copy of a shipped one, diff it against the shipped file and merge the changes by hand (--force replaces it)."""
+
 NEXT_STEPS = """Next steps:
   - Review the diff (git status / git diff); edit or delete any copied file or CLAUDE.md section you don't want.
   - Per machine, once: `uvx jevmem config init` and `uvx --from jevmem jevmem warmup`.
@@ -105,6 +108,8 @@ def main(argv: list[str] | None = None) -> int:
     print("\n".join(ctx.lines))
     for w in ctx.warnings:
         print(f"warning: {w}")
+    if any(line.lstrip().startswith("skipped") and "differs" in line for line in ctx.lines):
+        print(f"\n{KEPT_HINT}")
     if not args.dry_run:
         print(CODERS_NEXT_STEPS if args.coders_only else NEXT_STEPS)
         for name in sorted(args.with_coders):

{agent_guild-1.2.0 → agent_guild-1.2.1}/src/agent_guild/installer.py

@@ -85,6 +85,9 @@ def install_skills(ctx: Ctx, opts: Options) -> None:
     wanted = [c for c in COMPONENTS if opts.wants(c)]
     if not wanted:
         return
+    skills_dir = ctx.project / ".claude" / "skills"
+    if skills_dir.is_symlink():
+        ctx.log("note", skills_dir, f"is a symlink to {os.readlink(skills_dir)}: skills are written there")
     with tempfile.TemporaryDirectory(prefix="agent-guild-") as tmp:
         for component in wanted:
             url, subpath, skill_name = SKILL_SOURCES[component]
@@ -129,7 +132,7 @@ def install_rules(ctx: Ctx, opts: Options) -> None:
         added.append("@AGENTS.md import")
     for heading, body in _rule_sections(opts):
         if re.search(rf"^#+ .*{re.escape(heading)}", known, re.MULTILINE):
-            ctx.log("unchanged", claude_md, f"already has '{heading}' (CLAUDE.md or AGENTS.md)")
+            ctx.log("skipped", claude_md, f"section '{heading}' is already in CLAUDE.md or AGENTS.md")
             continue
         text = text.rstrip("\n") + ("\n\n" if text.strip() else "") + body + "\n"
         added.append(heading)

{agent_guild-1.2.0 → agent_guild-1.2.1}/tests/test_cli.py

@@ -364,3 +364,44 @@ def test_given_coders_only_without_with_then_it_is_refused(tmp_path, sources):
     # when / then
     with pytest.raises(SystemExit):
         run(project, sources, "--coders-only")
+
+
+def test_given_edited_agent_when_run_again_then_the_merge_hint_is_printed(tmp_path, sources, capsys):
+    # given
+    project = tmp_path / "p"
+    project.mkdir()
+    run(project, sources)
+    (project / ".claude/agents/coder.md").write_text("mine")
+    capsys.readouterr()
+    # when
+    run(project, sources)
+    # then
+    assert "merge the changes by hand" in capsys.readouterr().out
+
+
+def test_given_sections_already_in_agents_md_then_the_log_says_skipped_not_unchanged(tmp_path, sources, capsys):
+    # given
+    project = tmp_path / "p"
+    project.mkdir()
+    (project / "AGENTS.md").write_text("## Memory bank\n\n## Phase Gate Protocol\n")
+    # when
+    run(project, sources)
+    # then
+    out = capsys.readouterr().out
+    assert "skipped    CLAUDE.md  section 'Phase Gate Protocol' is already in CLAUDE.md or AGENTS.md" in out
+
+
+def test_given_symlinked_skills_dir_then_the_log_notes_where_skills_are_written(tmp_path, sources, capsys):
+    # given
+    project = tmp_path / "p"
+    (project / ".cursor/skills").mkdir(parents=True)
+    (project / ".claude").mkdir()
+    try:
+        (project / ".claude/skills").symlink_to(project / ".cursor/skills", target_is_directory=True)
+    except (OSError, NotImplementedError):
+        pytest.skip("symlinks are not available here")
+    # when
+    run(project, sources)
+    # then
+    assert "is a symlink to" in capsys.readouterr().out
+    assert (project / ".cursor/skills/sdd-feature/SKILL.md").is_file()

{agent_guild-1.2.0 → agent_guild-1.2.1}/uv.lock

@@ -4,7 +4,7 @@ requires-python = ">=3.11"
 
 [[package]]
 name = "agent-guild"
-version = "1.2.0"
+version = "1.2.1"
 source = { editable = "." }
 
 [package.dev-dependencies]