Query Hooks Improvements

public/locales/gsa-de.json

@@ -1604,6 +1604,7 @@
   "Port Counts: Total": "Portanzahl: Gesamt",
   "Port Counts: UDP": "Portanzahl: UDP",
   "Port List": "Portliste",
+  "Port List imported successfully.": "Portliste erfolgreich importiert.",
   "Port List: {{name}}": "Portliste: {{name}}",
   "Port Lists": "Portlisten",
   "Port Lists Filter": "Portlisten-Filter",

public/locales/gsa-en.json

@@ -1604,6 +1604,7 @@
   "Port Counts: Total": "Port Counts: Total",
   "Port Counts: UDP": "Port Counts: UDP",
   "Port List": "Port List",
+  "Port List imported successfully.": "Port List imported successfully.",
   "Port List: {{name}}": "Port List: {{name}}",
   "Port Lists": "Port Lists",
   "Port Lists Filter": "Port Lists Filter",

public/locales/gsa-it.json

@@ -1604,6 +1604,7 @@
   "Port Counts: Total": "Conteggi porte: Totale",
   "Port Counts: UDP": "Conteggi porte: UDP",
   "Port List": "Elenco porte",
+  "Port List imported successfully.": "",
   "Port List: {{name}}": "Elenco porte: {{name}}",
   "Port Lists": "Elenchi porte",
   "Port Lists Filter": "Filtro elenchi porte",

public/locales/gsa-ja.json

@@ -1604,6 +1604,7 @@
   "Port Counts: Total": "ポート数: 合計",
   "Port Counts: UDP": "ポート数: UDP",
   "Port List": "ポートリスト",
+  "Port List imported successfully.": "",
   "Port List: {{name}}": "ポートリスト: {{name}}",
   "Port Lists": "ポートリスト",
   "Port Lists Filter": "ポートリストフィルター",

public/locales/gsa-zh_CN.json

@@ -1604,6 +1604,7 @@
   "Port Counts: Total": "端口数量: 总数",
   "Port Counts: UDP": "端口数量: UDP",
   "Port List": "端口列表",
+  "Port List imported successfully.": "",
   "Port List: {{name}}": "端口列表：{{name}}",
   "Port Lists": "端口列表",
   "Port Lists Filter": "端口列表筛选",

public/locales/gsa-zh_TW.json

@@ -1604,6 +1604,7 @@
   "Port Counts: Total": "",
   "Port Counts: UDP": "",
   "Port List": "連接埠清單",
+  "Port List imported successfully.": "",
   "Port List: {{name}}": "",
   "Port Lists": "連接埠清單",
   "Port Lists Filter": "",

src/gmp/commands/__tests__/port-list.test.ts

@@ -8,7 +8,6 @@ import PortListCommand, {FROM_FILE} from 'gmp/commands/port-list';
 import {
   createHttp,
   createActionResultResponse,
-  createHttpMany,
   createEntityResponse,
 } from 'gmp/commands/testing';
 
@@ -95,14 +94,13 @@ describe('PortListCommand tests', () => {
       action: 'delete_port_range',
       id: '12345',
     });
-    const entityResponse = createEntityResponse('port_list', {id: '324'});
-    const http = createHttpMany([response, entityResponse]);
+    const http = createHttp(response);
     const command = new PortListCommand(http);
     const result = await command.deletePortRange({
       id: '12345',
-      portListId: '67890',
     });
-    expect(result.data.id).toEqual('324');
+    expect(result).toBeUndefined();
+    expect(http.request).toHaveBeenCalledTimes(1);
   });
 
   test('should allow to get a port list', async () => {

src/gmp/commands/__tests__/web-application-target.test.ts

@@ -122,7 +122,7 @@ describe('WebApplicationTargetCommand tests', () => {
         in_use: YES_VALUE,
       },
     });
-    expect(result.data.id).toEqual('wat-123');
+    expect(result).toBeUndefined();
   });
 
   test('should omit exclude_url when not provided in save', async () => {

src/gmp/commands/credential-store.ts

@@ -28,6 +28,10 @@ export interface CredentialStoreModifyParams {
   serverCaCertificate?: File;
 }
 
+export interface CredentialStoreVerifyParams {
+  id: string;
+}
+
 const log = logger.getLogger('gmp.commands.credentialStore');
 
 class CredentialStoreCommand extends EntityCommand<
@@ -90,7 +94,7 @@ class CredentialStoreCommand extends EntityCommand<
     });
   }
 
-  async verify({id}: {id: string}): Promise<EntityActionResponse> {
+  async verify({id}: CredentialStoreVerifyParams) {
     log.debug('Verifying credential store', {id});
 
     return this.entityAction({

src/gmp/commands/credential.ts

@@ -49,7 +49,7 @@ interface CredentialCommandSnmpFields {
 }
 
 // Create operation interfaces
-type CredentialCommandCreateArgs = CredentialCommandBaseArgs;
+export type CredentialCommandCreateParams = CredentialCommandBaseArgs;
 
 interface CredentialCommandKrb5Args
   extends CredentialCommandBaseArgs, CredentialCommandKrb5Fields {}
@@ -145,9 +145,9 @@ class CredentialCommand extends EntityCommand<
     };
   }
 
-  create(args: CredentialCommandCreateArgs) {
+  create(args: CredentialCommandCreateParams) {
     const baseData = this.createBase(args);
-    return this.action(baseData);
+    return this.entityAction(baseData);
   }
 
   createKrb5(args: CredentialCommandKrb5Args) {

src/gmp/commands/port-list.ts

@@ -26,19 +26,18 @@ export interface PortListCommandSaveParams {
   comment?: string;
 }
 
-interface PortListCommandCreatePortRangeParams {
+export interface PortListCommandCreatePortRangeParams {
   portListId: string;
   portRangeStart: number;
   portRangeEnd: number;
   portType: string;
 }
 
-interface PortListCommandDeletePortRangeParams {
+export interface PortListCommandDeletePortRangeParams {
   id: string;
-  portListId: string;
 }
 
-interface PortListCommandImportParams {
+export interface PortListCommandImportParams {
   xmlFile?: File;
 }
 
@@ -101,16 +100,12 @@ class PortListCommand extends EntityCommand<PortList, PortListElement> {
     });
   }
 
-  async deletePortRange({
-    id,
-    portListId,
-  }: PortListCommandDeletePortRangeParams) {
-    await this.httpPostWithTransform({
+  async deletePortRange({id}: PortListCommandDeletePortRangeParams) {
+    await this.action({
       cmd: 'delete_port_range',
       port_range_id: id,
       no_redirect: 1,
     });
-    return await this.get({id: portListId});
   }
 
   import({xmlFile}: PortListCommandImportParams) {

src/gmp/commands/target.ts

@@ -16,7 +16,7 @@ import {UNSET_VALUE} from 'web/utils/Render';
 export type TargetSource = 'manual' | 'file' | 'asset_hosts';
 export type TargetExcludeSource = 'manual' | 'file';
 
-interface TargetCommandCreateParams {
+export interface TargetCommandCreateParams {
   aliveTests?: AliveTest[];
   allowSimultaneousIPs?: boolean;
   comment?: string;

src/gmp/commands/user.ts

@@ -108,9 +108,9 @@ interface SaveArguments extends CreateArguments {
   oldName: string;
 }
 
-interface DeleteArguments {
+export interface UserCommandDeleteParams {
   id: string;
-  inheritorId: string;
+  inheritorId?: string;
 }
 
 const log = logger.getLogger('gmp.commands.users');
@@ -367,7 +367,7 @@ class UserCommand extends EntityCommand<User, PortListElement> {
     return this.action(data);
   }
 
-  async delete({id, inheritorId}: DeleteArguments) {
+  async delete({id, inheritorId}: UserCommandDeleteParams) {
     const data = {
       cmd: 'delete_user',
       id,

src/gmp/commands/web-application-target.ts

@@ -81,15 +81,15 @@ class WebApplicationTargetCommand extends EntityCommand<WebApplicationTarget> {
     reverseLookupOnly,
     reverseLookupUnify,
     inUse,
-  }: WebApplicationTargetSaveParams): Promise<EntityActionResponse> {
+  }: WebApplicationTargetSaveParams) {
     log.debug('Modifying web application target', {
       id,
       name,
       comment,
       urls,
       credentialId,
     });
-    return this.entityAction({
+    await this.entityAction({
       cmd: 'save_web_application_target',
       web_application_target_id: id,
       name,

src/web/entity/navigation.ts

@@ -7,21 +7,23 @@ import {type EntityType} from 'gmp/utils/entity-type';
 import {isDefined, isFunction} from 'gmp/utils/identity';
 import {entityListURL, entityURL, type EntityListType} from 'web/route-paths';
 
-export type NavigateFunc = (path: string) => void;
+type NavigateFunc = (path: string) => void;
 
-export interface NavigateObj {
+interface NavigateObj {
   navigate: NavigateFunc;
 }
 
-export type Navigate = NavigateFunc | NavigateObj;
+type Navigate = NavigateFunc | NavigateObj;
 
-export interface GotoDetailsObj {
-  data: {
-    id: string;
-  };
+interface GotoDetailsData {
+  id: string;
 }
 
-export type GotoDetailsFunc = ({data}: GotoDetailsObj) => void;
+interface GotoDetailsObj {
+  data: GotoDetailsData;
+}
+
+export type GotoDetailsFunc = (data: GotoDetailsObj | GotoDetailsData) => void;
 export type GotoListFunc = () => void;
 
 /**
@@ -47,7 +49,8 @@ export const goToDetails = (
       throw new Error('navigate function is required for goToDetails');
     }
   }
-  return ({data}) => navigate(entityURL(type, String(data.id)));
+  return data =>
+    navigate(entityURL(type, String('id' in data ? data.id : data.data.id)));
 };
 
 /**

src/web/hooks/use-query/__tests__/Agent.test.tsx

@@ -0,0 +1,97 @@
+/* SPDX-FileCopyrightText: 2026 Greenbone AG
+ *
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+import {describe, expect, test, testing} from '@gsa/testing';
+import {fireEvent, rendererWith, screen, waitFor} from 'web/testing';
+import {createSession} from 'gmp/testing';
+import {
+  useDeleteAgent,
+  useDownloadAgentSupportBundle,
+  useModifyAgent,
+} from 'web/hooks/use-query/agent';
+
+const createGmp = () => ({
+  session: createSession({token: 'test-token'}),
+  settings: {},
+  agent: {
+    save: testing.fn().mockResolvedValue(undefined),
+    delete: testing.fn().mockResolvedValue(undefined),
+    downloadSupportBundle: testing
+      .fn()
+      .mockResolvedValue({data: new ArrayBuffer(0)}),
+  },
+});
+
+describe('useModifyAgent', () => {
+  test('should save an agent', async () => {
+    const gmp = createGmp();
+    const {render} = rendererWith({gmp, router: true});
+    const input = {agentsIds: ['agent-1'], authorized: true};
+    const TestComponent = () => {
+      const mutation = useModifyAgent();
+      return <button onClick={() => mutation.mutate(input)}>Save</button>;
+    };
+
+    render(<TestComponent />);
+    fireEvent.click(screen.getByRole('button', {name: 'Save'}));
+    await waitFor(() => {
+      expect(gmp.agent.save.mock.calls[0][0]).toEqual(input);
+    });
+  });
+});
+
+describe('useDeleteAgent', () => {
+  test('should delete an agent', async () => {
+    const gmp = createGmp();
+    const {render} = rendererWith({gmp, router: true});
+    const TestComponent = () => {
+      const mutation = useDeleteAgent({});
+      return (
+        <button
+          onClick={() =>
+            mutation.mutate({
+              id: 'agent-1',
+              name: 'Deleted Agent',
+            })
+          }
+        >
+          Delete
+        </button>
+      );
+    };
+
+    render(<TestComponent />);
+    fireEvent.click(screen.getByRole('button', {name: 'Delete'}));
+    await waitFor(() => {
+      expect(gmp.agent.delete).toHaveBeenCalledWith({id: 'agent-1'});
+    });
+  });
+});
+
+describe('useDownloadAgentSupportBundle', () => {
+  test('should download an agent support bundle', async () => {
+    const gmp = createGmp();
+    const {render} = rendererWith({gmp, router: true});
+    const TestComponent = () => {
+      const mutation = useDownloadAgentSupportBundle();
+      return (
+        <button
+          onClick={() => mutation.mutate({id: 'agent-1', encryption: false})}
+        >
+          Download
+        </button>
+      );
+    };
+
+    render(<TestComponent />);
+    fireEvent.click(screen.getByRole('button', {name: 'Download'}));
+    await waitFor(() => {
+      expect(gmp.agent.downloadSupportBundle).toHaveBeenCalledWith(
+        'agent-1',
+        false,
+      );
+    });
+  });
+});