リリース: feedback-autofixの固定先更新と空本文フィードバックの拒否

.github/workflows/auto_fix_from_feedback.yml

@@ -67,7 +67,7 @@ jobs:
           fetch-depth: 0
           persist-credentials: false
 
-      - uses: TrainLCD/feedback-autofix/prepare@406de875a89dd1e640f8b66b71d4de9516a952ca # feedback-autofix#1
+      - uses: TrainLCD/feedback-autofix/prepare@c71e2b4abf4149b4b83e9a87cd185b1558ed8237 # feedback-autofix#4
         id: prepare
         with:
           stub_issue_number: ${{ github.event.issue.number }}
@@ -141,8 +141,12 @@ jobs:
       # always() を外さないこと。エージェントが失敗した場合こそ報告が要る。
       # 何も言わずに終わると、管理チケットを見た人には「調べたうえで何もしな
       # かった」のか「そもそも動かなかった」のかが分からない。
-      - uses: TrainLCD/feedback-autofix/report@406de875a89dd1e640f8b66b71d4de9516a952ca # feedback-autofix#1
-        if: ${{ always() && steps.prepare.outputs.eligible == 'true' }}
+      #
+      # failed も外さないこと。prepare が対象だと分かったあとで落ちた場合、
+      # eligible は 'true' になりません。この条件が無いとここが走らず、ジョブが
+      # 赤くなるだけで管理チケットには何も残りません。
+      - uses: TrainLCD/feedback-autofix/report@c71e2b4abf4149b4b83e9a87cd185b1558ed8237 # feedback-autofix#4
+        if: ${{ always() && (steps.prepare.outputs.eligible == 'true' || steps.prepare.outputs.failed == 'true') }}
         with:
           issue_number: ${{ steps.prepare.outputs.issue_number }}
           issues_repo_token: ${{ secrets.ISSUES_REPO_TOKEN }}

src/routes/feedback.test.ts

@@ -0,0 +1,77 @@
+import { CallableError } from '../lib/callable';
+import type { Env } from '../types';
+import { handleFeedback } from './feedback';
+
+jest.mock('../lib/auth/session', () => ({
+  verifySessionToken: jest.fn(async () => 'verified-uid'),
+}));
+
+const send = jest.fn(async (_message: unknown) => undefined);
+const env = { FEEDBACK_QUEUE: { send } } as unknown as Env;
+
+const buildRequest = (report: unknown): Request =>
+  new Request('https://example.com/postFeedback', {
+    method: 'POST',
+    headers: {
+      'content-type': 'application/json; charset=UTF-8',
+      Authorization: 'Bearer dummy',
+    },
+    body: JSON.stringify({ data: { report } }),
+  });
+
+const baseReport = {
+  id: 'feedback-id',
+  reportType: 'feedback',
+  description: '遅い',
+  reporterUid: 'client-claimed-uid',
+};
+
+describe('handleFeedback', () => {
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  // 文字数の下限は撤廃したため、短い本文もそのままキューへ流す
+  it('queues a short report', async () => {
+    const res = await handleFeedback(buildRequest(baseReport), env);
+
+    expect(res.status).toBe(200);
+    expect(send).toHaveBeenCalledTimes(1);
+    expect(send.mock.calls[0][0]).toMatchObject({
+      id: 'feedback-id',
+      report: { description: '遅い', reporterUid: 'verified-uid' },
+    });
+  });
+
+  // code は HTTP ステータスへ直結する（invalid-argument なら 400）。CallableError で
+  // あることしか見ていないと、internal へ変わって 500 を返すようになっても気づけない。
+  // アプリは 4xx と 5xx で扱いを変えられるため、コードまで固定する。
+  it.each([
+    ['empty', ''],
+    ['whitespace only', '  \n\t '],
+    ['number', 1],
+    ['null', null],
+  ])(
+    'rejects a %s description without queueing',
+    async (_label, description) => {
+      const rejected = handleFeedback(
+        buildRequest({ ...baseReport, description }),
+        env
+      );
+
+      await expect(rejected).rejects.toThrow(CallableError);
+      await expect(rejected).rejects.toMatchObject({
+        code: 'invalid-argument',
+      });
+      expect(send).not.toHaveBeenCalled();
+    }
+  );
+
+  it('rejects a missing description without queueing', async () => {
+    const rejected = handleFeedback(buildRequest({ id: 'feedback-id' }), env);
+
+    await expect(rejected).rejects.toThrow(/report.description required/);
+    await expect(rejected).rejects.toMatchObject({ code: 'invalid-argument' });
+    expect(send).not.toHaveBeenCalled();
+  });
+});

src/routes/feedback.ts

@@ -19,6 +19,11 @@ export const handleFeedback = async (
   if (!report?.id) {
     throw new CallableError('invalid-argument', 'report.id required');
   }
+  // 本文が空白のみのフィードバックはトリアージへ流さない。文字数の下限は設けず、
+  // 短い本文やクラッシュレポートの短いエラーメッセージはそのまま受け付ける。
+  if (typeof report.description !== 'string' || !report.description.trim()) {
+    throw new CallableError('invalid-argument', 'report.description required');
+  }
 
   // reporterUid はクライアント申告を信用せず、検証済みトークンの sub で上書きする。
   // （他ユーザーの UID を名乗って Issue/Discord に載せるなりすましを防ぐ）