feat: add source link target selector

Author: lisonge

web/src/utils/url.ts

@@ -40,6 +40,22 @@ const mirrorContentRegs = [
   /\/aosp\-mirror\/platform_frameworks_base\/[^\/]+(.*)$/g,
 ];
 
+export const sourceLinkTargetOptions = [
+  'cs.android.com',
+  'googlesource',
+  'githubusercontent',
+] as const;
+export type SourceLinkTarget = (typeof sourceLinkTargetOptions)[number];
+export const DEFAULT_SOURCE_LINK_TARGET: SourceLinkTarget = 'cs.android.com';
+
+export const getGoogleSourceUrl = (filePath: string): string => {
+  const tagEnd = filePath.indexOf('/');
+  if (tagEnd < 0) return sourceBaseurl;
+  const tag = filePath.substring(0, tagEnd);
+  const sourcePath = filePath.substring(tagEnd);
+  return `${sourceBaseurl}platform/frameworks/base/+/refs/tags/${tag}${sourcePath}`;
+};
+
 export const getMirrorContentUrl = (filePath: string): string => {
   for (const [tag, baseUrl] of manualTagMirrors) {
     const tagPrefix = `${tag}/`;

web/src/views/home/HomePage.vue

@@ -10,6 +10,7 @@ import DiffConcurrentSelect from './DiffConcurrentSelect.vue';
 import DiffResultList from './DiffResultList.vue';
 import { skipNextAutoDiffOnReload, useSharedHomeState } from './homeState';
 import MinSdkSelect from './MinSdkSelect.vue';
+import SourceLinkTargetSelect from './SourceLinkTargetSelect.vue';
 
 const title = document.title;
 const {
@@ -139,6 +140,7 @@ const handleClearLocalCache = async () => {
         </template>
       </MPopconfirm>
       <div flex-1></div>
+      <SourceLinkTargetSelect />
       <MinSdkSelect />
       <DiffConcurrentSelect />
       <a

web/src/views/home/SourceLinkTargetSelect.vue

@@ -0,0 +1,22 @@
+<script setup lang="ts">
+import MOption from '@/components/MOption.vue';
+import MSelect from '@/components/MSelect.vue';
+import { useSharedHomeState } from './homeState';
+
+const { sourceLinkTarget, sourceLinkTargetOptions } = useSharedHomeState();
+</script>
+
+<template>
+  <MSelect
+    v-model="sourceLinkTarget"
+    label="Link"
+    trigger-min-width="136px"
+    popover-min-width="136px"
+  >
+    <MOption
+      v-for="target in sourceLinkTargetOptions"
+      :key="target"
+      :value="target"
+    />
+  </MSelect>
+</template>

web/src/views/home/TagCard.vue

@@ -1,5 +1,9 @@
 <script setup lang="ts">
-import { getSourceUrlWithLine } from '@/utils/url';
+import {
+  getGoogleSourceUrl,
+  getMirrorContentUrl,
+  getSourceUrlWithLine,
+} from '@/utils/url';
 import { computed } from 'vue';
 import { ANDROID_PREFIX_LEN, useSharedHomeState } from './homeState';
 
@@ -8,7 +12,8 @@ const props = defineProps<{
   future?: boolean;
 }>();
 
-const { urlBuilder, getDiffResult, searchFromData } = useSharedHomeState();
+const { urlBuilder, getDiffResult, searchFromData, sourceLinkTarget } =
+  useSharedHomeState();
 const diffResult = computed(() => getDiffResult(props.tag));
 
 const title = computed<string | undefined>(() => {
@@ -28,9 +33,9 @@ const title = computed<string | undefined>(() => {
 });
 const notFound = computed(() => diffResult.value?.notFound);
 const sourceUrl = computed<string | undefined>(() => {
-  const t = urlBuilder.value?.templateUrl;
-  if (!t) return '';
-  const u = t[0] + props.tag + t[1];
+  const builder = urlBuilder.value;
+  if (!builder) return '';
+  const t = builder.templateUrl;
   const loc = (() => {
     if (searchFromData.value.targetKind === 'member') {
       return diffResult.value?.members?.[0]?.loc;
@@ -39,6 +44,14 @@ const sourceUrl = computed<string | undefined>(() => {
       return diffResult.value?.target?.loc;
     }
   })();
+  if (sourceLinkTarget.value === 'googlesource') {
+    const u = getGoogleSourceUrl(props.tag + builder.filePath);
+    return loc ? getSourceUrlWithLine(u, loc) : u;
+  }
+  if (sourceLinkTarget.value === 'githubusercontent') {
+    return getMirrorContentUrl(props.tag + builder.filePath);
+  }
+  const u = t[0] + props.tag + t[1];
   if (loc) {
     return getSourceUrlWithLine(u, loc);
   }

web/src/views/home/homeState.ts

@@ -8,8 +8,13 @@ import {
 import { colors, findStructByPath, useEqualComputed, useTask } from '@/utils';
 import androidVersionList from '@/utils/android.data';
 import { emptyArray } from '@/utils/constants';
-import { androidApiVersionList, DEFAULT_MIN_SDK, } from '@/utils/constants';
-import { getVersionUrlBuilder } from '@/utils/url';
+import { androidApiVersionList, DEFAULT_MIN_SDK } from '@/utils/constants';
+import {
+  DEFAULT_SOURCE_LINK_TARGET,
+  getVersionUrlBuilder,
+  sourceLinkTargetOptions,
+  type SourceLinkTarget,
+} from '@/utils/url';
 import {
   createSharedComposable,
   useStorage,
@@ -26,6 +31,7 @@ const SKIP_NEXT_AUTO_DIFF_STATE_KEY = '__androidApiDiffSkipNextAutoDiff';
 const DIFF_CONCURRENT_COUNT_STORAGE_KEY =
   'android-api-diff:diff-concurrent-count:v1';
 const MIN_SDK_STORAGE_KEY = 'android-api-diff:min-sdk:v1';
+const SOURCE_LINK_TARGET_STORAGE_KEY = 'android-api-diff:source-link-target:v1';
 const SEARCH_HISTORY_STORAGE_KEY = 'android-api-diff:search-history:v1';
 const DEFAULT_SEARCH_HISTORY = [
   'IActivityTaskManager.getTasks',
@@ -42,6 +48,8 @@ export const diffConcurrentCountOptions = Array.from(
 
 export const minSdkOptions = androidApiVersionList;
 
+export { sourceLinkTargetOptions };
+
 const normalizeDiffConcurrentCount = (value: unknown) => {
   const count = Number(value);
   if (!Number.isInteger(count)) return DEFAULT_DIFF_CONCURRENT_COUNT;
@@ -51,11 +59,21 @@ const normalizeDiffConcurrentCount = (value: unknown) => {
 const normalizeMinSdk = (value: unknown) => {
   const sdk = Number(value);
   if (!Number.isInteger(sdk)) return DEFAULT_MIN_SDK;
-  const minSdk = minSdkOptions[0] ;
+  const minSdk = minSdkOptions[0];
   const maxSdk = minSdkOptions.at(-1)!;
   return Math.min(Math.max(sdk, minSdk), maxSdk);
 };
 
+const normalizeSourceLinkTarget = (value: unknown): SourceLinkTarget => {
+  if (
+    typeof value === 'string' &&
+    (sourceLinkTargetOptions as readonly string[]).includes(value)
+  ) {
+    return value as SourceLinkTarget;
+  }
+  return DEFAULT_SOURCE_LINK_TARGET;
+};
+
 const normalizeSearchHistory = (value: unknown): string[] => {
   if (!Array.isArray(value)) return [];
   const seen = new Set<string>();
@@ -118,6 +136,18 @@ export const useSharedHomeState = createSharedComposable(() => {
       },
     },
   );
+  const sourceLinkTarget = useStorage<SourceLinkTarget>(
+    SOURCE_LINK_TARGET_STORAGE_KEY,
+    DEFAULT_SOURCE_LINK_TARGET,
+    undefined,
+    {
+      flush: 'sync',
+      serializer: {
+        read: (raw) => normalizeSourceLinkTarget(raw),
+        write: (value) => normalizeSourceLinkTarget(value),
+      },
+    },
+  );
   const searchHistory = useStorage<string[]>(
     SEARCH_HISTORY_STORAGE_KEY,
     () => [...DEFAULT_SEARCH_HISTORY],
@@ -413,5 +443,7 @@ export const useSharedHomeState = createSharedComposable(() => {
     diffConcurrentCountOptions,
     minSdk,
     minSdkOptions,
+    sourceLinkTarget,
+    sourceLinkTargetOptions,
   };
 });