ADFA-2455: remove debug logs from build output

[itsaky-adfa]

See ADFA-2455 for more details.

[coderabbitai]

📝 Walkthrough

Walkthrough

Formatting and logging adjustments across tooling, builder, and logging modules: minor Kotlin formatting changes, reduced/error→debug logging for tooling server streams, runtime feature-flag gating for tooling logging, and a runtime-configurable JVM stderr appender that can be disabled via system property.

Changes

Cohort / File(s)	Summary
Project sync helper (formatting/logging) subprojects/tooling-api/src/main/java/com/itsaky/androidide/tooling/api/sync/ProjectSyncHelper.kt	Whitespace/formatting and expression-body conversions; removed several debug log calls; trailing commas and minor string concatenation/indent tweaks. No signature or control-flow changes.
Tooling server / builder logging app/src/main/java/com/itsaky/androidide/services/builder/GradleBuildService.kt, app/src/main/java/com/itsaky/androidide/services/builder/ToolingServerRunner.kt	Changed tooling-server error-stream line logging from ERROR→DEBUG; replaced compile-time constant with runtime feature flag (FeatureFlags.isDebugLoggingEnabled) to control tooling logging behavior.
JVM stderr appender runtime gating logger/src/main/java/com/itsaky/androidide/logging/JvmStdErrAppender.java	Added runtime-enabled flag read from system property ide.logging.jvmStdErrAppenderEnabled combined with LogUtils.isJvm(); added append(ILoggingEvent) override that no-ops when disabled. Minor formatting changes and new import.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• ADFA-1830: Delete project fix #748 — touches tooling-server logging and JVM stderr appender behavior; likely overlaps with logging flag changes.
• ADFA-2419: force-stop Gradle daemons after a timeout #800 — modifies tooling server logging/daemon startup paths and logging-flag wiring; likely related to runtime flag adjustments.

Suggested reviewers

• jomen-adfa
• jatezzz

Poem

🐇 I nibbled logs and smoothed the seam,
Trailing commas caught my dream,
Flags now whisper when to shout,
Stderr hushes — quiet about,
A rabbit hops, the code looks clean.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 62.50% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description references the related issue tracker ticket (ADFA-2455) and aligns with the PR's stated objective of removing debug logs from build output.
Title check	✅ Passed	The title accurately reflects the main objective of the PR—removing debug logs from build output by changing logging levels and conditionally gating appender behavior.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/ADFA-2455

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Daniel-ADFA]

LGTM!