@dosu/cli 0.49.2 → 0.49.4

package/README.md

@@ -120,10 +120,10 @@ Once authenticated against a deployment, you can drive the Dosu platform without
 | `dosu review` | Document review workflow |
 | `dosu sources` | Manage connected data sources (list, sync, update) |
 | `dosu integrations` | List and inspect platform integrations (Slack, GitHub, …) |
-| `dosu tags` | List knowledge base tags and tagged pages |
-| `dosu members` | Manage team members and access requests |
+| `dosu topics` | List knowledge base topics and their pages |
+| `dosu members` | Invite organization members |
 | `dosu org` | Show organization information |
-| `dosu deployments` | List / show / switch deployments |
+| `dosu deployments` | List / show / switch Dosu MCP deployments |
 | `dosu analytics` | View usage statistics |
 | `dosu insights` | Open a visual report of your Dosu space activity |
 | `dosu skill` | Install / update / remove the Dosu agent skill |

package/bin/dosu.js

@@ -5000,7 +5000,7 @@ function isNpxInvocation(channel = INSTALL_CHANNEL, env = process.env) {
 function getVersionString() {
   return `v${VERSION}`;
 }
-var VERSION = "0.49.2", INSTALL_CHANNEL = "npm";
+var VERSION = "0.49.4", INSTALL_CHANNEL = "npm";
 
 // src/debug/logger.ts
 import {
@@ -8935,7 +8935,7 @@ var init_skill = __esm(() => {
 });
 
 // src/mcp/constants.ts
-var MCP_PROVIDER_SLUG = "dosu_mcp";
+var MCP_PROVIDER_SLUG2 = "dosu_mcp";
 
 // node_modules/signal-exit/dist/cjs/signals.js
 var require_signals = __commonJS((exports) => {
@@ -10593,7 +10593,7 @@ var init_settings = __esm(() => {
 });
 
 // src/telemetry/debug-id.ts
-var BUNDLE_DEBUG_ID_PLACEHOLDER = "20790a5a-a63f-06a4-6475-6e2164756e21";
+var BUNDLE_DEBUG_ID_PLACEHOLDER = "0e1efd83-4f86-ddb7-6475-6e2164756e21";
 
 // src/telemetry/telemetry.ts
 import { randomUUID as randomUUID3 } from "node:crypto";
@@ -11133,8 +11133,8 @@ function safeUuid(generate) {
   }
 }
 function eventId(generate) {
-  const uuid = safeUuid(generate);
-  return uuid?.replaceAll("-", "");
+  const uuid2 = safeUuid(generate);
+  return uuid2?.replaceAll("-", "");
 }
 function createCommandTelemetry(settings2, dependencies = {}) {
   const env = dependencies.env ?? process.env;
@@ -12728,7 +12728,7 @@ async function stepSelectDeployment(apiClient, org) {
 ${dim(deployments[0].name)}`);
       return deployments[0];
     }
-    const mcpDeployments = deployments.filter((d2) => d2.provider_slug === MCP_PROVIDER_SLUG);
+    const mcpDeployments = deployments.filter((d2) => d2.provider_slug === MCP_PROVIDER_SLUG2);
     if (mcpDeployments.length === 1) {
       logger.info("setup", `Selected deployment: ${mcpDeployments[0].name} (auto, single MCP)`);
       log.success(`Using MCP
@@ -13813,7 +13813,7 @@ async function resolveDeployment2(client, cfg, opts) {
       });
       return { code: 1, cfg };
     }
-    const mcpDeployments = allDeployments.filter((d) => d.provider_slug === MCP_PROVIDER_SLUG);
+    const mcpDeployments = allDeployments.filter((d) => d.provider_slug === MCP_PROVIDER_SLUG2);
     if (mcpDeployments.length === 0) {
       emitError({
         step: "deployment",
@@ -13939,6 +13939,38 @@ init_commander();
 init_trpc();
 var import_picocolors4 = __toESM(require_picocolors(), 1);
 
+// src/commands/arguments.ts
+init_commander();
+function positiveInteger(value) {
+  const parsed = Number(value);
+  if (!Number.isSafeInteger(parsed) || parsed <= 0) {
+    throw new InvalidArgumentError("must be a positive integer");
+  }
+  return parsed;
+}
+function positiveIntegerAtMost(maximum) {
+  return (value) => {
+    const parsed = positiveInteger(value);
+    if (parsed > maximum) {
+      throw new InvalidArgumentError(`must be at most ${maximum}`);
+    }
+    return parsed;
+  };
+}
+function messageLimit(value) {
+  const parsed = Number(value);
+  if (!Number.isSafeInteger(parsed) || parsed !== -1 && parsed <= 0) {
+    throw new InvalidArgumentError("must be -1 or a positive integer");
+  }
+  return parsed;
+}
+function uuid(value) {
+  if (!/^[0-9a-f]{8}-[0-9a-f]{4}-[1-8][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i.test(value)) {
+    throw new InvalidArgumentError("must be a UUID");
+  }
+  return value;
+}
+
 // src/commands/auth.ts
 init_config();
 var import_picocolors2 = __toESM(require_picocolors(), 1);
@@ -14027,12 +14059,12 @@ function requireConfig() {
   return cfg;
 }
 function analyticsCommand() {
-  const cmd = new Command("analytics").description("View usage statistics").option("--days <n>", "Number of days to analyze (default: 30)", "30").option("--json", "Output as JSON").action(async (opts) => {
+  const cmd = new Command("analytics").description("View usage statistics").addOption(new Option("--days <n>", "Number of days to analyze (default: 30)").argParser(positiveInteger).default(30)).option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig();
     const client = createTypedClient(cfg);
     const stats = await client.analytics.getUsageStats.query({
       spaceId: cfg.active_account.target.space_id,
-      days: Number.parseInt(opts.days ?? "30", 10)
+      days: opts.days
     });
     if (opts.json) {
       printResult(stats, opts);
@@ -14186,8 +14218,7 @@ var DEFAULT_DEPLOYMENT_CONFIG_GITHUB = {
       multiple_labels_per_group: false,
       include: [],
       exclude: [],
-      separator: null,
-      custom_instructions: null
+      separator: null
     },
     voting: {
       enabled: false
@@ -14213,8 +14244,7 @@ var DEFAULT_DEPLOYMENT_CONFIG_GITHUB = {
       multiple_labels_per_group: false,
       include: [],
       exclude: [],
-      separator: null,
-      custom_instructions: null
+      separator: null
     },
     lgtm_label: {
       enabled: false,
@@ -14233,10 +14263,6 @@ var DEFAULT_DEPLOYMENT_CONFIG_GITHUB = {
       color: "FFA500",
       description: "This PR is set to be merged"
     },
-    stale_doc_check: {
-      enabled: true,
-      monitored_paths: []
-    },
     diff_review_policies: []
   },
   discussions: {
@@ -14256,14 +14282,6 @@ var DEFAULT_DEPLOYMENT_CONFIG_GITHUB = {
     included_categories: ["Q&A", "Questions"],
     quality_checklist: null
   },
-  stale_bot: {
-    enabled: false,
-    days_before_stale: 90,
-    days_before_close: 7,
-    max_issues_close_per_day: 25,
-    excluded_label_ids: [],
-    tag_maintainer_response: false
-  },
   changelogs: {
     visibility: false,
     enabled: false
@@ -14680,6 +14698,26 @@ var REPO_REFRESH_POLL_TIMEOUT_MS = 1e4;
 var REPO_MULTISELECT_MAX_ITEMS = 10;
 var DATA_SOURCE_VERIFY_POLL_INTERVAL_MS = 1000;
 var DATA_SOURCE_VERIFY_POLL_TIMEOUT_MS = 1e4;
+function parseAvailableRepos(value) {
+  if (!Array.isArray(value))
+    throw new Error("githubRepository.listForOrg returned a non-array");
+  return value.map((repository, index) => {
+    if (repository === null || typeof repository !== "object" || !("repository_id" in repository) || typeof repository.repository_id !== "number" || !("name" in repository) || typeof repository.name !== "string" || !("slug" in repository) || typeof repository.slug !== "string" || !("is_deployed" in repository) || typeof repository.is_deployed !== "boolean" || "created_at" in repository && repository.created_at !== undefined && typeof repository.created_at !== "string") {
+      throw new Error(`githubRepository.listForOrg returned an invalid repository at index ${index}`);
+    }
+    return repository;
+  });
+}
+function parseDeploymentIds(value) {
+  if (!Array.isArray(value))
+    throw new Error("workspaces.listForSpace returned a non-array");
+  return value.map((deployment, index) => {
+    if (deployment === null || typeof deployment !== "object" || !("deployment_id" in deployment) || typeof deployment.deployment_id !== "string") {
+      throw new Error(`workspaces.listForSpace returned an invalid deployment at index ${index}`);
+    }
+    return deployment.deployment_id;
+  });
+}
 function detectGitRepo(cwd = process.cwd()) {
   let url;
   try {
@@ -14703,9 +14741,9 @@ function detectGitRepo(cwd = process.cwd()) {
 }
 async function fetchListForOrg(trpc, orgID) {
   try {
-    const repos = await trpc.githubRepository.listForOrg.query({
+    const repos = parseAvailableRepos(await trpc.githubRepository.listForOrg.query({
       org_id: orgID
-    });
+    }));
     return sortReposByRecency(repos);
   } catch (err) {
     const msg = err instanceof Error ? err.message : String(err);
@@ -14845,9 +14883,9 @@ async function createDeploymentForRepo(trpc, orgID, spaceID, repo) {
       space_id: spaceID,
       data_source_ids: [dataSourceID]
     });
-    const spaceDeployments = await trpc.workspaces.listForSpace.query(spaceID);
-    await Promise.all(spaceDeployments.map((d) => trpc.deploymentDataSource.create.mutate({
-      deployment_id: d.deployment_id,
+    const spaceDeploymentIds = parseDeploymentIds(await trpc.workspaces.listForSpace.query(spaceID));
+    await Promise.all(spaceDeploymentIds.map((deploymentID) => trpc.deploymentDataSource.create.mutate({
+      deployment_id: deploymentID,
       data_source_id: dataSourceID
     })));
     return {
@@ -15449,18 +15487,28 @@ function auditCommand() {
 
 // src/commands/deployments.ts
 init_commander();
+init_client();
 init_trpc();
 init_config();
 var import_picocolors10 = __toESM(require_picocolors(), 1);
+var MCP_PROVIDER_SLUG = "dosu_mcp";
 function requireConfig4() {
   return requireLoginConfig();
 }
+async function listAccessibleDeployments(client, activeOrgId) {
+  if (activeOrgId)
+    return client.workspaces.listForOrg.query(activeOrgId);
+  const orgs = await client.organization.getOrganizations.query({});
+  const deployments = await Promise.all(orgs.map((org) => client.workspaces.listForOrg.query(org.org_id)));
+  return deployments.flat();
+}
 function deploymentsCommand() {
   const cmd = new Command("deployments").description("Manage deployments");
   cmd.command("list").description("List all deployments").option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig4();
     const client = createTypedClient(cfg);
-    const deployments = cfg.active_account?.target?.org_id ? await client.workspaces.listForOrg.query(cfg.active_account?.target?.org_id) : await client.workspaces.listAll.query({});
+    const allDeployments = await listAccessibleDeployments(client, cfg.active_account?.target?.org_id);
+    const deployments = allDeployments.filter((deployment) => deployment.provider_slug === MCP_PROVIDER_SLUG);
     if (opts.json) {
       printResult(deployments, opts);
       return;
@@ -15515,11 +15563,17 @@ ${import_picocolors10.default.dim(`Current: ${cfg.active_account?.target?.deploy
       console.error(import_picocolors10.default.red(`Deployment not found: ${id}`));
       process.exit(1);
     }
+    if (deployment.provider_slug !== MCP_PROVIDER_SLUG) {
+      console.error(import_picocolors10.default.red(`Not a Dosu MCP deployment: ${id}`));
+      process.exit(1);
+    }
+    const apiKey = await new Client(cfg).createAPIKey(deployment.deployment_id, "dosu-cli");
     updateTarget(cfg, {
       deployment_id: deployment.deployment_id,
       deployment_name: deployment.name,
       org_id: deployment.org_id,
-      space_id: deployment.space_id
+      space_id: deployment.space_id,
+      api_key: apiKey.api_key
     });
     saveConfig(cfg);
     if (opts.json) {
@@ -15535,10 +15589,47 @@ ${import_picocolors10.default.dim(`Current: ${cfg.active_account?.target?.deploy
 init_dist8();
 init_dist();
 init_commander();
+init_client();
 init_trpc();
 init_logger();
 var import_picocolors11 = __toESM(require_picocolors(), 1);
 import { readFileSync as readFileSync6 } from "node:fs";
+
+// src/commands/doc-import.ts
+var IMPORT_PLATFORMS = [
+  "github",
+  "gitlab",
+  "azure_devops",
+  "confluence",
+  "notion",
+  "coda"
+];
+async function importDocuments(client, platform2, context, ids) {
+  const baseInput = {
+    knowledge_store_id: context.knowledgeStoreId,
+    space_id: context.spaceId
+  };
+  switch (platform2) {
+    case "github":
+      return client.docImports.importGithubFiles.mutate({ ...baseInput, file_ids: ids });
+    case "gitlab":
+      return client.docImports.importGitlabFiles.mutate({ ...baseInput, file_ids: ids });
+    case "azure_devops":
+      return client.docImports.importAzureDevopsFiles.mutate({ ...baseInput, file_ids: ids });
+    case "confluence":
+      return client.docImports.importConfluencePages.mutate({ ...baseInput, page_ids: ids });
+    case "notion":
+      return client.docImports.importNotionPages.mutate({ ...baseInput, page_ids: ids });
+    case "coda":
+      return client.docImports.importCodaPages.mutate({ ...baseInput, page_ids: ids });
+    default: {
+      const unsupported = platform2;
+      throw new Error(`Unsupported import platform: ${unsupported}`);
+    }
+  }
+}
+
+// src/commands/docs.ts
 function requireConfig5() {
   const cfg = requireLoginConfig();
   if (!cfg.active_account?.target?.space_id) {
@@ -15628,6 +15719,9 @@ async function backendPost2(path3, apiKey, body) {
     headers: { "Content-Type": "application/json", "X-Dosu-API-Key": apiKey },
     body: JSON.stringify(body)
   });
+  return parseBackendResponse2(resp);
+}
+async function parseBackendResponse2(resp) {
   if (!resp.ok) {
     let detail = `Request failed with status ${resp.status}`;
     try {
@@ -15640,15 +15734,15 @@ async function backendPost2(path3, apiKey, body) {
 }
 function docsCommand() {
   const cmd = new Command("docs").description("Document management");
-  cmd.command("list").description("List documents").option("--search <query>", "Search documents").option("--tag <id>", "Filter by tag ID").option("--limit <n>", "Maximum results", "20").option("--json", "Output as JSON").action(async (opts) => {
+  cmd.command("list").description("List documents").option("--search <query>", "Search documents").option("--topic <id>", "Filter by topic ID").addOption(new Option("--limit <n>", "Maximum results").argParser(positiveInteger).default(20)).option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig5();
     const client = createTypedClient(cfg);
     const ksId = await getKnowledgeStoreId(client, cfg.active_account.target.space_id);
     const result = await client.page.listWithTags.query({
       knowledge_store_id: ksId,
       searchTerm: opts.search,
-      topic_id: opts.tag,
-      limit: Number.parseInt(opts.limit ?? "20", 10)
+      topic_id: opts.topic,
+      limit: opts.limit
     });
     const pages = result.data;
     if (opts.json) {
@@ -15666,12 +15760,12 @@ function docsCommand() {
       formatDate(p2.created_at)
     ]), { rawData: pages });
   });
-  cmd.command("get").description("Get a document").argument("<id>", "Page ID").option("--version <v>", "Specific version number").option("--json", "Output as JSON").action(async (id, opts) => {
+  cmd.command("get").description("Get a document").argument("<id>", "Page ID").addOption(new Option("--revision <n>", "Specific revision number").argParser(positiveInteger)).option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig5();
     const client = createTypedClient(cfg);
     const page = await client.page.get.query({
       page_id: id,
-      version: opts.version ? Number.parseInt(opts.version, 10) : undefined
+      version: opts.revision
     });
     if (opts.json) {
       printResult(page, opts);
@@ -15692,7 +15786,7 @@ function docsCommand() {
 ${page.body}`);
     }
   });
-  cmd.command("create").description("Create a new document").requiredOption("--title <title>", "Document title").option("--body <markdown>", "Document body (markdown)").option("--body-file <path>", "Read body from file").option("--json", "Output as JSON").action(async (opts) => {
+  cmd.command("create").description("Create a new document").requiredOption("--title <title>", "Document title").option("--body <markdown>", "Document body (markdown)").addOption(new Option("--body-file <path>", "Read body from file").conflicts("body")).option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig5();
     const client = createTypedClient(cfg);
     const ksId = await getKnowledgeStoreId(client, cfg.active_account.target.space_id);
@@ -15708,19 +15802,22 @@ ${page.body}`);
     }
     console.log(import_picocolors11.default.green(`Document "${opts.title}" created.`));
   });
-  cmd.command("update").description("Update a document").argument("<id>", "Page ID").option("--title <title>", "New title").option("--body <markdown>", "New body (markdown)").option("--body-file <path>", "Read body from file").option("--json", "Output as JSON").action(async (id, opts) => {
+  cmd.command("update").description("Update a document").argument("<id>", "Page ID").option("--title <title>", "New title").option("--body <markdown>", "New body (markdown)").addOption(new Option("--body-file <path>", "Read body from file").conflicts("body")).option("--json", "Output as JSON").action(async (id, opts) => {
+    if (opts.title === undefined && opts.body === undefined && opts.bodyFile === undefined) {
+      throw new Error("Specify at least one of --title, --body, or --body-file.");
+    }
     const cfg = requireConfig5();
     const client = createTypedClient(cfg);
     const ksId = await getKnowledgeStoreId(client, cfg.active_account.target.space_id);
     const body = readBody(opts);
-    const result = await client.page.update.mutate({
+    await client.page.update.mutate({
       id,
       knowledge_store_id: ksId,
       title: opts.title,
       body
     });
     if (opts.json) {
-      printResult(result, opts);
+      printResult({ success: true, id }, opts);
       return;
     }
     console.log(import_picocolors11.default.green("Document updated."));
@@ -15765,18 +15862,18 @@ ${page.body}`);
       formatDate(v.created_at)
     ]), { rawData: versions });
   });
-  cmd.command("restore").description("Restore a document version").argument("<id>", "Page ID").requiredOption("--version <n>", "Version number to restore").option("--json", "Output as JSON").action(async (id, opts) => {
+  cmd.command("restore").description("Restore a document version").argument("<id>", "Page ID").addOption(new Option("--revision <n>", "Revision number to restore").argParser(positiveInteger).makeOptionMandatory()).option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig5();
     const client = createTypedClient(cfg);
     await client.page.restoreVersion.mutate({
       page_id: id,
-      version_to_restore: Number.parseInt(opts.version, 10)
+      version_to_restore: opts.revision
     });
     if (opts.json) {
-      printResult({ success: true, id, version: opts.version }, opts);
+      printResult({ success: true, id, revision: opts.revision }, opts);
       return;
     }
-    console.log(import_picocolors11.default.green(`Document restored to version ${opts.version}.`));
+    console.log(import_picocolors11.default.green(`Document restored to version ${opts.revision}.`));
   });
   cmd.command("generate").description("Generate a document using AI").requiredOption("--title <title>", "Document title").option("--instructions <text>", "Custom generation instructions").option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig5();
@@ -15802,31 +15899,19 @@ ${page.body}`);
     }
     console.log(import_picocolors11.default.green("Auto-tagging started."));
   });
-  cmd.command("import").description("Import documents from an external platform").argument("<platform>", "Platform: github, gitlab, azure_devops, confluence, notion, coda").requiredOption("--files <ids>", "Comma-separated file/page IDs to import").option("--json", "Output as JSON").action(async (platform2, opts) => {
+  cmd.command("import").description("Import documents from an external platform").addArgument(new Argument("<platform>", "Import platform").choices([...IMPORT_PLATFORMS])).requiredOption("--files <ids>", "Comma-separated file/page IDs to import").option("--json", "Output as JSON").action(async (platform2, opts) => {
     const cfg = requireConfig5();
     const client = createTypedClient(cfg);
     const ksId = await getKnowledgeStoreId(client, cfg.active_account.target.space_id);
-    const fileIds = opts.files.split(",").map((id) => id.trim());
-    const importFn = {
-      github: (input) => client.docImports.importGithubFiles.mutate(input),
-      gitlab: (input) => client.docImports.importGitlabFiles.mutate(input),
-      azure_devops: (input) => client.docImports.importAzureDevopsFiles.mutate(input),
-      confluence: (input) => client.docImports.importConfluencePages.mutate(input),
-      notion: (input) => client.docImports.importNotionPages.mutate(input),
-      coda: (input) => client.docImports.importCodaPages.mutate(input)
-    };
-    const fn = importFn[platform2.toLowerCase()];
-    if (!fn) {
-      console.error(import_picocolors11.default.red(`Unknown platform: ${platform2}. Use: github, gitlab, azure_devops, confluence, notion, coda`));
-      process.exit(1);
+    const fileIds = opts.files.split(",").map((id) => id.trim()).filter(Boolean);
+    if (fileIds.length === 0) {
+      throw new Error("--files must contain at least one file or page ID.");
     }
-    const idField = ["confluence", "notion", "coda"].includes(platform2.toLowerCase()) ? "page_ids" : "file_ids";
     try {
-      const result = await fn({
-        knowledge_store_id: ksId,
-        space_id: cfg.active_account.target.space_id,
-        [idField]: fileIds
-      });
+      const result = await importDocuments(client, platform2, {
+        knowledgeStoreId: ksId,
+        spaceId: cfg.active_account.target.space_id
+      }, fileIds);
       if (opts.json) {
         printResult(result, opts);
         return;
@@ -15850,10 +15935,10 @@ ${page.body}`);
       process.exit(1);
     }
   });
-  cmd.command("import-status").description("Check import task status").argument("<task-id>", "Import task ID").option("--json", "Output as JSON").action(async (taskId, opts) => {
+  cmd.command("import-status").description("Check import task status").addArgument(new Argument("<task-id>", "Import task ID").argParser(uuid)).option("--json", "Output as JSON").action(async (taskId, opts) => {
     const cfg = requireConfig5();
-    const client = createTypedClient(cfg);
-    const status = await client.docImports.getImportStatus.query(taskId);
+    const response = await new Client(cfg).get(`/doc-imports/status/${encodeURIComponent(taskId)}`);
+    const status = response.status === 404 ? null : await parseBackendResponse2(response);
     if (opts.json) {
       printResult(status, opts);
       return;
@@ -15960,17 +16045,25 @@ function requireConfig6() {
   }
   return cfg;
 }
+var DISPLAY_PLATFORMS = [
+  "github",
+  "gitlab",
+  "azure_devops",
+  "slack",
+  "confluence",
+  "notion",
+  "coda",
+  "teams"
+];
 var NANGO_PROBES = {
   gitlab: [
     { provider: "gitlab", providerConfigKey: "gitlab" },
     { provider: "gitlab", providerConfigKey: "gitlab-pat" }
   ],
-  "gitlab-pat": [{ provider: "gitlab", providerConfigKey: "gitlab-pat" }],
   confluence: [
     { provider: "confluence", providerConfigKey: "confluence" },
     { provider: "confluence", providerConfigKey: "confluence-basic" }
   ],
-  "confluence-basic": [{ provider: "confluence", providerConfigKey: "confluence-basic" }],
   notion: [{ provider: "notion", providerConfigKey: "notion" }],
   coda: [{ provider: "coda", providerConfigKey: "coda" }],
   azure_devops: [
@@ -15978,32 +16071,20 @@ var NANGO_PROBES = {
     { provider: "azure_devops", providerConfigKey: "azure-devops" }
   ]
 };
-var DISPLAY_PLATFORMS = [
-  "github",
-  "gitlab",
-  "azure_devops",
-  "slack",
-  "confluence",
-  "notion",
-  "coda",
-  "teams"
-];
 async function probeConnection(client, orgId, platform4) {
   const probes = NANGO_PROBES[platform4];
   if (!probes) {
-    return { queryable: false, connected: false, connection: null };
+    return { queryable: false, connected: null, connection: null };
   }
   for (const probe of probes) {
-    try {
-      const conn = await client.nango.getConnection.query({
-        provider: probe.provider,
-        providerConfigKey: probe.providerConfigKey,
-        orgId
-      });
-      if (conn != null) {
-        return { queryable: true, connected: true, connection: conn };
-      }
-    } catch {}
+    const conn = await client.nango.getConnection.query({
+      provider: probe.provider,
+      providerConfigKey: probe.providerConfigKey,
+      orgId
+    });
+    if (conn != null) {
+      return { queryable: true, connected: true, connection: conn };
+    }
   }
   return { queryable: true, connected: false, connection: null };
 }
@@ -16022,19 +16103,19 @@ function integrationsCommand() {
     }
     printTable(["Platform", "Status"], results.map((r2) => [
       r2.platform,
-      r2.connected ? import_picocolors15.default.green("connected") : import_picocolors15.default.dim("not connected")
+      r2.connected === null ? import_picocolors15.default.dim("status unavailable") : r2.connected ? import_picocolors15.default.green("connected") : import_picocolors15.default.dim("not connected")
     ]), { rawData: results });
   });
-  cmd.command("status").description("Check connection status of a specific platform").argument("<platform>", `Platform: ${DISPLAY_PLATFORMS.join(", ")}`).option("--json", "Output as JSON").action(async (platform4, opts) => {
+  cmd.command("status").description("Check connection status of a specific platform").addArgument(new Argument("<platform>", "Integration platform").choices([...DISPLAY_PLATFORMS])).option("--json", "Output as JSON").action(async (platform4, opts) => {
     const cfg = requireConfig6();
     const client = createTypedClient(cfg);
     const { queryable, connected, connection } = await probeConnection(client, cfg.active_account.target.org_id, platform4);
     if (!queryable) {
       if (opts.json) {
-        printResult({ platform: platform4, connected: false, note: "not queryable via nango" }, opts);
+        printResult({ platform: platform4, connected: null, note: "connection status unavailable via CLI" }, opts);
         return;
       }
-      console.log(`${platform4}: ${import_picocolors15.default.dim("not connected (not queryable)")}`);
+      console.log(`${platform4}: ${import_picocolors15.default.dim("connection status unavailable via CLI")}`);
       return;
     }
     if (opts.json) {
@@ -16070,10 +16151,10 @@ function integrationsCommand() {
     }
     console.log(import_picocolors15.default.green(`Joined Slack channel ${channelId}.`));
   });
-  cmd.command("github-collaborators").description("List GitHub repository collaborators").option("--json", "Output as JSON").action(async (opts) => {
+  cmd.command("github-collaborators").description("List GitHub repository collaborators").addArgument(new Argument("<repository-id>", "Numeric GitHub repository ID").argParser(positiveInteger)).option("--json", "Output as JSON").action(async (repositoryId, opts) => {
     const cfg = requireConfig6();
     const client = createTypedClient(cfg);
-    const collaborators = await client.githubRepository.getCollaborators.query(0);
+    const collaborators = await client.githubRepository.getCollaborators.query(repositoryId);
     if (opts.json) {
       printResult(collaborators, opts);
       return;
@@ -16101,7 +16182,7 @@ function requireConfig7() {
 }
 function knowledgeCommand() {
   const cmd = new Command("knowledge").description("Search and browse your knowledge base");
-  cmd.command("search").description("Search the knowledge base").argument("<query>", "Search query").option("--json", "Output as JSON").option("--limit <n>", "Maximum results", "10").action(async (query, opts) => {
+  cmd.command("search").description("Search the knowledge base").argument("<query>", "Search query").option("--json", "Output as JSON").addOption(new Option("--limit <n>", "Maximum results").argParser(positiveInteger).default(10)).action(async (query, opts) => {
     const cfg = requireConfig7();
     const client = createTypedClient(cfg);
     const dataSources = await client.dataSource.list.query({
@@ -16127,15 +16208,14 @@ function knowledgeCommand() {
       console.log(import_picocolors16.default.dim("No results found."));
       return;
     }
-    const limit = Number.parseInt(opts.limit ?? "10", 10);
-    const limited = results.slice(0, limit);
+    const limited = results.slice(0, opts.limit);
     printTable(["Title", "Type"], limited.map((r2) => [
       truncate(r2.title ?? "(untitled)", 60),
       r2.entity_type ?? "—"
     ]), { json: false, rawData: limited });
-    if (results.length > limit) {
+    if (results.length > opts.limit) {
       console.log(import_picocolors16.default.dim(`
-${results.length - limit} more results not shown.`));
+${results.length - opts.limit} more results not shown.`));
     }
   });
   cmd.command("list").description("Show knowledge store information").option("--json", "Output as JSON").action(async (opts) => {
@@ -16170,28 +16250,11 @@ function requireConfig8() {
   return cfg;
 }
 function membersCommand() {
-  const cmd = new Command("members").description("Manage team members");
-  cmd.command("list").description("List team members and invitations").option("--json", "Output as JSON").action(async (opts) => {
-    const cfg = requireConfig8();
-    const client = createTypedClient(cfg);
-    const data = await client.invitations.getInvitations.query();
-    if (opts.json) {
-      printResult(data, opts);
-      return;
-    }
-    if (!data.items || data.items.length === 0) {
-      console.log(import_picocolors17.default.dim("No members or invitations found."));
-      return;
-    }
-    printTable(["Email", "Org"], data.items.map((m3) => [
-      m3.email ?? "—",
-      m3.org?.name ?? "—"
-    ]), { rawData: data });
-  });
-  cmd.command("invite").description("Invite a member to the organization").argument("<email>", "Email address to invite").option("--role <role>", "Role: admin or member", "member").option("--json", "Output as JSON").action(async (email, opts) => {
+  const cmd = new Command("members").description("Invite organization members");
+  cmd.command("invite").description("Invite a member to the organization").argument("<email>", "Email address to invite").addOption(new Option("--role <role>", "Organization role").choices(["admin", "member"]).default("member")).option("--json", "Output as JSON").action(async (email, opts) => {
     const cfg = requireConfig8();
     const client = createTypedClient(cfg);
-    const role = opts.role.toUpperCase() === "ADMIN" ? "ADMIN" : "MEMBER";
+    const role = opts.role === "admin" ? "ADMIN" : "MEMBER";
     await client.invitations.invite.mutate({
       orgId: cfg.active_account.target.org_id,
       email,
@@ -16203,49 +16266,6 @@ function membersCommand() {
     }
     console.log(import_picocolors17.default.green(`Invitation sent to ${email} as ${role}.`));
   });
-  cmd.command("requests").description("List pending access requests").option("--json", "Output as JSON").action(async (opts) => {
-    const cfg = requireConfig8();
-    const client = createTypedClient(cfg);
-    const data = await client.invitations.getInvitations.query();
-    if (opts.json) {
-      printResult(data, opts);
-      return;
-    }
-    if (!data.items || data.items.length === 0) {
-      console.log(import_picocolors17.default.dim("No pending access requests."));
-      return;
-    }
-    printTable(["Email", "Org"], data.items.map((r2) => [
-      r2.email ?? "—",
-      r2.org?.name ?? "—"
-    ]), { rawData: data });
-  });
-  cmd.command("approve").description("Approve an access request").argument("<email>", "Email of requester").option("--json", "Output as JSON").action(async (email, opts) => {
-    const cfg = requireConfig8();
-    const client = createTypedClient(cfg);
-    await client.invitations.acceptInvitation.mutate({
-      orgId: cfg.active_account.target.org_id,
-      email
-    });
-    if (opts.json) {
-      printResult({ success: true, email, action: "approved" }, opts);
-      return;
-    }
-    console.log(import_picocolors17.default.green(`Access request from ${email} approved.`));
-  });
-  cmd.command("deny").description("Deny an access request").argument("<email>", "Email of requester").option("--json", "Output as JSON").action(async (email, opts) => {
-    const cfg = requireConfig8();
-    const client = createTypedClient(cfg);
-    await client.invitations.rejectInvitation.mutate({
-      orgId: cfg.active_account.target.org_id,
-      email
-    });
-    if (opts.json) {
-      printResult({ success: true, email, action: "denied" }, opts);
-      return;
-    }
-    console.log(import_picocolors17.default.green(`Access request from ${email} denied.`));
-  });
   return cmd;
 }
 
@@ -16459,7 +16479,7 @@ function reviewCommand() {
     console.log("");
     console.log(change.diff);
   });
-  cmd.command("edit").description("Edit a pending review item in place (doc body/title, or draft reply body)").argument("<id>", "Review item ID (from `dosu review list`)").option("--title <title>", "New title").option("--body <markdown>", "New body (markdown)").option("--body-file <path>", "Read body from file").option("--json", "Output as JSON").action(async (id, opts) => {
+  cmd.command("edit").description("Edit a pending doc or Dosu App draft in place").argument("<id>", "Review item ID (from `dosu review list`)").option("--title <title>", "New title").option("--body <markdown>", "New body (markdown)").option("--body-file <path>", "Read body from file").option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig10();
     const client = createTypedClient(cfg);
     if (opts.body !== undefined && opts.bodyFile !== undefined) {
@@ -16484,6 +16504,10 @@ function reviewCommand() {
         console.error(import_picocolors19.default.red("Draft replies support --body only (no --title)."));
         process.exit(1);
       }
+      if (body === undefined) {
+        console.error(import_picocolors19.default.red("Draft replies require --body or --body-file."));
+        process.exit(1);
+      }
       await requireDraft(client, id);
       await client.messages.saveDraft.mutate({
         messageId: bareMessageId(id),
@@ -16588,7 +16612,7 @@ function reviewCommand() {
       console.log(import_picocolors19.default.green(`Review ${name}: ${truncateId(id)}`));
     });
   }
-  cmd.command("revert").description("Revert a doc change to pending review (not supported for draft replies)").argument("<id>", "Review item ID (from `dosu review list`)").option("--json", "Output as JSON").action(async (id, opts) => {
+  cmd.command("revert").description("Reopen a previously accepted or declined doc change for review").argument("<id>", "Page version ID (not a pending item from `dosu review list`)").option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig10();
     const client = createTypedClient(cfg);
     if (isDraftId(id)) {
@@ -16681,6 +16705,9 @@ function sourcesCommand() {
     console.log(import_picocolors20.default.green(`Data source sync triggered for ${id.slice(0, 8)}.`));
   });
   cmd.command("update").description("Update a data source").argument("<id>", "Data source ID").option("--name <name>", "New name").option("--description <desc>", "New description").option("--json", "Output as JSON").action(async (id, opts) => {
+    if (opts.name === undefined && opts.description === undefined) {
+      throw new Error("Specify at least one of --name or --description.");
+    }
     const cfg = requireConfig11();
     const client = createTypedClient(cfg);
     const result = await client.dataSource.update.mutate({
@@ -16709,8 +16736,37 @@ function sourcesCommand() {
 
 // src/commands/suggest.ts
 init_commander();
-init_trpc();
 var import_picocolors21 = __toESM(require_picocolors(), 1);
+
+// src/client/supabase.ts
+async function listSpaceDataSourceIds(config, spaceId) {
+  const accessToken = config.active_account?.session.access_token;
+  if (!accessToken)
+    throw new Error("Not logged in. Run 'dosu login' first.");
+  const endpoint = new URL("/rest/v1/space_data_source", getSupabaseURL());
+  endpoint.searchParams.set("select", "data_source_id");
+  endpoint.searchParams.set("space_id", `eq.${spaceId}`);
+  const response = await fetch(endpoint, {
+    headers: {
+      apikey: getSupabaseAnonKey(),
+      Authorization: `Bearer ${accessToken}`
+    }
+  });
+  if (!response.ok) {
+    throw new Error(`Failed to list this deployment's data sources (status ${response.status}).`);
+  }
+  const rows = await response.json();
+  if (!Array.isArray(rows) || rows.some((row) => !isDataSourceLink(row))) {
+    throw new Error("Supabase returned an invalid space_data_source response.");
+  }
+  return rows.map((row) => row.data_source_id);
+}
+function isDataSourceLink(value) {
+  return value !== null && typeof value === "object" && "data_source_id" in value && typeof value.data_source_id === "string";
+}
+
+// src/commands/suggest.ts
+init_trpc();
 function requireConfig12() {
   const cfg = requireLoginConfig();
   if (!cfg.active_account?.target?.space_id) {
@@ -16755,15 +16811,10 @@ function suggestCommand() {
     const cfg = requireConfig12();
     const client = createTypedClient(cfg);
     const ksId = await getKnowledgeStoreId3(client, cfg.active_account.target.space_id);
-    if (!cfg.active_account?.target?.org_id) {
-      console.error(import_picocolors21.default.red("Missing org config. Run 'dosu setup' to reconfigure."));
-      process.exit(1);
+    const dataSourceIds = await listSpaceDataSourceIds(cfg, cfg.active_account.target.space_id);
+    if (dataSourceIds.length === 0) {
+      throw new Error("No data sources are connected to the active deployment.");
     }
-    const dataSources = await client.dataSource.list.query({
-      org_id: cfg.active_account?.target?.org_id,
-      excluded_provider_slugs: []
-    });
-    const dataSourceIds = dataSources.map((ds) => ds.id).filter((id) => id !== null);
     const result = await client.suggestedDoc.generate.mutate({
       knowledgeStoreId: ksId,
       dataSourceIds
@@ -16866,7 +16917,11 @@ function requireConfig13() {
 }
 function threadsCommand() {
   const cmd = new Command("threads").description("List and manage conversation threads");
-  cmd.command("list").description("List threads").option("--status <status>", "Filter by status: pending, resolved, archived").option("--search <query>", "Search threads").option("--limit <n>", "Maximum results (default: 20)", "20").option("--json", "Output as JSON").action(async (opts) => {
+  cmd.command("list").description("List threads").addOption(new Option("--status <status>", "Filter by status").choices([
+    "pending",
+    "resolved",
+    "archived"
+  ])).option("--search <query>", "Search threads").addOption(new Option("--limit <n>", "Maximum results (default: 20; max: 100)").argParser(positiveIntegerAtMost(100)).default(20)).option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig13();
     const client = createTypedClient(cfg);
     const input = {
@@ -16884,7 +16939,7 @@ function threadsCommand() {
       search: undefined,
       space_id: cfg.active_account.target.space_id,
       workspaces: undefined,
-      limit: Math.min(Number.parseInt(opts.limit ?? "20", 10), 100)
+      limit: opts.limit
     };
     if (opts.search)
       input.search = opts.search;
@@ -16916,14 +16971,14 @@ function threadsCommand() {
       formatDate(t2.created_at)
     ]), { rawData: threads });
   });
-  cmd.command("get").description("View a thread and its messages").argument("<id>", "Thread ID").option("--limit <n>", "Number of messages to show (default: 20)", "20").option("--json", "Output as JSON").action(async (id, opts) => {
+  cmd.command("get").description("View a thread and its messages").argument("<id>", "Thread ID").addOption(new Option("--limit <n>", "Number of messages to show (default: 20; -1 for all)").argParser(messageLimit).default(20)).option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig13();
     const client = createTypedClient(cfg);
     const [thread, messagesData] = await Promise.all([
       client.thread.get.query(id),
       client.messages.list.query({
         thread_id: id,
-        limit: Number.parseInt(opts.limit ?? "20", 10)
+        limit: opts.limit
       })
     ]);
     if (opts.json) {
@@ -16994,7 +17049,7 @@ async function getKnowledgeStoreId4(client, spaceId) {
   return store.id;
 }
 function topicsCommand() {
-  const cmd = new Command("topics").alias("tags").description("Browse Topics in your knowledge base");
+  const cmd = new Command("topics").description("Browse Topics in your knowledge base");
   cmd.command("list").description("List all topics").option("--json", "Output as JSON").action(async (opts) => {
     const cfg = requireConfig14();
     const client = createTypedClient(cfg);
@@ -17016,7 +17071,7 @@ function topicsCommand() {
       t2.description ?? "—"
     ]), { rawData: topics });
   });
-  cmd.command("pages").description("List pages with a specific topic").argument("<topic-id>", "Topic ID").option("--search <query>", "Search within the topic's pages").option("--limit <n>", "Maximum results", "10").option("--json", "Output as JSON").action(async (topicId, opts) => {
+  cmd.command("pages").description("List pages with a specific topic").argument("<topic-id>", "Topic ID").option("--search <query>", "Search within the topic's pages").addOption(new Option("--limit <n>", "Maximum results").argParser(positiveInteger).default(10)).option("--json", "Output as JSON").action(async (topicId, opts) => {
     const cfg = requireConfig14();
     const client = createTypedClient(cfg);
     const ksId = await getKnowledgeStoreId4(client, cfg.active_account.target.space_id);
@@ -17024,7 +17079,7 @@ function topicsCommand() {
       knowledge_store_id: ksId,
       topic_id: topicId,
       searchTerm: opts.search,
-      limit: Number.parseInt(opts.limit ?? "10", 10)
+      limit: opts.limit
     });
     const pages = result.data;
     if (opts.json) {
@@ -17898,4 +17953,4 @@ execute().catch((err) => {
   process.exit(1);
 });
 
-//# debugId=20790a5a-a63f-06a4-6475-6e2164756e21
+//# debugId=0e1efd83-4f86-ddb7-6475-6e2164756e21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dosu/cli",
-  "version": "0.49.2",
+  "version": "0.49.4",
   "type": "module",
   "description": "Dosu CLI - Manage MCP servers for AI tools",
   "license": "MIT",