@event4u/agent-config 2.25.0 → 3.0.0

package/.agent-src/templates/scripts/work_engine/state.py

@@ -49,6 +49,15 @@ The wire format adds five envelope fields on top of the legacy
   ``verdict`` (success / blocked / partial), and the
   ``integration_confirmed`` flag the user sets after reviewing the
   integration evidence.
+- ``halts`` — append-only log of :class:`HookHalt` events the engine
+  persisted into state. Populated by :func:`emitters._emit_halt` when
+  it runs against an already-persisted state file (the P3 branch table
+  is preserved: fresh-run halts before the first ``_save`` still leave
+  no state on disk). The ``explain_last`` trace builder reads the
+  tail entry to fill the ``trace.halt`` slot. Each entry is
+  ``{reason, step, surface, timestamp}``. The list defaults to empty
+  and is omitted from older state files via ``from_dict``'s tolerant
+  reader — no schema-version bump.
 
 All other fields keep their v0 names so the dispatcher can read the
 legacy slice unchanged once Phase 3 wires the steps over.
@@ -153,6 +162,7 @@ class WorkState:
     ui_polish: dict[str, Any] | None = None
     contract: dict[str, Any] | None = None
     stitch: dict[str, Any] | None = None
+    halts: list[dict[str, Any]] = field(default_factory=list)
     version: int = SCHEMA_VERSION
     persona: str = "senior-engineer"
     memory: list[dict[str, Any]] = field(default_factory=list)
@@ -187,6 +197,7 @@ def to_dict(state: WorkState) -> dict[str, Any]:
     _validate_ui_polish(state.ui_polish)
     _validate_contract(state.contract)
     _validate_stitch(state.stitch)
+    _validate_halts(state.halts)
     return {
         "version": state.version,
         "input": {"kind": state.input.kind, "data": state.input.data},
@@ -199,6 +210,7 @@ def to_dict(state: WorkState) -> dict[str, Any]:
         "ui_polish": state.ui_polish,
         "contract": state.contract,
         "stitch": state.stitch,
+        "halts": list(state.halts),
         "persona": state.persona,
         "memory": state.memory,
         "plan": state.plan,
@@ -269,6 +281,9 @@ def from_dict(payload: Any) -> WorkState:
     stitch = payload.get("stitch")
     _validate_stitch(stitch)
 
+    halts = payload.get("halts", [])
+    _validate_halts(halts)
+
     return WorkState(
         input=Input(kind=kind, data=data),
         intent=payload.get("intent", DEFAULT_INTENT),
@@ -280,6 +295,7 @@ def from_dict(payload: Any) -> WorkState:
         ui_polish=dict(ui_polish) if isinstance(ui_polish, dict) else None,
         contract=dict(contract) if isinstance(contract, dict) else None,
         stitch=dict(stitch) if isinstance(stitch, dict) else None,
+        halts=list(halts) if isinstance(halts, list) else [],
         version=version,
         persona=payload.get("persona", "senior-engineer"),
         memory=list(payload.get("memory", [])),
@@ -625,6 +641,43 @@ def _validate_stitch(stitch: Any) -> None:
         )
 
 
+def _validate_halts(halts: Any) -> None:
+    """Reject malformed ``halts`` envelopes; tolerate ``[]``.
+
+    Each entry must be a dict with at minimum ``reason`` (str) and
+    ``surface`` (list of str). ``step`` and ``timestamp`` are optional
+    metadata appended by ``emitters._emit_halt``. The schema enforces
+    only shape — the explain renderer is responsible for fallback
+    rendering when optional fields are absent.
+    """
+    if not isinstance(halts, list):
+        raise SchemaError(
+            f"state.halts must be a list; got {type(halts).__name__}",
+        )
+    for idx, entry in enumerate(halts):
+        if not isinstance(entry, dict):
+            raise SchemaError(
+                f"state.halts[{idx}] must be a JSON object; "
+                f"got {type(entry).__name__}",
+            )
+        reason = entry.get("reason")
+        if not isinstance(reason, str) or not reason:
+            raise SchemaError(
+                f"state.halts[{idx}].reason must be a non-empty string",
+            )
+        surface = entry.get("surface", [])
+        if not isinstance(surface, list):
+            raise SchemaError(
+                f"state.halts[{idx}].surface must be a list; "
+                f"got {type(surface).__name__}",
+            )
+        for j, line in enumerate(surface):
+            if not isinstance(line, str):
+                raise SchemaError(
+                    f"state.halts[{idx}].surface[{j}] must be a string",
+                )
+
+
 __all__ = [
     "DEFAULT_DIRECTIVE_SET",
     "DEFAULT_INTENT",

package/.agent-src/templates/skill-archive-note.md

@@ -1,6 +1,6 @@
 # Skill Archive Note Template
 
-> Template for `agents/archived-skills/<slug>.md`. Created during
+> Template for `agents/evidence/archived-skills/<slug>.md`. Created during
 > `step-2-skill-inventory-rationalization.md`
 > Phase 4 execution. Every skill removed from
 > `.agent-src.uncompressed/skills/` MUST have a matching archive note
@@ -8,7 +8,7 @@
 
 ## Instructions
 
-1. Create the file: `agents/archived-skills/{slug}.md`.
+1. Create the file: `agents/evidence/archived-skills/{slug}.md`.
 2. Copy the template body below and fill every required field.
 3. Commit alongside the SKILL.md removal in the same PR (the linter
    refuses to pass if the pair drifts).
@@ -88,12 +88,12 @@ references the removed slug without pointing at this note.}
 
 `scripts/lint_archived_skills.py` enforces:
 
-1. Every file under `agents/archived-skills/*.md` (except the README)
+1. Every file under `agents/evidence/archived-skills/*.md` (except the README)
    has a frontmatter block with the six required fields.
 2. `reason` is one of the four allowed values.
 3. When `reason ∈ {merged, superseded}`, the `replacement` slug exists
    under `.agent-src.uncompressed/skills/`.
-4. Every slug under `agents/archived-skills/` is *absent* from
+4. Every slug under `agents/evidence/archived-skills/` is *absent* from
    `.agent-src.uncompressed/skills/` (no zombies).
 5. No SKILL.md present under `.agent-src.uncompressed/skills/`
    references an archived slug as a router target.

package/.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "Shared agent configuration \u2014 skills for AI coding tools (Claude Code, Augment, Cursor, Cline, Windsurf, Gemini CLI).",
-    "version": "2.25.0",
+    "version": "3.0.0",
     "keywords": [
       "agent-config",
       "skills",
@@ -139,7 +139,6 @@
         "./.claude/skills/discovery-interview",
         "./.claude/skills/doc-coauthoring",
         "./.claude/skills/docker",
-        "./.claude/skills/dto-creator",
         "./.claude/skills/e2e-heal",
         "./.claude/skills/e2e-plan",
         "./.claude/skills/editorial-calendar",
@@ -199,15 +198,19 @@
         "./.claude/skills/judge-steps",
         "./.claude/skills/judge-test-coverage",
         "./.claude/skills/laravel",
+        "./.claude/skills/laravel-api-endpoint",
+        "./.claude/skills/laravel-dto",
         "./.claude/skills/laravel-horizon",
         "./.claude/skills/laravel-mail",
         "./.claude/skills/laravel-middleware",
+        "./.claude/skills/laravel-migration",
         "./.claude/skills/laravel-notifications",
         "./.claude/skills/laravel-pennant",
         "./.claude/skills/laravel-pulse",
         "./.claude/skills/laravel-reverb",
         "./.claude/skills/laravel-scheduling",
         "./.claude/skills/laravel-validation",
+        "./.claude/skills/laravel-websocket",
         "./.claude/skills/launch-readiness",
         "./.claude/skills/learning-to-rule-or-skill",
         "./.claude/skills/lint-skills",
@@ -230,7 +233,6 @@
         "./.claude/skills/merge-conflicts",
         "./.claude/skills/messaging-architecture",
         "./.claude/skills/migration-architect",
-        "./.claude/skills/migration-creator",
         "./.claude/skills/mobile-e2e-strategy",
         "./.claude/skills/mode",
         "./.claude/skills/module",
@@ -241,7 +243,6 @@
         "./.claude/skills/multi-tenancy",
         "./.claude/skills/nextjs-patterns",
         "./.claude/skills/okr-tree-modeling",
-        "./.claude/skills/onboard",
         "./.claude/skills/onboarding-design",
         "./.claude/skills/onboarding-program",
         "./.claude/skills/one-on-one-cadence",
@@ -324,6 +325,7 @@
         "./.claude/skills/roadmap-writing",
         "./.claude/skills/rtk-output-filtering",
         "./.claude/skills/rule-compliance-audit",
+        "./.claude/skills/rule-refactor",
         "./.claude/skills/rule-writing",
         "./.claude/skills/runway-cognition",
         "./.claude/skills/scenario-modeling",
@@ -380,7 +382,6 @@
         "./.claude/skills/vision-articulation",
         "./.claude/skills/voc-extract",
         "./.claude/skills/voice-and-tone-design",
-        "./.claude/skills/websocket",
         "./.claude/skills/work"
       ]
     }

package/AGENTS.md

@@ -18,10 +18,11 @@ task ci                # full pipeline — green before PR
 
 - **Package self-orientation** (beta) — identity, four-wing cognition map, repo layout, tech stack, key-rules table, telemetry, command-suggester: [`docs/contracts/package-self-orientation.md`](docs/contracts/package-self-orientation.md).
 - **Kernel + Router** (beta) — 9 always-loaded Iron-Law rules, tier-1 / tier-2 routing, cost profiles, per-rule char caps enforced by `task lint-rule-budget`: [`kernel-membership`](docs/contracts/kernel-membership.md) + [`rule-router`](docs/contracts/rule-router.md).
-- **Content pipelines** — A [source projection](docs/architecture/source-projection.md), B [Augment projection](docs/architecture/augment-projection.md), C [multi-tool projection](docs/architecture/multi-tool-projection.md), D [Claude.ai bundle](docs/architecture/claude-bundle.md). Index: [`docs/architecture.md`](docs/architecture.md).
-- **Editing this repo** — Iron-Law rules (portability, source-of-truth, skill-quality) + Thin-Root contract: [`augment-portability`](.agent-src/rules/augment-portability.md), [`augment-source-of-truth`](.agent-src/rules/augment-source-of-truth.md), [`skill-quality`](.agent-src/rules/skill-quality.md), [`agents-md-thin-root`](.agent-src/skills/agents-md-thin-root/SKILL.md).
-- **Consumer story** — `npx` + `scripts/install.sh` + `.agent-settings.yml` opt-in flags, sandbox/offline install paths, verified-offline manifest: [`README.md`](README.md).
-- **Personas** — 11 review-lens cast (6 core · 5 specialist), `personas:` vs `/mode` axes, citation map, override pattern: [`docs/personas.md`](docs/personas.md), schema [`docs/contracts/persona-schema.md`](docs/contracts/persona-schema.md) (beta).
+- **Content pipelines** — A→D source / Augment / multi-tool / Claude.ai-bundle projections, indexed at [`docs/architecture.md`](docs/architecture.md); sub-pipelines under [`docs/architecture/`](docs/architecture/).
+- **Editing this repo** — Iron-Law rules (edit-discipline, source-of-truth, skill-quality) + Thin-Root contract govern every change here: [`augment-source-of-truth`](.agent-src/rules/augment-source-of-truth.md) + [`agents-md-thin-root`](.agent-src/skills/agents-md-thin-root/SKILL.md).
+- **Consumer story** — `npx` + `scripts/install.sh` opt-in flags, sandbox / offline install paths, verified-offline manifest: [`README.md`](README.md).
+- **Personas** — 11 review-lens cast (6 core · 5 specialist), `personas:` vs `/mode` axes, override pattern: [`docs/personas.md`](docs/personas.md) (beta).
+- **Discovery** (workspaces / packs / `dist/discovery/discovery-manifest.json`) — contract [`ADR-013`](docs/decisions/ADR-013-discovery-frontmatter-contract.md), how-to [`customization § Workspaces & packs`](docs/customization.md#workspaces--packs-discovery).
 
 ## Emergency triage — read this when nothing else is reachable