Skip to content

feat(android): expose advanced debug tools on mobile - #890

Merged
H-Chris233 merged 2 commits into
Open-Less:betafrom
HKLHaoBin:feat/android-debug-tools-to-beta
Aug 3, 2026
Merged

feat(android): expose advanced debug tools on mobile#890
H-Chris233 merged 2 commits into
Open-Less:betafrom
HKLHaoBin:feat/android-debug-tools-to-beta

Conversation

@HKLHaoBin

@HKLHaoBin HKLHaoBin commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

User description

Summary

Related issues

Relates to #877, #865

Test plan

  • CI green (PR checks)
  • Signed release APK built via workflow_dispatch
  • Device: Settings -> Advanced -> Debug tools visible
  • Toggle recordAudioForDebug saves prefs
  • Export error log via SAF succeeds (content://)
  • Desktop Advanced tab unchanged

Made with Cursor


PR Type

Enhancement


Description

  • Show Advanced debug tools on Android

  • Add mobile-friendly layout for debug section

  • Display full export path on mobile success

  • Gate debug tools to desktop and Android


Diagram Walkthrough

flowchart LR
  A["AdvancedTab"] --> B["Detect platform"]
  B -- "desktop or android" --> C["DebugToolsSection"]
  B -- "desktop non-Windows" --> D["CodingAgent + ClaudeConsole"]
  C --> E["Mobile layout + full export path"]
Loading

File Walkthrough

Relevant files
Enhancement
DebugToolsSection.tsx
Mobile-friendly debug tools layout                                             

openless-all/app/src/pages/settings/DebugToolsSection.tsx

  • Add mobile layout detection via useMobileLayout
  • Stack audio max entries input vertically on mobile
  • Show description hint next to input on mobile
  • Allow success/error messages to wrap and show full export path
+31/-13 
tabs.tsx
Enable debug tools on Android                                                       

openless-all/app/src/pages/settings/tabs.tsx

  • Add Android to showDebugTools platform gating
  • Keep desktop-only sections gated to desktop
  • Clarify cross-platform intent with comments
+5/-1     

Show Advanced debug tools on Android with mobile-friendly layout. Backend SAF log export already landed in Open-Less#877.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit 0d2adb4)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ❌

877 - Partially compliant

Compliant requirements:

  • Advanced → Debug tools is now exposed on Android.
  • Debug section gets a mobile layout and shows the full export path on success.

Non-compliant requirements:

  • Backend filesDir resolution / JNI storage roots.
  • mobile_runtime initialization order (storage roots → logger → Coordinator).
  • Memory-only fallbacks, StylePack icon guard, log_dir_path() change, SAF ContentResolver export.
  • Kotlin filesDir/OpenLess prefs path change.

Requires further human verification:

  • Device check that Settings → Advanced → Debug tools is visible and usable on Android.
  • End-to-end SAF export of the error log on a real device.
  • Confirmation that prefs/logs are actually persisted under filesDir (backend behavior from fix(android): persist prefs/logs under app filesDir #877).

865 - Not compliant

Non-compliant requirements:

  • Provider-save and credential-save fixes are not included in this PR.
  • Generic 操作失败 toast remains unchanged; no actionable save error is shown.
  • Log distinguishability is not improved by this PR itself; it depends on the fix(android): persist prefs/logs under app filesDir #877 backend.

Requires further human verification:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit 0d2adb4

@H-Chris233
H-Chris233 merged commit f5fc5fd into Open-Less:beta Aug 3, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants