pi-crew 0.1.30 → 0.1.32

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## Unreleased
 
+## 0.1.31
+
+### Fixed
+
+- Added required Agent Skills frontmatter (`name` and `description`) to built-in coding skills so Pi loads them without conflicts.
+- Tightened built-in skill package coverage to require standards-compliant frontmatter.
+
+## 0.1.30
+
 ### Added
 
 - Added Phase 6 async hardening: jiti loader resolution/fail-fast, async startup marker files, and early background-runner exit detection.

package/README.md

@@ -132,9 +132,12 @@ User config path:
 Project config path:
 
 ```text
-.pi/teams/config.json
+.crew/config.json            # default (new projects)
+.pi/teams/config.json        # legacy (when the repo already has .pi/)
 ```
 
+The project root is auto-detected by walking up from the current directory and stopping at any of: `.git`, `.pi`, `.crew`, `.hg`, `.svn`, `.factory`, `.omc`, or any common manifest file (`package.json`, `pyproject.toml`, `Cargo.toml`, `go.mod`, `pom.xml`, `composer.json`, `build.gradle[.kts]`). If the project already has a `.pi/` directory, pi-crew reuses it under `.pi/teams/` to avoid creating a parallel layout; otherwise it uses `.crew/`.
+
 Config merge priority:
 
 ```text
@@ -179,6 +182,11 @@ Supported config:
     "showModel": true,
     "showTokens": true,
     "showTools": true
+  },
+  "tools": {
+    "enableClaudeStyleAliases": true,
+    "enableSteer": true,
+    "terminateOnForeground": false
   }
 }
 ```
@@ -186,6 +194,7 @@ Supported config:
 Safety notes:
 
 - Foreground child-process runs continue in the Pi extension process and return control to chat immediately, so large workflows do not block the interactive session. They are interrupted on session shutdown. Use `async: true` only for intentionally detached runs that may survive the current session.
+- `tools.terminateOnForeground` is an opt-in power-user setting. When true, foreground `Agent`/`crew_agent` calls return with `terminate: true` after the child result is available, saving one follow-up LLM turn. Default is false so the assistant can still summarize raw worker output.
 
 UI notes:
 
@@ -454,12 +463,20 @@ User resources:
 ~/.pi/agent/workflows/*.workflow.md
 ```
 
-Project resources:
+Project resources (new default layout):
+
+```text
+.crew/agents/*.md
+.crew/teams/*.team.md
+.crew/workflows/*.workflow.md
+```
+
+Legacy layout (when `.pi/` already exists in the repo):
 
 ```text
-.pi/agents/*.md
-.pi/teams/*.team.md
-.pi/workflows/*.workflow.md
+.pi/teams/agents/*.md
+.pi/teams/teams/*.team.md
+.pi/teams/workflows/*.workflow.md
 ```
 
 Discovery priority:
@@ -525,33 +542,41 @@ review
 
 ## State layout
 
-Project-local state is preferred when the cwd has `.git` or `.pi`; otherwise user-global state is used.
+Project-local state is preferred when the cwd is inside a recognised project (any of the markers listed in the Config section above). Otherwise pi-crew falls back to user-global state.
 
-Typical project-local state:
+The project state root (`<crewRoot>` below) resolves to:
 
 ```text
-.pi/teams/state/runs/{runId}/manifest.json
-.pi/teams/state/runs/{runId}/tasks.json
-.pi/teams/state/runs/{runId}/events.jsonl
-.pi/teams/artifacts/{runId}/...
-.pi/teams/worktrees/{runId}/{taskId}
-.pi/teams/imports/{runId}/run-export.json
+<repoRoot>/.crew/             # default, used for new projects
+<repoRoot>/.pi/teams/         # legacy reuse when .pi/ already exists
+```
+
+Typical project-local state (`<crewRoot>` is one of the two paths above):
+
+```text
+<crewRoot>/state/runs/{runId}/manifest.json
+<crewRoot>/state/runs/{runId}/tasks.json
+<crewRoot>/state/runs/{runId}/events.jsonl
+<crewRoot>/artifacts/{runId}/...
+<crewRoot>/worktrees/{runId}/{taskId}
+<crewRoot>/imports/{runId}/run-export.json
 ```
 
 Mailbox state:
 
 ```text
-.pi/teams/state/runs/{runId}/mailbox/inbox.jsonl
-.pi/teams/state/runs/{runId}/mailbox/outbox.jsonl
-.pi/teams/state/runs/{runId}/mailbox/delivery.json
-.pi/teams/state/runs/{runId}/mailbox/tasks/{taskId}/inbox.jsonl
-.pi/teams/state/runs/{runId}/mailbox/tasks/{taskId}/outbox.jsonl
+<crewRoot>/state/runs/{runId}/mailbox/inbox.jsonl
+<crewRoot>/state/runs/{runId}/mailbox/outbox.jsonl
+<crewRoot>/state/runs/{runId}/mailbox/delivery.json
+<crewRoot>/state/runs/{runId}/mailbox/tasks/{taskId}/inbox.jsonl
+<crewRoot>/state/runs/{runId}/mailbox/tasks/{taskId}/outbox.jsonl
 ```
 
-User-global fallback:
+User-global fallback (shared with other Pi tools):
 
 ```text
-~/.pi/agent/extensions/pi-crew/runs/...
+~/.pi/agent/extensions/pi-crew/state/runs/...
+~/.pi/agent/extensions/pi-crew/artifacts/...
 ~/.pi/agent/extensions/pi-crew/imports/...
 ```
 
@@ -570,18 +595,34 @@ Optionally copy builtin resources:
 /team-init --copy-builtins --overwrite
 ```
 
-Created directories:
+Created directories (new projects):
 
 ```text
-.pi/agents/
-.pi/teams/
-.pi/workflows/
+.crew/agents/
+.crew/teams/
+.crew/workflows/
+.crew/imports/
+```
+
+If the project already has `.pi/`, the legacy layout is initialised instead:
+
+```text
+.pi/teams/agents/
+.pi/teams/teams/
+.pi/teams/workflows/
 .pi/teams/imports/
 ```
 
-`.gitignore` entries:
+`.gitignore` entries are written for whichever layout is active, e.g.:
 
 ```text
+# new layout
+.crew/state/
+.crew/artifacts/
+.crew/worktrees/
+.crew/imports/
+
+# legacy layout
 .pi/teams/state/
 .pi/teams/artifacts/
 .pi/teams/worktrees/
@@ -597,14 +638,26 @@ Export writes:
 {artifactsRoot}/export/run-export.md
 ```
 
-Import stores bundles under:
+Import stores bundles under (new layout):
+
+```text
+.crew/imports/{runId}/run-export.json
+.crew/imports/{runId}/README.md
+```
+
+or under the legacy layout when `.pi/` already exists:
 
 ```text
 .pi/teams/imports/{runId}/run-export.json
 .pi/teams/imports/{runId}/README.md
 ```
 
-or user-global imports with `--user`.
+or user-global imports with `--user`:
+
+```text
+~/.pi/agent/extensions/pi-crew/imports/{runId}/run-export.json
+~/.pi/agent/extensions/pi-crew/imports/{runId}/README.md
+```
 
 ## Doctor and validation
 

package/agents/analyst.md

@@ -1,11 +1,11 @@
----
-name: analyst
-description: Analyze requirements, ambiguity, and hidden constraints
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls
----
-
-You are a requirements analyst. Identify what is known, unknown, risky, ambiguous, or underspecified. Produce clarifying assumptions and acceptance criteria.
+---
+name: analyst
+description: Analyze requirements, ambiguity, and hidden constraints
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls
+---
+
+You are a requirements analyst. Identify what is known, unknown, risky, ambiguous, or underspecified. Produce clarifying assumptions and acceptance criteria.

package/agents/critic.md

@@ -1,11 +1,11 @@
----
-name: critic
-description: Challenge plans and designs before execution
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls
----
-
-You are a critical reviewer. Find flaws, missing steps, unsafe assumptions, overengineering, underengineering, and verification gaps. Return concrete fixes to the plan.
+---
+name: critic
+description: Challenge plans and designs before execution
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls
+---
+
+You are a critical reviewer. Find flaws, missing steps, unsafe assumptions, overengineering, underengineering, and verification gaps. Return concrete fixes to the plan.

package/agents/executor.md

@@ -1,11 +1,11 @@
----
-name: executor
-description: Implement planned code changes
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls, bash, edit, write
----
-
-You are an implementation specialist. Follow the provided plan, make targeted changes, keep edits minimal, and report changed files plus validation status. Do not broaden scope without explaining why.
+---
+name: executor
+description: Implement planned code changes
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, bash, edit, write
+---
+
+You are an implementation specialist. Follow the provided plan, make targeted changes, keep edits minimal, and report changed files plus validation status. Do not broaden scope without explaining why.

package/agents/explorer.md

@@ -1,11 +1,11 @@
----
-name: explorer
-description: Fast codebase discovery and file/symbol mapping
-model: claude-haiku-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls
----
-
-You are a fast codebase explorer. Map relevant files, symbols, data flow, and constraints. Do not modify files. Return concise findings with paths and evidence.
+---
+name: explorer
+description: Fast codebase discovery and file/symbol mapping
+model: claude-haiku-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls
+---
+
+You are a fast codebase explorer. Map relevant files, symbols, data flow, and constraints. Do not modify files. Return concise findings with paths and evidence.

package/agents/planner.md

@@ -1,11 +1,11 @@
----
-name: planner
-description: Create an execution plan with clear sequencing and risk notes
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls
----
-
-You are a planning specialist. Convert the goal and discovery notes into a concrete, ordered plan. Identify dependencies, risks, validation steps, and handoff instructions for implementers.
+---
+name: planner
+description: Create an execution plan with clear sequencing and risk notes
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls
+---
+
+You are a planning specialist. Convert the goal and discovery notes into a concrete, ordered plan. Identify dependencies, risks, validation steps, and handoff instructions for implementers.

package/agents/reviewer.md

@@ -1,11 +1,11 @@
----
-name: reviewer
-description: Review code changes for correctness, maintainability, and regressions
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls, bash
----
-
-You are a code reviewer. Review the implementation for bugs, regressions, maintainability issues, missing tests, and project-rule violations. Return prioritized findings with evidence.
+---
+name: reviewer
+description: Review code changes for correctness, maintainability, and regressions
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, bash
+---
+
+You are a code reviewer. Review the implementation for bugs, regressions, maintainability issues, missing tests, and project-rule violations. Return prioritized findings with evidence.

package/agents/security-reviewer.md

@@ -1,11 +1,11 @@
----
-name: security-reviewer
-description: Review changes for security vulnerabilities and trust-boundary issues
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls, bash
----
-
-You are a security reviewer. Look for injection, authn/authz flaws, insecure defaults, secret exposure, unsafe filesystem/network behavior, and dependency risks. Return severity and remediation.
+---
+name: security-reviewer
+description: Review changes for security vulnerabilities and trust-boundary issues
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, bash
+---
+
+You are a security reviewer. Look for injection, authn/authz flaws, insecure defaults, secret exposure, unsafe filesystem/network behavior, and dependency risks. Return severity and remediation.

package/agents/test-engineer.md

@@ -1,11 +1,11 @@
----
-name: test-engineer
-description: Design and implement test strategy for a change
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls, bash, edit, write
----
-
-You are a test engineer. Identify the right test level, add or adjust tests when asked, detect flaky assumptions, and report exact validation commands and results.
+---
+name: test-engineer
+description: Design and implement test strategy for a change
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, bash, edit, write
+---
+
+You are a test engineer. Identify the right test level, add or adjust tests when asked, detect flaky assumptions, and report exact validation commands and results.

package/agents/verifier.md

@@ -1,11 +1,11 @@
----
-name: verifier
-description: Verify that implementation satisfies the requested goal
-model: claude-sonnet-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls, bash
----
-
-You are a verification specialist. Check whether the work is complete, correct, tested, and aligned with project constraints. Prefer evidence over assumptions. Return PASS or FAIL with reasons.
+---
+name: verifier
+description: Verify that implementation satisfies the requested goal
+model: claude-sonnet-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, bash
+---
+
+You are a verification specialist. Check whether the work is complete, correct, tested, and aligned with project constraints. Prefer evidence over assumptions. Return PASS or FAIL with reasons.

package/agents/writer.md

@@ -1,11 +1,11 @@
----
-name: writer
-description: Write concise documentation, migration notes, and summaries
-model: claude-haiku-4-5
-systemPromptMode: replace
-inheritProjectContext: true
-inheritSkills: false
-tools: read, grep, find, ls, edit, write
----
-
-You are a documentation specialist. Produce clear, concise, maintainable docs and summaries. Preserve technical accuracy and avoid marketing fluff.
+---
+name: writer
+description: Write concise documentation, migration notes, and summaries
+model: claude-haiku-4-5
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, edit, write
+---
+
+You are a documentation specialist. Produce clear, concise, maintainable docs and summaries. Preserve technical accuracy and avoid marketing fluff.