Internal change

PiperOrigin-RevId: 915305884

Author: OmniLab Team

src/devtools/mobileharness/fe/v6/angular/app/core/models/host_action.ts

@@ -63,15 +63,41 @@ export declare interface FlagPreset {
   readonly description: string;
 }
 
+/**
+ * Response for PreflightLabServerRelease API.
+ */
+export declare interface PreflightLabServerReleaseResponse {
+  readonly permissionDenied?: PermissionDenied;
+  readonly ready?: ReleaseReady;
+}
+
+/**
+ * No deploy permission.
+ */
+export declare interface PermissionDenied {}
+
+/**
+ * All checks passed. Here are the available versions to deploy.
+ */
+export declare interface ReleaseReady {
+  readonly versions: DeployableVersion[];
+}
+
+/**
+ * Represents the status of a Lab Server release.
+ */
+export type ReleaseStatus = 'Latest' | 'Current' | 'Deprecated' | '';
+
 /**
  * Configuration for a specific Lab Server release.
  */
-export declare interface HostReleaseConfig {
+export declare interface DeployableVersion {
   readonly name: string;
   readonly version: string;
-  readonly port: ReleasePort;
-  readonly syncCMD: string[];
-  readonly asyncCMD: string[];
+  readonly status: ReleaseStatus;
+  readonly buildTime: string;
+  readonly ports?: ReleasePort[];
+  readonly releaseDetails?: ReleaseDetails;
 }
 
 /**
@@ -82,6 +108,66 @@ export declare interface ReleasePort {
   readonly portNumber: number;
 }
 
+/**
+ * Details about a specific release.
+ */
+export declare interface ReleaseDetails {
+  readonly changeLogs?: ChangeLogGroup[];
+  readonly files?: FileRecord[];
+  readonly syncCommands?: CommandRecord[];
+  readonly asyncCommands?: CommandRecord[];
+}
+
+/**
+ * Represents a group of changes or bugs related to a release.
+ */
+export declare interface ChangeLogGroup {
+  readonly name: string;
+  readonly items: ChangeLogItem[];
+}
+
+/**
+ * Represents a single change record or bug record for a release.
+ */
+export declare interface ChangeLogItem {
+  readonly change?: ChangeRecord;
+  readonly bug?: BugRecord;
+}
+
+/**
+ * Represents a single change record for a release.
+ */
+export declare interface ChangeRecord {
+  readonly cl: number;
+  readonly author: string;
+  readonly text: string;
+  readonly bugs: number[];
+}
+
+/**
+ * Represents a single bug record for a release.
+ */
+export declare interface BugRecord {
+  readonly bug: number;
+  readonly text: string;
+}
+
+/**
+ * Represents a single file record for a release.
+ */
+export declare interface FileRecord {
+  readonly name: string;
+  readonly path: string;
+}
+
+/**
+ * Represents a single command record for a release.
+ */
+export declare interface CommandRecord {
+  readonly name: string;
+  readonly command: string;
+}
+
 /**
  * Response for DecommissionHost API.
  */

src/devtools/mobileharness/fe/v6/angular/app/core/services/config/fake_config_service.ts

@@ -63,7 +63,12 @@ export class FakeConfigService extends ConfigService {
     const isHostManaged = scenario.overview.host.name === 'host-x.example.com';
     const hostName = scenario.overview.host.name;
 
-    let uiStatus: Partial<DeviceConfigUiStatus> | undefined;
+    let uiStatus: Partial<DeviceConfigUiStatus> | undefined = {
+      permissions: {visible: true, editability: {editable: true}},
+      wifi: {visible: true, editability: {editable: true}},
+      dimensions: {visible: true, editability: {editable: true}},
+      settings: {visible: true, editability: {editable: true}},
+    };
     if (deviceId === 'WIFI_DIMENSIONS_ONLY_DEVICE') {
       uiStatus = {
         permissions: {visible: false},

src/devtools/mobileharness/fe/v6/angular/app/core/services/host/fake_host_service.ts

@@ -5,8 +5,8 @@ import {
   DecommissionHostResponse,
   GetHostDebugInfoResponse,
   HostHeaderInfo,
-  HostReleaseConfig,
   PopularFlag,
+  PreflightLabServerReleaseResponse,
   ReleaseLabServerRequest,
   ReleaseLabServerResponse,
   RestartLabServerResponse,
@@ -25,7 +25,10 @@ import {
   RemoteControlDevicesResponse,
 } from '../../models/host_overview';
 import {MOCK_HOST_SCENARIOS} from '../mock_data';
-import {createHostActions} from '../mock_data/hosts/ui_status_utils';
+import {
+  createDefaultReleaseResponse,
+  createHostActions,
+} from '../mock_data/hosts/ui_status_utils';
 import {HostService} from './host_service';
 
 /**
@@ -148,31 +151,13 @@ export class FakeHostService extends HostService {
     }
   }
 
-  override getReleaseConfigs(
+  override preflightLabServerRelease(
     hostName: string,
-  ): Observable<HostReleaseConfig[]> {
-    return of([
-      {
-        name: 'MH_SATELLITE_LAB',
-        version: '4.349.0',
-        port: {protocol: 'grpc', portNumber: 9994},
-        syncCMD: [
-          'sudo systemctl stop mobileharness-lab',
-          'sudo /usr/bin/mh_lab_installer --version 4.349.0',
-        ],
-        asyncCMD: ['sudo systemctl start mobileharness-lab'],
-      },
-      {
-        name: 'MH_SATELLITE_LAB',
-        version: '4.348.0',
-        port: {protocol: 'grpc', portNumber: 9994},
-        syncCMD: [
-          'sudo systemctl stop mobileharness-lab',
-          'sudo /usr/bin/mh_lab_installer --version 4.348.0',
-        ],
-        asyncCMD: ['sudo systemctl start mobileharness-lab'],
-      },
-    ]);
+  ): Observable<PreflightLabServerReleaseResponse> {
+    const scenario = MOCK_HOST_SCENARIOS.find((s) => s.hostName === hostName);
+    const preflightLabServerReleaseResponse =
+      scenario?.releaseResponse || createDefaultReleaseResponse();
+    return of(preflightLabServerReleaseResponse);
   }
 
   override decommissionMissingDevices(

src/devtools/mobileharness/fe/v6/angular/app/core/services/host/host_service.ts

@@ -2,12 +2,12 @@ import {InjectionToken} from '@angular/core';
 import {Observable} from 'rxjs';
 import {
   DecommissionHostResponse,
-  ReleaseLabServerRequest,
-  ReleaseLabServerResponse,
   GetHostDebugInfoResponse,
   HostHeaderInfo,
-  HostReleaseConfig,
   PopularFlag,
+  PreflightLabServerReleaseResponse,
+  ReleaseLabServerRequest,
+  ReleaseLabServerResponse,
   RestartLabServerResponse,
   StartLabServerResponse,
   StopLabServerResponse,
@@ -70,7 +70,9 @@ export abstract class HostService {
   /**
    * Retrieves release configurations for a host.
    */
-  abstract getReleaseConfigs(hostName: string): Observable<HostReleaseConfig[]>;
+  abstract preflightLabServerRelease(
+    hostName: string,
+  ): Observable<PreflightLabServerReleaseResponse>;
 
   /**
    * Decommissions missing devices on a specific host.

src/devtools/mobileharness/fe/v6/angular/app/core/services/host/http_host_service.spec.ts

@@ -8,10 +8,11 @@ import {TestBed} from '@angular/core/testing';
 import {APP_DATA, AppData} from '../../models/app_data';
 import {
   DecommissionHostResponse,
+  DeployableVersion,
   GetHostDebugInfoResponse,
   HostHeaderInfo,
-  HostReleaseConfig,
   PopularFlag,
+  PreflightLabServerReleaseResponse,
   ReleaseLabServerRequest,
   ReleaseLabServerResponse,
   RestartLabServerResponse,
@@ -154,17 +155,19 @@ describe('HttpHostService', () => {
   });
 
   it('should retrieve release configs', () => {
-    const mockConfigs: HostReleaseConfig[] = [
-      {name: 'config1'} as HostReleaseConfig,
-    ];
-    service.getReleaseConfigs('test-host').subscribe((configs) => {
-      expect(configs).toEqual(mockConfigs);
+    const mockResponse: PreflightLabServerReleaseResponse = {
+      ready: {
+        versions: [{name: 'config1'} as DeployableVersion],
+      },
+    };
+    service.preflightLabServerRelease('test-host').subscribe((response) => {
+      expect(response).toEqual(mockResponse);
     });
     const req = httpMock.expectOne(
-      'http://testdomain.com/v6/hosts/test-host/release-configs',
+      `http://testdomain.com/v6/hosts/test-host/preflightLabServerRelease`,
     );
     expect(req.request.method).toBe('GET');
-    req.flush(mockConfigs);
+    req.flush(mockResponse);
   });
 
   it('should decommission host', () => {

src/devtools/mobileharness/fe/v6/angular/app/core/services/host/http_host_service.ts

@@ -9,8 +9,8 @@ import {
   DecommissionHostResponse,
   GetHostDebugInfoResponse,
   HostHeaderInfo,
-  HostReleaseConfig,
   PopularFlag,
+  PreflightLabServerReleaseResponse,
   ReleaseLabServerRequest,
   ReleaseLabServerResponse,
   RestartLabServerResponse,
@@ -85,11 +85,11 @@ export class HttpHostService extends HostService {
     );
   }
 
-  override getReleaseConfigs(
+  override preflightLabServerRelease(
     hostName: string,
-  ): Observable<HostReleaseConfig[]> {
-    return this.http.get<HostReleaseConfig[]>(
-      `${this.apiUrl}/${hostName}/release-configs`,
+  ): Observable<PreflightLabServerReleaseResponse> {
+    return this.http.get<PreflightLabServerReleaseResponse>(
+      `${this.apiUrl}/${hostName}/preflightLabServerRelease`,
     );
   }
 

src/devtools/mobileharness/fe/v6/angular/app/core/services/mock_data/hosts/01_no_config.ts

@@ -18,4 +18,7 @@ export const SCENARIO_HOST_NO_CONFIG: MockHostScenario = {
   },
   defaultDeviceConfig: null,
   actions: createHostActions(),
+  releaseResponse: {
+    permissionDenied: {},
+  },
 };

src/devtools/mobileharness/fe/v6/angular/app/core/services/mock_data/hosts/02_basic_editable.ts

@@ -5,6 +5,7 @@ import {HostConfig} from '../../../models/host_config_models';
 import {MockHostScenario} from '../models';
 import {
   createDefaultHostOverview,
+  createDefaultReleaseResponse,
   createDefaultUiStatus,
   createHostActions,
 } from './ui_status_utils';
@@ -44,4 +45,5 @@ export const SCENARIO_HOST_BASIC_EDITABLE: MockHostScenario = {
   },
   defaultDeviceConfig: DEFAULT_DEVICE_CONFIG,
   actions: createHostActions(),
+  releaseResponse: createDefaultReleaseResponse(),
 };

src/devtools/mobileharness/fe/v6/angular/app/core/services/mock_data/hosts/multi_remote_control.ts

@@ -6,7 +6,11 @@ import {
   HostOverview,
 } from '../../../models/host_overview';
 import {MockHostScenario} from '../models';
-import {createDefaultUiStatus, createHostActions} from './ui_status_utils';
+import {
+  createDefaultUiStatus,
+  createDeviceActions,
+  createHostActions,
+} from './ui_status_utils';
 
 // Helper to create a basic HostOverview
 function createHostOverview(
@@ -278,6 +282,7 @@ const DEVICES_PROXY_MISMATCH: DeviceSummary[] = [
     requiredDims: '',
     model: 'Pixel 8',
     version: '14',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
   {
     id: 'RC-PROXY-MISMATCH-2',
@@ -292,6 +297,7 @@ const DEVICES_PROXY_MISMATCH: DeviceSummary[] = [
     requiredDims: '',
     model: 'Pixel 8 Pro',
     version: '21',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
   {
     id: 'RC-NO-PROXY',
@@ -302,6 +308,7 @@ const DEVICES_PROXY_MISMATCH: DeviceSummary[] = [
     requiredDims: '',
     model: 'Pixel 8 Pro',
     version: '21',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
   {
     id: 'RC-TESTBED-NO-PROXY',

src/devtools/mobileharness/fe/v6/angular/app/core/services/mock_data/hosts/overview_13.ts

@@ -4,6 +4,7 @@ import {MockHostScenario} from '../models';
 import {
   createDefaultHostOverview,
   createDefaultUiStatus,
+  createDeviceActions,
   createHostActions,
 } from './ui_status_utils';
 
@@ -55,6 +56,7 @@ const deviceSummaries: DeviceSummary[] = [
     requiredDims: 'pool:missing',
     model: 'Pixel 6',
     version: '13',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
   {
     id: 'MISSING-DEV-2',
@@ -69,6 +71,7 @@ const deviceSummaries: DeviceSummary[] = [
     requiredDims: 'pool:missing',
     model: 'Pixel 6 Pro',
     version: '13',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
   {
     id: 'MISSING-DEV-3',
@@ -83,6 +86,7 @@ const deviceSummaries: DeviceSummary[] = [
     requiredDims: 'pool:missing',
     model: 'Pixel 7',
     version: '14',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
   {
     id: 'ACTIVE-DEV-4',
@@ -97,6 +101,7 @@ const deviceSummaries: DeviceSummary[] = [
     requiredDims: 'pool:active',
     model: 'Pixel 8',
     version: '14',
+    actions: createDeviceActions('ALL_PERMISSIONS'),
   },
 ];