From 7c61c8b44e45280b6af1f7f7f5c5a33e3607b3e3 Mon Sep 17 00:00:00 2001 From: mschae23 <46165762+mschae23@users.noreply.github.com> Date: Wed, 24 Sep 2025 00:35:52 +0200 Subject: [PATCH] Use error log level, prefix messages --- src/components/options/AdvancedSkipOptionsComponent.tsx | 2 +- src/utils/skipRule.ts | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/components/options/AdvancedSkipOptionsComponent.tsx b/src/components/options/AdvancedSkipOptionsComponent.tsx index 20a7ed47..00f4230b 100644 --- a/src/components/options/AdvancedSkipOptionsComponent.tsx +++ b/src/components/options/AdvancedSkipOptionsComponent.tsx @@ -67,7 +67,7 @@ function compileConfig(config: string): AdvancedSkipRule[] | null { const { rules, errors } = parseConfig(config); for (const error of errors) { - console.log(`Error on line ${error.span.start.line}: ${error.message}`); + console.error(`[SB] Error on line ${error.span.start.line}: ${error.message}`); } if (errors.length === 0) { diff --git a/src/utils/skipRule.ts b/src/utils/skipRule.ts index 7c21a853..12830b57 100644 --- a/src/utils/skipRule.ts +++ b/src/utils/skipRule.ts @@ -737,11 +737,9 @@ export function parseConfig(config: string): { rules: AdvancedSkipRule[]; errors } expect(["if"], rule.comments.length !== 0 ? "expected `if` after `comment`" : "expected `if`", true); - rule.predicate = parsePredicate(); - expect(["disabled", "show overlay", "manual skip", "auto skip"], "expected skip option after predicate", true); - + expect(["disabled", "show overlay", "manual skip", "auto skip"], "expected skip option after condition", true); switch (previous.type) { case "disabled": rule.skipOption = CategorySkipOption.Disabled; @@ -801,7 +799,7 @@ export function parseConfig(config: string): { rules: AdvancedSkipRule[]; errors function parsePrimary(): AdvancedSkipPredicate { if (match(["("])) { const predicate = parsePredicate(); - expect([")"], "expected `)` after predicate", true); + expect([")"], "expected `)` after condition", true); return predicate; } else { return parseCheck();