docs(skill-usage): three more rule failures from the panel round (#160) - #161
Open
xenodeve wants to merge 1 commit into
Open
docs(skill-usage): three more rule failures from the panel round (#160)#161xenodeve wants to merge 1 commit into
xenodeve wants to merge 1 commit into
Conversation
The session entry now records eight rules that did not hold. The three added
here all come from one clink-brainstorm round on 2026-08-13, and they compound:
- The prompt described the reviewer's job as "whether the rules were
followed", which does not separate process from outcome. All three
panellists read it as outcome, correctly concluded that session history
cannot establish outcome, and unanimously recommended a design the
developer rejected in one sentence. The prompt was not vague — it carried
measured constraints, paths, three named options and a word count. One noun
with two readings was enough. Filed as #160, distinct from #131: that one
supplied an answer, this one bounded nothing.
- The same prompt handed the panel a 108-second transcript lag as a verified
constraint. It was one sample carried in from an earlier workflow and never
re-measured. The real distribution over 7,268 records is p50 0.2s, p90
15.7s, with 3.2% of gaps over a minute. Two panellists used the tail figure
as the deciding reason to refuse a layer.
- PRD #159 stated the master "may not answer with silence" as a rule of the
mechanism. A non-blocking hook cannot compel a response, so nothing
enforces it. A panellist caught that independently.
Recorded rather than summarised away, because the entry's value is the
embarrassing cases: a log of only the memorable sessions is the failure-selected
sample this layer exists to avoid.
T4-Gates: simplify=n-a code-review=n-a scrutinize=n-a security-review=n-a verify=ran
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CujMUuq14tpfU36ymaEhHi
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
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.
Refs #160
Three more rule failures appended to this session's skill-usage entry. All three came from one
clink-brainstormround, and they compound rather than sitting side by side.The prompt bounded nothing. It described the reviewer's job as "whether the invoked skills' rules were followed" — a phrase that does not separate process from outcome. All three panellists read it as outcome, correctly concluded that session history cannot establish outcome, and unanimously recommended a design the developer rejected in one sentence. The prompt was not vague: it carried measured constraints, file paths, three named options and a word count. One noun with two readings was enough. Filed as #160, and deliberately distinct from #131 — that one supplied an answer, this one supplied nothing and bounded nothing.
The same prompt handed the panel a figure I had not measured. It stated the transcript lags 108 seconds, as a verified constraint. That was one sample carried in from an earlier workflow. Measured properly across 7,268 timestamped records:
p50 0.2s · p90 15.7s · p99 189.7s, with 230 of 7,267 gaps over a minute. Two panellists used the tail figure as the deciding reason to refuse a layer. The defect is not the number — it is passing on someone else's measurement with my own confidence attached.PRD #159 claimed enforcement that does not exist. It said the master agent "may not answer with silence" as though it were a rule of the mechanism. A non-blocking hook cannot compel a response. A panellist caught it independently of everything above, and it is the same defect this repository documents as its worst failure mode.
Recorded rather than summarised away. The entry's value is the embarrassing cases — a log of only the memorable sessions is exactly the failure-selected sample this layer exists to avoid.
bash tests/hooks/run-all.sh→ ALL TESTS PASSED.สรุปภาษาไทย
Refs #160
เพิ่มกฎที่ไม่ถูกทำตามอีกสามข้อลงในรายการ skill-usage ของ session นี้ · ทั้งสามมาจากรอบ
clink-brainstormรอบเดียว และมันทบกัน ไม่ได้อยู่แยกกันprompt ไม่ได้กำหนดขอบเขตอะไรเลย · มันอธิบายหน้าที่ของ review agent ว่า "กฎของ skill ที่ถูกเรียกถูกทำตามไหม" — วลีที่ไม่ได้แยกกระบวนการออกจากผลลัพธ์ · panel ทั้งสามอ่านเป็นผลลัพธ์ สรุปถูกต้องว่าประวัติ session พิสูจน์ผลลัพธ์ไม่ได้ แล้วเสนอเป็นเอกฉันท์ให้ใช้ดีไซน์ที่ dev ปฏิเสธด้วยประโยคเดียว · prompt ไม่ได้คลุมเครือ มันมีข้อจำกัดที่วัดมาแล้ว มี path มีสามทางเลือกที่ตั้งชื่อไว้ และมีจำนวนคำ · คำนามคำเดียวที่อ่านได้สองแบบก็พอแล้ว · บันทึกเป็น #160 และตั้งใจแยกจาก #131 เพราะอันนั้นยื่นคำตอบให้ ส่วนอันนี้ไม่ได้ยื่นอะไรเลยและไม่ได้กำหนดขอบเขตอะไรเลย
prompt เดียวกันยื่นตัวเลขที่ผมไม่ได้วัดเองให้ panel · มันบอกว่า transcript ช้า 108 วินาที ในฐานะข้อจำกัดที่ตรวจแล้ว · นั่นเป็นตัวอย่างเดียวที่ยกมาจาก workflow รอบก่อน · วัดจริงจากเรคอร์ดที่มี timestamp 7,268 อัน:
p50 0.2 วินาที · p90 15.7 · p99 189.7โดยมี 230 จาก 7,267 ช่องว่างที่เกินหนึ่งนาที · สองใน panel ใช้ค่าหางนั้นเป็นเหตุผลชี้ขาดในการปฏิเสธชั้นหนึ่ง · ข้อบกพร่องไม่ใช่ตัวเลข แต่คือการส่งต่อการวัดของคนอื่นโดยติดความมั่นใจของตัวเองไปด้วยPRD #159 อ้างการบังคับใช้ที่ไม่มีอยู่จริง · มันเขียนว่า masteragent "ตอบด้วยความเงียบไม่ได้" เหมือนเป็นกฎของกลไก · hook ที่ไม่บล็อกบังคับให้เกิดคำตอบไม่ได้ · มี panellist จับได้เองโดยไม่เกี่ยวกับทุกอย่างข้างบน และมันคือข้อบกพร่องแบบเดียวกับที่ repo นี้บันทึกไว้ว่าเป็นความล้มเหลวที่แย่ที่สุดของมัน
บันทึกไว้แทนการสรุปทิ้ง · คุณค่าของรายการนี้อยู่ที่เคสที่น่าอาย — log ที่มีแต่ session ที่น่าจดจำ คือตัวอย่างที่คัดจากความล้มเหลวซึ่งชั้นนี้มีไว้เพื่อหลีกเลี่ยงพอดี
bash tests/hooks/run-all.sh→ ALL TESTS PASSED