fix(deepnote-server): Handle disabled IPv6 support (#192)

* fix(deepnote-server): Handle disabled IPv6 support

Signed-off-by: Tomas Kislan <tomas@kislan.sk>

* test: Add tests for disabled IPv6 network

Signed-off-by: Tomas Kislan <tomas@kislan.sk>

* feat(deepnote-server): Improve error handling for IPv6 checks

Signed-off-by: Tomas Kislan <tomas@kislan.sk>

* chore: format

---------

Signed-off-by: Tomas Kislan <tomas@kislan.sk>
Co-authored-by: Lukas Saltenas <lukas.saltenas@gmail.com>

Author: tkislan

src/kernels/deepnote/deepnoteServerStarter.node.ts

@@ -621,8 +621,17 @@ export class DeepnoteServerStarter implements IDeepnoteServerStarter, IExtension
             }
 
             // Also check IPv6 loopback to be safe
-            const inUseIpv6 = await tcpPortUsed.check(port, '::1');
-            return !inUseIpv6;
+            try {
+                const inUseIpv6 = await tcpPortUsed.check(port, '::1');
+                return !inUseIpv6;
+            } catch (error: unknown) {
+                if (error instanceof Error && 'code' in error && error.code === 'EAFNOSUPPORT') {
+                    logger.debug('IPv6 is not supported on this system');
+                    return true;
+                }
+                logger.warn(`Failed to check IPv6 port availability for ${port}:`, error);
+                return false;
+            }
         } catch (error) {
             logger.warn(`Failed to check port availability for ${port}:`, error);
             return false;

src/kernels/deepnote/deepnoteServerStarter.unit.test.ts

@@ -116,6 +116,67 @@ suite('DeepnoteServerStarter - Port Allocation Integration Tests', () => {
                 warnStub.restore();
             }
         });
+
+        test('should return true when IPv6 is disabled (EAFNOSUPPORT error)', async () => {
+            const port = 54324;
+            const ipv6Error = new Error('connect EAFNOSUPPORT ::1:54324');
+            (ipv6Error as any).code = 'EAFNOSUPPORT';
+
+            // IPv4 check succeeds (port is available)
+            checkStub.onFirstCall().resolves(false);
+
+            // IPv6 check throws EAFNOSUPPORT (IPv6 not supported)
+            checkStub.onSecondCall().rejects(ipv6Error);
+
+            const debugStub = sinon.stub(logger, 'debug');
+
+            try {
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                const isPortAvailable = getPrivateMethod(serverStarter as any, 'isPortAvailable');
+                const result = await isPortAvailable(port);
+
+                assert.isTrue(result, 'Expected port to be available when IPv4 is free and IPv6 is not supported');
+                assert.strictEqual(checkStub.callCount, 2, 'Should check both IPv4 and IPv6');
+                assert.deepEqual(checkStub.getCall(0).args, [port, '127.0.0.1']);
+                assert.deepEqual(checkStub.getCall(1).args, [port, '::1']);
+                assert.isTrue(
+                    debugStub.calledWith('IPv6 is not supported on this system'),
+                    'Should log debug message about IPv6 not being supported'
+                );
+            } finally {
+                debugStub.restore();
+            }
+        });
+
+        test('should return false when IPv6 check throws non-EAFNOSUPPORT error', async () => {
+            const port = 54325;
+            const ipv6Error = new Error('Some other IPv6 error');
+
+            // IPv4 check succeeds (port is available)
+            checkStub.onFirstCall().resolves(false);
+
+            // IPv6 check throws a different error
+            checkStub.onSecondCall().rejects(ipv6Error);
+
+            const warnStub = sinon.stub(logger, 'warn');
+
+            try {
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                const isPortAvailable = getPrivateMethod(serverStarter as any, 'isPortAvailable');
+                const result = await isPortAvailable(port);
+
+                assert.isFalse(
+                    result,
+                    'Expected port check to fail closed when IPv6 check fails with non-EAFNOSUPPORT error'
+                );
+                assert.strictEqual(checkStub.callCount, 2, 'Should check both IPv4 and IPv6');
+                assert.isTrue(warnStub.called, 'Should log warning when IPv6 check fails');
+                const warnCall = warnStub.getCall(0);
+                assert.include(warnCall.args[0], 'Failed to check IPv6 port availability');
+            } finally {
+                warnStub.restore();
+            }
+        });
     });
 
     suite('findAvailablePort', () => {