@aefree/pi-plastic 0.7.1 → 0.7.3

package/CHANGELOG.md

@@ -5,7 +5,18 @@ 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.1.0/),
 and this project follows semantic versioning for public package releases.
 
-## Unreleased
+## [0.7.3] - 2026-09-20
+
+### Changed
+
+- Update the Pi development dependencies and dynamic tool-loading eval contract to the 0.86.1 baseline while retaining explicit schema registration and sandbox authorization gates.
+
+## [0.7.2] - 2026-09-19
+
+### Fixed
+
+- Stop pending-diff Xlink ownership discovery before querying the workspace root, allowing parent-repository base resolution while preserving linked-repository qualification and refusal on unresolved descendant ownership.
+- Accept both `rep` and `repository` selector spellings when resolving workspace repository identity for pending diffs and shared selector consumers.
 
 ## [0.7.1] - 2026-09-17
 

package/README.md

@@ -75,7 +75,7 @@ PI_PLASTIC_TOOL_LOADING_MODE=loader-only # maximum initial schema reduction
 PI_PLASTIC_TOOL_LOADING_MODE=all-active  # all 30 currently exposed tools; loader omitted
 ```
 
-Pi 0.82 and newer use canonical `sourceInfo` provenance to identify this package's effective tools before deferring, restoring, or activating them. If canonical provenance or ownership of the effective loader cannot be proven, `pi-plastic` fails safe: it preserves the complete current active set exactly, does not defer, remove, or activate any `plastic_*` name, and an effective package loader can only report known tools that are already active rather than activating inactive names. On sourceInfo-capable Pi instances, providers without native deferred definitions still receive the complete current active set after a loader call. Reload or restart Pi after source edits; source files are not watched automatically.
+Pi uses canonical `sourceInfo` provenance to identify this package's effective tools before deferring, restoring, or activating them. If canonical provenance or ownership of the effective loader cannot be proven, `pi-plastic` fails safe: it preserves the complete current active set exactly, does not defer, remove, or activate any `plastic_*` name, and an effective package loader can only report known tools that are already active rather than activating inactive names. On sourceInfo-capable Pi instances, providers without native deferred definitions still receive the complete current active set after a loader call. Reload or restart Pi after source edits; source files are not watched automatically.
 
 ## Safety behavior
 
@@ -119,13 +119,13 @@ pi install npm:@aefree/pi-plastic
 Install a pinned GitHub release over HTTPS:
 
 ```bash
-pi install git:github.com/aefreedman/pi-plastic@v0.7.1
+pi install git:github.com/aefreedman/pi-plastic@v0.7.2
 ```
 
 Equivalent SSH install:
 
 ```bash
-pi install git:git@github.com/aefreedman/pi-plastic@v0.7.1
+pi install git:git@github.com/aefreedman/pi-plastic@v0.7.2
 ```
 
 To intentionally track the moving default branch instead of a release tag:
@@ -151,7 +151,7 @@ Project-local installation protects only Pi processes that load those project se
 ## Requirements
 
 - Node.js 22.19.0 or newer
-- Pi 0.82.0 or newer; the current development and eval baseline is Pi 0.83
+- Pi 0.86.1 or newer; the current development and eval baseline is Pi 0.86.1
 - Plastic SCM / Unity Version Control CLI (`cm`) available on `PATH`, or `PI_PLASTIC_CM_EXECUTABLE` set to its full executable path
 - A GNU/POSIX-compatible text `diff` available on `PATH`, or `PI_PLASTIC_DIFF_EXECUTABLE` set to its full executable path (including paths containing spaces), for text-only diff tools. Pi invokes that resolved executable as-is and does not discover Git Bash paths automatically. On macOS, bare `diff` is PATH-dependent: it may be GNU Diffutils (often installed as `diff` ahead of system paths) or Apple’s BSD `/usr/bin/diff`; the package does not substitute one for the other. Configure the intended text executable explicitly when that distinction matters.
 - Patch generation has a separate executable policy: `toolPath` is the one-call highest-priority override, followed by `PI_PLASTIC_PATCH_EXECUTABLE`. On Windows, set one to a verified patch-capable non-GUI executable such as Git's `diff.exe`; the package deliberately does not reuse `PI_PLASTIC_DIFF_EXECUTABLE`/GnuWin32 as a patch default. macOS validation with Plastic 11 found that Apple BSD `/usr/bin/diff` supports text-only diffs but rejects the `--binary` argument supplied by `cm patch`; configure a verified GNU Diffutils-compatible executable explicitly for `plastic_patch`. The non-Windows fallback to `PI_PLASTIC_DIFF_EXECUTABLE` or bare `diff` remains only for environments whose resolved executable satisfies that patch contract. In an interactive Pi session, an invalid configured patch override or missing Windows patch setting produces one package capability warning per Pi runtime; the warning omits configured paths, and `plastic_patch` retains its tool-time validation.
@@ -178,7 +178,7 @@ The live test requires `/main`, no pending changes, and no merge in progress. It
 
 ### Dynamic tool-loading eval
 
-The package-local behavioral eval uses fresh Pi 0.83 JSON subprocesses against an explicitly attested dedicated Plastic sandbox and is not a skill eval. It compares all-active, balanced, and loader-only mode behavior, checks exact smallest-sufficient loader activations, blocks destructive calls unless they are supported `preflight: true` previews, captures tool calls and sanitized provider-schema measurements, and deletes raw provider payload captures by default.
+The package-local behavioral eval uses fresh Pi 0.86.1 JSON subprocesses against an explicitly attested dedicated Plastic sandbox and is not a skill eval. It compares all-active, balanced, and loader-only mode behavior, checks exact smallest-sufficient loader activations, blocks destructive calls unless they are supported `preflight: true` previews, captures tool calls and sanitized provider-schema measurements, and deletes raw provider payload captures by default.
 
 ```bash
 npm run eval:tool-loading -- --dry-run
@@ -189,7 +189,7 @@ See [`evals/tool-loading/README.md`](evals/tool-loading/README.md) for approved
 
 ## Constrained sampling compatibility
 
-Pi 0.82 introduced provider-side constrained sampling for tools. `pi-plastic` does not currently opt in: every public Plastic schema includes optional fields (at minimum `workdir`), while OpenAI strict function schemas require closed objects and all declared properties to be required. Pi forwards the registered schema without converting those optional fields.
+Pi supports provider-side constrained sampling for tools. `pi-plastic` does not currently opt in: every public Plastic schema includes optional fields (at minimum `workdir`), while OpenAI strict function schemas require closed objects and all declared properties to be required. Pi forwards the registered schema without converting those optional fields.
 
 Enabling `strict: "prefer"` now would therefore either produce invalid strict OpenAI requests or require a breaking redesign of the ordinary tool arguments. The test suite audits all registered tools and prevents accidental opt-in until a schema is genuinely strict-compatible. Existing TypeBox validation and Plastic runtime safety checks remain authoritative.
 

package/index.ts

@@ -631,7 +631,7 @@ export default function plasticTools(pi: ExtensionAPI) {
       const ownership = getEffectivePlasticToolOwnership(pi.getAllTools(), EXTENSION_SOURCE_PATH);
       const unknownToolNames = getUnknownExactPlasticToolNames(params.toolNames);
       const active = pi.getActiveTools();
-      // Never infer ownership without Pi 0.82 canonical sourceInfo. In that
+      // Never infer ownership without canonical sourceInfo. In that
       // compatibility mode we may describe known tools that are already active,
       // but cannot safely activate an inactive same-named definition.
       const matches = searchPlasticTools(params, coreDescriptions).filter((match) =>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aefree/pi-plastic",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "type": "module",
   "engines": {
     "node": ">=22.19.0"
@@ -43,8 +43,8 @@
   },
   "devDependencies": {
     "@aefree/pi-file-discovery": "^0.1.1",
-    "@earendil-works/pi-coding-agent": "0.83.0",
-    "@earendil-works/pi-tui": "0.83.0",
+    "@earendil-works/pi-coding-agent": "0.86.1",
+    "@earendil-works/pi-tui": "0.86.1",
     "@types/node": "22.20.2",
     "tsx": "4.23.5",
     "typebox": "^1.3.8",

package/src/plastic-core.ts

@@ -1876,7 +1876,7 @@ const parseRepositoryIdentity = (selector: string | undefined): RepositoryIdenti
 
 const parseRepositorySelector = (output: string): RepositoryIdentity | null =>
 {
-    const match = output.match(/^\s*repository\s+(?:"([^"]+)"|(\S+))\s*$/mi);
+    const match = output.match(/^\s*(?:repository|rep)\s+(?:"([^"]+)"|(\S+))\s*$/mi);
     return parseRepositoryIdentity(match?.[1] ?? match?.[2]);
 };
 
@@ -1974,6 +1974,12 @@ const createPendingBaseIdentityResolver = (cwd: string, workdir?: string): Pendi
                 toPathComparisonKeyFromAbsolutePath(rootIdentity),
             ))
             {
+                // The workspace root belongs to its selector; cm rejects Xlink queries there.
+                if (toPathComparisonKeyFromAbsolutePath(await toFilesystemIdentityPath(candidate))
+                    === toPathComparisonKeyFromAbsolutePath(rootIdentity))
+                {
+                    break;
+                }
                 // A missing ancestor can be a removed Xlink mount. Status does not
                 // retain ownership, so parent-repository fallback would be unsafe.
                 if ((item.kind === "deleted" || item.kind === "moved") && !(await fs.stat(candidate).catch(() => null)))
@@ -1986,11 +1992,6 @@ const createPendingBaseIdentityResolver = (cwd: string, workdir?: string): Pendi
                     item.baseRepository = `${repository.repository}@${repository.server}`;
                     return `revid:${item.revisionId}@rep:${repository.repository}@repserver:${repository.server}`;
                 }
-                if (toPathComparisonKeyFromAbsolutePath(await toFilesystemIdentityPath(candidate))
-                    === toPathComparisonKeyFromAbsolutePath(rootIdentity))
-                {
-                    break;
-                }
                 const parent = dirname(candidate);
                 if (parent === candidate)
                 {

package/src/plastic-tool-loading.ts

@@ -103,7 +103,7 @@ function hasSourcePath(tool: PlasticToolInfo, expectedSourcePath: string): boole
 }
 
 /**
- * Pi 0.82 exposes effective tool provenance. Compare effective definitions to
+ * Pi exposes effective tool provenance. Compare effective definitions to
  * this module's own import path, so even a foreign first-registration-wins
  * collision on the public loader name cannot become an ownership anchor.
  * Without that proof, callers preserve the active set unchanged.

package/src/plastic-workspace.ts

@@ -84,7 +84,7 @@ export function parsePlasticSelector(content: string): ParseOutcome<PlasticSelec
 
   for (const line of content.split(/\r?\n/)) {
     if (/^\s*(?:#|$)/.test(line)) continue;
-    if (/^\s*repository(?:\s|$)/i.test(line)) repository = parseSelectorValue(line, "repository");
+    if (/^\s*(?:repository|rep)(?:\s|$)/i.test(line)) repository = parseSelectorValue(line, "(?:repository|rep)");
     if (/^\s*smartbranch(?:\s|$)/i.test(line)) {
       sawBranchKey = true;
       smartBranch = parseSelectorValue(line, "smartbranch");