@event4u/agent-config 1.41.1 → 2.0.0

package/scripts/install.py

@@ -1217,188 +1217,13 @@ def _smoke_test_hooks(project_root: Path, package_root: Path) -> int:
     return 1 if failed else 0
 
 
-# --- Global user-level install (Phase 3 — road-to-simplicity-and-everywhere) ---
+# --- Global user-level install — RETIRED (road-to-portable-runtime P0.5) ---
 #
-# `scripts/install --global` ships kernel rules + a curated top-N of skills
-# into per-tool user-scope directories so the agent has them in every
-# project on the machine. Curation lives in templates/global-install-manifest.yml.
-#
-# Files are written under an `event4u/` namespace so `--global --uninstall`
-# can wipe the namespace dir without touching user-added files.
-
-GLOBAL_NAMESPACE = "event4u"
-GLOBAL_MANIFEST_REL = Path("templates") / "global-install-manifest.yml"
-
-# Per-tool global directories. Each surface gets a `rules/` and `skills/`
-# bucket under the event4u/ namespace. claude-desktop reuses claude-code's
-# ~/.claude/ (the two surfaces share the dir per Anthropic's docs).
-GLOBAL_TOOL_DIRS: dict[str, dict[str, Path]] = {
-    "claude-code": {
-        "rules":  Path.home() / ".claude" / "rules"  / GLOBAL_NAMESPACE,
-        "skills": Path.home() / ".claude" / "skills" / GLOBAL_NAMESPACE,
-    },
-    "cursor": {
-        "rules":  Path.home() / ".cursor" / "rules"  / "imported" / GLOBAL_NAMESPACE / "rules",
-        "skills": Path.home() / ".cursor" / "rules"  / "imported" / GLOBAL_NAMESPACE / "skills",
-    },
-    "windsurf": {
-        "rules":  Path.home() / ".codeium" / "windsurf" / "global_workflows" / GLOBAL_NAMESPACE / "rules",
-        "skills": Path.home() / ".codeium" / "windsurf" / "global_workflows" / GLOBAL_NAMESPACE / "skills",
-    },
-    "fallback": {
-        "rules":  Path.home() / ".config" / "agent-config" / "rules"  / GLOBAL_NAMESPACE,
-        "skills": Path.home() / ".config" / "agent-config" / "skills" / GLOBAL_NAMESPACE,
-    },
-}
-
-
-def _load_global_manifest(package_root: Path) -> dict:
-    """Parse templates/global-install-manifest.yml without a YAML dep.
-
-    Tiny hand-rolled parser — only handles the manifest's flat shape:
-    `key: value`, `- item`, and `- key: value` indented under a parent
-    list. We avoid pulling in PyYAML so install.py stays zero-dep.
-    """
-    path = package_root / GLOBAL_MANIFEST_REL
-    if not path.is_file():
-        fail(f"global manifest missing: {path}")
-    out: dict = {"kernel_rules": [], "top_skills": []}
-    section: str | None = None
-    pending: dict | None = None
-    for raw in path.read_text(encoding="utf-8").splitlines():
-        line = raw.split("#", 1)[0].rstrip()
-        if not line.strip():
-            continue
-        if line.startswith("kernel_rules:"):
-            section = "kernel_rules"; pending = None; continue
-        if line.startswith("top_skills:"):
-            section = "top_skills"; pending = None; continue
-        stripped = line.lstrip()
-        indent = len(line) - len(stripped)
-        if section == "kernel_rules" and stripped.startswith("- "):
-            out["kernel_rules"].append(stripped[2:].strip())
-        elif section == "top_skills":
-            if stripped.startswith("- id:"):
-                if pending is not None:
-                    out["top_skills"].append(pending)
-                pending = {"id": stripped.split(":", 1)[1].strip(), "surfaces": []}
-            elif pending is not None and stripped.startswith("surfaces:"):
-                raw_list = stripped.split(":", 1)[1].strip()
-                if raw_list.startswith("[") and raw_list.endswith("]"):
-                    pending["surfaces"] = [s.strip() for s in raw_list[1:-1].split(",") if s.strip()]
-            elif indent == 0 and not stripped.startswith("-"):
-                section = None
-    if pending is not None:
-        out["top_skills"].append(pending)
-    return out
-
-
-def _resolve_skill_source(package_root: Path, skill_id: str) -> Path | None:
-    """Locate a skill's SKILL.md in the package. Prefers .claude/skills/."""
-    for base in (package_root / ".claude" / "skills",
-                 package_root / ".agent-src" / "skills"):
-        candidate = base / skill_id / "SKILL.md"
-        if candidate.is_file():
-            return candidate.parent
-    return None
-
-
-def _resolve_rule_source(package_root: Path, rule_id: str) -> Path | None:
-    """Locate a rule's .md in the package. Prefers .agent-src/rules/."""
-    for base in (package_root / ".agent-src" / "rules",
-                 package_root / ".augment" / "rules"):
-        candidate = base / f"{rule_id}.md"
-        if candidate.is_file():
-            return candidate
-    return None
-
-
-def _global_targets(tools: set[str]) -> dict[str, dict[str, Path]]:
-    """Return the subset of GLOBAL_TOOL_DIRS to write to. Fallback always on."""
-    targets: dict[str, dict[str, Path]] = {"fallback": GLOBAL_TOOL_DIRS["fallback"]}
-    for tool_id, dirs in GLOBAL_TOOL_DIRS.items():
-        if tool_id == "fallback":
-            continue
-        # claude-desktop shares claude-code's dir — both flip the same target.
-        if tool_id == "claude-code" and ("claude-code" in tools or "claude-desktop" in tools):
-            targets[tool_id] = dirs
-        elif tool_id in tools:
-            targets[tool_id] = dirs
-    return targets
-
-
-def install_global(package_root: Path, tools: set[str], force: bool) -> int:
-    """S13/S14: ship kernel rules + curated skills to user-scope dirs."""
-    import shutil
-    manifest = _load_global_manifest(package_root)
-    targets = _global_targets(tools)
-
-    if not QUIET:
-        info(f"Global install — surfaces: {', '.join(sorted(targets))}")
-        info(f"  rules:  {len(manifest['kernel_rules'])}")
-        info(f"  skills: {len(manifest['top_skills'])}")
-
-    for surface, dirs in targets.items():
-        for kind in ("rules", "skills"):
-            dirs[kind].mkdir(parents=True, exist_ok=True)
-        # Rules: copy each kernel rule (file).
-        for rule_id in manifest["kernel_rules"]:
-            src = _resolve_rule_source(package_root, rule_id)
-            if src is None:
-                warn(f"global: rule '{rule_id}' missing in package — skipped")
-                continue
-            dst = dirs["rules"] / f"{rule_id}.md"
-            if dst.exists() and not force and dst.read_bytes() == src.read_bytes():
-                continue
-            dst.write_bytes(src.read_bytes())
-        # Skills: copy each curated skill (directory).
-        for entry in manifest["top_skills"]:
-            if surface != "fallback" and surface not in entry.get("surfaces", []):
-                continue
-            src_dir = _resolve_skill_source(package_root, entry["id"])
-            if src_dir is None:
-                warn(f"global: skill '{entry['id']}' missing in package — skipped")
-                continue
-            dst_dir = dirs["skills"] / entry["id"]
-            if dst_dir.exists() and not force:
-                shutil.rmtree(dst_dir)
-            shutil.copytree(src_dir, dst_dir)
-        if not QUIET:
-            success(f"  {surface}: {dirs['rules']}, {dirs['skills']}")
-    return 0
-
-
-def uninstall_global(tools: set[str]) -> int:
-    """S15: remove the event4u/ namespace dir from each enabled surface."""
-    import shutil
-    targets = _global_targets(tools)
-    removed: list[str] = []
-    # Collect ancestor dirs named GLOBAL_NAMESPACE so we can drop the
-    # shared parent (e.g. cursor/windsurf put rules + skills as siblings
-    # under one event4u/ dir; removing both leaves a stranded namespace
-    # dir). We never delete anything not literally named event4u/.
-    namespace_parents: set[Path] = set()
-    for surface, dirs in targets.items():
-        for kind in ("rules", "skills"):
-            d = dirs[kind]
-            if d.exists():
-                shutil.rmtree(d)
-                removed.append(str(d))
-            for ancestor in d.parents:
-                if ancestor.name == GLOBAL_NAMESPACE:
-                    namespace_parents.add(ancestor)
-    for parent in namespace_parents:
-        if parent.is_dir() and not any(parent.iterdir()):
-            parent.rmdir()
-            removed.append(str(parent))
-    if not QUIET:
-        if removed:
-            for r in removed:
-                success(f"removed {r}")
-        else:
-            skip("global uninstall: nothing to remove")
-    return 0
-
+# The `--global` symlink-install scheme was retired under the npx-only
+# distribution model. The curated manifest (templates/global-install-manifest.yml)
+# and its helpers/dispatch are gone. The replacement is `npx @event4u/agent-config`
+# resolving the runtime per-invocation; the project's `.agent-settings.yml`
+# carries the version pin.
 
 # --- Argument parsing ---
 
@@ -1457,21 +1282,6 @@ def parse_options(argv: list[str]) -> argparse.Namespace:
         action="store_true",
         help="skip the post-install hook smoke test (default: dry-fire dispatch:hook against every installed bridge)",
     )
-    parser.add_argument(
-        "--global",
-        dest="global_install",
-        action="store_true",
-        help=(
-            "Phase-3 mode: ship kernel rules + curated top-N skills to user-scope "
-            "dirs (~/.claude/, ~/.cursor/, ~/.codeium/windsurf/, ~/.config/agent-config/) "
-            "so the agent has them in every project. Curation: templates/global-install-manifest.yml."
-        ),
-    )
-    parser.add_argument(
-        "--uninstall",
-        action="store_true",
-        help="With --global: remove the event4u/ namespace dir from each enabled surface.",
-    )
     return parser.parse_args(argv)
 
 
@@ -1539,17 +1349,6 @@ def main(argv: list[str]) -> int:
         info(f"Profile:  {opts.profile}")
         print()
 
-    # --global: short-circuits the project-bridge path. Ships kernel rules
-    # + curated skills to user-scope dirs and exits. --uninstall pairs
-    # with --global to remove the namespace dir.
-    if opts.global_install:
-        tools = _parse_tools(opts.tools)
-        if opts.uninstall:
-            return uninstall_global(tools)
-        return install_global(package_root, tools, opts.force)
-    if opts.uninstall:
-        fail("--uninstall is only valid combined with --global")
-
     ensure_agent_settings(project_root, package_root, opts.profile, opts.force)
 
     tools = _parse_tools(opts.tools)

package/bin/install.php

@@ -1,45 +0,0 @@
-#!/usr/bin/env php
-<?php
-
-declare(strict_types=1);
-
-/**
- * Agent Config — Project Installer (Composer wrapper)
- *
- * Thin wrapper that delegates to scripts/install, the primary orchestrator.
- * The orchestrator chains payload sync (install.sh) and bridge generation
- * (install.py); this wrapper only exists so that PHP/Composer users can
- * invoke it via `php vendor/bin/install.php` without knowing about bash.
- *
- * Usage:
- *   php vendor/bin/install.php                              # full install (every tool)
- *   php vendor/bin/install.php --profile=balanced           # pick cost profile
- *   php vendor/bin/install.php --tools=claude-code,cursor   # only those two surfaces
- *   php vendor/bin/install.php --tools=cursor --yes         # CI-friendly, single tool
- *   php vendor/bin/install.php --list-tools                 # show valid tool IDs
- *   php vendor/bin/install.php --force                      # overwrite existing bridges
- *   php vendor/bin/install.php --skip-bridges               # payload only
- *   php vendor/bin/install.php --skip-sync                  # bridges only
- *   php vendor/bin/install.php --help                       # full option list
- *
- * Valid --tools IDs (default: all):
- *   claude-code, claude-desktop, cursor, windsurf, cline,
- *   gemini-cli, copilot, augment, aider, codex, all
- */
-
-$packageRoot = dirname(__DIR__);
-$installer = $packageRoot . '/scripts/install';
-
-if (! file_exists($installer)) {
-    fwrite(STDERR, "  ❌  Installer script not found: {$installer}\n");
-    exit(1);
-}
-
-// Forward all CLI arguments to the bash orchestrator. Drop $argv[0] (this script).
-$forwarded = array_slice($argv, 1);
-$command = array_merge(['bash', $installer], $forwarded);
-$escaped = array_map('escapeshellarg', $command);
-
-$exitCode = 0;
-passthru(implode(' ', $escaped), $exitCode);
-exit($exitCode);

package/composer.json

@@ -1,33 +0,0 @@
-{
-    "name": "event4u/agent-config",
-    "description": "Shared agent configuration — skills, rules, commands, guidelines, and templates for AI coding tools",
-    "type": "library",
-    "license": "MIT",
-    "require": {
-        "php": ">=8.0"
-    },
-    "suggest": {
-        "@event4u/agent-memory": "Optional MCP-based memory backend (npm: @event4u/agent-memory ^1.1.0). Adds persistent agent learnings across sessions. Install with `npm install --save-dev @event4u/agent-memory` if your team wants the memory layer; otherwise agent-config falls back to file-based memory."
-    },
-    "bin": [
-        "bin/install.php",
-        "scripts/agent-config"
-    ],
-    "archive": {
-        "exclude": [
-            "/.agent-src.uncompressed",
-            "/.compression-hashes.json",
-            "/agents",
-            "/tests",
-
-            "/Taskfile.yml",
-            "/.github",
-            "/.idea",
-            "/.claude",
-            "/.cursor",
-            "/.clinerules",
-            "/.windsurfrules",
-            "/GEMINI.md"
-        ]
-    }
-}

package/scripts/postinstall.sh

@@ -1,76 +0,0 @@
-#!/usr/bin/env bash
-# postinstall.sh — npm postinstall wrapper for scripts/install
-#
-# Runs the orchestrator and decides on an exit strategy that is honest
-# without breaking `npm install` for unrelated reasons:
-#
-#   - Success        → exit 0, silent.
-#   - Soft failure   → exit 0, print actionable hint. The orchestrator
-#                      already handles environment issues (no python3)
-#                      internally and continues with the payload sync.
-#   - Hard failure   → print a loud error block, print the captured output,
-#                      exit 0. `npm install` keeps working; the developer
-#                      sees exactly what failed and the command to retry.
-#
-# Delegates to scripts/install (the orchestrator). Never calls install.sh
-# or install.py directly.
-
-set -u
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-INSTALLER="$SCRIPT_DIR/install"
-
-if [[ ! -f "$INSTALLER" ]]; then
-    # Nothing to do — shipping this wrapper without the installer would be a
-    # packaging bug, not a consumer problem.
-    echo "agent-config postinstall: $INSTALLER missing, skipping." >&2
-    exit 0
-fi
-
-LOG="$(mktemp -t agent-config-postinstall.XXXXXX.log 2>/dev/null || mktemp)"
-trap 'rm -f "$LOG"' EXIT
-
-bash "$INSTALLER" --quiet >"$LOG" 2>&1
-CODE=$?
-if [[ $CODE -eq 0 ]]; then
-    # Optional companion: @event4u/agent-memory. Suggest it once, only if
-    # the consumer hasn't already installed it (locally or on PATH). The
-    # hint is purely informational; agent-config falls back to file-based
-    # memory when the backend is absent.
-    if ! command -v memory >/dev/null 2>&1 \
-       && ! command -v agent-memory >/dev/null 2>&1 \
-       && [[ ! -d "$SCRIPT_DIR/../../@event4u/agent-memory" ]]; then
-        cat >&2 <<'HINT'
-💡 agent-config tip: install @event4u/agent-memory for persistent agent
-   learnings across sessions (optional, dev-only):
-
-       npm install --save-dev @event4u/agent-memory
-
-   Skip if you don't need it — agent-config falls back to file-based memory.
-HINT
-    fi
-    exit 0
-fi
-
-cat >&2 <<EOF
-
-────────────────────────────────────────────────────────────────
-⚠️  agent-config: postinstall failed (exit $CODE)
-────────────────────────────────────────────────────────────────
-Output:
-$(sed 's/^/    /' "$LOG")
-
-The rest of \`npm install\` continues — but agent-config was NOT
-installed. Re-run manually to see the full trace:
-
-    bash node_modules/@event4u/agent-config/scripts/install
-
-If this keeps happening, please open an issue:
-    https://github.com/event4u-app/agent-config/issues
-────────────────────────────────────────────────────────────────
-
-EOF
-
-# Exit 0 so npm install doesn't break for unrelated consumer workflows.
-# The loud block above makes the failure visible — unlike `|| true`.
-exit 0

package/templates/global-install-manifest.yml

@@ -1,91 +0,0 @@
-# Global Install Manifest — road-to-simplicity-and-everywhere Phase 3
-#
-# Curated subset of rules + skills shipped to user-scope directories
-# (~/.claude/, ~/.cursor/, ~/.codeium/windsurf/, ~/.config/agent-config/)
-# by `scripts/install --global`. Without curation we'd dump 200+ skills
-# into user dirs — that is the failure mode anthropics/claude-code#53950
-# flagged.
-#
-# Schema (v1):
-#   kernel_rules: list[str]   — rule IDs from router.json's `kernel`.
-#                               Always shipped to every enabled surface.
-#   top_skills:   list[entry] — curated universal skills.
-#     entry.id        : skill directory under .agent-src/skills/
-#     entry.surfaces  : subset of [claude-code, cursor, windsurf, fallback]
-#                       — fallback is always written.
-#
-# Source: kernel_rules mirrors router.json; top_skills is a 15-pick
-# distilled from .agent-src/skills/ — universal coding-loop helpers
-# (work, commit, PR, review, quality, analyze, handoff). Per-stack
-# skills (laravel, react-shadcn-ui, php-coder) stay project-local.
-#
-# Per-tool target paths (handled by scripts/install.py global subcommand):
-#   claude-code  : ~/.claude/skills/event4u/<id>/SKILL.md
-#                  ~/.claude/rules/event4u/<rule>.md
-#   cursor       : ~/.cursor/rules/imported/event4u/<rule>.mdc
-#                  ~/.cursor/skills/event4u/<id>/  (skills surface; mirrors agent-os)
-#   windsurf     : ~/.codeium/windsurf/global_workflows/event4u/<id>.md
-#   fallback     : ~/.config/agent-config/{rules,skills}/event4u/
-#
-# All writes are namespaced under `event4u/` so `--global --uninstall`
-# can `rm -rf` the namespace dir without touching user-added files.
-
-schema_version: 1
-
-# Always-loaded rules — mirrors router.json's `kernel` array.
-# Symlinked / copied into every enabled surface's rules dir.
-kernel_rules:
-  - agent-authority
-  - ask-when-uncertain
-  - commit-policy
-  - direct-answers
-  - language-and-tone
-  - no-cheap-questions
-  - non-destructive-by-default
-  - scope-control
-  - verify-before-complete
-
-# Top-N curated skills. The 15 picks below are coding-loop universals:
-# the agent reaches for them on almost any task, regardless of stack.
-# Per-stack and domain skills are deliberately omitted — they belong in
-# project-local installs where the stack signal is unambiguous.
-top_skills:
-  # --- Workflow drivers (most-touched commands) ---
-  - id: work
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: commit
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: create-pr
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: review-changes
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: quality-fix
-    surfaces: [claude-code, cursor, windsurf, fallback]
-
-  # --- Session / context management ---
-  - id: agent-handoff
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: agent-status
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: project-analyze
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: project-health
-    surfaces: [claude-code, cursor, windsurf, fallback]
-
-  # --- Investigation / debugging ---
-  - id: bug-investigate
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: bug-fix
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: systematic-debugging
-    surfaces: [claude-code, cursor, windsurf, fallback]
-
-  # --- Ticket / PR drivers ---
-  - id: implement-ticket
-    surfaces: [claude-code, cursor, windsurf, fallback]
-  - id: prepare-for-review
-    surfaces: [claude-code, cursor, windsurf, fallback]
-
-  # --- Onboarding (single command, surfaces a project) ---
-  - id: onboard
-    surfaces: [claude-code, cursor, windsurf, fallback]