fix: enable type checking of zoe/tools and fix errors

packages/zoe/jsconfig.json

@@ -14,5 +14,5 @@
     "strictNullChecks": true,
     "moduleResolution": "node",
   },
-  "include": ["src/**/*.js", "exports.js"],
+  "include": ["src/**/*.js", "exported.js", "tools/**/*.js"],
 }

packages/zoe/tools/manualTimer.js

@@ -16,7 +16,7 @@ import './internal-types';
 export default function buildManualTimer(log, startValue = 0) {
   let ticks = startValue;
 
-  /** @type {Store<Timestamp, Array<Waker>>} */
+  /** @type {Store<Timestamp, Array<TimerWaker>>} */
   const schedule = makeStore('Timestamp');
 
   /** @type {ManualTimer} */
@@ -43,7 +43,7 @@ export default function buildManualTimer(log, startValue = 0) {
       if (baseTime <= ticks) {
         log(`&& task was past its deadline when scheduled: ${baseTime} &&`);
         E(waker).wake(ticks);
-        return undefined;
+        return baseTime;
       }
       log(`@@ schedule task for:${baseTime}, currently: ${ticks} @@`);
       if (!schedule.has(baseTime)) {
@@ -72,10 +72,10 @@ export default function buildManualTimer(log, startValue = 0) {
       return harden(baseTimes);
     },
     createRepeater(delay, interval) {
-      /** @type {Array<Waker> | null} */
+      /** @type {Array<TimerWaker> | null} */
       let wakers = [];
 
-      /** @type {Waker} */
+      /** @type {TimerWaker} */
       const repeaterWaker = {
         async wake(timestamp) {
           if (!wakers) {
@@ -91,6 +91,9 @@ export default function buildManualTimer(log, startValue = 0) {
       /** @type {TimerRepeater} */
       const repeater = {
         schedule(waker) {
+          if (!wakers) {
+            throw Error(`Cannot schedule on a disabled repeater`);
+          }
           wakers.push(waker);
         },
         disable() {

packages/zoe/tools/types.js

@@ -3,9 +3,9 @@
  * schedule a single wake() call, create a repeater that will allow scheduling
  * of events at regular intervals, or remove scheduled calls.
  * @property {() => Timestamp} getCurrentTimestamp Retrieve the latest timestamp
- * @property {(baseTime: Timestamp, waker: Waker) => Timestamp} setWakeup Return
+ * @property {(baseTime: Timestamp, waker: TimerWaker) => Timestamp} setWakeup Return
  * value is the time at which the call is scheduled to take place
- * @property {(waker: Waker) => Array<Timestamp>} removeWakeup Remove the waker
+ * @property {(waker: TimerWaker) => Array<Timestamp>} removeWakeup Remove the waker
  * from all its scheduled wakeups, whether produced by `timer.setWakeup(h)` or
  * `repeater.schedule(h)`.
  * @property {(delay: RelativeTime, interval: RelativeTime) => TimerRepeater}
@@ -28,14 +28,14 @@
  */
 
 /**
- * @typedef {Object} Waker
+ * @typedef {Object} TimerWaker
  * @property {(timestamp: Timestamp) => void} wake The timestamp passed to
  * `wake()` is the time that the call was scheduled to occur.
  */
 
 /**
  * @typedef {Object} TimerRepeater
- * @property {(waker: Waker) => void} schedule Returns the time scheduled for
+ * @property {(waker: TimerWaker) => void} schedule Returns the time scheduled for
  * the first call to `E(waker).wake()`.  The waker will continue to be scheduled
  * every interval until the repeater is disabled.
  * @property {() => void} disable Disable this repeater, so `schedule(w)` can't