Tags: SysAdminDoc/OpenTasker
Tags
fix: deep audit — thread safety, correctness, UX, and design system h… …ardening Thread safety: ArrayStore uses ConcurrentHashMap; VariableStore local scopes use ConcurrentHashMap; WiFi/Connectivity monitors mark lastState volatile; CameraMicContextEvents callback fields marked volatile. Resource leak: AutomationService.onDestroy now calls CameraMicContextEvents.stop() to unregister AppOps watchers. Data corruption: HTTP readBounded collects into ByteArrayOutputStream before UTF-8 decode to prevent multi-byte character corruption at read boundaries. Correctness: BrightnessAction auto mode sets SCREEN_BRIGHTNESS_MODE instead of writing -1; ScreenTimeoutAction clamps to 30 min max; SunEventCalculator uses event-time offset instead of noon for DST accuracy; battery_level/charging seeded from sticky broadcast at service start; FlowGraphCard uses firstOrNull to prevent NoSuchElementException; TTS SayAction guards double-resume with AtomicBoolean. Safety: vibration capped at 10s; queued task depth capped at 50; WAL checkpoint uses TRUNCATE for safer backups; PendingIntent request codes use hash to prevent overflow. UX: disabledAlpha uses Modifier.alpha instead of black overlay (fixes light theme); warning color uses amber/peach instead of green; nav icons have contentDescription; card radius uses DesignSystem.Radii.xxl token.