@remogram/cli 0.1.0-beta.3 → 0.1.0-beta.5

package/index.js

@@ -21,6 +21,8 @@ import {
   throwIfStaleHeadByNumber,
   FACT_INVENTORY_PACKET_TYPES,
   forgeFactInventoryPacket,
+  assertWriteCommandConfigured,
+  parseSinceObservedAt,
 } from '@remogram/core';
 import { provider as giteaApi } from '@remogram/provider-gitea-api';
 import { provider as githubApi } from '@remogram/provider-github-api';
@@ -36,6 +38,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);
@@ -212,6 +221,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(', ')}. Add ids for Remogram CLI/MCP writes, or use forge/CI tooling for those actions outside Remogram.`
+            : '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 {
@@ -270,7 +295,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 {
@@ -361,12 +392,139 @@ export async function runCli(argv, options = {}) {
           ),
         });
       }
+      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,
-        await provider.crInventory(ctx, {
-          slice_ref: flags.slice_ref,
-          limit: parsePositiveInt(flags.limit, '--limit'),
+        inventoryBody,
+      );
+    } else if (group === 'cr' && sub === 'files') {
+      const number = parsePositiveInt(flags.number, '--number');
+      if (number == null) {
+        throw Object.assign(new Error('--number required'), {
+          forgeError: forgeError(ERROR_CODES.INVALID_ARGS, '--number required for cr files'),
+        });
+      }
+      if (typeof provider.crFiles !== 'function') {
+        throw Object.assign(new Error('cr files not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'cr files not implemented for provider',
+          ),
+        });
+      }
+      packet = forgePacket(
+        PACKET_TYPES.CR_FILES,
+        ctx,
+        await provider.crFiles(ctx, { number }),
+      );
+    } else if (group === 'cr' && sub === 'comments') {
+      const number = parsePositiveInt(flags.number, '--number');
+      if (number == null) {
+        throw Object.assign(new Error('--number required'), {
+          forgeError: forgeError(ERROR_CODES.INVALID_ARGS, '--number required for cr comments'),
+        });
+      }
+      if (typeof provider.crComments !== 'function') {
+        throw Object.assign(new Error('cr comments not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'cr comments not implemented for provider',
+          ),
+        });
+      }
+      packet = forgePacket(
+        PACKET_TYPES.CR_COMMENTS,
+        ctx,
+        await provider.crComments(ctx, { number }),
+      );
+    } else if (group === 'forge' && sub === 'changes') {
+      let sinceIso;
+      try {
+        sinceIso = parseSinceObservedAt(flags.since);
+      } catch (err) {
+        throw err;
+      }
+      if (typeof provider.forgeChanges !== 'function') {
+        throw Object.assign(new Error('forge changes not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'forge changes not implemented for provider',
+          ),
+        });
+      }
+      packet = forgePacket(
+        PACKET_TYPES.FORGE_CHANGES,
+        ctx,
+        await provider.forgeChanges(ctx, { since: sinceIso }),
+      );
+    } 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 === 'status' && sub === 'set') {
+      if (typeof provider.statusSet !== 'function') {
+        throw Object.assign(new Error('status set not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'status set not implemented for provider',
+          ),
+        });
+      }
+      assertWriteCommandConfigured(ctx.config, 'status_set');
+      packet = forgePacket(
+        PACKET_TYPES.COMMIT_STATUS_SET,
+        ctx,
+        await provider.statusSet(ctx, {
+          sha: flags.sha,
+          context: flags.context,
+          state: flags.state,
+          target_url: flags.target_url,
+          description: flags.description,
         }),
       );
     } else if (group === 'pr' && sub === 'view') {
@@ -415,7 +573,49 @@ 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 === 'branch' && sub === 'protection') {
+      const branchRef = flags.branch_ref;
+      if (!branchRef) {
+        throw Object.assign(new Error('--branch-ref required'), {
+          forgeError: forgeError(
+            ERROR_CODES.INVALID_ARGS,
+            '--branch-ref required for branch protection',
+          ),
+        });
+      }
+      assertGitRef(branchRef, '--branch-ref');
+      if (typeof provider.branchProtection !== 'function') {
+        throw Object.assign(new Error('branch protection not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'branch protection not implemented for provider',
+          ),
+        });
+      }
+      packet = forgePacket(
+        PACKET_TYPES.BRANCH_PROTECTION,
+        ctx,
+        await provider.branchProtection(ctx, { branchRef }),
+      );
+    } else if (group === 'whoami' && sub == null) {
+      if (typeof provider.whoami !== 'function') {
+        throw Object.assign(new Error('whoami not implemented for provider'), {
+          forgeError: forgeError(
+            ERROR_CODES.PROVIDER_UNSUPPORTED,
+            'whoami not implemented for provider',
+          ),
+        });
+      }
+      packet = forgePacket(PACKET_TYPES.PROVIDER_IDENTITY, ctx, await provider.whoami(ctx));
     } else if (group === 'sync' && sub === 'plan') {
       const remote = flags.remote || ctx.config.remote;
       assertGitRemote(remote, '--remote');
@@ -428,7 +628,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, refs inventory, cr inventory, pr view, pr checks, merge plan, sync plan',
+          'Unknown command. Try: provider capabilities, repo status, refs compare, refs inventory, cr inventory, cr files, cr comments, cr open, status set, forge changes, pr view, pr checks, merge plan, sync plan, whoami, branch protection',
         ),
       });
     }

package/package.json

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