Skip to content

Commit f14a2ff

Browse files
authored
Merge pull request #9716 from sarthakNITT/#9548
fix: improve error messages for invalid rule files
2 parents 49af89c + 9694b24 commit f14a2ff

File tree

1 file changed

+22
-1
lines changed

1 file changed

+22
-1
lines changed

core/config/yaml/loadYaml.ts

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,9 +258,30 @@ export async function configYamlToContinueConfig(options: {
258258
continueConfig.slashCommands?.push(slashCommand);
259259
}
260260
} catch (e) {
261+
// If the file is in a rules directory, we can provide a more helpful error message
262+
// because we know it's likely a rule definition
263+
const isRuleFile =
264+
file.path.toLowerCase().includes("/rules/") ||
265+
file.path.toLowerCase().includes("\\rules\\");
266+
267+
let message = `Failed to convert prompt file ${file.path} to slash command: ${e instanceof Error ? e.message : e}`;
268+
269+
if (isRuleFile) {
270+
const isYamlError =
271+
e instanceof Error &&
272+
(e.name?.includes("YAML") || e.message.includes("flow sequence"));
273+
274+
const prefix = isYamlError
275+
? "Failed to parse rule definition"
276+
: "Failed to process rule definition";
277+
278+
const errorDetails = e instanceof Error ? e.message : String(e);
279+
message = `${prefix} ${file.path}: ${errorDetails}`;
280+
}
281+
261282
localErrors.push({
262283
fatal: false,
263-
message: `Failed to convert prompt file ${file.path} to slash command: ${e instanceof Error ? e.message : e}`,
284+
message,
264285
});
265286
}
266287
});

0 commit comments

Comments
 (0)