Remove unused isCloudDebugEnabled and isTerminalDebugEnabled

[kridaydave]

What Changed

Removed two unused exports from the mobile debug loggers:

• isCloudDebugEnabled in apps/mobile/src/features/cloud/cloudDebugLog.ts
• isTerminalDebugEnabled in apps/mobile/src/features/terminal/terminalDebugLog.ts

8 deleted lines, nothing added. Both files stay, because each holds a second export that is still imported:

• cloudDebugLog is imported by managedRelayState.ts:19
• terminalDebugLog is imported by terminalBufferReplay.ts:1, ThreadTerminalRouteScreen.tsx:54, NativeTerminalSurface.tsx:24, and threads/ThreadRouteScreen.tsx:71

So only the dead accessor in each file is cut, and the logger consts stay, since each is still used by the surviving log() wrapper.

Why

Neither is*DebugEnabled had a single caller. git grep -In at base 0fcd5f9061 across all tracked file types returns only each function's own definition. There is no namespace import, no import *, no index barrel in either feature directory, and no dynamic import() or require() touching either module, so no caller is hiding behind reflection either.

The change is deletion only, so there is nothing to review beyond confirming the importers.

UI Changes

None. Both functions were unreachable, so no runtime path touched them.

Verification

Run on this branch:

• pnpm test in apps/mobile: 1726 passed, 5 skipped, exit 0
• pnpm typecheck in apps/mobile: exit 0, 0 errors
• pnpm lint: exit 0, remaining warnings are pre-existing and all in apps/web
• CI: all checks pass, including Macroscope - Effect Service Conventions

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes (not applicable, no UI change)
• I included a video for animation/interaction changes (not applicable, none)

[macroscopeapp]

Approvability

Verdict: Approved at 19763e1

Macroscope's review found this PR approvable — This is a deletion-only cleanup of two unused debug-enabled accessors; the active logging paths and logger configuration remain unchanged. Repository-wide searches show no callers, leaving no material runtime or product-default impact.

Notes:

• Diff unchanged. Approvability was decided on eligibility alone.

No code changes detected at 0c43359. Prior analysis still applies.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 1c3fb5af-bee8-46fc-9ae5-7882e09f137a

📥 Commits

Reviewing files that changed from the base of the PR and between bd5237c and 0c43359.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: a32a46c9-2730-4023-9ce9-1bc914ee9fa4

📥 Commits

Reviewing files that changed from the base of the PR and between 0fcd5f9 and d20f422.

📒 Files selected for processing (2)

• apps/mobile/src/features/cloud/cloudDebugLog.ts
• apps/mobile/src/features/terminal/terminalDebugLog.ts

💤 Files with no reviewable changes (2)

• apps/mobile/src/features/terminal/terminalDebugLog.ts
• apps/mobile/src/features/cloud/cloudDebugLog.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

The cloud and terminal debug log modules no longer export functions that report whether their loggers are enabled.

Changes

Debug log API removals

Layer / File(s)	Summary
Remove debug-enabled queries apps/mobile/src/features/cloud/cloudDebugLog.ts, apps/mobile/src/features/terminal/terminalDebugLog.ts	Removed the exported isCloudDebugEnabled and isTerminalDebugEnabled functions.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to d20f4

The unused debug-state accessors can be removed without changing logging behavior or breaking repository callers. No actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely identifies the two unused exports removed by the pull request.
Description check	✅ Passed	The description covers what changed, why it changed, UI impact, verification results, and the checklist. It is focused and provides sufficient technical context.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

---

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

Dismissing prior approval to re-evaluate 19763e1