dcp-worker 3.2.30-6 → 3.2.30-7

package/bin/dcp-worker

@@ -4,12 +4,7 @@
  *              Standalone NodeJS DCP Worker
  *
  *  @author     Ryan Rossiter, ryan@kingsds.network
- *              Paul, paul@distributive.network
- *              Wes Garland, wes@distributive.network
- *
  *  @date       April 2020
- *              April-May 2023
- *              May-June 2023
  */
 'use strict';
 
@@ -21,7 +16,6 @@ const path    = require('path');
 const crypto  = require('crypto');
 const chalk   = require('chalk');
 const telnetd = require('../lib/remote-console');
-const utils   = require('../lib/utils');
 
 const configName = process.env.DCP_CONFIG || '../etc/dcp-worker-config';
 const EXIT_UNHANDLED = 5;
@@ -41,9 +35,7 @@ const replHelpers = {
 };
 telnetd.init(replHelpers);
 
-/* Initialize dcp-client with local config defaults and run the main function. DCP_CONFIG_COOKIE becomes dcpConfig.cookie.
- * And dcpConfig is defined as a side effect of initializing dcp-client.
- */
+/* Initialize dcp-client with local config defaults and run the main function. DCP_CONFIG_COOKIE becomes dcpConfig.cookie. */
 process.env.DCP_CONFIG_COOKIE = (Math.random().toString(16)).slice(2) + '-' + process.pid + '-' + Date.now();
 require('dcp-client').init({ configName }).then(main).catch(handleUnhandled);
 
@@ -69,6 +61,7 @@ function parseCliArgs()
         alias: 'd',
         describe: 'default proportion of CPU,GPU to use when cores not specified',
         type: 'string',
+        default: JSON.stringify(dcpConfig.worker.defaultCoreDensity),
       },
       verbose: {
         alias: 'v',
@@ -407,10 +400,23 @@ async function main()
 
       function fetchEventHandler(ev)
       {
+        var slicesFetched;
+
         if (ev instanceof Error)
+        {
           console.error('Error fetching task:', ev);
+          return;
+        }
+
+        if (typeof ev === 'number' || typeof ev === 'string') /* <= June 2023 Worker events: remove ~ Sep 2023 /wg */
+          slicesFetched = ev;
         else
-          dcpWorkerOptions.leavePublicGroup = Boolean(utils.slicesFetched(ev) > 0);
+        {
+          const task = ev;
+          slicesFetched = task.slices.length;
+        }
+
+        dcpWorkerOptions.leavePublicGroup = Boolean(slicesFetched > 0);
       }
     }
   }
@@ -493,13 +499,12 @@ function processCoresAndDensity (dcpWorkerOptions, cliArgs)
 
   const parseArg = (which) => {
     if (!cliArgs[which])
-      dcpWorkerOptions[which] = defaultTargets[which];
-    else
-    {
-      const [cpu, gpu] = cliArgs[which].split(',');
-      dcpWorkerOptions[which] = { cpu: Number(cpu || defaultTargets[which].cpu),
-                                  gpu: Number(gpu || defaultTargets[which].gpu) };
-    }
+      return false;
+
+    const [cpu, gpu] = cliArgs[which].split(',');
+    dcpWorkerOptions[which] = { cpu: Number(cpu || defaultTargets[which].cpu),
+                                gpu: Number(gpu || defaultTargets[which].gpu) };
+    return true;
   };
 
   parseArg('density');
@@ -507,8 +512,8 @@ function processCoresAndDensity (dcpWorkerOptions, cliArgs)
 
   if (dcpWorkerOptions.cores)
     debugging() && console.debug('dcp-worker: cores =', dcpWorkerOptions.cores);
-  if (dcpWorkerOptions.density)
-    debugging() && console.debug('dcp-worker: core density =', dcpWorkerOptions.density);
+  else
+    debugging() && console.debug('dcp-worker: core density =', dcpWorkerOptions.defaultCoreDensity);
 }
 
 /**
@@ -632,14 +637,14 @@ function sliceReport()
 
   report += ('Progress:') + '\n';
   worker.workingSandboxes.forEach(sb => {
-    const jobName = sb.job?.public?.name || `idek (${sb.jobAddress})`;
+    const jobName = sb.job && sb.job.public && sb.job.public.name || `idek (${sb.jobAddress})`;
     let el = Date.now() - sb.sliceStartTime;
     const t = el < 1000000
           ? toInterval(el)
           : 'new';
 
     el = sb.progressReports && sb.progressReports.last
-      ? Date.now() - (sb.sliceStartTime + (sb.progressReports.last?.timestamp ?? 0))
+      ? Date.now() - (sb.sliceStartTime + sb.progressReports.last.timestamp)
       : 0;
     const pct = (typeof sb.progress) === 'number'
           ? `${Number(sb.progress).toFixed(0).padStart(2)}%`

package/etc/dcp-worker-config.js

@@ -49,7 +49,7 @@
       // keystore('~/.dcp/scott'),
     ],
 
-    jobAddresses: false,               /* If specified, restrict the worker to only these jobs */
+    jobAddresses: [],               /* If specified, restrict the worker to only these jobs */
     paymentAddress: undefined,      /* Bank account where earned funds are transfered if not specified on command-line */
   },
 

package/etc/dcp-worker-config.js.md5

@@ -1,2 +1,2 @@
-4d49057dd014344e4be5a266e9754824
+8edf1c57bee521ed7f151a21b8f3f34d
 ### DO NOT MODIFY THIS FILE!!! ###

package/lib/blessed-components/sandboxes.js

@@ -67,7 +67,11 @@ class Sandboxes extends Box {
   update(data=this.data) {
     this.data = data;
     for (let i = 0; i < this.data.length; i++) {
-      this.updateProgressBar(i, this.data[i]);
+      if (i < this.progressBars.length) {
+        this.updateProgressBar(i, this.data[i]);
+      } else {
+        this.createProgressBar();
+      }
     }
 
     if (this.data.length < this.progressBars.length) {
@@ -76,15 +80,7 @@ class Sandboxes extends Box {
       }
     }
 
-    this.setLabel(`${this.options.label} (${this.progressBars.length})`);
-  }
-
-  // Deletes last progress bar
-  deleteProgressBar() {
-    let i = this.progressBars.length - 1;
-    this.progressBars[i].label.destroy()
-    this.progressBars[i].progressBar.destroy()
-    this.progressBars.pop(i)
+    this.setLabel(`${this.options.label} (${this.data.length})`);
   }
 }
 

package/lib/dashboard-tui.js

@@ -15,7 +15,6 @@ const chalk      = require('chalk');
 const blessed    = require('blessed');
 const contrib    = require('blessed-contrib');
 const components = require('./blessed-components');
-const utils      = require('../lib/utils');
 
 const { replaceWorkerEvent, replaceSandboxEvent }  = require('./default-ui-events');
 
@@ -53,7 +52,7 @@ exports.init = function dashboard$$init(worker, options)
 
   const sandboxPane = grid.set(0, 0, 2, 2, components.sandboxes, {
     label: 'Sandboxes',
-    defaultProgressBars: 0,
+    defaultProgressBars: Math.floor(worker.workerOptions.cores.cpu) || 1,
     scrollable: true,
     alwaysScroll: true,
     mouse: true,
@@ -62,47 +61,6 @@ exports.init = function dashboard$$init(worker, options)
     },
   });
 
-  const passwordBox = blessed.textbox({
-    parent: screen,
-    border: 'line',
-    top: 'center',
-    left: 'center',
-    width: '50%',
-    height: 'shrink',
-    padding: {
-      top: 1
-    },
-    censor: true,
-    inputOnFocus: true,
-    label: 'Password Prompt:',
-    hidden: true,
-  });
-
-  function askPassword(promptMessage)
-  {
-    return new Promise((resolve, reject) => {
-      passwordBox.focus();
-      passwordBox.show();
-      passwordBox.setLabel(promptMessage);
-
-      function passwordSubmitFn(value)
-      {
-        passwordBox.hide();
-        passwordBox.removeListener('submit', passwordSubmitFn);
-        passwordBox.setValue('');
-        resolve(value);
-      }
-
-      passwordBox.on('submit', passwordSubmitFn);
-      screen.render();
-    });
-  }
-
-  // override wallet.passphrasePrompt with password box
-  require('dcp/wallet').passphrasePrompt = (promptMessage) => {
-    return askPassword(promptMessage);
-  };
-
   delete exports.init; /* singleton */
 
   if (!usingDebugger)
@@ -141,27 +99,23 @@ exports.init = function dashboard$$init(worker, options)
 
   /* Override default event behaviour to work better with the Dashboard. */
 
-  /** XXXpfr @todo Is this correct?  Or should we init progressData inside 'slice' like we used to.  */
-  replaceSandboxEvent('ready', function dashboard$$job(sandbox, sandboxData, ev) {
+  replaceSandboxEvent('start', function dashboard$$sliceStart(sandbox, sandboxData, ev) {
     sandboxData.progressData = {
       indeterminate: true,
       progress: 0,
-      label: sandbox?.public ? sandbox.public.name: '<no-label>',
+      label: sandbox.public.name,
     };
-  });
-  
-  replaceSandboxEvent('slice', function dashboard$$slice(sandbox, sandboxData, ev) {
+
     sandboxPane.data.push(sandboxData.progressData);
     sandboxPane.update();
   });
 
-  replaceSandboxEvent('progress', function dashboard$$progress(sandbox, sandboxData, ev) {
-    if (!ev)
+  replaceSandboxEvent('sliceProgress', function dashboard$$sliceProgress(sandbox, sandboxData, ev) {
+    if (ev.indeterminate)
     {
       sandboxData.progressData.progress = 100;
       setTimeout(() => {
-        if (sandboxData.progressData.indeterminate)
-        {
+        if (sandboxData.progressData.indeterminate) {
           sandboxData.progressData.progress = 0;
           sandboxPane.update();
         }
@@ -169,54 +123,53 @@ exports.init = function dashboard$$init(worker, options)
     }
     else
     {
-      sandboxData.progressData.progress = ev;
+      sandboxData.progressData.progress = ev.progress;
       sandboxData.progressData.indeterminate = false;
     }
 
     sandboxPane.update();
   });
   
-  replaceSandboxEvent('sliceEnd', function dashboard$$sliceEnd(sandbox, sandboxData, ev) {
+  replaceSandboxEvent('sliceFinish', function dashboard$$sliceFinish(sandbox, sandboxData, ev) {
     sandboxPane.data = sandboxPane.data.filter(d => d != sandboxData.progressData);
-    sandboxData.progressData.progress = 0;
     sandboxPane.update();
   });
 
-  replaceSandboxEvent('end', function dashboard$$end(sandbox, sandboxData, ev) {
-    sandboxPane.data = sandboxPane.data.filter(d => d != sandboxData.progressData);
-    sandboxPane.deleteProgressBar();
-    sandboxData.progressData.progress = 0;
-    sandboxPane.update();
-  });
+  worker.on('payment', updateWorkerInfo);
 
-  replaceWorkerEvent('beforeFetch', function dashboard$$beforeFetch(ev) {
+  replaceWorkerEvent('fetchStart', function dashboard$$fetchStart(ev) {
     sliceFetchStatus = SLICE_FETCH_STATUS.FETCHING;
     updateWorkerInfo();
   });
   
   replaceWorkerEvent('fetch', function dashboard$$fetch(ev) {
-    sliceFetchStatus = SLICE_FETCH_STATUS.NO_WORK;
+    var slicesFetched;
+
     if (ev instanceof Error)
+    {
       console.error('Error fetching slices:', ev);
-    else if ( !(utils.slicesFetched(ev) === 0 && sandboxPane.data.length === 0))
-      sliceFetchStatus = SLICE_FETCH_STATUS.WORKING;
-    updateWorkerInfo();
-  });
+      return;
+    }
 
-  worker.on('end', () =>  { screen.destroy(); });
+    if (typeof ev === 'number' || typeof ev === 'string') /* <= June 2023 Worker events: remove ~ Sep 2023 /wg */
+      slicesFetched = ev;
+    else
+    {
+      const task = ev;
+      slicesFetched = task.slices.length;
+    }
 
-  worker.on('sandbox', function dashboard$$sandbox(ev) {
-    sandboxPane.createProgressBar();
-    sandboxPane.update();
-  });
+    if (slicesFetched === 0 && sandboxPane.data.length === 0) {
+      sliceFetchStatus = SLICE_FETCH_STATUS.NO_WORK;
+    } else {
+      sliceFetchStatus = SLICE_FETCH_STATUS.WORKING;
+    }
 
-  worker.on('payment', function dashboard$$payment(ev) {
-    const payment = parseFloat(ev);
-    
-    if (!isNaN(payment))
-      totalDCCs += payment;
+    updateWorkerInfo();
+  });
 
-    sandboxPane.update();
+  replaceWorkerEvent('fetchError', function dashabord$$fetchError() {
+    sliceFetchStatus = SLICE_FETCH_STATUS.NO_WORK;
     updateWorkerInfo();
   });
 };

package/lib/default-ui-events.js

@@ -26,8 +26,6 @@
  */
 'use strict';
 
-const utils = require('../lib/utils');
-
 const sandboxEventHandlers = {};
 const workerEventHandlers = {};
 
@@ -52,28 +50,35 @@ exports.hook = function hookWorkerEvents$$hook(worker, options)
     if (!sandbox.jobAddress)
       return '<no job>';
     
-    const address = sandbox.jobAddress.slice(0, truncationLength);
-    const baseInfo = sandbox?.public ? `${address}: ${sandbox.public.name}` : address;
+    const address = sandbox.jobAddress ? sandbox.jobAddress.slice(0, truncationLength) : 'null';
+    const baseInfo = sandbox.public?.name ? `${address}: ${sandbox.public.name}` : `${address}:`;
 
     if (!sliceNumber)
-      sliceNumber = sandbox.sliceNumber;
-    return sliceNumber > 0 ? `slice ${sliceNumber}, ${baseInfo}` : baseInfo;
+      sliceNumber = sandbox.slice ? sandbox.slice.sliceNumber : 0;
+    return sliceNumber ? `slice ${sliceNumber}, ${baseInfo}` : baseInfo;
   }
 
   sandboxEventHandlers.ready = function sandboxReadyHandler(sandbox, sandboxData, ev) {
     console.log(` . Sandbox ${sandboxData.shortId}:  Initialized`);
   };
 
-  sandboxEventHandlers.slice = function sliceHandler(sandbox, sandboxData, ev) {
-    sliceMap[sandbox.id] = { slice: sandbox.sliceNumber, t0: Date.now() };
+  sandboxEventHandlers.terminated = function sandboxTerminatedHandler(sandbox, sandboxData, ev) {
+    const sliceInfo = sliceMap[sandbox.id];
+    console.log(` * Sandbox ${sandboxData.shortId}:  Terminated:      ${makeSliceId(sandbox, sliceInfo?.slice)}`);
+    delete sliceMap[sandbox.id];
+  };
+
+  sandboxEventHandlers.start = function sliceStartHandler(sandbox, sandboxData, ev) {
+    const sliceNumber = sandbox.slice ? sandbox.slice.sliceNumber : 0;
+    sliceMap[sandbox.id] = { slice: sliceNumber, t0: Date.now() };
     console.log(` . Sandbox ${sandboxData.shortId}:  Slice Started:   ${makeSliceId(sandbox)}`);
   };
 
-  sandboxEventHandlers.progress = function progressHandler(sandbox, sandboxdData, ev) {
-    // Overridden in dashboard-tui.js
+  sandboxEventHandlers.sliceProgress = function sliceProgressHandler(sandbox, sandbodData, ev) {
+    // NOP
   };
 
-  sandboxEventHandlers.sliceEnd = function sliceEndHandler(sandbox, sandboxData, ev) {
+  sandboxEventHandlers.sliceFinish = function sliceFinishHandler(sandbox, sandboxData, ev) {
     const sliceInfo = sliceMap[sandbox.id];
     if (sliceInfo)
       console.log(` * Sandbox ${sandboxData.shortId}:  Slice Completed: ${makeSliceId(sandbox, sliceInfo.slice)}: dt ${Date.now() - sliceInfo.t0}ms`);
@@ -81,14 +86,8 @@ exports.hook = function hookWorkerEvents$$hook(worker, options)
       console.log(` * Sandbox ${sandboxData.shortId}:  Slice Completed: ${makeSliceId(sandbox)}`);
   };
 
-  sandboxEventHandlers.end = function endHandler(sandbox,sandboxData, ev) {
-    const sliceInfo = sliceMap[sandbox.id];
-    console.log(` * Sandbox ${sandboxData.shortId}:  Terminated:      ${makeSliceId(sandbox, sliceInfo?.slice)}`);
-    delete sliceMap[sandbox.id];
-  };
-
   workerEventHandlers.payment = function paymentHandler(ev) {
-    const payment = parseFloat(ev);
+    const payment = parseFloat(ev.payment);
 
     if (isNaN(payment))
       console.error(' ! Failed to parse payment:', payment);
@@ -96,27 +95,44 @@ exports.hook = function hookWorkerEvents$$hook(worker, options)
       console.log(` . Payment: ${payment.toFixed(3)} ⊇`);
   };
 
-  workerEventHandlers.beforeFetch = function beforeFetchHandler() {
+  workerEventHandlers.fetchStart = function fetchStartHandler() {
     options.verbose && console.log(' * Fetching slices...');
   };
 
   workerEventHandlers.fetch = function fetchHandler(ev) {
+    var slicesFetched;
+
     if (ev instanceof Error)
+    {
       console.error(' ! Failed to fetch slices:', ev);
+      return;
+    }
+
+    if (typeof ev === 'number' || typeof ev === 'string') /* <= June 2023 Worker events: remove ~ Sep 2023 /wg */
+      slicesFetched = ev;
     else
-      options.verbose && console.log(' . Fetched', utils.slicesFetched(ev), 'slices');
+    {
+      const task = ev;
+      slicesFetched = task.slices.length;
+    }
+
+    options.verbose && console.log(' . Fetched', slicesFetched, 'slices');
   };
 
+  workerEventHandlers.fetchError = function fetchErrorHandler(ev) {
+    console.log(' ! Failed to fetch slices:', ev);
+  };
 
-  workerEventHandlers.beforeResult = function beforeResultHandler() {
+  workerEventHandlers.submitStart = function submitStartHandler() {
     options.verbose >= 2 && console.log(' * Submitting results...');
   };
 
-  workerEventHandlers.result = function resultHandler(ev) {
-    if (ev instanceof Error)
-      console.error(" ! Failed to submit results:", ev);
-    else
-      options.verbose >= 2 && console.log(' . Submitted');
+  workerEventHandlers.submit = function submitHandler() {
+    options.verbose >= 2 && console.log(' . Submitted');
+  };
+
+  workerEventHandlers.submitError = function submitErrorHandler(ev) {
+    console.log(' ! Failed to submit results:', ev);
   };
 
   /* Register the appropriate event handlers on the worker and on each sandbox. The handlers are
@@ -128,14 +144,14 @@ exports.hook = function hookWorkerEvents$$hook(worker, options)
    * called `sandboxData` which is just arbitrary storage for the eventHandlers' use, eg for memos.
    */
   for (let eventName in workerEventHandlers)
-    worker.on(eventName, (...args) => workerEventHandlers[eventName](...args));
+    worker.addEventListener(eventName, workerEventHandlers[eventName]);
 
   worker.on('sandbox', function newSandboxHandler(sandbox) {
     const sandboxData = {
       shortId: sandbox.id.toString(10).padStart(3)
     };
     for (let eventName in sandboxEventHandlers)
-      sandbox.on(eventName, (...args) => sandboxEventHandlers[eventName](sandbox, sandboxData, ...args));
+      sandbox.addEventListener(eventName, (...args) => sandboxEventHandlers[eventName](sandbox, sandboxData, ...args));
   });
   
   exports.sandboxEventHandlers = sandboxEventHandlers;

package/lib/startWorkerLogger.js

@@ -36,10 +36,7 @@ function getLogger(options)
   catch (error)
   {
     if (error.code === 'MODULE_NOT_FOUND')
-    {
-      const errorMessageStart = error.message.replace(/\n.*/g, '');
-      error.message = `Unknown outputMode "${options.outputMode} (${errorMessageStart})`;
-    }
+      throw new Error(`Unknown outputMode "${options.outputMode}"`);
     throw error;
   }
 }
@@ -126,5 +123,5 @@ function format(...argv)
  * Options for util.inspect. Loggers which cannot deal with colours should force this false.
  */
 exports.inspectOptions = {
-  colors: process.stdout.isTTY && process.stdout.hasColors() || process.env.FORCE_COLOR,
+  colors: process.stdout.hasColors() || process.env.FORCE_COLOR,
 };

package/lib/worker-loggers/dashboard.js

@@ -45,4 +45,3 @@ exports.init = function dashboardLogger$$init(options)
   for (let level of ['log', 'warn', 'debug', 'info', 'error'])
     exports[level] = logWrapperFactory(level);
 }
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dcp-worker",
-  "version": "3.2.30-6",
+  "version": "3.2.30-7",
   "description": "JavaScript portion of DCP Workers for Node.js",
   "main": "bin/dcp-worker",
   "keywords": [
@@ -37,7 +37,7 @@
     "blessed": "^0.1.81",
     "blessed-contrib": "^4.11.0",
     "chalk": "^4.1.0",
-    "dcp-client": "4.3.0-4",
+    "dcp-client": "4.2.32",
     "semver": "^7.3.8",
     "syslog-client": "1.1.1"
   },
@@ -46,18 +46,10 @@
   },
   "devDependencies": {
     "@kingsds/eslint-config": "^1.0.1",
-    "eslint": ">=8"
-  },
-  "peerDependencies": {
-    "node-eventlog": "https://gitpkg.now.sh/Distributive-Network/node-eventlog/package?dcp/0.0.1"
-  },
-  "peerDependenciesMeta": {
-    "node-eventlog": {
-      "optional": true
-    }
+    "eslint": "7.30.0"
   },
   "engines": {
-    "node": ">=18",
+    "node": ">=16",
     "npm": ">=7"
   }
 }

package/lib/utils.js

@@ -1,28 +0,0 @@
-
-/**
- * @file        utils.js
- *              Shared library code.
- *
- * @author      Paul, paul@distributive.network
- * @date        August 2023
- */
-'use strict';
-
-/**
- * Figure out #slices fetched from the different forms of the 'fetch' event.
- * @param {*|string|number} task
- * @returns {number}
- */
-function slicesFetched (task)
-{
-  if (typeof task === 'number') /* <= June 2023 Worker events: remove ~ Sep 2023 /wg */
-    return task;
-  if (typeof task === 'string') /* <= June 2023 Worker events: remove ~ Sep 2023 /wg */
-    return parseInt(task) || 0;
-  let slicesFetched = 0;
-  for (const job in task.slices)
-    slicesFetched += task.slices[job];
-  return slicesFetched;
-}
-
-exports.slicesFetched = slicesFetched;