Skip to content

Commit 406d665

Browse files
committed
chore(cli): remove debug logs from parser and direct script runner for production
2 parents 43072f8 + af81e23 commit 406d665

2 files changed

Lines changed: 1 addition & 27 deletions

File tree

src/ErrorHandler.cpp

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -206,15 +206,6 @@ namespace vix::cli
206206
const std::string cleanedLog = ::trim_build_preamble(buildLog);
207207
auto errors = ClangGccParser::parse(cleanedLog);
208208

209-
std::cerr << "[DEBUG PARSER] errors.size()=" << errors.size() << "\n";
210-
for (const auto &e : errors)
211-
{
212-
std::cerr << "[DEBUG PARSER] file=" << e.file
213-
<< " line=" << e.line
214-
<< " col=" << e.column
215-
<< " msg=" << e.message << "\n";
216-
}
217-
218209
if (errors.empty())
219210
{
220211
if (handle_unrecognized_cli_option_as_script_runtime_args(cleanedLog))

src/commands/run/detail/DirectScriptRunner.cpp

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -470,43 +470,30 @@ namespace vix::commands::RunCommand::detail
470470

471471
if (plan.shouldCompile)
472472
{
473-
info("Direct compile: " + plan.scriptPath.filename().string());
474-
475473
const LiveRunResult build = run_cmd_live_filtered_capture(
476474
plan.compileCmd,
477-
"Compiling script...",
475+
"",
478476
false,
479477
0,
480478
opt.enableSanitizers || opt.enableUbsanOnly,
481479
true);
482480

483481
if (build.exitCode != 0)
484482
{
485-
std::cerr << "[DEBUG NEW VIX] direct compile failure path reached\n";
486-
std::cerr << "[DEBUG NEW VIX] exitCode=" << build.exitCode << "\n";
487-
std::cerr << "[DEBUG NEW VIX] stdout empty=" << std::boolalpha << build.stdoutText.empty() << "\n";
488-
std::cerr << "[DEBUG NEW VIX] stderr empty=" << std::boolalpha << build.stderrText.empty() << "\n";
489-
490483
bool handled = false;
491484

492485
if (!build.stdoutText.empty() || !build.stderrText.empty())
493486
{
494487
const std::string compileLog = build.stdoutText + build.stderrText;
495488

496-
std::cerr << "[DEBUG NEW VIX] compileLog size=" << compileLog.size() << "\n";
497-
std::cerr << "[DEBUG NEW VIX] calling printBuildErrors(...)\n";
498-
499489
handled = vix::cli::ErrorHandler::printBuildErrors(
500490
compileLog,
501491
plan.scriptPath,
502492
"Script compile failed");
503-
504-
std::cerr << "[DEBUG NEW VIX] printBuildErrors handled=" << handled << "\n";
505493
}
506494

507495
if (!handled)
508496
{
509-
std::cerr << "[DEBUG NEW VIX] fallback: error(\"Script compile failed.\")\n";
510497
error("Script compile failed.");
511498
}
512499

@@ -516,10 +503,6 @@ namespace vix::commands::RunCommand::detail
516503
const std::string meta = make_direct_cache_meta(plan.scriptPath, plan);
517504
text::write_text_file(cache.metaFile, meta);
518505
}
519-
else
520-
{
521-
success("Using cached direct build");
522-
}
523506

524507
if (!plan.shouldRun)
525508
return 0;

0 commit comments

Comments
 (0)