github-issue-tower-defence-management 1.89.0 → 1.91.0

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

@@ -0,0 +1,22 @@
+import { renderHook } from '@testing-library/react';
+import { useConsoleCaches } from './useConsoleCaches';
+
+describe('useConsoleCaches', () => {
+  beforeEach(() => {
+    localStorage.clear();
+    window.history.replaceState({}, '', '/?k=token');
+  });
+
+  it('exposes one cache per resource and a stable identity across renders', () => {
+    const { result, rerender } = renderHook(() => useConsoleCaches());
+    const first = result.current;
+    expect(first.body).toBeDefined();
+    expect(first.comments).toBeDefined();
+    expect(first.files).toBeDefined();
+    expect(first.commits).toBeDefined();
+    expect(first.relatedPrs).toBeDefined();
+    expect(first.state).toBeDefined();
+    rerender();
+    expect(result.current).toBe(first);
+  });
+});

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

@@ -0,0 +1,42 @@
+import { useMemo } from 'react';
+import {
+  type ConsoleApiClient,
+  createConsoleApiClient,
+} from '../lib/consoleApi';
+import { ResourceCache } from '../lib/resourceCache';
+import type {
+  ConsoleChangedFile,
+  ConsoleComment,
+  ConsoleCommit,
+  ConsoleIssueState,
+  ConsoleRelatedPullRequest,
+} from '../types';
+import { useConsoleToken } from './useConsoleToken';
+
+export type ConsoleCaches = {
+  body: ResourceCache<string>;
+  comments: ResourceCache<ConsoleComment[]>;
+  files: ResourceCache<ConsoleChangedFile[]>;
+  commits: ResourceCache<ConsoleCommit[]>;
+  relatedPrs: ResourceCache<ConsoleRelatedPullRequest[]>;
+  state: ResourceCache<ConsoleIssueState>;
+  client: ConsoleApiClient;
+};
+
+export const useConsoleCaches = (): ConsoleCaches => {
+  const { appendToken } = useConsoleToken();
+  return useMemo(() => {
+    const client = createConsoleApiClient(appendToken);
+    return {
+      client,
+      body: new ResourceCache<string>(client.fetchItemBody),
+      comments: new ResourceCache<ConsoleComment[]>(client.fetchComments),
+      files: new ResourceCache<ConsoleChangedFile[]>(client.fetchPrFiles),
+      commits: new ResourceCache<ConsoleCommit[]>(client.fetchPrCommits),
+      relatedPrs: new ResourceCache<ConsoleRelatedPullRequest[]>(
+        client.fetchRelatedPrs,
+      ),
+      state: new ResourceCache<ConsoleIssueState>(client.fetchIssueState),
+    };
+  }, [appendToken]);
+};

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

@@ -0,0 +1,126 @@
+import { renderHook, waitFor } from '@testing-library/react';
+import { ResourceCache } from '../lib/resourceCache';
+import type {
+  ConsoleChangedFile,
+  ConsoleComment,
+  ConsoleCommit,
+  ConsoleIssueState,
+  ConsoleListItem,
+  ConsoleRelatedPullRequest,
+} from '../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,
+  story: 'Story',
+  labels: [],
+  createdAt: '2026-06-10T00:00:00.000Z',
+};
+
+const issueItem: ConsoleListItem = {
+  ...prItem,
+  isPr: false,
+  url: 'https://github.com/o/r/issues/2',
+  number: 2,
+};
+
+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,
+      },
+    ],
+    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,
+    }),
+  };
+  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),
+  };
+};
+
+describe('useConsoleItemDetailData', () => {
+  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);
+    });
+  });
+
+  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,
+        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('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

@@ -0,0 +1,167 @@
+import { useEffect, useState } from 'react';
+import type { ConsoleRelatedPullRequestView } from '../components/ConsoleItemDetail';
+import type {
+  ConsoleChangedFile,
+  ConsoleComment,
+  ConsoleCommit,
+  ConsoleIssueState,
+  ConsoleListItem,
+  ConsoleRelatedPullRequest,
+} from '../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,
+};
+
+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;
+  relatedPullRequests: ConsoleRelatedPullRequestView[];
+};
+
+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 relatedPrs = useConsoleResource(
+    caches.relatedPrs,
+    !isPr ? key : null,
+    !isPr ? 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,
+    relatedPullRequests: relatedViews,
+  };
+};

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

@@ -0,0 +1,198 @@
+import { act, renderHook } from '@testing-library/react';
+import {
+  consoleListItemsFixture,
+  consoleStatusOptionsFixture,
+} from '../fixtures';
+import { overlayStorageKey } from '../overlay';
+import { useConsoleOperations } from './useConsoleOperations';
+import { useConsoleOverlay } from './useConsoleOverlay';
+
+const prItem = consoleListItemsFixture[0];
+const issueItem = consoleListItemsFixture[2];
+
+const captureFetch = (): jest.Mock => {
+  const fetchMock = jest.fn(async () => ({
+    ok: true,
+    status: 200,
+    json: async () => ({ ok: true }),
+  }));
+  global.fetch = fetchMock as unknown as typeof fetch;
+  return fetchMock;
+};
+
+const lastBody = (fetchMock: jest.Mock): Record<string, unknown> =>
+  JSON.parse((fetchMock.mock.calls.at(-1)?.[1] as { body: string }).body);
+
+const setup = () => {
+  localStorage.clear();
+  window.history.replaceState({}, '', '/projects/umino/prs?k=token');
+  return renderHook(() => {
+    const overlay = useConsoleOverlay('umino');
+    const operations = useConsoleOperations('umino', 'prs', overlay);
+    return { overlay, operations };
+  });
+};
+
+describe('useConsoleOperations', () => {
+  it('posts a totally wrong review as a close with the totally wrong comment body', async () => {
+    const fetchMock = captureFetch();
+    const { result } = setup();
+    await act(async () => {
+      await result.current.operations.reviewPullRequest(
+        prItem,
+        prItem.url,
+        'totally_wrong',
+      );
+    });
+    expect(fetchMock.mock.calls[0][0]).toBe('/api/review?k=token');
+    expect(lastBody(fetchMock)).toMatchObject({
+      pjcode: 'umino',
+      action: 'close',
+      prUrl: prItem.url,
+      commentBody: 'totally wrong',
+    });
+  });
+
+  it('posts an unnecessary review as a close with the unnecessary comment body', async () => {
+    const fetchMock = captureFetch();
+    const { result } = setup();
+    await act(async () => {
+      await result.current.operations.reviewPullRequest(
+        prItem,
+        prItem.url,
+        'unnecessary',
+      );
+    });
+    expect(lastBody(fetchMock)).toMatchObject({
+      action: 'close',
+      commentBody: 'This pull request is unnecessary.',
+    });
+  });
+
+  it('posts an approve review and marks the item done in the overlay', async () => {
+    captureFetch();
+    const { result } = setup();
+    await act(async () => {
+      await result.current.operations.reviewPullRequest(
+        prItem,
+        prItem.url,
+        'approve',
+      );
+    });
+    const stored = JSON.parse(
+      localStorage.getItem(overlayStorageKey('umino')) ?? '{}',
+    );
+    expect(stored[prItem.projectItemId].done).toBe(true);
+  });
+
+  it('posts a not-planned close through the triage endpoint', async () => {
+    const fetchMock = captureFetch();
+    const { result } = setup();
+    await act(async () => {
+      await result.current.operations.closeIssue(
+        issueItem,
+        'close_not_planned',
+      );
+    });
+    expect(fetchMock.mock.calls[0][0]).toBe('/api/triage?k=token');
+    expect(lastBody(fetchMock)).toMatchObject({
+      pjcode: 'umino',
+      action: 'close_not_planned',
+      issueUrl: issueItem.url,
+    });
+  });
+
+  it('posts set_status and records the overlay status', async () => {
+    const fetchMock = captureFetch();
+    const { result } = setup();
+    const option = consoleStatusOptionsFixture[1];
+    await act(async () => {
+      await result.current.operations.setStatus(issueItem, option);
+    });
+    expect(lastBody(fetchMock)).toMatchObject({
+      action: 'set_status',
+      statusName: option.name,
+    });
+    const stored = JSON.parse(
+      localStorage.getItem(overlayStorageKey('umino')) ?? '{}',
+    );
+    expect(stored[issueItem.projectItemId].status.name).toBe(option.name);
+  });
+
+  it('posts set_intmux through the intmux endpoint', async () => {
+    const fetchMock = captureFetch();
+    const { result } = setup();
+    const option = consoleStatusOptionsFixture[5];
+    await act(async () => {
+      await result.current.operations.setInTmuxByHuman(issueItem, option);
+    });
+    expect(fetchMock.mock.calls[0][0]).toBe('/api/intmux?k=token');
+    expect(lastBody(fetchMock)).toMatchObject({
+      pjcode: 'umino',
+      action: 'set_intmux',
+    });
+  });
+
+  it('does not mark done on snooze outside the todo-by-human tab', async () => {
+    captureFetch();
+    const { result } = setup();
+    await act(async () => {
+      await result.current.operations.setNextActionDate(
+        issueItem,
+        'snooze_1day',
+      );
+    });
+    const stored = JSON.parse(
+      localStorage.getItem(overlayStorageKey('umino')) ?? '{}',
+    );
+    expect(stored[issueItem.projectItemId]?.done).toBeUndefined();
+  });
+
+  it('marks done on snooze in the todo-by-human tab so the item is skipped', async () => {
+    captureFetch();
+    localStorage.clear();
+    window.history.replaceState(
+      {},
+      '',
+      '/projects/umino/todo-by-human?k=token',
+    );
+    const { result } = renderHook(() => {
+      const overlay = useConsoleOverlay('umino');
+      const operations = useConsoleOperations(
+        'umino',
+        'todo-by-human',
+        overlay,
+      );
+      return { overlay, operations };
+    });
+    await act(async () => {
+      await result.current.operations.setNextActionDate(
+        issueItem,
+        'snooze_1week',
+      );
+    });
+    const stored = JSON.parse(
+      localStorage.getItem(overlayStorageKey('umino')) ?? '{}',
+    );
+    expect(stored[issueItem.projectItemId].done).toBe(true);
+  });
+
+  it('rejects an operation and posts nothing when no pjcode is available', async () => {
+    const fetchMock = captureFetch();
+    localStorage.clear();
+    window.history.replaceState({}, '', '/?k=token');
+    const { result } = renderHook(() => {
+      const overlay = useConsoleOverlay('console');
+      const operations = useConsoleOperations(null, 'prs', overlay);
+      return { overlay, operations };
+    });
+    await expect(
+      result.current.operations.reviewPullRequest(
+        prItem,
+        prItem.url,
+        'approve',
+      ),
+    ).rejects.toThrow('No project specified in the URL path.');
+    expect(fetchMock).not.toHaveBeenCalled();
+  });
+});