Conversation
Simultaneously upgrade Gradle (8.12.1 → 9.1.0) and AGP (8.10.1 → 9.0.1)
since AGP 8.x does not support Gradle 9.x.
Uses opt-out flags (android.newDsl=false, android.builtInKotlin=false) to
preserve existing kotlin-android/kapt plugins while on AGP 9.
Version bumps:
- Gradle 8.12.1 → 9.1.0
- AGP 8.10.1 → 9.0.1
- Dagger/Hilt 2.58 → 2.59.2
- Fladle 0.19.0 → 0.21.0
AGP 9 migration fixes:
- Remove archivesBaseName from defaultConfig (use base.archivesName)
- Move compileSdk from defaultConfig to android block
- Replace proguard-android.txt with proguard-android-optimize.txt
- Rename lintOptions → lint in root build.gradle
- Replace deprecated buildDir with layout.buildDirectory
- Remove obsolete android.useAndroidX and android.enableJetifier properties
- Fix kotlin {} → java {} for sourceCompatibility in JVM-only modules
- Remove allowBackup from posttypes library manifest (app-level concern)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use rememberSaveable instead of remember for the selected index in StatusDialog, FormatDialog, and AuthorDialog so the user's choice persists through configuration changes. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Remove LoginEmailPasswordFragment * Remove Login Magic Link Fragments * Remove username/password WP.com flow * Remove Google Login * Remove LoginEmailFragment * Remove Login 2FA fragment * Remove SignupMagicLinkFragment * Remove orphaned files * Remove the “Find your site address” button * Remove SignupEpilogueFragment * Remove native account signup code * Remove `PostSignupInterstitialActivity` * Remove SmartLock * Remove `WOO_LOGIN_MODE` * Don’t show the username/password fields for Application Password sites * In the Me tab, start WP.com login directly * Fix strings * Fix WP.com login UI * Improve the WP.org login device name * Add self-hosted site reauthentication * Display more detailed error messages * Fix WP.com sharing login flow * Fix share flow login * Remove `LoginMode.JETPACK_SELFHOSTED` * Move resources out of login module into app * Flatten theme heirarchies * Modernize login prologue * Fix edge-to-edge for login * Automatically load notifications when switching to that tab * Fix Application Password login focus * Remove login module from CI * Remove unused styles * Add nullable annotation * Remove deprecated Smart Lock Credentials API and play-services-auth The only usage of play-services-auth was the deprecated Smart Lock for Passwords (Credentials API) in AppInitializer, which connected a GoogleApiClient on startup and called disableAutoSignIn on logout. play-services-fido was only present as a transitive dependency of play-services-auth. Neither is needed anymore. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove trailing comma in localization.rb array Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix trailing backslash in run-unit-tests.sh Removing :libs:login:koverXmlReportDebug left a trailing backslash on the previous line, causing TESTS_EXIT_STATUS=$? to be passed as a gradle task argument instead of being a variable assignment. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove unused imports left over from login lib removal Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Suppress detekt LongMethod for application password error handling These methods are long due to exhaustive error handling across multiple WpRequestResult variants, which is reasonable to suppress. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Delete unused bg_oval_translucent_stroke_3dp drawable Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove references to deleted signup epilogue analytics constants Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix constructor mismatches in application password test files Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove unused login prologue resources Delete login prologue drawables, colors, dimens, strings, and menu resources that became unused after the login library removal. Also update TrainOfIcons preview to use app_icon instead of deleted drawables. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Restore login_prologue_revamped colors used by wordpress source set These colors are referenced by LoginPrologueRevampedFragment.kt and Tagline.kt in src/wordpress/, so they must not be removed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove remaining unused resources from login lib removal Delete unused palette colors (purple_10, purple_20, purple_60, pink_10, orange_10, green_60, celadon_30), login prologue dimens, indicator drawables, promo illustration PNGs, PromoTitle style, and orphaned LoginSiteAddressValidatorTest. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Move login prologue revamped colors to wordpress source set These colors are only used by WordPress-flavor code in src/wordpress/, so they belong in src/wordpress/res/ rather than shared src/main/res/. This fixes the Jetpack lint check which correctly flagged them as unused. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix dark/light mode switching issues * Add a release note * Remove unused View import in LoginActivity Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove stale Google Login warnings from build scripts Google Login no longer exists in the app, so these warnings about it not working are no longer relevant. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Use bodyLarge text style for site address label The headlineSmall style competes with the TopAppBar title. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Make login form scrollable for landscape support Flatten the two-column layout into a single scrollable column so the text field and button don't get crushed when the keyboard opens on small devices in landscape mode. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix ANR risk and error handling in WP.com OAuth login - Replace runBlocking with async coroutine scope to avoid blocking the main thread during token exchange (Critical #1) - Use dedicated CoroutineScope instead of Dispatchers.IO so dispose() only cancels this helper's coroutines (Critical #2) - Propagate login errors to the caller via Consumer<Exception> instead of swallowing them - Show error state on the loading screen with a retry button instead of silently failing - Cancel coroutine scope in LoginActivity.onDestroy() Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Handle null and unknown values in LoginFlow.fromIntent() getStringExtra() can return null even when hasExtra() is true, and valueOf() throws for old enum names that no longer exist. Fall back to PROLOGUE safely in both cases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Connect site comparison logic after OAuth login After fetching sites post-OAuth login, route back through loggedInAndFinish() so FINISH_WITH_SITE flows correctly return the newly added site ID to the calling activity. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Persist login flow state in SharedPreferences instead of static fields Static fields don't survive process death and can cause issues in multi-window mode. Move sPendingLoginFlow and sIsShareFlowPending to AppPrefs with dedicated getter/setter methods. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add system bar transparency to login night theme The light theme had transparent status/navigation bars but the night theme was missing these attributes, causing opaque system bars in dark mode. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add @singleton to LoginAnalyticsTracker provider method The class had @singleton but the Dagger provider didn't, so a new instance was created on every injection. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Call startPostLoginServices and remove dead stub methods startPostLoginServices was defined but never called, so Reader tags and notification services weren't started after login. Also removes unused startOver and gotConnectedSiteInfo methods. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove leftover debug Log.e in WPMainActivity Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove plaintext username from debug logs The hasApiRestUsername boolean check is sufficient for debugging without exposing the actual credential value. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Replace deprecated onActivityCreated with onViewCreated Move activity title setup into onViewCreated, removing the suppressed deprecation warning. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Align ARG_LOGIN_FLOW constant name with its string value Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Unbind CustomTabsServiceConnection on dispose The service was bound but never unbound, causing a minor resource leak. Now unbinds when the helper is disposed in onDestroy. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Use Channel instead of SharedFlow for discovery URL event Channel guarantees the event is buffered and consumed exactly once, even if the collector starts slightly after emission. Safer than SharedFlow(replay=0) for one-shot navigation events. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Suppress overdraw lint warning on login loading layout The window background from LoginTheme triggers a false positive overdraw warning since the ConstraintLayout itself has no explicit background. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add tools:keep for flavor-specific login prologue colors Android Lint can't trace cross-flavor resource usage, so it flags these colors as unused. They're referenced by Compose code in the wordpress flavor. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Redirect WP.com site addresses to OAuth login flow When a user enters a WordPress.com site address in the site address field, redirect them to the WP.com OAuth flow instead of proceeding with application password authorization. This ensures WP.com account state is properly established so the Me tab shows the user's profile. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove old login lib references from e2e tests The LoginFlow e2e helper imported resources from the deleted login library and referenced views (login_username_row, login_password_row) that no longer exist. Remove the dead enterUsernameAndPassword and enterSiteAddress methods since login forms are now Compose-based and WP.com login uses OAuth via Custom Tabs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Call onFailure when OAuth callback is missing code parameter Previously tryLoginWithDataString silently returned if the code query parameter was absent, leaving the caller on a loading screen indefinitely. Now reports the error via the onFailure callback. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt violations and CI unit test task name - Remove unused AppLog imports from WPcomLoginHelper - Replace generic Exception catch with Result.fold to satisfy detekt - Fix kover task name to match trunk's flavor rename (no more Wasabi) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Persist post-OAuth loading state across configuration changes mIsWaitingForSitesToLoad and mOldSitesIdsForLoginUpdate were not saved in onSaveInstanceState, so a device rotation during the post-OAuth account/sites fetch would reset the flags and leave the user stuck on the loading screen. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix share flow race condition in LoginActivity onResume Gate the onResume completion check on a new mShareFlowLoginLaunched flag so LoginActivity doesn't immediately finish with RESULT_OK when the user already has sites but hasn't completed the share-flow login. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Nick Bradbury <nick.bradbury@gmail.com>
Remove android.newDsl=false and android.builtInKotlin=false to fully adopt AGP 9's new DSL and built-in Kotlin compilation support. - Remove kotlin-android plugin from all modules (AGP 9 built-in) - Migrate kapt → legacy-kapt in fluxc (AGP 9 requirement) - Migrate applicationVariants API → androidComponents.onVariants - Remove kotlin-android and kapt entries from version catalog - Keep kotlin-compose plugin (still required for Compose compiler) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove android.nonTransitiveRClass and android.nonFinalResIds (now always enabled in AGP 9) and android.enableR8.fullMode=false to adopt the new default of full R8 mode. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Fix issues introduced by login lib removal - Use single root layout with loading overlay in LoginActivity to avoid multiple setContentView() calls that orphan fragments - Wrap unbindService() in try-catch for IllegalArgumentException - Reset cached mLoginFlow in onNewIntent() to prevent stale values - Remove custom get() on loadingStateFlow to avoid recreating wrapper - Remove redundant e.printStackTrace() already logged via appLogWrapper - Require dot in site URL validation to reject single-word inputs - Make loading dialog dismissible by adding cancel discovery callback Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add hideLoadingOverlay() to LoginActivity Adds the inverse of showLoadingOverlay() so future code paths can transition back from the loading state to the fragment UI. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Re-add android.enableR8.fullMode=false to preserve the previous R8 behavior. Enabling full mode can be done as a separate follow-up after verifying keep rules with a release build. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Add diagnostic error handling for application password login failures Surface detailed error messages when application password login fails, instead of showing a generic toast or silently crashing. Catches exceptions in SiteStore encryption/decryption paths and threads error details through to the UI toast. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Send Sentry report on application password login failure Every error path in the login flow now sends a crash report via CrashLogging so we get visibility into failures in the wild. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add analytics tracking and crash logging for app password storing failures Re-applies the reverted changes from #22694 that add trackStoringFailed() calls with specific reason codes (empty_raw_data, empty_fetch_params, fetch_sites_exception, site_changed_failed, bad_data, site_not_found) and CrashLogging reports for better debugging. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Address code review: hide internal errors from toast, fix non-null assertion, add tests - Show only user-friendly message in toast, keep detailed errors in logs/crash reports - Replace site!! with safe ?: return@launch - Fix import ordering (com.* before org.*) - Add TODO comments on broad catch blocks to narrow once root cause identified - Add CrashLogging mock and 5 new tests for error branches (SiteStore error, no rows affected, bad credentials, DB exception, crash report verification) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Refactor onSiteChanged to fix LongMethod detekt finding, remove TODOs Split onSiteChanged into smaller focused methods to stay under the 60-line detekt limit: handleSiteChangedError, handleSiteChangedSuccess, validateSiteChanged, and logAndEmitSiteChangedError. Also remove TODO comments from SiteStore catch blocks. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix double Sentry reporting, unreported storeCredentials exception, and error message leaks - Remove crashLogging from ApplicationPasswordLoginHelper.trackStoringFailed so only the ViewModel's emitError sends Sentry reports with full context - Add trackStoringFailed and crashLogging.sendReportWithTag to the storeCredentials catch block so KeyStore failures are tracked - Replace technical error messages in NavigationActionData.errorMessage with opaque error codes (e.g. site_store_error, no_rows_affected) - Use e.message ?: e.javaClass.simpleName in SiteStore catch blocks as fallback for exceptions without a message Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Restore Sentry reports in helper for storeCredentials false-return paths Add crashLogging.sendReportWithTag calls in storeApplicationPasswordCredentialsFrom for the bad_data and site_not_found paths, which return false without throwing. These are not duplicates of the ViewModel's emitError reports — they cover failures where execution continues to fetchSites and the ViewModel never calls emitError. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Add commented-out forced error for testing Sentry reporting path TODO: Remove before merging. Uncomment the throw line to verify that the storeCredentials error path sends analytics and Sentry reports. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Refactor helper to fix LongMethod detekt finding Extract logAndReportBadData and logAndReportSiteNotFound from storeApplicationPasswordCredentialsFrom to bring it under the 60-line detekt limit. Also extract reportStoringFailedToSentry to deduplicate Sentry exception construction. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Remove commented-out forced error used for testing Sentry reporting Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: adalpari <adalpari@gmail.com>
Bumps [io.sentry.android.gradle](https://github.com/getsentry/sentry-android-gradle-plugin) from 6.1.0 to 6.2.0. - [Release notes](https://github.com/getsentry/sentry-android-gradle-plugin/releases) - [Changelog](https://github.com/getsentry/sentry-android-gradle-plugin/blob/main/CHANGELOG.md) - [Commits](getsentry/sentry-android-gradle-plugin@6.1.0...6.2.0) --- updated-dependencies: - dependency-name: io.sentry.android.gradle dependency-version: 6.2.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Activity Log: Show MCP agent metadata in list and detail views
Display "via {client}" label (e.g. "via Claude") in the Activity Log
when an entry was performed by an MCP agent. Adds isMCPAgent and
mcpClient fields through the full data pipeline: API response parsing,
database persistence with migration, and UI rendering in both the
list and detail views.
Ref: AIINT-290
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Fix detekt and lint issues in MCP agent metadata changes
Suppress LongParameterList for Actor class and extract hardcoded
dot separator to a string resource.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Refactor MCP metadata: extract shared logic, use uiHelpers, add tests
- Extract duplicated MCP formatting into ActivityActorExtensions
- Remove unnecessary intermediate variable in detail ViewModel
- Pass uiHelpers to EventItemViewHolder for consistent visibility handling
- Add unit tests for MCP metadata in both list and detail ViewModels
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: adalpari <adalpari@gmail.com>
Generated by 🚫 Danger |
|
|
|
|
🤖 Build Failure AnalysisThis build has failures. Claude has analyzed them - check the build annotations for details. |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Bumps [json](https://github.com/ruby/json) from 2.18.1 to 2.19.2. - [Release notes](https://github.com/ruby/json/releases) - [Changelog](https://github.com/ruby/json/blob/master/CHANGES.md) - [Commits](ruby/json@v2.18.1...v2.19.2) --- updated-dependencies: - dependency-name: json dependency-version: 2.19.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* RS Post Settings: Add polish and error handling
Replace Toast with Compose Snackbar for inline error messages with retry
actions, add pull-to-refresh support, show "Saving..." label alongside
the spinner, display "Not Set" placeholder for empty status, and improve
accessibility with content descriptions for password toggle and featured
image placeholder. Extract shared error utilities from PostRsListViewModel
into PostRsErrorUtils for reuse across both list and settings screens.
Add unit tests for PostRsSettingsViewModel.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Fix review issues from polish PR
- Preserve unsaved edits when refreshing post from server
- Remove unreachable snackbar when site is null (activity finishes immediately)
- Fix PullToRefreshBox content indentation
- Remove redundant offline mocks in refresh tests
- Rename misleading test name for status selection
- Add test for save-online sets isSaving
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Fix detekt and checkstyle issues
Extract preserveEdits() helper to shorten refreshPost() below the
60-line detekt limit, and remove empty line after opening brace in
test class.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Fix snackbar navbar overlap and center error content
Add navigationBarsPadding to SnackbarHost so snackbars aren't
obscured by the system navigation bar. Wrap ErrorContent in a
centered Box so the error message displays in the center of the
screen instead of at the top.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Center error content on term selection screen
Add fillMaxWidth and TextAlign.Center to the ErrorContent on the
term selection screen so the network error message is properly
centered horizontally.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Add retry action to save network error snackbar
The network error snackbar shown when saving in airplane mode was
missing an actionLabel and onAction callback, so no Retry button
appeared. Add both so the user can retry saving after reconnecting.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Use primary color for snackbar action button
Set the snackbar action color to MaterialTheme.colorScheme.primary
so the Retry text on snackbars matches the primary color used by
the Retry button on the error empty state.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Remove pre-flight network check from save to avoid race condition
When disabling airplane mode and immediately tapping Retry, Android may not
have re-established connectivity yet, causing a spurious network error. Now
the save always proceeds to the API call, which handles network errors
naturally via its catch block using PostRsErrorUtils.friendlyErrorMessage().
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Simplify code and remove status bar hiding
- Use BackHandler(onBack=) instead of wrapping lambda
- Use ?.let { } ?: Modifier for conditional click modifiers
- Replace PostApiException with RuntimeException
- Inline isAuthError wrapper in PostRsListViewModel
- Remove status bar hiding to fix PTR triggering on system bar pull
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Use named exception to satisfy detekt
Replace RuntimeException with PostApiRequestException to fix
TooGenericExceptionThrown detekt violations.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Cache WpApiClient to avoid repeated instantiation
On self-hosted sites, a new WpApiClient was created for every API call
(5 instantiations just to open post settings). Cache self-hosted clients
in WpApiClientProvider (mirroring the existing WP.com pattern), add a
per-site client cache in PostRsRestClient, and use a lazy property in
PostRsSettingsViewModel to reuse the same client across fetch and save.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Remove dead parameters and redundant client cache
Now that WpApiClientProvider caches self-hosted clients, the per-site
client cache in PostRsRestClient is redundant — remove it. Also remove
the now-unused site parameters from fetchPost() and savePost() in the
ViewModel, and replace shadowed locals with simple null guards.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Fix import ordering, replace crash with handled exception, remove unused methods
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Fix detekt ThrowsCount and ReturnCount violations
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Replace loading spinner with shimmer skeleton
Show hero image shimmer and placeholder rows while loading instead
of a titled app bar with a centered spinner.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Simplify code and remove redundant null checks
- Extract shared HeroOverlay composable to deduplicate gradient
and back button between loading skeleton and hero layout
- Inline statusResId variable
- Remove unreachable site null guards in ViewModel
- Remove extra blank line in WpApiClientProvider
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Add author search to author dialog
Add a debounced search field to the author selection dialog, matching
the existing pattern used for category/tag term search. Also fixes
author selection from index-based to ID-based to prevent incorrect
selection when the list changes during search or load-more.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Simplify author dialog and deduplicate constant
Replace itemsIndexed with items in AuthorDialog since the index was
unused, and consolidate AUTHORS_PER_PAGE to a single constant in
PostRsRestClient.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Clear author search state when dialog closes
Reset the search query, search flag, and cached author list on
dismiss so reopening the dialog shows a fresh state instead of
stale search results.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* RS Post Settings: Remove redundant state clearing in onAuthorClicked
The authorSearchQuery and isSearchingAuthors fields are already
reset by onDismissDialog(), so clearing them again on open is
unnecessary.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Fix detekt issues: remove unused import and extract hideStatusBar()
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Author search: reset pagination on dismiss and show error snackbar
Reset nextAuthorPageParams and canLoadMoreAuthors when the author
dialog is dismissed to prevent stale pagination state. Show a
snackbar when author search fails, consistent with other error
handling in the ViewModel.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Author search: disable OK when no results and clear search on confirm
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Bump version number * Update WordPress metadata translations for 26.7 * Update Jetpack metadata translations for 26.7
Bumps [org.mockito.kotlin:mockito-kotlin](https://github.com/mockito/mockito-kotlin) from 6.2.3 to 6.3.0. - [Release notes](https://github.com/mockito/mockito-kotlin/releases) - [Commits](mockito/mockito-kotlin@v6.2.3...v6.3.0) --- updated-dependencies: - dependency-name: org.mockito.kotlin:mockito-kotlin dependency-version: 6.3.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* RS Post Settings: Use device timezone for date picker The date/time picker was using UTC for display and selection, causing users to see times offset from their local timezone. Switch to device timezone to match the old post settings behavior. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * RS Post Settings: Move UTC constant to its only usage site Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * RS Post Settings: Append timezone label to displayed date Show the device timezone abbreviation (e.g., EST, PST) after the formatted date so users know which timezone the date refers to. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Update wordpress-rs to 1219-9cb722b47c Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add Insights tab with Year in Review card to new stats screen Populate the Insights tab with card management (add, remove, move) mirroring the Traffic tab architecture. Add Year in Review as the first Insights card, fetching yearly summaries (posts, words, likes, comments) via the wordpress-rs statsInsights endpoint. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Restyle Year in Review card with 2x2 grid of mini stat cards Update the card title to show "YEAR in review" instead of a generic title. Replace the table layout with a 2x2 grid of mini cards, each displaying an icon, label, and formatted value for Posts, Words, Likes, and Comments, matching the web design. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add Year in Review detail screen with View All functionality Add a detail screen showing all years with full stats (posts, comments, avg comments/post, likes, avg likes/post, words, avg words/post). The card shows the most recent year and a "View all" link when multiple years are available. Years are sorted newest first. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Always show current year in Year in Review card and always display Show All Ensure the current year is always present in the data, adding it with zero values if not returned by the API. The Show All footer is now always visible regardless of the number of years available. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Minor cleanup: cache current year and remove redundant variable Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix code review issues and add missing tests - Use resource string for exception errors instead of raw e.message - Add duplicate guard in addCard() - Change Error from data class to class to fix lambda equality - Use Year.now() per-call instead of cached static val - Fix isValidConfiguration to check for null entries - Remove 0dp Spacer from detail screen - Add StatsFormatterTest with 12 tests - Add repository moveCard and addCard duplicate tests Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix second round of review issues: stale success flag, race condition, siteId fallback, API type inconsistency - Reset isLoadedSuccessfully on error/exception so loadDataIfNeeded recovers after failed refresh - Add Mutex to InsightsCardsConfigurationRepository to prevent concurrent mutation races - Guard siteId in InsightsViewModel mutations to avoid operating with invalid 0L - Align fetchStatsInsights parameter from String to Long for interface consistency Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update Year in Review labels to match web app Card uses short labels (Words, Likes) without "Total" prefix. Detail screen uses exact web labels: Total posts, Total comments, Avg comments per post, Total likes, Avg likes per post, Total words, Avg words per post. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * CMM-1936: Add All-time Stats and Most Popular Day Insights cards (#22673) * Update wordpress-rs library hash Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add stats summary data layer with shared caching Add fetchStatsSummary endpoint and StatsSummaryData model to StatsDataSource. Implement Mutex-based caching in StatsRepository so All-time Stats and Most Popular Day cards share a single API call. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add All-time Stats Insights card New card showing Views, Visitors, Posts, and Comments from the statsSummary endpoint. Uses rememberShimmerBrush for loading state. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add Most Popular Day Insights card New card showing the best day for views with date, view count, and percentage. Shares the statsSummary API call with All-time Stats via repository caching. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Wire All-time Stats and Most Popular Day into Insights tab Add card types to InsightsCardType enum and default card list. Wire ViewModels and cards into InsightsTabContent. Add config migration to automatically show new card types for existing users. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix review issues: locale, empty best day, config migration, tests - Create DateTimeFormatter at format time instead of caching in companion object to respect locale changes - Handle empty viewsBestDay by returning loaded state with empty values instead of throwing - Fix config migration by persisting hiddenCards field so new card types can be distinguished from user-removed cards - Add missing tests: empty viewsBestDay, zero views percentage, exception path, dayAndMonth assertion, addNewCardTypes migration, full config no-migration Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update RS library and fix insights locale, detekt, and date casing - Update wordpress-rs to 1de57afce924622700bcc8d3a1f3ce893d8dad5b - Add locale parameter to StatsInsightsParams matching other endpoints - Fix detekt MagicNumber and TooGenericExceptionCaught violations - Capitalize first letter of formatted date in Most Popular Day card Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Move stats summary cache from StatsRepository to InsightsViewModel StatsRepository was not @singleton, so the shared cache for statsSummary data was broken — each ViewModel got its own instance. Move caching to InsightsViewModel which is activity-scoped and shared. Child ViewModels now receive a summary provider function wired from the UI layer. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix double-write and concurrency in InsightsCardsConfigurationRepository addNewCardTypes wrote to prefs directly, then the caller wrote again via saveConfiguration. Also getConfiguration was not mutex-protected. Make addNewCardTypes pure, extract loadAndMigrate for atomic read-migrate-save within the mutex, and make getConfiguration go through the mutex. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Rename hiddenCards() to computeHiddenCards() to avoid naming conflict InsightsCardsConfiguration had both a hiddenCards property (persisted list) and a hiddenCards() method (computed from visibleCards). Rename the method to make the distinction clear. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add NoData state for MostPopularDay and fix percentage precision When a site has no best day data, show a dedicated NoData state with a "No data yet" message instead of a blank Loaded card. Also reduce percentage format from 3 to 1 decimal place for consistency with typical stats UIs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Suppress TooGenericExceptionThrown detekt warning in test files Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix thread-safety, DI design, and Error state in Insights cards - Add @volatile to isLoading/isLoadedSuccessfully flags in ViewModels - Extract StatsSummaryUseCase singleton to replace summaryProvider var, removing temporal coupling and null-check error paths - Change Error from class to data class, move onRetry out of state and into composable parameters to avoid unnecessary recompositions Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * Add Most Popular Time card and centralize Insights data fetching (#22684) * Add Most Popular Time Insights card Introduce a new "Most popular time" card in the stats insights tab that shows the best day of week and best hour with their view percentages. The card reuses the insights API endpoint via a new shared StatsInsightsUseCase (following the StatsSummaryUseCase caching pattern), which also refactors YearInReviewViewModel to use the same use case. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Clean up MostPopularTimeViewModel: locale-aware hour formatting and remove unnecessary @volatile Use DateTimeFormatter.ofLocalizedTime instead of hardcoded AM/PM to respect device locale settings. Remove unnecessary @volatile annotations since all access is on the main thread via viewModelScope. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add tests for MostPopularTimeViewModel and StatsInsightsUseCase Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix day-of-week mapping, NoData condition, and add bounds check - Fix day mapping: WordPress API uses 0=Monday (not Sunday) - Show NoData when either day or hour percent is zero (not both) - Add bounds check for invalid day values (returns empty string) - Update and add tests for new behavior Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt: suppress LongMethod and remove unused import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Centralize Insights data fetching in InsightsViewModel Move data fetching from individual card ViewModels to InsightsViewModel as coordinator, ensuring each API endpoint (stats summary and insights) is called only once per load. Card ViewModels now receive results via SharedFlow instead of fetching independently, reducing duplicate network calls from 4 to 2. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix race condition, consistent onRetry pattern, and remove unused siteId - Set isDataLoading in refreshData() to prevent duplicate fetches - Move onRetry from YearInReviewCardUiState.Error to composable param - Remove unused siteId property, use resolvedSiteId() directly Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add formatHour bounds check, remove duplicate string, clean up import - Guard formatHour against invalid hour values (crash prevention) - Remove duplicate stats_insights_percent_of_views string resource - Use import for kotlin.math.round instead of fully qualified call Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt LongMethod: extract fetchSummary and fetchInsights Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Reduce duplication in MostPopularTimeCard using shared components Replace manual card container, header, error content, and shimmer boxes with StatsCardContainer, StatsCardHeader, StatsCardErrorContent, and ShimmerBox. Extract repeated day/hour section into StatSection. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Rename views percent string resource and add NoData preview - Rename stats_insights_most_popular_day_percent to stats_insights_views_percent for neutral naming - Add missing NoData preview to MostPopularTimeCard Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Trigger PR checks * Use device 24h/12h setting for hour formatting Use android.text.format.DateFormat.is24HourFormat() to respect the device time format preference instead of relying on locale. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix thread safety, CancellationException handling, and lambda allocation - Add @volatile to isDataLoaded/isDataLoading flags - Rethrow CancellationException to preserve structured concurrency - Wrap onRetryData lambda with remember to avoid recomposition allocations Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * Add Tags & Categories insights card to new stats screen (#22687) * Add Most Popular Time Insights card Introduce a new "Most popular time" card in the stats insights tab that shows the best day of week and best hour with their view percentages. The card reuses the insights API endpoint via a new shared StatsInsightsUseCase (following the StatsSummaryUseCase caching pattern), which also refactors YearInReviewViewModel to use the same use case. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Clean up MostPopularTimeViewModel: locale-aware hour formatting and remove unnecessary @volatile Use DateTimeFormatter.ofLocalizedTime instead of hardcoded AM/PM to respect device locale settings. Remove unnecessary @volatile annotations since all access is on the main thread via viewModelScope. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add tests for MostPopularTimeViewModel and StatsInsightsUseCase Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix day-of-week mapping, NoData condition, and add bounds check - Fix day mapping: WordPress API uses 0=Monday (not Sunday) - Show NoData when either day or hour percent is zero (not both) - Add bounds check for invalid day values (returns empty string) - Update and add tests for new behavior Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt: suppress LongMethod and remove unused import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Centralize Insights data fetching in InsightsViewModel Move data fetching from individual card ViewModels to InsightsViewModel as coordinator, ensuring each API endpoint (stats summary and insights) is called only once per load. Card ViewModels now receive results via SharedFlow instead of fetching independently, reducing duplicate network calls from 4 to 2. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix race condition, consistent onRetry pattern, and remove unused siteId - Set isDataLoading in refreshData() to prevent duplicate fetches - Move onRetry from YearInReviewCardUiState.Error to composable param - Remove unused siteId property, use resolvedSiteId() directly Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add formatHour bounds check, remove duplicate string, clean up import - Guard formatHour against invalid hour values (crash prevention) - Remove duplicate stats_insights_percent_of_views string resource - Use import for kotlin.math.round instead of fully qualified call Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt LongMethod: extract fetchSummary and fetchInsights Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Reduce duplication in MostPopularTimeCard using shared components Replace manual card container, header, error content, and shimmer boxes with StatsCardContainer, StatsCardHeader, StatsCardErrorContent, and ShimmerBox. Extract repeated day/hour section into StatSection. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Rename views percent string resource and add NoData preview - Rename stats_insights_most_popular_day_percent to stats_insights_views_percent for neutral naming - Add missing NoData preview to MostPopularTimeCard Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Trigger PR checks * Use device 24h/12h setting for hour formatting Use android.text.format.DateFormat.is24HourFormat() to respect the device time format preference instead of relying on locale. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix thread safety, CancellationException handling, and lambda allocation - Add @volatile to isDataLoaded/isDataLoading flags - Rethrow CancellationException to preserve structured concurrency - Wrap onRetryData lambda with remember to avoid recomposition allocations Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add Tags & Categories insights card Add a new top-list card to the Insights tab showing tags and categories with view counts. Includes expandable multi-tag groups, percentage bars, folder/tag icons, and a detail screen via Show All. Updates wordpress-rs to 1230 for the statsTags endpoint. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add expand/collapse for multi-tag groups in detail screen Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add tests for Tags & Categories feature ViewModel, repository, and display type unit tests covering success/error states, data mapping, refresh, and edge cases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Simplify Tags & Categories by reusing shared components - Use StatsCardContainer, StatsCardHeader, StatsListHeader, StatsCardEmptyContent, StatsListRowContainer from StatsCardCommon - Extract TagTypeIcon and ExpandedTagsSection into shared TagsAndCategoriesComponents to eliminate duplication between Card and DetailActivity - Add fromTagType() to TagGroupDisplayType to avoid list allocation per tag in ExpandedTagsSection - Add modifier parameter to StatsListRowContainer for clickable rows - Remove duplicated constants (CardCornerRadius, BAR_BACKGROUND_ALPHA, VERTICAL_LINE_ALPHA) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove unused stubUnknownError from ViewModel test Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix review issues: error recovery, conditional refresh, loading state - Only set isLoaded on success so loadData() retries after errors - Guard pull-to-refresh to skip tags refresh when card is hidden - Move loading state into refresh() so callers don't need showLoading() - Remove showLoading() public method - Add test for loadData() retry after error Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Address review feedback: deduplicate row composable, remove unused link field, fix thread safety and Intent size - Extract shared TagGroupRow composable into TagsAndCategoriesComponents.kt with optional position parameter, removing duplicate from Card and DetailActivity - Remove unused TagData.link field from data source, impl, and all tests - Replace Intent extras with in-memory static holder in DetailActivity to avoid TransactionTooLargeException risk - Remove unnecessary Parcelable from UI models - Use AtomicBoolean for isLoaded/isLoading flags in ViewModel for thread safety Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix concurrent refresh, process death, isExpandable duplication, and accessibility - Cancel in-flight fetch job on refresh() to prevent stale overwrites - Finish detail activity on process death instead of showing blank screen - Extract TagGroupUiItem.isExpandable computed property to deduplicate logic - Add content descriptions for TagTypeIcon and expand/collapse chevron icons Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update configuration tests to include TAGS_AND_CATEGORIES card type Fixes CI test failures caused by the new TAGS_AND_CATEGORIES card not being reflected in test fixtures and assertions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * CMM-1952: stats insights: tags card and some fixes (#22701) * Add Most Popular Time Insights card Introduce a new "Most popular time" card in the stats insights tab that shows the best day of week and best hour with their view percentages. The card reuses the insights API endpoint via a new shared StatsInsightsUseCase (following the StatsSummaryUseCase caching pattern), which also refactors YearInReviewViewModel to use the same use case. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Clean up MostPopularTimeViewModel: locale-aware hour formatting and remove unnecessary @volatile Use DateTimeFormatter.ofLocalizedTime instead of hardcoded AM/PM to respect device locale settings. Remove unnecessary @volatile annotations since all access is on the main thread via viewModelScope. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add tests for MostPopularTimeViewModel and StatsInsightsUseCase Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix day-of-week mapping, NoData condition, and add bounds check - Fix day mapping: WordPress API uses 0=Monday (not Sunday) - Show NoData when either day or hour percent is zero (not both) - Add bounds check for invalid day values (returns empty string) - Update and add tests for new behavior Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt: suppress LongMethod and remove unused import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Centralize Insights data fetching in InsightsViewModel Move data fetching from individual card ViewModels to InsightsViewModel as coordinator, ensuring each API endpoint (stats summary and insights) is called only once per load. Card ViewModels now receive results via SharedFlow instead of fetching independently, reducing duplicate network calls from 4 to 2. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix race condition, consistent onRetry pattern, and remove unused siteId - Set isDataLoading in refreshData() to prevent duplicate fetches - Move onRetry from YearInReviewCardUiState.Error to composable param - Remove unused siteId property, use resolvedSiteId() directly Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add formatHour bounds check, remove duplicate string, clean up import - Guard formatHour against invalid hour values (crash prevention) - Remove duplicate stats_insights_percent_of_views string resource - Use import for kotlin.math.round instead of fully qualified call Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix detekt LongMethod: extract fetchSummary and fetchInsights Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Reduce duplication in MostPopularTimeCard using shared components Replace manual card container, header, error content, and shimmer boxes with StatsCardContainer, StatsCardHeader, StatsCardErrorContent, and ShimmerBox. Extract repeated day/hour section into StatSection. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Rename views percent string resource and add NoData preview - Rename stats_insights_most_popular_day_percent to stats_insights_views_percent for neutral naming - Add missing NoData preview to MostPopularTimeCard Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Trigger PR checks * Use device 24h/12h setting for hour formatting Use android.text.format.DateFormat.is24HourFormat() to respect the device time format preference instead of relying on locale. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix thread safety, CancellationException handling, and lambda allocation - Add @volatile to isDataLoaded/isDataLoading flags - Rethrow CancellationException to preserve structured concurrency - Wrap onRetryData lambda with remember to avoid recomposition allocations Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add Tags & Categories insights card Add a new top-list card to the Insights tab showing tags and categories with view counts. Includes expandable multi-tag groups, percentage bars, folder/tag icons, and a detail screen via Show All. Updates wordpress-rs to 1230 for the statsTags endpoint. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add expand/collapse for multi-tag groups in detail screen Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add tests for Tags & Categories feature ViewModel, repository, and display type unit tests covering success/error states, data mapping, refresh, and edge cases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Simplify Tags & Categories by reusing shared components - Use StatsCardContainer, StatsCardHeader, StatsListHeader, StatsCardEmptyContent, StatsListRowContainer from StatsCardCommon - Extract TagTypeIcon and ExpandedTagsSection into shared TagsAndCategoriesComponents to eliminate duplication between Card and DetailActivity - Add fromTagType() to TagGroupDisplayType to avoid list allocation per tag in ExpandedTagsSection - Add modifier parameter to StatsListRowContainer for clickable rows - Remove duplicated constants (CardCornerRadius, BAR_BACKGROUND_ALPHA, VERTICAL_LINE_ALPHA) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove unused stubUnknownError from ViewModel test Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix review issues: error recovery, conditional refresh, loading state - Only set isLoaded on success so loadData() retries after errors - Guard pull-to-refresh to skip tags refresh when card is hidden - Move loading state into refresh() so callers don't need showLoading() - Remove showLoading() public method - Add test for loadData() retry after error Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Address review feedback: deduplicate row composable, remove unused link field, fix thread safety and Intent size - Extract shared TagGroupRow composable into TagsAndCategoriesComponents.kt with optional position parameter, removing duplicate from Card and DetailActivity - Remove unused TagData.link field from data source, impl, and all tests - Replace Intent extras with in-memory static holder in DetailActivity to avoid TransactionTooLargeException risk - Remove unnecessary Parcelable from UI models - Use AtomicBoolean for isLoaded/isLoading flags in ViewModel for thread safety Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix concurrent refresh, process death, isExpandable duplication, and accessibility - Cancel in-flight fetch job on refresh() to prevent stale overwrites - Finish detail activity on process death instead of showing blank screen - Extract TagGroupUiItem.isExpandable computed property to deduplicate logic - Add content descriptions for TagTypeIcon and expand/collapse chevron icons Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update configuration tests to include TAGS_AND_CATEGORIES card type Fixes CI test failures caused by the new TAGS_AND_CATEGORIES card not being reflected in test fixtures and assertions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fetch tags data independently in detail screen instead of using static field The detail screen now has its own ViewModel that fetches up to 100 items directly from the API, while the card continues to fetch 10. This removes the static data holder pattern that was prone to data loss on process death. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Extract shared mapper, add detail VM tests, guard double calls, show all 10 card items - Extract TagsAndCategoriesMapper to deduplicate TagGroupData to TagGroupUiItem mapping between card and detail ViewModels - Add unit tests for TagsAndCategoriesDetailViewModel - Add isLoaded/isLoading guards to detail VM to prevent double fetches - Remove CARD_MAX_ITEMS limit so card displays all 10 fetched items - Remove unused import in DetailActivity Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Skip data fetching for hidden Insights cards Only call summary/insights endpoints when visible cards need them, avoiding unnecessary network calls for hidden cards. Track which endpoint groups have been fetched so re-adding a hidden card triggers a fetch for its missing data. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Early return in fetchData when no endpoints are needed Prevents setting isDataLoaded=true when no cards require fetching, which would block future fetches when cards are re-added to the visible list. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Address code review findings: reduce duplication and improve tests - Replace duplicate shimmer animation in YearInReviewCard with shared rememberShimmerBrush() utility - Extract StatsTagsUseCase to centralize token validation and repository init, removing duplication between Tags ViewModels - Add card reordering tests for middle elements in InsightsCardsConfigurationRepositoryTest - Fix locale-dependent assertions in MostPopularDayViewModelTest Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Suppress LargeClass detekt warning on InsightsViewModelTest Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix process-death restore and add caching to StatsTagsUseCase - Call loadData() unconditionally in TagsAndCategoriesDetailActivity onCreate to handle process-death restore (loadData guard prevents double fetch on rotation) - Add Mutex-protected in-memory cache to StatsTagsUseCase, consistent with StatsSummaryUseCase and StatsInsightsUseCase - Pass forceRefresh=true on pull-to-refresh in TagsAndCategoriesViewModel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Extract BaseTagsAndCategoriesViewModel and use localized error messages Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix thread safety and error handling in ViewModels Use AtomicBoolean with compareAndSet in InsightsViewModel to prevent race conditions, rethrow CancellationException in base tags ViewModel, and only mark endpoints as fetched on success results. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Extract isCacheHit method to fix detekt ComplexCondition Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Cancel in-flight fetch job before refreshing in InsightsViewModel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix compilation errors after trunk merge Update wordpress-rs to version with stats tags types and remove duplicate NoConnectionContent composable and redundant else branches that caused -Werror failures. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Address code review findings and add base ViewModel tests - Move network call outside mutex in StatsTagsUseCase to avoid blocking concurrent callers during slow requests - Add main-thread-confinement comments for fetchJob fields - Document why TAGS_AND_CATEGORIES is absent from needsSummary/needsInsights (uses its own fetch path) - Restore card max items to 7 (was unintentionally changed to 10 during refactoring) - Add tests for BaseTagsAndCategoriesViewModel Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix detekt findings: suppress ReturnCount, remove unused composable - Suppress ReturnCount on StatsTagsUseCase.invoke (3 returns are clearer than restructuring with nested conditions) - Remove unused PlaceholderTabContent composable Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix TOCTOU race, config fetch trigger, and detail empty state - StatsTagsUseCase: use in-flight Deferred to coalesce concurrent requests with the same params, eliminating the TOCTOU race where two callers could both miss cache and fire duplicate requests - BaseTagsAndCategoriesViewModel: make isLoaded private since no subclass accesses it directly - InsightsViewModel: trigger loadDataIfNeeded() from updateFromConfiguration when new endpoints are required, instead of relying on the UI to call it - TagsAndCategoriesDetailActivity: add empty-state message when the loaded items list is empty Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Deduplicate detail VM tests, fix Mockito import, add cancellation test - Remove duplicate tests from TagsAndCategoriesDetailViewModelTest that are already covered by BaseTagsAndCategoriesViewModelTest; keep only initial-state and maxItems-specific tests - Replace fully-qualified org.mockito.Mockito.times() with the imported mockito-kotlin times() throughout InsightsViewModelTest - Add test verifying that rapid double-refresh cancels the first job so only one forceRefresh call completes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Clear stats caches on screen open to prevent stale data Add clearCache() to StatsInsightsUseCase, StatsSummaryUseCase, and StatsTagsUseCase. Call all three from InsightsViewModel init so that reopening the insights screen always fetches fresh data while still sharing results between cards within a single session. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Update wordpress-rs to trunk-262a778ead5f163f3450d62adfac21fb32048714 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * Simplify: fix StatsTagsUseCase error handling, deduplicate bottom sheets - Fix critical bug in StatsTagsUseCase where an exception from fetchTags() would leave the CompletableDeferred incomplete, hanging all awaiters - Extract generic AddCardBottomSheet<T> to replace three duplicate implementations (Stats, Insights, Subscribers) - Merge duplicate LaunchedEffect(cardsToLoad) blocks in InsightsTabContent Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix PR #22711 review issues: race conditions, code quality, API compat - Fix race condition in InsightsViewModel.refreshData() by nulling fetchJob before cancel and guarding finally block for current job - Fix StatsTagsUseCase cancellation propagation: complete deferred with error instead of completeExceptionally for CancellationException - Remove redundant hiddenCards constructor param, make it a computed property; use PersistedConfig for backward-compatible JSON migration - Key expandedGroups on items in TagsAndCategoriesCard and detail activity so expansion state resets on data refresh - Use StatsCardContainer in AllTimeStatsCard and MostPopularDayCard instead of duplicating border/clip/background pattern - Replace Year.now() (API 26+) with Calendar in YearInReviewViewModel - Move @Suppress off inline catch to function level in StatsTagsUseCase - Fix fully qualified references in repository test (import never, Gson) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix insights tab bugs: cancellation, race conditions, perf, and quality - Fix StatsTagsUseCase cancellation propagation (deferred.cancel instead of error) - Fix InsightsViewModel card-add race by cancelling in-flight fetch on config change - Release mutex during network calls in StatsSummaryUseCase and StatsInsightsUseCase - Replace Calendar with java.time.Year in YearInReviewViewModel - Add NoData state for empty tags response instead of empty Loaded card - Show period selector only on Traffic tab, not Insights - Add accessibility onClickLabel to AddCardItem bottom sheet Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Suppress ReturnCount detekt warning in use cases The mutex refactor introduced a third return path (guard clause pattern) which is idiomatic and readable. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix retry, error handling, race condition, and localization bugs - Reset isLoaded on exception in BaseTagsAndCategoriesViewModel so retry is not a no-op after a failed load - Complete deferred with TagsResult.Error instead of completeExceptionally in StatsTagsUseCase so non-owner callers get a proper result - Replace raw English error strings with localized R.string.stats_error_unknown in InsightsViewModel via ResourceProvider - Fix updateFromConfiguration race window by cancelling job first and calling fetchData() directly with isDataLoading already set to true Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Deduplicate error UI, remove raw Context, fix Intent data passing, add logging - Replace duplicate ErrorContent composables in AllTimeStatsCard and MostPopularDayCard with shared StatsCardErrorContent - Replace raw Context injection in MostPopularTimeViewModel with DateFormatWrapper for better testability - Refactor YearInReviewDetailActivity to fetch data via ViewModel and StatsInsightsUseCase instead of passing Parcelable list via Intent - Add AppLog warning in MostPopularDayViewModel.parseBestDay catch block to aid debugging of unparseable date strings Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Use LazyColumn, shared shimmer, and reduce line wrapping in insights tab - Convert Column+verticalScroll to LazyColumn in InsightsTabContent for lazy composition of off-screen cards - Add ProvideShimmerBrush/LocalShimmerBrush CompositionLocal so all cards share a single synchronized shimmer animation - Reduce overly aggressive line wrapping across ViewModels and InsightsTabContent, keeping lines within 120 chars but no longer wrapping well under the limit Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Extract shared YearSummary mapping, sync detail screen shimmer - Move toUiModel/ensureCurrentYear from both YearInReview ViewModels to YearSummary.Companion to eliminate duplication - Wrap detail screen loading state in ProvideShimmerBrush so multiple ShimmerBox instances share a single synchronized animation Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Remove unused Box import in YearInReviewDetailActivity Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix missing resourceProvider in InsightsViewModelTest constructors Add resourceProvider parameter to two remaining InsightsViewModel constructor calls in tests, and stub stats_error_unknown string resource to fix error-path test. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Project dependencies changeslist- Removed Dependencies
androidx.credentials:credentials:1.2.0
androidx.credentials:credentials-play-services-auth:1.2.0
com.google.android.gms:play-services-auth:20.7.0
com.google.android.gms:play-services-auth-api-phone:18.0.1
com.google.android.gms:play-services-auth-base:18.0.4
com.google.android.gms:play-services-fido:20.1.0
com.google.android.libraries.identity.googleid:googleid:1.1.0
! Upgraded Dependencies
rs.wordpress.api:android:trunk-262a778ead5f163f3450d62adfac21fb32048714, (changed from trunk-502e9561f2a68294f0065867bab9214cc9a6b78c)
rs.wordpress.api:kotlin:trunk-262a778ead5f163f3450d62adfac21fb32048714, (changed from trunk-502e9561f2a68294f0065867bab9214cc9a6b78c)tree +--- project :libs:fluxc
-| \--- rs.wordpress.api:android:trunk-502e9561f2a68294f0065867bab9214cc9a6b78c
-| +--- com.squareup.okhttp3:okhttp:5.3.2 (*)
-| +--- com.squareup.okhttp3:okhttp-tls:5.3.2
-| | +--- com.squareup.okhttp3:okhttp:5.3.2 (*)
-| | +--- com.squareup.okio:okio:3.16.4 (*)
-| | \--- org.jetbrains.kotlin:kotlin-stdlib:2.2.21 -> 2.3.10 (*)
-| +--- net.java.dev.jna:jna:5.18.1
-| +--- rs.wordpress.api:kotlin:trunk-502e9561f2a68294f0065867bab9214cc9a6b78c
-| | +--- com.squareup.okhttp3:okhttp:5.3.2 (*)
-| | +--- com.squareup.okhttp3:okhttp-tls:5.3.2 (*)
-| | +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.10.2 (*)
-| | \--- org.jetbrains.kotlin:kotlin-stdlib:2.1.21 -> 2.3.10 (*)
-| \--- org.jetbrains.kotlin:kotlin-stdlib:2.1.21 -> 2.3.10 (*)
+| \--- rs.wordpress.api:android:trunk-262a778ead5f163f3450d62adfac21fb32048714
+| +--- com.squareup.okhttp3:okhttp:5.3.2 (*)
+| +--- com.squareup.okhttp3:okhttp-tls:5.3.2
+| | +--- com.squareup.okhttp3:okhttp:5.3.2 (*)
+| | +--- com.squareup.okio:okio:3.16.4 (*)
+| | \--- org.jetbrains.kotlin:kotlin-stdlib:2.2.21 -> 2.3.10 (*)
+| +--- net.java.dev.jna:jna:5.18.1
+| +--- rs.wordpress.api:kotlin:trunk-262a778ead5f163f3450d62adfac21fb32048714
+| | +--- com.squareup.okhttp3:okhttp:5.3.2 (*)
+| | +--- com.squareup.okhttp3:okhttp-tls:5.3.2 (*)
+| | +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.10.2 (*)
+| | \--- org.jetbrains.kotlin:kotlin-stdlib:2.1.21 -> 2.3.10 (*)
+| \--- org.jetbrains.kotlin:kotlin-stdlib:2.1.21 -> 2.3.10 (*)
++--- com.gravatar:gravatar:2.5.0
+| +--- org.jetbrains.kotlin:kotlin-parcelize-runtime:2.0.21 -> 2.3.10 (*)
+| +--- com.squareup.moshi:moshi:1.15.1
+| | +--- com.squareup.okio:okio:3.7.0 -> 3.16.4 (*)
+| | \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.21 -> 1.9.24 (*)
+| +--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.7.3 -> 1.10.2 (*)
+| +--- com.squareup.okhttp3:okhttp:4.12.0 -> 5.3.2 (*)
+| \--- org.jetbrains.kotlin:kotlin-stdlib:2.0.21 -> 2.3.10 (*)
++--- com.google.android.play:app-update:2.1.0
+| +--- com.google.android.gms:play-services-basement:18.1.0 -> 18.8.0
+| | +--- androidx.collection:collection:1.0.0 -> 1.5.0 (*)
+| | +--- androidx.core:core:1.2.0 -> 1.16.0 (*)
+| | \--- androidx.fragment:fragment:1.1.0 -> 1.8.9 (*)
+| \--- com.google.android.gms:play-services-tasks:18.0.2 -> 18.4.0
+| \--- com.google.android.gms:play-services-basement:18.8.0 (*)
++--- com.google.mlkit:barcode-scanning-common:17.0.0
+| \--- com.google.mlkit:vision-common:17.0.0 -> 17.3.0
+| \--- com.google.android.gms:play-services-base:18.1.0 -> 18.5.0
+| +--- androidx.collection:collection:1.0.0 -> 1.5.0 (*)
+| +--- androidx.core:core:1.2.0 -> 1.16.0 (*)
+| +--- androidx.fragment:fragment:1.0.0 -> 1.8.9 (*)
+| +--- com.google.android.gms:play-services-basement:18.4.0 -> 18.8.0 (*)
+| \--- com.google.android.gms:play-services-tasks:18.2.0 -> 18.4.0 (*)
++--- com.zendesk:support:5.5.3
+| \--- com.google.dagger:dagger-android:2.48.1 -> 2.59.2
+| +--- com.google.dagger:dagger:2.59.2 (*)
+| +--- com.google.dagger:dagger-lint-aar:2.59.2
+| +--- androidx.annotation:annotation:1.7.1 -> 1.9.1 (*)
+| \--- javax.inject:javax.inject:1
++--- com.google.dagger:dagger-android-support:2.59.2
+| +--- com.google.dagger:dagger:2.59.2 (*)
+| +--- com.google.dagger:dagger-android:2.59.2 (*)
+| +--- com.google.dagger:dagger-lint-aar:2.59.2
+| +--- androidx.activity:activity:1.5.1 -> 1.10.1 (*)
+| +--- androidx.annotation:annotation:1.7.1 -> 1.9.1 (*)
+| +--- androidx.appcompat:appcompat:1.3.1 -> 1.7.1 (*)
+| +--- androidx.fragment:fragment:1.5.1 -> 1.8.9 (*)
+| +--- androidx.lifecycle:lifecycle-common:2.5.1 -> 2.10.0 (*)
+| +--- androidx.lifecycle:lifecycle-viewmodel:2.5.1 -> 2.10.0 (*)
+| +--- androidx.lifecycle:lifecycle-viewmodel-savedstate:2.5.1 -> 2.10.0 (*)
+| \--- javax.inject:javax.inject:1
-+--- project :libs:login
-| +--- com.google.android.gms:play-services-auth:20.4.1 -> 20.7.0
-| | +--- androidx.fragment:fragment:1.0.0 -> 1.8.9 (*)
-| | +--- androidx.loader:loader:1.0.0 (*)
-| | +--- com.google.android.gms:play-services-auth-api-phone:18.0.1
-| | | +--- com.google.android.gms:play-services-base:18.0.1 -> 18.5.0
-| | | | +--- androidx.collection:collection:1.0.0 -> 1.5.0 (*)
-| | | | +--- androidx.core:core:1.2.0 -> 1.16.0 (*)
-| | | | +--- androidx.fragment:fragment:1.0.0 -> 1.8.9 (*)
-| | | | +--- com.google.android.gms:play-services-basement:18.4.0 -> 18.8.0
-| | | | | +--- androidx.collection:collection:1.0.0 -> 1.5.0 (*)
-| | | | | +--- androidx.core:core:1.2.0 -> 1.16.0 (*)
-| | | | | \--- androidx.fragment:fragment:1.1.0 -> 1.8.9 (*)
-| | | | \--- com.google.android.gms:play-services-tasks:18.2.0 -> 18.4.0
-| | | | \--- com.google.android.gms:play-services-basement:18.8.0 (*)
-| | | +--- com.google.android.gms:play-services-basement:18.0.0 -> 18.8.0 (*)
-| | | \--- com.google.android.gms:play-services-tasks:18.0.1 -> 18.4.0 (*)
-| | +--- com.google.android.gms:play-services-auth-base:18.0.4
-| | | +--- androidx.collection:collection:1.0.0 -> 1.5.0 (*)
-| | | +--- com.google.android.gms:play-services-base:18.0.1 -> 18.5.0 (*)
-| | | +--- com.google.android.gms:play-services-basement:18.0.0 -> 18.8.0 (*)
-| | | \--- com.google.android.gms:play-services-tasks:18.0.1 -> 18.4.0 (*)
-| | +--- com.google.android.gms:play-services-base:18.0.1 -> 18.5.0 (*)
-| | +--- com.google.android.gms:play-services-basement:18.2.0 -> 18.8.0 (*)
-| | +--- com.google.android.gms:play-services-fido:20.0.1 -> 20.1.0
-| | | +--- com.google.android.gms:play-services-base:18.0.1 -> 18.5.0 (*)
-| | | +--- com.google.android.gms:play-services-basement:18.0.0 -> 18.8.0 (*)
-| | | \--- com.google.android.gms:play-services-tasks:18.0.1 -> 18.4.0 (*)
-| | \--- com.google.android.gms:play-services-tasks:18.0.1 -> 18.4.0 (*)
-| +--- org.jetbrains.kotlin:kotlin-stdlib:2.3.10 (*)
-| +--- org.wordpress:utils:3.14.0 (*)
-| +--- com.gravatar:gravatar:2.5.0
-| | +--- org.jetbrains.kotlin:kotlin-parcelize-runtime:2.0.21 -> 2.3.10 (*)
-| | +--- com.squareup.moshi:moshi:1.15.1
-| | | +--- com.squareup.okio:okio:3.7.0 -> 3.16.4 (*)
-| | | \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.21 -> 1.9.24 (*)
-| | +--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.7.3 -> 1.10.2 (*)
-| | +--- com.squareup.okhttp3:okhttp:4.12.0 -> 5.3.2 (*)
-| | \--- org.jetbrains.kotlin:kotlin-stdlib:2.0.21 -> 2.3.10 (*)
-| +--- androidx.appcompat:appcompat:1.7.1 (*)
-| +--- androidx.constraintlayout:constraintlayout:2.2.1 (*)
-| +--- com.google.android.material:material:1.12.0 (*)
-| +--- androidx.core:core:1.16.0 (*)
-| +--- project :libs:fluxc (*)
-| +--- com.github.bumptech.glide:glide:5.0.5 (*)
-| +--- androidx.credentials:credentials:1.2.0
-| | +--- androidx.annotation:annotation:1.5.0 -> 1.9.1 (*)
-| | +--- org.jetbrains.kotlin:kotlin-stdlib:1.8.22 -> 2.3.10 (*)
-| | +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.7.1 -> 1.10.2 (*)
-| | \--- androidx.credentials:credentials-play-services-auth:1.2.0 (c)
-| +--- androidx.credentials:credentials-play-services-auth:1.2.0
-| | +--- androidx.credentials:credentials:1.2.0 (*)
-| | +--- com.google.android.gms:play-services-auth:20.7.0 (*)
-| | +--- com.google.android.gms:play-services-fido:20.1.0 (*)
-| | +--- com.google.android.libraries.identity.googleid:googleid:1.1.0
-| | | +--- org.jetbrains.kotlin:kotlin-stdlib:1.8.0 -> 2.3.10 (*)
-| | | \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0 -> 1.9.24 (*)
-| | +--- org.jetbrains.kotlin:kotlin-stdlib:1.8.22 -> 2.3.10 (*)
-| | \--- androidx.credentials:credentials:1.2.0 (c)
-| +--- com.google.dagger:dagger:2.59.2 (*)
-| \--- com.google.dagger:dagger-android-support:2.59.2
-| +--- com.google.dagger:dagger:2.59.2 (*)
-| +--- com.google.dagger:dagger-android:2.59.2
-| | +--- com.google.dagger:dagger:2.59.2 (*)
-| | +--- com.google.dagger:dagger-lint-aar:2.59.2
-| | +--- androidx.annotation:annotation:1.7.1 -> 1.9.1 (*)
-| | \--- javax.inject:javax.inject:1
-| +--- com.google.dagger:dagger-lint-aar:2.59.2
-| +--- androidx.activity:activity:1.5.1 -> 1.10.1 (*)
-| +--- androidx.annotation:annotation:1.7.1 -> 1.9.1 (*)
-| +--- androidx.appcompat:appcompat:1.3.1 -> 1.7.1 (*)
-| +--- androidx.fragment:fragment:1.5.1 -> 1.8.9 (*)
-| +--- androidx.lifecycle:lifecycle-common:2.5.1 -> 2.10.0 (*)
-| +--- androidx.lifecycle:lifecycle-viewmodel:2.5.1 -> 2.10.0 (*)
-| +--- androidx.lifecycle:lifecycle-viewmodel-savedstate:2.5.1 -> 2.10.0 (*)
-| \--- javax.inject:javax.inject:1
-+--- com.gravatar:gravatar:2.5.0 (*)
-+--- com.google.android.play:app-update:2.1.0
-| +--- com.google.android.gms:play-services-basement:18.1.0 -> 18.8.0 (*)
-| \--- com.google.android.gms:play-services-tasks:18.0.2 -> 18.4.0 (*)
-+--- com.google.android.gms:play-services-auth:20.4.1 -> 20.7.0 (*)
-+--- com.google.mlkit:barcode-scanning-common:17.0.0
-| \--- com.google.mlkit:vision-common:17.0.0 -> 17.3.0
-| \--- com.google.android.gms:play-services-base:18.1.0 -> 18.5.0 (*)
-+--- com.zendesk:support:5.5.3
-| \--- com.google.dagger:dagger-android:2.48.1 -> 2.59.2 (*)
-+--- com.google.dagger:dagger-android-support:2.59.2 (*)
-\--- rs.wordpress.api:android:trunk-502e9561f2a68294f0065867bab9214cc9a6b78c (*)
+\--- rs.wordpress.api:android:trunk-262a778ead5f163f3450d62adfac21fb32048714 (*) |
Project manifest changes for WordPressThe following changes in the --- ./build/reports/diff_manifest/WordPress/wordpressRelease/base_manifest.txt 2026-03-20 12:17:06.890018301 +0000
+++ ./build/reports/diff_manifest/WordPress/wordpressRelease/head_manifest.txt 2026-03-20 12:17:41.000065287 +0000
@@ -2,8 +2,8 @@
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="org.wordpress.android"
android:installLocation="auto"
- android:versionCode="1487"
- android:versionName="26.7-rc-1" >
+ android:versionCode="1488"
+ android:versionName="26.7" >
<uses-sdk
android:minSdkVersion="26"
@@ -205,6 +205,14 @@
android:exported="false"
android:theme="@style/WordPress.NoActionBar" />
<activity
+ android:name="org.wordpress.android.ui.newstats.yearinreview.YearInReviewDetailActivity"
+ android:exported="false"
+ android:theme="@style/WordPress.NoActionBar" />
+ <activity
+ android:name="org.wordpress.android.ui.newstats.tagsandcategories.TagsAndCategoriesDetailActivity"
+ android:exported="false"
+ android:theme="@style/WordPress.NoActionBar" />
+ <activity
android:name="org.wordpress.android.ui.newstats.subscribers.subscriberslist.SubscribersListDetailActivity"
android:exported="false"
android:theme="@style/WordPress.NoActionBar" />
@@ -219,7 +227,8 @@
<activity
android:name="org.wordpress.android.ui.accounts.LoginActivity"
android:exported="true"
- android:theme="@style/LoginTheme.TransparentSystemBars"
+ android:launchMode="singleTop"
+ android:theme="@style/LoginTheme"
android:windowSoftInputMode="adjustResize" >
<intent-filter>
<action android:name="android.intent.action.VIEW" />
@@ -235,7 +244,7 @@
<activity
android:name="org.wordpress.android.ui.accounts.applicationpassword.ApplicationPasswordLoginActivity"
android:exported="true"
- android:theme="@style/LoginTheme.TransparentSystemBars"
+ android:theme="@style/LoginTheme"
android:windowSoftInputMode="adjustResize" >
<intent-filter>
<action android:name="android.intent.action.VIEW" />
@@ -276,14 +285,6 @@
</intent-filter>
</activity>
<activity
- android:name="org.wordpress.android.ui.accounts.SignupEpilogueActivity"
- android:label="@string/signup_epilogue_screen_title"
- android:theme="@style/LoginTheme" />
- <activity
- android:name="org.wordpress.android.ui.accounts.PostSignupInterstitialActivity"
- android:label="@string/post_signup_interstitial_title"
- android:theme="@style/LoginTheme" />
- <activity
android:name="org.wordpress.android.ui.sitecreation.SiteCreationActivity"
android:theme="@style/WordPress.NoActionBar"
android:windowSoftInputMode="adjustResize" />
@@ -1118,11 +1119,6 @@
android:label="Installation Referrer Service"
android:permission="android.permission.BIND_JOB_SERVICE" />
<service
- android:name="org.wordpress.android.login.LoginWpcomService"
- android:exported="false"
- android:foregroundServiceType="shortService"
- android:label="Login to WPCOM Service" />
- <service
android:name="org.wordpress.android.ui.sitecreation.services.SiteCreationService"
android:exported="false"
android:foregroundServiceType="dataSync"
@@ -1537,37 +1533,6 @@
android:name="com.google.firebase.components:com.google.firebase.datatransport.TransportRegistrar"
android:value="com.google.firebase.components.ComponentRegistrar" />
</service>
- <service
- android:name="androidx.credentials.playservices.CredentialProviderMetadataHolder"
- android:enabled="true"
- android:exported="false" >
- <meta-data
- android:name="androidx.credentials.CREDENTIAL_PROVIDER_KEY"
- android:value="androidx.credentials.playservices.CredentialProviderPlayServicesImpl" />
- </service>
-
- <activity
- android:name="androidx.credentials.playservices.HiddenActivity"
- android:configChanges="orientation|screenSize|screenLayout|keyboardHidden"
- android:enabled="true"
- android:exported="false"
- android:fitsSystemWindows="true"
- android:theme="@style/Theme.Hidden" >
- </activity>
- <activity
- android:name="com.google.android.gms.auth.api.signin.internal.SignInHubActivity"
- android:excludeFromRecents="true"
- android:exported="false"
- android:theme="@android:style/Theme.Translucent.NoTitleBar" />
- <!--
- Service handling Google Sign-In user revocation. For apps that do not integrate with
- Google Sign-In, this service will never be started.
- -->
- <service
- android:name="com.google.android.gms.auth.api.signin.RevocationBoundService"
- android:exported="true"
- android:permission="com.google.android.gms.auth.api.signin.permission.REVOCATION_NOTIFICATION"
- android:visibleToInstantApps="true" />
<activity
android:name="com.google.android.gms.common.api.GoogleApiActivity"Go to https://buildkite.com/automattic/wordpress-android/builds/25607/canvas?sid=019d0b25-258d-4b91-b76b-969de6c68941, click on the |
Project manifest changes for WordPressThe following changes in the --- ./build/reports/diff_manifest/WordPress/jetpackRelease/base_manifest.txt 2026-03-20 12:17:11.552091158 +0000
+++ ./build/reports/diff_manifest/WordPress/jetpackRelease/head_manifest.txt 2026-03-20 12:17:48.262097831 +0000
@@ -2,8 +2,8 @@
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.jetpack.android"
android:installLocation="auto"
- android:versionCode="1487"
- android:versionName="26.7-rc-1" >
+ android:versionCode="1488"
+ android:versionName="26.7" >
<uses-sdk
android:minSdkVersion="26"
@@ -397,6 +397,14 @@
android:exported="false"
android:theme="@style/WordPress.NoActionBar" />
<activity
+ android:name="org.wordpress.android.ui.newstats.yearinreview.YearInReviewDetailActivity"
+ android:exported="false"
+ android:theme="@style/WordPress.NoActionBar" />
+ <activity
+ android:name="org.wordpress.android.ui.newstats.tagsandcategories.TagsAndCategoriesDetailActivity"
+ android:exported="false"
+ android:theme="@style/WordPress.NoActionBar" />
+ <activity
android:name="org.wordpress.android.ui.newstats.subscribers.subscriberslist.SubscribersListDetailActivity"
android:exported="false"
android:theme="@style/WordPress.NoActionBar" />
@@ -411,7 +419,8 @@
<activity
android:name="org.wordpress.android.ui.accounts.LoginActivity"
android:exported="true"
- android:theme="@style/LoginTheme.TransparentSystemBars"
+ android:launchMode="singleTop"
+ android:theme="@style/LoginTheme"
android:windowSoftInputMode="adjustResize" >
<intent-filter>
<action android:name="android.intent.action.VIEW" />
@@ -427,7 +436,7 @@
<activity
android:name="org.wordpress.android.ui.accounts.applicationpassword.ApplicationPasswordLoginActivity"
android:exported="true"
- android:theme="@style/LoginTheme.TransparentSystemBars"
+ android:theme="@style/LoginTheme"
android:windowSoftInputMode="adjustResize" >
<intent-filter>
<action android:name="android.intent.action.VIEW" />
@@ -468,14 +477,6 @@
</intent-filter>
</activity>
<activity
- android:name="org.wordpress.android.ui.accounts.SignupEpilogueActivity"
- android:label="@string/signup_epilogue_screen_title"
- android:theme="@style/LoginTheme" />
- <activity
- android:name="org.wordpress.android.ui.accounts.PostSignupInterstitialActivity"
- android:label="@string/post_signup_interstitial_title"
- android:theme="@style/LoginTheme" />
- <activity
android:name="org.wordpress.android.ui.sitecreation.SiteCreationActivity"
android:theme="@style/WordPress.NoActionBar"
android:windowSoftInputMode="adjustResize" />
@@ -1143,11 +1144,6 @@
android:label="Installation Referrer Service"
android:permission="android.permission.BIND_JOB_SERVICE" />
<service
- android:name="org.wordpress.android.login.LoginWpcomService"
- android:exported="false"
- android:foregroundServiceType="shortService"
- android:label="Login to WPCOM Service" />
- <service
android:name="org.wordpress.android.ui.sitecreation.services.SiteCreationService"
android:exported="false"
android:foregroundServiceType="dataSync"
@@ -1562,37 +1558,6 @@
android:name="com.google.firebase.components:com.google.firebase.datatransport.TransportRegistrar"
android:value="com.google.firebase.components.ComponentRegistrar" />
</service>
- <service
- android:name="androidx.credentials.playservices.CredentialProviderMetadataHolder"
- android:enabled="true"
- android:exported="false" >
- <meta-data
- android:name="androidx.credentials.CREDENTIAL_PROVIDER_KEY"
- android:value="androidx.credentials.playservices.CredentialProviderPlayServicesImpl" />
- </service>
-
- <activity
- android:name="androidx.credentials.playservices.HiddenActivity"
- android:configChanges="orientation|screenSize|screenLayout|keyboardHidden"
- android:enabled="true"
- android:exported="false"
- android:fitsSystemWindows="true"
- android:theme="@style/Theme.Hidden" >
- </activity>
- <activity
- android:name="com.google.android.gms.auth.api.signin.internal.SignInHubActivity"
- android:excludeFromRecents="true"
- android:exported="false"
- android:theme="@android:style/Theme.Translucent.NoTitleBar" />
- <!--
- Service handling Google Sign-In user revocation. For apps that do not integrate with
- Google Sign-In, this service will never be started.
- -->
- <service
- android:name="com.google.android.gms.auth.api.signin.RevocationBoundService"
- android:exported="true"
- android:permission="com.google.android.gms.auth.api.signin.permission.REVOCATION_NOTIFICATION"
- android:visibleToInstantApps="true" />
<activity
android:name="com.google.android.gms.common.api.GoogleApiActivity"Go to https://buildkite.com/automattic/wordpress-android/builds/25607/canvas?sid=019d0b25-258e-4b2d-931d-7816ff58870b, click on the |
| private boolean mIsSiteLoginAvailableFromPrologue; | ||
|
|
||
| private LoginMode mLoginMode; | ||
| private LoginFlow mLoginFlow; |
Check notice
Code scanning / Android Lint
Nullable/NonNull annotation missing on field
|
|
||
| private static final String GOOGLE_ERROR_DIALOG_TAG = "google_error_dialog_tag"; | ||
| private int mFragmentContainerId; | ||
| private View mLoadingOverlay; |
Check notice
Code scanning / Android Lint
Nullable/NonNull annotation missing on field
| showMainActivity(context, bypassMigration, false); | ||
| } | ||
|
|
||
| public static void showMainActivity(Context context, boolean bypassMigration, boolean selectPrimarySite) { |
Check notice
Code scanning / Android Lint
Nullable/NonNull annotation missing on method parameter
* Enable Insights and Subscribers stats tabs for all users Show all three stats tabs (Traffic, Insights, Subscribers) for everyone by removing the BuildConfig.DEBUG gate. The NEW_STATS feature flag now defaults to true and controls access to the new stats screen itself via ListItemActionHandler, not the tab visibility. - Add defaultEnabled parameter to ExperimentalFeatures.Feature enum - Set NEW_STATS defaultEnabled to true - Propagate default through AppPrefs/AppPrefsWrapper - Always show all StatsTab entries in NewStatsScreen Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Revert AppPrefs, AppPrefsWrapper, and ExperimentalFeatures changes Keep only the NewStatsActivity change that always shows all tabs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Bumps com.google.firebase:firebase-bom from 34.10.0 to 34.11.0. --- updated-dependencies: - dependency-name: com.google.firebase:firebase-bom dependency-version: 34.11.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the kotlin-ksp group with 9 updates: | Package | From | To | | --- | --- | --- | | [org.jetbrains.kotlin:kotlin-reflect](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin:kotlin-test-junit](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.kapt](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.plugin.allopen](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.android](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.plugin.compose](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.jvm](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.plugin.parcelize](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | | [org.jetbrains.kotlin.plugin.serialization](https://github.com/JetBrains/kotlin) | `2.3.10` | `2.3.20` | Updates `org.jetbrains.kotlin:kotlin-reflect` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin:kotlin-test-junit` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.kapt` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.allopen` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.android` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.compose` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.jvm` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.parcelize` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.serialization` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin:kotlin-test-junit` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.kapt` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.allopen` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.android` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.compose` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.jvm` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.parcelize` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) Updates `org.jetbrains.kotlin.plugin.serialization` from 2.3.10 to 2.3.20 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md) - [Commits](JetBrains/kotlin@v2.3.10...v2.3.20) --- updated-dependencies: - dependency-name: org.jetbrains.kotlin:kotlin-reflect dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin:kotlin-test-junit dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.kapt dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.allopen dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.android dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.compose dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.jvm dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.parcelize dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.serialization dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin:kotlin-test-junit dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.kapt dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.allopen dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.android dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.compose dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.jvm dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.parcelize dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp - dependency-name: org.jetbrains.kotlin.plugin.serialization dependency-version: 2.3.20 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: kotlin-ksp ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Remove Application Password experimental feature flag Enable Application Passwords for all users by removing the EXPERIMENTAL_APPLICATION_PASSWORD_FEATURE toggle from the experimental features screen. - Remove feature enum entry and all isEnabled/setEnabled checks - Remove enableApplicationPasswordIfNecessary() from dialog ViewModels - Remove application password dialog state from ExperimentalFeaturesViewModel - Remove experimentalFeatures dependency where no longer needed - Update tests to reflect always-on behavior Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Remove dead shouldBuildCard method from ApplicationPasswordViewModelSlice Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Remove dead AP dialog composables and unused experimentalFeatures dependency - Remove ApplicationPasswordOffConfirmationDialog and ApplicationPasswordInfoDialog from ExperimentalFeaturesScreen - Remove experimentalFeatures from SiteListItemBuilder constructor - Clean up unused imports in source and test files Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix missing imports and remove unused AP string resources Add missing imports (verticalScroll, rememberScrollState, TextAlign) in ExperimentalFeaturesScreen after trunk merge, and remove orphaned string resources from the AP feature flag removal. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Remove orphaned AP translations from 27 locale files Clean up translated strings for experimental_application_password_feature and application_password_disable_feature that were left behind after removing the feature flag from the default strings.xml. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Remove stale "Experimental Feature" note from AP dialogs Application Passwords are now always-on, so the note telling users it is an experimental feature is misleading. Removed the string and its 24 translations, and stopped appending it in ApplicationPasswordRequiredDialogActivity. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|





Description
This PR implements phases 4 & 5 of our AGP 9.x upgrade plan by removing obsolete
gradle.propertiesflags that are now defaults in AGP 9:android.nonTransitiveRClass=true— non-transitive R classes are the defaultandroid.nonFinalResIds=true— non-final resource IDs are the defaultAlso keeps
android.enableR8.fullMode=falseto reduce migration risk.Testing instructions
./gradlew assembleJetpackDebug./gradlew :WordPress:testWordPressDebugUnitTest