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

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

@@ -1,262 +0,0 @@
-import { fireEvent, render } from '@testing-library/react';
-import {
-  type ConsoleOfflinePendingActionItem,
-  ConsoleOfflinePendingActionsPanel,
-} from './ConsoleOfflinePendingActionsPanel';
-
-const approveAction: ConsoleOfflinePendingActionItem = {
-  id: 'offline-1',
-  message: 'Approved — PR #851',
-  color: 'green',
-  itemNumber: 851,
-  isPr: true,
-  currentTitle: 'Add serveConsole subcommand under entry-points',
-  currentState: 'open',
-  currentPrStatus: null,
-  fetchError: false,
-};
-
-const baseProps = {
-  actions: [approveAction],
-  isOnline: true,
-  isConfirming: false,
-  onConfirm: () => {},
-  onDiscard: () => {},
-};
-
-describe('ConsoleOfflinePendingActionsPanel', () => {
-  it('renders nothing when the action list is empty', () => {
-    const { container } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} actions={[]} />,
-    );
-    expect(container.firstChild).toBeNull();
-  });
-
-  it('shows the action count and a confirm-before-sending message when online', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} />,
-    );
-    expect(getByText(/1 action held/)).toBeInTheDocument();
-    expect(getByText(/confirm each before sending/)).toBeInTheDocument();
-  });
-
-  it('shows an awaiting-connectivity message when offline', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} isOnline={false} />,
-    );
-    expect(getByText(/awaiting connectivity/)).toBeInTheDocument();
-  });
-
-  it('shows per-action rows with item target when offline', () => {
-    const { getByText, queryByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} isOnline={false} />,
-    );
-    expect(getByText(/PR #851 — Add serveConsole/)).toBeInTheDocument();
-    expect(getByText('Approved — PR #851')).toBeInTheDocument();
-    expect(queryByText('Send')).not.toBeInTheDocument();
-  });
-
-  it('shows Discard button even when offline', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} isOnline={false} />,
-    );
-    expect(getByText('Discard')).toBeInTheDocument();
-  });
-
-  it('shows the action message and item label when online', () => {
-    const { getByText, container } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} />,
-    );
-    expect(getByText('Approved — PR #851')).toBeInTheDocument();
-    expect(
-      container.querySelector('.console-offline-panel-item-target')
-        ?.textContent,
-    ).toContain('PR #851');
-  });
-
-  it('shows the current item title and state when loaded', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} />,
-    );
-    expect(
-      getByText(/Add serveConsole subcommand under entry-points/),
-    ).toBeInTheDocument();
-    expect(getByText(/open/)).toBeInTheDocument();
-  });
-
-  it('shows a loading indicator when the current state is not yet fetched', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[{ ...approveAction, currentTitle: null, currentState: null }]}
-      />,
-    );
-    expect(getByText(/loading current state/)).toBeInTheDocument();
-  });
-
-  it('disables the Send button while the current state is loading', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[{ ...approveAction, currentTitle: null, currentState: null }]}
-      />,
-    );
-    expect(getByText('Send')).toBeDisabled();
-  });
-
-  it('enables the Send button once the current state is loaded', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} />,
-    );
-    expect(getByText('Send')).not.toBeDisabled();
-  });
-
-  it('enables the Send button when the state fetch failed so the operator can still proceed', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[
-          {
-            ...approveAction,
-            currentTitle: null,
-            currentState: null,
-            fetchError: true,
-          },
-        ]}
-      />,
-    );
-    expect(getByText('Send')).not.toBeDisabled();
-  });
-
-  it('shows a fetch error message when the state could not be loaded', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[
-          {
-            ...approveAction,
-            currentTitle: null,
-            currentState: null,
-            fetchError: true,
-          },
-        ]}
-      />,
-    );
-    expect(getByText(/Failed to load current state/)).toBeInTheDocument();
-  });
-
-  it('disables the Send button while a confirmation is in progress', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} isConfirming={true} />,
-    );
-    expect(getByText('Send')).toBeDisabled();
-  });
-
-  it('calls onConfirm with the action id when Send is clicked', () => {
-    const onConfirm = jest.fn();
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        onConfirm={onConfirm}
-      />,
-    );
-    fireEvent.click(getByText('Send'));
-    expect(onConfirm).toHaveBeenCalledWith('offline-1');
-  });
-
-  it('calls onDiscard with the action id when Discard is clicked', () => {
-    const onDiscard = jest.fn();
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        onDiscard={onDiscard}
-      />,
-    );
-    fireEvent.click(getByText('Discard'));
-    expect(onDiscard).toHaveBeenCalledWith('offline-1');
-  });
-
-  it('uses plural phrasing for the count when more than one action is queued', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[
-          approveAction,
-          { ...approveAction, id: 'offline-2', message: 'Rejected — PR #853' },
-        ]}
-      />,
-    );
-    expect(getByText(/2 actions held/)).toBeInTheDocument();
-  });
-
-  it('renders a Send and Discard button for each queued action when online', () => {
-    const { getAllByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[
-          approveAction,
-          { ...approveAction, id: 'offline-2', message: 'Rejected — PR #853' },
-        ]}
-      />,
-    );
-    expect(getAllByText('Send')).toHaveLength(2);
-    expect(getAllByText('Discard')).toHaveLength(2);
-  });
-
-  it('applies the color modifier class to the action row', () => {
-    const { container } = render(
-      <ConsoleOfflinePendingActionsPanel {...baseProps} />,
-    );
-    expect(
-      container.querySelector('.console-offline-panel-item-green'),
-    ).toBeInTheDocument();
-  });
-
-  it('shows PR status information when the action targets a PR and status is loaded', () => {
-    const { getByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        actions={[
-          {
-            ...approveAction,
-            currentPrStatus: {
-              found: true,
-              isConflicted: false,
-              mergeableStatus: 'MERGEABLE',
-              isPassedAllCiJob: true,
-              isCiStateSuccess: true,
-              isBranchOutOfDate: false,
-              missingRequiredCheckNames: [],
-            },
-          },
-        ]}
-      />,
-    );
-    expect(getByText(/CI: pass/)).toBeInTheDocument();
-    expect(getByText(/Merge: mergeable/)).toBeInTheDocument();
-  });
-
-  it('does not show PR status when offline even if currentPrStatus is set', () => {
-    const { queryByText } = render(
-      <ConsoleOfflinePendingActionsPanel
-        {...baseProps}
-        isOnline={false}
-        actions={[
-          {
-            ...approveAction,
-            currentPrStatus: {
-              found: true,
-              isConflicted: false,
-              mergeableStatus: 'MERGEABLE',
-              isPassedAllCiJob: true,
-              isCiStateSuccess: true,
-              isBranchOutOfDate: false,
-              missingRequiredCheckNames: [],
-            },
-          },
-        ]}
-      />,
-    );
-    expect(queryByText(/CI: pass/)).not.toBeInTheDocument();
-  });
-});

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

@@ -1,123 +0,0 @@
-import type { ConsoleToastColor } from '../../logic/actionToast';
-import type { ConsolePullRequestStatus } from '../../logic/types';
-
-export type ConsoleOfflinePendingActionItem = {
-  id: string;
-  message: string;
-  color: ConsoleToastColor;
-  itemNumber: number;
-  isPr: boolean;
-  currentTitle: string | null;
-  currentState: 'open' | 'closed' | null;
-  currentPrStatus: ConsolePullRequestStatus | null;
-  fetchError: boolean;
-};
-
-export type ConsoleOfflinePendingActionsPanelProps = {
-  actions: ConsoleOfflinePendingActionItem[];
-  isOnline: boolean;
-  isConfirming: boolean;
-  onConfirm: (id: string) => void;
-  onDiscard: (id: string) => void;
-};
-
-const itemLabel = (item: ConsoleOfflinePendingActionItem): string =>
-  `${item.isPr ? 'PR' : 'Issue'} #${item.itemNumber}`;
-
-const stateLabel = (state: 'open' | 'closed' | null): string => {
-  if (state === null) return 'loading…';
-  return state;
-};
-
-const prStatusLabel = (prStatus: ConsolePullRequestStatus): string => {
-  const parts: string[] = [];
-  parts.push(`CI: ${prStatus.isCiStateSuccess ? 'pass' : 'fail'}`);
-  parts.push(`Merge: ${prStatus.mergeableStatus.toLowerCase()}`);
-  if (prStatus.isBranchOutOfDate) {
-    parts.push('branch out of date');
-  }
-  return parts.join(' · ');
-};
-
-const targetText = (action: ConsoleOfflinePendingActionItem): string => {
-  if (action.fetchError) {
-    return `${itemLabel(action)} — failed to load current state`;
-  }
-  if (action.currentTitle === null) {
-    return `${itemLabel(action)} — loading current state…`;
-  }
-  return `${itemLabel(action)} — ${action.currentTitle} (${stateLabel(action.currentState)})`;
-};
-
-export const ConsoleOfflinePendingActionsPanel = ({
-  actions,
-  isOnline,
-  isConfirming,
-  onConfirm,
-  onDiscard,
-}: ConsoleOfflinePendingActionsPanelProps) => {
-  if (actions.length === 0) return null;
-
-  return (
-    <section
-      className="console-offline-panel"
-      aria-label="Offline actions queue"
-    >
-      <div className="console-offline-panel-header">
-        {actions.length} action{actions.length !== 1 ? 's' : ''} held
-        {isOnline
-          ? ' — confirm each before sending'
-          : ' — awaiting connectivity'}
-      </div>
-      {actions.map((action) => (
-        <div
-          key={action.id}
-          className={`console-offline-panel-item console-offline-panel-item-${action.color}`}
-        >
-          <div className="console-offline-panel-item-message">
-            {action.message}
-          </div>
-          <div className="console-offline-panel-item-target">
-            {targetText(action)}
-          </div>
-          {action.fetchError && (
-            <div className="console-offline-panel-item-fetch-error">
-              Failed to load current state — sending will act on the last known
-              state
-            </div>
-          )}
-          {isOnline &&
-            action.isPr &&
-            action.currentPrStatus !== null &&
-            !action.fetchError && (
-              <div className="console-offline-panel-item-pr-status">
-                {prStatusLabel(action.currentPrStatus)}
-              </div>
-            )}
-          <div className="console-offline-panel-item-controls">
-            {isOnline && (
-              <button
-                type="button"
-                className="console-offline-panel-confirm"
-                disabled={
-                  isConfirming ||
-                  (action.currentTitle === null && !action.fetchError)
-                }
-                onClick={() => onConfirm(action.id)}
-              >
-                Send
-              </button>
-            )}
-            <button
-              type="button"
-              className="console-offline-panel-discard"
-              onClick={() => onDiscard(action.id)}
-            >
-              Discard
-            </button>
-          </div>
-        </div>
-      ))}
-    </section>
-  );
-};

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

@@ -1,65 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react-vite';
-import type { ConsoleOperationHandlers } from '../../logic/operations';
-import {
-  consoleListItemsFixture,
-  consoleStatusOptionsFixture,
-} from '../../testing/fixtures';
-import { ConsoleOperationMenu } from './ConsoleOperationMenu';
-
-const handlers: ConsoleOperationHandlers = {
-  onReview: () => {},
-  onSetNextActionDate: () => {},
-  onSetStory: () => {},
-  onSetStatus: () => {},
-  onSetInTmuxByHuman: () => {},
-  onClose: () => {},
-  onOkAndAwaitingWorkspace: () => {},
-  onDeleteAllComments: () => {},
-};
-
-const meta: Meta<typeof ConsoleOperationMenu> = {
-  title: 'Console/ConsoleOperationMenu',
-  component: ConsoleOperationMenu,
-  args: {
-    rejectEnabled: false,
-    statusOptions: consoleStatusOptionsFixture,
-    handlers,
-  },
-};
-
-export default meta;
-
-type Story = StoryObj<typeof ConsoleOperationMenu>;
-
-export const PrsTabPullRequest: Story = {
-  args: {
-    tab: 'prs',
-    item: consoleListItemsFixture[0],
-    hasPullRequest: true,
-  },
-};
-
-export const PrsTabPullRequestRejectEnabled: Story = {
-  args: {
-    tab: 'prs',
-    item: consoleListItemsFixture[0],
-    hasPullRequest: true,
-    rejectEnabled: true,
-  },
-};
-
-export const TodoByHumanTabIssue: Story = {
-  args: {
-    tab: 'todo-by-human',
-    item: consoleListItemsFixture[2],
-    hasPullRequest: false,
-  },
-};
-
-export const TodoByAgentTabIssue: Story = {
-  args: {
-    tab: 'todo-by-agent',
-    item: consoleListItemsFixture[2],
-    hasPullRequest: false,
-  },
-};

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

@@ -1,159 +0,0 @@
-import { render } from '@testing-library/react';
-import type { ConsoleOperationHandlers } from '../../logic/operations';
-import {
-  consoleListItemsFixture,
-  consoleStatusOptionsFixture,
-} from '../../testing/fixtures';
-import { ConsoleOperationMenu } from './ConsoleOperationMenu';
-
-const handlers: ConsoleOperationHandlers = {
-  onReview: jest.fn(),
-  onSetNextActionDate: jest.fn(),
-  onSetStory: jest.fn(),
-  onSetStatus: jest.fn(),
-  onSetInTmuxByHuman: jest.fn(),
-  onClose: jest.fn(),
-  onOkAndAwaitingWorkspace: jest.fn(),
-  onDeleteAllComments: jest.fn(),
-};
-
-const prItem = consoleListItemsFixture[0];
-const issueItem = consoleListItemsFixture[2];
-
-describe('ConsoleOperationMenu', () => {
-  it('shows the review and close groups for a PR on the prs tab', () => {
-    const { getByText, queryByText } = render(
-      <ConsoleOperationMenu
-        tab="prs"
-        item={prItem}
-        hasPullRequest
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('Approve & Merge')).toBeInTheDocument();
-    expect(getByText('+1 day')).toBeInTheDocument();
-    expect(getByText('Awaiting Workspace')).toBeInTheDocument();
-    expect(getByText('Close')).toBeInTheDocument();
-    expect(getByText('Close as not planned')).toBeInTheDocument();
-    expect(queryByText('Move to Okinawa')).toBeNull();
-    expect(queryByText('TDPM Console port')).toBeNull();
-  });
-
-  it('shows +1 week and skip on the todo-by-human tab', () => {
-    const { getByText } = render(
-      <ConsoleOperationMenu
-        tab="todo-by-human"
-        item={issueItem}
-        hasPullRequest={false}
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('+1 week and skip')).toBeInTheDocument();
-  });
-
-  it('shows +1 week and skip on the todo-by-agent tab', () => {
-    const { getByText } = render(
-      <ConsoleOperationMenu
-        tab="todo-by-agent"
-        item={issueItem}
-        hasPullRequest={false}
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('+1 week and skip')).toBeInTheDocument();
-  });
-
-  it('hides the review group when there is no pull request', () => {
-    const { queryByText } = render(
-      <ConsoleOperationMenu
-        tab="todo-by-human"
-        item={issueItem}
-        hasPullRequest={false}
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(queryByText('Approve & Merge')).toBeNull();
-    expect(queryByText('Close')).not.toBeNull();
-  });
-
-  it('shows status, close and next-action groups on the workflow-blocker tab', () => {
-    const { getByText, queryByText } = render(
-      <ConsoleOperationMenu
-        tab="workflow-blocker"
-        item={issueItem}
-        hasPullRequest={false}
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('Awaiting Workspace')).toBeInTheDocument();
-    expect(getByText('Close')).toBeInTheDocument();
-    expect(getByText('Close as not planned')).toBeInTheDocument();
-    expect(getByText('+1 day')).toBeInTheDocument();
-    expect(queryByText('Move to Okinawa')).toBeNull();
-    expect(queryByText('Approve & Merge')).toBeNull();
-  });
-
-  it('shows the review group on the workflow-blocker tab when the item has a pull request', () => {
-    const { getByText } = render(
-      <ConsoleOperationMenu
-        tab="workflow-blocker"
-        item={prItem}
-        hasPullRequest
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('Approve & Merge')).toBeInTheDocument();
-    expect(getByText('Close')).toBeInTheDocument();
-  });
-
-  it('forwards the reject-enabled state to the review group', () => {
-    const { getByText, rerender } = render(
-      <ConsoleOperationMenu
-        tab="prs"
-        item={prItem}
-        hasPullRequest
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('Reject')).toBeDisabled();
-    rerender(
-      <ConsoleOperationMenu
-        tab="prs"
-        item={prItem}
-        hasPullRequest
-        rejectEnabled={true}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('Reject')).not.toBeDisabled();
-  });
-
-  it('renders the dangerous actions toggle on all tabs', () => {
-    const { getByText } = render(
-      <ConsoleOperationMenu
-        tab="prs"
-        item={prItem}
-        hasPullRequest
-        rejectEnabled={false}
-        statusOptions={consoleStatusOptionsFixture}
-        handlers={handlers}
-      />,
-    );
-    expect(getByText('⚠')).toBeInTheDocument();
-  });
-});

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

@@ -1,55 +0,0 @@
-import {
-  type ConsoleOperationHandlers,
-  isManualTriageTab,
-} from '../../logic/operations';
-import type {
-  ConsoleFieldOption,
-  ConsoleListItem,
-  ConsoleTabName,
-} from '../../logic/types';
-import { ConsoleCloseActions } from './ConsoleCloseActions';
-import { ConsoleDangerousActions } from './ConsoleDangerousActions';
-import { ConsoleNextActionDateActions } from './ConsoleNextActionDateActions';
-import { ConsolePullRequestReviewActions } from './ConsolePullRequestReviewActions';
-import { ConsoleStatusActions } from './ConsoleStatusActions';
-
-export type ConsoleOperationBarProps = {
-  tab: ConsoleTabName;
-  item: ConsoleListItem;
-  hasPullRequest: boolean;
-  rejectEnabled: boolean;
-  statusOptions: ConsoleFieldOption[];
-  handlers: ConsoleOperationHandlers;
-};
-
-export const ConsoleOperationMenu = ({
-  tab,
-  hasPullRequest,
-  rejectEnabled,
-  statusOptions,
-  handlers,
-}: ConsoleOperationBarProps) => {
-  return (
-    <div className="console-operation-bar">
-      <ConsoleDangerousActions
-        onDeleteAllComments={handlers.onDeleteAllComments}
-      />
-      {hasPullRequest && (
-        <ConsolePullRequestReviewActions
-          onReview={handlers.onReview}
-          rejectEnabled={rejectEnabled}
-        />
-      )}
-      <ConsoleNextActionDateActions
-        isManualTriage={isManualTriageTab(tab)}
-        onSetNextActionDate={handlers.onSetNextActionDate}
-      />
-      <ConsoleStatusActions
-        statusOptions={statusOptions}
-        onSetStatus={handlers.onSetStatus}
-        onSetInTmuxByHuman={handlers.onSetInTmuxByHuman}
-      />
-      <ConsoleCloseActions onClose={handlers.onClose} />
-    </div>
-  );
-};

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

@@ -1,20 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react-vite';
-import { ConsolePullRequestReviewActions } from './ConsolePullRequestReviewActions';
-
-const meta: Meta<typeof ConsolePullRequestReviewActions> = {
-  title: 'Console/ConsolePullRequestReviewActions',
-  component: ConsolePullRequestReviewActions,
-  args: { onReview: () => {}, rejectEnabled: false },
-};
-
-export default meta;
-
-type Story = StoryObj<typeof ConsolePullRequestReviewActions>;
-
-export const RejectDisabled: Story = {
-  args: { rejectEnabled: false },
-};
-
-export const RejectEnabled: Story = {
-  args: { rejectEnabled: true },
-};