why does the unit test has const subnetsOverlap = (subnet1: string, subnet2: string): boolean => {
const [ip1, cidr1] = subnet1.split('/');
const [ip2, cidr2] = subnet2.split('/');

  const ipToNumber = (ip: string): number => {
    return ip.split('.').reduce((acc, octet) => (acc << 8) + parseInt(octet, 10), 0) >>> 0;
  };

  const getNetworkRange = (ip: string, cidr: string): [number, number] => {
    const ipNum = ipToNumber(ip);
    const maskBits = parseInt(cidr, 10);
    const mask = (0xffffffff << (32 - maskBits)) >>> 0;
    const networkAddr = (ipNum & mask) >>> 0;
    const broadcastAddr = (networkAddr | ~mask) >>> 0;
    return [networkAddr, broadcastAddr];
  };

  const [start1, end1] = getNetworkRange(ip1, cidr1);
  const [start2, end2] = getNetworkRange(ip2, cidr2);

  return (start1 <= end2 && end1 >= start2);
};

this should be expose by the docker-manager.ts, no? The unit test shuldn't dupliate the same logic

The user has attached the following files as relevant context:

• AGENTS.md