Skip to content

ADFA-3421 Change from AndroidIDE to CodeOnTheGo for two log use cases#1107

Merged
hal-eisen-adfa merged 2 commits intostagefrom
ADFA-3421-Change-log-file-name-in-two-places
Mar 24, 2026
Merged

ADFA-3421 Change from AndroidIDE to CodeOnTheGo for two log use cases#1107
hal-eisen-adfa merged 2 commits intostagefrom
ADFA-3421-Change-log-file-name-in-two-places

Conversation

@hal-eisen-adfa
Copy link
Copy Markdown
Collaborator

(1) log file name, and
(2) thread logging label

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f8733730-7ee0-46f4-aeb9-a8bba0a578be

📥 Commits

Reviewing files that changed from the base of the PR and between 19e414c and f3aaf6f.

📒 Files selected for processing (2)
  • app/src/main/java/com/itsaky/androidide/activities/editor/IDELogcatReader.kt
  • logsender/src/main/java/com/itsaky/androidide/logsender/utils/LogReader.java

📝 Walkthrough

Release Notes

Changes:

  • Updated log file naming from AndroidIDE-LOG-<timestamp>.txt to CodeOnTheGo-LOG-<timestamp>.txt in the IDE logcat reader
  • Updated thread name from AndroidIDE-LogReader to CodeOnTheGo-LogReader in the log sender utility

Risks:

  • Breaking change for log file consumers: Any scripts, monitoring tools, or user workflows that depend on the old AndroidIDE-LOG-* filename pattern will need to be updated to reference the new CodeOnTheGo-LOG-* naming convention
  • Missing migration guidance: No migration path or documentation provided for users with existing log parsing/archival scripts that reference the old filename pattern

Best Practices Notes:

  • Minimal, isolated changes affecting only string literals (no logic changes)
  • No public API modifications or behavioral changes
  • Consider documenting this breaking change in migration guides or release notes for end users

Walkthrough

This PR updates branding strings across the logging infrastructure. Two files are modified: the logcat reader's log filename prefix changes from "AndroidIDE" to "CodeOnTheGo", and the LogReader thread name is similarly updated.

Changes

Cohort / File(s) Summary
Logcat Reader Branding
app/src/main/java/com/itsaky/androidide/activities/editor/IDELogcatReader.kt
Log output filename prefix updated from AndroidIDE-LOG-<timestamp>.txt to CodeOnTheGo-LOG-<timestamp>.txt.
LogReader Thread Naming
logsender/src/main/java/com/itsaky/androidide/logsender/utils/LogReader.java
Thread name passed to superclass constructor changed from "AndroidIDE-LogReader" to "CodeOnTheGo-LogReader".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • jomen-adfa
  • Daniel-ADFA

Poem

🐰 Hopping through the logs so bright,
From AndroidIDE to CodeOnTheGo in sight,
Thread names and filenames all aligned,
A fresh rebrand, perfectly designed! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% 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
Title check ✅ Passed The title accurately describes the main changes: updating references from AndroidIDE to CodeOnTheGo in two log-related locations (filename and thread name).
Description check ✅ Passed The description directly relates to the changeset, specifying the two log use cases being modified (log file name and thread logging label).

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ADFA-3421-Change-log-file-name-in-two-places

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

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

@hal-eisen-adfa hal-eisen-adfa merged commit 41cc01f into stage Mar 24, 2026
2 checks passed
@hal-eisen-adfa hal-eisen-adfa deleted the ADFA-3421-Change-log-file-name-in-two-places branch March 24, 2026 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants