github-issue-tower-defence-management 1.112.6 → 1.114.0

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsolePullRequestDetail.test.tsx

@@ -38,6 +38,52 @@ describe('ConsolePullRequestDetail', () => {
     expect(getByText('27 files')).toBeInTheDocument();
   });
 
+  it('renders the CI status badge derived from the pull request fields', () => {
+    const { getByText } = render(
+      <ConsolePullRequestDetail
+        pullRequest={pullRequest}
+        body={pullRequest.summary?.body ?? ''}
+        bodyIsLoading={false}
+        files={consoleChangedFilesFixture}
+        filesAreLoading={false}
+        filesError={null}
+        commits={consoleCommitsFixture}
+        commitsAreLoading={false}
+        commitsError={null}
+        now={now}
+      />,
+    );
+    expect(getByText('CI passing')).toBeInTheDocument();
+  });
+
+  it('renders failing CI, conflict and out-of-date badges for an unhealthy pull request', () => {
+    const { getByText } = render(
+      <ConsolePullRequestDetail
+        pullRequest={{
+          ...pullRequest,
+          isConflicted: true,
+          isPassedAllCiJob: false,
+          isCiStateSuccess: false,
+          isBranchOutOfDate: true,
+          missingRequiredCheckNames: ['build'],
+        }}
+        body={pullRequest.summary?.body ?? ''}
+        bodyIsLoading={false}
+        files={consoleChangedFilesFixture}
+        filesAreLoading={false}
+        filesError={null}
+        commits={consoleCommitsFixture}
+        commitsAreLoading={false}
+        commitsError={null}
+        now={now}
+      />,
+    );
+    expect(getByText('CI failing')).toBeInTheDocument();
+    expect(getByText(/missing: build/)).toBeInTheDocument();
+    expect(getByText('Conflict')).toBeInTheDocument();
+    expect(getByText('Out of date')).toBeInTheDocument();
+  });
+
   it('renders a copy URL button for the pull request url', () => {
     const { getByRole } = render(
       <ConsolePullRequestDetail

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsolePullRequestDetail.tsx

@@ -4,12 +4,14 @@ import type {
   ConsoleCommit,
   ConsoleRelatedPullRequest,
 } from '../../logic/types';
+import type { ConsoleReferenceLinkRenderer } from '../content/ConsoleMarkdownContent';
 import { ConsoleMarkdownContent } from '../content/ConsoleMarkdownContent';
 import { ConsolePanel } from '../layout/ConsolePanel';
 import { ConsoleChangedFileList } from './ConsoleChangedFileList';
 import { ConsoleCommitList } from './ConsoleCommitList';
 import { ConsoleCopyUrlButton } from './ConsoleCopyUrlButton';
 import type { ConsoleAddInlineComment } from './ConsoleFileDiff';
+import { ConsolePullRequestStatusBadges } from './ConsolePullRequestStatusBadges';
 
 export type ConsolePullRequestSectionProps = {
   pullRequest: ConsoleRelatedPullRequest;
@@ -23,6 +25,7 @@ export type ConsolePullRequestSectionProps = {
   commitsError: string | null;
   now: number;
   buildImageProxyUrl?: ImageProxyUrlBuilder;
+  renderReferenceLink?: ConsoleReferenceLinkRenderer;
   onAddInlineComment?: ConsoleAddInlineComment;
 };
 
@@ -38,6 +41,7 @@ export const ConsolePullRequestDetail = ({
   commitsError,
   now,
   buildImageProxyUrl,
+  renderReferenceLink,
   onAddInlineComment,
 }: ConsolePullRequestSectionProps) => {
   const summary = pullRequest.summary;
@@ -59,6 +63,13 @@ export const ConsolePullRequestDetail = ({
         {pullRequest.isDraft && (
           <span className="console-pr-section-state">draft</span>
         )}
+        <ConsolePullRequestStatusBadges
+          isConflicted={pullRequest.isConflicted}
+          isPassedAllCiJob={pullRequest.isPassedAllCiJob}
+          isCiStateSuccess={pullRequest.isCiStateSuccess}
+          isBranchOutOfDate={pullRequest.isBranchOutOfDate}
+          missingRequiredCheckNames={pullRequest.missingRequiredCheckNames}
+        />
         <ConsoleCopyUrlButton url={pullRequest.url} label="Copy PR URL" />
         <div className="console-pr-statbar">
           {pullRequest.branchName !== null && (
@@ -82,6 +93,7 @@ export const ConsolePullRequestDetail = ({
           <ConsoleMarkdownContent
             body={summary?.body ?? body}
             buildImageProxyUrl={buildImageProxyUrl}
+            renderReferenceLink={renderReferenceLink}
           />
         )}
       </ConsolePanel>

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsolePullRequestStatusBadges.stories.tsx

@@ -0,0 +1,41 @@
+import type { Meta, StoryObj } from '@storybook/react-vite';
+import { ConsolePullRequestStatusBadges } from './ConsolePullRequestStatusBadges';
+
+const meta: Meta<typeof ConsolePullRequestStatusBadges> = {
+  title: 'Console/ConsolePullRequestStatusBadges',
+  component: ConsolePullRequestStatusBadges,
+};
+
+export default meta;
+
+type Story = StoryObj<typeof ConsolePullRequestStatusBadges>;
+
+export const Passing: Story = {
+  args: {
+    isConflicted: false,
+    isPassedAllCiJob: true,
+    isCiStateSuccess: true,
+    isBranchOutOfDate: false,
+    missingRequiredCheckNames: [],
+  },
+};
+
+export const FailingWithMissingChecks: Story = {
+  args: {
+    isConflicted: false,
+    isPassedAllCiJob: false,
+    isCiStateSuccess: false,
+    isBranchOutOfDate: false,
+    missingRequiredCheckNames: ['build', 'test'],
+  },
+};
+
+export const ConflictedAndOutOfDate: Story = {
+  args: {
+    isConflicted: true,
+    isPassedAllCiJob: false,
+    isCiStateSuccess: false,
+    isBranchOutOfDate: true,
+    missingRequiredCheckNames: ['build', 'test'],
+  },
+};

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsolePullRequestStatusBadges.test.tsx

@@ -0,0 +1,72 @@
+import { render } from '@testing-library/react';
+import { ConsolePullRequestStatusBadges } from './ConsolePullRequestStatusBadges';
+
+describe('ConsolePullRequestStatusBadges', () => {
+  it('renders a passing CI badge when all CI jobs passed and the CI state is success', () => {
+    const { getByText, queryByText } = render(
+      <ConsolePullRequestStatusBadges
+        isConflicted={false}
+        isPassedAllCiJob={true}
+        isCiStateSuccess={true}
+        isBranchOutOfDate={false}
+        missingRequiredCheckNames={[]}
+      />,
+    );
+    expect(getByText('CI passing')).toBeInTheDocument();
+    expect(queryByText('Conflict')).toBeNull();
+    expect(queryByText('Out of date')).toBeNull();
+  });
+
+  it('renders a failing CI badge with missing required checks', () => {
+    const { getByText } = render(
+      <ConsolePullRequestStatusBadges
+        isConflicted={false}
+        isPassedAllCiJob={false}
+        isCiStateSuccess={true}
+        isBranchOutOfDate={false}
+        missingRequiredCheckNames={['build', 'test']}
+      />,
+    );
+    expect(getByText('CI failing')).toBeInTheDocument();
+    expect(getByText(/missing: build, test/)).toBeInTheDocument();
+  });
+
+  it('renders the conflict badge when the pull request is conflicted', () => {
+    const { getByText } = render(
+      <ConsolePullRequestStatusBadges
+        isConflicted={true}
+        isPassedAllCiJob={true}
+        isCiStateSuccess={true}
+        isBranchOutOfDate={false}
+        missingRequiredCheckNames={[]}
+      />,
+    );
+    expect(getByText('Conflict')).toBeInTheDocument();
+  });
+
+  it('renders the out-of-date badge when the branch is behind the base branch', () => {
+    const { getByText } = render(
+      <ConsolePullRequestStatusBadges
+        isConflicted={false}
+        isPassedAllCiJob={true}
+        isCiStateSuccess={true}
+        isBranchOutOfDate={true}
+        missingRequiredCheckNames={[]}
+      />,
+    );
+    expect(getByText('Out of date')).toBeInTheDocument();
+  });
+
+  it('treats a successful CI state but unfinished jobs as failing', () => {
+    const { getByText } = render(
+      <ConsolePullRequestStatusBadges
+        isConflicted={false}
+        isPassedAllCiJob={false}
+        isCiStateSuccess={true}
+        isBranchOutOfDate={false}
+        missingRequiredCheckNames={[]}
+      />,
+    );
+    expect(getByText('CI failing')).toBeInTheDocument();
+  });
+});

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsolePullRequestStatusBadges.tsx

@@ -0,0 +1,71 @@
+import { colorFromEnum } from '../../logic/colors';
+import type { ConsoleColor } from '../../logic/types';
+
+export type ConsolePullRequestStatusBadgesProps = {
+  isConflicted: boolean;
+  isPassedAllCiJob: boolean;
+  isCiStateSuccess: boolean;
+  isBranchOutOfDate: boolean;
+  missingRequiredCheckNames: string[];
+};
+
+const badgeStyle = (color: ConsoleColor) => {
+  const palette = colorFromEnum(color);
+  return {
+    color: palette.fg,
+    borderColor: palette.border,
+    backgroundColor: palette.bg,
+  };
+};
+
+export const ConsolePullRequestStatusBadges = ({
+  isConflicted,
+  isPassedAllCiJob,
+  isCiStateSuccess,
+  isBranchOutOfDate,
+  missingRequiredCheckNames,
+}: ConsolePullRequestStatusBadgesProps) => {
+  const ciPassing = isPassedAllCiJob && isCiStateSuccess;
+  const ciColor: ConsoleColor = ciPassing ? 'GREEN' : 'RED';
+  const ciLabel = ciPassing ? 'CI passing' : 'CI failing';
+  const missingChecksLabel =
+    !ciPassing && missingRequiredCheckNames.length > 0
+      ? `missing: ${missingRequiredCheckNames.join(', ')}`
+      : null;
+
+  return (
+    <span className="console-detail-pr-status">
+      <span
+        className="console-detail-status-chip console-detail-ci-chip"
+        style={badgeStyle(ciColor)}
+        title={missingChecksLabel ?? ciLabel}
+      >
+        {ciLabel}
+        {missingChecksLabel !== null && (
+          <span className="console-detail-ci-missing">
+            {' '}
+            ({missingChecksLabel})
+          </span>
+        )}
+      </span>
+      {isConflicted && (
+        <span
+          className="console-detail-status-chip console-detail-conflict-chip"
+          style={badgeStyle('RED')}
+          title="This pull request has merge conflicts"
+        >
+          Conflict
+        </span>
+      )}
+      {isBranchOutOfDate && (
+        <span
+          className="console-detail-status-chip console-detail-outofdate-chip"
+          style={badgeStyle('YELLOW')}
+          title="This branch is out of date with the base branch"
+        >
+          Out of date
+        </span>
+      )}
+    </span>
+  );
+};

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

@@ -9,6 +9,7 @@ import type {
   ConsoleComment,
   ConsoleCommit,
   ConsoleIssueState,
+  ConsolePullRequestStatus,
   ConsoleRelatedPullRequest,
 } from '../logic/types';
 import { useConsoleToken } from './useConsoleToken';
@@ -20,6 +21,7 @@ export type ConsoleCaches = {
   commits: ResourceCache<ConsoleCommit[]>;
   relatedPrs: ResourceCache<ConsoleRelatedPullRequest[]>;
   state: ResourceCache<ConsoleIssueState>;
+  prStatus: ResourceCache<ConsolePullRequestStatus>;
   client: ConsoleApiClient;
 };
 
@@ -37,6 +39,9 @@ export const useConsoleCaches = (): ConsoleCaches => {
         client.fetchRelatedPrs,
       ),
       state: new ResourceCache<ConsoleIssueState>(client.fetchIssueState),
+      prStatus: new ResourceCache<ConsolePullRequestStatus>(
+        client.fetchPullRequestStatus,
+      ),
     };
   }, [appendToken]);
 };

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

@@ -6,6 +6,7 @@ import type {
   ConsoleCommit,
   ConsoleIssueState,
   ConsoleListItem,
+  ConsolePullRequestStatus,
   ConsoleRelatedPullRequest,
 } from '../logic/types';
 import type { ConsoleCaches } from './useConsoleCaches';
@@ -62,6 +63,15 @@ const buildCaches = (related: ConsoleRelatedPullRequest[]): ConsoleCaches => {
       state: 'open',
       merged: false,
       isPullRequest: true,
+      title: 'Console item detail fixture title',
+    }),
+    fetchPullRequestStatus: async (): Promise<ConsolePullRequestStatus> => ({
+      found: true,
+      isConflicted: true,
+      isPassedAllCiJob: false,
+      isCiStateSuccess: false,
+      isBranchOutOfDate: true,
+      missingRequiredCheckNames: ['build'],
     }),
   };
   return {
@@ -72,6 +82,7 @@ const buildCaches = (related: ConsoleRelatedPullRequest[]): ConsoleCaches => {
     commits: new ResourceCache(client.fetchPrCommits),
     relatedPrs: new ResourceCache(client.fetchRelatedPrs),
     state: new ResourceCache(client.fetchIssueState),
+    prStatus: new ResourceCache(client.fetchPullRequestStatus),
   };
 };
 
@@ -85,9 +96,19 @@ describe('useConsoleItemDetailData', () => {
       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[] = [
       {

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

@@ -6,6 +6,7 @@ import type {
   ConsoleCommit,
   ConsoleIssueState,
   ConsoleListItem,
+  ConsolePullRequestStatus,
   ConsoleRelatedPullRequest,
 } from '../logic/types';
 import type { ConsoleCaches } from './useConsoleCaches';
@@ -19,6 +20,15 @@ const DEFAULT_STATE: ConsoleIssueState = {
   state: 'open',
   merged: false,
   isPullRequest: false,
+  title: '',
+};
+const DEFAULT_PR_STATUS: ConsolePullRequestStatus = {
+  found: false,
+  isConflicted: false,
+  isPassedAllCiJob: false,
+  isCiStateSuccess: false,
+  isBranchOutOfDate: false,
+  missingRequiredCheckNames: [],
 };
 
 export type ConsoleItemDetailData = {
@@ -35,6 +45,7 @@ export type ConsoleItemDetailData = {
   commits: ConsoleCommit[];
   commitsAreLoading: boolean;
   commitsError: string | null;
+  pullRequestStatus: ConsolePullRequestStatus | null;
   relatedPullRequests: ConsoleRelatedPullRequestView[];
 };
 
@@ -66,6 +77,12 @@ export const useConsoleItemDetailData = (
     isPr ? url : null,
     EMPTY_COMMITS,
   );
+  const prStatus = useConsoleResource(
+    caches.prStatus,
+    isPr ? key : null,
+    isPr ? url : null,
+    DEFAULT_PR_STATUS,
+  );
   const relatedPrs = useConsoleResource(
     caches.relatedPrs,
     !isPr ? key : null,
@@ -162,6 +179,7 @@ export const useConsoleItemDetailData = (
     commits: commits.data,
     commitsAreLoading: commits.isLoading,
     commitsError: commits.error,
+    pullRequestStatus: isPr ? prStatus.data : null,
     relatedPullRequests: relatedViews,
   };
 };

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

@@ -0,0 +1,48 @@
+import { renderHook, waitFor } from '@testing-library/react';
+import { ResourceCache } from '../lib/resourceCache';
+import type { ConsoleIssueState } from '../logic/types';
+import { useConsoleReferenceLink } from './useConsoleReferenceLink';
+
+const buildState = (title: string): ConsoleIssueState => ({
+  state: 'open',
+  merged: false,
+  isPullRequest: true,
+  title,
+});
+
+describe('useConsoleReferenceLink', () => {
+  const url = 'https://github.com/octo/repo/pull/7';
+
+  it('returns null then the resolved state', async () => {
+    const cache = new ResourceCache<ConsoleIssueState>(async () =>
+      buildState('Decorate links'),
+    );
+    const { result } = renderHook(() => useConsoleReferenceLink(cache, url));
+    expect(result.current).toBeNull();
+    await waitFor(() => {
+      expect(result.current?.title).toBe('Decorate links');
+    });
+  });
+
+  it('returns null when the fetch fails', async () => {
+    const cache = new ResourceCache<ConsoleIssueState>(async () => {
+      throw new Error('boom');
+    });
+    const { result } = renderHook(() => useConsoleReferenceLink(cache, url));
+    await waitFor(() => {
+      expect(cache.has(url)).toBe(false);
+    });
+    expect(result.current).toBeNull();
+  });
+
+  it('reuses an already cached value without refetching', () => {
+    const fetcher = jest.fn(async () => buildState('cached'));
+    const cache = new ResourceCache<ConsoleIssueState>(fetcher);
+    cache.load(url, url);
+    return cache.load(url, url).then(() => {
+      const { result } = renderHook(() => useConsoleReferenceLink(cache, url));
+      expect(result.current?.title).toBe('cached');
+      expect(fetcher).toHaveBeenCalledTimes(1);
+    });
+  });
+});

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

@@ -0,0 +1,39 @@
+import { useEffect, useState } from 'react';
+import type { ResourceCache } from '../lib/resourceCache';
+import type { ConsoleIssueState } from '../logic/types';
+
+export const useConsoleReferenceLink = (
+  cache: ResourceCache<ConsoleIssueState>,
+  url: string,
+): ConsoleIssueState | null => {
+  const [state, setState] = useState<ConsoleIssueState | null>(
+    () => cache.peek(url) ?? null,
+  );
+
+  useEffect(() => {
+    const cached = cache.peek(url);
+    if (cached !== undefined) {
+      setState(cached);
+      return;
+    }
+    let cancelled = false;
+    setState(null);
+    cache
+      .load(url, url)
+      .then((value) => {
+        if (!cancelled) {
+          setState(value);
+        }
+      })
+      .catch(() => {
+        if (!cancelled) {
+          setState(null);
+        }
+      });
+    return () => {
+      cancelled = true;
+    };
+  }, [cache, url]);
+
+  return state;
+};

package/src/adapter/entry-points/console/ui/src/features/console/lib/consoleApi.test.ts

@@ -48,6 +48,7 @@ describe('createConsoleApiClient', () => {
     readers.push((url) => client.fetchPrCommits(url));
     readers.push((url) => client.fetchRelatedPrs(url));
     readers.push((url) => client.fetchIssueState(url));
+    readers.push((url) => client.fetchPullRequestStatus(url));
     const expectedPaths = [
       '/api/itembody',
       '/api/comments',
@@ -55,6 +56,7 @@ describe('createConsoleApiClient', () => {
       '/api/prcommits',
       '/api/relatedprs',
       '/api/issuetitle',
+      '/api/pullrequeststatus',
     ];
     for (let index = 0; index < readers.length; index += 1) {
       const fetchMock = mockFetchOnce({});
@@ -103,11 +105,34 @@ describe('createConsoleApiClient', () => {
   });
 
   it('parses the issue state', async () => {
-    mockFetchOnce({ state: 'closed', merged: true, isPullRequest: true });
+    mockFetchOnce({
+      state: 'closed',
+      merged: true,
+      isPullRequest: true,
+      title: 'Decorate PR links',
+    });
     const client = createConsoleApiClient(appendToken);
     expect(
       await client.fetchIssueState('https://github.com/o/r/pull/1'),
-    ).toEqual({ state: 'closed', merged: true, isPullRequest: true });
+    ).toEqual({
+      state: 'closed',
+      merged: true,
+      isPullRequest: true,
+      title: 'Decorate PR links',
+    });
+  });
+
+  it('defaults the title to an empty string when absent', async () => {
+    mockFetchOnce({ state: 'open', merged: false, isPullRequest: false });
+    const client = createConsoleApiClient(appendToken);
+    expect(
+      await client.fetchIssueState('https://github.com/o/r/issues/1'),
+    ).toEqual({
+      state: 'open',
+      merged: false,
+      isPullRequest: false,
+      title: '',
+    });
   });
 
   it('parses pull request commits', async () => {
@@ -161,6 +186,45 @@ describe('createConsoleApiClient', () => {
     expect(related[0].missingRequiredCheckNames).toEqual(['build']);
   });
 
+  it('parses the pull request status when found', async () => {
+    mockFetchOnce({
+      found: true,
+      status: {
+        isConflicted: true,
+        isPassedAllCiJob: false,
+        isCiStateSuccess: false,
+        isBranchOutOfDate: true,
+        missingRequiredCheckNames: ['build', 'test'],
+      },
+    });
+    const client = createConsoleApiClient(appendToken);
+    expect(
+      await client.fetchPullRequestStatus('https://github.com/o/r/pull/1'),
+    ).toEqual({
+      found: true,
+      isConflicted: true,
+      isPassedAllCiJob: false,
+      isCiStateSuccess: false,
+      isBranchOutOfDate: true,
+      missingRequiredCheckNames: ['build', 'test'],
+    });
+  });
+
+  it('returns a not-found pull request status when the server reports none', async () => {
+    mockFetchOnce({ found: false, status: null });
+    const client = createConsoleApiClient(appendToken);
+    expect(
+      await client.fetchPullRequestStatus('https://github.com/o/r/pull/1'),
+    ).toEqual({
+      found: false,
+      isConflicted: false,
+      isPassedAllCiJob: false,
+      isCiStateSuccess: false,
+      isBranchOutOfDate: false,
+      missingRequiredCheckNames: [],
+    });
+  });
+
   it('throws on a non-ok response', async () => {
     mockFetchOnce({}, false);
     const client = createConsoleApiClient(appendToken);