@aefree/pi-plastic 0.5.3 → 0.5.4

package/CHANGELOG.md

@@ -5,6 +5,13 @@ 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.
 
+## [0.5.4] - 2026-08-13
+
+### Fixed
+
+- Recognize both `smartbranch` and `br` branch forms in Plastic workspace selectors, including quoted values produced by different checkout workflows.
+- Use a valid local selector as the footer's authoritative branch source instead of launching an unnecessary `cm status` process; malformed or missing branch selectors retain the bounded CLI fallback.
+
 ## [0.5.3] - 2026-08-09
 
 ### Changed

package/README.md

@@ -4,7 +4,7 @@ Pi tools, footer status, and skill guidance for Plastic SCM / Unity Version Cont
 
 ## Plastic branch footer status
 
-When Pi starts inside a Plastic workspace, this package adds a themed `Plastic <branch>` status to Pi's built-in footer. It discovers the nearest enclosing `.plastic/plastic.workspace`, reads the local selector as a fast credential-free fallback, and confirms the branch with a bounded `cm status` call. Selector changes and successful same-workspace `plastic_*` tools refresh the status; sibling workspaces are ignored.
+When Pi starts inside a Plastic workspace, this package adds a themed `Plastic <branch>` status to Pi's built-in footer. It discovers the nearest enclosing `.plastic/plastic.workspace` and reads either the selector's `smartbranch` or `br` form as the credential-free branch source. A bounded `cm status` call is used only when the selector has no valid branch. Selector changes and successful same-workspace `plastic_*` tools refresh the status; sibling workspaces are ignored.
 
 The extension owns only the `plastic-branch` status key. It does not replace Pi's footer or suppress Pi's Git branch display, so Git and Plastic information can appear together in nested workspaces. If the Plastic marker exists but neither the selector nor `cm` yields a branch, the footer shows `Plastic branch unavailable`. Non-Plastic directories show no Plastic status.
 
@@ -107,13 +107,13 @@ pi install npm:@aefree/pi-plastic
 Install a pinned GitHub release over HTTPS:
 
 ```bash
-pi install git:github.com/aefreedman/pi-plastic@v0.5.3
+pi install git:github.com/aefreedman/pi-plastic@v0.5.4
 ```
 
 Equivalent SSH install:
 
 ```bash
-pi install git:git@github.com/aefreedman/pi-plastic@v0.5.3
+pi install git:git@github.com/aefreedman/pi-plastic@v0.5.4
 ```
 
 To intentionally track the moving default branch instead of a release tag:

package/extensions/plastic-branch-status.ts

@@ -219,7 +219,8 @@ export function createPlasticBranchStatusExtension(overrides: Partial<BranchStat
       if (!active || generation !== ownedGeneration || refreshSequence !== ownedRefresh) return;
 
       const hasSelectorBranch = selectorBranch ? displayBranch(selectorBranch) : false;
-      if (!hasSelectorBranch) displayUnavailable();
+      if (hasSelectorBranch) return;
+      displayUnavailable();
 
       const controller = new AbortController();
       confirmationController = controller;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aefree/pi-plastic",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "type": "module",
   "engines": {
     "node": ">=22.19.0"

package/src/plastic-workspace.ts

@@ -78,7 +78,8 @@ const parseSelectorValue = (line: string, key: string): string | undefined => {
 /** Parse the local selector without treating it as proof that a workspace exists. */
 export function parsePlasticSelector(content: string): ParseOutcome<PlasticSelector> {
   let repository: string | undefined;
-  let branch: string | undefined;
+  let smartBranch: string | undefined;
+  let explicitBranch: string | undefined;
   let sawBranchKey = false;
 
   for (const line of content.split(/\r?\n/)) {
@@ -86,13 +87,18 @@ export function parsePlasticSelector(content: string): ParseOutcome<PlasticSelec
     if (/^\s*repository(?:\s|$)/i.test(line)) repository = parseSelectorValue(line, "repository");
     if (/^\s*smartbranch(?:\s|$)/i.test(line)) {
       sawBranchKey = true;
-      branch = parseSelectorValue(line, "smartbranch");
+      smartBranch = parseSelectorValue(line, "smartbranch");
+    }
+    if (/^\s*br(?:\s|$)/i.test(line)) {
+      sawBranchKey = true;
+      explicitBranch = parseSelectorValue(line, "br");
     }
   }
 
   if (!sawBranchKey) return { kind: "not_found" };
+  const branch = smartBranch ?? explicitBranch;
   if (!branch || /[\u0000-\u001f\u007f]/.test(branch)) {
-    return { kind: "malformed", reason: "Selector smartbranch is blank or contains control characters." };
+    return { kind: "malformed", reason: "Selector branch is blank or contains control characters." };
   }
 
   return {