Skip to content

feat(report): add user instruction input and UI improvements (#612)#613

Merged
Kewton merged 2 commits intodevelopfrom
feature/612-report-ui
Apr 2, 2026
Merged

feat(report): add user instruction input and UI improvements (#612)#613
Kewton merged 2 commits intodevelopfrom
feature/612-report-ui

Conversation

@Kewton
Copy link
Copy Markdown
Owner

@Kewton Kewton commented Apr 2, 2026

Summary

  • レポート生成時にユーザー指示を入力できる複数行テキストボックスを追加
  • Review画面のタブ名を「Review/Report」に変更
  • Sessions画面の表示順序をリポジトリ名→ブランチ名に修正
  • summary-prompt-builder にユーザー指示を安全に組み込む機能を追加

Closes #612

Test plan

  • lint / tsc / test:unit / build 全パス(6037 tests passed)
  • ユーザー指示がAIプロンプトに反映される
  • Review/Reportタブ名が正しく表示される

🤖 Generated with Claude Code

Kewton and others added 2 commits April 2, 2026 14:02
- Add userInstruction textarea to ReportTab for customizing AI summary generation
- Propagate userInstruction through API -> generator -> prompt builder pipeline
- Add prompt injection isolation rules in system prompt for <user_instruction>
- Apply sanitizeMessage to userInstruction in buildSummaryPrompt (DR1-005)
- Add body shape validation and userInstruction validation to daily-summary API
- Change nav label from 'Review' to 'Review/Report' (Header + GlobalMobileNav)
- Change Sessions default sort to repositoryName(asc) with branchName second sort
- Add second sort key for status sort (repositoryName)
- Add MAX_USER_INSTRUCTION_LENGTH=1000 constant to review-config
- All 6037 unit tests pass, 0 ESLint errors, 0 TypeScript errors

Resolves #612

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Extract serializeReport() in daily-summary route to eliminate 3x
  duplicated report-to-JSON serialization (DRY)
- Extract STATUS_LABELS record and STATUS_BADGE_CLASSES in Sessions page
  to replace switch statement and nested ternary chain (readability)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Kewton Kewton merged commit df495ca into develop Apr 2, 2026
5 checks passed
@Kewton Kewton deleted the feature/612-report-ui branch April 2, 2026 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant