@analyticscli/growth-engineer 0.1.1-preview.20 → 0.1.1-preview.22

package/dist/runtime/openclaw-growth-wizard.mjs

@@ -16,6 +16,7 @@ const ENABLE_ISOLATED_SECRET_RUNNER_WIZARD = false;
 const DEFAULT_GROWTH_INTERVAL_MINUTES = 90;
 const DEFAULT_CONNECTOR_HEALTH_INTERVAL_MINUTES = 360;
 const DEFAULT_SCHEDULER_PROOF_PATH = 'data/openclaw-growth-engineer/runtime/scheduler-proof.jsonl';
+const DELETE_SECRET = '__OPENCLAW_DELETE_SECRET__';
 const GROWTH_ENGINEER_PACKAGE_SPEC = process.env.OPENCLAW_GROWTH_ENGINEER_PACKAGE || '@analyticscli/growth-engineer@preview';
 const RUNTIME_DIR = path.dirname(fileURLToPath(import.meta.url));
 const ACCOUNT_SIGNAL_CONNECTOR_KEYS = [
@@ -2126,6 +2127,9 @@ function getPassingConnectorKeys(payload, failedConnectors = new Set()) {
 }
 function summarizeFailureReason(detail) {
     const text = String(detail || '').replace(/\s+/g, ' ').trim();
+    if (/ASC .*\.p8 private key is invalid|invalid private key|failed to parse|sequence truncated|malformed|asn1/i.test(text)) {
+        return 'ASC auth failed: the .p8 key could not be parsed';
+    }
     if (/token has been revoked/i.test(text))
         return 'token has been revoked';
     if (/unauthorized|UNAUTHORIZED/i.test(text))
@@ -2168,6 +2172,9 @@ function summarizeFailureFix(connector, blockers) {
         return 'Paste a Coolify base URL and read-only API token from Keys & Tokens / API tokens, then rerun setup.';
     }
     if (connector === 'asc') {
+        if (/invalid|truncated|malformed|private key/i.test(combined)) {
+            return 'Use the original downloaded AuthKey_<KEY_ID>.p8 for the Reports key. Old pasted ASC_PRIVATE_KEY values are removed when you choose a file path.';
+        }
         return 'Rerun ASC setup and verify ASC credentials, key role access, and `asc apps list --output json`.';
     }
     if (isAccountSignalConnector(connector)) {
@@ -2216,21 +2223,15 @@ function printConciseSetupBlockers(payload, command, options = {}) {
         process.stdout.write(`Live checks passed: ${passingConnectors.map(connectorTitle).join(', ')}.\n`);
     }
     if (groups.size > 0) {
-        process.stdout.write('\nNeeds fix:\n');
+        process.stdout.write('\nNeeds attention:\n');
         for (const [connector, connectorBlockers] of groups.entries()) {
             const primary = connectorBlockers[0] || {};
-            const reasons = [
-                ...new Set(connectorBlockers
-                    .map((blocker) => summarizeFailureReason(blocker.detail || blocker.check))
-                    .filter(Boolean)),
-            ];
             process.stdout.write(`- ${connectorTitle(connector)}: ${summarizeFailureReason(primary.detail || primary.check)}\n`);
-            process.stdout.write(`  Why: ${reasons.join('; ')}.\n`);
             process.stdout.write(`  Fix: ${summarizeFailureFix(connector, connectorBlockers)}\n`);
         }
     }
     printDeferredSetupNotes(blockers, focusConnectors);
-    if (groups.size > 0 || !options.hideRerunWhenClean) {
+    if (!options.hideRerun && (groups.size > 0 || !options.hideRerunWhenClean)) {
         process.stdout.write(`\nRerun: ${command}\n`);
     }
 }
@@ -2375,9 +2376,9 @@ function isDeferredGitHubFailure(failure) {
     return (name === 'project:github-repo' ||
         (name === 'connection:github' && /project\.githubRepo|repo is missing|repo is not configured/i.test(detail)));
 }
-function healthStatusLabel(status) {
+function healthStatusLabel(status, spinner = '') {
     if (status === 'running')
-        return 'running';
+        return spinner ? `running ${spinner}` : 'running';
     if (status === 'pass')
         return 'done';
     if (status === 'warn')
@@ -2386,18 +2387,27 @@ function healthStatusLabel(status) {
         return 'needs attention';
     if (status === 'deferred')
         return 'deferred';
-    return 'pending';
+    return spinner ? `pending ${spinner}` : 'pending';
 }
-function renderHealthProgress(items, message = 'Live checks running...', title = 'Health check') {
+function renderHealthProgress(items, message = 'Live checks running...', title = 'Health check', options = {}) {
     if (process.stdout.isTTY)
         clearTerminal();
-    const finished = items.filter((item) => !['pending', 'running'].includes(String(item.status || ''))).length;
+    const final = Boolean(options.final);
+    const visibleItems = final
+        ? items.filter((item) => !['pending', 'running'].includes(String(item.status || '')) && item.key !== 'finalize')
+        : items;
+    const finished = visibleItems.filter((item) => !['pending', 'running'].includes(String(item.status || ''))).length;
     process.stdout.write(`${title}\n`);
     process.stdout.write('------------\n');
     process.stdout.write(`${message}\n\n`);
-    process.stdout.write(`${finished}/${items.length} checks finished.\n\n`);
-    for (const item of items) {
-        process.stdout.write(`[${healthStatusLabel(item.status)}] ${item.label}: ${item.detail}\n`);
+    if (final) {
+        process.stdout.write('');
+    }
+    else {
+        process.stdout.write(`${finished}/${visibleItems.length} checks finished.\n\n`);
+    }
+    for (const item of visibleItems) {
+        process.stdout.write(`[${healthStatusLabel(item.status, options.spinner || '')}] ${item.label}: ${item.detail}\n`);
     }
 }
 function updateHealthProgress(items, event) {
@@ -2508,22 +2518,41 @@ function updateProgressItem(items, key, status, detail) {
 }
 async function runSetupCommandWithProgress(command, env, selected, message) {
     const plan = buildSetupTestProgressPlan(selected);
-    renderHealthProgress(plan, `${message}\nDo not close this terminal yet.`, 'Connector setup test');
+    const spinnerFrames = ['-', '\\', '|', '/'];
+    let spinnerIndex = 0;
+    let currentMessage = message;
+    const render = (nextMessage = currentMessage, options = {}) => {
+        currentMessage = nextMessage;
+        renderHealthProgress(plan, currentMessage, 'Connector setup test', {
+            ...options,
+            spinner: spinnerFrames[spinnerIndex++ % spinnerFrames.length],
+        });
+    };
+    render(message);
+    const spinnerInterval = process.stdout.isTTY
+        ? setInterval(() => render(currentMessage), 800)
+        : null;
     const progressCommand = command.includes('--progress-json') ? command : `${command} --progress-json`;
     const result = await runCommandCaptureWithProgress(progressCommand, (event) => {
-        if (updateHealthProgress(plan, event)) {
-            const primaryFinished = primaryProgressItemsFinished(plan);
-            if (primaryFinished) {
-                updateProgressItem(plan, 'finalize', 'running', 'command still running; parsing final output and follow-up work');
-            }
-            const message = primaryFinished
-                ? 'Checks finished. Finalizing result; do not close this terminal yet.'
-                : 'Connector setup test is still running. Do not close this terminal yet.';
-            renderHealthProgress(plan, message, 'Connector setup test');
+        if (!updateHealthProgress(plan, event))
+            return;
+        const primaryFinished = primaryProgressItemsFinished(plan);
+        if (primaryFinished) {
+            updateProgressItem(plan, 'finalize', 'running', 'finishing');
         }
-    }, { env, timeoutMs: 180_000 });
+        render(primaryFinished ? 'Finishing setup test...' : 'Testing connector setup...');
+    }, { env, timeoutMs: 180_000 }).finally(() => {
+        if (spinnerInterval)
+            clearInterval(spinnerInterval);
+    });
+    const payload = parseJsonFromStdout(result.stdout);
+    if (Array.isArray(payload?.blockers) && payload.blockers.length > 0) {
+        if (process.stdout.isTTY)
+            clearTerminal();
+        return result;
+    }
     updateProgressItem(plan, 'finalize', 'pass', 'result received');
-    renderHealthProgress(plan, 'Connector setup test finished.', 'Connector setup test');
+    renderHealthProgress(plan, 'Connector setup test finished.', 'Connector setup test', { final: true });
     return result;
 }
 async function saveSecretsImmediately(secrets) {
@@ -2531,7 +2560,7 @@ async function saveSecretsImmediately(secrets) {
         return false;
     const secretsFile = resolveSecretsFile();
     await writeSecretsFile(secretsFile, secrets);
-    Object.assign(process.env, secrets);
+    applySecretsToProcessEnv(secrets);
     process.stdout.write(`Saved local secrets to ${secretsFile} with chmod 600.\n`);
     return true;
 }
@@ -2556,6 +2585,7 @@ async function runImmediateConnectorHealthCheck({ rl, configPath, connector, sec
         printConciseSetupBlockers(payload, command, {
             focusConnectors: [connector],
             hideRerunWhenClean: true,
+            hideRerun: true,
         });
         const retry = await askYesNo(rl, `Re-enter ${connectorLabel(connector)} configuration now?`, true);
         return { ok: false, retry, result, payload };
@@ -2722,6 +2752,10 @@ async function readSecretsFile(filePath) {
 async function writeSecretsFile(filePath, nextValues) {
     const current = await readSecretsFile(filePath);
     for (const [key, value] of Object.entries(nextValues)) {
+        if (value === DELETE_SECRET) {
+            current.delete(key);
+            continue;
+        }
         if (value.trim())
             current.set(key, value.trim());
     }
@@ -2735,6 +2769,16 @@ async function writeSecretsFile(filePath, nextValues) {
     await fs.writeFile(filePath, lines.join('\n'), { encoding: 'utf8', mode: 0o600 });
     await fs.chmod(filePath, 0o600);
 }
+function applySecretsToProcessEnv(nextValues) {
+    for (const [key, value] of Object.entries(nextValues)) {
+        if (value === DELETE_SECRET) {
+            delete process.env[key];
+            continue;
+        }
+        if (value.trim())
+            process.env[key] = value.trim();
+    }
+}
 function renderBashSingleQuoted(value) {
     return `'${String(value).replace(/'/g, `'\\''`)}'`;
 }
@@ -3911,7 +3955,7 @@ async function guideAscConnector(rl, secrets) {
     process.stdout.write(`${bold('Enter the Reports key now:')}\n`);
     printBullets([
         `${bold('.p8 path')} to Apple\'s original ${bold('AuthKey_<KEY_ID>.p8')} file. ${bold('Do not rename it')}; KEY_ID is read from the filename.`,
-        `${bold('Issuer ID')} from the API keys page.`,
+        `${bold('Issuer ID')} from the API keys page. Same value for both keys.`,
         `${bold('Vendor Number')} from Sales and Trends > Reports.`,
     ]);
     const normalKeyPath = await askAscPrivateKeyPathWithKeyId(rl, {
@@ -3922,10 +3966,12 @@ async function guideAscConnector(rl, secrets) {
     let keyId = normalKeyPath.keyId;
     if (normalKeyPath.privateKeyPath) {
         secrets.ASC_PRIVATE_KEY_PATH = normalKeyPath.privateKeyPath;
+        secrets.ASC_PRIVATE_KEY = DELETE_SECRET;
+        secrets.ASC_PRIVATE_KEY_B64 = DELETE_SECRET;
         secrets.ASC_KEY_ID = keyId;
         process.stdout.write(`Inferred ASC_KEY_ID=${keyId} from ${path.basename(normalKeyPath.privateKeyPath)}.\n`);
     }
-    const issuerId = await ask(rl, 'ASC_ISSUER_ID (reports key, empty = skip)', process.env.ASC_ISSUER_ID || '');
+    const issuerId = await ask(rl, 'ASC_ISSUER_ID (same for both keys, empty = skip)', process.env.ASC_ISSUER_ID || '');
     if (issuerId.trim())
         secrets.ASC_ISSUER_ID = issuerId.trim();
     if (!normalKeyPath.privateKeyPath) {
@@ -3942,6 +3988,8 @@ async function guideAscConnector(rl, secrets) {
         await fs.writeFile(privateKeyPath, privateKeyContent, { encoding: 'utf8', mode: 0o600 });
         await fs.chmod(privateKeyPath, 0o600);
         secrets.ASC_PRIVATE_KEY_PATH = privateKeyPath;
+        secrets.ASC_PRIVATE_KEY = DELETE_SECRET;
+        secrets.ASC_PRIVATE_KEY_B64 = DELETE_SECRET;
         process.stdout.write(`Saved ASC private key to ${privateKeyPath} with chmod 600.\n`);
     }
     const vendorNumber = await ask(rl, 'ASC_VENDOR_NUMBER (Sales and Trends > Reports)', process.env.ASC_VENDOR_NUMBER || '');
@@ -3963,7 +4011,10 @@ async function guideAscBootstrapAdminKey(rl, issuerIdDefault = '') {
         keyLabel: 'the temporary Admin key',
     });
     let bootstrapKeyId = bootstrapKeyPath.keyId;
-    const bootstrapIssuerId = await ask(rl, 'ASC_BOOTSTRAP_ISSUER_ID', issuerIdDefault);
+    let bootstrapIssuerId = String(issuerIdDefault || '').trim();
+    if (!bootstrapIssuerId) {
+        bootstrapIssuerId = await ask(rl, 'ASC_ISSUER_ID (same API keys page)', process.env.ASC_ISSUER_ID || '');
+    }
     if (bootstrapKeyPath.privateKeyPath) {
         bootstrapEnv.ASC_BOOTSTRAP_PRIVATE_KEY_PATH = bootstrapKeyPath.privateKeyPath;
         process.stdout.write(`Inferred ASC_BOOTSTRAP_KEY_ID=${bootstrapKeyId} from ${path.basename(bootstrapKeyPath.privateKeyPath)}.\n`);