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

package/index.js

@@ -16,6 +16,8 @@ import {
   sanitizeField,
   assertGitRef,
   assertGitRemote,
+  getEffectiveIngestMaxBytes,
+  FORGE_INGEST_MAX_BYTES_ENV,
 } from '@remogram/core';
 import { provider as giteaApi } from '@remogram/provider-gitea-api';
 import { provider as githubApi } from '@remogram/provider-github-api';
@@ -87,6 +89,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 +118,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 +128,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 implemented 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 +213,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 +276,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;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remogram/cli",
-  "version": "0.1.0-beta.0",
+  "version": "0.1.0-beta.1",
   "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.1",
+    "@remogram/provider-gitea-api": "0.1.0-beta.1",
+    "@remogram/provider-github-api": "0.1.0-beta.1",
+    "@remogram/provider-gitlab-api": "0.1.0-beta.1",
+    "@remogram/provider-gitea-tea": "0.1.0-beta.1",
+    "@remogram/provider-github-gh": "0.1.0-beta.1"
   }
 }