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

package/index.js

@@ -16,6 +16,9 @@ import {
   sanitizeField,
   assertGitRef,
   assertGitRemote,
+  getEffectiveIngestMaxBytes,
+  FORGE_INGEST_MAX_BYTES_ENV,
+  throwIfStaleHeadByNumber,
 } from '@remogram/core';
 import { provider as giteaApi } from '@remogram/provider-gitea-api';
 import { provider as githubApi } from '@remogram/provider-github-api';
@@ -57,6 +60,11 @@ function handleError(err, ctx, asJson) {
     remoteName: 'origin',
     repoId: 'unknown/unknown',
   };
+  if (err.staleHeadPacket) {
+    output(forgePacket(err.staleHeadPacket.type, baseCtx, err.staleHeadPacket.body, fe), asJson);
+    process.exitCode = 1;
+    return;
+  }
   output(forgeErrorPacket(baseCtx, fe), asJson);
   process.exitCode = 1;
 }
@@ -87,6 +95,24 @@ function contextFromConfig(config, cwd, parsed = null) {
   };
 }
 
+function finalizeDoctorPacket(ctx, checks, providerCapabilities) {
+  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,
+  );
+}
+
 async function buildDoctorPacket(cwd, providers) {
   const checks = [];
   const configPath = findConfigPath(cwd);
@@ -98,11 +124,7 @@ async function buildDoctorPacket(cwd, providers) {
 
   if (!configPath) {
     checks.push(doctorCheck('config', 'fail', 'No .remogram.json found'));
-    return forgePacket(PACKET_TYPES.PROVIDER_DOCTOR, ctx, {
-      summary: doctorSummary(checks),
-      checks,
-      provider_capabilities: null,
-    });
+    return finalizeDoctorPacket(ctx, checks, null);
   }
 
   try {
@@ -112,22 +134,30 @@ async function buildDoctorPacket(cwd, providers) {
     checks.push(doctorCheck('config', 'pass', '.remogram.json is present and valid'));
   } catch (err) {
     checks.push(doctorCheck('config', 'fail', err.forgeError?.message || err.message));
-    return forgePacket(PACKET_TYPES.PROVIDER_DOCTOR, ctx, {
-      summary: doctorSummary(checks),
-      checks,
-      provider_capabilities: null,
-    });
+    return finalizeDoctorPacket(ctx, checks, null);
   }
 
   const provider = providers[config.provider];
   if (!provider) {
     checks.push(doctorCheck('provider', 'fail', `Unsupported provider: ${config.provider}`));
   } else {
-    checks.push(doctorCheck('provider', 'pass', `${config.provider} is registered`));
     if (typeof provider.providerCapabilities === 'function') {
       providerCapabilities = await provider.providerCapabilities(ctx);
+      const stubProvider =
+        providerCapabilities.commands?.length > 0
+        && providerCapabilities.commands.every((command) => command.implemented === false);
+      checks.push(
+        doctorCheck(
+          'provider',
+          stubProvider ? 'warn' : 'pass',
+          stubProvider
+            ? `${config.provider} is not fully supported in v1; use an *-api provider`
+            : `${config.provider} is registered`,
+        ),
+      );
       checks.push(doctorCheck('capabilities', 'pass', 'Provider capabilities are available'));
     } else {
+      checks.push(doctorCheck('provider', 'pass', `${config.provider} is registered`));
       checks.push(doctorCheck('capabilities', 'fail', 'Provider capabilities are not implemented'));
     }
   }
@@ -189,13 +219,40 @@ async function buildDoctorPacket(cwd, providers) {
     }
   }
 
+  const { bytes: ingestCapBytes, envOverride: ingestEnvOverride, invalidEnv: ingestInvalidEnv } =
+    getEffectiveIngestMaxBytes();
+  if (ingestInvalidEnv) {
+    checks.push(
+      doctorCheck(
+        'forge_ingest_cap',
+        'warn',
+        `${FORGE_INGEST_MAX_BYTES_ENV} is invalid; using default 8192 bytes`,
+        { effective_bytes: ingestCapBytes, env_override: false },
+      ),
+    );
+  } else if (ingestEnvOverride) {
+    checks.push(
+      doctorCheck(
+        'forge_ingest_cap',
+        'warn',
+        `${FORGE_INGEST_MAX_BYTES_ENV} overrides default ingest cap; agent-safe guarantee is weakened`,
+        { effective_bytes: ingestCapBytes, env_override: true },
+      ),
+    );
+  } else {
+    checks.push(
+      doctorCheck(
+        'forge_ingest_cap',
+        'pass',
+        'Forge HTTP ingest cap is default 8192 bytes',
+        { effective_bytes: ingestCapBytes, env_override: false },
+      ),
+    );
+  }
+
   checks.push(doctorCheck('api_reachability', 'skipped', 'Live API reachability is not checked by default'));
 
-  return forgePacket(PACKET_TYPES.PROVIDER_DOCTOR, ctx, {
-    summary: doctorSummary(checks),
-    checks,
-    provider_capabilities: providerCapabilities,
-  });
+  return finalizeDoctorPacket(ctx, checks, providerCapabilities);
 }
 
 export async function runCli(argv, options = {}) {
@@ -225,7 +282,9 @@ export async function runCli(argv, options = {}) {
   const [group, sub] = positional;
 
   if (group === 'doctor' && sub == null) {
-    output(await buildDoctorPacket(cwd, providers), asJson);
+    const packet = await buildDoctorPacket(cwd, providers);
+    output(packet, asJson);
+    if (!packet.ok) process.exitCode = 1;
     return;
   }
 
@@ -284,7 +343,15 @@ export async function runCli(argv, options = {}) {
           forgeError: forgeError(ERROR_CODES.INVALID_ARGS, '--number required for pr view'),
         });
       }
-      packet = forgePacket(PACKET_TYPES.PR_STATUS, ctx, await provider.prView(ctx, { number }));
+      const body = await provider.prView(ctx, { number });
+      throwIfStaleHeadByNumber(
+        ctx,
+        PACKET_TYPES.PR_STATUS,
+        body,
+        body.head_ref,
+        body.head_sha,
+      );
+      packet = forgePacket(PACKET_TYPES.PR_STATUS, ctx, body);
     } else if (group === 'pr' && sub === 'checks') {
       const number = parsePositiveInt(flags.number, '--number');
       if (number == null && !flags.ref) {
@@ -293,6 +360,16 @@ export async function runCli(argv, options = {}) {
         });
       }
       if (flags.ref) assertGitRef(flags.ref, '--ref');
+      if (number != null && !flags.ref) {
+        const view = await provider.prView(ctx, { number });
+        throwIfStaleHeadByNumber(
+          ctx,
+          PACKET_TYPES.PR_CHECKS,
+          { head_sha: view.head_sha },
+          view.head_ref,
+          view.head_sha,
+        );
+      }
       packet = forgePacket(
         PACKET_TYPES.PR_CHECKS,
         ctx,

package/package.json

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