scripter-x 1.0.23 → 1.0.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scripter-x",
-  "version": "1.0.23",
+  "version": "1.0.24",
   "description": "ScripterX — local Flipkart session extractor (runs on your residential IP, syncs to marthunt)",
   "type": "module",
   "bin": {

package/src/commands.js

@@ -3,7 +3,7 @@
 // stdin/stdout). Each command: async (io, api, args) => void.
 import * as config from './config.js';
 import { ApiClient, AuthError } from './api.js';
-import { saveSessions, CANCEL } from './util.js';
+import { saveSessions, saveSessionsLocal, CANCEL } from './util.js';
 import * as tp from './providers/tempotp.js';
 import * as oc from './providers/otpcart.js';
 import * as kuku from './providers/kuku.js';
@@ -205,18 +205,27 @@ export async function run(io, api, args = {}) {
 
   io.print(`  ✓ done — ${stats.succeeded} succeeded · ${stats.failed} failed · ${stats.cancelled} cancelled · ₹${stats.charges} spent`);
   if (stats.succeeded > 0) {
-    let saved = null, lastErr = null;
-    for (let a = 0; a < 5 && !saved; a++) {
-      if (a > 0) await new Promise((r) => setTimeout(r, 1000));
-      try { saved = await saveSessions(api, cid, { campaignName: name }); } catch (e) { lastErr = e; }
+    // PRIMARY: save LOCALLY from the worker's in-memory buffer — never depends on the
+    // server, so an expired dashboard JWT can't lose the JSONs the user paid for.
+    let saved = null;
+    try {
+      saved = saveSessionsLocal(worker.results, cid, { campaignName: name, out: args.out, checkMinutes });
+    } catch (e) {
+      io.print(`  ! local save error: ${e.message}`, 'danger');
+    }
+    // FALLBACK: only if the local buffer was somehow empty, try the server export.
+    if (!saved && worker.results.length === 0) {
+      for (let a = 0; a < 3 && !saved; a++) {
+        if (a > 0) await new Promise((r) => setTimeout(r, 1000));
+        try { saved = await saveSessions(api, cid, { campaignName: name, out: args.out }); } catch { /* */ }
+      }
     }
     if (saved) {
       io.print('  ✓ saved session(s) to:');
-      for (const res of saved) {
-        io.print(`     ${res.path} (${res.count} sessions)`, 'accent');
-      }
+      for (const res of saved) io.print(`     ${res.label}  ${res.path} (${res.count} sessions)`, 'accent');
+    } else {
+      io.print('  ! could not write the session file — check disk permissions', 'danger');
     }
-    else io.print(`  ! couldn't auto-save${lastErr ? ` (${lastErr.message})` : ''} — run \`export ${name}\` to retry`);
   } else io.print('  ◉ no sessions extracted — nothing to save.');
   if (worker.logPath) io.print(`  ◉ log saved to: ${worker.logPath}`);
 }

package/src/util.js

@@ -31,34 +31,19 @@ export const log = {
   err: (m) => console.log(`  ${paint.err('✗')} ${m}`),
 };
 
-// Resolve + write a campaign's sessions to a file. Returns {path, count} or null
-// (null when there are no extracted sessions). Mirrors the Python CLI's behavior:
-//   out=<file>  → that file;  out=<dir>/ → default name inside;  no out → ~/Downloads/scripterx/
-export async function saveSessions(api, cid, { campaignName, out } = {}) {
-  let name = (campaignName || cid.slice(0, 8)).replace(/[^A-Za-z0-9._-]+/g, '-').replace(/^-+|-+$/g, '') || cid.slice(0, 8);
-  const accounts = await api.exportCampaign(cid);
+// Core: partition an accounts array by Minutes status and write the JSON file(s).
+// accounts = [{ session, coupon_eligible? }]. Pure local I/O — no server.
+// Returns [{label, path, count}] or null.
+function writeAccounts(accounts, { name, out, checkMinutes } = {}) {
   if (!accounts || !accounts.length) return null;
 
-  let checkMinutes = false;
-  try {
-    const { campaign } = await api.getCampaign(cid);
-    checkMinutes = !!campaign.check_minutes;
-  } catch (e) {
-    // ignore
-  }
-
   // Resolve base directory and optional forced filename.
   let baseDir, forcedName = null;
-  const defaultName = `${name}-${cid.slice(0, 8)}.json`;
   if (out) {
     const expanded = out.startsWith('~') ? join(homedir(), out.slice(1)) : out;
     const isDir = (existsSync(expanded) && statSync(expanded).isDirectory()) || /[/\\]$/.test(out);
-    if (isDir) {
-      baseDir = expanded;
-    } else {
-      baseDir = dirname(expanded);
-      forcedName = expanded.split(/[/\\]/).pop();
-    }
+    if (isDir) baseDir = expanded;
+    else { baseDir = dirname(expanded); forcedName = expanded.split(/[/\\]/).pop(); }
   } else {
     const downloads = join(homedir(), 'Downloads');
     const base = existsSync(downloads) ? downloads : homedir();
@@ -67,22 +52,19 @@ export async function saveSessions(api, cid, { campaignName, out } = {}) {
 
   function write(suffix, sessionsToWrite) {
     if (!sessionsToWrite || !sessionsToWrite.length) return null;
-    let fname;
-    if (forcedName) {
-      fname = suffix === '' ? forcedName : forcedName.replace(/\.json$/i, `${suffix}.json`);
-    } else {
-      fname = `${name}-${cid.slice(0, 8)}${suffix}.json`;
-    }
+    const fname = forcedName
+      ? (suffix === '' ? forcedName : forcedName.replace(/\.json$/i, `${suffix}.json`))
+      : `${name}${suffix}.json`;
     const dest = join(baseDir, fname);
     mkdirSync(baseDir, { recursive: true });
     writeFileSync(dest, JSON.stringify(sessionsToWrite, null, 2));
     return dest;
   }
 
-  // Partition by coupon_eligible when minutes check was requested.
+  // Partition by coupon_eligible when the Minutes check was requested.
   const minutesFree = [];   // coupon_eligible=true  → no Minutes order, ₹100 coupon available
   const minutesUsed = [];   // coupon_eligible=false → already placed a Minutes order
-  const unchecked = [];      // coupon_eligible absent → check wasn't done for this account
+  const unchecked = [];      // coupon_eligible absent → check wasn't done
 
   let hasChecked = false;
   for (const a of accounts) {
@@ -101,10 +83,8 @@ export async function saveSessions(api, cid, { campaignName, out } = {}) {
   if (hasChecked || (checkMinutes && (minutesFree.length || minutesUsed.length))) {
     const destFree = write('-minutes-free', minutesFree);
     if (destFree) results.push({ label: '🟢 minutes-free (₹100 coupon)', path: destFree, count: minutesFree.length });
-
     const destUsed = write('-minutes-used', minutesUsed);
     if (destUsed) results.push({ label: '🔴 minutes-used (coupon gone)', path: destUsed, count: minutesUsed.length });
-
     if (unchecked.length) {
       const destUnk = write('-unchecked', unchecked);
       if (destUnk) results.push({ label: '⚪ unchecked', path: destUnk, count: unchecked.length });
@@ -114,6 +94,27 @@ export async function saveSessions(api, cid, { campaignName, out } = {}) {
     const dest = write('', allSessions);
     if (dest) results.push({ label: 'combined', path: dest, count: allSessions.length });
   }
-
   return results.length ? results : null;
 }
+
+function campaignFileStem(campaignName, cid) {
+  const name = (campaignName || cid.slice(0, 8)).replace(/[^A-Za-z0-9._-]+/g, '-').replace(/^-+|-+$/g, '') || cid.slice(0, 8);
+  return `${name}-${cid.slice(0, 8)}`;
+}
+
+// LOCAL autosave — the primary save path. Writes directly from the worker's
+// in-memory results, so a campaign's JSONs are ALWAYS saved even if the dashboard
+// JWT expired mid-run (no server round-trip). Returns [{label,path,count}] or null.
+export function saveSessionsLocal(accounts, cid, { campaignName, out, checkMinutes } = {}) {
+  return writeAccounts(accounts, { name: campaignFileStem(campaignName, cid), out, checkMinutes });
+}
+
+// Resolve + write a campaign's sessions to a file, fetching from the SERVER.
+// Used by the standalone `export` command (no local buffer available there).
+export async function saveSessions(api, cid, { campaignName, out } = {}) {
+  const accounts = await api.exportCampaign(cid);
+  if (!accounts || !accounts.length) return null;
+  let checkMinutes = false;
+  try { const { campaign } = await api.getCampaign(cid); checkMinutes = !!campaign.check_minutes; } catch { /* ignore */ }
+  return writeAccounts(accounts, { name: campaignFileStem(campaignName, cid), out, checkMinutes });
+}

package/src/worker.js

@@ -39,6 +39,10 @@ export class Worker {
     this.slots = {};
     this.seq = 0;
     this.stopped = false;
+    // Local buffer of every successful session — the source of truth for autosave.
+    // We NEVER depend on a server round-trip to save: even if the dashboard JWT
+    // expires mid-run, the file still gets written from here.
+    this.results = [];
     this.logger = new CampaignLogger({ campaignId, name: name || campaignId, provider: provider.name });
   }
 
@@ -69,6 +73,21 @@ export class Worker {
     else { this.stats.failed++; this.stats.charges += res.cost || 0; } // failed w/ SMS = charged by provider
     this.onEvent('progress', this.stats);
     this.onEvent('row', res);
+    // Buffer every successful session LOCALLY first — autosave reads from here, so a
+    // failed/expired server ingest can never lose the JSON the user paid for.
+    if (res.status === 'success' && res.session) {
+      this.results.push({
+        id_no: res.id_no, mobile: res.mobile || '', session: res.session,
+        minutes_checked: !!res.minutes_checked,
+        coupon_eligible: res.coupon_eligible ?? undefined,
+        has_minutes_order: res.has_minutes_order ?? undefined,
+        minutes_order_count: res.minutes_order_count ?? undefined,
+        minutes_orders: res.minutes_orders ?? undefined,
+        linked_email: res.linked_email ?? undefined,
+        email_linked: res.email_linked ?? undefined,
+        email_reason: res.email_reason ?? undefined,
+      });
+    }
     try {
       await this.api.ingestAccount(this.cid, {
         id_no: res.id_no, mobile: res.mobile || '', status: res.status, cost: res.cost || 0,