bbs.profullstack/logicsrc
1
0
Fork 0
mirror of https://github.com/profullstack/logicsrc.git synced 2026-08-14 23:07:29 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Reject empty config path segments

Browse source
This commit is contained in:
Codex Microtask Operator 2026-06-12 18:20:37 +02:00
parent 8f4691584c
commit c6cc21409b
2 changed files with 20 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

9
packages/cli/src/config.test.ts Normal file
View file

@ -0,0 +1,9 @@
import { describe, expect, it } from "vitest";
import { getConfigValue, setConfigValue } from "./config.js";
describe("CLI config", () => {
it("rejects empty config path segments", () => {
expect(() => getConfigValue("waiting..arcade", {})).toThrow(/empty segments/);
expect(() => setConfigValue("waiting..arcade", "true", {})).toThrow(/empty segments/);
});
});

16
packages/cli/src/config.ts
View file

@ -40,14 +40,12 @@ export function writeConfig(config: JsonObject) {
} }
export function getConfigValue(path: string, config = readConfig()) { export function getConfigValue(path: string, config = readConfig()) {
return path.split(".").reduce<unknown>((current, key) => (isObject(current) ? current[key] : undefined), config); const parts = splitConfigPath(path);
return parts.reduce<unknown>((current, key) => (isObject(current) ? current[key] : undefined), config);
} }
export function setConfigValue(path: string, rawValue: string, config = readConfig()) { export function setConfigValue(path: string, rawValue: string, config = readConfig()) {
const parts = path.split(".").filter(Boolean); const parts = splitConfigPath(path);
if (parts.length === 0) {
throw new Error("Config path cannot be empty.");
}
let current: JsonObject = config; let current: JsonObject = config;
for (const part of parts.slice(0, -1)) { for (const part of parts.slice(0, -1)) {
if (!isObject(current[part])) { if (!isObject(current[part])) {
@ -59,6 +57,14 @@ export function setConfigValue(path: string, rawValue: string, config = readConf
return config; return config;
} }
function splitConfigPath(path: string) {
const parts = path.split(".");
if (parts.length === 0 || parts.some((part) => part.length === 0)) {
throw new Error("Config path cannot contain empty segments.");
}
return parts;
}
export function parseConfigValue(value: string): unknown { export function parseConfigValue(value: string): unknown {
if (value === "true") return true; if (value === "true") return true;
if (value === "false") return false; if (value === "false") return false;