Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Web watcher: add support for Brave (`com.brave.browser` missing from KNOWN_BROWSER_PACKAGES)

Abierto Apto para principiantes
  • ¿Algo no está bien?
#306 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
android, kotlin
Área
mobile-dev

Línea de trabajo

Busca WebWatcher, extractUrl() y KNOWN_BROWSER_PACKAGES; después, inspecciona cómo se enruta el paquete de Chrome existente a su extractor de URL. Añade Brave usando su paquete y el ID de vista de la barra de direcciones, y verifica que un evento de accesibilidad de Brave produzca una URL registrada en lugar de finalizar la sesión.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Summary

The Android web watcher only extracts URLs for a hardcoded list of browsers, so users of Chromium-based browsers that aren't on that list end up with an empty aw-watcher-android-web bucket even when the Accessibility permission is granted and working.

Current behaviour

WebWatcher resolves URLs via extractUrl(), gated on KNOWN_BROWSER_PACKAGES:

  • com.android.chrome — com.android.chrome:id/url_bar
  • org.mozilla.firefox — several extractors (ADDRESSBAR_URL_BOX, url_bar_title, mozac_browser_toolbar_url_view)
  • com.sec.android.app.sbrowser — two view ids
  • com.opera.browser — two view ids
  • com.microsoft.emmx — com.microsoft.emmx:id/url_bar

Any other browser package is not matched: the event is classified as "window changed away from a tracked browser", the session is ended, and no URL is recorded. Affected browsers include Brave, Vivaldi, DuckDuckGo, and most OEM browsers.

Request

Add com.brave.browser to KNOWN_BROWSER_PACKAGES and route it to the same extractor as Chrome. Brave is Chromium-based and exposes its address bar as com.brave.browser:id/url_bar, so the existing extractTextByViewId(event, "…:id/url_bar") call should work unchanged.

Longer term

Consider a generic fallback for unknown packages: try the url_bar / url_bar_title family of view ids by matching the suffix of a node's viewIdResourceName rather than requiring a full package-qualified id, before giving up. That would cover the rest of the Chromium-derived browsers and avoid one PR per browser.

It would also help to list the currently supported browsers in the app's UI and/or in the docs on web tracking, so users don't have to read the source to find out why their web bucket is empty.

Lenguaje dominante
Kotlin
Estrellas
270
Forks
57
Merge medio
20 h 32 min
PR fusionados (30 d)
38

Preparar el entorno

Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de ActivityWatch/aw-android

Todos los issues de ActivityWatch/aw-android

Issues similares

Más issues de Kotlin

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.