fix: remove VITEST bypass from version getters

Version getters should fail if environment variables aren't set,
even in test mode, because values should be properly defined in
external-tools.json.

Author: jdalton

packages/cli/src/env/coana-version.mts

@@ -8,15 +8,9 @@
 
 import process from 'node:process'
 
-import { VITEST } from './vitest.mts'
-
 export function getCoanaVersion(): string {
   const version = process.env['INLINED_SOCKET_CLI_COANA_VERSION']
   if (!version) {
-    // In test mode, return empty string instead of throwing.
-    if (VITEST) {
-      return ''
-    }
     throw new Error(
       'INLINED_SOCKET_CLI_COANA_VERSION not found. Please ensure @coana-tech/cli is properly configured in external-tools.json.',
     )

packages/cli/src/env/sfw-version.mts

@@ -8,15 +8,9 @@
 
 import process from 'node:process'
 
-import { VITEST } from './vitest.mts'
-
 export function getSwfVersion(): string {
   const version = process.env['INLINED_SOCKET_CLI_SFW_VERSION']
   if (!version) {
-    // In test mode, return empty string instead of throwing.
-    if (VITEST) {
-      return ''
-    }
     throw new Error(
       'INLINED_SOCKET_CLI_SFW_VERSION not found. Please ensure sfw is properly configured in external-tools.json.',
     )

packages/cli/src/env/socket-patch-version.mts

@@ -8,15 +8,9 @@
 
 import process from 'node:process'
 
-import { VITEST } from './vitest.mts'
-
 export function getSocketPatchVersion(): string {
   const version = process.env['INLINED_SOCKET_CLI_SOCKET_PATCH_VERSION']
   if (!version) {
-    // In test mode, return empty string instead of throwing.
-    if (VITEST) {
-      return ''
-    }
     throw new Error(
       'INLINED_SOCKET_CLI_SOCKET_PATCH_VERSION not found. Please ensure socket-patch is properly configured in external-tools.json.',
     )