npm - github-issue-tower-defence-management - Versions diffs - 1.91.3 → 1.91.4 - Mend

github-issue-tower-defence-management 1.91.3 → 1.91.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (51) hide show

package/bin/adapter/entry-points/console/ui-dist/index.html CHANGED Viewed

@@ -4,8 +4,8 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>TDPM Console</title>
-    <script type="module" crossorigin src="/assets/index-BvuSQN9s.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-BU6p3cGU.css">
+    <script type="module" crossorigin src="/assets/index-D04N09aG.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-0IuY3q4G.css">
   </head>
   <body>
     <div id="root"></div>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "github-issue-tower-defence-management",
-  "version": "1.91.3",
+  "version": "1.91.4",
   "description": "",
   "main": "bin/index.js",
   "scripts": {

package/src/adapter/entry-points/console/consoleOperationApi.test.ts CHANGED Viewed

@@ -7,6 +7,7 @@ import { Project } from '../../../domain/entities/Project';
 import { Issue } from '../../../domain/entities/Issue';
 import {
   ConsoleOperationContext,
+  handleComment,
   handleIntmux,
   handleReview,
   handleTriage,
@@ -555,4 +556,75 @@ describe('consoleOperationApi', () => {
       expect(response.statusCode).toBe(200);
     });
   });
+  describe('handleComment', () => {
+    it('posts a comment and returns the created comment', async () => {
+      issueRepository.getIssueOrPullRequestComments.mockResolvedValue([
+        {
+          author: 'github-actions',
+          body: 'All required checks have passed.',
+          createdAt: new Date('2026-06-17T07:48:11.000Z'),
+        },
+        {
+          author: 'HiromiShikata',
+          body: 'Please rebase onto the latest main branch.',
+          createdAt: new Date('2026-06-17T09:03:27.000Z'),
+        },
+      ]);
+      const response = await handleComment(context, {
+        pjcode: 'umino',
+        url: 'https://github.com/o/r/issues/1',
+        body: 'Please rebase onto the latest main branch.',
+      });
+      expect(response.statusCode).toBe(200);
+      expect(issueRepository.createCommentByUrl).toHaveBeenCalledWith(
+        'https://github.com/o/r/issues/1',
+        'Please rebase onto the latest main branch.',
+      );
+      expect(response.body).toEqual({
+        ok: true,
+        comment: {
+          author: 'HiromiShikata',
+          body: 'Please rebase onto the latest main branch.',
+          createdAt: '2026-06-17T09:03:27.000Z',
+        },
+      });
+    });
+    it('falls back to the posted body when no comment is returned', async () => {
+      issueRepository.getIssueOrPullRequestComments.mockResolvedValue([]);
+      const response = await handleComment(context, {
+        pjcode: 'umino',
+        url: 'https://github.com/o/r/issues/1',
+        body: 'A first comment on this issue.',
+      });
+      expect(response.statusCode).toBe(200);
+      expect(response.body).toEqual({
+        ok: true,
+        comment: {
+          author: '',
+          body: 'A first comment on this issue.',
+          createdAt: '',
+        },
+      });
+    });
+    it('rejects when url is missing', async () => {
+      const response = await handleComment(context, {
+        pjcode: 'umino',
+        body: 'A comment without a target url.',
+      });
+      expect(response.statusCode).toBe(400);
+      expect(issueRepository.createCommentByUrl).not.toHaveBeenCalled();
+    });
+    it('rejects when body is missing', async () => {
+      const response = await handleComment(context, {
+        pjcode: 'umino',
+        url: 'https://github.com/o/r/issues/1',
+      });
+      expect(response.statusCode).toBe(400);
+      expect(issueRepository.createCommentByUrl).not.toHaveBeenCalled();
+    });
+  });
 });

package/src/adapter/entry-points/console/consoleOperationApi.ts CHANGED Viewed

@@ -296,6 +296,38 @@ export const handleTriage = async (
   return badRequest(`unknown triage action "${action}"`);
 };
+export const handleComment = async (
+  context: ConsoleOperationContext,
+  body: Record<string, unknown>,
+): Promise<ConsoleOperationResponse> => {
+  const url = body.url;
+  const commentBody = body.body;
+  if (!isNonEmptyString(url)) {
+    return badRequest('url is required');
+  }
+  if (!isNonEmptyString(commentBody)) {
+    return badRequest('body is required');
+  }
+  await context.issueRepository.createCommentByUrl(url, commentBody);
+  const comments =
+    await context.issueRepository.getIssueOrPullRequestComments(url);
+  const posted = comments[comments.length - 1] ?? null;
+  return {
+    statusCode: 200,
+    body: {
+      ok: true,
+      comment:
+        posted === null
+          ? { author: '', body: commentBody, createdAt: '' }
+          : {
+              author: posted.author,
+              body: posted.body,
+              createdAt: posted.createdAt.toISOString(),
+            },
+    },
+  };
+};
 export const handleIntmux = async (
   context: ConsoleOperationContext,
   body: Record<string, unknown>,

package/src/adapter/entry-points/console/consoleServer.test.ts CHANGED Viewed

@@ -533,6 +533,55 @@ describe('consoleServer new routes integration', () => {
     }
   });
+  it('posts a comment through the comment operation api', async () => {
+    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'console-server-'));
+    const issueRepository = mock<IssueRepository>();
+    issueRepository.getIssueOrPullRequestComments.mockResolvedValue([
+      {
+        author: 'HiromiShikata',
+        body: 'Thanks, this resolves the parity gap.',
+        createdAt: new Date('2026-06-18T03:21:00.000Z'),
+      },
+    ]);
+    const server = await startConsoleServer({
+      accessToken: testToken,
+      uiDistDir: path.join(tmpDir, 'ui-dist'),
+      consoleDataOutputDir: null,
+      issueRepository,
+      resolveProject: async (pjcode) =>
+        pjcode === 'umino' ? { pjcode, project: buildProject() } : null,
+      port: 0,
+    });
+    try {
+      const response = await request(
+        server,
+        'POST',
+        `/api/comment?k=${testToken}`,
+        {
+          pjcode: 'umino',
+          url: 'https://github.com/o/r/issues/1',
+          body: 'Thanks, this resolves the parity gap.',
+        },
+      );
+      expect(response.statusCode).toBe(200);
+      expect(issueRepository.createCommentByUrl).toHaveBeenCalledWith(
+        'https://github.com/o/r/issues/1',
+        'Thanks, this resolves the parity gap.',
+      );
+      expect(JSON.parse(response.body)).toEqual({
+        ok: true,
+        comment: {
+          author: 'HiromiShikata',
+          body: 'Thanks, this resolves the parity gap.',
+          createdAt: '2026-06-18T03:21:00.000Z',
+        },
+      });
+    } finally {
+      await closeServer(server);
+      fs.rmSync(tmpDir, { recursive: true, force: true });
+    }
+  });
   it('runs an operation api and records the done exclusion', async () => {
     const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'console-server-'));
     const dataDir = path.join(tmpDir, 'data');

package/src/adapter/entry-points/console/consoleServer.ts CHANGED Viewed

@@ -19,6 +19,7 @@ import {
 import {
   ConsoleOperationContext,
   ConsoleProjectResolver,
+  handleComment,
   handleIntmux,
   handleReview,
   handleTriage,
@@ -306,6 +307,8 @@ const handleOperationApi = async (
       return handleTriage(context, body);
     case '/api/intmux':
       return handleIntmux(context, body);
+    case '/api/comment':
+      return handleComment(context, body);
     default:
       return null;
   }

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsoleCommentComposer.stories.tsx ADDED Viewed

@@ -0,0 +1,30 @@
+import type { Meta, StoryObj } from '@storybook/react-vite';
+import type { ConsoleComment } from '../../logic/types';
+import { ConsoleCommentComposer } from './ConsoleCommentComposer';
+const acceptComment = async (body: string): Promise<ConsoleComment> => ({
+  author: 'HiromiShikata',
+  body,
+  createdAt: '2026-06-19T11:58:00.000Z',
+});
+const meta: Meta<typeof ConsoleCommentComposer> = {
+  title: 'Console/ConsoleCommentComposer',
+  component: ConsoleCommentComposer,
+  args: {
+    now: Date.parse('2026-06-19T12:00:00.000Z'),
+    onSubmit: acceptComment,
+  },
+};
+export default meta;
+type Story = StoryObj<typeof ConsoleCommentComposer>;
+export const IssueComposerOpen: Story = {
+  args: { isPr: false },
+};
+export const PullRequestComposerCollapsed: Story = {
+  args: { isPr: true },
+};

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsoleCommentComposer.test.tsx ADDED Viewed

@@ -0,0 +1,81 @@
+import { fireEvent, render, waitFor } from '@testing-library/react';
+import type { ConsoleComment } from '../../logic/types';
+import { ConsoleCommentComposer } from './ConsoleCommentComposer';
+jest.mock('../../lib/mermaidLoader', () => ({
+  renderMermaidToSvg: jest.fn(async () => '<svg></svg>'),
+}));
+const now = Date.parse('2026-06-19T12:00:00.000Z');
+describe('ConsoleCommentComposer', () => {
+  it('opens the form by default for an issue item', () => {
+    const { getByPlaceholderText } = render(
+      <ConsoleCommentComposer
+        isPr={false}
+        now={now}
+        onSubmit={async (body) => ({
+          author: 'HiromiShikata',
+          body,
+          createdAt: '2026-06-19T11:58:00.000Z',
+        })}
+      />,
+    );
+    expect(getByPlaceholderText('Leave a comment…')).toBeInTheDocument();
+  });
+  it('keeps the form closed by default for a pull request item', () => {
+    const { queryByPlaceholderText, getByText } = render(
+      <ConsoleCommentComposer
+        isPr
+        now={now}
+        onSubmit={async (body) => ({
+          author: 'HiromiShikata',
+          body,
+          createdAt: '2026-06-19T11:58:00.000Z',
+        })}
+      />,
+    );
+    expect(queryByPlaceholderText('Leave a comment…')).toBeNull();
+    expect(getByText('💬 Add a comment')).toBeInTheDocument();
+  });
+  it('submits the comment and shows the posted comment', async () => {
+    const onSubmit = jest.fn(
+      async (body: string): Promise<ConsoleComment> => ({
+        author: 'HiromiShikata',
+        body,
+        createdAt: '2026-06-19T11:58:00.000Z',
+      }),
+    );
+    const { getByPlaceholderText, getByText } = render(
+      <ConsoleCommentComposer isPr={false} now={now} onSubmit={onSubmit} />,
+    );
+    fireEvent.change(getByPlaceholderText('Leave a comment…'), {
+      target: { value: 'Looks good after the rebase.' },
+    });
+    fireEvent.click(getByText('Comment'));
+    await waitFor(() => {
+      expect(getByText('Looks good after the rebase.')).toBeInTheDocument();
+    });
+    expect(onSubmit).toHaveBeenCalledWith('Looks good after the rebase.');
+  });
+  it('shows a failure message when the submission rejects', async () => {
+    const { getByPlaceholderText, getByText, findByRole } = render(
+      <ConsoleCommentComposer
+        isPr={false}
+        now={now}
+        onSubmit={async () => {
+          throw new Error('HTTP 500');
+        }}
+      />,
+    );
+    fireEvent.change(getByPlaceholderText('Leave a comment…'), {
+      target: { value: 'This should fail to post.' },
+    });
+    fireEvent.click(getByText('Comment'));
+    const alert = await findByRole('alert');
+    expect(alert.textContent).toContain('HTTP 500');
+  });
+});

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsoleCommentComposer.tsx ADDED Viewed

@@ -0,0 +1,118 @@
+import { useState } from 'react';
+import {
+  formatFullTimestamp,
+  formatRelativeTime,
+} from '../../logic/relativeTime';
+import type { ConsoleComment } from '../../logic/types';
+import { ConsoleMarkdownContent } from '../content/ConsoleMarkdownContent';
+export type ConsoleCommentComposerProps = {
+  isPr: boolean;
+  now: number;
+  onSubmit: (body: string) => Promise<ConsoleComment>;
+};
+type ComposerStatus =
+  | { kind: 'idle' }
+  | { kind: 'posting' }
+  | { kind: 'error'; message: string };
+export const ConsoleCommentComposer = ({
+  isPr,
+  now,
+  onSubmit,
+}: ConsoleCommentComposerProps) => {
+  const [open, setOpen] = useState<boolean>(!isPr);
+  const [draft, setDraft] = useState<string>('');
+  const [status, setStatus] = useState<ComposerStatus>({ kind: 'idle' });
+  const [posted, setPosted] = useState<ConsoleComment[]>([]);
+  const submit = async (): Promise<void> => {
+    const body = draft.trim();
+    if (body.length === 0 || status.kind === 'posting') {
+      return;
+    }
+    setStatus({ kind: 'posting' });
+    try {
+      const comment = await onSubmit(body);
+      setPosted((previous) => [...previous, comment]);
+      setDraft('');
+      setStatus({ kind: 'idle' });
+    } catch (error) {
+      setStatus({
+        kind: 'error',
+        message: error instanceof Error ? error.message : 'failed to post',
+      });
+    }
+  };
+  return (
+    <div className="console-composer">
+      <button
+        type="button"
+        className="console-composer-toggle"
+        aria-expanded={open}
+        onClick={() => setOpen((value) => !value)}
+      >
+        {open ? '✕ Cancel' : '💬 Add a comment'}
+      </button>
+      {posted.length > 0 && (
+        <div className="console-composer-posted">
+          {posted.map((comment) => (
+            <article
+              key={`${comment.author}:${comment.createdAt}:${comment.body}`}
+              className="console-comment"
+            >
+              <header
+                className="console-comment-header"
+                title={formatFullTimestamp(comment.createdAt)}
+              >
+                <span className="console-comment-author">
+                  {comment.author === '' ? 'you' : comment.author}
+                </span>
+                <span className="console-comment-time">
+                  {formatRelativeTime(comment.createdAt, now)}
+                </span>
+              </header>
+              <ConsoleMarkdownContent body={comment.body} />
+            </article>
+          ))}
+        </div>
+      )}
+      {open && (
+        <div className="console-composer-form">
+          <textarea
+            className="console-composer-input"
+            rows={3}
+            placeholder="Leave a comment…"
+            value={draft}
+            onChange={(event) => setDraft(event.target.value)}
+          />
+          <div className="console-composer-row">
+            <button
+              type="button"
+              className="console-composer-submit"
+              disabled={status.kind === 'posting'}
+              onClick={() => {
+                void submit();
+              }}
+            >
+              Comment
+            </button>
+            {status.kind === 'posting' && (
+              <span className="console-composer-status">Posting…</span>
+            )}
+            {status.kind === 'error' && (
+              <span
+                role="alert"
+                className="console-composer-status console-composer-error"
+              >
+                Failed: {status.message}
+              </span>
+            )}
+          </div>
+        </div>
+      )}
+    </div>
+  );
+};

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsoleItemDetail.stories.tsx CHANGED Viewed

@@ -14,6 +14,7 @@ const meta: Meta<typeof ConsoleItemDetail> = {
   component: ConsoleItemDetail,
   args: {
     now: Date.parse('2026-06-19T12:00:00.000Z'),
+    commentComposer: null,
     operationBar: null,
   },
 };

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsoleItemDetail.test.tsx CHANGED Viewed

@@ -34,18 +34,35 @@ const baseProps = {
   commitsError: null,
   relatedPullRequests: [],
   now,
+  commentComposer: <div>comment-composer</div>,
   operationBar: <div>operation-bar</div>,
 };
 describe('ConsoleItemDetail', () => {
-  it('renders the PR title with the PR number, sub bar and changed files panel', () => {
+  it('renders the PR title with the PR number, sub bar and counted panels', () => {
     const { getByText, getAllByText } = render(
       <ConsoleItemDetail item={prItem} {...baseProps} />,
     );
     expect(getAllByText(`PR #${prItem.number}`).length).toBeGreaterThan(0);
-    expect(getByText('Changed files')).toBeInTheDocument();
-    expect(getByText('Commits')).toBeInTheDocument();
+    expect(
+      getByText(`Changed files (${consoleChangedFilesFixture.length})`),
+    ).toBeInTheDocument();
+    expect(
+      getByText(`Comments (${consoleCommentsFixture.length})`),
+    ).toBeInTheDocument();
+    expect(
+      getByText(`Commits (${consoleCommitsFixture.length})`),
+    ).toBeInTheDocument();
     expect(getByText('operation-bar')).toBeInTheDocument();
+    expect(getByText('comment-composer')).toBeInTheDocument();
+  });
+  it('renders the Description open link to the item url', () => {
+    const { getByText } = render(
+      <ConsoleItemDetail item={prItem} {...baseProps} />,
+    );
+    const openLink = getByText('open');
+    expect(openLink).toHaveAttribute('href', prItem.url);
   });
   it('renders the story tag and opened relative time', () => {

package/src/adapter/entry-points/console/ui/src/features/console/components/detail/ConsoleItemDetail.tsx CHANGED Viewed

@@ -52,6 +52,7 @@ export type ConsoleItemDetailProps = {
   commitsError: string | null;
   relatedPullRequests: ConsoleRelatedPullRequestView[];
   now: number;
+  commentComposer: ReactNode;
   operationBar: ReactNode;
 };
@@ -75,6 +76,7 @@ export const ConsoleItemDetail = ({
   commitsError,
   relatedPullRequests,
   now,
+  commentComposer,
   operationBar,
 }: ConsoleItemDetailProps) => {
   const resolvedState = state?.state ?? 'open';
@@ -85,6 +87,12 @@ export const ConsoleItemDetail = ({
   const statusPalette = overlayStatus
     ? colorFromEnum(overlayStatus.color)
     : null;
+  const filesCount =
+    filesAreLoading || filesError !== null ? null : files.length;
+  const commentsCount =
+    commentsAreLoading || commentsError !== null ? null : comments.length;
+  const commitsCount =
+    commitsAreLoading || commitsError !== null ? null : commits.length;
   return (
     <article className="console-detail">
@@ -164,7 +172,19 @@ export const ConsoleItemDetail = ({
         opened {formatRelativeTime(item.createdAt, now)}
       </div>
-      <ConsolePanel title="Description">
+      <ConsolePanel
+        title="Description"
+        headerAction={
+          <a
+            href={item.url}
+            className="console-panel-open-link"
+            target="_blank"
+            rel="noopener noreferrer"
+          >
+            open
+          </a>
+        }
+      >
         {bodyError !== null ? (
           <p role="alert" className="console-detail-body-error">
             Failed to load description: {bodyError}
@@ -177,7 +197,7 @@ export const ConsoleItemDetail = ({
       </ConsolePanel>
       {item.isPr && (
-        <ConsolePanel title="Changed files">
+        <ConsolePanel title="Changed files" count={filesCount}>
           <ConsoleChangedFileList
             files={files}
             isLoading={filesAreLoading}
@@ -186,7 +206,11 @@ export const ConsoleItemDetail = ({
         </ConsolePanel>
       )}
-      <ConsolePanel title="Comments" defaultCollapsed={item.isPr}>
+      <ConsolePanel
+        title="Comments"
+        count={commentsCount}
+        defaultCollapsed={item.isPr}
+      >
         <ConsoleCommentList
           comments={comments}
           isLoading={commentsAreLoading}
@@ -196,7 +220,7 @@ export const ConsoleItemDetail = ({
       </ConsolePanel>
       {item.isPr && (
-        <ConsolePanel title="Commits" defaultCollapsed>
+        <ConsolePanel title="Commits" count={commitsCount} defaultCollapsed>
           <ConsoleCommitList
             commits={commits}
             isLoading={commitsAreLoading}
@@ -223,7 +247,9 @@ export const ConsoleItemDetail = ({
           />
         ))}
-      <div className="console-detail-operations">{operationBar}</div>
+      {commentComposer}
+      <div className="console-actionbar">{operationBar}</div>
     </article>
   );
 };

package/src/adapter/entry-points/console/ui/src/features/console/components/layout/ConsolePanel.stories.tsx CHANGED Viewed

@@ -24,3 +24,28 @@ export const Collapsed: Story = {
     children: <p style={{ padding: 12 }}>Panel body content</p>,
   },
 };
+export const WithCountAndOpenLink: Story = {
+  args: {
+    title: 'Description',
+    headerAction: (
+      <a
+        href="https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/pull/851"
+        className="console-panel-open-link"
+        target="_blank"
+        rel="noopener noreferrer"
+      >
+        open
+      </a>
+    ),
+    children: <p style={{ padding: 12 }}>Panel body content</p>,
+  },
+};
+export const ChangedFilesWithCount: Story = {
+  args: {
+    title: 'Changed files',
+    count: 3,
+    children: <p style={{ padding: 12 }}>Panel body content</p>,
+  },
+};

package/src/adapter/entry-points/console/ui/src/features/console/components/layout/ConsolePanel.test.tsx CHANGED Viewed

@@ -20,6 +20,33 @@ describe('ConsolePanel', () => {
     expect(queryByText('content')).toBeNull();
   });
+  it('appends the count to the title when provided', () => {
+    const { getByText } = render(
+      <ConsolePanel title="Changed files" count={3}>
+        <p>content</p>
+      </ConsolePanel>,
+    );
+    expect(getByText('Changed files (3)')).toBeInTheDocument();
+  });
+  it('omits the count when it is null', () => {
+    const { getByText } = render(
+      <ConsolePanel title="Description" count={null}>
+        <p>content</p>
+      </ConsolePanel>,
+    );
+    expect(getByText('Description')).toBeInTheDocument();
+  });
+  it('renders the header action node', () => {
+    const { getByText } = render(
+      <ConsolePanel title="Description" headerAction={<a href="/x">open</a>}>
+        <p>content</p>
+      </ConsolePanel>,
+    );
+    expect(getByText('open')).toBeInTheDocument();
+  });
   it('toggles collapsed state on the header button', () => {
     const { getByRole, queryByText } = render(
       <ConsolePanel title="Comments" defaultCollapsed>

package/src/adapter/entry-points/console/ui/src/features/console/components/layout/ConsolePanel.tsx CHANGED Viewed

@@ -2,6 +2,7 @@ import { type ReactNode, useState } from 'react';
 export type ConsolePanelProps = {
   title: string;
+  count?: number | null;
   defaultCollapsed?: boolean;
   headerAction?: ReactNode;
   children: ReactNode;
@@ -9,11 +10,13 @@ export type ConsolePanelProps = {
 export const ConsolePanel = ({
   title,
+  count = null,
   defaultCollapsed = false,
   headerAction,
   children,
 }: ConsolePanelProps) => {
   const [collapsed, setCollapsed] = useState<boolean>(defaultCollapsed);
+  const heading = count === null ? title : `${title} (${count})`;
   return (
     <section className="console-panel">
       <header className="console-panel-header">
@@ -24,7 +27,7 @@ export const ConsolePanel = ({
           onClick={() => setCollapsed((value) => !value)}
         >
           <span className="console-panel-caret">{collapsed ? '▸' : '▾'}</span>
-          <span className="console-panel-title">{title}</span>
+          <span className="console-panel-title">{heading}</span>
         </button>
         {headerAction !== undefined && (
           <div className="console-panel-action">{headerAction}</div>