@gotgenes/pi-subagents 1.0.1 → 1.0.2

package/CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.2](https://github.com/gotgenes/pi-packages/compare/pi-subagents-v1.0.1...pi-subagents-v1.0.2) (2026-05-17)
+
+
+### Documentation
+
+* plan update ADR 0001 to reflect hard-fork decision ([#51](https://github.com/gotgenes/pi-packages/issues/51)) ([bd4899a](https://github.com/gotgenes/pi-packages/commit/bd4899a0d7e72c43b80c3c07c07ccc32dc0df8ed))
+* update ADR 0001 to reflect hard-fork decision ([#51](https://github.com/gotgenes/pi-packages/issues/51)) ([387e0ad](https://github.com/gotgenes/pi-packages/commit/387e0ad06ec1cf015c1d3d1d9852a1be015fc283))
+
 ## [1.0.1](https://github.com/gotgenes/pi-packages/compare/pi-subagents-v1.0.0...pi-subagents-v1.0.1) (2026-05-17)
 
 

package/docs/decisions/0001-deferred-patches.md

@@ -1,5 +1,5 @@
 ---
-status: accepted
+status: superseded
 date: 2026-05-11
 ---
 
@@ -7,7 +7,8 @@ date: 2026-05-11
 
 ## Status
 
-Accepted
+Superseded by [`docs/architecture/architecture.md`](../architecture/architecture.md), which commits to a hard fork with material scope reduction (scheduling removal, `SubagentsAPI` boundary, `index.ts` decomposition).
+The original rationale below remains useful context.
 
 ## Context
 
@@ -51,7 +52,8 @@ All three divergences now have upstream PRs, opened after production validation
 2. **Post-bind re-filter** — [tintinweb/pi-subagents#72](https://github.com/tintinweb/pi-subagents/pull/72) (`fix(agent-runner): re-filter active tools after bindExtensions so extension tools land in child`)
 3. **Active-agent tag** — [tintinweb/pi-subagents#73](https://github.com/tintinweb/pi-subagents/pull/73) (`feat(prompts): inject <active_agent name="..."/> tag for permission resolution`)
 
-Once these land upstream, the fork's divergence reduces to package naming and tooling.
+If these land upstream, upstream gains the peer-dep fix and the two RepOne patches.
+However, the fork now diverges intentionally beyond those patches — see [`docs/architecture/architecture.md`](../architecture/architecture.md) for the full scope of planned changes.
 
 ## Consequences
 
@@ -67,5 +69,6 @@ Once these land upstream, the fork's divergence reduces to package naming and to
 
 ### Operational
 
-- Upstream PRs are open and linked above. Once merged, the fork's behavioral divergence is eliminated.
+- Upstream PRs are open and linked above. If merged, upstream gains the three patches, but the fork continues independently with broader architectural changes per [`docs/architecture/architecture.md`](../architecture/architecture.md).
+- The architecture document governs the fork's direction going forward; this ADR's original "thin-patch" framing no longer describes the fork's trajectory.
 - When Patch 1 is eventually added, it should be a separate ADR in `docs/decisions/` with its own follow-up.

package/docs/plans/0051-update-adr-0001-hard-fork.md

@@ -0,0 +1,74 @@
+---
+issue: 51
+issue_title: "docs: update ADR 0001 to reflect hard-fork decision"
+---
+
+# Update ADR 0001 to reflect hard-fork decision
+
+## Problem Statement
+
+ADR 0001 (`docs/decisions/0001-deferred-patches.md`) was written when the fork was a thin-patch layer over `tintinweb/pi-subagents`.
+The new architecture document (`docs/architecture/architecture.md`) commits to a hard fork with material scope reduction — scheduling removal, a `SubagentsAPI` boundary, `index.ts` decomposition, and more.
+
+Several claims in ADR 0001 are now outdated:
+
+1. The status is "accepted" but the decision has been superseded by the architecture doc.
+2. The Upstream PRs section states "the fork's divergence reduces to package naming and tooling," which is no longer true.
+3. The Consequences → Operational section implies that merging the upstream PRs eliminates behavioral divergence, which no longer holds.
+
+## Goals
+
+- Add a supersession note to ADR 0001 pointing to `docs/architecture/architecture.md`.
+- Update the "Upstream PRs are open" subsection so the "divergence reduces to…" claim reflects reality.
+- Update the Consequences → Operational section to note intentional divergence per the architecture document.
+- Preserve all existing rationale — no information is removed.
+
+## Non-Goals
+
+- Rewriting the ADR from scratch — the original context is still useful.
+- Updating the architecture document itself.
+- Any code changes.
+
+## Background
+
+ADR 0001 has YAML frontmatter (`status: accepted`, `date: 2026-05-11`) and follows a standard ADR structure: Status, Context, Decision, Consequences.
+
+The architecture document (`docs/architecture/architecture.md`) describes a six-phase plan that materially diverges from upstream: scheduling removal, ad-hoc RPC replacement, group-join and output-file removal, a typed `SubagentsAPI` boundary, and `index.ts` decomposition.
+
+The three upstream PRs (#71, #72, #73) are still open and factually accurate — that section just needs the concluding sentence revised.
+
+## Design Overview
+
+The update touches three areas of the ADR:
+
+1. **Frontmatter + Status section** — change `status: accepted` to `status: superseded` in frontmatter, and update the Status section body to read "Superseded" with a pointer to `docs/architecture/architecture.md`.
+2. **"Upstream PRs are open" subsection** — keep the PR list and factual statements intact; revise the final sentence ("Once these land upstream, the fork's divergence reduces to package naming and tooling.") to note that the fork now diverges intentionally beyond those patches, per the architecture document.
+3. **Consequences → Operational** — add a sentence noting that the fork diverges intentionally beyond patches, and that the architecture document governs the fork's direction going forward. Keep the existing bullet about upstream PRs.
+
+No structural changes (new sections, removed sections, reordered content).
+
+## Module-Level Changes
+
+| File                                      | Change                                                                                                                                                                                    |
+| ----------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| `docs/decisions/0001-deferred-patches.md` | Update frontmatter `status` from `accepted` to `superseded`; revise Status section; revise closing sentence in "Upstream PRs are open"; add divergence note to Consequences → Operational |
+
+## Test Impact Analysis
+
+No tests are affected — this is a docs-only change.
+
+## TDD Order
+
+1. Update ADR 0001 with all four edits described above.
+   Commit: `docs: update ADR 0001 to reflect hard-fork decision (#51)`
+
+## Risks and Mitigations
+
+| Risk                                           | Mitigation                                                                                                           |
+| ---------------------------------------------- | -------------------------------------------------------------------------------------------------------------------- |
+| Supersession note makes the ADR look stale     | Keep all original rationale intact; the note clarifies evolution, not obsolescence                                   |
+| Wording drift between ADR and architecture doc | Use a direct pointer (`docs/architecture/architecture.md`) rather than paraphrasing the architecture doc's decisions |
+
+## Open Questions
+
+None — the issue's acceptance criteria are unambiguous.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotgenes/pi-subagents",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "A pi extension that brings Claude Code-style autonomous sub-agents to pi. Friendly fork of @tintinweb/pi-subagents.",
   "author": {
     "name": "Chris Lasher"