Fix CBM_SKIP_PERF to only skip incremental delta tests, not tool tests

CBM_SKIP_PERF=1 now skips only phases 2-7 (incremental delta, stress,
perf regression) inside the suite. Phase 1 (initial index) and phases
8-30 (135 tool integration tests) always run. This keeps ~2672 tests
on CI vs the full ~2692 locally.

Also fix Windows build: guard symlink() with #ifndef _WIN32.

Author: DeusData

tests/test_incremental.c

@@ -2771,40 +2771,48 @@ SUITE(incremental) {
         return;
     }
 
-    /* Phase 1: Full index baseline + performance */
+    int skip_perf = (getenv("CBM_SKIP_PERF") != NULL &&
+                     getenv("CBM_SKIP_PERF")[0] != '0' &&
+                     getenv("CBM_SKIP_PERF")[0] != '\0');
+
+    /* Phase 1: Full index baseline (needed for tool tests below) */
     RUN_TEST(incr_full_index);
     RUN_TEST(incr_full_has_functions);
     RUN_TEST(incr_full_edge_types);
 
-    /* Phase 2: Noop */
-    RUN_TEST(incr_noop_reindex);
-
-    /* Phase 3: Incremental deltas */
-    RUN_TEST(incr_modify_file);
-    RUN_TEST(incr_formatter_run);
-    RUN_TEST(incr_add_file);
-    RUN_TEST(incr_delete_file);
-    RUN_TEST(incr_simultaneous_changes);
-
-    /* Phase 4: Adversarial */
-    RUN_TEST(incr_empty_file);
-    RUN_TEST(incr_syntax_error);
-    RUN_TEST(incr_huge_single_function);
-    RUN_TEST(incr_binary_content);
-    RUN_TEST(incr_large_generated);
-    RUN_TEST(incr_new_subdir);
-
-    /* Phase 5: Stress */
-    RUN_TEST(incr_rapid_reindex);
-    RUN_TEST(incr_replace_file_content);
-    RUN_TEST(incr_batch_add_delete);
-
-    /* Phase 6: Recovery + accuracy */
-    RUN_TEST(incr_db_deleted_recovery);
-    RUN_TEST(incr_accuracy_vs_full);
-
-    /* Phase 7: Performance regression */
-    RUN_TEST(incr_perf_single_file_fast);
+    if (!skip_perf) {
+        /* Phase 2: Noop */
+        RUN_TEST(incr_noop_reindex);
+
+        /* Phase 3: Incremental deltas */
+        RUN_TEST(incr_modify_file);
+        RUN_TEST(incr_formatter_run);
+        RUN_TEST(incr_add_file);
+        RUN_TEST(incr_delete_file);
+        RUN_TEST(incr_simultaneous_changes);
+
+        /* Phase 4: Adversarial */
+        RUN_TEST(incr_empty_file);
+        RUN_TEST(incr_syntax_error);
+        RUN_TEST(incr_huge_single_function);
+        RUN_TEST(incr_binary_content);
+        RUN_TEST(incr_large_generated);
+        RUN_TEST(incr_new_subdir);
+
+        /* Phase 5: Stress */
+        RUN_TEST(incr_rapid_reindex);
+        RUN_TEST(incr_replace_file_content);
+        RUN_TEST(incr_batch_add_delete);
+
+        /* Phase 6: Recovery + accuracy */
+        RUN_TEST(incr_db_deleted_recovery);
+        RUN_TEST(incr_accuracy_vs_full);
+
+        /* Phase 7: Performance regression */
+        RUN_TEST(incr_perf_single_file_fast);
+    } else {
+        printf("  [phases 2-7 SKIPPED — CBM_SKIP_PERF=1]\n");
+    }
 
     /* Phase 8: list_projects + index_status + schema */
     RUN_TEST(tool_list_projects_basic);

tests/test_main.c

@@ -10,7 +10,6 @@ int tf_skip_count = 0;
 
 #include "test_framework.h"
 #include <sqlite3.h>
-#include <stdlib.h>
 
 /* Forward declarations of suite functions */
 extern void suite_arena(void);
@@ -56,13 +55,7 @@ extern void suite_yaml(void);
 extern void suite_integration(void);
 extern void suite_incremental(void);
 
-static int env_is_set(const char *name) {
-    const char *v = getenv(name);
-    return v && v[0] && v[0] != '0';
-}
-
 int main(void) {
-    int skip_perf = env_is_set("CBM_SKIP_PERF");
     printf("\n  codebase-memory-mcp  C test suite\n");
 
     /* Foundation */
@@ -153,11 +146,7 @@ int main(void) {
 
     /* Integration (end-to-end) */
     RUN_SUITE(integration);
-    if (!skip_perf) {
-        RUN_SUITE(incremental);
-    } else {
-        printf("\n=== incremental (SKIPPED — CBM_SKIP_PERF=1) ===\n");
-    }
+    RUN_SUITE(incremental);
 
     /* Release sqlite3 internal caches so ASan doesn't report them as leaks */
     sqlite3_shutdown();