@jwilger/pi-development-system 0.14.0 → 0.15.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jwilger/pi-development-system",
-  "version": "0.14.0",
+  "version": "0.15.0",
   "description": "A pi extension package representing a seasoned approach to software development using a full AI SDLC.",
   "keywords": [
     "pi-package"

package/src/core/spawn-overrides.ts

@@ -16,7 +16,7 @@ export type Settings = {
   readonly thinkingLevel: ThinkingLevel;
 };
 
-const MODEL_REF = /^[^\s/]+\/\S+$/;
+const MODEL_REF = /^[^\s/*]+\/[^\s*]+$/;
 
 const isLevel = (value: unknown): value is ThinkingLevel =>
   THINKING_LEVELS.some((level) => level === value);
@@ -66,7 +66,7 @@ export type ModelSource = "current" | "preferences" | "restored" | "inherited";
  * thread keeps the model it was already running instead of falling back to the preferences.
  */
 export function modelSource(input: {
-  mode: "use-current" | "pick-first-scoped" | "pick-first";
+  mode: "use-current" | "pick-first-scoped" | "pick-first-available";
   pinned: boolean;
   hasPreferences: boolean;
   hasRestored: boolean;

package/src/subagents/VENDORED.md

@@ -38,3 +38,4 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
+6. Empty scope (I5 review): `orch/runtime.ts` `availableScopedModels` treats an empty `/scoped-models` as no restriction (pi's meaning) instead of "matches nothing", because every devsys agent declares `models:` and would otherwise be unspawnable by default. Pinned models are also re-applied on resume (`resolveInitialSettings`), since a restored session may hold only inherited history.

package/src/subagents/orch/runtime.ts

@@ -165,6 +165,8 @@ export function createDriverFactory(
       .map((model) => ({ model }));
   const availableScopedModels = (scopedModels: readonly ScopedModel[]) => {
     const available = new Set(availableModelCandidates().map(({ model }) => modelIdentity(model)));
+    // devsys: an empty scope means no /scoped-models restriction in pi, not "matches nothing".
+    if (scopedModels.length === 0) return [...availableModelCandidates()];
     return scopedModels.filter(({ model }) => available.has(modelIdentity(model)));
   };
   const selectTypeModelPreference = (
@@ -232,9 +234,12 @@ export function createDriverFactory(
         : undefined;
     const thinkingLevel =
       savedThinking ?? type.thinkingLevel ?? parent?.thinking ?? ctx.thinkingLevel ?? "off";
-    // devsys: a per-spawn pin from agent_spawn wins; restored sessions already persist their choice.
-    if (restored === undefined)
-      return applySpawnOverrides({ provider, id, thinkingLevel }, type.spawnOverrides);
+    // devsys: a per-spawn pin from agent_spawn wins. On resume the thinking level is the saved one,
+    // but the pinned model still applies: the restored session may only hold inherited history.
+    const pins = type.spawnOverrides;
+    if (restored === undefined) return applySpawnOverrides({ provider, id, thinkingLevel }, pins);
+    if (pins?.model !== undefined)
+      return applySpawnOverrides({ provider, id, thinkingLevel }, { model: pins.model });
     return { provider, id, thinkingLevel };
   };
   const resolveAgentSettings = (type: AgentType, parentPath: string): ResolvedAgentSettings => {