@gotgenes/pi-subagents 1.0.0

package/.markdownlint-cli2.yaml

@@ -0,0 +1,19 @@
+# CHANGELOG.md is machine-generated by release-please. Its double blank lines
+# between sections are intentional; exclude it from linting entirely.
+#
+# README.md is largely the upstream tintinweb/pi-subagents README — we
+# preserve the original copy as-is and add a "Fork notice" block at the top
+# rather than reformatting the entire document.
+ignores:
+  - "CHANGELOG.md"
+  - "README.md"
+
+config:
+  line-length: false
+  no-duplicate-heading:
+    siblings_only: true
+  no-inline-html:
+    allowed_elements:
+      - p
+      - img
+  first-line-heading: false

package/.prettierignore

@@ -0,0 +1,5 @@
+# This fork does not use Prettier — formatting is handled by Biome
+# (see biome.json and the sibling Pi-package conventions). Ignore all
+# files to prevent Prettier from being run accidentally (e.g. by an
+# agent harness with project-level write-time Prettier formatting).
+*

package/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "1.0.0"
+}

package/AGENTS.md

@@ -0,0 +1,85 @@
+# AGENTS.md
+
+## Project Purpose
+
+This repository is a Pi extension that adds Claude Code-style autonomous subagent dispatch to the Pi coding agent.
+
+This package is a friendly fork of [`tintinweb/pi-subagents`](https://github.com/tintinweb/pi-subagents).
+It carries a small number of patches needed for downstream consumers (notably [RepOne](https://github.com/Tiny-IG-Software/repone)) that intend to use it as a normal Pi extension dependency:
+
+1. **Peer-dep rename** — peer dependencies point at `@earendil-works/pi-*` (the active scope) rather than the deprecated `@mariozechner/pi-*` scope.
+2. **Patch 2 (post-bind active-tool re-filter)** — `runAgent` re-runs the active-tool filter after `session.bindExtensions(...)` so extension-registered tools land in the child's active tool set. Without this, the `extensions: string[]` allowlist branch is functionally dead for extension tools.
+3. **Patch 3 (active_agent tag)** — `runAgent` prepends `<active_agent name="${agentConfig.name}"/>` to every assembled child system prompt so `@gotgenes/pi-permission-system` can resolve per-agent `permission:` frontmatter inside the child.
+
+See `docs/decisions/0001-deferred-patches.md` for a fourth patch (mirror parent resource paths) that was scoped out, and the rationale for not opening upstream PRs yet.
+
+## Workflow
+
+- Keep scope tight — this fork stays as close to upstream as possible.
+- Prefer small, reversible changes.
+- Preserve upstream behavior unless there is a clear reason to diverge.
+- When in doubt about whether a change should land here or be proposed upstream, prefer upstream.
+
+## Implementation Priorities
+
+- Maintain compatibility with upstream's public API.
+- Keep the patch set minimal and clearly identified in the code (search for `Patch 2 (RepOne` / `Patch 3 (RepOne` comments).
+- Mirror sibling Pi-package conventions for tooling (pnpm, biome, vitest, prek, markdownlint-cli2, release-please).
+- Track the upstream `tintinweb/pi-subagents` repository for fixes and incorporate them as merges or cherry-picks.
+
+## Code Style
+
+Use TypeScript. This project uses **pnpm** exclusively — never `npm` or `npx`.
+
+Formatting is handled by Biome (`biome check`, `biome format`). The repo intentionally does not use Prettier — a top-level `.prettierignore` blocks any harness with project-level write-time Prettier formatting from reformatting files here.
+
+## Build, Test, Lint Commands
+
+```bash
+pnpm install                    # Install dependencies + run prek install
+pnpm run build                  # tsc
+pnpm run typecheck              # tsc --noEmit
+pnpm run lint                   # biome check src/ test/
+pnpm run lint:fix               # biome check --fix src/ test/
+pnpm run lint:md                # markdownlint-cli2 '*.md' 'docs/**/*.md'
+pnpm run lint:md:fix            # markdownlint-cli2 --fix '*.md' 'docs/**/*.md'
+pnpm run lint:all               # lint + lint:md
+pnpm run format                 # biome format --write src/ test/
+pnpm test                       # vitest run
+pnpm run test:watch             # vitest
+pnpm run check                  # build + lint:all + test (full local CI)
+```
+
+## Markdown
+
+This project uses `markdownlint-cli2` with the config in `.markdownlint-cli2.yaml`.
+The `CHANGELOG.md` file is machine-generated by release-please and is excluded from linting.
+
+## Testing
+
+The fork preserves upstream's full `vitest` suite (362 tests) plus tests added for Patches 2 and 3.
+All tests must pass before publishing.
+Use `vi.hoisted(...)` for module-level mocks, matching the existing patterns in `test/agent-runner.test.ts`.
+
+## Releases
+
+This repo uses [release-please](https://github.com/googleapis/release-please) and npm trusted publishing via OIDC.
+
+- Conventional Commits drive the version bump (`feat:` → minor, `fix:` → patch, `BREAKING CHANGE:` → major).
+- Pushing to `main` triggers `release-please-action` which opens a release PR.
+- Merging the release PR creates a tag and triggers the publish job (no `NPM_TOKEN` — trusted publishing via OIDC).
+
+## Commits
+
+Use Conventional Commits.
+Commit at meaningful checkpoints without waiting for an explicit reminder.
+Prefer small, reviewable commits that leave the repository in a valid state.
+
+## Notes for Agents
+
+When working in this repo:
+
+1. The two RepOne-specific patches are clearly marked in source — search for `// Patch 2 (RepOne` or `// Patch 3 (RepOne` to find them.
+2. Do not introduce a third or fourth patch without first documenting the rationale in `docs/decisions/`.
+3. Upstream PRs to `tintinweb/pi-subagents` for Patches 2 and 3 are deferred pending production validation in RepOne — see `docs/decisions/0001-deferred-patches.md`.
+4. When syncing with upstream (rare), reapply the peer-dep rename and the two patches; the upstream `vitest` suite is the canary that nothing regressed.