Repository navigation
feat: append Product Design composition pointer on bootstrap - #48
Merged
Merged
Conversation
When skills/product-design/SKILL.md exists, SessionStart, OpenCode, Pi, and Kimi append the same short delivery-layer pointer: process skills stay primary, Product Design is secondary, conflicts keep process gates. Co-authored-by: 悟空非空也 <1390128154@qq.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.
Who is submitting this PR? (required)
cursor-grok-4.6-xhigh-fast)bc-bd1c37b4-8cfd-5e03-95f4-867927b16d0d; Project coordinatorbc-00bc4604-2af3-4dc7-9e8c-350a7f19ede4)550ab1884cce3a0c04a931b05ed20db9865eec6c, 15 files, +491/−17) in the Project chat on 2026-10-04 and then said to open this PR.What problem are you trying to solve?
Session bootstrap injects
using-wukong-code(plus optional harness tool mapping) and never mentions Product Design. Process-primary / PD-secondary / keep process gates lives only inreferences/wukong-product-design-composition.md, which the model must find on its own. After language-guidance got an injected Secondary domain slot, PD — the other secondary domain in this fork — still had no delivery-layer pointer.This is leftover architecture item 4, reconfirmed as item 2 in the 2026-10-04 review on
origin/main(thenbcadfde; this branch starts from8146c8aafter #47). Not a live “Let's make a react todo list” failure. The user experience: a “do a feature / add disabled to Button” prompt can raceproduct-designagainst process skills with no forced composition rule.What does this PR change?
When
skills/product-design/SKILL.mdexists, SessionStart, OpenCode, Pi, and Kimi append the same short sentence at the end of bootstrap: process skills stay primary, Product Design is secondary, conflicts keep process gates, seereferences/wukong-product-design-composition.md. Shared text isskills/using-wukong-code/references/product-design-composition-pointer.md. No Scope table, PD skill, Red Flags, or “human partner” edits. PD stays in this plugin.Is this change appropriate for the core library?
Yes. This is the existing bootstrap delivery layer in this fork, which already ships 10 PD skills in the same plugin.
using-wukong-codealready live in core; a second plugin cannot be the session-start pointer those injectors append.tests/hooks/test-product-design-bootstrap-pointer.shasserts present when PD exists, absent when it does not (fixture), and thatusing-wukong-code/SKILL.mdstill has zero Product Design routing. SessionStart / OpenCode / Pi / Kimi / canonical mapping / PD integrity suites passed. No live acceptance-test session.What alternatives did you consider?
using-wukong-code/SKILL.mdlike language-guidance. Rejected for this PR: that is behavior-shaping skill prose and needs evals. The report said injector pointer first.Does this PR contain multiple unrelated changes?
No. Shared pointer file, four injectors, Kimi
skillInstructionsregen, and tests that lock present/absent are one delivery-layer change. No openai.yaml, Codex archive-list, or marketplace description work.Existing PRs
No open PR on
cursor/pd-bootstrap-pointer-6d0d. No closed PR appends this composition pointer. #47 is process-skill yaml. #36 is PD transport paths. #26 folded frontend-design into PD. #33 is harness tool mappings.Environment tested
tests/hooks/test-product-design-bootstrap-pointer.sh,test-session-start.sh,test-tool-mapping-canonical.sh, OpenCode / Pi / Kimi / Cursor / PD integrity / language-guidance / language-router / Codex package / marketplace / shell-lint fixture / skill-slim-gates)cursor-grok-4.6-xhigh-fastscripts/test.sh --suite corepreflight failed on that VM (rsyncmissing, needed only by Codex sync tests).scripts/lint-shell.shnot run (shellcheckabsent).bash -non touched shell.New harness support (required if this PR adds a new harness)
Not applicable. This PR does not add a new harness.
Clean-session transcript for "Let's make a react todo list"
Evaluation
the session that led to this change?
After merging item 1 (#47), the human partner said to implement item 2: append a PD composition pointer at the end of bootstrap.
Zero live harness sessions. After the change we ran the static suites listed above.
Before: bootstrap never mentioned Product Design; composition lived only in a file the model had to discover. After: all four injectors append the same sentence when PD is installed, and omit it when a fixture removes
skills/product-design/SKILL.md.using-wukong-codeScope still has zero PD routing. We did not measure live brainstorming vs product-design trigger rates.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
This is injector wiring plus a one-sentence pointer file, not a skill-prose rewrite.
writing-skillswas not used. No Red Flags / “human partner” / Scope-table edits. Coverage is present/absent fixtures across four injectors. We did not run live multi-session pressure tests. Remaining risk: a model may still preferproduct-designon “Let's make a react todo list”; that needs a live session (report item 2 step 2).Human review