github-issue-tower-defence-management 2.19.1 → 2.19.2

package/src/adapter/entry-points/console/ui/src/features/console/hooks/useConsoleFeaturesConfig.test.ts

@@ -1,51 +0,0 @@
-import { renderHook, waitFor } from '@testing-library/react';
-import { useConsoleFeaturesConfig } from './useConsoleFeaturesConfig';
-
-describe('useConsoleFeaturesConfig', () => {
-  afterEach(() => {
-    jest.restoreAllMocks();
-  });
-
-  it('returns airplaneMode false while loading (initial state)', () => {
-    global.fetch = jest.fn().mockReturnValue(new Promise(() => {}));
-    const { result } = renderHook(() => useConsoleFeaturesConfig());
-    expect(result.current.airplaneMode).toBe(false);
-  });
-
-  it('returns airplaneMode true when /api/features responds { airplaneMode: true }', async () => {
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: true,
-      json: async () => ({ airplaneMode: true }),
-    });
-    const { result } = renderHook(() => useConsoleFeaturesConfig());
-    await waitFor(() => expect(result.current.airplaneMode).toBe(true));
-  });
-
-  it('returns airplaneMode false when /api/features responds { airplaneMode: false }', async () => {
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: true,
-      json: async () => ({ airplaneMode: false }),
-    });
-    const { result } = renderHook(() => useConsoleFeaturesConfig());
-    await waitFor(() => expect(global.fetch).toHaveBeenCalled());
-    expect(result.current.airplaneMode).toBe(false);
-  });
-
-  it('returns airplaneMode false on fetch failure', async () => {
-    global.fetch = jest.fn().mockRejectedValue(new Error('network error'));
-    const consoleSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
-    const { result } = renderHook(() => useConsoleFeaturesConfig());
-    await waitFor(() => expect(consoleSpy).toHaveBeenCalled());
-    expect(result.current.airplaneMode).toBe(false);
-  });
-
-  it('returns airplaneMode false when response is not ok', async () => {
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: false,
-      json: async () => ({ airplaneMode: true }),
-    });
-    const { result } = renderHook(() => useConsoleFeaturesConfig());
-    await waitFor(() => expect(global.fetch).toHaveBeenCalled());
-    expect(result.current.airplaneMode).toBe(false);
-  });
-});

package/src/adapter/entry-points/console/ui/src/features/console/hooks/useConsoleFeaturesConfig.ts

@@ -1,39 +0,0 @@
-import { useEffect, useState } from 'react';
-
-export type ConsoleFeaturesConfig = {
-  airplaneMode: boolean;
-};
-
-const FEATURES_PATH = '/api/features';
-
-export const useConsoleFeaturesConfig = (): ConsoleFeaturesConfig => {
-  const [config, setConfig] = useState<ConsoleFeaturesConfig>({
-    airplaneMode: false,
-  });
-
-  useEffect(() => {
-    fetch(FEATURES_PATH)
-      .then(async (response) => {
-        if (!response.ok) {
-          return;
-        }
-        const data: unknown = await response.json();
-        if (
-          data !== null &&
-          typeof data === 'object' &&
-          !Array.isArray(data) &&
-          'airplaneMode' in data &&
-          typeof (data as { airplaneMode: unknown }).airplaneMode === 'boolean'
-        ) {
-          setConfig({
-            airplaneMode: (data as { airplaneMode: boolean }).airplaneMode,
-          });
-        }
-      })
-      .catch((err: unknown) => {
-        console.warn('Failed to fetch /api/features', err);
-      });
-  }, []);
-
-  return config;
-};

package/src/adapter/entry-points/console/ui/src/features/console/hooks/useConsoleItemDetailData.test.ts

@@ -1,229 +0,0 @@
-import { renderHook, waitFor } from '@testing-library/react';
-import { ResourceCache } from '../lib/resourceCache';
-import type {
-  ConsoleChangedFile,
-  ConsoleComment,
-  ConsoleCommit,
-  ConsoleIssueState,
-  ConsoleListItem,
-  ConsolePullRequestStatus,
-  ConsoleRelatedPullRequest,
-} from '../logic/types';
-import type { ConsoleCaches } from './useConsoleCaches';
-import { useConsoleItemDetailData } from './useConsoleItemDetailData';
-
-const prItem: ConsoleListItem = {
-  number: 1,
-  title: 'PR',
-  url: 'https://github.com/o/r/pull/1',
-  repo: 'o/r',
-  nameWithOwner: 'o/r',
-  projectItemId: 'PVTI_1',
-  itemId: 'PVTI_1',
-  isPr: true,
-  relatedOpenPullRequestUrls: [],
-  story: 'Story',
-  status: null,
-  agent: null,
-  nextActionDate: null,
-  nextActionHour: null,
-  dependedIssueUrls: [],
-  labels: [],
-  createdAt: '2026-06-10T00:00:00.000Z',
-};
-
-const issueItem: ConsoleListItem = {
-  ...prItem,
-  isPr: false,
-  relatedOpenPullRequestUrls: ['https://github.com/o/r/pull/9'],
-  url: 'https://github.com/o/r/issues/2',
-  number: 2,
-};
-
-const issueItemWithoutRelatedPullRequests: ConsoleListItem = {
-  ...issueItem,
-  relatedOpenPullRequestUrls: [],
-  url: 'https://github.com/o/r/issues/3',
-  number: 3,
-};
-
-const buildCaches = (related: ConsoleRelatedPullRequest[]): ConsoleCaches => {
-  const client = {
-    fetchItemBody: async () => 'body',
-    fetchComments: async (): Promise<ConsoleComment[]> => [],
-    fetchPrFiles: async (): Promise<ConsoleChangedFile[]> => [
-      {
-        path: 'a.ts',
-        additions: 1,
-        deletions: 0,
-        status: 'added',
-        patch: null,
-        rawUrl: null,
-      },
-    ],
-    fetchPrCommits: async (): Promise<ConsoleCommit[]> => [
-      {
-        sha: 'abc',
-        message: 'm',
-        author: 'a',
-        authoredAt: '2026-06-10T00:00:00.000Z',
-      },
-    ],
-    fetchRelatedPrs: async (): Promise<ConsoleRelatedPullRequest[]> => related,
-    fetchIssueState: async (): Promise<ConsoleIssueState> => ({
-      state: 'open',
-      merged: false,
-      isPullRequest: true,
-      title: 'Console item detail fixture title',
-    }),
-    fetchPullRequestStatus: async (): Promise<ConsolePullRequestStatus> => ({
-      found: true,
-      isConflicted: true,
-      mergeableStatus: 'CONFLICTING',
-      isPassedAllCiJob: false,
-      isCiStateSuccess: false,
-      isBranchOutOfDate: true,
-      missingRequiredCheckNames: ['build'],
-    }),
-  };
-  return {
-    client,
-    body: new ResourceCache(client.fetchItemBody),
-    comments: new ResourceCache(client.fetchComments),
-    files: new ResourceCache(client.fetchPrFiles),
-    commits: new ResourceCache(client.fetchPrCommits),
-    relatedPrs: new ResourceCache(client.fetchRelatedPrs),
-    state: new ResourceCache(client.fetchIssueState),
-    prStatus: new ResourceCache(client.fetchPullRequestStatus),
-  };
-};
-
-const buildFailingCaches = (): ConsoleCaches => {
-  const caches = buildCaches([]);
-  return {
-    ...caches,
-    relatedPrs: new ResourceCache<ConsoleRelatedPullRequest[]>(() =>
-      Promise.reject(new Error('related pull request read failed')),
-    ),
-    state: new ResourceCache<ConsoleIssueState>(() =>
-      Promise.reject(new Error('state read failed')),
-    ),
-    prStatus: new ResourceCache<ConsolePullRequestStatus>(() =>
-      Promise.reject(new Error('pull request status read failed')),
-    ),
-  };
-};
-
-describe('useConsoleItemDetailData', () => {
-  it('exposes the related pull request read failure for an issue item', async () => {
-    const caches = buildFailingCaches();
-    const { result } = renderHook(() =>
-      useConsoleItemDetailData(caches, issueItem),
-    );
-    await waitFor(() => {
-      expect(result.current.relatedPullRequestsError).toBe(
-        'related pull request read failed',
-      );
-    });
-    expect(result.current.relatedPullRequests).toEqual([]);
-    expect(result.current.stateError).toBe('state read failed');
-  });
-
-  it('exposes the pull request status read failure for a PR item', async () => {
-    const caches = buildFailingCaches();
-    const { result } = renderHook(() =>
-      useConsoleItemDetailData(caches, prItem),
-    );
-    await waitFor(() => {
-      expect(result.current.pullRequestStatusError).toBe(
-        'pull request status read failed',
-      );
-    });
-  });
-
-  it('loads body, files and commits for a PR item', async () => {
-    const caches = buildCaches([]);
-    const { result } = renderHook(() =>
-      useConsoleItemDetailData(caches, prItem),
-    );
-    await waitFor(() => {
-      expect(result.current.body).toBe('body');
-      expect(result.current.files.length).toBe(1);
-      expect(result.current.commits.length).toBe(1);
-      expect(result.current.pullRequestStatus?.found).toBe(true);
-      expect(result.current.pullRequestStatus?.isConflicted).toBe(true);
-    });
-  });
-
-  it('does not expose pull request status for an issue item', () => {
-    const caches = buildCaches([]);
-    const { result } = renderHook(() =>
-      useConsoleItemDetailData(caches, issueItem),
-    );
-    expect(result.current.pullRequestStatus).toBeNull();
-  });
-
-  it('loads related pull request views for an issue item', async () => {
-    const related: ConsoleRelatedPullRequest[] = [
-      {
-        url: 'https://github.com/o/r/pull/9',
-        branchName: 'feat',
-        createdAt: '2026-06-10T00:00:00.000Z',
-        isDraft: false,
-        isConflicted: false,
-        mergeableStatus: 'MERGEABLE',
-        isPassedAllCiJob: true,
-        isCiStateSuccess: true,
-        isResolvedAllReviewComments: true,
-        isBranchOutOfDate: false,
-        missingRequiredCheckNames: [],
-        summary: {
-          title: 'Linked PR',
-          body: 'body',
-          additions: 5,
-          deletions: 1,
-          changedFiles: 2,
-        },
-      },
-    ];
-    const caches = buildCaches(related);
-    const { result } = renderHook(() =>
-      useConsoleItemDetailData(caches, issueItem),
-    );
-    await waitFor(() => {
-      expect(result.current.relatedPullRequests.length).toBe(1);
-      expect(result.current.relatedPullRequests[0].filesAreLoading).toBe(false);
-    });
-    expect(result.current.relatedPullRequests[0].files.length).toBe(1);
-  });
-
-  it('reads no related pull requests for an issue the board lists no open pull request for', async () => {
-    const fetchRelatedPrs = jest.fn(
-      async (): Promise<ConsoleRelatedPullRequest[]> => [],
-    );
-    const caches: ConsoleCaches = {
-      ...buildCaches([]),
-      relatedPrs: new ResourceCache<ConsoleRelatedPullRequest[]>(
-        fetchRelatedPrs,
-      ),
-    };
-    const { result } = renderHook(() =>
-      useConsoleItemDetailData(caches, issueItemWithoutRelatedPullRequests),
-    );
-
-    await waitFor(() => {
-      expect(result.current.body).toBe('body');
-    });
-
-    expect(fetchRelatedPrs).not.toHaveBeenCalled();
-    expect(result.current.relatedPullRequests).toEqual([]);
-    expect(result.current.relatedPullRequestsError).toBeNull();
-  });
-
-  it('returns defaults when no item is selected', () => {
-    const caches = buildCaches([]);
-    const { result } = renderHook(() => useConsoleItemDetailData(caches, null));
-    expect(result.current.body).toBe('');
-    expect(result.current.relatedPullRequests).toEqual([]);
-  });
-});

package/src/adapter/entry-points/console/ui/src/features/console/hooks/useConsoleItemDetailData.ts

@@ -1,194 +0,0 @@
-import { useEffect, useState } from 'react';
-import type { ConsoleRelatedPullRequestView } from '../components/detail/ConsoleItemDetail';
-import type {
-  ConsoleChangedFile,
-  ConsoleComment,
-  ConsoleCommit,
-  ConsoleIssueState,
-  ConsoleListItem,
-  ConsolePullRequestStatus,
-  ConsoleRelatedPullRequest,
-} from '../logic/types';
-import type { ConsoleCaches } from './useConsoleCaches';
-import { useConsoleResource } from './useConsoleResource';
-
-const EMPTY_COMMENTS: ConsoleComment[] = [];
-const EMPTY_FILES: ConsoleChangedFile[] = [];
-const EMPTY_COMMITS: ConsoleCommit[] = [];
-const EMPTY_RELATED: ConsoleRelatedPullRequest[] = [];
-const DEFAULT_STATE: ConsoleIssueState = {
-  state: 'open',
-  merged: false,
-  isPullRequest: false,
-  title: '',
-};
-const DEFAULT_PR_STATUS: ConsolePullRequestStatus = {
-  found: false,
-  isConflicted: false,
-  mergeableStatus: 'UNKNOWN',
-  isPassedAllCiJob: false,
-  isCiStateSuccess: false,
-  isBranchOutOfDate: false,
-  missingRequiredCheckNames: [],
-};
-
-export type ConsoleItemDetailData = {
-  state: ConsoleIssueState | null;
-  body: string;
-  bodyIsLoading: boolean;
-  bodyError: string | null;
-  comments: ConsoleComment[];
-  commentsAreLoading: boolean;
-  commentsError: string | null;
-  files: ConsoleChangedFile[];
-  filesAreLoading: boolean;
-  filesError: string | null;
-  commits: ConsoleCommit[];
-  commitsAreLoading: boolean;
-  commitsError: string | null;
-  stateError: string | null;
-  pullRequestStatus: ConsolePullRequestStatus | null;
-  pullRequestStatusError: string | null;
-  relatedPullRequests: ConsoleRelatedPullRequestView[];
-  relatedPullRequestsError: string | null;
-};
-
-export const useConsoleItemDetailData = (
-  caches: ConsoleCaches,
-  item: ConsoleListItem | null,
-): ConsoleItemDetailData => {
-  const key = item !== null ? `${item.repo}#${item.number}` : null;
-  const url = item !== null ? item.url : null;
-  const isPr = item?.isPr ?? false;
-
-  const body = useConsoleResource(caches.body, key, url, '');
-  const state = useConsoleResource(caches.state, key, url, DEFAULT_STATE);
-  const comments = useConsoleResource(
-    caches.comments,
-    key,
-    url,
-    EMPTY_COMMENTS,
-  );
-  const files = useConsoleResource(
-    caches.files,
-    isPr ? key : null,
-    isPr ? url : null,
-    EMPTY_FILES,
-  );
-  const commits = useConsoleResource(
-    caches.commits,
-    isPr ? key : null,
-    isPr ? url : null,
-    EMPTY_COMMITS,
-  );
-  const prStatus = useConsoleResource(
-    caches.prStatus,
-    isPr ? key : null,
-    isPr ? url : null,
-    DEFAULT_PR_STATUS,
-  );
-  const boardListsRelatedPullRequests =
-    !isPr && (item?.relatedOpenPullRequestUrls.length ?? 0) > 0;
-  const relatedPrs = useConsoleResource(
-    caches.relatedPrs,
-    boardListsRelatedPullRequests ? key : null,
-    boardListsRelatedPullRequests ? url : null,
-    EMPTY_RELATED,
-  );
-
-  const [relatedViews, setRelatedViews] = useState<
-    ConsoleRelatedPullRequestView[]
-  >([]);
-
-  useEffect(() => {
-    if (isPr || relatedPrs.data.length === 0) {
-      setRelatedViews([]);
-      return;
-    }
-    let cancelled = false;
-    const initial: ConsoleRelatedPullRequestView[] = relatedPrs.data.map(
-      (pullRequest) => ({
-        pullRequest,
-        files: EMPTY_FILES,
-        filesAreLoading: true,
-        filesError: null,
-        commits: EMPTY_COMMITS,
-        commitsAreLoading: true,
-        commitsError: null,
-      }),
-    );
-    setRelatedViews(initial);
-
-    const updateView = (
-      prUrl: string,
-      patch: Partial<ConsoleRelatedPullRequestView>,
-    ): void => {
-      if (cancelled) {
-        return;
-      }
-      setRelatedViews((current) =>
-        current.map((view) =>
-          view.pullRequest.url === prUrl ? { ...view, ...patch } : view,
-        ),
-      );
-    };
-
-    for (const pullRequest of relatedPrs.data) {
-      const relatedKey = pullRequest.url;
-      caches.files
-        .load(relatedKey, pullRequest.url)
-        .then((value) =>
-          updateView(pullRequest.url, {
-            files: value,
-            filesAreLoading: false,
-          }),
-        )
-        .catch((cause: unknown) =>
-          updateView(pullRequest.url, {
-            filesAreLoading: false,
-            filesError: cause instanceof Error ? cause.message : String(cause),
-          }),
-        );
-      caches.commits
-        .load(relatedKey, pullRequest.url)
-        .then((value) =>
-          updateView(pullRequest.url, {
-            commits: value,
-            commitsAreLoading: false,
-          }),
-        )
-        .catch((cause: unknown) =>
-          updateView(pullRequest.url, {
-            commitsAreLoading: false,
-            commitsError:
-              cause instanceof Error ? cause.message : String(cause),
-          }),
-        );
-    }
-
-    return () => {
-      cancelled = true;
-    };
-  }, [caches, isPr, relatedPrs.data]);
-
-  return {
-    state: state.data,
-    body: body.data,
-    bodyIsLoading: body.isLoading,
-    bodyError: body.error,
-    comments: comments.data,
-    commentsAreLoading: comments.isLoading,
-    commentsError: comments.error,
-    files: files.data,
-    filesAreLoading: files.isLoading,
-    filesError: files.error,
-    commits: commits.data,
-    commitsAreLoading: commits.isLoading,
-    commitsError: commits.error,
-    stateError: state.error,
-    pullRequestStatus: isPr ? prStatus.data : null,
-    pullRequestStatusError: isPr ? prStatus.error : null,
-    relatedPullRequests: relatedViews,
-    relatedPullRequestsError: isPr ? null : relatedPrs.error,
-  };
-};

package/src/adapter/entry-points/console/ui/src/features/console/hooks/useConsoleNavigation.test.ts

@@ -1,151 +0,0 @@
-import { act, renderHook } from '@testing-library/react';
-import type { ConsoleTabName } from '../logic/types';
-import {
-  parseItemKeyFromHash,
-  parseTabFromPath,
-  useConsoleNavigation,
-} from './useConsoleNavigation';
-
-const counts = (
-  overrides: Partial<Record<ConsoleTabName, number>> = {},
-): Record<ConsoleTabName, number> => ({
-  'workflow-blocker': 0,
-  prs: 0,
-  'failed-preparation': 0,
-  'todo-by-human': 0,
-  'todo-by-agent': 0,
-  queued: 0,
-  stories: 0,
-  ...overrides,
-});
-
-describe('parseTabFromPath', () => {
-  it('reads a known tab from the project path', () => {
-    expect(parseTabFromPath('/projects/acme/prs')).toBe('prs');
-  });
-
-  it('returns null for an unknown tab segment', () => {
-    expect(parseTabFromPath('/projects/acme/unknown')).toBeNull();
-  });
-
-  it('returns null for the removed triage segment', () => {
-    expect(parseTabFromPath('/projects/acme/triage')).toBeNull();
-  });
-
-  it('returns null when there is no tab segment', () => {
-    expect(parseTabFromPath('/projects/acme')).toBeNull();
-  });
-});
-
-describe('parseItemKeyFromHash', () => {
-  it('decodes the item key from the hash', () => {
-    expect(parseItemKeyFromHash('#item/PVTI_lADO%20123')).toBe('PVTI_lADO 123');
-  });
-
-  it('returns null when the hash is not an item hash', () => {
-    expect(parseItemKeyFromHash('#other')).toBeNull();
-  });
-});
-
-describe('useConsoleNavigation', () => {
-  beforeEach(() => {
-    window.history.replaceState({}, '', '/projects/acme/prs?k=token');
-  });
-
-  it('reads the active tab from the path and no selected item', () => {
-    const { result } = renderHook(() => useConsoleNavigation('acme', counts()));
-    expect(result.current.activeTab).toBe('prs');
-    expect(result.current.selectedItemKey).toBeNull();
-  });
-
-  it('builds a project tab href', () => {
-    const { result } = renderHook(() => useConsoleNavigation('acme', counts()));
-    expect(result.current.tabHref('todo-by-human')).toBe(
-      '/projects/acme/todo-by-human',
-    );
-  });
-
-  it('selects a tab and updates the path', () => {
-    const { result } = renderHook(() => useConsoleNavigation('acme', counts()));
-    act(() => {
-      result.current.selectTab('todo-by-human');
-    });
-    expect(result.current.activeTab).toBe('todo-by-human');
-    expect(window.location.pathname).toBe('/projects/acme/todo-by-human');
-  });
-
-  it('opens an item and reflects it in the hash', () => {
-    const { result } = renderHook(() => useConsoleNavigation('acme', counts()));
-    act(() => {
-      result.current.openItem('PVTI_open');
-    });
-    expect(result.current.selectedItemKey).toBe('PVTI_open');
-    expect(window.location.hash).toBe('#item/PVTI_open');
-  });
-
-  it('closes an item and clears the hash', () => {
-    window.history.replaceState({}, '', '/projects/acme/prs#item/PVTI_open');
-    const { result } = renderHook(() => useConsoleNavigation('acme', counts()));
-    expect(result.current.selectedItemKey).toBe('PVTI_open');
-    act(() => {
-      result.current.closeItem();
-    });
-    expect(result.current.selectedItemKey).toBeNull();
-    expect(window.location.hash).toBe('');
-  });
-});
-
-describe('useConsoleNavigation default tab without a tab segment', () => {
-  beforeEach(() => {
-    window.history.replaceState({}, '', '/projects/acme?k=token');
-  });
-
-  it('lands on the left-most tab when all tabs are non-empty', () => {
-    const { result } = renderHook(() =>
-      useConsoleNavigation(
-        'acme',
-        counts({
-          'workflow-blocker': 3,
-          prs: 5,
-          'failed-preparation': 1,
-          'todo-by-human': 4,
-        }),
-      ),
-    );
-    expect(result.current.activeTab).toBe('workflow-blocker');
-  });
-
-  it('skips the empty left-most tab and lands on the next non-empty tab', () => {
-    const { result } = renderHook(() =>
-      useConsoleNavigation(
-        'acme',
-        counts({ 'workflow-blocker': 0, prs: 0, 'failed-preparation': 8 }),
-      ),
-    );
-    expect(result.current.activeTab).toBe('failed-preparation');
-  });
-
-  it('falls back to the first tab when every tab is empty', () => {
-    const { result } = renderHook(() => useConsoleNavigation('acme', counts()));
-    expect(result.current.activeTab).toBe('workflow-blocker');
-  });
-
-  it('updates the default tab when counts arrive after the initial render', () => {
-    const { result, rerender } = renderHook(
-      ({ tabCounts }: { tabCounts: Record<ConsoleTabName, number> }) =>
-        useConsoleNavigation('acme', tabCounts),
-      { initialProps: { tabCounts: counts() } },
-    );
-    expect(result.current.activeTab).toBe('workflow-blocker');
-    rerender({ tabCounts: counts({ 'failed-preparation': 6 }) });
-    expect(result.current.activeTab).toBe('failed-preparation');
-  });
-
-  it('keeps the tab from the path even when counts are present', () => {
-    window.history.replaceState({}, '', '/projects/acme/todo-by-human?k=token');
-    const { result } = renderHook(() =>
-      useConsoleNavigation('acme', counts({ 'todo-by-human': 6 })),
-    );
-    expect(result.current.activeTab).toBe('todo-by-human');
-  });
-});