badgr-cli 1.1.2 → 1.1.4

package/src/errors.js

@@ -17,7 +17,12 @@ export const CATALOG = {
   // ── Capacity ──────────────────────────────────────────────────────────────
 
   NO_CAPACITY: {
+    // Prefer the backend's specific detail (e.g. "CPU launch is not
+    // configured on this backend (missing HETZNER_API_TOKEN)") over the
+    // generic template — a real config/provider reason is far more
+    // actionable than "not available right now" when that's not why.
     message: (ctx) =>
+      ctx.server_message ||
       `No ${ctx.gpu || 'GPU'} available${ctx.region ? ` in ${ctx.region}` : ''} right now.`,
     billing:  'never_started',
     retried:  false,

package/src/fallback.js

@@ -6,6 +6,13 @@ import { CATALOG, formatCliError } from './errors.js';
 /** Rates above this threshold trigger a visible warning when no --max-cost is set. */
 export const HIGH_RATE_THRESHOLD = 3.00;
 
+// ── badgr run --smoke defaults ────────────────────────────────────────────────
+// Hardcoded default smoke caps, not a config system — --max-cost/--max-runtime
+// still override them when passed explicitly (see run.js). The goal is to stop
+// expensive local smoke tests by default, not build another configuration system.
+export const SMOKE_MAX_COST_USD = 0.25;
+export const SMOKE_MAX_RUNTIME_MINUTES = 10;
+
 /** Normalise --tier flag variants to '1' or '2'. */
 export function normalizeTier(tier) {
   return (tier === '2' || tier === 'tier2' || tier === 'tier-2') ? '2' : (tier || '1');
@@ -37,12 +44,16 @@ export class CapacityError extends Error {
  * @param {object}   labels            - { thing: 'job'|'endpoint', cmd: 'badgr run'|'badgr serve' }
  * @param {object}   [opts]
  * @param {boolean}  [opts.allowTier2Fallback=true] - set false to disable tier-2 expansion
+ * @param {boolean}  [opts.singleAttempt=false] - set true to skip both the same-tier
+ *   provider retry and tier-2 expansion, failing immediately on the first error
+ *   (badgr run --smoke's "one attempt" guarantee)
  */
 export async function callWithFallback(endpoint, callOpts, buildBody, effectiveTier, chalk, labels, opts = {}) {
   const { callApi } = await import('./api.js');
   const thing = labels?.thing ?? 'job';
   const cmd   = labels?.cmd   ?? 'badgr run';
-  const allowTier2Fallback = opts.allowTier2Fallback !== false; // default true
+  const singleAttempt = opts.singleAttempt === true;
+  const allowTier2Fallback = !singleAttempt && opts.allowTier2Fallback !== false; // default true
 
   // 220s: comfortably above backend's BADGR_PROVISION_TIMEOUT_SECONDS (default
   // 200s, itself set above deployment_service.py's 180s routing-search
@@ -82,7 +93,10 @@ export async function callWithFallback(endpoint, callOpts, buildBody, effectiveT
     if (key) {
       const ctx = {
         gpu:              d.filters?.gpu ?? d.gpu,
-        region:           d.filters?.region,
+        // Server uses the literal string "any" as a filters.region placeholder
+        // when no region was requested — don't surface that as if the user
+        // had actually asked for a region named "any" (e.g. "in any right now").
+        region:           d.filters?.region && d.filters.region !== 'any' ? d.filters.region : undefined,
         failure_category: d.failure_category,
         low_cost_failed:  d.low_cost_provider_failed,
         server_message:   d.message,
@@ -113,7 +127,7 @@ export async function callWithFallback(endpoint, callOpts, buildBody, effectiveT
   // Provider retry: PROVISIONING_FAILED means the selected provider couldn't launch the slot.
   // Retry once with prefer_different_provider so the backend routes to a different provider
   // (e.g. RunPod failed → try Vast.ai or Hyperstack) within the same max_cost budget.
-  if (d?.code === 'PROVISIONING_FAILED' || d?.code === 'PROVIDER_ADAPTER_ERROR') {
+  if (!singleAttempt && (d?.code === 'PROVISIONING_FAILED' || d?.code === 'PROVIDER_ADAPTER_ERROR')) {
     console.log(chalk.dim('\n  Provider unavailable — trying alternative provider...\n'));
     try {
       const retryBody = { ...buildBody(), prefer_different_provider: true };

package/src/commands/task.js

@@ -1,25 +0,0 @@
-import { launchCommand } from './launch.js';
-
-/**
- * badgr task "<description>" -- <command>
- *
- * The MVP version of "assign a task" — a thin label wrapper over
- * `badgr launch . -- <command>`. It does not queue, schedule, or track
- * anything beyond what `badgr launch` already does; the description is
- * printed for the human and otherwise discarded (the receipt/status page
- * still key off the deployment ID, same as any other launch).
- */
-export async function taskCommand(config, args, chalk) {
-  const description = args[0];
-  if (!description || description.startsWith('-')) {
-    console.error(chalk.red('\nUsage: badgr task "<description>" [badgr launch flags...] -- <command>\n'));
-    console.error(chalk.dim('  The description must come first, before any flags.'));
-    console.error(chalk.dim('  Example: badgr task "Run the Chromium tests and tell me what failed" --max-cost 1 -- npm run test:chromium'));
-    console.error(chalk.dim('  Example: badgr task "Fix the failing checkout test" --max-cost 1 -- claude -p "Fix the failing checkout test"\n'));
-    process.exitCode = 1;
-    return;
-  }
-
-  console.log(chalk.dim(`  Task: ${description}`));
-  return launchCommand(config, ['.', ...args.slice(1)], chalk);
-}