Skip to content

feat(workflow): マイグレーション振り返りフィードバックの一般化改善#15

Merged
fecot merged 1 commit into
mainfrom
feature/workflow-feedback-improvements
May 29, 2026
Merged

feat(workflow): マイグレーション振り返りフィードバックの一般化改善#15
fecot merged 1 commit into
mainfrom
feature/workflow-feedback-improvements

Conversation

@fecot
Copy link
Copy Markdown
Owner

@fecot fecot commented May 29, 2026

概要

マイグレーション案件の振り返りフィードバックのうち、特定プロジェクト固有でない一般化可能な改善 をワークフローに反映する。フィードバックは AngularJS→React 書き換えに由来するが、その依存部分は意図的に除外し、UI 変更・開発プロセス全般に効く形に一般化した。

変更内容

フィードバック 対応 反映先
受入基準の明示承認 列挙だけでは進まず、人間が明示承認するまで停止する Stop Condition 化 workflows(Phase 1) / product-interpreter
視覚仕様レビューゲート UI 変更全般で、Phase 4 に視覚仕様スケッチ(ASCII/mock)作成 → Phase 5 前に人間合意するゲートを新設 workflows(Phase 4) / implementation-driver / run-feature-workflow / safe-implement
Known Risks 蓄積 Phase 8 に振り返りを追加。「次回また踏む罠」を Known Risks 追記提案する運用 workflows(Phase 8) / release-captain / project-rules-template
タスク粒度 追跡は Phase 単位まで、Phase 内ステップは計画書/実装ログで管理 workflows / run-feature-workflow
現状スケッチ必須化(補強) Migration Phase 2 に「全画面スクショ+可視要素インベントリ」を必須化(要素丸ごと見落とし防止) migration-spec-capture / workflows(§6.1)
SCSS パス特定(補強) 「ロジックと CSS の両方を Discovery(CSS を後回しにしない)」を明記 workflows(§6.1)
キャッシュ対策(補強) キャッシュバイパスを no-cacheno-store に強化 browser-verification

設計判断

意図的に追加しなかったもの(Angular→React 固有)

  • 具体パス(html/member/src/scss/...)、nginx 固有のキャッシュ設定、c3→chart.js 等の特定ライブラリ対応表 — 一般リポジトリに適用できないため除外。

🤖 Generated with Claude Code

… feedback

マイグレーション振り返りフィードバックのうち、特定プロジェクト固有でない
一般化可能な改善を反映する。

- 視覚仕様レビューゲート(Phase 4→5): UI変更全般で実装前にスケッチ合意
- Known Risks 蓄積運用(Phase 8): 振り返りで次回防ぐべき罠を追記提案
- 受入基準の明示承認を Stop Condition 化(Phase 1)
- タスク粒度の指針: 追跡は Phase 単位まで
- Migration Phase 2 補強: 全画面スクショ+可視要素インベントリを必須化
- browser-verification: キャッシュバイパスを no-store に強化

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@fecot fecot merged commit c5a7ed7 into main May 29, 2026
@fecot fecot deleted the feature/workflow-feedback-improvements branch May 29, 2026 13: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