@remogram/cli 0.1.0-beta.2 → 0.1.0-beta.4

package/index.js

@@ -19,6 +19,9 @@ import {
   getEffectiveIngestMaxBytes,
   FORGE_INGEST_MAX_BYTES_ENV,
   throwIfStaleHeadByNumber,
+  FACT_INVENTORY_PACKET_TYPES,
+  forgeFactInventoryPacket,
+  assertWriteCommandConfigured,
 } from '@remogram/core';
 import { provider as giteaApi } from '@remogram/provider-gitea-api';
 import { provider as githubApi } from '@remogram/provider-github-api';
@@ -34,6 +37,13 @@ const PROVIDERS = {
   'github-gh': githubGh,
 };
 
+const REPEATABLE_FLAGS = new Set(['allowed_path']);
+
+function parseAllowedPathFlags(flags) {
+  if (flags.allowed_path == null) return undefined;
+  return Array.isArray(flags.allowed_path) ? flags.allowed_path : [flags.allowed_path];
+}
+
 function parsePositiveInt(value, name) {
   if (value == null) return undefined;
   const n = Number(value);
@@ -210,6 +220,22 @@ async function buildDoctorPacket(cwd, providers) {
       ),
     );
 
+    if (providerCapabilities.write_support) {
+      const providerWrites = (providerCapabilities.write_commands || []).filter(Boolean);
+      const configuredWrites = Array.isArray(config?.write_commands) ? config.write_commands : [];
+      const missing = providerWrites.filter((name) => !configuredWrites.includes(name));
+      checks.push(
+        doctorCheck(
+          'write_config',
+          missing.length ? 'warn' : 'pass',
+          missing.length
+            ? `Provider supports write commands but .remogram.json write_commands omits: ${missing.join(', ')}`
+            : 'Consumer write_commands matches provider write surface',
+          { provider_write_commands: providerWrites, configured_write_commands: configuredWrites },
+        ),
+      );
+    }
+
     if (!providerCapabilities.check_sources?.length) {
       checks.push(doctorCheck('checks', 'warn', 'Provider does not report forge check sources'));
     } else {
@@ -268,7 +294,13 @@ export async function runCli(argv, options = {}) {
     else if (arg.startsWith('--')) {
       const key = arg.slice(2).replace(/-/g, '_');
       const next = argv[i + 1];
-      if (next != null && !next.startsWith('--')) {
+      if (REPEATABLE_FLAGS.has(key)) {
+        if (!flags[key]) flags[key] = [];
+        if (next != null && !next.startsWith('--')) {
+          flags[key].push(next);
+          i += 1;
+        }
+      } else if (next != null && !next.startsWith('--')) {
         flags[key] = next;
         i += 1;
       } else {
@@ -336,6 +368,83 @@ export async function runCli(argv, options = {}) {
         ctx,
         await provider.refsCompare(ctx, flags.base, flags.head),
       );
+    } else if (group === 'refs' && sub === 'inventory') {
+      if (typeof provider.refsInventory !== 'function') {
+        throw Object.assign(new Error('refs inventory not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'refs inventory not implemented for provider',
+          ),
+        });
+      }
+      packet = forgeFactInventoryPacket(
+        FACT_INVENTORY_PACKET_TYPES.REF_INVENTORY,
+        ctx,
+        await provider.refsInventory(ctx),
+      );
+    } else if (group === 'cr' && sub === 'inventory') {
+      if (typeof provider.crInventory !== 'function') {
+        throw Object.assign(new Error('cr inventory not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'cr inventory not implemented for provider',
+          ),
+        });
+      }
+      const inventoryBody = await provider.crInventory(ctx, {
+          slice_ref: flags.slice_ref,
+          limit: parsePositiveInt(flags.limit, '--limit'),
+          sort: flags.sort,
+        });
+      if (inventoryBody.list_truncated === true) {
+        throw Object.assign(new Error('Open CR list incomplete'), {
+          forgeError: forgeError(
+            ERROR_CODES.INVENTORY_LIST_INCOMPLETE,
+            'Open change request list could not be proved complete within pagination bounds',
+            null,
+            {
+              inventory_list: {
+                entry_count: inventoryBody.entry_count,
+              },
+            },
+          ),
+        });
+      }
+      packet = forgeFactInventoryPacket(
+        FACT_INVENTORY_PACKET_TYPES.CR_INVENTORY_SLICE,
+        ctx,
+        inventoryBody,
+      );
+    } else if (group === 'cr' && sub === 'open') {
+      if (typeof provider.crOpen !== 'function') {
+        throw Object.assign(new Error('cr open not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'cr open not implemented for provider',
+          ),
+        });
+      }
+      if (!flags.head || !flags.base || !flags.title) {
+        throw Object.assign(new Error('--head, --base, and --title required'), {
+          forgeError: forgeError(
+            ERROR_CODES.INVALID_ARGS,
+            '--head, --base, and --title required for cr open',
+          ),
+        });
+      }
+      assertGitRef(flags.head, '--head');
+      assertGitRef(flags.base, '--base');
+      assertWriteCommandConfigured(ctx.config, 'cr_open');
+      packet = forgePacket(
+        PACKET_TYPES.CHANGE_REQUEST_OPENED,
+        ctx,
+        await provider.crOpen(ctx, {
+          head: flags.head,
+          base: flags.base,
+          title: flags.title,
+          body: flags.body,
+        }),
+      );
     } else if (group === 'pr' && sub === 'view') {
       const number = parsePositiveInt(flags.number, '--number');
       if (number == null) {
@@ -382,7 +491,15 @@ export async function runCli(argv, options = {}) {
           forgeError: forgeError(ERROR_CODES.INVALID_ARGS, '--number required for merge plan'),
         });
       }
-      packet = forgePacket(PACKET_TYPES.MERGE_PLAN, ctx, await provider.mergePlan(ctx, { number }));
+      const allowedPaths = parseAllowedPathFlags(flags);
+      packet = forgePacket(
+        PACKET_TYPES.MERGE_PLAN,
+        ctx,
+        await provider.mergePlan(ctx, {
+          number,
+          ...(allowedPaths ? { allowed_paths: allowedPaths } : {}),
+        }),
+      );
     } else if (group === 'sync' && sub === 'plan') {
       const remote = flags.remote || ctx.config.remote;
       assertGitRemote(remote, '--remote');
@@ -395,7 +512,7 @@ export async function runCli(argv, options = {}) {
       throw Object.assign(new Error(`Unknown command: ${positional.join(' ')}`), {
         forgeError: forgeError(
           ERROR_CODES.INVALID_ARGS,
-          'Unknown command. Try: provider capabilities, repo status, refs compare, pr view, pr checks, merge plan, sync plan',
+          'Unknown command. Try: provider capabilities, repo status, refs compare, refs inventory, cr inventory, cr open, pr view, pr checks, merge plan, sync plan',
         ),
       });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remogram/cli",
-  "version": "0.1.0-beta.2",
+  "version": "0.1.0-beta.4",
   "description": "Remogram forge boundary CLI",
   "type": "module",
   "license": "MIT",
@@ -27,11 +27,11 @@
     "node": ">=20"
   },
   "dependencies": {
-    "@remogram/core": "0.1.0-beta.2",
-    "@remogram/provider-gitea-api": "0.1.0-beta.2",
-    "@remogram/provider-github-api": "0.1.0-beta.2",
-    "@remogram/provider-gitlab-api": "0.1.0-beta.2",
-    "@remogram/provider-gitea-tea": "0.1.0-beta.2",
-    "@remogram/provider-github-gh": "0.1.0-beta.2"
+    "@remogram/core": "0.1.0-beta.4",
+    "@remogram/provider-gitea-api": "0.1.0-beta.4",
+    "@remogram/provider-github-api": "0.1.0-beta.4",
+    "@remogram/provider-gitlab-api": "0.1.0-beta.4",
+    "@remogram/provider-gitea-tea": "0.1.0-beta.4",
+    "@remogram/provider-github-gh": "0.1.0-beta.4"
   }
 }