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

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

@@ -1,235 +0,0 @@
-import { render, waitFor } from '@testing-library/react';
-import { parseGitHubReferenceUrl } from '../../logic/references';
-import {
-  consoleChangedFilesFixture,
-  consoleCommitsFixture,
-  consoleRelatedPullRequestsFixture,
-} from '../../testing/fixtures';
-import { ConsolePullRequestDetail } from './ConsolePullRequestDetail';
-
-jest.mock('../../lib/mermaidLoader', () => ({
-  renderMermaidToSvg: jest.fn(async () => '<svg></svg>'),
-}));
-
-const now = Date.parse('2026-06-19T12:00:00.000Z');
-const pullRequest = consoleRelatedPullRequestsFixture[0];
-const pullRequestReference = parseGitHubReferenceUrl(pullRequest.url);
-
-if (pullRequestReference === null) {
-  throw new Error('the fixture pull request url must be a GitHub url');
-}
-
-describe('ConsolePullRequestDetail', () => {
-  it('renders the title, stat bar and changed files', () => {
-    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(
-        'Scaffold React console UI under entry-points with build bundling',
-      ),
-    ).toBeInTheDocument();
-    expect(getByText('+1184')).toBeInTheDocument();
-    expect(getByText('27 files')).toBeInTheDocument();
-    expect(
-      getByText(`${pullRequestReference.owner}/${pullRequestReference.repo}`),
-    ).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,
-          mergeableStatus: 'CONFLICTING',
-          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('opens the description panel when the view is first rendered', () => {
-    const { getByRole } = 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(getByRole('button', { name: /^▾ Description$/ })).toHaveAttribute(
-      'aria-expanded',
-      'true',
-    );
-  });
-
-  it('renders an issue reference in the description against the repository owning the pull request', async () => {
-    const referencedIssueNumber = 1234;
-    const renderReferenceLink = jest.fn((href: string) => (
-      <span className="decorated-reference" data-href={href}>
-        decorated
-      </span>
-    ));
-    const { container } = render(
-      <ConsolePullRequestDetail
-        pullRequest={{
-          ...pullRequest,
-          summary:
-            pullRequest.summary === null
-              ? null
-              : {
-                  ...pullRequest.summary,
-                  body: `- close #${referencedIssueNumber}`,
-                },
-        }}
-        body={`- close #${referencedIssueNumber}`}
-        bodyIsLoading={false}
-        files={consoleChangedFilesFixture}
-        filesAreLoading={false}
-        filesError={null}
-        commits={consoleCommitsFixture}
-        commitsAreLoading={false}
-        commitsError={null}
-        now={now}
-        renderReferenceLink={renderReferenceLink}
-      />,
-    );
-    await waitFor(() => {
-      expect(container.querySelectorAll('.decorated-reference').length).toBe(1);
-    });
-    expect(
-      container
-        .querySelector('.decorated-reference')
-        ?.getAttribute('data-href'),
-    ).toBe(
-      `https://github.com/${pullRequestReference.owner}/${pullRequestReference.repo}/issues/${referencedIssueNumber}`,
-    );
-  });
-
-  it('leaves an issue reference untouched when the pull request url names no repository', async () => {
-    const renderReferenceLink = jest.fn((href: string) => (
-      <span className="decorated-reference" data-href={href}>
-        decorated
-      </span>
-    ));
-    const { container } = render(
-      <ConsolePullRequestDetail
-        pullRequest={{
-          ...pullRequest,
-          url: 'https://example.com/not-a-github-pull-request',
-          summary:
-            pullRequest.summary === null
-              ? null
-              : { ...pullRequest.summary, body: '- close #1234' },
-        }}
-        body={'- close #1234'}
-        bodyIsLoading={false}
-        files={consoleChangedFilesFixture}
-        filesAreLoading={false}
-        filesError={null}
-        commits={consoleCommitsFixture}
-        commitsAreLoading={false}
-        commitsError={null}
-        now={now}
-        renderReferenceLink={renderReferenceLink}
-      />,
-    );
-    await waitFor(() => {
-      expect(container.querySelector('.console-markdown')).not.toBeNull();
-    });
-    expect(container.querySelector('.decorated-reference')).toBeNull();
-    expect(renderReferenceLink).not.toHaveBeenCalled();
-  });
-
-  it('renders a copy URL button for the pull request url', () => {
-    const { getByRole } = 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(getByRole('button', { name: 'Copy PR URL' })).toBeInTheDocument();
-  });
-
-  it('offers a link painted as a link that opens the pull request', () => {
-    const { getByRole } = render(
-      <ConsolePullRequestDetail
-        pullRequest={pullRequest}
-        body={pullRequest.summary?.body ?? ''}
-        bodyIsLoading={false}
-        files={consoleChangedFilesFixture}
-        filesAreLoading={false}
-        filesError={null}
-        commits={consoleCommitsFixture}
-        commitsAreLoading={false}
-        commitsError={null}
-        now={now}
-      />,
-    );
-
-    const openLink = getByRole('link', { name: 'open' });
-    expect(openLink).toHaveAttribute('href', pullRequest.url);
-    expect(openLink).toHaveAttribute('target', '_blank');
-    expect(openLink.className).toContain('console-panel-open-link');
-  });
-});

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

@@ -1,145 +0,0 @@
-import type { ImageProxyUrlBuilder } from '../../lib/imageProxy';
-import { parseGitHubReferenceUrl } from '../../logic/references';
-import type {
-  ConsoleChangedFile,
-  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 { ConsolePullRequestMergeableChip } from './ConsolePullRequestMergeableChip';
-import { ConsolePullRequestStatusBadges } from './ConsolePullRequestStatusBadges';
-
-export type ConsolePullRequestSectionProps = {
-  pullRequest: ConsoleRelatedPullRequest;
-  body: string;
-  bodyIsLoading: boolean;
-  files: ConsoleChangedFile[];
-  filesAreLoading: boolean;
-  filesError: string | null;
-  commits: ConsoleCommit[];
-  commitsAreLoading: boolean;
-  commitsError: string | null;
-  now: number;
-  buildImageProxyUrl?: ImageProxyUrlBuilder;
-  renderReferenceLink?: ConsoleReferenceLinkRenderer;
-  onAddInlineComment?: ConsoleAddInlineComment;
-};
-
-export const ConsolePullRequestDetail = ({
-  pullRequest,
-  body,
-  bodyIsLoading,
-  files,
-  filesAreLoading,
-  filesError,
-  commits,
-  commitsAreLoading,
-  commitsError,
-  now,
-  buildImageProxyUrl,
-  renderReferenceLink,
-  onAddInlineComment,
-}: ConsolePullRequestSectionProps) => {
-  const summary = pullRequest.summary;
-  const pullRequestReference = parseGitHubReferenceUrl(pullRequest.url);
-  const repoContext =
-    pullRequestReference === null
-      ? undefined
-      : {
-          owner: pullRequestReference.owner,
-          repo: pullRequestReference.repo,
-        };
-  const filesCount =
-    filesAreLoading || filesError !== null ? null : files.length;
-  const commitsCount =
-    commitsAreLoading || commitsError !== null ? null : commits.length;
-  return (
-    <>
-      <div className="console-pr-header">
-        <a
-          href={pullRequest.url}
-          className="console-pr-section-title"
-          target="_blank"
-          rel="noopener noreferrer"
-        >
-          {summary?.title ?? pullRequest.url}
-        </a>
-        {pullRequest.isDraft && (
-          <span className="console-pr-section-state">draft</span>
-        )}
-        <ConsolePullRequestMergeableChip
-          mergeableStatus={pullRequest.mergeableStatus}
-        />
-        <ConsolePullRequestStatusBadges
-          isPassedAllCiJob={pullRequest.isPassedAllCiJob}
-          isCiStateSuccess={pullRequest.isCiStateSuccess}
-          isBranchOutOfDate={pullRequest.isBranchOutOfDate}
-          missingRequiredCheckNames={pullRequest.missingRequiredCheckNames}
-        />
-        <a
-          href={pullRequest.url}
-          className="console-panel-open-link"
-          target="_blank"
-          rel="noopener noreferrer"
-        >
-          open
-        </a>
-        <ConsoleCopyUrlButton url={pullRequest.url} label="Copy PR URL" />
-        <div className="console-pr-statbar">
-          {pullRequestReference !== null && (
-            <span className="console-pr-repo">
-              {pullRequestReference.owner}/{pullRequestReference.repo}
-            </span>
-          )}
-          {pullRequest.branchName !== null && (
-            <span className="console-pr-branch">{pullRequest.branchName}</span>
-          )}
-          {summary !== null && (
-            <>
-              <span className="console-pr-add">+{summary.additions}</span>
-              <span className="console-pr-del">-{summary.deletions}</span>
-              <span className="console-pr-files-count">
-                {summary.changedFiles} files
-              </span>
-            </>
-          )}
-        </div>
-      </div>
-      <ConsolePanel title="Description">
-        {bodyIsLoading ? (
-          <p className="console-pr-body-loading">Loading description...</p>
-        ) : (
-          <ConsoleMarkdownContent
-            body={summary?.body ?? body}
-            buildImageProxyUrl={buildImageProxyUrl}
-            renderReferenceLink={renderReferenceLink}
-            repoContext={repoContext}
-          />
-        )}
-      </ConsolePanel>
-      <ConsolePanel title="Changed files" count={filesCount}>
-        <ConsoleChangedFileList
-          files={files}
-          isLoading={filesAreLoading}
-          error={filesError}
-          buildImageProxyUrl={buildImageProxyUrl}
-          onAddInlineComment={onAddInlineComment}
-        />
-      </ConsolePanel>
-      <ConsolePanel title="Commits" count={commitsCount} defaultCollapsed>
-        <ConsoleCommitList
-          commits={commits}
-          isLoading={commitsAreLoading}
-          error={commitsError}
-          now={now}
-        />
-      </ConsolePanel>
-    </>
-  );
-};

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

@@ -1,29 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react-vite';
-import { ConsolePullRequestMergeableChip } from './ConsolePullRequestMergeableChip';
-
-const meta: Meta<typeof ConsolePullRequestMergeableChip> = {
-  title: 'Console/ConsolePullRequestMergeableChip',
-  component: ConsolePullRequestMergeableChip,
-};
-
-export default meta;
-
-type Story = StoryObj<typeof ConsolePullRequestMergeableChip>;
-
-export const Mergeable: Story = {
-  args: {
-    mergeableStatus: 'MERGEABLE',
-  },
-};
-
-export const Conflicting: Story = {
-  args: {
-    mergeableStatus: 'CONFLICTING',
-  },
-};
-
-export const CheckingMergeStatus: Story = {
-  args: {
-    mergeableStatus: 'UNKNOWN',
-  },
-};

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

@@ -1,30 +0,0 @@
-import { render } from '@testing-library/react';
-import { ConsolePullRequestMergeableChip } from './ConsolePullRequestMergeableChip';
-
-describe('ConsolePullRequestMergeableChip', () => {
-  it('renders the green no-conflict chip when the pull request is mergeable', () => {
-    const { getByText, queryByText } = render(
-      <ConsolePullRequestMergeableChip mergeableStatus="MERGEABLE" />,
-    );
-    expect(getByText('No conflict')).toBeInTheDocument();
-    expect(queryByText('Conflict')).toBeNull();
-    expect(queryByText('Checking merge status')).toBeNull();
-  });
-
-  it('renders the red conflict chip when the pull request is conflicting', () => {
-    const { getByText, queryByText } = render(
-      <ConsolePullRequestMergeableChip mergeableStatus="CONFLICTING" />,
-    );
-    expect(getByText('Conflict')).toBeInTheDocument();
-    expect(queryByText('No conflict')).toBeNull();
-  });
-
-  it('renders the gray checking chip when the merge status is unknown', () => {
-    const { getByText, queryByText } = render(
-      <ConsolePullRequestMergeableChip mergeableStatus="UNKNOWN" />,
-    );
-    expect(getByText('Checking merge status')).toBeInTheDocument();
-    expect(queryByText('Conflict')).toBeNull();
-    expect(queryByText('No conflict')).toBeNull();
-  });
-});

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

@@ -1,50 +0,0 @@
-import { colorFromEnum } from '../../logic/colors';
-import type { ConsoleColor, ConsoleMergeableStatus } from '../../logic/types';
-
-export type ConsolePullRequestMergeableChipProps = {
-  mergeableStatus: ConsoleMergeableStatus;
-};
-
-const MERGEABLE_BADGE: Record<
-  ConsoleMergeableStatus,
-  { color: ConsoleColor; label: string; title: string; modifier: string }
-> = {
-  MERGEABLE: {
-    color: 'GREEN',
-    label: 'No conflict',
-    title: 'This pull request has no merge conflicts',
-    modifier: 'console-detail-mergeable-chip-ok',
-  },
-  CONFLICTING: {
-    color: 'RED',
-    label: 'Conflict',
-    title: 'This pull request has merge conflicts',
-    modifier: 'console-detail-mergeable-chip-conflict',
-  },
-  UNKNOWN: {
-    color: 'GRAY',
-    label: 'Checking merge status',
-    title: 'GitHub has not finished computing the merge status yet',
-    modifier: 'console-detail-mergeable-chip-unknown',
-  },
-};
-
-export const ConsolePullRequestMergeableChip = ({
-  mergeableStatus,
-}: ConsolePullRequestMergeableChipProps) => {
-  const badge = MERGEABLE_BADGE[mergeableStatus];
-  const palette = colorFromEnum(badge.color);
-  return (
-    <span
-      className={`console-detail-status-chip console-detail-mergeable-chip ${badge.modifier}`}
-      style={{
-        color: palette.fg,
-        borderColor: palette.border,
-        backgroundColor: palette.bg,
-      }}
-      title={badge.title}
-    >
-      {badge.label}
-    </span>
-  );
-};

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

@@ -1,38 +0,0 @@
-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: {
-    isPassedAllCiJob: true,
-    isCiStateSuccess: true,
-    isBranchOutOfDate: false,
-    missingRequiredCheckNames: [],
-  },
-};
-
-export const FailingWithMissingChecks: Story = {
-  args: {
-    isPassedAllCiJob: false,
-    isCiStateSuccess: false,
-    isBranchOutOfDate: false,
-    missingRequiredCheckNames: ['build', 'test'],
-  },
-};
-
-export const FailingAndOutOfDate: Story = {
-  args: {
-    isPassedAllCiJob: false,
-    isCiStateSuccess: false,
-    isBranchOutOfDate: true,
-    missingRequiredCheckNames: ['build', 'test'],
-  },
-};

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

@@ -1,54 +0,0 @@
-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
-        isPassedAllCiJob={true}
-        isCiStateSuccess={true}
-        isBranchOutOfDate={false}
-        missingRequiredCheckNames={[]}
-      />,
-    );
-    expect(getByText('CI passing')).toBeInTheDocument();
-    expect(queryByText('Out of date')).toBeNull();
-  });
-
-  it('renders a failing CI badge with missing required checks', () => {
-    const { getByText } = render(
-      <ConsolePullRequestStatusBadges
-        isPassedAllCiJob={false}
-        isCiStateSuccess={true}
-        isBranchOutOfDate={false}
-        missingRequiredCheckNames={['build', 'test']}
-      />,
-    );
-    expect(getByText('CI failing')).toBeInTheDocument();
-    expect(getByText(/missing: build, test/)).toBeInTheDocument();
-  });
-
-  it('renders the out-of-date badge when the branch is behind the base branch', () => {
-    const { getByText } = render(
-      <ConsolePullRequestStatusBadges
-        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
-        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

@@ -1,60 +0,0 @@
-import { colorFromEnum } from '../../logic/colors';
-import type { ConsoleColor } from '../../logic/types';
-
-export type ConsolePullRequestStatusBadgesProps = {
-  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 = ({
-  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>
-      {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>
-  );
-};