refactor: clean up API v2 before release

Author: stefan-ernst

api/openapi-v2.json

@@ -50,11 +50,6 @@ export const actions = {
     fetchV2Data(`/workspaces/${workspaceId}/actions/${id}`, {
       method: 'DELETE',
     }),
-  toggle: (workspaceId, id, isEnabled) =>
-    fetchV2Data(`/workspaces/${workspaceId}/actions/${id}/toggle`, {
-      method: 'POST',
-      body: JSON.stringify({ is_enabled: isEnabled }),
-    }),
   execute: (workspaceId, actionId, itemId) =>
     fetchV2Data(`/workspaces/${workspaceId}/actions/${actionId}/execute`, {
       method: 'POST',

frontend/src/lib/api/actions.js

@@ -7,8 +7,8 @@ import { buildQueryString } from './utils.js';
  * (core.js sets credentials: 'same-origin').
  */
 export const pages = {
-  /** Fetch the workspace page tree + flat list. */
-  getTree: (workspaceId) => fetchV2Data(`/workspaces/${workspaceId}/pages/tree`),
+  /** Fetch every visible page as an ordered, flat metadata list. */
+  getAll: (workspaceId) => fetchV2Data(`/workspaces/${workspaceId}/pages`),
 
   /** Fetch a single page (404 on missing or no view permission). */
   getPage: (workspaceId, pageId) => fetchV2Data(`/workspaces/${workspaceId}/pages/${pageId}`),

frontend/src/lib/api/pages.js

@@ -10,13 +10,11 @@ import { testPlans } from './testPlans.js';
 import { testResults } from './testResults.js';
 import { testRuns } from './testRuns.js';
 import { testRunTemplates } from './testRunTemplates.js';
-import { testSets } from './testSets.js';
 
 export const tests = {
   testFolders,
   testLabels,
   testCases,
-  testSets,
   testPlans,
   testRunTemplates,
   testRuns,

frontend/src/lib/api/tests/index.js

@@ -1,4 +1,4 @@
-import { fetchAllV2Pages, fetchV2Data } from '../core.js';
+import { fetchAllV2Pages, fetchAPIV2, fetchV2Data } from '../core.js';
 import { createCrudClient } from '../createCrudClient.js';
 
 export const testCases = {
@@ -23,7 +23,10 @@ export const testCases = {
     const endpoint = `/workspaces/${workspaceId}/test-cases${queryString ? `?${queryString}` : ''}`;
     return params.all ? fetchAllV2Pages(endpoint) : fetchV2Data(endpoint);
   },
-  count: (workspaceId) => fetchV2Data(`/workspaces/${workspaceId}/test-cases/count`),
+  count: async (workspaceId) => {
+    const document = await fetchAPIV2(`/workspaces/${workspaceId}/test-cases?page=1&page_size=1`);
+    return { count: document?.pagination?.total_items ?? 0 };
+  },
   move: (workspaceId, id, data) =>
     fetchV2Data(`/workspaces/${workspaceId}/test-cases/${id}/move`, {
       method: 'POST',

frontend/src/lib/api/tests/testCases.js

@@ -1,7 +1,6 @@
 import { fetchV2Data } from '../core.js';
 import { createCrudClient } from '../createCrudClient.js';
 
-// Test Plans (preferred terminology, same as testSets)
 export const testPlans = {
   ...createCrudClient('/test-plans', { parentPath: '/workspaces', v2: true, allV2: true }),
   getTestCases: (workspaceId, id) =>

frontend/src/lib/api/tests/testPlans.js

@@ -30,7 +30,6 @@ export const statuses = {
   get: async (...args) => normalizeStatus(await statusCRUD.get(...args)),
   create: async (...args) => normalizeStatus(await statusCRUD.create(...args)),
   update: async (...args) => normalizeStatus(await statusCRUD.update(...args)),
-  getNonDoneIds: () => fetchV2Data('/statuses/non-completed-ids'),
 };
 
 const workflowCRUD = createCrudClient('/workflows', { v2: true });
@@ -73,6 +72,4 @@ export const workflows = {
       method: 'PUT',
       body: JSON.stringify({ transitions: data }),
     }),
-  getAvailableTransitions: (id, statusId) =>
-    fetchV2Data(`/workflows/${id}/statuses/${statusId}/transitions`),
 };

frontend/src/lib/api/tests/testSets.js

@@ -136,7 +136,7 @@
 
   async function handleToggle(action) {
     try {
-      await api.actions.toggle(workspaceId, action.id, !action.is_enabled);
+      await api.actions.update(workspaceId, action.id, { is_enabled: !action.is_enabled });
       await loadActions();
       successToast(action.is_enabled ? t('actions.disabled') : t('actions.enabled'));
     } catch (error) {

frontend/src/lib/api/workflows.js

@@ -2,14 +2,19 @@
   import MethodBadge from './MethodBadge.svelte';
   import ApiSchema from './ApiSchema.svelte';
   import { renderMarkdown } from '../../utils/render-markdown.js';
-  import { resolveRef } from './openapi-store.svelte.js';
+  import {
+    operationRequiredScopes,
+    resolveOperationParameters,
+    resolveRef,
+  } from './openapi-store.svelte.js';
 
   let { spec, entry } = $props();
 
   // entry = { tag, path, method, operation, id }
   const op = $derived(entry.operation);
-  const params = $derived(op.parameters || []);
+  const params = $derived(resolveOperationParameters(spec, entry));
   const grouped = $derived(groupParams(params));
+  const requiredScopes = $derived(operationRequiredScopes(op));
 
   function groupParams(list) {
     const buckets = { path: [], query: [], header: [], cookie: [] };
@@ -82,12 +87,15 @@
           {#each Object.keys(req) as scheme}
             <span class="security-pill">
               {scheme}
-              {#if req[scheme] && req[scheme].length > 0}
-                <span class="security-scopes">({req[scheme].join(', ')})</span>
-              {/if}
             </span>
           {/each}
         {/each}
+        {#if requiredScopes.length > 0}
+          <span class="security-pill">
+            Required scopes
+            <span class="security-scopes">({requiredScopes.join(', ')})</span>
+          </span>
+        {/if}
       </div>
     </section>
   {/if}