NETSCRIPT: Add additionalMsec to BasicHGWOptions

This option adds additional time to the sleep in hack/grow/weaken before
the command takes effect. The critical difference between this and doing
your own sleep is that it creates a single, uninterruptible sleep: This
opens up multiple new avenues of gameplay for batching.

Note that use of this new feature is theoretically always suboptimal,
since extending the sleep time wastes RAM.

src/Netscript/NetscriptHelpers.ts

@@ -434,17 +434,22 @@ function hack(
   ctx: NetscriptContext,
   hostname: string,
   manual: boolean,
-  { threads: requestedThreads, stock }: BasicHGWOptions = {},
+  { threads: requestedThreads, stock, additionalMsec: requestedSec }: BasicHGWOptions = {},
 ): Promise<number> {
   const ws = ctx.workerScript;
   const threads = helpers.resolveNetscriptRequestedThreads(ctx, requestedThreads);
+  const additionalMsec = number(ctx, "opts.additionalMsec", requestedSec ?? 0);
+  if (additionalMsec < 0) {
+    throw makeRuntimeErrorMsg(ctx, `additionalMsec must be non-negative, got ${additionalMsec}`);
+  }
   const server = getServer(ctx, hostname);
   if (!(server instanceof Server)) {
     throw makeRuntimeErrorMsg(ctx, "Cannot be executed on this server.");
   }
 
   // Calculate the hacking time
-  const hackingTime = calculateHackingTime(server, Player); // This is in seconds
+  // This is in seconds
+  const hackingTime = calculateHackingTime(server, Player) + additionalMsec / 1000.0;
 
   // No root access or skill level too low
   const canHack = netscriptCanHack(server);

src/NetscriptFunctions.ts

@@ -144,7 +144,7 @@ export const ns: InternalAPI<NSFull> = {
       // TODO 2.2: better type safety rework for functions using assertObjectType, then remove function.
       const optsValidator: BasicHGWOptions = {};
       assertObjectType(ctx, "opts", opts, optsValidator);
-      return helpers.hack(ctx, hostname, false, { threads: opts.threads, stock: opts.stock });
+      return helpers.hack(ctx, hostname, false, opts);
     },
   hackAnalyzeThreads: (ctx) => (_hostname, _hackAmount) => {
     const hostname = helpers.string(ctx, "hostname", _hostname);
@@ -241,11 +241,11 @@ export const ns: InternalAPI<NSFull> = {
       const hostname = helpers.string(ctx, "hostname", _hostname);
       const optsValidator: BasicHGWOptions = {};
       assertObjectType(ctx, "opts", opts, optsValidator);
-      const requestedThreads =
+      const threads = helpers.resolveNetscriptRequestedThreads(ctx, opts.threads);
-        opts.threads === undefined
+      const additionalMsec = helpers.number(ctx, "opts.additionalMsec", opts.additionalMsec ?? 0);
-          ? ctx.workerScript.scriptRef.threads
+      if (additionalMsec < 0) {
-          : helpers.number(ctx, "opts.threads", opts.threads);
+        throw helpers.makeRuntimeErrorMsg(ctx, `additionalMsec must be non-negative, got ${additionalMsec}`);
-      const threads = helpers.resolveNetscriptRequestedThreads(ctx, requestedThreads);
+      }
 
       const server = helpers.getServer(ctx, hostname);
       if (!(server instanceof Server)) {
@@ -264,7 +264,7 @@ export const ns: InternalAPI<NSFull> = {
         throw helpers.makeRuntimeErrorMsg(ctx, canHack.msg || "");
       }
 
-      const growTime = calculateGrowTime(server, Player);
+      const growTime = calculateGrowTime(server, Player) + additionalMsec / 1000.0;
       helpers.log(
         ctx,
         () =>
@@ -348,11 +348,11 @@ export const ns: InternalAPI<NSFull> = {
       const hostname = helpers.string(ctx, "hostname", _hostname);
       const optsValidator: BasicHGWOptions = {};
       assertObjectType(ctx, "opts", opts, optsValidator);
-      const requestedThreads =
+      const threads = helpers.resolveNetscriptRequestedThreads(ctx, opts.threads);
-        opts.threads === undefined
+      const additionalMsec = helpers.number(ctx, "opts.additionalMsec", opts.additionalMsec ?? 0);
-          ? ctx.workerScript.scriptRef.threads
+      if (additionalMsec < 0) {
-          : helpers.number(ctx, "opts.threads", opts.threads);
+        throw helpers.makeRuntimeErrorMsg(ctx, `additionalMsec must be non-negative, got ${additionalMsec}`);
-      const threads = helpers.resolveNetscriptRequestedThreads(ctx, requestedThreads);
+      }
 
       const server = helpers.getServer(ctx, hostname);
       if (!(server instanceof Server)) {
@@ -366,7 +366,7 @@ export const ns: InternalAPI<NSFull> = {
         throw helpers.makeRuntimeErrorMsg(ctx, canHack.msg || "");
       }
 
-      const weakenTime = calculateWeakenTime(server, Player);
+      const weakenTime = calculateWeakenTime(server, Player) + additionalMsec / 1000.0;
       helpers.log(
         ctx,
         () =>

src/ScriptEditor/NetscriptDefinitions.d.ts

@@ -256,6 +256,12 @@ interface BasicHGWOptions {
   threads?: number;
   /** Set to true this action will affect the stock market. */
   stock?: boolean;
+  /** Number of additional milliseconds that will be spent waiting between the
+   *  start of the function and when it completes. There will only be one,
+   *  contiguous wait, which is relevant because stats such as server security
+   *  level are computed at the *start* of the function.
+   *  Must be non-negative. */
+  additionalMsec?: number;
 }
 
 /**