@remogram/cli 0.1.0-beta.8 → 0.1.0-beta.9

package/cli-dispatch.js

@@ -11,6 +11,9 @@ import {
   forgeFactInventoryPacket,
   assertWriteCommandConfigured,
   parseSinceObservedAt,
+  decodeForgeChangesCursor,
+  paginateForgeChangesBody,
+  DEFAULT_FORGE_CHANGES_PAGE_SIZE,
   normalizeAllowedPaths,
   assertExpectedSha,
   buildMergeExecuteBeforeFacts,
@@ -21,6 +24,7 @@ import {
   buildMergeExecuteMergeFacts,
   mergeExecuteViewFacts,
   isOpenPrState,
+  bindIdempotencyScope,
 } from '@remogram/core';
 import { parseAllowedPathFlags, parsePositiveInt } from './cli-argv.js';
 
@@ -77,8 +81,9 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
       slice_ref: flags.slice_ref,
       limit: parsePositiveInt(flags.limit, '--limit'),
       sort: flags.sort,
+      cursor: flags.cursor,
     });
-    if (inventoryBody.list_truncated === true) {
+    if (inventoryBody.list_truncated === true && !flags.cursor) {
       throw Object.assign(new Error('Open CR list incomplete'), {
         forgeError: forgeError(
           ERROR_CODES.INVENTORY_LIST_INCOMPLETE,
@@ -141,7 +146,6 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
     );
   }
   if (group === 'forge' && sub === 'changes') {
-    const sinceIso = parseSinceObservedAt(flags.since);
     if (typeof provider.forgeChanges !== 'function') {
       throw Object.assign(new Error('forge changes not implemented for provider'), {
         forgeError: forgeError(
@@ -150,11 +154,19 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
         ),
       });
     }
-    return forgePacket(
-      PACKET_TYPES.FORGE_CHANGES,
-      ctx,
-      await provider.forgeChanges(ctx, { since: sinceIso }),
-    );
+    let sinceIso;
+    let cursorOffset = 0;
+    const pageLimit = parsePositiveInt(flags.limit, '--limit') ?? DEFAULT_FORGE_CHANGES_PAGE_SIZE;
+    if (flags.cursor) {
+      const decoded = decodeForgeChangesCursor(flags.cursor, { since: flags.since });
+      sinceIso = decoded.since;
+      cursorOffset = decoded.offset;
+    } else {
+      sinceIso = parseSinceObservedAt(flags.since);
+    }
+    const body = await provider.forgeChanges(ctx, { since: sinceIso });
+    const paginated = paginateForgeChangesBody(body, { offset: cursorOffset, limit: pageLimit });
+    return forgePacket(PACKET_TYPES.FORGE_CHANGES, ctx, paginated);
   }
   if (group === 'cr' && sub === 'open') {
     if (typeof provider.crOpen !== 'function') {
@@ -176,6 +188,9 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
     assertGitRef(flags.head, '--head');
     assertGitRef(flags.base, '--base');
     assertWriteCommandConfigured(ctx.config, 'cr_open');
+    const idempotencyFingerprint = flags.idempotency_key
+      ? bindIdempotencyScope(ctx.repo_id, flags.idempotency_key, [flags.head, flags.base])
+      : null;
     return forgePacket(
       PACKET_TYPES.CHANGE_REQUEST_OPENED,
       ctx,
@@ -184,6 +199,35 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
         base: flags.base,
         title: flags.title,
         body: flags.body,
+        idempotencyFingerprint,
+      }),
+    );
+  }
+  if (group === 'issue' && sub === 'open') {
+    if (typeof provider.issueOpen !== 'function') {
+      throw Object.assign(new Error('issue open not implemented for provider'), {
+        forgeError: forgeError(
+          ERROR_CODES.PROVIDER_UNSUPPORTED,
+          'issue open not implemented for provider',
+        ),
+      });
+    }
+    if (!flags.title) {
+      throw Object.assign(new Error('--title required'), {
+        forgeError: forgeError(ERROR_CODES.INVALID_ARGS, '--title required for issue open'),
+      });
+    }
+    assertWriteCommandConfigured(ctx.config, 'issue_open');
+    const idempotencyFingerprint = flags.idempotency_key
+      ? bindIdempotencyScope(ctx.repo_id, flags.idempotency_key, [flags.title])
+      : null;
+    return forgePacket(
+      PACKET_TYPES.ISSUE_OPENED,
+      ctx,
+      await provider.issueOpen(ctx, {
+        title: flags.title,
+        body: flags.body,
+        idempotencyFingerprint,
       }),
     );
   }
@@ -197,6 +241,13 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
       });
     }
     assertWriteCommandConfigured(ctx.config, 'status_set');
+    const idempotencyFingerprint = flags.idempotency_key
+      ? bindIdempotencyScope(ctx.repo_id, flags.idempotency_key, [
+          flags.sha,
+          flags.context,
+          flags.state,
+        ])
+      : null;
     return forgePacket(
       PACKET_TYPES.COMMIT_STATUS_SET,
       ctx,
@@ -206,6 +257,7 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
         state: flags.state,
         target_url: flags.target_url,
         description: flags.description,
+        idempotencyFingerprint,
       }),
     );
   }
@@ -312,7 +364,13 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
     let forgeHeadRefSha = null;
     const headRef = viewFacts.sourceBranchRef ? String(viewFacts.sourceBranchRef).trim() : '';
     if (!headRef && isOpenPrState(view.state)) {
-      const before = buildMergeExecuteBeforeFacts(view, checks, mergePlanBody, null);
+      const before = buildMergeExecuteBeforeFacts(
+      view,
+      checks,
+      mergePlanBody,
+      null,
+      ctx.mergePolicy,
+    );
       return forgePacket(
         PACKET_TYPES.CR_MERGE_BLOCKED,
         ctx,
@@ -327,7 +385,13 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
     }
     if (headRef) {
       if (typeof provider.branchHeadSha !== 'function') {
-        const before = buildMergeExecuteBeforeFacts(view, checks, mergePlanBody, null);
+        const before = buildMergeExecuteBeforeFacts(
+      view,
+      checks,
+      mergePlanBody,
+      null,
+      ctx.mergePolicy,
+    );
         return forgePacket(
           PACKET_TYPES.CR_MERGE_BLOCKED,
           ctx,
@@ -346,7 +410,13 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
       try {
         assertGitRef(headRef, 'head_ref');
       } catch (err) {
-        const before = buildMergeExecuteBeforeFacts(view, checks, mergePlanBody, null);
+        const before = buildMergeExecuteBeforeFacts(
+      view,
+      checks,
+      mergePlanBody,
+      null,
+      ctx.mergePolicy,
+    );
         return forgePacket(
           PACKET_TYPES.CR_MERGE_BLOCKED,
           ctx,
@@ -368,7 +438,16 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
           repoId: view.forge_source_repo_id ?? null,
         });
       } catch (err) {
-        const before = buildMergeExecuteBeforeFacts(view, checks, mergePlanBody, null);
+        if (err.forgeError?.code === ERROR_CODES.INVALID_ARGS) {
+          throw err;
+        }
+        const before = buildMergeExecuteBeforeFacts(
+      view,
+      checks,
+      mergePlanBody,
+      null,
+      ctx.mergePolicy,
+    );
         return forgePacket(
           PACKET_TYPES.CR_MERGE_BLOCKED,
           ctx,
@@ -387,13 +466,19 @@ export async function dispatchForgeCommand({ group, sub, flags, positional, ctx,
       }
     }
 
-    const before = buildMergeExecuteBeforeFacts(view, checks, mergePlanBody, forgeHeadRefSha);
+    const before = buildMergeExecuteBeforeFacts(
+      view,
+      checks,
+      mergePlanBody,
+      forgeHeadRefSha,
+      ctx.mergePolicy,
+    );
     const blockers = collectMergeExecuteBlockers(
       view,
       checks,
       mergePlanBody,
       expected,
-      { forgeHeadRefSha },
+      { forgeHeadRefSha, mergePolicy: ctx.mergePolicy },
     );
 
     if (blockers.length > 0) {

package/cli-doctor.js

@@ -16,6 +16,12 @@ import {
   getEffectiveIngestMaxBytes,
   FORGE_INGEST_MAX_BYTES_ENV,
   MAX_FORGE_INGEST_ENV_BYTES,
+  resolveMergePolicy,
+  ALLOW_MISSING_CHECKS_ENV,
+  ALLOW_PENDING_CHECKS_ENV,
+  buildWriteReadiness,
+  writeReadinessHasWarnings,
+  buildApiReachabilityCheck,
 } from '@remogram/core';
 import { contextFromConfig } from './cli-io.js';
 
@@ -34,25 +40,23 @@ export function doctorSummary(checks) {
   return 'pass';
 }
 
-function finalizeDoctorPacket(ctx, checks, providerCapabilities) {
+function finalizeDoctorPacket(ctx, checks, providerCapabilities, writeConfig = null) {
   const summary = doctorSummary(checks);
   const error =
     summary === 'fail'
       ? forgeError(ERROR_CODES.CONFIG_INVALID, 'Doctor checks failed')
       : null;
-  return forgePacket(
-    PACKET_TYPES.PROVIDER_DOCTOR,
-    ctx,
-    {
-      summary,
-      checks,
-      provider_capabilities: providerCapabilities,
-    },
-    error,
-  );
+  const body = {
+    summary,
+    checks,
+    provider_capabilities: providerCapabilities,
+  };
+  if (writeConfig) body.write_config = writeConfig;
+  return forgePacket(PACKET_TYPES.PROVIDER_DOCTOR, ctx, body, error);
 }
 
-export async function buildDoctorPacket(cwd, providers) {
+export async function buildDoctorPacket(cwd, providers, options = {}) {
+  const { live = false } = options;
   const checks = [];
   const configPath = findConfigPath(cwd);
   let loaded = null;
@@ -139,9 +143,12 @@ export async function buildDoctorPacket(cwd, providers) {
     }
   }
 
+  let writeConfig = null;
+
   if (providerCapabilities) {
     const envNames = providerCapabilities.auth_envs || [];
     const presentEnv = envNames.find((name) => Boolean(process.env[name])) || null;
+    const authPresent = Boolean(presentEnv);
     checks.push(
       doctorCheck(
         'auth',
@@ -152,17 +159,22 @@ export 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));
+      writeConfig = buildWriteReadiness(config, providerCapabilities, { authPresent });
+      const warn = writeReadinessHasWarnings(writeConfig);
+      const notReady = writeConfig.commands.filter(
+        (entry) => entry.provider_supported && !entry.ready,
+      );
+      const missingConfig = notReady.filter((entry) => !entry.configured).map((entry) => entry.id);
       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 },
+          warn ? 'warn' : 'pass',
+          warn
+            ? missingConfig.length
+              ? `Provider supports write commands but .remogram.json write_commands omits: ${missingConfig.join(', ')}. Add ids for Remogram CLI/MCP writes, or use forge/CI tooling for those actions outside Remogram.`
+              : 'One or more configured write commands are not ready (check auth or provider support)'
+            : 'All provider write commands are configured and ready',
+          writeConfig,
         ),
       );
     }
@@ -209,7 +221,46 @@ export async function buildDoctorPacket(cwd, providers) {
     );
   }
 
-  checks.push(doctorCheck('api_reachability', 'skipped', 'Live API reachability is not checked by default'));
+  const mergePolicy = resolveMergePolicy(config);
+  if (mergePolicy.allow_missing_checks || mergePolicy.allow_pending_checks) {
+    checks.push(
+      doctorCheck(
+        'merge_policy',
+        'warn',
+        'Merge policy relaxes check blockers for merge plan and merge execute',
+        {
+          allow_missing_checks: mergePolicy.allow_missing_checks,
+          allow_pending_checks: mergePolicy.allow_pending_checks,
+          source: mergePolicy.source,
+          env_names: [ALLOW_MISSING_CHECKS_ENV, ALLOW_PENDING_CHECKS_ENV],
+        },
+      ),
+    );
+  } else {
+    checks.push(
+      doctorCheck(
+        'merge_policy',
+        'pass',
+        'Default merge policy — missing and pending checks block merge',
+        {
+          allow_missing_checks: false,
+          allow_pending_checks: false,
+          source: mergePolicy.source,
+        },
+      ),
+    );
+  }
+
+  const hostBindingPass = checks.some(
+    (check) => check.name === 'host_binding' && check.status === 'pass',
+  );
+  const configPass = checks.some((check) => check.name === 'config' && check.status === 'pass');
+  checks.push(
+    await buildApiReachabilityCheck(ctx, provider, {
+      live,
+      prerequisitesPass: live && configPass && hostBindingPass && parsed != null,
+    }),
+  );
 
-  return finalizeDoctorPacket(ctx, checks, providerCapabilities);
+  return finalizeDoctorPacket(ctx, checks, providerCapabilities, writeConfig);
 }

package/cli-io.js

@@ -5,6 +5,7 @@ import {
   ERROR_CODES,
   normalizedForgeOrigin,
   trustedBaseUrl,
+  resolveMergePolicy,
 } from '@remogram/core';
 
 export function output(packet, asJson) {
@@ -47,5 +48,6 @@ export function contextFromConfig(config, cwd, parsed = null) {
   if (config.baseUrl && (!parsed || trustedBaseUrl(config, parsed.host))) {
     ctx.baseUrl = normalizedForgeOrigin(config);
   }
+  ctx.mergePolicy = resolveMergePolicy(config);
   return ctx;
 }

package/index.js

@@ -30,7 +30,7 @@ export async function runCli(argv, options = {}) {
   const [group, sub] = positional;
 
   if (group === 'doctor' && sub == null) {
-    const packet = await buildDoctorPacket(cwd, providers);
+    const packet = await buildDoctorPacket(cwd, providers, { live: flags.live === true });
     output(packet, asJson);
     if (!packet.ok) process.exitCode = 1;
     return;

package/package.json

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