Skip to content

ci(review): 非 suggestion 指摘の分類・task化コアを追加#2378

Merged
ootakazuhiko merged 3 commits intomainfrom
feat/2373-review-comment-classifier
Mar 2, 2026
Merged

ci(review): 非 suggestion 指摘の分類・task化コアを追加#2378
ootakazuhiko merged 3 commits intomainfrom
feat/2373-review-comment-classifier

Conversation

@ootakazuhiko
Copy link
Copy Markdown
Collaborator

概要

変更内容

  • scripts/ci/lib/review-comment-classifier.mjs(新規)
    • extractSuggestionBlocks:
      • review comment 本文から suggestion ブロックを抽出
    • classifyReviewCommentBody:
      • suggestion / actionable / informational のルールベース分類
    • buildActionTaskFromComment:
      • actionable comment から deterministic な task オブジェクトを生成
    • summarizeActionTasks:
      • task 一覧を PR コメント向けの要約行へ変換
  • tests/unit/ci/review-comment-classifier.test.ts(新規)
    • suggestion 抽出
    • 分類ルール
    • task 生成
    • 要約出力

テスト

  • pnpm vitest run tests/unit/ci/review-comment-classifier.test.ts

Refs: #2373

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

❓ Code Generation Drift Detection

Status:
Regeneration needed: No


Automated by AE-Framework Codegen

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

Quality Summary

  • Adapters:

  • Formal: n/a

  • Replay: n/a

  • Trace IDs:

Harness Health

  • severity: warn
  • source: workflow=PR Maintenance, runId=22562712322, sha=4157bd4047c88c46ec073a22c7b4708b014f724b
Gate Status Checks
artifactsSchema skip 0
testingHarness warn 1
contextPack skip 0
runtimeConformance skip 0
ciExtended skip 1

Change Package

  • risk=risk:low (inferred=risk:low) | files=0 | areas=unknown | evidence=1/4 present/missing
  • required labels: (none) | missing: (none)
  • reproducibility: pnpm run verify:lite

Change Package Validation

  • result: WARN
  • strict: false
  • schema: PASS
  • required evidence: verifyLiteSummary
  • missing required evidence: verifyLiteSummary
  • evidence present/missing(actual): 1/4
  • warnings:
    • missing required evidence: verifyLiteSummary

@github-actions github-actions bot added the enforce-testing Make testing DDD scripts strict/blocking label Mar 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

Progress Summary

  • Missing: metrics, quality, traceability, phaseState

@github-actions github-actions bot added the risk:low Low risk PR (auto-merge candidate) label Mar 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

Generate Artifacts Preview

Generated at: 2026-03-02T05:28:17.803Z

  • tests/api/generated: clean
  • artifacts/codex: clean
  • artifacts/spec: clean

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

CodeX Artifacts Summary

  • • Contract/E2E templates: 1 files (dir: tests/api/generated)
  • • Tests: PBT files=123, BDD features=0
  • • Formal specs: TLA=1, Alloy=1

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 09c9575fec

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

KvOnce Trace Validation

  • OTLP: ✅ Success (Issues: 0)
  • NDJSON: ✅ Success (Issues: 0)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

suggestion のレビュー指摘を自動処理ループに載せるための基盤として、レビューコメント本文のルールベース分類と actionable 指摘の task 化(+ 要約出力)を追加するPRです(#2373 / PR-1 相当)。

Changes:

  • レビューコメント本文から suggestion ブロックを抽出する処理を追加
  • 本文を suggestion / actionable / informational に分類し、actionable から deterministic な task を生成するコアを追加
  • 上記の単体テスト(抽出/分類/task生成/要約)を追加

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
scripts/ci/lib/review-comment-classifier.mjs suggestion 抽出・本文分類・actionable task 生成・task 要約のコア実装を追加
tests/unit/ci/review-comment-classifier.test.ts classifier の主要機能に対する unit test を追加

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

@artifacts/quality/comment-body.txt

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

❓ Code Generation Drift Detection

Status:
Regeneration needed: No


Automated by AE-Framework Codegen

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

CodeX Artifacts Summary

  • • Contract/E2E templates: 1 files (dir: tests/api/generated)
  • • Tests: PBT files=123, BDD features=0
  • • Formal specs: TLA=1, Alloy=1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

CI Status Snapshot (2026-03-02T05:21:56.225Z)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

@artifacts/quality/comment-body.txt

@ootakazuhiko
Copy link
Copy Markdown
Collaborator Author

レビュー本文・インラインコメントを全件確認し、以下を実施しました。

  • 本文レビュー:
    • Codex/Copilot本文は概要確認中心で、追加の修正要求はありませんでした。
  • インラインコメント:
    • 5件すべてに返信済み。
    • 必要な修正を反映(混在コメントでactionable優先、commentId欠落時fail-closed、line範囲正規化、追加テスト)。
    • 返信後に全threadをresolve済み。
  • CI:
    • Copilot Review Gate の過去失敗runを再実行し、required checksがすべてpassであることを確認しました。

追加で確認が必要な点があれば指示ください。

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

❓ Code Generation Drift Detection

Status:
Regeneration needed: No


Automated by AE-Framework Codegen

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

CodeX Artifacts Summary

  • • Contract/E2E templates: 1 files (dir: tests/api/generated)
  • • Tests: PBT files=123, BDD features=0
  • • Formal specs: TLA=1, Alloy=1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

@artifacts/quality/comment-body.txt

@ootakazuhiko ootakazuhiko merged commit dc7a3a4 into main Mar 2, 2026
82 checks passed
@ootakazuhiko ootakazuhiko deleted the feat/2373-review-comment-classifier branch March 2, 2026 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enforce-testing Make testing DDD scripts strict/blocking risk:low Low risk PR (auto-merge candidate)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants