fix(types): introduce the MapLike-type instead of using Map

[aslushnikov]

As of today, the https://github.com/isaacs/node-lru-cache cannot be used as a cache for memoization since LRUCache doesn't define the Symbol.toStringTag.

This patch narrows down type requirements for cache implementations.

[peterbe]

import memoize from "@github/memoize";

import { LRUCache } from "lru-cache";

// const myCache = new Map<string, string>();
const myCache = new LRUCache<string, string>({ max: 2 });

const randomName = memoize(
  function randomName(length: number) {
    const upper = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
    const lower = "abcdefghijklmnopqrstuvwxyz";
    let result = "";
    for (let i = 0; i < length; i++) {
      result += (i === 0 ? upper : lower).charAt(
        Math.floor(Math.random() * upper.length)
      );
    }
    return result;
  },
  { cache: myCache }
);

function main() {
  console.log("hello", randomName(5));
  console.log("Hullo", randomName(5));
  console.log("Heil", randomName(5));

  console.log(myCache);
}

main();

works.

Was it perhaps not working in late 2023?

[arelia]

👋🏾 from Web Systems - bumping this comment #25 (comment)