fix: use pathToFileURL for Windows path comparison in generate-base-config-schema

easyteacher · BradGroux · commit f77464d7f485 · 2026-05-09T03:57:57.000-05:00
The script was using new URL() to compare import.meta.url with process.argv[1],
which fails on Windows because new URL() doesn't properly convert Windows paths
to file URLs. Using pathToFileURL from node:url correctly handles this conversion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -47,6 +47,7 @@ Docs: https://docs.openclaw.ai
 - Codex app-server: annotate message-tool-only direct chat turns in the dynamic `message` tool spec so visible replies are sent through `message(action="send")` instead of staying private. (#79704)
 - Codex app-server: default implicit local stdio app-server permissions to guardian when Codex system requirements disallow the YOLO approval, reviewer, or sandbox value, including hostname-scoped remote sandbox entries, avoiding turn-start failures on managed hosts that permit only reviewed approval or narrower sandboxes.
 - Plugins/install: run managed npm-root install, uninstall, prune, and repair commands from the managed root without a redundant `--prefix .`, avoiding npm 10.9.3 Arborist crashes on native Windows WhatsApp plugin installs. Fixes #78514. (#78902) Thanks @melihselamett-stack.
+- Config/schema/Windows: detect direct execution of the base config schema generator with `pathToFileURL` so Windows paths with backslashes still run the `--check` and `--write` command body. (#52989) Thanks @easyteacher.
 - Discord/voice: stream ElevenLabs TTS directly into Discord playback and send ElevenLabs latency optimization as the documented query parameter so spoken replies can start sooner.
 - Discord/voice: keep TTS playback running when another user starts speaking, ignore new capture during playback to avoid feedback loops, and downgrade expected receive-stream aborts to verbose diagnostics.
 - iMessage: expose native private-API message actions through `imsg rpc` for reactions, edits, unsends, replies, rich sends, attachments, and group management when `imsg status --json` reports the required bridge capabilities.
diff --git a/scripts/generate-base-config-schema.ts b/scripts/generate-base-config-schema.ts
@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+import { pathToFileURL } from "node:url";
 import { computeBaseConfigSchemaResponse } from "../src/config/schema-base.js";
 
 export function checkBaseConfigSchema(): void {
@@ -12,7 +13,7 @@ if (args.has("--check") && args.has("--write")) {
   throw new Error("Use either --check or --write, not both.");
 }
 
-if (import.meta.url === new URL(process.argv[1] ?? "", "file://").href) {
+if (import.meta.url === pathToFileURL(process.argv[1] ?? "").href) {
   checkBaseConfigSchema();
   if (args.has("--write")) {
     console.log("[base-config-schema] runtime-computed; no generated file to write");
