Repository navigation
feat: grilling HANDOFF writes a derived technical spec - #55
Merged
Merged
Conversation
Record confirmed grilling consensus on GitHub issues for tracking. Do not treat the Shared-Understanding Record as the spec; write a separate technical specification, wait for spec-file review, then writing-plans. Leave skills/grilling/SKILL.md unchanged. Co-authored-by: ficehuang <ficehuang@gmail.com>
After the Shared-Understanding Record is confirmed, write a separate technical specification (not a dump of the record), wait for spec-file review, then writing-plans. Slim AGENTS.md to repo issue tracking only. Co-authored-by: ficehuang <ficehuang@gmail.com>
1 task done
This was referenced Oct 14, 2026
Merged
cursor Bot
added a commit
that referenced
this pull request
Oct 14, 2026
…4bb1 Release v6.6.0 notes: product commits are #55
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 #49
Who is submitting this PR? (required)
What problem are you trying to solve?
Plugin users of Wukong Code read
skills/grilling/SKILL.md, not this repository’sAGENTS.md. After grilling, agents were dumping the in-chat Shared-Understanding Record intodocs/wukong-code/specs/as if that eight-section confirmation record were the spec (the #32 contract), while this repo’sAGENTS.mdlater forbade writing specs at all (#50) and put tracking on GitHub issues. 悟空非空也g, in a live session, rejected both: the confirmation record is not the spec; after the record is confirmed, write a separate technical specification derived from it, have the human review that spec file, thenwriting-plans. Issue/board tracking stays a habit of this repo inAGENTS.md/CLAUDE.md, not a skill capability.This is not a theoretical cleanup. The failure mode was agents treating the interview transcript dump as the design document, or skipping a spec because AGENTS.md said “issue not spec.”
What does this PR change?
skills/grilling/SKILL.mdHANDOFF now writes a derived six-section technical spec (not a paste of the eight Completion Gate headings), waits for spec-file review, and loadswriting-plansonly after that file is approved.AGENTS.md/CLAUDE.mdgrilling section is slimmed to this-repo issue tracking and a pointer at the skill. Resident probes intests/skills/grilling-scenarios.mdand a Cursor isolated RED/GREEN note underdocs/wukong-code/evals/2026-10-13-grilling-derived-spec-handoff.mdmatch the new contract.Is this change appropriate for the core library?
Yes.
grillingis already a core process skill. Plugin users in any project that loads the skill need the HANDOFF sequence; they will never read this repo’sAGENTS.md. Scope, provenance, and maintenance stay with Wukong Code process skills. Zero new dependencies, no third-party runtime, no GitHub Issues/Projects in the skill. Evidence is the isolated RED/GREEN below, proportionate to a HANDOFF wording change (not a new skill).What alternatives did you consider?
AGENTS.mdand leave the skill dumping the record (feat: persist confirmed grilling records as specs before writing-plans #32). Rejected by 悟空非空也g: future plugin users read the skill.AGENTS.md). Rejected: confirmation record and issue tracking are not the spec; a separate technical specification is required beforewriting-plans.skills/grilling/SKILL.md. Rejected: that is this repository’s development habit only.Does this PR contain multiple unrelated changes?
No. Skill HANDOFF, matching scenario text, this-repo AGENTS slim so it does not contradict the skill, and the eval note for that HANDOFF change. No harness, language pack, or unrelated docs.
Existing PRs
#32 (merged) persisted the confirmed eight-section record as the spec. This PR keeps persist-then-file-review-then-writing-plans but forbids treating the record as the spec and requires a derived technical specification. #50 (merged) recorded grilling consensus on issues and forbade new specs in this repo; that AGENTS override is removed so it does not fight the skill. #13 added grilling. No open PR covers this derived-spec HANDOFF.
Environment tested
New harness support (required if this PR adds a new harness)
This PR does not add a new harness. Transcript below is N/A.
Clean-session transcript for "Let's make a react todo list"
Evaluation
悟空非空也g decided grilling HANDOFF belongs in the skill: 确认记录 → 根据确认后的记录写 spec 技术规格文档(不要把确认记录自己当成 spec 文档)→ 审核 spec 文档 → 再 writing-plans.
Three isolated GREEN probes (M4/M5 derived spec + file review; M6
按这个写计划then writing-plans; M7 early-stop). Matching RED: M4 dumped the eight-section record as the spec; M7 already passed (no file). Officialwukong-code-evalsgrilling cohort: not run (repo has JS/TS jsonl only on the language-guidance branch; no API key). Do not treat this as a Drill/tmux result.Before: HANDOFF copied Objective…Success Criteria into the spec file. After: spec used Problem and objective / Architecture / Interfaces / Constraints / Errors / Acceptance; asked for review of that file; writing-plans only after spec-file approval; early-stop still wrote nothing. Details:
docs/wukong-code/evals/2026-10-13-grilling-derived-spec-handoff.md.Language-pack evidence
Rigor
wukong-code:writing-skillsandcompleted adversarial pressure testing (paste results below)
rationalizations, "human partner" language) without extensive evals
showing the change is an improvement
RED (current skill, new contract): M4 actor
bc-6dd09e92-fff4-5d84-aed3-c4757f76884awrote a spec whose outline was the eight Completion Gate headings. M7 actorbc-30288003-776b-5222-acb1-cd0b217b4290stopped with a partial record and wrote no file.GREEN (candidate skill): M4/M6 actor
bc-89eb817f-89b3-53d1-a6d1-d558430a991ewrote/tmp/grilling-green-iso-handoff/docs/wukong-code/specs/2026-10-13-1205-in-process-request-deduplication-design.mdwith the six derived sections, reported commit failure (isolated tree is not a git repo), asked for spec-file review; on按这个写计划loaded writing-plans. M7 actorbc-475400e1-f5b9-546a-850c-e4f3d0d75a24wrote none.S1–S5 Turn Contract and U1 router probes were not re-run (HANDOFF-only). Isolated
/tmptrees, not session-start bootstrap.Human review
悟空非空也g reviewed the work on branch
cursor/grilling-spec-handoff-9b62and said 合入.