@waron97/prbot 3.2.1 → 3.4.0

package/src/agrippa/commands/cloneLrp.js

@@ -0,0 +1,114 @@
+import search from '@inquirer/search';
+import inquirer from 'inquirer';
+import { getToken } from '../../lib/auth.js';
+import { log, warn } from '../../lib/logger.js';
+import { loadEffectiveEnv, readConfig, writeConfig } from '../lib/config.js';
+import { getLrpXml, listLrps, resolveLrpByName } from '../lib/lrpApi.js';
+import { checksumOfPayload, comparePayload, decompose, recompose } from '../lib/pbProject.js';
+import { projectReader, writeProject } from '../lib/pbWorkspace.js';
+
+// Clone a long-running process (LRP). Structurally identical to a PB clone
+// (same decompose/recompose/checksum machinery — LRPs are plain Activiti BPMN
+// too), but selection and identity are name-based: the tabulator `id` changes
+// on every save/version bump (verified live — a re-fetched id differs from a
+// previously captured one for the same process), so it can never be used as
+// a stable workspace key the way PB's guid is.
+async function cloneLrp(opts) {
+    const config = readConfig();
+    loadEffectiveEnv(config);
+
+    if (!process.env.IMPORTEXPORT_URL) {
+        throw new Error(
+            'IMPORTEXPORT_URL is not configured. Run `prbot init` or set it in agrippa.yaml.'
+        );
+    }
+
+    const token = await getToken();
+
+    // Selection: by --name (exact) or interactive server-side search (LRPs
+    // are searched server-side by name — see listLrps — not fetched-all-then-
+    // fuzzy-filtered like PBs).
+    let chosen;
+    if (opts.name) {
+        chosen = await resolveLrpByName(token, opts.name);
+    } else {
+        log('Fetching long-running process list...');
+        const initial = await listLrps(token, null);
+        let controller = null;
+        chosen = await search({
+            message: 'Select a long-running process:',
+            source: async (input) => {
+                if (!input) return initial.map((p) => ({ name: p.name, value: p }));
+                if (controller) controller.abort();
+                controller = new AbortController();
+                try {
+                    const list = await listLrps(token, input, controller.signal);
+                    return list.map((p) => ({ name: p.name, value: p }));
+                } catch {
+                    return initial.map((p) => ({ name: p.name, value: p }));
+                }
+            },
+        });
+    }
+
+    // Destination directory.
+    let dest = opts.path ?? null;
+    if (!dest) {
+        const { inputPath } = await inquirer.prompt([
+            {
+                type: 'input',
+                name: 'inputPath',
+                message: 'Destination directory:',
+                default: chosen.name.replace(/^B2WA_/, ''),
+            },
+        ]);
+        dest = inputPath;
+    }
+
+    log(`Fetching "${chosen.name}"...`);
+    const { xml, description } = await getLrpXml(token, chosen.id);
+    const payload = { built_page: xml };
+
+    const { files } = decompose(payload);
+    writeProject(dest, files);
+
+    const scriptCount = Object.keys(files).filter((p) => p.startsWith('scripts/')).length;
+    log(`Cloned to ${dest}/  (${scriptCount} script(s)).`);
+
+    // Prove the clone reconstructs the original XML (0-loss bar A). LRPs have
+    // no `pages` at all — recompose always synthesizes `pages: []`, which
+    // comparePayload (shared with PB, where it's meaningful) would otherwise
+    // flag as noise on every single LRP clone.
+    const rebuilt = recompose(projectReader(dest));
+    const diffs = comparePayload(payload, rebuilt).filter((d) => !d.startsWith('pages:'));
+    if (diffs.length) {
+        warn('WARNING: round-trip verification found differences:');
+        diffs.forEach((d) => warn('  - ' + d));
+    } else {
+        log('Round-trip verified: recomposed payload is identical (0 information loss).');
+    }
+
+    // Register in the workspace for later pull/push — keyed by NAME, not id.
+    config.workspace = config.workspace || [];
+    const existing = config.workspace.findIndex(
+        (e) => e.object_type === 'long_running_process' && e.name === chosen.name
+    );
+    const entry = {
+        path: dest,
+        object_type: 'long_running_process',
+        name: chosen.name,
+        tenant_id: chosen.tenantId,
+        svg: chosen.bpmnFileSvg,
+        description,
+        // Baseline for push classification (see pull.js/push.js): checksum of
+        // the *recomposed* payload, changes only when local files change.
+        checksum_at_pull: checksumOfPayload(rebuilt),
+        version: chosen.version,
+        status: chosen.status,
+    };
+    if (existing >= 0) config.workspace[existing] = entry;
+    else config.workspace.push(entry);
+    writeConfig(config);
+}
+
+export { cloneLrp };

package/src/agrippa/commands/clonePb.js

@@ -2,10 +2,10 @@ import search from '@inquirer/search';
 import inquirer from 'inquirer';
 import { getToken } from '../../lib/auth.js';
 import { fuzzyMatch } from '../../lib/fuzzy.js';
-import { computeChecksum } from '../lib/checksum.js';
+import { log, warn } from '../../lib/logger.js';
 import { loadEffectiveEnv, readConfig, writeConfig } from '../lib/config.js';
 import { getProcess, listProcesses } from '../lib/pbApi.js';
-import { comparePayload, decompose, recompose, stableStringify } from '../lib/pbProject.js';
+import { checksumOfPayload, comparePayload, decompose, recompose } from '../lib/pbProject.js';
 import { projectReader, writeProject } from '../lib/pbWorkspace.js';
 
 async function clonePb(opts) {
@@ -16,11 +16,11 @@ async function clonePb(opts) {
         throw new Error('PB_URL is not configured. Run `prbot init` or set it in agrippa.yaml.');
     }
 
-    console.log('Fetching process list...');
+    log('Fetching process list...');
     const token = await getToken();
     const processes = await listProcesses(token);
     if (!processes.length) {
-        console.log('No process-builder wizards found.');
+        log('No process-builder wizards found.');
         return;
     }
 
@@ -61,7 +61,7 @@ async function clonePb(opts) {
         dest = inputPath;
     }
 
-    console.log(`Fetching "${chosen.process_name}"...`);
+    log(`Fetching "${chosen.process_name}"...`);
     const payload = await getProcess(token, chosen.guid);
 
     const { files } = decompose(payload);
@@ -69,17 +69,17 @@ async function clonePb(opts) {
 
     const scriptCount = Object.keys(files).filter((p) => p.startsWith('scripts/')).length;
     const pageCount = Object.keys(files).filter((p) => p.startsWith('pages/')).length;
-    console.log(`Cloned to ${dest}/  (${scriptCount} script(s), ${pageCount} page(s))`);
+    log(`Cloned to ${dest}/  (${scriptCount} script(s), ${pageCount} page(s))`);
 
     // Prove the clone reconstructs the original payload (0-loss bar A) by
     // reading the files back from disk and recomposing.
     const rebuilt = recompose(projectReader(dest));
     const diffs = comparePayload(payload, rebuilt);
     if (diffs.length) {
-        console.warn('WARNING: round-trip verification found differences:');
-        diffs.forEach((d) => console.warn('  - ' + d));
+        warn('WARNING: round-trip verification found differences:');
+        diffs.forEach((d) => warn('  - ' + d));
     } else {
-        console.log('Round-trip verified: recomposed payload is identical (0 information loss).');
+        log('Round-trip verified: recomposed payload is identical (0 information loss).');
     }
 
     // Register in the workspace for later pull/push.
@@ -95,7 +95,7 @@ async function clonePb(opts) {
         name: chosen.process_name,
         // Baselines for `push` classification: checksum of the *recomposed* payload
         // (changes only when local files change) + upstream updated_date/status.
-        checksum_at_pull: computeChecksum(stableStringify(rebuilt)),
+        checksum_at_pull: checksumOfPayload(rebuilt),
         updated_date: payload.updated_date,
         status: payload.status,
     };

package/src/agrippa/commands/diff.js

@@ -12,6 +12,7 @@ import {
 import { tmpdir } from 'os';
 import { join } from 'path';
 import { getToken } from '../../lib/auth.js';
+import { error, log } from '../../lib/logger.js';
 import { computeChecksum } from '../lib/checksum.js';
 import { loadEffectiveEnv, readConfig } from '../lib/config.js';
 import { getProcess } from '../lib/pbApi.js';
@@ -25,13 +26,13 @@ async function diff(targetArg) {
     loadEffectiveEnv(config);
 
     if (!config.workspace.length) {
-        console.log('No tracked resources. Run `agrippa clone` first.');
+        log('No tracked resources. Run `agrippa clone` first.');
         return;
     }
 
     const entries = filterEntries(config.workspace, targetArg);
     if (!entries.length) {
-        console.log('No tracked files match the given path.');
+        log('No tracked files match the given path.');
         return;
     }
 
@@ -43,7 +44,7 @@ async function diff(targetArg) {
     if (pbEntries.length && !process.env.PB_URL)
         throw new Error('PB_URL is not configured. Run `prbot init` or set it in agrippa.yaml.');
 
-    console.log('Fetching remote code...');
+    log('Fetching remote code...');
     const token = await getToken();
 
     let diffCount = 0;
@@ -65,13 +66,13 @@ async function diff(targetArg) {
     }
 
     if (diffCount === 0) {
-        console.log('No differences found — all tracked files match the remote.');
+        log('No differences found — all tracked files match the remote.');
     } else {
         const combined = Buffer.concat(chunks);
         const pager = process.env.PAGER || 'less';
         const pagerArgs = pager === 'less' ? ['-R', '-F'] : [];
         spawnSync(pager, pagerArgs, { input: combined, stdio: ['pipe', 'inherit', 'inherit'] });
-        console.log(`\n${diffCount} file(s) differ from remote.`);
+        log(`\n${diffCount} file(s) differ from remote.`);
     }
 }
 
@@ -90,7 +91,7 @@ function diffCodeEntries(entries, remoteCodeMap) {
             if (computeChecksum(localCode) === computeChecksum(remoteCode)) continue;
 
             if (!fileExists(entry.path)) {
-                console.log(`\n--- ${entry.path} (local file missing)`);
+                log(`\n--- ${entry.path} (local file missing)`);
                 continue;
             }
 
@@ -105,7 +106,7 @@ function diffCodeEntries(entries, remoteCodeMap) {
             );
             // exit code 1 means differences found (normal), 0 means identical, >1 means error
             if (result.status !== null && result.status > 1) {
-                console.error(`git diff failed for ${entry.path}`);
+                error(`git diff failed for ${entry.path}`);
             }
             const header = Buffer.from(`\n=== ${entry.path}  [${entry.name}] ===\n`);
             chunks.push(Buffer.concat([header, result.stdout ?? Buffer.alloc(0)]));
@@ -156,7 +157,7 @@ async function diffPbEntry(token, entry) {
             { cwd: tmpRoot, stdio: ['ignore', 'pipe', 'pipe'] }
         );
         if (result.status !== null && result.status > 1) {
-            console.error(`git diff failed for ${entry.path}`);
+            error(`git diff failed for ${entry.path}`);
         }
         const header = Buffer.from(`\n=== ${entry.path}  [${entry.name}] (process-builder) ===\n`);
         const chunk = Buffer.concat([header, result.stdout ?? Buffer.alloc(0)]);

package/src/agrippa/commands/init.js

@@ -9,6 +9,7 @@ import {
 import { join } from 'path';
 import { fileURLToPath } from 'url';
 import inquirer from 'inquirer';
+import { log } from '../../lib/logger.js';
 import { WORKSPACE_FILE } from '../lib/config.js';
 
 const TEMPLATE = `# agrippa workspace configuration
@@ -93,23 +94,23 @@ async function init() {
             },
         ]);
         if (!overwrite) {
-            console.log('Skipped workspace file.');
+            log('Skipped workspace file.');
         } else {
             writeFileSync(WORKSPACE_FILE, TEMPLATE, 'utf-8');
-            console.log(`Created ${WORKSPACE_FILE}`);
-            console.log(`Run 'agrippa clone' to add resources to this workspace.`);
-            console.log(`Add ${WORKSPACE_FILE} to .gitignore if it contains credentials.`);
+            log(`Created ${WORKSPACE_FILE}`);
+            log(`Run 'agrippa clone' to add resources to this workspace.`);
+            log(`Add ${WORKSPACE_FILE} to .gitignore if it contains credentials.`);
         }
     } else {
         writeFileSync(WORKSPACE_FILE, TEMPLATE, 'utf-8');
-        console.log(`Created ${WORKSPACE_FILE}`);
-        console.log(`Run 'agrippa clone' to add resources to this workspace.`);
-        console.log(`Add ${WORKSPACE_FILE} to .gitignore if it contains credentials.`);
+        log(`Created ${WORKSPACE_FILE}`);
+        log(`Run 'agrippa clone' to add resources to this workspace.`);
+        log(`Add ${WORKSPACE_FILE} to .gitignore if it contains credentials.`);
     }
 
     if (!existsSync('pyproject.toml')) {
         writeFileSync('pyproject.toml', PYPROJECT_TOML, 'utf-8');
-        console.log('Created pyproject.toml (ruff builtins)');
+        log('Created pyproject.toml (ruff builtins)');
     }
 
     const { importTypings } = await inquirer.prompt([
@@ -124,7 +125,7 @@ async function init() {
     if (importTypings) {
         if (!existsSync('pyrightconfig.json')) {
             writeFileSync('pyrightconfig.json', PYRIGHTCONFIG, 'utf-8');
-            console.log('Created pyrightconfig.json');
+            log('Created pyrightconfig.json');
         }
 
         const typingsDir = fileURLToPath(new URL('../../../agrippa_typings', import.meta.url));
@@ -132,7 +133,7 @@ async function init() {
         for (const file of TYPING_FILES) {
             copyFileSync(join(typingsDir, file), join('typings', file));
         }
-        console.log(`Copied ${TYPING_FILES.length} type stubs to typings/`);
+        log(`Copied ${TYPING_FILES.length} type stubs to typings/`);
     }
 
     const { importInstructions } = await inquirer.prompt([
@@ -149,13 +150,13 @@ async function init() {
         const block = pbBlock(readFileSync(guidePath, 'utf-8'));
         if (!existsSync('CLAUDE.md')) {
             writeFileSync('CLAUDE.md', block, 'utf-8');
-            console.log('Created CLAUDE.md with agrippa-pb guidance');
+            log('Created CLAUDE.md with agrippa-pb guidance');
         } else {
             const current = readFileSync('CLAUDE.md', 'utf-8');
             const begin = current.indexOf(PB_BEGIN);
             if (begin === -1) {
                 appendFileSync('CLAUDE.md', `\n${block}`, 'utf-8');
-                console.log('Appended agrippa-pb guidance to CLAUDE.md');
+                log('Appended agrippa-pb guidance to CLAUDE.md');
             } else {
                 // Replace the existing managed block in place; leave the rest as-is.
                 const endIdx = current.indexOf(PB_END, begin);
@@ -165,7 +166,7 @@ async function init() {
                     );
                 const after = current.slice(endIdx + PB_END.length).replace(/^\n/, '');
                 writeFileSync('CLAUDE.md', current.slice(0, begin) + block + after, 'utf-8');
-                console.log('Refreshed agrippa-pb guidance in CLAUDE.md');
+                log('Refreshed agrippa-pb guidance in CLAUDE.md');
             }
         }
     }

package/src/agrippa/commands/initPhase.js

@@ -2,6 +2,7 @@ import search from '@inquirer/search';
 import inquirer from 'inquirer';
 import { getToken } from '../../lib/auth.js';
 import { fuzzyMatch } from '../../lib/fuzzy.js';
+import { log } from '../../lib/logger.js';
 import {
     createConfigurator,
     deleteConfigurator,
@@ -75,12 +76,12 @@ async function initPhase() {
     if (!ripUrl)
         throw new Error('RIP_URL is not configured. Run `prbot init` or set it in agrippa.yaml.');
 
-    console.log('Fetching workflows...');
+    log('Fetching workflows...');
     const token = await getToken();
     const workflows = await listWorkflows(token, ripUrl);
 
     if (!workflows.length) {
-        console.log('No workflows found.');
+        log('No workflows found.');
         return;
     }
 
@@ -92,70 +93,78 @@ async function initPhase() {
         },
     });
 
-    console.log(`Fetching phases for "${workflow.name}"...`);
+    log(`Fetching phases for "${workflow.name}"...`);
     const phases = await getPhasesByWorkflow(token, ripUrl, workflow.id);
 
     if (!phases.length) {
-        console.log('No phases found for this workflow.');
+        log('No phases found for this workflow.');
         return;
     }
 
-    const phase = await search({
-        message: 'Select a phase:',
-        source: (input) => {
-            const filtered = input ? phases.filter((p) => fuzzyMatch(p.name, input)) : phases;
-            return filtered.map((p) => ({ name: p.name, value: p }));
+    const { selectedPhases } = await inquirer.prompt([
+        {
+            type: 'checkbox',
+            name: 'selectedPhases',
+            message: 'Select phases to initialize:',
+            choices: phases.map((p) => ({ name: p.name, value: p })),
+            loop: false,
         },
-    });
+    ]);
 
-    // Resolve result objects — API may return IDs or full objects
-    let results = phase.allowed_phase_result_ids || [];
-    if (results.length > 0 && typeof results[0] === 'number') {
-        results = await getPhaseResults(token, ripUrl, results);
+    if (!selectedPhases.length) {
+        log('No phases selected. Aborted.');
+        return;
     }
 
-    const code = generateCode(results);
-
     const { confirm } = await inquirer.prompt([
         {
             type: 'confirm',
             name: 'confirm',
-            message: `Initialize phase "${phase.name}"? This will overwrite existing code.`,
+            message: `Initialize ${selectedPhases.length} phase(s)? This will overwrite existing code.`,
             default: true,
         },
     ]);
 
     if (!confirm) {
-        console.log('Aborted.');
+        log('Aborted.');
         return;
     }
 
-    // Delete existing configurator records for this phase
-    const existing = await getPhaseConfigurators(token, ripUrl, phase.id);
-    for (const cfg of existing) {
-        await deleteConfigurator(token, ripUrl, cfg.id);
+    for (const phase of selectedPhases) {
+        // Resolve result objects — API may return IDs or full objects
+        let results = phase.allowed_phase_result_ids || [];
+        if (results.length > 0 && typeof results[0] === 'number') {
+            results = await getPhaseResults(token, ripUrl, results);
+        }
+
+        const code = generateCode(results);
+
+        // Delete existing configurator records for this phase
+        const existing = await getPhaseConfigurators(token, ripUrl, phase.id);
+        for (const cfg of existing) {
+            await deleteConfigurator(token, ripUrl, cfg.id);
+        }
+
+        // Update phase code + set_result_automatically
+        await initPhaseRemote(token, ripUrl, phase.id, code);
+
+        // Create new configurator records for each result
+        const vars = results.map((_, i) => `RES${i + 1}`);
+        for (let i = 0; i < results.length; i++) {
+            await createConfigurator(token, ripUrl, {
+                result_value: vars[i],
+                code_phase_id: phase.id,
+                triplet_phase_result_id: results[i].id,
+            });
+        }
+
+        log(`Initialized phase "${phase.name}".`);
+        if (results.length > 0) {
+            log(`  ${results.length} configurator(s) created: ${vars.join(', ')}`);
+        }
     }
 
-    // Update phase code + set_result_automatically
-    await initPhaseRemote(token, ripUrl, phase.id, code);
-
-    // Create new configurator records for each result
-    const vars = results.map((_, i) => `RES${i + 1}`);
-    for (let i = 0; i < results.length; i++) {
-        await createConfigurator(token, ripUrl, {
-            result_value: vars[i],
-            code_phase_id: phase.id,
-            triplet_phase_result_id: results[i].id,
-        });
-    }
-
-    console.log(`Initialized phase "${phase.name}".`);
-    if (results.length > 0) {
-        console.log(`  ${results.length} configurator(s) created: ${vars.join(', ')}`);
-    }
-    console.log(
-        `Run 'agrippa pull' in workspaces that track this workflow to fetch the updated code.`
-    );
+    log(`Run 'agrippa pull' in workspaces that track this workflow to fetch the updated code.`);
 }
 
 export { initPhase };