scripter-x 1.0.26 → 1.0.27

package/package.json

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

package/src/ui/RunView.js

@@ -30,14 +30,35 @@ function SlotRow({ slot, mobile, phase, detail, wait }) {
   );
 }
 
-function ResultRow({ status, mobile, detail, coupon, cost }) {
-  const st = STATUS[status] || STATUS.pending;
+// Map a result to a clean { glyph, label, color, note } for display. Driven by the
+// worker's `category` so every outcome reads consistently (no "successed"/"success" mix).
+function resultDisplay({ status, category, detail }) {
+  const C = COLORS;
+  switch (category) {
+    case 'fresh':            return { glyph: '🆕', label: 'fresh account',  color: C.accent,  note: 'new account created' };
+    case 'coupon':           return { glyph: '✓',  label: 'success',        color: C.success, note: '₹100 coupon available' };
+    case 'no_coupon':        return { glyph: '✓',  label: 'success',        color: C.success, note: 'no ₹100 coupon' };
+    case 'extracted':        return { glyph: '✓',  label: 'success',        color: C.success, note: 'extracted' };
+    case 'timeout':          return { glyph: '⏱',  label: 'timeout',        color: C.warn,    note: 'no OTP in 90s' };
+    case 'cancelled_user':   return { glyph: '⊘',  label: 'cancelled',      color: C.warn,    note: 'cancelled by user' };
+    case 'wrong_otp':        return { glyph: '✗',  label: 'failed',         color: C.danger,  note: 'wrong OTP code' };
+    case 'goal_reached':     return { glyph: '•',  label: 'skipped',        color: C.muted,   note: 'goal reached' };
+    case 'failed':           return { glyph: '✗',  label: 'failed',         color: C.danger,  note: detail || 'failed' };
+    default: {
+      const st = STATUS[status] || STATUS.pending;
+      const note = status === 'success' ? 'extracted' : (detail || '');
+      return { glyph: st.glyph, label: status, color: st.color, note };
+    }
+  }
+}
+
+function ResultRow({ status, category, mobile, detail, cost }) {
+  const d = resultDisplay({ status, category, detail });
   const mob = (mobile || '—').slice(-10) || '—';
-  const text = status === 'success' ? (coupon ? '₹100 coupon' : 'extracted') : (detail || '');
   return h(Box, null,
     h(Box, { width: 13 }, h(Text, null, mob)),
-    h(Box, { width: 13 }, h(Text, { color: st.color }, `${st.glyph} ${status}`)),
-    h(Box, { flexGrow: 1 }, h(Text, { color: 'gray' }, text)),
+    h(Box, { width: 14 }, h(Text, { color: d.color }, `${d.glyph} ${d.label}`)),
+    h(Box, { flexGrow: 1 }, h(Text, { color: 'gray' }, d.note)),
     h(Box, { width: 7, justifyContent: 'flex-end' }, h(Text, { color: 'gray' }, cost ? `₹${cost}` : '')),
   );
 }
@@ -76,7 +97,7 @@ export function RunView({ controller }) {
       slotList.length ? slotList.map((sl) => h(SlotRow, { key: sl.slot, ...sl }))
         : h(Text, { color: 'gray' }, 'starting…')),
     h(Panel, { title: 'results' },
-      lastRows.length ? lastRows.map((r, i) => h(ResultRow, { key: i, status: r.status, mobile: r.mobile, detail: r.detail, coupon: r.coupon_eligible, cost: r.cost }))
+      lastRows.length ? lastRows.map((r, i) => h(ResultRow, { key: i, status: r.status, category: r.category, mobile: r.mobile, detail: r.detail, cost: r.cost }))
         : h(Text, { color: 'gray' }, 'waiting for first result…')),
     h(Box, { borderStyle: 'round', borderColor: COLORS.accent, paddingX: 1 },
       h(Text, { color: COLORS.success }, `✓ ${stats.succeeded}`),

package/src/util.js

@@ -61,36 +61,51 @@ function writeAccounts(accounts, { name, out, checkMinutes } = {}) {
     return dest;
   }
 
-  // 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
+  // Partition into the THREE buckets the user wants:
+  //   fresh    → brand-new account just created (signup)
+  //   coupon   → existing account with the ₹100 Minutes coupon available
+  //   noCoupon → existing account, coupon already used
+  // plus `unchecked` (Minutes wasn't checked). Categorisation prefers the explicit
+  // `category` set by the worker; falls back to is_new_account / coupon_eligible.
+  const fresh = [];
+  const coupon = [];
+  const noCoupon = [];
+  const unchecked = [];
 
   let hasChecked = false;
   for (const a of accounts) {
     const sess = a.session;
     if (!sess) continue;
+    const cat = a.category;
     const eligible = a.coupon_eligible;
-    if (eligible !== undefined && eligible !== null) {
+    if (cat === 'fresh' || a.is_new_account) {
+      fresh.push(sess);
+      hasChecked = true;
+    } else if (cat === 'coupon' || eligible === true) {
+      coupon.push(sess);
+      hasChecked = true;
+    } else if (cat === 'no_coupon' || eligible === false) {
+      noCoupon.push(sess);
       hasChecked = true;
-      (eligible ? minutesFree : minutesUsed).push(sess);
     } else {
       unchecked.push(sess);
     }
   }
 
   const results = [];
-  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 (hasChecked || (checkMinutes && (fresh.length || coupon.length || noCoupon.length))) {
+    const dFresh = write('-fresh-new-account', fresh);
+    if (dFresh) results.push({ label: '🆕 fresh new account', path: dFresh, count: fresh.length });
+    const dCoupon = write('-with-100-coupon', coupon);
+    if (dCoupon) results.push({ label: '🟢 ₹100 coupon available', path: dCoupon, count: coupon.length });
+    const dNo = write('-no-100-coupon', noCoupon);
+    if (dNo) results.push({ label: '🔴 no ₹100 coupon', path: dNo, count: noCoupon.length });
     if (unchecked.length) {
-      const destUnk = write('-unchecked', unchecked);
-      if (destUnk) results.push({ label: '⚪ unchecked', path: destUnk, count: unchecked.length });
+      const dUnk = write('-unchecked', unchecked);
+      if (dUnk) results.push({ label: '⚪ unchecked', path: dUnk, count: unchecked.length });
     }
   } else {
-    const allSessions = [...minutesFree, ...minutesUsed, ...unchecked];
+    const allSessions = [...fresh, ...coupon, ...noCoupon, ...unchecked];
     const dest = write('', allSessions);
     if (dest) results.push({ label: 'combined', path: dest, count: allSessions.length });
   }

package/src/worker.js

@@ -78,6 +78,8 @@ export class Worker {
     if (res.status === 'success' && res.session) {
       this.results.push({
         id_no: res.id_no, mobile: res.mobile || '', session: res.session,
+        category: res.category,                 // 'fresh' | 'coupon' | 'no_coupon' | 'extracted'
+        is_new_account: !!res.is_new_account,
         minutes_checked: !!res.minutes_checked,
         coupon_eligible: res.coupon_eligible ?? undefined,
         has_minutes_order: res.has_minutes_order ?? undefined,
@@ -152,18 +154,25 @@ export class Worker {
       this._emit(slot, { phase: 'done', detail: 'goal reached' });
       res.status = 'cancelled';
       res.detail = 'goal reached';
+      res.category = 'goal_reached';
       return res;
     }
 
     this._emit(slot, { mobile: '', phase: 'renting', detail: 'requesting a number', wait: 0 });
     const number = await this._rent(slot);
-    if (!number) { res.status = 'cancelled'; res.detail = 'stopped while renting'; return res; }
+    if (!number) {
+      res.status = 'cancelled';
+      res.detail = this.stopped ? 'cancelled by user' : 'stopped while renting';
+      res.category = this.stopped ? 'cancelled_user' : 'goal_reached';
+      return res;
+    }
     if (this.stats.succeeded >= this.requested) {
       this.log(`goal reached by another slot; releasing ${number.mobile} immediately`);
       this._release(number, Date.now(), slot);
       this._emit(slot, { phase: 'done', detail: 'goal reached' });
       res.status = 'cancelled';
       res.detail = 'goal reached';
+      res.category = 'goal_reached';
       return res;
     }
     const rentedAt = Date.now();
@@ -194,11 +203,13 @@ export class Worker {
       this._release(number, rentedAt, slot);
     };
 
-    const fail = (detail) => {
+    const fail = (detail, category) => {
       const charged = smsArrived.v;
       res.status = charged ? 'failed' : 'cancelled';
       res.cost = charged ? number.cost : 0;
       res.detail = detail;
+      // category: timeout (90s, no OTP), wrong_otp, send_blocked, or generic failed/cancelled
+      res.category = category || (res.status === 'failed' ? 'failed' : 'cancelled');
       this.log(`FAILED ${number.mobile}: ${detail} (charged: ${charged})`);
       releaseOnce(charged, 'failed');
       return res;
@@ -224,13 +235,14 @@ export class Worker {
       const deadline = this.provider.name === 'tempotp' ? TEMPOTP_DEADLINE : OTPCART_DEADLINE;
       const doResend = this.provider.name === 'otpcart';
       const otp = await this._awaitOtp(slot, stream, fk, number, deadline, doResend, smsArrived);
-      if (!otp) return fail(`no OTP within ${deadline / 1000}s — abandoned`);
+      if (!otp) return fail(`no OTP in ${deadline / 1000}s (timeout)`, 'timeout');
 
       // 3. VERIFY (already retries transients internally via _verify; re-verify on blockage)
       this._emit(slot, { phase: 'verify', detail: 'verifying OTP' });
       const { session, error: verifyErr } = await this._verify(slot, fk, stream, number, otp, smsArrived);
       if (!session) {
-        const failed = fail(verifyErr || 'verify failed');
+        const isWrongOtp = /incorrect|wrong|invalid|expired|code/i.test(verifyErr || '');
+        const failed = fail(verifyErr || 'verify failed', isWrongOtp ? 'wrong_otp' : 'failed');
         if (this.onFailure && !this.stopped && !this._asking) {
           this._asking = true;
           try {
@@ -282,7 +294,17 @@ export class Worker {
       res.status = 'success';
       res.cost = number.cost;
       res.session = session;
-      const doneDetail = res.email_linked ? `extracted + ${res.linked_email}` : 'extracted';
+      res.is_new_account = !!fk.isNewUser; // a freshly-created (signup) account
+      // Derive the result CATEGORY for display + file routing:
+      //   fresh     → brand-new account just created
+      //   coupon    → existing account, ₹100 Minutes coupon available
+      //   no_coupon → existing account, coupon already used
+      //   extracted → success but Minutes wasn't checked
+      res.category = res.is_new_account ? 'fresh'
+        : (res.minutes_checked ? (res.coupon_eligible ? 'coupon' : 'no_coupon') : 'extracted');
+      const doneDetail = res.is_new_account ? 'fresh account'
+        : res.minutes_checked ? (res.coupon_eligible ? '₹100 coupon' : 'no coupon')
+        : 'extracted';
       this._emit(slot, { phase: 'done', detail: doneDetail });
       releaseOnce(true, doneDetail, true); // success: OTP consumed → no cancel
       return res;