Refine skill-optimization patterns and wire through creator/reviewer (1.25.3)#184
Merged
Merged
Conversation
Port the Adopted design validity check from the plugin distribution (claude-code-workflows PR #147) into the -en sources, selectively so this repo's already-ahead checks (behavioral claim, fact-disposition semantic alignment, cross-layer assumptions, minimal-surface detail) and its TypeScript-specialized frontmatter/category set stay intact. - document-reviewer.md: add requirements_verbatim / confirmed_decisions inputs, the effective-requirements derivation step, the confirmed_decisions clause in the LLM-facing clarity check, the Adopted design validity check block, and its Self-Validation enumeration entry. - design.md / front-design.md: wire requirements_verbatim / confirmed_decisions into the Design Doc document-reviewer invocation and pass the confirmed scope and user answers to the technical-designer. Scope: -en only; ja sync is a separate step. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ments (en) Port the latest rashomon prompt-optimization BP refinements into the English skill-optimization skill and wire them through the consuming agents: - BP-002: least-restrictive sufficient criterion with precision contribution / constraint cost; decompose the dense cell into resolution steps; source-derived example with an unresolved fallback - BP-003: consumer-required output shape - BP-005: rename to Missing or Excess Context; add excess-context removal - BP-006: per-step output evidence and transition condition - BP-007: rename to Unnecessary or Biased Examples; shift to minimal necessary examples - BP-008: observed/inferred/unknown classification, stop with the evidence or user decision required - principles #4/#7 aligned with the refined vocabulary - review-criteria: content-observable constraint-necessity check - skill-creator: sync BP transforms, add bp002Resolution branch schema - skill-reviewer: constraintNecessity balance check (content-observable) Scope: English sources only; ja/active mirror deferred. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ements into ja sources Translate the English skill-optimization refinements into the Japanese sources, matching existing tone and terminology and avoiding literal renderings: - BP-002: least-restrictive sufficient criterion with precision contribution / constraint cost and resolution steps; source-derived example with an unresolved fallback - BP-003: consumer-required output shape - BP-005: rename to Missing or Excess Context; add excess-context removal - BP-006: per-step observable completion output and transition condition - BP-007: rename to Unnecessary or Biased Examples; minimal necessary examples - BP-008: observed-fact/inferred/unknown classification, stop with the evidence or user decision required - principles #4/#7 aligned with the refined vocabulary - review-criteria: content-observable constraint-necessity check - skill-creator: sync BP transforms, add bp002Resolution branch schema - skill-reviewer: constraintNecessity balance check (content-observable) Verified en/ja skills-index consistency and ja agent JSON validity. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <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
1 participant
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.
Summary
Refines the
skill-optimizationskill's content patterns and wires them through theskill-creatorandskill-revieweragents so generated and reviewed skills come out less over-constrained and easier to follow. Applied across en/ja and released as 1.25.3.review-criteriagains a content-observable constraint-necessity checkskill-creator:bp002Resolutionbranch schema;skill-reviewer:constraintNecessitybalance checkVerification
check:skills-index— all 28 skills consistent across en / ja🤖 Generated with Claude Code