feat: session-end reflection を qa_schema 3 の Transferable Principles として inline する (Phase 1) - #6
Merged
Conversation
…iples Fix the structural skip where agents close a session recording only what was fixed, never asking whether the session produced a learning that outlives it (the "matches an existing pattern, no new intent needed" rationalization observed downstream). - qa_schema 3 = 2 + a required Transferable Principles section on verification docs; legacy 2 stays accepted, test-plan requirements are unchanged. - validate-qa accepts candidates or a reasoned "None: <reason>" only; a blank section or bare None fails, so "considered, nothing found" is structurally distinct from a silent skip. Content quality stays with user review, mirroring the enforcement level of intent-code traceability. - The verification template ships the section as comment-only guidance, so an untouched copy fails validation and the reflection must actually be written. - post-implementation / qa-review inline the reflection step; the prompt asks "what made you stop and think why" instead of "write an intent" so the record extends thinking rather than adding ceremony. Promotion to intent is gated on the user. - Cross-cutting principles promote to _docs/intent/<Area>/conventions/decision.md per the new promotion rule in documentation_operations, with provenance back to the verification. - New eval case transferable-principle-skip reproduces the skip so the step can be regression-checked against agent behavior, not anecdotes. Closes #1 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1
何を変えるか
agent が「既存 pattern と一致するから新規 intent 不要」と inline で正当化し、session で得た transferable principle の昇格検討そのものを skip する構造的な抜け穴を塞ぐ。fix の説明 (what) は書かれるが一般化された学び (why generalized) が残らない、という downstream で観測された failure mode への対応。
設計判断
Transferable Principles必須。legacy 2 は受理継続、test-plan の要件は不変。既存文書の一斉移行は要求しない。- TP: ...) または理由付きNone: <reason>のみ受理。空欄と裸のNoneは fail。これにより「検討したが無かった」と「無言の skip」が構造的に区別される。中身の質は user review の領分 (intent ↔ code traceability の強制レベルと同じ哲学)。_docs/intent/<Area>/conventions/decision.mdに集約 (documentation_operations 昇格ルール 5)。昇格 DEC は昇格元 verification への provenance を references に残す。transferable-principle-skipを追加: 昇格 skip を再現するシナリオで、この step が agent 挙動を変えるかを逸話ではなく回帰確認できるようにする。検証
./scripts/check-docs.shPASS (fmt / check / 全 validator / fixture 回帰 / hook test / smoke)missing heading/must record candidates or explicit None)🤖 Generated with Claude Code