feat: rules/ を現行モデル観点でスリム化(#117、v0.67.0)#132
Merged
Merged
Conversation
- patterns.md 廃止(有効 2 行は coding-style.md へ統合)、hooks.md 廃止(要点は CLAUDE.md へ移設) - agents.md を委譲閾値 + 出力照合の 2 行に書換 - anti-patterns / coding-style / permissions-guide から harness 標準・デフォルト挙動の再記述を削除 - README の rules 数を 10→8 に追随 Closes #117
There was a problem hiding this comment.
Pull request overview
This PR reduces the always-injected rules/ footprint by removing low-signal or harness-duplicate guidance and relocating hook-authoring guidance to repository documentation, aligning the starter kit’s global rules with “current model” expectations.
Changes:
- Retired
rules/patterns.mdandrules/hooks.md, consolidating the two useful “patterns” bullets intorules/coding-style.mdand moving hook-authoring guidance intoCLAUDE.md. - Slimmed always-loaded rules by deleting duplicated/default-behavior lines in
rules/anti-patterns.md,rules/agents.md,rules/coding-style.md, andrules/permissions-guide.md. - Updated documentation and release notes to reflect the new “8 rules” set and added a
0.67.0changelog entry.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| rules/permissions-guide.md | Removes regression-test instruction from always-loaded rules. |
| rules/patterns.md | Retired; patterns guidance removed from global rules set. |
| rules/hooks.md | Retired; hook-authoring guidance removed from global rules set. |
| rules/coding-style.md | Slimmed and absorbs the two retained “patterns” bullets. |
| rules/anti-patterns.md | Removes a low-signal context-discipline line. |
| rules/agents.md | Replaces agent-use enumeration with a delegation threshold rule; keeps reconciliation guardrail. |
| README.md | Updates rule count to 8 in directory structure overview. |
| README.en.md | Updates rules list/count to 8 in features + directory structure. |
| CLAUDE.md | Adds hook-authoring conventions and regression-test note as repo-level guidance. |
| CHANGELOG.md | Adds 0.67.0 entry documenting rule slimming and retirements. |
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.
概要
LLM 性能監査計画 #121 の P4-11(#117)。常時注入される rules/ から、現行モデルに情報量のない行(harness 標準動作・デフォルト挙動の再記述、ルール作者向けメタ指示、開発規約の漏出)を削除する。10 → 8 ファイル。
変更内容
検証
Closes #117