@distributionos/cli 0.1.7 → 0.1.9

package/README.md

@@ -6,7 +6,7 @@ Installer for connecting an app repo to DistributionOS.
 npx @distributionos/cli setup --app <appId>
 ```
 
-Default setup opens the DistributionOS MCP OAuth approval flow when needed, then prints a plan first. In an interactive terminal, approve the plan to apply the managed setup changes. In non-interactive agent runs, use `--apply` only after reviewing the dry-run output.
+Default setup opens the DistributionOS MCP OAuth approval flow when needed, then prints a setup review first. In an interactive terminal, approve the plan to apply the managed setup changes. In non-interactive agent runs, use `--apply` only after reviewing the setup output.
 
 Agents that support installable skills can also load the public DistributionOS agent skill before using the CLI:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@distributionos/cli",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "DistributionOS repo setup CLI for agent onboarding and first-party analytics.",
   "homepage": "https://distributionos.dev",
   "repository": {

package/src/cli.js

@@ -226,8 +226,8 @@ export async function runCli(argv, runtime) {
       : null;
 
     stdout.write(args.json
-      ? `${JSON.stringify({ plan, changes, initialization, validation, inspection }, null, 2)}\n`
-      : formatApplyResult(plan, changes, initialization, validation, inspection, args.skipValidate));
+      ? `${JSON.stringify({ plan, changes, initialization, validation, inspection }, null, 2)}\n`
+      : formatApplyResult(plan, changes, initialization, validation, inspection, args.skipValidate, args.apiBase ?? DEFAULT_API_BASE));
     return hasBlockingApplyFailure(validation, inspection) ? 1 : 0;
   } catch (error) {
     stderr.write(`${error instanceof Error ? error.message : 'Setup apply failed'}\n`);
@@ -449,10 +449,13 @@ function formatReportResult(result) {
   return `${lines.join('\n')}\n`;
 }
 
-function formatApplyResult(plan, changes, initialization, validation, inspection, skipValidate) {
-  const lines = [
-    formatPlanText({ ...plan, mode: 'apply' }).trimEnd(),
-    '',
+function formatApplyResult(plan, changes, initialization, validation, inspection, skipValidate, apiBase = DEFAULT_API_BASE) {
+  const reviewUrl = initialization?.reviewUrl
+    ? absoluteDistributionOsUrl(initialization.reviewUrl, apiBase)
+    : null;
+  const lines = [
+    formatPlanText({ ...plan, mode: 'apply' }).trimEnd(),
+    '',
     'Applied changes',
   ];
 
@@ -466,9 +469,9 @@ function formatApplyResult(plan, changes, initialization, validation, inspection
     }
   }
 
-  lines.push(initialization?.reviewUrl
-    ? `- Initialization submitted. Review URL: ${initialization.reviewUrl}`
-    : '- Initialization was not submitted because no auth token was available.');
+  lines.push(reviewUrl
+    ? `- Initialization submitted. Review URL: ${reviewUrl}`
+    : '- Initialization was not submitted because no auth token was available.');
 
   lines.push('', 'Validation');
   if (skipValidate) {
@@ -497,11 +500,33 @@ function formatApplyResult(plan, changes, initialization, validation, inspection
     for (const result of inspection) {
       lines.push(`- ${result.name}: ${result.status} - ${result.message}`);
     }
-  }
-  lines.push('- Nothing was committed, pushed, or deployed.');
-  return `${lines.join('\n')}\n`;
-}
-
-function oneLine(value) {
-  return String(value).split(/\r?\n/).filter(Boolean).slice(-1)[0]?.slice(0, 240) ?? '';
-}
+  }
+  lines.push('- Nothing was committed, pushed, or deployed.');
+  lines.push('', 'Next step');
+  if (reviewUrl) {
+    lines.push(
+      '- Open this DistributionOS review page now:',
+      `  ${reviewUrl}`,
+      '- Review the generated Brain Doc and Brain Vault context before asking your agent to keep working.',
+      '- After review, come back to this repo and inspect git diff before committing, pushing, or deploying.',
+    );
+  } else {
+    lines.push(
+      '- Reconnect DistributionOS OAuth or provide an app-scoped token, then rerun setup so initialization can be submitted.',
+      '- No DistributionOS review page is ready yet.',
+    );
+  }
+  return `${lines.join('\n')}\n`;
+}
+
+function oneLine(value) {
+  return String(value).split(/\r?\n/).filter(Boolean).slice(-1)[0]?.slice(0, 240) ?? '';
+}
+
+function absoluteDistributionOsUrl(pathOrUrl, apiBase) {
+  try {
+    return new URL(pathOrUrl, apiBase || DEFAULT_API_BASE).toString();
+  } catch {
+    return String(pathOrUrl);
+  }
+}

package/src/constants.js

@@ -48,9 +48,11 @@ export const PRIVATE_ROUTE_PATTERNS = [
   '/products/**',
   '/project/**',
   '/projects/**',
-  '/workspace/**',
-  '/workspaces/**',
-  '/customer/**',
+  '/workspace/**',
+  '/workspaces/**',
+  '/workflow/**',
+  '/workflows/**',
+  '/customer/**',
   '/customers/**',
   '/client/**',
   '/clients/**',

package/src/plan.js

@@ -39,7 +39,7 @@ export async function createSetupPlan(options) {
   return {
     appId,
     cwd,
-    mode: options.apply ? 'apply' : 'dry-run',
+    mode: options.apply ? 'apply' : 'review',
     packageName: CLI_PACKAGE_NAME,
     futureCommand: buildSetupCommand(appId),
     localCommand: `npm run cli:setup -- --app ${appId}`,