@event4u/agent-config 1.29.0 → 1.32.0

package/scripts/release.py

@@ -319,8 +319,15 @@ def render_changelog_entry(
     prev: str | None,
     commits: list[Commit],
     today: str,
+    *,
+    test_trend_line: str | None = None,
 ) -> tuple[str, str]:
-    """Return (heading-aware full entry, body-only for GitHub Release notes)."""
+    """Return (heading-aware full entry, body-only for GitHub Release notes).
+
+    ``test_trend_line`` — optional pre-computed ``Tests: N (+M …)`` footer
+    (road-to-feedback-followups P3.2). Computed by the caller so tests
+    don't trigger a recursive pytest collection.
+    """
     if prev:
         heading = (
             f"## [{version}](https://github.com/{REPO_SLUG}/compare/"
@@ -364,6 +371,12 @@ def render_changelog_entry(
         for c in other:
             body_lines.append(_changelog_line(c))
 
+    # Test-count trend footer (road-to-feedback-followups P3.2). Silent
+    # on errors — never a release blocker.
+    if test_trend_line:
+        body_lines.append("")
+        body_lines.append(test_trend_line)
+
     body = "\n".join(body_lines).lstrip("\n")
     full = heading + "\n\n" + body + "\n"
     return full, body
@@ -376,6 +389,65 @@ def _changelog_line(c: Commit) -> str:
     return f"* {scope}{c.subject} ([{short}]({link}))"
 
 
+# ─── test-count trend (road-to-feedback-followups P3.2) ───────────────────────
+
+_TEST_COUNT_LINE_RE = re.compile(r"^Tests:\s+(\d+)", re.MULTILINE)
+_PYTEST_COLLECTED_RE = re.compile(r"^(\d+)\s+tests?\s+collected", re.MULTILINE)
+
+
+def _count_tests_current() -> int | None:
+    """Return the count from `pytest --collect-only -q` on the current
+    tree. Returns None when pytest isn't available or collection fails —
+    the trend line is informational, never a release blocker.
+    """
+    try:
+        result = subprocess.run(
+            ["python3", "-m", "pytest", "--collect-only", "-q"],
+            cwd=str(REPO_ROOT),
+            capture_output=True,
+            text=True,
+            timeout=120,
+        )
+    except (FileNotFoundError, subprocess.TimeoutExpired):
+        return None
+    if result.returncode != 0:
+        return None
+    match = _PYTEST_COLLECTED_RE.search(result.stdout)
+    return int(match.group(1)) if match else None
+
+
+def _previous_test_count_from_changelog(prev_tag: str | None) -> int | None:
+    """Read CHANGELOG.md and return the most recent ``Tests: N`` footer
+    under the ``prev_tag`` heading, or None when not found.
+    """
+    if not prev_tag or not CHANGELOG.exists():
+        return None
+    text = CHANGELOG.read_text(encoding="utf-8")
+    heading_re = re.compile(rf"^##\s+\[?{re.escape(prev_tag)}\b", re.MULTILINE)
+    m = heading_re.search(text)
+    if not m:
+        return None
+    next_heading = re.search(r"^##\s+\[?\d+\.\d+\.\d+", text[m.end():], re.MULTILINE)
+    section = text[m.end(): m.end() + (next_heading.start() if next_heading else len(text))]
+    count_match = _TEST_COUNT_LINE_RE.search(section)
+    return int(count_match.group(1)) if count_match else None
+
+
+def _render_test_trend_line(prev_tag: str | None) -> str | None:
+    """Return the ``Tests: N (+M since X.Y.Z)`` footer line, or None when
+    the current count cannot be determined. Silent on collection errors.
+    """
+    current = _count_tests_current()
+    if current is None:
+        return None
+    previous = _previous_test_count_from_changelog(prev_tag)
+    if previous is None or not prev_tag:
+        return f"Tests: {current}"
+    delta = current - previous
+    sign = "+" if delta >= 0 else ""
+    return f"Tests: {current} ({sign}{delta} since {prev_tag})"
+
+
 def prepend_changelog(path: Path, entry: str) -> None:
     """Insert `entry` directly above the most recent `## [` heading."""
     text = path.read_text(encoding="utf-8")
@@ -787,7 +859,10 @@ def main(argv: list[str] | None = None) -> int:
         preflight(target, resume=args.resume)
 
     today = _date.today().isoformat()
-    full, body = render_changelog_entry(target, prev, commits, today)
+    test_trend_line = _render_test_trend_line(prev)
+    full, body = render_changelog_entry(
+        target, prev, commits, today, test_trend_line=test_trend_line
+    )
     plan = Plan(
         current=current,
         target=target,

package/scripts/skill_linter.py

@@ -1291,12 +1291,19 @@ def lint_command(path: Path, text: str) -> LintResult:
     if not H1_PATTERN.search(text):
         issues.append(Issue("error", "missing_h1", "Command is missing an H1 heading (# Title)"))
 
-    # Must have at least one ## section with steps
+    # Must have at least one ## section with steps. Cluster-head and
+    # router-style commands (frontmatter cluster:/routes_to: or ≥ 3 .md
+    # links) are exempt — they delegate procedure to sub-commands or
+    # skills (road-to-feedback-followups P2.1).
     sections = extract_sections(text)
     has_steps = any(s.lower().startswith("step") for s in sections)
-    has_numbered = bool(re.search(r"^###?\s+\d+\.\s+", text, re.MULTILINE))
+    # Accept both ``## 1.`` / ``### 1.`` numbered headings AND
+    # ``### Step N`` / ``## Step N`` step-prefixed sub-headings.
+    has_numbered = bool(re.search(r"^###?\s+(?:\d+\.|step\s+\d+)\s+", text, re.MULTILINE | re.IGNORECASE))
     if not has_steps and not has_numbered:
-        issues.append(Issue("warning", "no_steps", "Command has no Steps section or numbered sub-headings"))
+        delegated = _command_delegation_signal(text, frontmatter)
+        if not delegated:
+            issues.append(Issue("warning", "no_steps", "Command has no Steps section or numbered sub-headings"))
 
     # --- Size check (docs/contracts/linter-structural-model.md) ---
     # Structural-density gate replaces sub-section + code-block heuristic

package/.agent-src/commands/agents/cleanup.md

@@ -1,194 +0,0 @@
----
-name: agents:cleanup
-cluster: agents
-sub: cleanup
-skills: [agent-docs-writing]
-description: Execute cleanup actions from an agents-audit — move, merge, delete, and update agent docs
-disable-model-invocation: true
-suggestion:
-  eligible: false
-  rationale: "Consumes prior audit output; only meaningful right after /agents-audit."
----
-
-# /agents cleanup
-## Instructions
-
-### 1. Check for audit roadmap
-
-Check the audit-roadmap path created by `/agents audit`:
-
-```
-agents/roadmaps/agents-cleanup.md
-```
-
-- If it exists → load it and show the phases.
-- If not → ask:
-
-```
-⚠️  No audit roadmap found.
-
-1. 🔍 Run /agents-audit first
-2. 💬 I know what to do — start directly
-```
-
-**Recommendation: 1 — Run /agents-audit first** — without an audit roadmap, the cleanup is shooting in the dark. Caveat: pick 2 only if you already know exactly which docs to touch.
-
-If option 2, ask what to clean up.
-
-### 2. Show action plan
-
-If a roadmap exists, display the phases:
-
-```
-═══════════════════════════════════════════════
-  🧹 AGENTS CLEANUP
-═══════════════════════════════════════════════
-
-Phase 1: Critical fixes ({count} actions)
-  - [ ] {action}
-  - [ ] {action}
-
-Phase 2: Structural cleanup ({count} actions)
-  - [ ] {action}
-  - [ ] {action}
-
-Phase 3: Fill gaps ({count} actions)
-  - [ ] {action}
-
-Phase 4: Cleanup ({count} actions)
-  - [ ] {action}
-
-═══════════════════════════════════════════════
-
-Which phase to work on?
-
-1. Start Phase 1 (recommended — top to bottom)
-2. Choose a specific phase
-3. Choose a specific action
-```
-
-### 3. Execute actions
-
-For each action, follow the appropriate workflow:
-
-**Move file:**
-```
-📁 Move: {source} → {target}
-Reason: {why}
-
-Confirm? (y/n)
-```
-- Move the file.
-- Update all references in other docs that link to the old path.
-- Check `.augment/skills/` and `.augment/commands/` for references.
-
-**Merge files:**
-```
-🔗 Merge:
-  {file1} + {file2} → {target}
-Reason: {why — what overlaps}
-
-Confirm? (y/n)
-```
-- Read both files.
-- Show the proposed merged content.
-- Create the merged file, delete the originals.
-- Update references.
-
-**Delete file:**
-```
-🗑️  Delete: {file}
-Reason: {why — what's obsolete}
-
-Content (preview):
-  {first 5 lines}
-
-Confirm? (y/n)
-```
-- Delete the file.
-- Check for and remove references in other docs.
-
-**Update file:**
-```
-✏️  Update: {file}
-Reason: {what's outdated}
-
-Changes:
-  - {reference to deleted class} → remove
-  - {outdated section} → update
-  - {missing info} → add
-
-Confirm? (y/n)
-```
-- Read the file.
-- Make the changes.
-- Show a summary of what changed.
-
-**Create context:**
-```
-📄 Create context: {area}
-Reason: {why it's needed}
-
-> 1. Yes — start /context-create
-> 2. Skip
-```
-- Transition to `/context-create` with the area pre-selected.
-
-### 4. Update roadmap progress
-
-After each action:
-- Mark the step as `[x]` in the roadmap file.
-- Show progress:
-
-```
-✅  Action complete: {description}
-
-Progress Phase {n}: [{completed}/{total}]
-██████████░░░░░░ 60%
-```
-
-Per `verbosity.routine_confirmations` (default `false`):
-
-- `false` → continue to the next action silently. User can interrupt
-  any time; "continue" is dominant once cleanup is in flight.
-- `true` → ask:
-  ```
-  > 1. Continue with next action
-  > 2. Stop here
-  ```
-
-### 5. Summary (verbosity-gated)
-
-After completing a phase or all actions, read
-`verbosity.post_action_reports` from `.agent-settings.yml` (default
-`minimal`).
-
-- `off` → emit nothing on success; surface errors only.
-- `minimal` (default) → one line:
-  `→ Cleanup: {moved} moved · {deleted} deleted · {updated} updated · {remaining} remaining`.
-- `full` → multi-line block:
-
-  ```
-  ═══════════════════════════════════════════════
-    ✅  CLEANUP SUMMARY
-  ═══════════════════════════════════════════════
-
-    📁 Moved:     {count} files
-    🔗 Merged:    {count} files
-    🗑️  Deleted:   {count} files
-    ✏️  Updated:   {count} files
-    📄 Created:   {count} contexts
-
-    Remaining:        {count} actions in {phases} phases
-
-  ═══════════════════════════════════════════════
-  ```
-
-### Rules
-
-- **Do NOT commit or push.**
-- **Always confirm before each destructive action** (delete, merge, move).
-- **Always update references** when moving or renaming files.
-- **Update the roadmap** after each completed action.
-- **Show file content** before deleting — the user should see what's being removed.
-- **Check `.augment/` references too** — skills and commands may link to agents docs.

package/.agent-src/commands/agents/prepare.md

@@ -1,141 +0,0 @@
----
-name: agents:prepare
-cluster: agents
-sub: prepare
-skills: [agent-docs-writing]
-description: Scaffold the agents/ directory structure with all required subdirectories and .gitkeep files
-disable-model-invocation: true
-suggestion:
-  eligible: false
-  rationale: "One-shot project scaffolding; only run during initial setup."
----
-
-# /agents prepare
-## Instructions
-
-### 1. Check project root
-
-Verify you're in the project root (look for `composer.json`, `artisan`, or `package.json`).
-
-### 2. Create directory structure
-
-Create the following directories if they don't exist, with a `.gitkeep` in each **empty** directory:
-
-```
-agents/
-├── roadmaps/
-│   └── .gitkeep
-├── features/
-│   └── .gitkeep
-└── contexts/
-    └── .gitkeep
-```
-
-Also create the guidelines directory in `.augment/` if it doesn't exist:
-
-```
-.augment/guidelines/
-└── php/
-    └── .gitkeep
-```
-
-**Rules:**
-- Only add `.gitkeep` to directories that have **no other files** in them.
-- Do NOT overwrite existing files.
-- Do NOT create directories that already exist with content.
-
-### 3. Check for module support
-
-Check if the project has a module system (`app/Modules/` directory):
-
-- If `app/Modules/` exists → list all modules.
-- For each module, create the agent directories if they don't exist:
-
-```
-app/Modules/{Module}/agents/
-├── roadmaps/
-│   └── .gitkeep
-├── features/
-│   └── .gitkeep
-├── contexts/
-│   └── .gitkeep
-```
-
-### 4. Verify templates exist
-
-Check that `.augment/templates/` contains the required templates:
-
-```
-.augment/templates/
-├── features.md    # Feature plan template
-├── roadmaps.md    # Roadmap template
-└── contexts.md    # Context document template
-```
-
-If any template is missing, warn:
-
-```
-⚠️  Missing templates in .augment/templates/:
-  - {missing-file}
-
-These templates are required by the Feature/Roadmap commands.
-```
-
-### 5. Clean up old templates
-
-Check for **old template files** in the agents directories and offer to remove them:
-
-```
-agents/features/template.md
-agents/roadmaps/template.md
-app/Modules/*/agents/roadmaps/template.md
-app/Modules/*/agents/features/template.md
-```
-
-If any are found:
-
-```
-ℹ️  Old template files found (now in .augment/templates/):
-
-  - agents/features/template.md
-  - agents/roadmaps/template.md
-
-Delete these? (y/n)
-```
-
-If yes → delete them.
-
-### 6. Show summary
-
-```
-═══════════════════════════════════════════════
-  ✅  AGENTS-VERZEICHNIS VORBEREITET
-═══════════════════════════════════════════════
-
-📁 Projekt-Root:
-  agents/roadmaps/     {✅ existiert | 🆕 erstellt}
-  agents/features/     {✅ existiert | 🆕 erstellt}
-  .augment/guidelines/ {✅ existiert | 🆕 erstellt}
-
-📁 Templates:
-  .augment/templates/features.md   {✅ | ⚠️ fehlt}
-  .augment/templates/roadmaps.md   {✅ | ⚠️ fehlt}
-
-{If modules exist:}
-📁 Module:
-  {Module}/agents/roadmaps/   {✅ | 🆕}
-  {Module}/agents/features/   {✅ | 🆕}
-  ...
-
-{If old templates were cleaned up:}
-🗑️  Old templates removed: {count}
-
-═══════════════════════════════════════════════
-```
-
-### Rules
-
-- **Do NOT commit or push.**
-- **Do NOT overwrite existing files.**
-- **Do NOT create `.gitkeep` in directories that already have files.**
-- **Always ask before deleting** old template files.