pi-crew 0.8.13 → 0.8.14

package/CHANGELOG.md

@@ -2628,3 +2628,28 @@ The reply on the issue asks again for a concrete repro.
 
 typecheck clean; +6 user-scope cleanup tests + 1 routing test update; full
 suite 2963/0.
+
+## [0.8.14] — stop injecting AGENTS.md on init (Issue #35, redundant) (2026-06-18)
+
+`team action=init` no longer writes a guidance block into the project's
+`AGENTS.md`. AGENTS.md is the USER's project-instructions file (Pi loads it as
+project guidance), and the injected block was **redundant**: the `team` tool
+already self-describes via its tool registration (`description` + `promptSnippet`
+in `src/extension/registration/team-tool.ts:63-64`), which Pi injects into the
+agent's system prompt every session. So agents still learn pi-crew's commands —
+from the tool, not AGENTS.md.
+
+- Removed `injectGuidance(AGENTS.md, ...)` call from `initializeProject()`
+  (`src/extension/project-init.ts`).
+- Removed `guidancePath` / `guidanceModified` from `ProjectInitResult`.
+- Removed now-unused `getPackageVersion()` + the markers import.
+- `team action=cleanup` STILL removes any block injected by older versions
+  (<0.8.14) — backward-compat preserved via `removeGuidance`.
+- README Uninstall section + install.mjs warning note the v0.8.14 behavior
+  change.
+
+Scope rationale: pi-crew is a sub-agent orchestration extension. Modifying a
+user's project-instructions file was out-of-scope and unnecessary.
+
++4 regression tests (init does NOT create/modify AGENTS.md; API fields removed).
+typecheck clean; full suite 2972/0.

package/README.md

@@ -113,6 +113,12 @@ node ./pi-crew/install.mjs   # from local clone
 extension uninstall hook, so several things pi-crew created are left behind.
 Reverse them explicitly with `team action=cleanup`. There are **two scopes**:
 
+> **v0.8.14+**: `team action=init` **no longer injects a guidance block into
+> AGENTS.md** (it was redundant — the `team` tool self-describes via its tool
+> registration, so the agent learns pi-crew's commands from there, not AGENTS.md).
+> The cleanup steps below still work for removing blocks injected by **older
+> versions** (<0.8.14).
+
 #### Project scope (reverse `team action=init`)
 
 ```bash

package/install.mjs

@@ -70,11 +70,12 @@ console.log("To force-disable or force-enable workers in a shell, use PI_TEAMS_E
 console.log("\n--- What pi-crew writes (and how to undo it) ---");
 console.log("pi-crew itself writes nothing on install. The following only happens when you");
 console.log("explicitly run `team action=init` in a project:");
-console.log("  - A marker-delimited block is injected into the project's AGENTS.md.");
-console.log("    (Wrapped in <!-- PI-CREW:GUIDANCE:START/END --> — your content is never touched.)");
 console.log("  - A `.crew/` runtime state dir is created in the project (run history + artifacts).");
 console.log("  - With --copy-builtins: bundled agents/teams/workflows are copied into the project.");
 console.log("This install also created the global config above (`~/.pi/agent/pi-crew.json`).");
+console.log("Note: pi-crew v0.8.14+ no longer injects a guidance block into AGENTS.md on init");
+console.log("      (it was redundant — the `team` tool self-describes via tool registration).");
+console.log("      Versions <0.8.14 did inject one; `team action=cleanup` removes it.");
 console.log("\nFull uninstall (in order):");
 console.log("  team action=cleanup dryRun=true       # preview what would be removed (project)");
 console.log("  team action=cleanup                    # remove the AGENTS.md guidance block");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-crew",
-  "version": "0.8.13",
+  "version": "0.8.14",
   "description": "Pi extension for coordinated AI teams, workflows, worktrees, and async task orchestration",
   "author": "baphuongna",
   "license": "MIT",

package/src/extension/project-init.ts

@@ -2,7 +2,6 @@ import * as fs from "node:fs";
 import * as path from "node:path";
 import { configPath as globalConfigPath } from "../config/config.ts";
 import { DEFAULT_UI } from "../config/defaults.ts";
-import { injectGuidance, standardGuidanceBlocks } from "../config/markers.ts";
 import { packageRoot, projectCrewRoot, projectPiRoot } from "../utils/paths.ts";
 
 export interface ProjectInitOptions {
@@ -22,8 +21,6 @@ export interface ProjectInitResult {
 	configScope: "global" | "project" | "none";
 	configCreated: boolean;
 	configSkipped: boolean;
-	guidancePath: string;
-	guidanceModified: boolean;
 }
 
 function ensureDir(dir: string, createdDirs: string[]): void {
@@ -147,22 +144,12 @@ export function initializeProject(cwd: string, options: ProjectInitOptions = {})
 		gitignoreUpdated = true;
 	}
 
-	// Inject guidance into project AGENTS.md (or similar) using marker-based injection.
-	const guidancePath = path.join(cwd, "AGENTS.md");
-	const version = getPackageVersion();
-	const guidanceResult = injectGuidance(guidancePath, standardGuidanceBlocks(version));
+	// v0.8.14: pi-crew no longer injects a guidance block into AGENTS.md on init.
+	// AGENTS.md is the USER's project-instructions file (Pi loads it as project
+	// guidance) — extensions modifying it was out-of-scope and redundant: the
+	// `team` tool already self-describes via its schema description, so the agent
+	// learns pi-crew's commands from tool registration, not AGENTS.md.
+	// `team action=cleanup` still removes any block injected by older versions.
 
-	return { createdDirs, copiedFiles, skippedFiles, gitignorePath, gitignoreUpdated, configPath, configScope, configCreated, configSkipped, guidancePath, guidanceModified: guidanceResult.modified };
-}
-
-/** Read the current package version from the nearest package.json. */
-function getPackageVersion(): string {
-	try {
-		const pkgPath = path.join(packageRoot(), "package.json");
-		const raw = fs.readFileSync(pkgPath, "utf-8");
-		const parsed: { version?: string } = JSON.parse(raw) as { version?: string };
-		return parsed.version ?? "0.0.0";
-	} catch {
-		return "0.0.0";
-	}
+	return { createdDirs, copiedFiles, skippedFiles, gitignorePath, gitignoreUpdated, configPath, configScope, configCreated, configSkipped };
 }