EditorLauncher Scaffold Implementation (#22111)

* Add EditorLauncher scaffold with centralized editor routing

Implement hybrid approach for editor activity routing that will support
both legacy EditPostActivity and future EditPostGutenbergKitActivity.

Changes:
- Add EditorLauncher singleton with feature-flag-based routing logic
- Migrate all ActivityLauncher methods to use EditorLauncher helper
- Add EditorLauncher to dependency injection (AppComponent)
- Include analytics tracking and source identification for launches
- Route to EditPostActivity for now (scaffold implementation)

This provides infrastructure for clean editor activity separation while
maintaining backward compatibility. Future changes will:
- Add EditorLauncherParams data class with builder pattern
- Route to EditPostGutenbergKitActivity when implemented
- Add redirection fallback with monitoring

Covers ActivityLauncher methods:
- openEditorForSiteInNewStack()
- openEditorForPostInNewStack()
- openEditorForReblog()
- addNewPostForResult() variants
- editPostOrPageForResult() variants
- editPageForResult() variants
- addNewPageForResult() variants

* Add EditorLauncherParams for type-safe editor launch parameters

Replace Bundle-based Intent extras with type-safe parameter objects
to improve maintainability and reduce errors.

Changes:
- Add EditorLauncherParams.kt with Kotlin data class and Java Builder pattern
- Update EditorLauncher.kt to accept EditorLauncherParams instead of Intent bundles
- Refactor ActivityLauncher.java methods to use EditorLauncherParams.Builder
- Add backward compatibility with deprecated Intent-based method
- Centralize all Intent extra handling in EditorLauncher.addEditorExtras()
- Maintain analytics tracking with type-safe parameter source detection

This provides compile-time safety while maintaining Java interoperability
through the builder pattern.

* Remove deprecated Intent-based EditorLauncher method

Complete migration to EditorLauncherParams for all ActivityLauncher methods.

Changes:
- Remove deprecated createEditorIntent(context, Intent) method from EditorLauncher
- Remove unused getIntentSource() legacy method
- Update remaining ActivityLauncher methods to use EditorLauncherParams.Builder:
  - addNewPostWithContentFromAIForResult()
  - addNewPageForResult() (both Activity and Fragment variants)
- All editor launches now use type-safe parameters instead of Bundle operations

This completes the EditorLauncherParams migration for ActivityLauncher,
providing compile-time safety and better maintainability.

* Fix Detekt code quality violations in EditorLauncher

Address all code quality issues identified by Detekt static analysis.

Changes:
- Remove unused imports (Bundle) and unused property (analyticsTracker)
- Replace TODO comments with regular comments to avoid ForbiddenComment violations
- Break down complex addEditorExtras() method into smaller helper methods:
  - addBasicExtras() - site, page, promo flags
  - addPostExtras() - post IDs, autosave, quickpress, landing editor flags
  - addReblogExtras() - reblog title, quote, image, citation, action
  - addPageExtras() - page title, content, template
  - addMiscExtras() - voice content, media, prompt ID, entry point
- Fix MaxLineLength violations by splitting long builder method calls
- Add missing newline at end of EditorLauncherParams.kt

This reduces cyclomatic complexity and improves code maintainability
while maintaining all existing functionality.

* Move analytics source field handling into EditorLauncher

Centralizes the EXTRA_CREATION_SOURCE_DETAIL Intent extra creation in
EditorLauncher.addMiscExtras instead of requiring manual addition in
each ActivityLauncher method. This eliminates code duplication and
ensures analytics tracking is consistently applied across all editor launches.

Changes:
- Add AnalyticsUtils.EXTRA_CREATION_SOURCE_DETAIL to EditorLauncher.addMiscExtras
- Remove redundant analytics field additions from ActivityLauncher methods
- Add AnalyticsUtils import to EditorLauncher

* Require non-null SiteModel in EditorLauncherParams.Builder constructor

Makes the Java Builder pattern more type-safe by requiring SiteModel in the constructor
rather than checking for null at build time.

Changes:
- EditorLauncherParams.Builder now takes @nonnull SiteModel in constructor
- Remove site() setter method from Builder
- Remove runtime requireNotNull check in build()
- Update all call sites to pass site to constructor
- Add missing @nonnull annotations in ActivityLauncher

* Add analytics tracking to EditorLauncher

Tracks editor launches with should_use_gutenberg_kit property to understand
feature flag usage patterns.

* Add documentation and test for EditorLauncherParams field mapping

Ensures all EditorLauncherParams fields are handled in EditorLauncher.addEditorExtras()
through documentation and a unit test that validates completeness.

Changes:
- Add cross-reference documentation in EditorLauncherParams and EditorLauncher
- Add EditorLauncherTest with detailed field-to-method mapping documentation
- Test validates that all fields are accounted for in addEditorExtras methods
- Filter out Kotlin synthetic fields in test validation

* Fix line length in EditorLauncherTest

* Update analytics property name to will_use_gutenberg_kit

* Add tracking for EditorLauncher vs direct Intent launches

Adds a way to distinguish editor launches that went through EditorLauncher
vs direct Intent creation for analytics purposes.

Changes:
- EditorLauncher: Add EXTRA_LAUNCHED_VIA_EDITOR_LAUNCHER intent extra
- EditPostActivity: Track EDITOR_LAUNCHED_VIA_EDITOR_LAUNCHER when extra is present (only on initial creation, not configuration changes)
- AnalyticsTracker: Add EDITOR_LAUNCHED_VIA_EDITOR_LAUNCHER stat

* Fix line length in EditPostActivity

* Add null checks for PostModel in editPostOrPageForResult methods

Prevents crashes when PostModel is null (e.g., when mPostStore.getPostByLocalPostId
returns null for non-existent posts) by showing a user-friendly error message instead.

Changes:
- Add null check and error toast in both editPostOrPageForResult variants
- Prevents crashes from calling post.getId() or post.isPage() on null objects

* Make all boolean parameters nullable to avoid unintended defaults

The trunk editPostOrPageForResult methods never set boolean extras like
EXTRA_IS_PAGE, EXTRA_IS_PROMO, or EXTRA_LOAD_AUTO_SAVE_REVISION unless
explicitly required. Making all boolean parameters nullable ensures they're
only set when explicitly provided, matching trunk behavior exactly.

Changes:
- Make all boolean fields nullable in EditorLauncherParams and Builder
- Update EditorLauncher to only set extras when parameters are non-null
- Remove unnecessary loadAutoSaveRevision(false) from first editPostOrPageForResult
- Preserves explicit boolean settings in other methods (pages, promos, etc.)

* Don't set source in ActivityLauncher.openEditorForReblog

Author: oguzkocer

WordPress/src/main/java/org/wordpress/android/modules/AppComponent.java

@@ -109,6 +109,7 @@
 import org.wordpress.android.ui.posts.prepublishing.categories.addcategory.PrepublishingAddCategoryFragment;
 import org.wordpress.android.ui.posts.prepublishing.home.PrepublishingHomeAdapter;
 import org.wordpress.android.ui.posts.prepublishing.home.PrepublishingHomeFragment;
+import org.wordpress.android.ui.posts.EditorLauncher;
 import org.wordpress.android.ui.posts.prepublishing.publishsettings.PrepublishingPublishSettingsFragment;
 import org.wordpress.android.ui.posts.prepublishing.social.PrepublishingSocialFragment;
 import org.wordpress.android.ui.posts.prepublishing.tags.PrepublishingTagsFragment;
@@ -561,4 +562,7 @@ public interface AppComponent {
     void inject(PostResolutionOverlayFragment object);
 
     void inject(LoginSiteApplicationPasswordFragment object);
+
+    // Provide access to EditorLauncher for static utility classes
+    EditorLauncher editorLauncher();
 }

WordPress/src/main/java/org/wordpress/android/ui/ActivityLauncher.java

@@ -87,8 +87,9 @@
 import org.wordpress.android.ui.plugins.PluginBrowserActivity;
 import org.wordpress.android.ui.plugins.PluginDetailActivity;
 import org.wordpress.android.ui.plugins.PluginUtils;
-import org.wordpress.android.ui.posts.EditPostActivity;
 import org.wordpress.android.ui.posts.EditPostActivityConstants;
+import org.wordpress.android.ui.posts.EditorLauncher;
+import org.wordpress.android.ui.posts.EditorLauncherParams;
 import org.wordpress.android.ui.posts.JetpackSecuritySettingsActivity;
 import org.wordpress.android.ui.posts.PostListType;
 import org.wordpress.android.ui.posts.PostUtils;
@@ -401,9 +402,11 @@ public static void openEditorForSiteInNewStack(Context context, @NonNull SiteMod
         TaskStackBuilder taskStackBuilder = TaskStackBuilder.create(context);
         Intent mainActivityIntent = getMainActivityInNewStack(context);
 
-        Intent editorIntent = new Intent(context, EditPostActivity.class);
-        editorIntent.putExtra(WordPress.SITE, site);
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_IS_PAGE, false);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+            .isPage(false)
+            .build();
+
+        Intent editorIntent = EditorLauncher.getInstance().createEditorIntent(context, params);
 
         taskStackBuilder.addNextIntent(mainActivityIntent);
         taskStackBuilder.addNextIntent(editorIntent);
@@ -415,10 +418,12 @@ public static void openEditorForPostInNewStack(Context context, @NonNull SiteMod
         TaskStackBuilder taskStackBuilder = TaskStackBuilder.create(context);
         Intent mainActivityIntent = getMainActivityInNewStack(context);
 
-        Intent editorIntent = new Intent(context, EditPostActivity.class);
-        editorIntent.putExtra(WordPress.SITE, site);
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_POST_LOCAL_ID, localPostId);
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_IS_PAGE, false);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+                .postLocalId(localPostId)
+                .isPage(false)
+                .build();
+
+        Intent editorIntent = EditorLauncher.getInstance().createEditorIntent(context, params);
 
         taskStackBuilder.addNextIntent(mainActivityIntent);
         taskStackBuilder.addNextIntent(editorIntent);
@@ -457,12 +462,15 @@ public static void openEditorForReblog(
             site.getSiteId()
         );
 
-        Intent editorIntent = new Intent(activity, EditPostActivity.class);
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_REBLOG_POST_TITLE, post.getTitle());
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_REBLOG_POST_QUOTE, post.getExcerpt());
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_REBLOG_POST_IMAGE, post.getFeaturedImage());
-        editorIntent.putExtra(EditPostActivityConstants.EXTRA_REBLOG_POST_CITATION, post.getUrl());
-        editorIntent.setAction(EditPostActivityConstants.ACTION_REBLOG);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+                .reblogPostTitle(post.getTitle())
+                .reblogPostQuote(post.getExcerpt())
+                .reblogPostImage(post.getFeaturedImage())
+                .reblogPostCitation(post.getUrl())
+                .reblogAction(EditPostActivityConstants.ACTION_REBLOG)
+                .build();
+
+        Intent editorIntent = EditorLauncher.getInstance().createEditorIntent(activity, params);
 
         addNewPostForResult(editorIntent, activity, site, false, reblogSource, -1, null);
     }
@@ -980,26 +988,35 @@ public static void addNewPostWithContentFromAIForResult(
             return;
         }
 
-        Intent intent = new Intent(activity, EditPostActivity.class);
-        intent.putExtra(WordPress.SITE, site);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_PAGE, false);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_PROMO, isPromo);
-        intent.putExtra(AnalyticsUtils.EXTRA_CREATION_SOURCE_DETAIL, source);
-        intent.putExtra(EditPostActivityConstants.EXTRA_VOICE_CONTENT, content);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+            .isPage(false)
+            .isPromo(isPromo)
+            .source(source)
+            .voiceContent(content)
+            .build();
+
+        Intent intent = EditorLauncher.getInstance().createEditorIntent(activity, params);
         activity.startActivityForResult(intent, RequestCodes.EDIT_POST);
     }
 
     public static void addNewPostForResult(
             Activity activity,
-            SiteModel site,
+            @NonNull SiteModel site,
             boolean isPromo,
             PagePostCreationSourcesDetail source,
             final int promptId,
             final EntryPoint entryPoint
     ) {
-        addNewPostForResult(
-            new Intent(activity, EditPostActivity.class), activity, site, isPromo, source, promptId, entryPoint
-        );
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+                .isPage(false)
+                .isPromo(isPromo)
+                .source(source)
+                .promptId(promptId)
+                .entryPoint(entryPoint)
+                .build();
+
+        Intent editorIntent = EditorLauncher.getInstance().createEditorIntent(activity, params);
+        activity.startActivityForResult(editorIntent, RequestCodes.EDIT_POST);
     }
 
     public static void addNewPostForResult(
@@ -1024,14 +1041,35 @@ public static void addNewPostForResult(
         activity.startActivityForResult(intent, RequestCodes.EDIT_POST);
     }
 
-    public static void editPostOrPageForResult(Activity activity, SiteModel site, PostModel post) {
-        editPostOrPageForResult(new Intent(activity, EditPostActivity.class), activity, site, post.getId(), false);
+    public static void editPostOrPageForResult(Activity activity, @NonNull SiteModel site, @Nullable PostModel post) {
+        if (post == null) {
+            ToastUtils.showToast(activity, R.string.post_not_found, ToastUtils.Duration.SHORT);
+            return;
+        }
+
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+                .postLocalId(post.getId())
+                .loadAutoSaveRevision(false)
+                .build();
+
+        Intent editorIntent = EditorLauncher.getInstance().createEditorIntent(activity, params);
+        activity.startActivityForResult(editorIntent, RequestCodes.EDIT_POST);
     }
 
-    public static void editPostOrPageForResult(Activity activity, SiteModel site, PostModel post,
+    public static void editPostOrPageForResult(Activity activity, @NonNull SiteModel site, @Nullable PostModel post,
                                                boolean loadAutoSaveRevision) {
-        editPostOrPageForResult(new Intent(activity, EditPostActivity.class), activity, site, post.getId(),
-                loadAutoSaveRevision);
+        if (post == null) {
+            ToastUtils.showToast(activity, R.string.post_not_found, ToastUtils.Duration.SHORT);
+            return;
+        }
+
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+            .postLocalId(post.getId())
+            .loadAutoSaveRevision(loadAutoSaveRevision)
+            .build();
+
+        Intent editorIntent = EditorLauncher.getInstance().createEditorIntent(activity, params);
+        activity.startActivityForResult(editorIntent, RequestCodes.EDIT_POST);
     }
 
     public static void editPostOrPageForResult(Intent intent, Activity activity, SiteModel site, int postLocalId) {
@@ -1056,8 +1094,14 @@ public static void editPostOrPageForResult(Intent intent, Activity activity, Sit
 
     public static void editPageForResult(@NonNull Fragment fragment, @NonNull SiteModel site,
                                          int pageLocalId, boolean loadAutoSaveRevision) {
-        Intent intent = new Intent(fragment.getContext(), EditPostActivity.class);
-        editPageForResult(intent, fragment, site, pageLocalId, loadAutoSaveRevision, RequestCodes.EDIT_POST);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+                .postLocalId(pageLocalId)
+                .loadAutoSaveRevision(loadAutoSaveRevision)
+                .isPage(true)
+                .build();
+
+        Intent intent = EditorLauncher.getInstance().createEditorIntent(fragment.getContext(), params);
+        fragment.startActivityForResult(intent, RequestCodes.EDIT_POST);
     }
 
     public static void editPageForResult(Intent intent, @NonNull Fragment fragment, @NonNull SiteModel site,
@@ -1067,10 +1111,16 @@ public static void editPageForResult(Intent intent, @NonNull Fragment fragment,
 
     public static void editLandingPageForResult(@NonNull Fragment fragment, @NonNull SiteModel site, int homeLocalId,
                                                 boolean isNewSite) {
-        Intent intent = new Intent(fragment.getContext(), EditPostActivity.class);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_LANDING_EDITOR, true);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_LANDING_EDITOR_OPENED_FOR_NEW_SITE, isNewSite);
-        editPageForResult(intent, fragment, site, homeLocalId, false, RequestCodes.EDIT_LANDING_PAGE);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+                .postLocalId(homeLocalId)
+                .loadAutoSaveRevision(false)
+                .isPage(true)
+                .isLandingEditor(true)
+                .isLandingEditorOpenedForNewSite(isNewSite)
+                .build();
+
+        Intent intent = EditorLauncher.getInstance().createEditorIntent(fragment.getContext(), params);
+        fragment.startActivityForResult(intent, RequestCodes.EDIT_LANDING_PAGE);
     }
 
     public static void editPageForResult(Intent intent, @NonNull Fragment fragment, @NonNull SiteModel site,
@@ -1089,14 +1139,16 @@ public static void addNewPageForResult(
             @Nullable String template,
             @NonNull PagePostCreationSourcesDetail source
     ) {
-        Intent intent = new Intent(activity, EditPostActivity.class);
-        intent.putExtra(WordPress.SITE, site);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_PAGE, true);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_PROMO, false);
-        intent.putExtra(EditPostActivityConstants.EXTRA_PAGE_TITLE, title);
-        intent.putExtra(EditPostActivityConstants.EXTRA_PAGE_CONTENT, content);
-        intent.putExtra(EditPostActivityConstants.EXTRA_PAGE_TEMPLATE, template);
-        intent.putExtra(AnalyticsUtils.EXTRA_CREATION_SOURCE_DETAIL, source);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+            .isPage(true)
+            .isPromo(false)
+            .pageTitle(title)
+            .pageContent(content)
+            .pageTemplate(template)
+            .source(source)
+            .build();
+
+        Intent intent = EditorLauncher.getInstance().createEditorIntent(activity, params);
         activity.startActivityForResult(intent, RequestCodes.EDIT_POST);
     }
 
@@ -1107,14 +1159,16 @@ public static void addNewPageForResult(
             @NonNull String content,
             @Nullable String template,
             @NonNull PagePostCreationSourcesDetail source) {
-        Intent intent = new Intent(fragment.getContext(), EditPostActivity.class);
-        intent.putExtra(WordPress.SITE, site);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_PAGE, true);
-        intent.putExtra(EditPostActivityConstants.EXTRA_IS_PROMO, false);
-        intent.putExtra(EditPostActivityConstants.EXTRA_PAGE_TITLE, title);
-        intent.putExtra(EditPostActivityConstants.EXTRA_PAGE_CONTENT, content);
-        intent.putExtra(EditPostActivityConstants.EXTRA_PAGE_TEMPLATE, template);
-        intent.putExtra(AnalyticsUtils.EXTRA_CREATION_SOURCE_DETAIL, source);
+        EditorLauncherParams params = new EditorLauncherParams.Builder(site)
+            .isPage(true)
+            .isPromo(false)
+            .pageTitle(title)
+            .pageContent(content)
+            .pageTemplate(template)
+            .source(source)
+            .build();
+
+        Intent intent = EditorLauncher.getInstance().createEditorIntent(fragment.getContext(), params);
         fragment.startActivityForResult(intent, RequestCodes.EDIT_POST);
     }
 

WordPress/src/main/java/org/wordpress/android/ui/notifications/NotificationsListFragmentPage.kt

@@ -316,7 +316,9 @@ class NotificationsListFragmentPage : ViewPagerFragment(R.layout.notifications_l
             return
         }
         if (tabPosition == Unread.ordinal) {
-            ActivityLauncher.addNewPostForResult(activity, selectedSite, false, POST_FROM_NOTIFS_EMPTY_VIEW, -1, null)
+            selectedSite?.let {
+                ActivityLauncher.addNewPostForResult(activity, it, false, POST_FROM_NOTIFS_EMPTY_VIEW, -1, null)
+            }
         } else if (activity is WPMainActivity) {
             (requireActivity() as WPMainActivity).setReaderPageActive()
         }

WordPress/src/main/java/org/wordpress/android/ui/posts/EditPostActivity.kt

@@ -527,6 +527,12 @@ class EditPostActivity : BaseAppCompatActivity(), EditorFragmentActivity, Editor
         (application as WordPress).component().inject(this)
         initializeViewModels()
 
+        // Track if this editor launch came via EditorLauncher (only on initial creation)
+        if (savedInstanceState == null
+            && intent.getBooleanExtra(EditorLauncher.EXTRA_LAUNCHED_VIA_EDITOR_LAUNCHER, false)) {
+            analyticsTrackerWrapper.track(Stat.EDITOR_LAUNCHED_VIA_EDITOR_LAUNCHER)
+        }
+
         setContentView(R.layout.new_edit_post_activity)
         val callback: OnBackPressedCallback = object : OnBackPressedCallback(true) {
             override fun handleOnBackPressed() {