From df64404d21be9560a50f87bde52005a657be3854 Mon Sep 17 00:00:00 2001
From: Evgeny Gamov <smartass@smartass.pw>
Date: Tue, 16 Jun 2026 19:51:13 +0500
Subject: [PATCH] test(core): cover base schema + respond; neutralize tunnel
 log prefix

---
 packages/core/src/net/tunnel.ts               |  4 +--
 .../core/test/unit/config/baseSchema.test.ts  | 32 +++++++++++++++++++
 packages/core/test/unit/mcp/respond.test.ts   | 21 ++++++++++++
 packages/core/vitest.config.ts                |  7 +++-
 4 files changed, 61 insertions(+), 3 deletions(-)
 create mode 100644 packages/core/test/unit/config/baseSchema.test.ts
 create mode 100644 packages/core/test/unit/mcp/respond.test.ts

diff --git a/packages/core/src/net/tunnel.ts b/packages/core/src/net/tunnel.ts
index 54249a4..de966ea 100644
--- a/packages/core/src/net/tunnel.ts
+++ b/packages/core/src/net/tunnel.ts
@@ -67,7 +67,7 @@ export const openTunnel = async (
 
     // After 'ready' resolved, later ssh errors must not crash the process.
     client.on('error', (error) => {
-        console.error('dbmole: ssh tunnel error: ' + error.message)
+        console.error('ssh-tunnel: ssh tunnel error: ' + error.message)
     })
 
     let closed = false
@@ -140,7 +140,7 @@ export const openTunnel = async (
     // without a persistent one, a later listener error is swallowed and the
     // tunnel silently serves a dead socket. Mark it closed instead.
     server.on('error', (error) => {
-        console.error('dbmole: tunnel listener error: ' + error.message)
+        console.error('ssh-tunnel: tunnel listener error: ' + error.message)
         markClosed()
     })
 
diff --git a/packages/core/test/unit/config/baseSchema.test.ts b/packages/core/test/unit/config/baseSchema.test.ts
new file mode 100644
index 0000000..cf9f8bb
--- /dev/null
+++ b/packages/core/test/unit/config/baseSchema.test.ts
@@ -0,0 +1,32 @@
+import { describe, expect, it } from 'vitest'
+import * as z from 'zod'
+import { baseConnectionShape, sshConfigSchema } from '../../../src/config/baseSchema.js'
+
+// A minimal concrete schema built from the shared base, the way a leaf package
+// would compose it.
+const testSchema = z.object({ ...baseConnectionShape, type: z.literal('test') }).strict()
+
+describe('sshConfigSchema', () => {
+    it('defaults port to 22', () => {
+        expect(sshConfigSchema.parse({ host: 'h', user: 'u' }).port).toBe(22)
+    })
+    it('rejects unknown keys (strict)', () => {
+        expect(() => sshConfigSchema.parse({ host: 'h', user: 'u', bogus: 1 })).toThrow()
+    })
+})
+
+describe('baseConnectionShape', () => {
+    it('accepts a valid connection and defaults readonly to false', () => {
+        const c = testSchema.parse({ name: 'c1', host: 'h', user: 'u', type: 'test' })
+        expect(c.readonly).toBe(false)
+        expect(c.port).toBeUndefined()
+    })
+    it('requires user', () => {
+        expect(() => testSchema.parse({ name: 'c1', host: 'h', type: 'test' })).toThrow()
+    })
+    it('rejects an invalid name', () => {
+        expect(() =>
+            testSchema.parse({ name: 'bad name!', host: 'h', user: 'u', type: 'test' })
+        ).toThrow()
+    })
+})
diff --git a/packages/core/test/unit/mcp/respond.test.ts b/packages/core/test/unit/mcp/respond.test.ts
new file mode 100644
index 0000000..409cb73
--- /dev/null
+++ b/packages/core/test/unit/mcp/respond.test.ts
@@ -0,0 +1,21 @@
+import { describe, expect, it } from 'vitest'
+import { errorMessage, fail, ok } from '../../../src/mcp/respond.js'
+
+describe('respond helpers', () => {
+    it('ok wraps a JSON payload as text content', () => {
+        const r = ok({ a: 1 })
+        expect(r).toEqual({ content: [{ type: 'text', text: JSON.stringify({ a: 1 }) }] })
+        expect(r.isError).toBeUndefined()
+    })
+    it('fail returns an error result with the message text', () => {
+        const r = fail('boom')
+        expect(r.content).toEqual([{ type: 'text', text: 'boom' }])
+        expect(r.isError).toBe(true)
+    })
+    it('errorMessage extracts Error.message', () => {
+        expect(errorMessage(new Error('nope'))).toBe('nope')
+    })
+    it('errorMessage stringifies non-Errors', () => {
+        expect(errorMessage('plain')).toBe('plain')
+    })
+})
diff --git a/packages/core/vitest.config.ts b/packages/core/vitest.config.ts
index 5dfe3d7..f1fd33e 100644
--- a/packages/core/vitest.config.ts
+++ b/packages/core/vitest.config.ts
@@ -3,6 +3,11 @@ import { defineConfig } from 'vitest/config'
 export default defineConfig({
     test: {
         include: ['test/unit/**/*.test.ts'],
-        coverage: { provider: 'v8', include: ['src/**'], thresholds: { lines: 90, functions: 90 } }
+        coverage: {
+            provider: 'v8',
+            include: ['src/**'],
+            exclude: ['src/index.ts'],
+            thresholds: { lines: 90, functions: 90 }
+        }
     }
 })