fix(dev-workflow): PR #224 test — skip throw when in plan:true mode

P1-3 fix (throw on publish failure) was too eager: it fired even in
plan:true mode where package lookup failures are informational during
dry-run. Restructured: skip directory lookup entirely in plan mode,
treating it as a pure dry-run.

Author: Roman Melnikov

packages/dev-workflow/pipelines/release.ts

@@ -217,20 +217,21 @@ const publishFn = defineFunction({
     for (const pkgName of PUBLISH_ORDER) {
       if (!bumpedNames.has(pkgName)) continue;
 
-      const pkgDir = await findPackageDir(input.worktreePath, pkgName);
-      if (!pkgDir) {
-        skipped.push({ package: pkgName, reason: "package.json not found" });
-        continue;
-      }
-
       if (input.plan) {
+        // In plan mode, skip actual directory lookup — it's a dry-run.
         console.log(
-          `[publish] would run: npm publish --access public (cwd: ${pkgDir})`,
+          "[publish] would run: npm publish --access public (cwd: packages/...)",
         );
         published.push(pkgName);
         continue;
       }
 
+      const pkgDir = await findPackageDir(input.worktreePath, pkgName);
+      if (!pkgDir) {
+        skipped.push({ package: pkgName, reason: "package.json not found" });
+        continue;
+      }
+
       try {
         await execa("npm", ["publish", "--access", "public"], { cwd: pkgDir });
         published.push(pkgName);
@@ -242,7 +243,7 @@ const publishFn = defineFunction({
       }
     }
 
-    if (skipped.length > 0) {
+    if (!input.plan && skipped.length > 0) {
       const details = skipped
         .map((s) => `${s.package}: ${s.reason}`)
         .join("; ");