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

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsolePullRequestReviewActions.test.tsx

@@ -1,93 +0,0 @@
-import { fireEvent, render } from '@testing-library/react';
-import { ConsolePullRequestReviewActions } from './ConsolePullRequestReviewActions';
-
-describe('ConsolePullRequestReviewActions', () => {
-  it('renders the four buttons left to right', () => {
-    const { getAllByRole } = render(
-      <ConsolePullRequestReviewActions
-        onReview={() => {}}
-        rejectEnabled={false}
-      />,
-    );
-    expect(getAllByRole('button').map((button) => button.textContent)).toEqual([
-      'Unnecessary',
-      'Totally wrong',
-      'Reject',
-      'Approve & Merge',
-    ]);
-  });
-
-  it('disables Reject until an inline comment has been entered', () => {
-    const { getByText, rerender } = render(
-      <ConsolePullRequestReviewActions
-        onReview={() => {}}
-        rejectEnabled={false}
-      />,
-    );
-    expect(getByText('Reject')).toBeDisabled();
-    expect(getByText('Approve & Merge')).not.toBeDisabled();
-    rerender(
-      <ConsolePullRequestReviewActions
-        onReview={() => {}}
-        rejectEnabled={true}
-      />,
-    );
-    expect(getByText('Reject')).not.toBeDisabled();
-  });
-
-  it('states on screen why Reject cannot be used', () => {
-    const { getByText } = render(
-      <ConsolePullRequestReviewActions
-        onReview={() => {}}
-        rejectEnabled={false}
-      />,
-    );
-    expect(
-      getByText('Reject needs an inline comment on the diff.'),
-    ).toBeInTheDocument();
-  });
-
-  it('stops stating why Reject cannot be used once an inline comment exists', () => {
-    const { queryByText } = render(
-      <ConsolePullRequestReviewActions
-        onReview={() => {}}
-        rejectEnabled={true}
-      />,
-    );
-    expect(
-      queryByText('Reject needs an inline comment on the diff.'),
-    ).toBeNull();
-  });
-
-  it('does not report a reject action while disabled', () => {
-    const onReview = jest.fn();
-    const { getByText } = render(
-      <ConsolePullRequestReviewActions
-        onReview={onReview}
-        rejectEnabled={false}
-      />,
-    );
-    fireEvent.click(getByText('Reject'));
-    expect(onReview).not.toHaveBeenCalled();
-  });
-
-  it('reports each review action when reject is enabled', () => {
-    const onReview = jest.fn();
-    const { getByText } = render(
-      <ConsolePullRequestReviewActions
-        onReview={onReview}
-        rejectEnabled={true}
-      />,
-    );
-    fireEvent.click(getByText('Unnecessary'));
-    fireEvent.click(getByText('Totally wrong'));
-    fireEvent.click(getByText('Reject'));
-    fireEvent.click(getByText('Approve & Merge'));
-    expect(onReview.mock.calls.map((call) => call[0])).toEqual([
-      'unnecessary',
-      'totally_wrong',
-      'request_changes',
-      'approve_and_merge',
-    ]);
-  });
-});

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsolePullRequestReviewActions.tsx

@@ -1,45 +0,0 @@
-import type { ConsoleReviewAction } from '../../logic/operations';
-
-export type ConsolePullRequestReviewGroupProps = {
-  onReview: (action: ConsoleReviewAction) => void;
-  rejectEnabled: boolean;
-};
-
-const REVIEW_BUTTONS: {
-  action: ConsoleReviewAction;
-  label: string;
-  variant: string;
-}[] = [
-  { action: 'unnecessary', label: 'Unnecessary', variant: 'unneeded' },
-  { action: 'totally_wrong', label: 'Totally wrong', variant: 'wrong' },
-  { action: 'request_changes', label: 'Reject', variant: 'reject' },
-  { action: 'approve_and_merge', label: 'Approve & Merge', variant: 'approve' },
-];
-
-const REJECT_REQUIREMENT_MESSAGE =
-  'Reject needs an inline comment on the diff.';
-
-export const ConsolePullRequestReviewActions = ({
-  onReview,
-  rejectEnabled,
-}: ConsolePullRequestReviewGroupProps) => (
-  <div className="console-op-group console-op-group-review">
-    {!rejectEnabled && (
-      <p className="console-op-requirement">{REJECT_REQUIREMENT_MESSAGE}</p>
-    )}
-    {REVIEW_BUTTONS.map((button) => {
-      const disabled = button.action === 'request_changes' && !rejectEnabled;
-      return (
-        <button
-          key={button.action}
-          type="button"
-          className={`console-op-button console-op-button-${button.variant}`}
-          disabled={disabled}
-          onClick={() => onReview(button.action)}
-        >
-          {button.label}
-        </button>
-      );
-    })}
-  </div>
-);

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsoleStatusActions.stories.tsx

@@ -1,17 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react-vite';
-import { consoleStatusOptionsFixture } from '../../testing/fixtures';
-import { ConsoleStatusActions } from './ConsoleStatusActions';
-
-const meta: Meta<typeof ConsoleStatusActions> = {
-  title: 'Console/ConsoleStatusActions',
-  component: ConsoleStatusActions,
-  args: { onSetStatus: () => {}, onSetInTmuxByHuman: () => {} },
-};
-
-export default meta;
-
-type Story = StoryObj<typeof ConsoleStatusActions>;
-
-export const AllStatusOptions: Story = {
-  args: { statusOptions: consoleStatusOptionsFixture },
-};

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsoleStatusActions.test.tsx

@@ -1,49 +0,0 @@
-import { fireEvent, render } from '@testing-library/react';
-import { consoleStatusOptionsFixture } from '../../testing/fixtures';
-import { ConsoleStatusActions } from './ConsoleStatusActions';
-
-describe('ConsoleStatusActions', () => {
-  it('renders only the four routed status buttons that exist in the data', () => {
-    const { getByText, queryByText } = render(
-      <ConsoleStatusActions
-        statusOptions={consoleStatusOptionsFixture}
-        onSetStatus={() => {}}
-        onSetInTmuxByHuman={() => {}}
-      />,
-    );
-    expect(getByText('In Tmux by agent')).toBeInTheDocument();
-    expect(getByText('In Tmux by human')).toBeInTheDocument();
-    expect(getByText('Todo by human')).toBeInTheDocument();
-    expect(getByText('Awaiting Workspace')).toBeInTheDocument();
-    expect(queryByText('Preparation')).toBeNull();
-  });
-
-  it('routes In Tmux by human to the intmux handler and others to the status handler', () => {
-    const onSetStatus = jest.fn();
-    const onSetInTmuxByHuman = jest.fn();
-    const { getByText } = render(
-      <ConsoleStatusActions
-        statusOptions={consoleStatusOptionsFixture}
-        onSetStatus={onSetStatus}
-        onSetInTmuxByHuman={onSetInTmuxByHuman}
-      />,
-    );
-    fireEvent.click(getByText('In Tmux by human'));
-    fireEvent.click(getByText('In Tmux by agent'));
-    expect(onSetInTmuxByHuman).toHaveBeenCalledTimes(1);
-    expect(onSetInTmuxByHuman.mock.calls[0][0].name).toBe('In Tmux by human');
-    expect(onSetStatus).toHaveBeenCalledTimes(1);
-    expect(onSetStatus.mock.calls[0][0].name).toBe('In Tmux by agent');
-  });
-
-  it('renders nothing when no routed option exists', () => {
-    const { container } = render(
-      <ConsoleStatusActions
-        statusOptions={[{ id: 'x', name: 'Preparation', color: 'YELLOW' }]}
-        onSetStatus={() => {}}
-        onSetInTmuxByHuman={() => {}}
-      />,
-    );
-    expect(container.firstChild).toBeNull();
-  });
-});

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsoleStatusActions.tsx

@@ -1,66 +0,0 @@
-import { colorFromEnum } from '../../logic/colors';
-import {
-  IN_TMUX_BY_HUMAN_NAME,
-  STATUS_BUTTON_NAMES,
-} from '../../logic/operations';
-import type { ConsoleFieldOption } from '../../logic/types';
-
-export type ConsoleStatusButtonGroupProps = {
-  statusOptions: ConsoleFieldOption[];
-  onSetStatus: (option: ConsoleFieldOption) => void;
-  onSetInTmuxByHuman: (option: ConsoleFieldOption) => void;
-};
-
-const findStatusOption = (
-  statusOptions: ConsoleFieldOption[],
-  name: string,
-): ConsoleFieldOption | null => {
-  const lower = name.toLowerCase();
-  return (
-    statusOptions.find((option) => option.name.toLowerCase() === lower) ?? null
-  );
-};
-
-export const ConsoleStatusActions = ({
-  statusOptions,
-  onSetStatus,
-  onSetInTmuxByHuman,
-}: ConsoleStatusButtonGroupProps) => {
-  const buttons = STATUS_BUTTON_NAMES.map((name) => ({
-    name,
-    option: findStatusOption(statusOptions, name),
-  })).filter(
-    (entry): entry is { name: string; option: ConsoleFieldOption } =>
-      entry.option !== null,
-  );
-
-  if (buttons.length === 0) {
-    return null;
-  }
-
-  return (
-    <div className="console-op-group">
-      {buttons.map(({ name, option }) => {
-        const palette = colorFromEnum(option.color);
-        const isInTmuxByHuman = name === IN_TMUX_BY_HUMAN_NAME;
-        return (
-          <button
-            key={option.id}
-            type="button"
-            className="console-op-button"
-            style={{
-              color: palette.fg,
-              borderColor: palette.border,
-              backgroundColor: palette.bg,
-            }}
-            onClick={() =>
-              isInTmuxByHuman ? onSetInTmuxByHuman(option) : onSetStatus(option)
-            }
-          >
-            {option.name}
-          </button>
-        );
-      })}
-    </div>
-  );
-};

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsoleUndoToast.stories.tsx

@@ -1,47 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react-vite';
-import { ConsoleUndoToast } from './ConsoleUndoToast';
-
-const meta: Meta<typeof ConsoleUndoToast> = {
-  title: 'Console/ConsoleUndoToast',
-  component: ConsoleUndoToast,
-  args: {
-    message: 'Approved — PR #851',
-    color: 'green',
-    remainingSeconds: 5,
-    progress: 1,
-    onUndo: () => {},
-  },
-};
-
-export default meta;
-
-type Story = StoryObj<typeof ConsoleUndoToast>;
-
-export const Approve: Story = {};
-
-export const Reject: Story = {
-  args: {
-    message: 'Rejected — PR #853',
-    color: 'amber',
-    remainingSeconds: 3,
-    progress: 0.6,
-  },
-};
-
-export const StatusChange: Story = {
-  args: {
-    message: 'Status → Awaiting Workspace — #845',
-    color: 'blue',
-    remainingSeconds: 2,
-    progress: 0.4,
-  },
-};
-
-export const CloseIssue: Story = {
-  args: {
-    message: 'Closed — #845',
-    color: 'red',
-    remainingSeconds: 1,
-    progress: 0.2,
-  },
-};

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsoleUndoToast.test.tsx

@@ -1,95 +0,0 @@
-import { fireEvent, render } from '@testing-library/react';
-import { ConsoleErrorToast, ConsoleUndoToast } from './ConsoleUndoToast';
-
-describe('ConsoleUndoToast', () => {
-  const baseProps = {
-    message: 'Approved — PR #851',
-    color: 'green' as const,
-    remainingSeconds: 5,
-    progress: 1,
-    onUndo: () => {},
-  };
-
-  it('shows the action message and the countdown', () => {
-    const { getByText } = render(<ConsoleUndoToast {...baseProps} />);
-    expect(getByText('Approved — PR #851')).toBeInTheDocument();
-    expect(getByText('5s')).toBeInTheDocument();
-  });
-
-  it('renders the color modifier class', () => {
-    const { container } = render(<ConsoleUndoToast {...baseProps} />);
-    expect(
-      container.querySelector('.console-undo-toast-green'),
-    ).toBeInTheDocument();
-  });
-
-  it('shrinks the progress bar with the remaining fraction', () => {
-    const { container } = render(
-      <ConsoleUndoToast {...baseProps} progress={0.4} />,
-    );
-    const bar = container.querySelector(
-      '.console-undo-toast-bar',
-    ) as HTMLElement;
-    expect(bar.style.width).toBe('40%');
-  });
-
-  it('clamps the progress bar width within 0 and 100 percent', () => {
-    const { container } = render(
-      <ConsoleUndoToast {...baseProps} progress={1.4} />,
-    );
-    const bar = container.querySelector(
-      '.console-undo-toast-bar',
-    ) as HTMLElement;
-    expect(bar.style.width).toBe('100%');
-  });
-
-  it('invokes onUndo when the Undo control is clicked', () => {
-    const onUndo = jest.fn();
-    const { getByText } = render(
-      <ConsoleUndoToast {...baseProps} onUndo={onUndo} />,
-    );
-    fireEvent.click(getByText('Undo'));
-    expect(onUndo).toHaveBeenCalledTimes(1);
-  });
-});
-
-describe('ConsoleErrorToast', () => {
-  it('shows the failure message with the error modifier class and alert role', () => {
-    const { getByText, container, getByRole } = render(
-      <ConsoleErrorToast
-        message="Operation failed: HTTP 422"
-        onDismiss={() => {}}
-      />,
-    );
-    expect(getByText('Operation failed: HTTP 422')).toBeInTheDocument();
-    expect(container.querySelector('.console-error-toast')).toBeInTheDocument();
-    expect(getByRole('alert')).toBeInTheDocument();
-  });
-
-  it('does not render a countdown or progress bar', () => {
-    const { container } = render(
-      <ConsoleErrorToast
-        message="Operation failed: boom"
-        onDismiss={() => {}}
-      />,
-    );
-    expect(
-      container.querySelector('.console-undo-toast-countdown'),
-    ).not.toBeInTheDocument();
-    expect(
-      container.querySelector('.console-undo-toast-bar'),
-    ).not.toBeInTheDocument();
-  });
-
-  it('invokes onDismiss when the Dismiss control is clicked', () => {
-    const onDismiss = jest.fn();
-    const { getByText } = render(
-      <ConsoleErrorToast
-        message="Operation failed: boom"
-        onDismiss={onDismiss}
-      />,
-    );
-    fireEvent.click(getByText('Dismiss'));
-    expect(onDismiss).toHaveBeenCalledTimes(1);
-  });
-});

package/src/adapter/entry-points/console/ui/src/features/console/components/operations/ConsoleUndoToast.tsx

@@ -1,62 +0,0 @@
-import type { ConsoleToastColor } from '../../logic/actionToast';
-
-export type ConsoleUndoToastProps = {
-  message: string;
-  color: ConsoleToastColor;
-  remainingSeconds: number;
-  progress: number;
-  onUndo: () => void;
-};
-
-export const ConsoleUndoToast = ({
-  message,
-  color,
-  remainingSeconds,
-  progress,
-  onUndo,
-}: ConsoleUndoToastProps) => (
-  <div
-    className={`console-undo-toast console-undo-toast-${color}`}
-    role="status"
-    aria-live="polite"
-  >
-    <span className="console-undo-toast-message">{message}</span>
-    <button type="button" className="console-undo-toast-undo" onClick={onUndo}>
-      Undo
-    </button>
-    <span className="console-undo-toast-countdown" aria-hidden="true">
-      {remainingSeconds}s
-    </span>
-    <span
-      className="console-undo-toast-bar"
-      aria-hidden="true"
-      style={{ width: `${Math.max(0, Math.min(100, progress * 100))}%` }}
-    />
-  </div>
-);
-
-export type ConsoleErrorToastProps = {
-  title?: string;
-  message: string;
-  onDismiss: () => void;
-};
-
-export const ConsoleErrorToast = ({
-  title,
-  message,
-  onDismiss,
-}: ConsoleErrorToastProps) => (
-  <div className="console-error-toast" role="alert" aria-live="assertive">
-    {title !== undefined && (
-      <span className="console-error-toast-title">{title}</span>
-    )}
-    <span className="console-error-toast-message">{message}</span>
-    <button
-      type="button"
-      className="console-error-toast-dismiss"
-      onClick={onDismiss}
-    >
-      Dismiss
-    </button>
-  </div>
-);

package/src/adapter/entry-points/console/ui/src/features/console/components/shared/ConsoleClipboardCopyButton.stories.tsx

@@ -1,32 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react-vite';
-import { ConsoleClipboardCopyButton } from './ConsoleClipboardCopyButton';
-
-const meta: Meta<typeof ConsoleClipboardCopyButton> = {
-  title: 'Console/ConsoleClipboardCopyButton',
-  component: ConsoleClipboardCopyButton,
-};
-
-export default meta;
-
-type Story = StoryObj<typeof ConsoleClipboardCopyButton>;
-
-export const UrlValue: Story = {
-  args: {
-    value:
-      'https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/issues/845',
-    idleText: 'Copy URL',
-    idleAriaLabel: 'Copy URL',
-    copiedAriaLabel: 'URL copied to clipboard',
-    className: 'console-copy-url-button',
-  },
-};
-
-export const CodeValue: Story = {
-  args: {
-    value: 'npm run build:console-ui\nnode scripts/copyConsoleUiDist.mjs\n',
-    idleText: 'Copy code',
-    idleAriaLabel: 'Copy code',
-    copiedAriaLabel: 'Code copied to clipboard',
-    className: 'console-copy-code-button',
-  },
-};

package/src/adapter/entry-points/console/ui/src/features/console/components/shared/ConsoleClipboardCopyButton.test.tsx

@@ -1,101 +0,0 @@
-import { act, fireEvent, render } from '@testing-library/react';
-import { ConsoleClipboardCopyButton } from './ConsoleClipboardCopyButton';
-
-const value = 'first line\nsecond line\n';
-
-const renderButton = () =>
-  render(
-    <ConsoleClipboardCopyButton
-      value={value}
-      idleText="Copy code"
-      idleAriaLabel="Copy code"
-      copiedAriaLabel="Code copied to clipboard"
-      className="console-copy-code-button"
-    />,
-  );
-
-describe('ConsoleClipboardCopyButton', () => {
-  const writeText = jest.fn(async () => {});
-
-  beforeEach(() => {
-    writeText.mockClear();
-    Object.defineProperty(navigator, 'clipboard', {
-      configurable: true,
-      value: { writeText },
-    });
-  });
-
-  it('renders the idle text and idle accessible name', () => {
-    const { getByRole } = renderButton();
-    expect(getByRole('button', { name: 'Copy code' })).toHaveTextContent(
-      'Copy code',
-    );
-  });
-
-  it('writes the value verbatim and switches to the copied accessible name', async () => {
-    const { getByRole } = renderButton();
-    await act(async () => {
-      fireEvent.click(getByRole('button'));
-    });
-    expect(writeText).toHaveBeenCalledWith(value);
-    expect(
-      getByRole('button', { name: 'Code copied to clipboard' }),
-    ).toHaveTextContent('Copied');
-  });
-
-  it('reverts to the idle state after the feedback delay', async () => {
-    jest.useFakeTimers();
-    try {
-      const { getByRole, queryByText } = renderButton();
-      await act(async () => {
-        fireEvent.click(getByRole('button'));
-      });
-      act(() => {
-        jest.advanceTimersByTime(1500);
-      });
-      expect(queryByText('Copied')).toBeNull();
-      expect(getByRole('button', { name: 'Copy code' })).toBeInTheDocument();
-    } finally {
-      jest.useRealTimers();
-    }
-  });
-
-  it('copies through the document selection command when the clipboard api is absent', async () => {
-    Reflect.deleteProperty(navigator, 'clipboard');
-    const execCommand = jest.fn().mockReturnValue(true);
-    Object.defineProperty(document, 'execCommand', {
-      configurable: true,
-      value: execCommand,
-    });
-    const { getByRole } = renderButton();
-
-    await act(async () => {
-      fireEvent.click(getByRole('button'));
-    });
-
-    expect(execCommand).toHaveBeenCalledWith('copy');
-    expect(
-      getByRole('button', { name: 'Code copied to clipboard' }),
-    ).toHaveTextContent('Copied');
-  });
-
-  it('shows a failed state when no copy mechanism succeeds', async () => {
-    Reflect.deleteProperty(navigator, 'clipboard');
-    Object.defineProperty(document, 'execCommand', {
-      configurable: true,
-      value: jest.fn().mockReturnValue(false),
-    });
-    const { getByRole } = renderButton();
-
-    await act(async () => {
-      fireEvent.click(getByRole('button'));
-    });
-
-    expect(getByRole('button')).toHaveTextContent('Copy failed');
-  });
-
-  it('applies the given class name to the rendered button', () => {
-    const { getByRole } = renderButton();
-    expect(getByRole('button')).toHaveClass('console-copy-code-button');
-  });
-});

package/src/adapter/entry-points/console/ui/src/features/console/components/shared/ConsoleClipboardCopyButton.tsx

@@ -1,80 +0,0 @@
-import { useEffect, useRef, useState } from 'react';
-import { Button } from '@/components/ui/button';
-import { copyTextToClipboard } from '../../logic/clipboard';
-
-export type ConsoleClipboardCopyButtonProps = {
-  value: string;
-  idleText: string;
-  idleAriaLabel: string;
-  copiedAriaLabel: string;
-  className: string;
-};
-
-type ConsoleClipboardCopyState = 'idle' | 'copied' | 'failed';
-
-const COPIED_FEEDBACK_MS = 1500;
-const FAILED_TEXT = 'Copy failed';
-const FAILED_ARIA_LABEL = 'Copying to the clipboard failed';
-
-export const ConsoleClipboardCopyButton = ({
-  value,
-  idleText,
-  idleAriaLabel,
-  copiedAriaLabel,
-  className,
-}: ConsoleClipboardCopyButtonProps) => {
-  const [copyState, setCopyState] = useState<ConsoleClipboardCopyState>('idle');
-  const resetTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
-
-  useEffect(
-    () => () => {
-      if (resetTimerRef.current !== null) {
-        clearTimeout(resetTimerRef.current);
-      }
-    },
-    [],
-  );
-
-  const handleCopy = async () => {
-    try {
-      await copyTextToClipboard(value);
-      setCopyState('copied');
-    } catch (error) {
-      console.error(error);
-      setCopyState('failed');
-    }
-    if (resetTimerRef.current !== null) {
-      clearTimeout(resetTimerRef.current);
-    }
-    resetTimerRef.current = setTimeout(() => {
-      setCopyState('idle');
-      resetTimerRef.current = null;
-    }, COPIED_FEEDBACK_MS);
-  };
-
-  const buttonText =
-    copyState === 'copied'
-      ? 'Copied'
-      : copyState === 'failed'
-        ? FAILED_TEXT
-        : idleText;
-  const buttonAriaLabel =
-    copyState === 'copied'
-      ? copiedAriaLabel
-      : copyState === 'failed'
-        ? FAILED_ARIA_LABEL
-        : idleAriaLabel;
-
-  return (
-    <Button
-      type="button"
-      variant="ghost"
-      size="sm"
-      className={className}
-      aria-label={buttonAriaLabel}
-      onClick={handleCopy}
-    >
-      {buttonText}
-    </Button>
-  );
-};