fix(tests,windows): repair PR #12 Windows failures and status JSON errors

1. tests/test_index_resilience.c: the `outside` buffer in
   index_metrics_out_outside_root_refused was 64 bytes. On Windows
   cbm_mkdtemp expands the template into a ~86-byte temp path via strcpy,
   overflowing the buffer. Enlarged to 256 bytes. The other metrics_out
   test and the extra/metrics buffers (300 bytes) were checked and are
   large enough.

2. tests/test_cli.c: four Windows legacy-hook migration tests called
   cbm_install_agent_configs, which in this fork installs skills only, so
   cbm_remove_owned_legacy_hook_script never ran. They now call
   cbm_install_agent_hooks_for_testing; other calls are unchanged.

3. src/foundation/compat_fs.c: the Windows cbm_opendir returned NULL
   without setting errno when FindFirstFileW failed. execute_projects in
   src/operations/read.c treats NULL with errno != ENOENT as an
   unreadable cache directory, so `projects` failed on Windows when the
   cache directory did not exist. GetLastError() is now mapped:
   FILE_NOT_FOUND/PATH_NOT_FOUND -> ENOENT, ACCESS_DENIED -> EACCES,
   ERROR_DIRECTORY -> ENOTDIR, otherwise EIO.

4. src/main.c: the daemon status JSON printer ignored the return values
   of the yyjson add calls and could print partial or empty JSON. The
   results are now combined; on failure it prints "error: could not build
   status JSON" to stderr, and a NULL from yyjson_mut_write prints
   "error: could not serialize status JSON". The function stays void.

Author: ngallodev

src/foundation/compat_fs.c

@@ -77,6 +77,18 @@ cbm_dir_t *cbm_opendir(const char *path) {
 
     d->find_handle = FindFirstFileW(d->wide_pattern, &d->find_data);
     if (d->find_handle == INVALID_HANDLE_VALUE) {
+        /* Map the Win32 error so callers can tell a missing directory
+         * (ENOENT) from a real failure, as on POSIX. */
+        DWORD werr = GetLastError();
+        if (werr == ERROR_FILE_NOT_FOUND || werr == ERROR_PATH_NOT_FOUND) {
+            errno = ENOENT;
+        } else if (werr == ERROR_ACCESS_DENIED) {
+            errno = EACCES;
+        } else if (werr == ERROR_DIRECTORY) {
+            errno = ENOTDIR;
+        } else {
+            errno = EIO;
+        }
         free(d);
         return NULL;
     }

src/main.c

@@ -2166,21 +2166,30 @@ static void main_daemon_ctl_print_status_json(bool active,
         return;
     }
     yyjson_mut_doc_set_root(doc, root);
-    yyjson_mut_obj_add_bool(doc, root, "active", true);
-    yyjson_mut_obj_add_str(doc, root, "status", "active");
-    yyjson_mut_obj_add_bool(doc, root, "permanent", status->permanent);
-    yyjson_mut_obj_add_uint(doc, root, "pid", (unsigned long)status->daemon_pid);
-    yyjson_mut_obj_add_uint(doc, root, "clients", (unsigned)status->client_count);
-    yyjson_mut_obj_add_uint(doc, root, "committed_clients", (unsigned)status->committed_clients);
-    yyjson_mut_obj_add_bool(doc, root, "stopping", status->stopping);
-    yyjson_mut_obj_add_strcpy(doc, root, "version", status->semantic_version);
-    yyjson_mut_obj_add_strcpy(doc, root, "build", status->build_fingerprint);
+    bool ok = true;
+    ok = ok && yyjson_mut_obj_add_bool(doc, root, "active", true);
+    ok = ok && yyjson_mut_obj_add_str(doc, root, "status", "active");
+    ok = ok && yyjson_mut_obj_add_bool(doc, root, "permanent", status->permanent);
+    ok = ok && yyjson_mut_obj_add_uint(doc, root, "pid", (unsigned long)status->daemon_pid);
+    ok = ok && yyjson_mut_obj_add_uint(doc, root, "clients", (unsigned)status->client_count);
+    ok = ok && yyjson_mut_obj_add_uint(doc, root, "committed_clients",
+                                       (unsigned)status->committed_clients);
+    ok = ok && yyjson_mut_obj_add_bool(doc, root, "stopping", status->stopping);
+    ok = ok && yyjson_mut_obj_add_strcpy(doc, root, "version", status->semantic_version);
+    ok = ok && yyjson_mut_obj_add_strcpy(doc, root, "build", status->build_fingerprint);
+    if (!ok) {
+        yyjson_mut_doc_free(doc);
+        (void)fprintf(stderr, "error: could not build status JSON\n");
+        return;
+    }
     char *json = yyjson_mut_write(doc, 0, NULL);
     yyjson_mut_doc_free(doc);
-    if (json) {
-        printf("%s\n", json);
-        free(json);
+    if (!json) {
+        (void)fprintf(stderr, "error: could not serialize status JSON\n");
+        return;
     }
+    printf("%s\n", json);
+    free(json);
 }
 
 #ifdef _WIN32

tests/test_cli.c

@@ -8908,7 +8908,7 @@ TEST(cli_windows_claude_lifecycle_migrates_only_exact_owned_legacy_state) {
     free(initial_json);
     yyjson_mut_doc_free(initial_doc);
 
-    int install_rc = seeded ? cbm_install_agent_configs(tmpdir, binary_path, false, false) : -1;
+    int install_rc = seeded ? cbm_install_agent_hooks_for_testing(tmpdir, binary_path, false, false) : -1;
     char *installed_settings = read_test_file_alloc(settings_path);
     yyjson_doc *installed_doc =
         installed_settings ? yyjson_read(installed_settings, strlen(installed_settings), 0) : NULL;
@@ -9009,7 +9009,7 @@ TEST(cli_windows_claude_hook_scripts_migrate_and_uninstall_all_owned_shapes) {
     };
     char *current_scripts[sizeof(current_names) / sizeof(current_names[0])] = {0};
 
-    int initial_install_rc = cbm_install_agent_configs(tmpdir, binary_path, false, false);
+    int initial_install_rc = cbm_install_agent_hooks_for_testing(tmpdir, binary_path, false, false);
     bool current_scripts_ready = initial_install_rc == 0;
     for (size_t i = 0U; i < sizeof(current_names) / sizeof(current_names[0]); i++) {
         char current_path[768];
@@ -9022,7 +9022,7 @@ TEST(cli_windows_claude_hook_scripts_migrate_and_uninstall_all_owned_shapes) {
     }
 
     int current_upgrade_rc =
-        current_scripts_ready ? cbm_install_agent_configs(tmpdir, binary_path, false, false) : -1;
+        current_scripts_ready ? cbm_install_agent_hooks_for_testing(tmpdir, binary_path, false, false) : -1;
     bool current_legacy_removed = current_upgrade_rc == 0;
     for (size_t i = 0U; i < sizeof(legacy_names) / sizeof(legacy_names[0]); i++) {
         char current_path[768];
@@ -9049,7 +9049,7 @@ TEST(cli_windows_claude_hook_scripts_migrate_and_uninstall_all_owned_shapes) {
     }
 
     int released_upgrade_rc =
-        released_ready ? cbm_install_agent_configs(tmpdir, binary_path, false, false) : -1;
+        released_ready ? cbm_install_agent_hooks_for_testing(tmpdir, binary_path, false, false) : -1;
     bool released_legacy_removed = released_upgrade_rc == 0;
     for (size_t i = 0U; i < sizeof(legacy_names) / sizeof(legacy_names[0]); i++) {
         char legacy_path[768];

tests/test_index_resilience.c

@@ -809,7 +809,7 @@ TEST(index_metrics_out_outside_root_refused) {
         FAIL("mkdtemp failed");
     }
     rh_to_fwd_slashes(lp.tmpdir);
-    char outside[64];
+    char outside[256];
     snprintf(outside, sizeof(outside), "/tmp/cbm_resil_out_XXXXXX");
     if (!cbm_mkdtemp(outside)) {
         FAIL("mkdtemp failed");