@stjoseph/prumo 0.6.0 → 0.6.1

package/dist/add.js

@@ -1,6 +1,6 @@
 import { existsSync } from 'node:fs';
 import { readFile, rm, writeFile } from 'node:fs/promises';
-import { join, relative } from 'node:path';
+import { join, relative, sep } from 'node:path';
 import { appsOf, appsOfType, PORTS, typesOf, validateApps } from "./apps.js";
 import { adaptAppConfig, CLIENTS, collectCatalog, copyApp, copyTemplate, extractContract, joinContract, rootDevPrefix, takeBuildApprovals, } from "./compose.js";
 import { areasFor, growContext } from "./context.js";
@@ -85,14 +85,16 @@ export async function addApp({ templates, knowledge, cwd, type, name = type, ins
         const result = await write({ templates, knowledge, root, project, apps, app, answers, journal });
         if (install) {
             await journal.keep(join(root, 'pnpm-lock.yaml'));
-            run('pnpm', ['install'], root, childOutput);
+            // The new app is a new importer in the lockfile, which pnpm under CI would otherwise refuse to write.
+            run('pnpm', ['install', '--no-frozen-lockfile'], root, childOutput);
             // Only what this run wrote: the rest of the workspace is the team's, formatted as they left it.
             run('pnpm', [
                 'exec',
                 'biome',
                 'check',
                 '--write',
-                ...result.written.map((path) => relative(root, path)),
+                // Written with forward slashes, which every system takes and a Windows shell needs no quotes for.
+                ...result.written.map((path) => relative(root, path).split(sep).join('/')),
             ], root, childOutput);
             if (type === 'web') {
                 run('pnpm', ['exec', 'tsr', 'generate'], directory, childOutput);
@@ -174,7 +176,10 @@ async function write({ templates, knowledge, root, project, apps, app, answers,
         if (app.type === 'web' && api !== undefined && existsSync(join(at(api), file))) {
             await rewrite(join(at(api), file), (text) => {
                 const list = (envLine(text, 'WEB_ORIGIN') ?? '').split(',').filter((entry) => entry !== '');
-                return withEnvLine(text, 'WEB_ORIGIN', [...list, origin].join(','));
+                // An API with no web app yet already names the first web port, which is where this one lands.
+                return list.includes(origin)
+                    ? text
+                    : withEnvLine(text, 'WEB_ORIGIN', [...list, origin].join(','));
             });
         }
         // A new API trusts the web apps there already, on the ports they hold.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stjoseph/prumo",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Scaffolds a TypeScript project and ships it the conventions it follows, in a .prumo/ folder any AI assistant reads.",
   "keywords": [
     "scaffolding",