@joshuafolkken/kit 1.1455.0 → 1.1471.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.claude/skills/dependency-update/SKILL.md +26 -55
- package/.claude/skills/epic-commands/SKILL.md +87 -148
- package/.claude/skills/epic-commands/epic-bundle.md +45 -77
- package/.claude/skills/epic-commands/execution-waves.md +44 -81
- package/.claude/skills/verify-ui/SKILL.md +25 -50
- package/.claude/skills/workflow-commands/background-commands.md +36 -47
- package/.claude/skills/workflow-commands/backlogrun-child.md +72 -85
- package/.claude/skills/workflow-commands/backlogrun-lanes.md +91 -121
- package/.claude/skills/workflow-commands/backlogrun-park.md +73 -117
- package/.claude/skills/workflow-commands/backlogrun-progress.md +7 -9
- package/.claude/skills/workflow-commands/backlogrun-recovery.md +11 -20
- package/.claude/skills/workflow-commands/backlogrun-steps.md +43 -52
- package/.claude/skills/workflow-commands/chain-rule.md +24 -34
- package/.claude/skills/workflow-commands/followup.md +57 -76
- package/.claude/skills/workflow-commands/issue-comments.md +43 -60
- package/.claude/skills/workflow-commands/kickoff.md +65 -74
- package/.claude/skills/workflow-commands/observation-filing.md +62 -107
- package/.claude/skills/workflow-commands/observation-ledger.md +56 -111
- package/.claude/skills/workflow-commands/pre-gate-cut.md +89 -138
- package/.claude/skills/workflow-commands/prerequisite.md +10 -15
- package/.claude/skills/workflow-commands/progress-watcher.md +21 -26
- package/.claude/skills/workflow-commands/retrospective.md +31 -50
- package/.claude/skills/workflow-commands/split-assessment.md +34 -58
- package/.claude/skills/workflow-commands/working-tree-hold.md +54 -71
- package/CLAUDE.md +6 -5
- package/dist/CLAUDE.md +6 -5
- package/dist/hooks/batch-guard.js +6 -6
- package/dist/hooks/{chunk-7WQBPLUN.js → chunk-43ZHT22P.js} +31 -15
- package/dist/hooks/{chunk-LIAFCXL7.js → chunk-AS74OU3D.js} +23 -7
- package/dist/hooks/{chunk-3C4XRYG6.js → chunk-E7OIKX5Z.js} +17 -12
- package/dist/hooks/{chunk-Z4XI7CZS.js → chunk-EUMACUK2.js} +18 -12
- package/dist/hooks/{chunk-WNLVGT7E.js → chunk-H5HVWYXK.js} +4 -4
- package/dist/hooks/{chunk-HHI6FO3R.js → chunk-IIGFYXKC.js} +345 -240
- package/dist/hooks/{chunk-GPLGZ6IC.js → chunk-J47HU5H5.js} +14 -3
- package/dist/hooks/{chunk-OLMO72ON.js → chunk-K4EOONYM.js} +1 -1
- package/dist/hooks/{chunk-D5QBTOCZ.js → chunk-K4LMBQC2.js} +4 -4
- package/dist/hooks/{chunk-TBJFTSUY.js → chunk-KC5AK2GF.js} +8 -6
- package/dist/hooks/chunk-SHGPANFB.js +56 -0
- package/dist/hooks/{chunk-5H4SSDSD.js → chunk-YHBZAJSV.js} +3 -3
- package/dist/hooks/{chunk-HFGWYGUV.js → chunk-YYPI7SJZ.js} +5 -5
- package/dist/hooks/codex-hook-adapter.js +13 -13
- package/dist/hooks/duplicate-read-guard.js +7 -7
- package/dist/hooks/format-edited.js +3 -3
- package/dist/hooks/inputs.json +216 -214
- package/dist/hooks/investigation-guard.js +7 -7
- package/dist/hooks/pretool-guard.js +12 -12
- package/dist/hooks/session-lang.js +6 -5
- package/dist/hooks/stop-guard.js +18 -14
- package/dist/josh.js +5 -6
- package/dist/managed-marker/index.js +1 -2
- package/dist/version/index.js +15 -12
- package/eslint/base.js +2 -1
- package/eslint/rules/unicorn.js +16 -0
- package/package.json +3 -3
- package/prompts/coding-standards.md +12 -22
- package/prompts/collaboration-workflow/file-edits.md +26 -44
- package/prompts/collaboration-workflow/gh-rest.md +20 -36
- package/prompts/collaboration-workflow/glossary.md +5 -7
- package/prompts/collaboration-workflow/issue-citation.md +7 -13
- package/prompts/collaboration-workflow/issue-template.md +29 -29
- package/prompts/collaboration-workflow/operating-rules.md +35 -44
- package/prompts/collaboration-workflow/output-bounds.md +15 -15
- package/prompts/collaboration-workflow/overview.md +14 -19
- package/prompts/collaboration-workflow/plan-comment.md +13 -13
- package/prompts/collaboration-workflow/principles.md +50 -53
- package/prompts/collaboration-workflow/proposal-request.md +1 -9
- package/prompts/collaboration-workflow/report-format.md +32 -49
- package/prompts/collaboration-workflow/residency.md +10 -10
- package/prompts/collaboration-workflow/rule-delivery.md +9 -9
- package/prompts/collaboration-workflow/shell-body.md +10 -20
- package/prompts/collaboration-workflow/turn-batching.md +8 -14
- package/prompts/collaboration-workflow/upstream-interrupt.md +26 -34
- package/prompts/collaboration-workflow/wip-cap.md +24 -30
- package/prompts/collaboration-workflow.md +24 -25
- package/prompts/refactoring.md +33 -43
- package/prompts/review-rubric.md +84 -129
- package/prompts/review.md +111 -141
- package/prompts/sonar-hotspot-handling.md +22 -50
- package/prompts/testing-guide.md +48 -99
- package/scripts/adopt/adopt-toolkits.ts +2 -4
- package/scripts/agent/agent-exit-record.ts +23 -1
- package/scripts/agent/codex-agent-argv.ts +7 -2
- package/scripts/backlog/backlog-budget-cli.ts +1 -2
- package/scripts/backlog/backlog-drive-carried.ts +26 -0
- package/scripts/backlog/backlog-drive-cli.ts +10 -5
- package/scripts/backlog/backlog-drive-named.ts +2 -4
- package/scripts/backlog/backlog-drive-restore.ts +19 -1
- package/scripts/backlog/backlog-drive-seats.ts +27 -0
- package/scripts/backlog/backlog-drive.ts +11 -6
- package/scripts/backlog/backlog-ready.ts +13 -6
- package/scripts/behavior/behavior-rules.ts +3 -2
- package/scripts/cost-runtime/codex-usage.ts +4 -2
- package/scripts/cost-runtime/cost-cli.ts +1 -2
- package/scripts/delegation/investigation-guard.ts +1 -1
- package/scripts/delegation/investigation-reads.ts +1 -1
- package/scripts/discovery/repo-discovery.ts +4 -2
- package/scripts/discovery/repo-origin.ts +2 -4
- package/scripts/doctor/doctor-logic.ts +6 -3
- package/scripts/document/document-byte-budget.ts +27 -27
- package/scripts/document/document-byte-check.ts +1 -2
- package/scripts/document/entry-read-budget.ts +2 -2
- package/scripts/epic/epic-audit-rationale.ts +8 -2
- package/scripts/epic/epic-bundle-cli.ts +1 -2
- package/scripts/epic/epic-chains.ts +6 -2
- package/scripts/epic/epic-close-comment.ts +1 -2
- package/scripts/gate/verification-gate.ts +7 -2
- package/scripts/gh/git-pr.ts +6 -2
- package/scripts/git/git-common-directory.ts +3 -3
- package/scripts/git/git-diff-reads.ts +16 -9
- package/scripts/git/git-execa-error.ts +1 -2
- package/scripts/hooks/format-edited-file.ts +10 -4
- package/scripts/hooks/pre-push-unit.ts +15 -4
- package/scripts/hooks/pretool-guard.ts +23 -29
- package/scripts/hooks/step-zero-notice.ts +3 -2
- package/scripts/init/hook-command-rewrite.ts +6 -2
- package/scripts/init/init-logic-deploy-vps.ts +4 -2
- package/scripts/issue/behavior-change-lint.ts +12 -6
- package/scripts/issue/issue-scout-cli.ts +3 -2
- package/scripts/josh/file-map-stamp.ts +1 -3
- package/scripts/josh/hook-decision.ts +17 -2
- package/scripts/lane/lane-await.ts +3 -2
- package/scripts/lane/lane-child-invocation.ts +14 -1
- package/scripts/lane/lane-guard-policy.ts +4 -4
- package/scripts/lane/lane-ledger.ts +9 -0
- package/scripts/lane/lane-load.ts +9 -2
- package/scripts/lane/lane-measure-cli.ts +2 -0
- package/scripts/lane/lane-parked.ts +18 -0
- package/scripts/lane/lane-process-load.ts +125 -0
- package/scripts/lane/lane-process-stats.ts +93 -0
- package/scripts/lane/lane-registry.ts +8 -3
- package/scripts/lane/lane-session-broker.ts +29 -5
- package/scripts/lane/lane-spent.ts +14 -3
- package/scripts/lane/lane-stats.ts +1 -0
- package/scripts/managed-marker/managed-marker-logic.ts +1 -2
- package/scripts/overrides/lockfile-overrides.ts +1 -3
- package/scripts/propagate/propagate.ts +1 -2
- package/scripts/release/release-progress.ts +3 -2
- package/scripts/report/report-lint.ts +7 -4
- package/scripts/review/review-record-cli.ts +6 -2
- package/scripts/rules/destructive-command.ts +6 -9
- package/scripts/rules/direct-filing.ts +3 -7
- package/scripts/rules/early-heartbeat.ts +7 -18
- package/scripts/rules/file-body.ts +6 -13
- package/scripts/rules/filing-cap.ts +3 -6
- package/scripts/rules/git-force.ts +5 -9
- package/scripts/rules/implementation-cut.ts +37 -35
- package/scripts/rules/index-guard.ts +4 -7
- package/scripts/rules/issue-comments.ts +5 -15
- package/scripts/rules/josh-git-bare.ts +3 -6
- package/scripts/rules/lane-background.ts +10 -21
- package/scripts/rules/lane-carry-conflict.ts +7 -17
- package/scripts/rules/lane-interactive-ask.ts +6 -11
- package/scripts/rules/lane-park.ts +5 -13
- package/scripts/rules/lane-split-park.ts +2 -7
- package/scripts/rules/lane-switch-main.ts +9 -12
- package/scripts/rules/oracle-consulted.ts +4 -5
- package/scripts/rules/piped-verification.ts +81 -31
- package/scripts/rules/poll-loop.ts +7 -17
- package/scripts/rules/pre-gate-cut.ts +7 -21
- package/scripts/rules/prior-comment-read.ts +7 -5
- package/scripts/rules/protected-files.ts +5 -9
- package/scripts/rules/raw-field-body.ts +3 -7
- package/scripts/rules/rule-body-guard.ts +4 -11
- package/scripts/rules/run-tail-rule.ts +6 -22
- package/scripts/rules/shell-body-trigger.ts +4 -6
- package/scripts/rules/shell-segments.ts +7 -0
- package/scripts/rules/stop-rules.ts +16 -29
- package/scripts/rules/test-declared-commit.ts +4 -10
- package/scripts/rules/third-party-write.ts +5 -10
- package/scripts/rules/worktree-guard.ts +5 -12
- package/scripts/run/board/run-board-github.ts +11 -3
- package/scripts/run/board/run-board-idle.ts +31 -0
- package/scripts/run/board/run-board-read.ts +9 -12
- package/scripts/run/board/run-board-reason.ts +7 -3
- package/scripts/run/board/run-board-state.ts +4 -0
- package/scripts/run/board/run-board-status.ts +18 -3
- package/scripts/run/board/run-board-thanks.ts +7 -2
- package/scripts/run/board/run-board-tick.ts +32 -7
- package/scripts/run/carry/run-carry-args.ts +1 -3
- package/scripts/run/carry/run-carry-conversation.ts +7 -2
- package/scripts/run/cut/run-cut-cli.ts +15 -20
- package/scripts/run/cut/run-cut-decline.ts +72 -0
- package/scripts/run/cut/run-cut-ready.ts +74 -0
- package/scripts/run/cut/run-cut-report.ts +13 -0
- package/scripts/run/cut/run-cut.ts +10 -1
- package/scripts/run/hold/run-hold-cli.ts +14 -3
- package/scripts/run/hold/run-hold.ts +6 -3
- package/scripts/run/merge/run-merge-cli.ts +30 -7
- package/scripts/run/merge/run-merge-ship.ts +59 -1
- package/scripts/run/merge/run-merge.ts +3 -2
- package/scripts/run/run-ending.ts +31 -3
- package/scripts/run/run-parent-cut-hook.ts +5 -10
- package/scripts/run/run-preflight.ts +1 -2
- package/scripts/run/run-stop-notify.ts +1 -2
- package/scripts/run/run-watcher-guard.ts +6 -3
- package/scripts/run/ship/run-ship-review-steps.ts +13 -4
- package/scripts/run/ship/run-ship-review.ts +15 -2
- package/scripts/run/tidy/run-tidy-cli.ts +3 -3
- package/scripts/run/tidy/run-tidy-lanes.ts +3 -2
- package/scripts/safe-chain/project-config.ts +7 -2
- package/scripts/sync/sync-hook-safety.ts +8 -2
- package/scripts/test/unit-worker-share.ts +3 -2
- package/scripts/time-runtime/time-batch-guard.ts +3 -3
- package/scripts/time-runtime/time-hook-transcript.ts +3 -2
- package/scripts/time-runtime/time-shell.ts +10 -0
- package/scripts/version/effective-upstream.ts +3 -4
- package/scripts/version/kit-version-config.ts +8 -2
- package/scripts/version/latest-corepack.ts +1 -2
- package/scripts/version/release-hold.ts +11 -4
- package/scripts/version/version-commands.ts +12 -4
- package/dist/hooks/chunk-UT7YC3N2.js +0 -56
|
@@ -1,38 +1,35 @@
|
|
|
1
1
|
# 原則
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
経緯と適用例は [`docs/maintainers/principles-rationale.md`](../../docs/maintainers/principles-rationale.md) の各節にあり、実行中には読まない。
|
|
4
4
|
|
|
5
5
|
## no-clones — クローン禁止・単一ソース化
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
既存の非自明なロジック(別ファイル・別モジュール・上流依存を含む別パッケージにあるもの)を、ソースを変えずに済ませるために**コピー・移植・再実装しない**。複製したくなったら、全消費者が import する共有モジュール/export/パッケージへ単一ソース化する。
|
|
8
8
|
|
|
9
|
-
-
|
|
10
|
-
-
|
|
11
|
-
-
|
|
12
|
-
-
|
|
9
|
+
- 既存の重複は再重複の理由にならない(例: josh-game が kit をコピー)。3 つ目のコピーを足さず、重複を surface する
|
|
10
|
+
- 「上流を参照する」は再利用することで、コードを貼り付けることではない
|
|
11
|
+
- 「X を触るな」という制約でクローンを黙って正当化しない。「クリーンな修正は kit の変更が必要、代替はクローン、どちらにする?」と先に提示する
|
|
12
|
+
- 複製は、単一ソース化の代替案とコストを提示し、ユーザーの明示承認を得た後にのみ行う
|
|
13
13
|
|
|
14
|
-
|
|
14
|
+
計測は `pnpm josh clone:scan`(`pnpm josh oracle:list` の決定オラクル)。リポジトリ内と `JOSH_REPO_PATHS` の他リポジトリを跨いだ複製の件数と各 `file:line`(`[same-file|cross-file|cross-repo]`、なしは `clean`)を印字する(`docs/maintainers/principles-rationale.md` → "Why clone:scan exists")。
|
|
15
15
|
|
|
16
16
|
## elegant-design — 設計はエレガント・シンプルを第一目標にする
|
|
17
17
|
|
|
18
|
-
|
|
18
|
+
フック・ガード・ルール・抽象化を足す前に、**原因側を直せばその仕組みが要らなくならないか**を問う。要らなくなるなら足さず、原因を直す。検出・通知・叱責を強化する前に、その入力を生んでいる側を断つ。適用例は `docs/maintainers/principles-rationale.md` → "An application of simplicity-first"。
|
|
19
19
|
|
|
20
|
-
-
|
|
21
|
-
- **足す前に断つ**: 検出・通知・叱責を強化する前に、その入力を生んでいる側を探す。入力が消えれば、それを捌く仕組みそのものが消える。適用例は `docs/maintainers/principles-rationale.md` → "An application of simplicity-first"。
|
|
22
|
-
|
|
23
|
-
**書かれたコードの可読性**(意図が伝わる命名、過度な抽象化を避ける、など)は `prompts/coding-standards.md` の担当で、こちらは**設計時にどの仕組みを足すか・どこへ置くかを決める**規則。両者は別レイヤーなので、片方の本文をもう片方へ写さず、互いを指すだけにする。
|
|
20
|
+
書かれたコードの可読性は `prompts/coding-standards.md` の担当で、ここはどの仕組みをどこへ足すかの規則である。互いの本文を写さず、指すだけにする。
|
|
24
21
|
|
|
25
22
|
## quality-priority — 品質目標が衝突したときの優先順位
|
|
26
23
|
|
|
27
|
-
|
|
24
|
+
上の段は、下の段のために犠牲にしない。
|
|
28
25
|
|
|
29
|
-
1. **正しさ** —
|
|
26
|
+
1. **正しさ** — 仕様どおりに動き、検証ゲートが緑であること
|
|
30
27
|
2. **同列で、どれも悪化させない** — 次の 4 つに上下はない
|
|
31
|
-
- **複雑さ**(読みやすさを含む)—
|
|
32
|
-
- **冗長さ**(重複を含む)— 同じ知識を 1 か所に置く(no-clones
|
|
28
|
+
- **複雑さ**(読みやすさを含む)— 初見の読み手が前提知識なしに意図を追えること。部品や間接層は読む量が減るときだけ足す
|
|
29
|
+
- **冗長さ**(重複を含む)— 同じ知識を 1 か所に置く(no-clones)
|
|
33
30
|
- **パフォーマンス** — gate・CI・josh 起動・テストの所要時間
|
|
34
31
|
- **AI コスト** — エージェントが読み込み・出力するトークン量
|
|
35
|
-
3. **コメント最小** —
|
|
32
|
+
3. **コメント最小** — まず命名・分割・型で表現する。残すのは「なぜ」だけ
|
|
36
33
|
|
|
37
34
|
2 段目の目標同士が衝突したら、次の順で決める。
|
|
38
35
|
|
|
@@ -40,63 +37,63 @@
|
|
|
40
37
|
2. **1 つを悪くするなら、同じ変更の中で別の場所を削って埋め合わせる** — 合計を悪化させない
|
|
41
38
|
3. **それでも両立しないときは、数字を示して利用者に聞く**(Tier B)— 何をどれだけ悪くして、何を得るか
|
|
42
39
|
|
|
43
|
-
##
|
|
40
|
+
## concise-prose — 文章は条件と動作だけを書く
|
|
44
41
|
|
|
45
|
-
|
|
42
|
+
docs・prompts・skills・拒否文・Issue・PR の文章に適用する。
|
|
46
43
|
|
|
47
|
-
-
|
|
48
|
-
-
|
|
49
|
-
-
|
|
44
|
+
- 本文には、いつ何をするか(条件と動作)だけを書く
|
|
45
|
+
- 理由と経緯は rationale 文書に置き、本文からは見出しで指す
|
|
46
|
+
- 太字は 1 節に 1 つまで。箇条書き先頭の見出し語は数えない。`→ "…"` で引かれている太字は消さない
|
|
47
|
+
- 同じ内容を言い換えて繰り返さない
|
|
48
|
+
- ファイルや節が自分自身を説明する文(「このファイルは〜である」)を書かない
|
|
50
49
|
|
|
51
|
-
|
|
50
|
+
導入時のベースラインは `docs/maintainers/principles-rationale.md` → "Why concise-prose exists"。
|
|
52
51
|
|
|
53
|
-
|
|
52
|
+
## consult-vs-execute — 相談と実行を区別する
|
|
54
53
|
|
|
55
|
-
|
|
54
|
+
ユーザーが進め方・理由を尋ねたり目標を述べたりした(「どうすべき?」「how should we…」「なぜ」「〜したい」「〜の方が良い?」)ときは、**分析と推奨のみ**を返し、ファイル編集・Issue 作成などのアクションは取らない。目標の表明は計画の依頼であって、実行の承認ではない。
|
|
56
55
|
|
|
57
|
-
|
|
58
|
-
|
|
59
|
-
- 新ルール/エラーが正当なら **消費者コードを直す**
|
|
60
|
-
- ルール上書きが要るなら **正しいレイヤー**(kit / app-kit の共有設定)でスコープする。消費者リポジトリでの場当たり的な一回限りの disable にしない
|
|
61
|
-
- 破壊が first-party パッケージ(kit / app-kit)起因なら、**そこに Issue を立てて適切な altitude で直す**。消費者側の回避だけで済ませない(→ `upstream-interrupt.md`)
|
|
62
|
-
3. **pin-back は最終手段**: fix-forward が本当に不可能/ブロックされている(例: 未リリースの上流修正待ち)ときだけ、古いバージョンへ固定する。固定するときは **理由を記録し、最新へ戻すためのトラッキング Issue を立てる**。pin-back を既定の推奨として提示してはならない
|
|
63
|
-
4. **既存の保護を尊重する**: fix-forward は保護された pin を黙って書き換える許可ではない。overrides と `devEngines` の承認ゲートの単一ソースは `CLAUDE.md` → "Decision autonomy" と `dependency-update` skill である
|
|
64
|
-
5. **タイムリーに**: バンプ起因の破壊は、可能な限り同じ作業セッション内で速やかに対処し、pin の裏に先送りしない
|
|
56
|
+
- アクションを取るのは、明示的な命令(「do it」「書き換えて」「作成して」「implement」)かワークフローキーワード(`kickoff` / `halfrun` / `prrun` / `fullrun` / `backlogrun`)があるときだけ
|
|
57
|
+
- 判別できないときは相談として扱う。実行が指示された後の判断点は `CLAUDE.md` → "Decision autonomy" の Tier で決まる
|
|
65
58
|
|
|
66
|
-
##
|
|
59
|
+
## latest-first — 最新優先・fix-forward(pin-back は最終手段)
|
|
60
|
+
|
|
61
|
+
依存・ツールチェーンは**既定で最新版を採用する**。`dependency-update` skill → "4. When the bump breaks something — fix forward" はこの節を指す。
|
|
67
62
|
|
|
68
|
-
|
|
63
|
+
1. 適応の手間を避けるために古いバージョンに留まったり戻したりしない
|
|
64
|
+
2. バンプで lint クラッシュ・新規ルール・型エラー等が出たら fix-forward で解消する:
|
|
65
|
+
- 新ルール/エラーが正当なら消費者コードを直す
|
|
66
|
+
- ルール上書きが要るなら kit / app-kit の共有設定でスコープする。消費者リポジトリで一回限りの disable にしない
|
|
67
|
+
- first-party パッケージ(kit / app-kit)起因なら、そこに Issue を立てて直す(→ `upstream-interrupt.md`)
|
|
68
|
+
3. **pin-back は最終手段**: fix-forward が不可能/ブロックされている(例: 未リリースの上流修正待ち)ときだけ固定し、理由を記録して最新へ戻すトラッキング Issue を立てる。pin-back を既定の推奨にしない
|
|
69
|
+
4. fix-forward は保護された pin を黙って書き換える許可ではない。overrides と `devEngines` の承認ゲートは `CLAUDE.md` → "Decision autonomy" と `dependency-update` skill に従う
|
|
70
|
+
5. バンプ起因の破壊は同じセッション内で対処し、pin の裏に先送りしない
|
|
69
71
|
|
|
70
|
-
-
|
|
71
|
-
- **MEMORY は 1 台のマシン × 1 つのリポジトリ**にスコープされたローカルストアで、コミットされない。保存は最小限に留め、用途は本質的にローカルな文脈(マシン固有のパス、個人の環境の癖など)だけ。どこでも適用されるべきルールの唯一の置き場所にしてはならない
|
|
72
|
+
## durable-rules — 恒久ルールは MEMORY ではなくプロンプト/ドキュメントに書く
|
|
72
73
|
|
|
73
|
-
|
|
74
|
+
共有価値のある恒久ルールに気付いたら、**kit が配布するプロンプト/ドキュメント**(`CLAUDE.md` / `prompts/*`)の変更として書き、自動 MEMORY(`~/.claude/projects/<repo-slug>/memory/`)で済ませない。MEMORY はマシン固有のパスなど本質的にローカルな文脈だけに使う。
|
|
74
75
|
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
4. 既に MEMORY にあるルールが実は共有すべきものだと分かったときは、プロンプト/ドキュメントへ移し、MEMORY 側の重複エントリは削除する
|
|
76
|
+
- ターンがドキュメント変更を承認していない(相談中など)なら、MEMORY に黙って保存せず変更を提案する
|
|
77
|
+
- 消費者リポジトリで気付いたら、kit 側へ Issue/PR として上流化する
|
|
78
|
+
- MEMORY にあるルールが共有すべきものなら、プロンプト/ドキュメントへ移し、MEMORY 側を削除する
|
|
79
79
|
|
|
80
80
|
## upstream-to-kit — 配布ドキュメント・設定の変更は kit に上流化する
|
|
81
81
|
|
|
82
|
-
`CLAUDE.md`
|
|
83
|
-
|
|
84
|
-
- **消費者リポジトリ(app-kit / game-kit)ではこれらをローカル編集しない**: `josh sync` が編集を上書きするうえ、変更は本来上流(kit)に属する。ドキュメント/設定を編集する前に、それが配布物かどうかを確認し、配布物なら kit 側に変更を提案(Issue/PR)する
|
|
85
|
-
- **kit リポジトリ自身ではあなたが配布元**なので、ここでは編集してよい。編集先は `CLAUDE.md` 1 本である(次節)
|
|
82
|
+
- 消費者リポジトリ(app-kit / game-kit)では、kit が配布する `CLAUDE.md` や他のドキュメント/設定をローカル編集しない(`josh sync` が上書きする)。編集前に配布物か確認し、配布物なら kit 側に Issue/PR で提案する
|
|
83
|
+
- kit リポジトリ自身では編集してよい。編集先は `CLAUDE.md` 1 本である(次節)
|
|
86
84
|
|
|
87
85
|
## エージェント規則の単一ソースは `CLAUDE.md`(`AGENTS.md` / `GEMINI.md` / `.cursorrules` は導線)
|
|
88
86
|
|
|
89
|
-
**規則の本体は `CLAUDE.md` にしか無い。** `AGENTS.md`・`GEMINI.md`・`.cursorrules` は「規則は `CLAUDE.md`
|
|
87
|
+
**規則の本体は `CLAUDE.md` にしか無い。** `AGENTS.md`・`GEMINI.md`・`.cursorrules` は「規則は `CLAUDE.md` にある。作業前に全文を読むこと」と「規則を書き戻さない」だけを書いた導線である(`.cursorrules` は Svelte MCP の案内だけを足す)。
|
|
90
88
|
|
|
91
|
-
-
|
|
92
|
-
- **導線文書に規則を書き戻さない。** そこに書かれた規則は、次の改訂で更新し忘れる 4 つ目の置き場になる
|
|
89
|
+
- 規則の追加・変更・節の追加は `CLAUDE.md` に 1 回だけ書き、導線文書には書き戻さない
|
|
93
90
|
- 経緯と機械的な検査は `docs/maintainers/principles-rationale.md` → "Why AGENTS and GEMINI are pointers"、`@CLAUDE.md` 行と他ツールでの未検証は `docs/maintainers/principles-rationale.md` → "The include line, and what has not been verified"
|
|
94
91
|
|
|
95
92
|
### Claude Code 以外のエージェントでの読み替え
|
|
96
93
|
|
|
97
|
-
`CLAUDE.md`
|
|
94
|
+
Codex・Gemini CLI など別のエージェントは `CLAUDE.md` を次のように読み替える(joshuafolkken/kit#3079)。
|
|
98
95
|
|
|
99
|
-
- **フック**:
|
|
100
|
-
- **Skill・`/verify-ui`・`/code-review`**:
|
|
101
|
-
- **`.claude/settings.json` の deny list**
|
|
96
|
+
- **フック**: フックが走らない・出力が見えないツールでは、[`rule-delivery.md`](./rule-delivery.md) →「配送されている規則」の各項目を自分で守る。加えて、最初の実行時コード編集の前に Step 0 の作業サマリを提示し、編集後に `pnpm josh lint:related` と `pnpm josh cspell:dot` を走らせ、`JOSH_SESSION_LANG` を `.env` から解決する。フックの全体は `.claude/settings.json` の `hooks`
|
|
97
|
+
- **Skill・`/verify-ui`・`/code-review`**: その skill の `SKILL.md` を読んで自分のセッションで従う
|
|
98
|
+
- **`.claude/settings.json` の deny list** は効かないが、`CLAUDE.md` → "Git Rules" はそのまま拘束する
|
|
102
99
|
- **`AskUserQuestion`** は、ユーザーに平文で尋ねることを指す
|
|
@@ -1,37 +1,32 @@
|
|
|
1
1
|
## 報告フォーマット(平易な概要 + 技術詳細)
|
|
2
2
|
|
|
3
|
-
作業前サマリ(Step 7)と完了報告(Step 5
|
|
3
|
+
作業前サマリ(Step 7)と完了報告(Step 5)は、**平易な概要を先頭に置き、技術詳細をその下に置く 2 層構造**で書く。
|
|
4
4
|
|
|
5
5
|
### 機械的な半分は `josh report:lint` が検査する
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
ラベルの有無、概要行の字数上限、コードフェンス囲み、概要へのファイルパス・CLI フラグの混入、ケース段と「壊れるとしたら」段の形は、サマリを標準入力で `pnpm josh report:lint` に渡して検査する(`ok` か違反行を印字する — joshuafolkken/kit#2123)。
|
|
8
8
|
|
|
9
9
|
### 出力はコードフェンスで囲まない(必須)
|
|
10
10
|
|
|
11
|
-
|
|
11
|
+
以下のテンプレートは**フェンスの中身だけ**をセッションに書く(空白での桁揃えと単独行の `---` も使わない)。
|
|
12
12
|
|
|
13
13
|
### ラベルはセッション言語に訳し、注釈は付けない(必須)
|
|
14
14
|
|
|
15
|
-
**(1) ラベルはセッション言語に訳す。**
|
|
15
|
+
**(1) ラベルはセッション言語に訳す。** 日本語セッションでの対応:
|
|
16
16
|
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
| `Cause` / `Fix` / `Result`(完了報告) | `原因` / `対応` / `結果` |
|
|
24
|
-
| `■ Hand-off`(区切り) | `■ 区切り` |
|
|
25
|
-
| `Done` / `Remaining` / `Why it stopped` / `Next command` | `終わったこと` / `残っていること` / `止めた理由` / `次に打つコマンド` |
|
|
17
|
+
- `■ Overview` → `■ 概要`
|
|
18
|
+
- `Now` / `Change` / `Check` → `今こうなっている` / `こう直す` / `確かめ方`
|
|
19
|
+
- `Details` → `技術詳細`、`Changes and tests` → `変更とテスト`
|
|
20
|
+
- `Cause` / `Fix` / `Result`(完了報告)→ `原因` / `対応` / `結果`
|
|
21
|
+
- `■ Hand-off`(区切り)→ `■ 区切り`
|
|
22
|
+
- `Done` / `Remaining` / `Why it stopped` / `Next command` → `終わったこと` / `残っていること` / `止めた理由` / `次に打つコマンド`
|
|
26
23
|
|
|
27
|
-
成果物の散文(Issue 本文、Issue/PR コメント、Telegram
|
|
24
|
+
成果物の散文(Issue 本文、Issue/PR コメント、Telegram 本文)のラベルもこの表で訳す。英語のまま残るのは「出力の言語(`JOSH_SESSION_LANG`)」が挙げた 3 つだけ。
|
|
28
25
|
|
|
29
|
-
**(2) ラベルに注釈を付けない。** `■ 概要(平易な説明)`
|
|
26
|
+
**(2) ラベルに注釈を付けない。** `■ 概要(平易な説明)` は誤り。注意書きはラベルの下の説明文に置く。
|
|
30
27
|
|
|
31
28
|
### 作業前サマリはいつ・どう出すか
|
|
32
29
|
|
|
33
|
-
`CLAUDE.md` → "Code Change Rules" の Step 0 が指す細目の単一ソース。
|
|
34
|
-
|
|
35
30
|
- **いつ**: 実装コードを書く前に、Issue ごとに 1 回出す。`fullrun` / `halfrun` / `prrun` / `backlogrun` で必須、`kickoff` は対象外。確認のための停止ではない — 出したらそのまま実装に進む
|
|
36
31
|
- **概要**: `Now` / `Change` / `Check` を各 1 文、セッション言語で、プログラマでない人にも追える文で書く。各行に具体的な主語を書き、主語のない文は不可(下記「具体的な主語を必ず書く」)。ファイルパス・関数名や型名・CLI のオプションフラグは書かない — 禁じるのは内部識別子だけである
|
|
37
32
|
- **技術詳細**: すべての変更をそのテストとともに `<変更内容> — Test: <Unit|E2E> — <file path> — <検証する挙動>` の形で並べる
|
|
@@ -64,20 +59,20 @@
|
|
|
64
59
|
2. ...
|
|
65
60
|
```
|
|
66
61
|
|
|
67
|
-
|
|
62
|
+
テスト宣言はケース単位の 2 段で書く(joshuafolkken/kit#2246)。
|
|
68
63
|
|
|
69
|
-
-
|
|
70
|
-
-
|
|
71
|
-
-
|
|
72
|
-
-
|
|
64
|
+
- 第 1 段(ケース)は固定語彙を並べ、**該当しないものは `該当なし` に続けて理由を書いて消す**
|
|
65
|
+
- 第 2 段(壊れるとしたら)は、この変更に固有の壊れうる条件を**最低 3 件**書く。書けないときは `どう呼ばれても壊れない` と明示する
|
|
66
|
+
- 複数ケースは `it.each` 1 本に畳んでよい
|
|
67
|
+
- 差分が HTTP・プロセス起動・ファイル IO・時間の境界を跨ぐときは、`pnpm josh cases <path...>` が印字する異常系を第 2 段に載せる
|
|
73
68
|
|
|
74
69
|
### 行数予算は編集前に読む
|
|
75
70
|
|
|
76
|
-
**「対象」に書くファイルは、書き始める前に `pnpm josh lines <path> [<path>...]`
|
|
71
|
+
**「対象」に書くファイルは、書き始める前に `pnpm josh lines <path> [<path>...]` に渡し、上限に近いと言われたファイルには分割方針を「対象」に書く**(joshuafolkken/kit#1425)。行数は `wc -l` で数えない。関数単位の上限は `pnpm josh lint` が報告する。
|
|
77
72
|
|
|
78
73
|
### 変更対象はまとめて読んでから編集する(joshuafolkken/kit#2202)
|
|
79
74
|
|
|
80
|
-
|
|
75
|
+
**「変更とテスト」が変更対象ファイルを列挙したら、1 回の `pnpm josh read:files <path> [<path> ...]` で読み、続く独立した編集は 1 回の `pnpm josh edit:files -` で送る**(個別の `Edit` は依存する編集・1 件だけの編集・`Write` のとき)。詳細は [`turn-batching.md`](turn-batching.md) →「実装中の独立編集に効く合成コマンド」。
|
|
81
76
|
|
|
82
77
|
### 完了報告(セッション向け)
|
|
83
78
|
|
|
@@ -97,42 +92,32 @@
|
|
|
97
92
|
- 残課題: <なければ「なし」>
|
|
98
93
|
```
|
|
99
94
|
|
|
100
|
-
**実行証跡は「動くこと」の証拠であり、gate
|
|
95
|
+
**実行証跡は「動くこと」の証拠であり、gate の緑ではない**(joshuafolkken/kit#2246)。同じ証跡を PR 本文の `## 実機証跡` 節(コマンド+実出力のフェンス)にも置く。無い実行時の変更は `pnpm josh followup` がマージを拒否する(#2446)。
|
|
101
96
|
|
|
102
|
-
**`run:carry` の実行証跡は `JOSH_TEMP_ROOT` を新しい一時ディレクトリに向けて取る(必須)。**
|
|
97
|
+
**`run:carry` の実行証跡は `JOSH_TEMP_ROOT` を新しい一時ディレクトリに向けて取る(必須)。** 状態レコードは本物の run と共有なので、`JOSH_TEMP_ROOT="$(mktemp -d)" pnpm josh run:carry --begin …` のように各コマンドの頭に付ける(`docs/environment-variables.md` → "Runs")。`run:merge` は main の同期・lane の close・GitHub への書き込みを行うため、これでは切り離せない。
|
|
103
98
|
|
|
104
99
|
### 区切りの報告(完了報告と区別する・必須)
|
|
105
100
|
|
|
106
|
-
`backlogrun`
|
|
101
|
+
`backlogrun` の区切り(hand-off)は完了報告ではない — **`原因` / `対応` / `結果` を使わない**。書式は [`.claude/skills/workflow-commands/backlogrun-handoff-report.md`](../../.claude/skills/workflow-commands/backlogrun-handoff-report.md)。
|
|
107
102
|
|
|
108
103
|
### 概要 3 行の書き方(ここが本体)
|
|
109
104
|
|
|
110
|
-
字数と内部識別子の混入は `report:lint`
|
|
105
|
+
字数と内部識別子の混入は `report:lint` が検査する。平易にするために語彙と抽象度を制約する:
|
|
111
106
|
|
|
112
107
|
- 専門用語は使わないか、`〜(=…のこと)` の形でその場で言い換える
|
|
113
|
-
-
|
|
114
|
-
-
|
|
108
|
+
- 変更点の羅列ではなく、原因 → 対応 → 効果の因果で書く
|
|
109
|
+
- 主語は利用者・システムの振る舞い。「利用者から見て何が違うか」が書けない行は技術詳細に置く
|
|
115
110
|
|
|
116
|
-
**送信前セルフチェック**: コードを見ていない読み手が、概要 3
|
|
111
|
+
**送信前セルフチェック**: コードを見ていない読み手が、概要 3 行だけでどれの話か特定できるか。できないなら具体的な名詞を 1 つ足す。
|
|
117
112
|
|
|
118
113
|
### 具体的な主語を必ず書く(禁止と対になる要求)
|
|
119
114
|
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
-
|
|
123
|
-
-
|
|
124
|
-
- **書いてよい具体名**: 利用者が画面で目にするもの — パッケージ名、表示されるコマンドの目的、画面名、メッセージの種類
|
|
125
|
-
- **書いてはいけない具体名**: 内部識別子のみ — ファイルパス、関数名・型名、CLI のオプションフラグ
|
|
115
|
+
- **何かが「古い」「壊れている」「変わらない」と書くときは、その対象を名指しする** — 対象パッケージ名・画面名・出力の種類のいずれかを含める
|
|
116
|
+
- 主語のない「状況」「案内」「問題」「その処理」だけで済ませない
|
|
117
|
+
- 書いてよい具体名: 利用者が画面で目にするもの — パッケージ名、表示されるコマンドの目的、画面名、メッセージの種類
|
|
118
|
+
- 書いてはいけない具体名: 内部識別子 — ファイルパス、関数名・型名、CLI のオプションフラグ
|
|
126
119
|
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
```md
|
|
130
|
-
- **今こうなっている**: 古いままだと警告が出るのに、案内されたコマンドを実行しても状況が変わらないことがある。
|
|
131
|
-
- **こう直す**: 効き目のない案内は理由の説明に置き換え、同じ案内が何度も並ぶのもやめる。
|
|
132
|
-
- **確かめ方**: 自動テストで、無駄な案内が消え・重複が消え・従来の表示は変わらないことを確かめる。
|
|
133
|
-
```
|
|
134
|
-
|
|
135
|
-
**良い例**(同じ内容で、対象を名指ししている):
|
|
120
|
+
悪い例は「古いままだと警告が出るのに、案内されたコマンドを実行しても状況が変わらない」(何の話か特定できない)。良い例:
|
|
136
121
|
|
|
137
122
|
```md
|
|
138
123
|
- **今こうなっている**: バージョン確認が「全体で使う app-kit が古い」と警告するのに、一緒に出る更新コマンドは既に入っている版を指していて、実行しても何も変わらず同じ警告がまた出る。
|
|
@@ -140,8 +125,6 @@
|
|
|
140
125
|
- **確かめ方**: 自動テストで、効かないコマンドが説明に置き換わり、重複が 1 行になり、通常の表示は変わらないことを確かめる。
|
|
141
126
|
```
|
|
142
127
|
|
|
143
|
-
違いは長さではなく、**名指ししているかどうか**にある。良い例には内部識別子は 1 つも出てこない。
|
|
144
|
-
|
|
145
128
|
### 成果物(Issue / PR / Telegram)側
|
|
146
129
|
|
|
147
|
-
言語は「出力の言語(`JOSH_SESSION_LANG
|
|
130
|
+
言語は「出力の言語(`JOSH_SESSION_LANG`)」に従い、構造もセッション向けと同じ 2 層にする。`--notify-message` は `Added ... / Changed ...` の羅列ではなく、`Cause / Fix / Result` の 3 行を先頭に置き、変更点を `Details:` 以下にまとめる(実例は [`followup.md`](../../.claude/skills/workflow-commands/followup.md))。
|
|
@@ -1,37 +1,37 @@
|
|
|
1
1
|
## 常駐ドキュメントと skill の分担(何を常駐に残すか)
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
規則を置く・移す・引退させるターンに読む(joshuafolkken/kit#2891)。規則のトリガは `CLAUDE.md` に 1 度だけ、手順と数値は導線の先に 1 度だけ書く。`CLAUDE.md` の上限は `scripts/document/resident-budget.ts` の `RESIDENT_CEILING_BYTES`(残りは `pnpm josh bytes CLAUDE.md`)。**置き場所は重要度ではなく、下の問いを順に当てて決める。** 経緯は `docs/maintainers/claude-md-history.md` と `docs/maintainers/residency-rationale.md`。
|
|
4
4
|
|
|
5
5
|
### 第 0 問(先に問う)
|
|
6
6
|
|
|
7
7
|
> **その規則の答えは、機械的に読める入力だけから計算できるか。**
|
|
8
8
|
|
|
9
|
-
|
|
9
|
+
できるなら、コマンドが答え、文書には導線 1 行だけを置く。既存のオラクルは `pnpm josh oracle:list`、単一ソースは `scripts/rules/decision-oracle.ts`。
|
|
10
10
|
|
|
11
11
|
### 順序の問い(第 0 問の次に問う)
|
|
12
12
|
|
|
13
13
|
> **その規則は「何を」ではなく「いつ・どの順で・どの引数で」を決めているか。**
|
|
14
14
|
|
|
15
|
-
|
|
15
|
+
そうであれば、規則はドライバの状態遷移に書き、文書は導線 1 行だけにする。`pnpm josh run:step <N>` が次の 1 手を計算する(`scripts/run/run-step.ts`、joshuafolkken/kit#2248)。
|
|
16
16
|
|
|
17
17
|
### 第 1 問 — 引き金を名指しできるか
|
|
18
18
|
|
|
19
19
|
> **その規則が効き始める瞬間を、1 件のツール呼び出しとして名指しできるか。**
|
|
20
20
|
|
|
21
|
-
|
|
21
|
+
できるなら、規則本文を常駐から出し、配送ガードの行に `scripts/rules/rule-registry.ts` の項目を添え、[`rule-delivery.md`](./rule-delivery.md) が指す `pnpm josh rule:list` の一覧に載せる(joshuafolkken/kit#1524)。**規則ごとのトリガ行は常駐に残さない** — `CLAUDE.md` には一覧への導線 1 行だけを置く。フックが走らないセッションは一覧を自己点検に使う(`principles.md` →「Claude Code 以外のエージェントでの読み替え」)。根拠は `docs/maintainers/residency-rationale.md` → "Why a delivered rule leaves no resident line"。
|
|
22
22
|
|
|
23
23
|
### 第 2 問 — skill なしのターンでも効く必要があるか
|
|
24
24
|
|
|
25
25
|
> **その規則は、skill がロードされていないターンでも効く必要があるか。**
|
|
26
26
|
|
|
27
|
-
**`CLAUDE.md` に残るのは、skill がロードされていないターンでも効く必要がある規則だけである。** 入力は「最初に効くのはコマンド開始の前か後か」の 1
|
|
27
|
+
**`CLAUDE.md` に残るのは、skill がロードされていないターンでも効く必要がある規則だけである。** 入力は「最初に効くのはコマンド開始の前か後か」の 1 点。後なら `CLAUDE.md` から導線を張り、再掲しない。
|
|
28
28
|
|
|
29
29
|
### 残すときの書き方と予算
|
|
30
30
|
|
|
31
31
|
- **常駐規則は、トリガと導線の 2 つで書く。** **判定は、導線を一度も開かないターンでも常駐の記述だけで正しく振る舞えるかである。**
|
|
32
|
-
- **削ることは移すことであり、削除は理由を示して初めて許される例外である。** 先に導線の先へ移し、マーカーテストも付け替える。引退の 3 条件は `docs/maintainers/residency-rationale.md` → "Trigger and pointer, and the narrow retirement route"
|
|
33
|
-
-
|
|
34
|
-
-
|
|
35
|
-
-
|
|
32
|
+
- **削ることは移すことであり、削除は理由を示して初めて許される例外である。** 先に導線の先へ移し、マーカーテストも付け替える。引退の 3 条件は `docs/maintainers/residency-rationale.md` → "Trigger and pointer, and the narrow retirement route"
|
|
33
|
+
- 経緯(Issue 番号・なぜ・計測・却下案)は `docs/maintainers/<topic>-rationale.md` に置く。Issue 番号引用の件数は `scripts/document/issue-citation-budget.ts` のラチェットに従う — 減らしたら記録も下げる(`docs/maintainers/residency-rationale.md` → "The issue-citation ratchet")
|
|
34
|
+
- 引用は本文があるファイルを直接指す(`docs/maintainers/residency-rationale.md` → "A pointer-only topic file is never cited")
|
|
35
|
+
- 予算が詰まったら回収で解く。**引き上げは Tier C として扱う** — `RESIDENT_CEILING_BYTES`・`RESIDENT_HEADROOM_BYTES`・`RE_INLINE_GUARD_HEADROOM_BYTES` の緩和は、`docs/maintainers/residency-rationale.md` → "When the ceiling may be raised" の 3 条件を示し、ユーザーの明示指示を得てから行う
|
|
36
36
|
|
|
37
|
-
|
|
37
|
+
常駐に残る規則と配送へ移した規則の一覧は `docs/maintainers/residency-rationale.md` → "The resident-rule list"。そのほかの経緯: `docs/maintainers/residency-rationale.md` → "The reduction freeze and its retraction"、`docs/maintainers/residency-rationale.md` → "Measurement decides what leaves"、`docs/maintainers/residency-rationale.md` → "The `rule:value` readings"、`docs/maintainers/residency-rationale.md` → "Why the delivered list carries no count"、`docs/maintainers/residency-rationale.md` → "Why the workflow index is split by topic"、`docs/maintainers/residency-rationale.md` → "One list, not two"、`docs/maintainers/residency-rationale.md` → "The resident budget"。
|
|
@@ -1,31 +1,31 @@
|
|
|
1
1
|
# 引き金つき配送 — 規則を「効く瞬間」に届ける(joshuafolkken/kit#1524)
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
規則の届け方は**常駐**(`CLAUDE.md` に書き毎ターン読み込ませる)と**引き金つき配送**(効く瞬間にフックが本文を突きつける)の 2 つ。
|
|
4
4
|
|
|
5
5
|
## なぜ配送が要るのか
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
`docs/maintainers/rule-delivery-rationale.md` → "Why delivery is needed"。
|
|
8
8
|
|
|
9
9
|
## 判定基準 — 引き金を特定できるか
|
|
10
10
|
|
|
11
|
-
|
|
11
|
+
どの規則を配送へ移すかは [`residency.md`](./residency.md) →「第 1 問」に従う。
|
|
12
12
|
|
|
13
13
|
## 機構 — 1 本だけ、新規に作らない
|
|
14
14
|
|
|
15
|
-
|
|
15
|
+
入口は `PreToolUse` の `pnpm josh pretool:guard`(バッチング / 調査 / 規則の 3 ガード)と Stop フックの `pnpm josh stop:guard` の 2 つ。**2 本目の配送経路を作るのは `CLAUDE.md` →「No clones」が禁じるクローン**である。配送は呼び出しの書き換え(`updatedInput`)でもよい — 規則の結果そのものを機械的に作れる形に限る。書き換えは `pnpm josh rule:value` が `rewritten` として数える。配線・停止スイッチ・拒否の優先順位: `docs/maintainers/rule-delivery-rationale.md` → "The mechanism"。
|
|
16
16
|
|
|
17
17
|
## 配送されている規則
|
|
18
18
|
|
|
19
|
-
一覧は `pnpm josh rule:list` が `scripts/rules/delivered-rules.ts`
|
|
19
|
+
一覧は `pnpm josh rule:list` が `scripts/rules/delivered-rules.ts` から生成する(joshuafolkken/kit#3399)。各項目は規則の単一ソース・発火点・発火しないときを持ち、フックが走らないエージェントはその出力を自己適用のチェックリストにする(`principles.md`)。
|
|
20
20
|
|
|
21
|
-
**引き金はシェルのコマンド文字列しか見えない**(node 内の REST や `gh api --input <file>`
|
|
21
|
+
**引き金はシェルのコマンド文字列しか見えない**(node 内の REST や `gh api --input <file>` は掛からない)ので、常駐側にトリガ 1 行を残す(`docs/maintainers/rule-delivery-rationale.md` → "Shell evaluation of a body is one more row of the same mechanism")。
|
|
22
22
|
|
|
23
|
-
|
|
23
|
+
引き金が発火しないターンでは何も起きないのが仕様で、各項目の発火しない状態は「規則が守られている状態」と一致させる。一致が取れない規則は常駐に残す。
|
|
24
24
|
|
|
25
25
|
## 配送は 1 ラン 1 回
|
|
26
26
|
|
|
27
|
-
|
|
27
|
+
配送文はラン 1 回につき 1 度しか出ないので、**「同じ呼び出しをもう一度出せ」と明記する**。例外: 繰り返す行為を止める項目は毎回発火し(joshuafolkken/kit#1570)、前提の行為を求める項目は前提が末尾に現れるまで毎回拒否する(joshuafolkken/kit#2807)— その配送文にはそれぞれ「これは毎回発火する」「前提が末尾に現れるまで拒否する」と明記する。バッチングと調査の 2 ガードは違反が間隔分続けば再び発火する(joshuafolkken/kit#2164)。前提の証拠を読めないときは拒否する。経緯: `docs/maintainers/rule-delivery-rationale.md` → "One delivery per run"。配送文の長さの上限と経緯の置き場所: `docs/maintainers/rule-delivery-rationale.md` → "Why a refusal is the action and the next command"。
|
|
28
28
|
|
|
29
29
|
## マーカーテスト
|
|
30
30
|
|
|
31
|
-
|
|
31
|
+
`docs/maintainers/rule-delivery-rationale.md` → "Marker tests"。
|
|
@@ -1,25 +1,22 @@
|
|
|
1
1
|
## 本文をシェルの二重引用符に載せない(joshuafolkken/kit#1198)
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
`CLAUDE.md` からは [`rule-delivery.md`](./rule-delivery.md) →「配送されている規則」を経て届く。[`file-edits.md`](./file-edits.md) が防ぐのはコストの二重払い、こちらが防ぐのは**テキストが実行されること**である。
|
|
4
4
|
|
|
5
5
|
### 何が起きるのか
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
二重引用符の中のバッククォートと `$` は実行前に評価される。次の呼び出しは `pnpm josh ms` を実行し、その出力を本文に埋め込む。
|
|
8
8
|
|
|
9
9
|
```bash
|
|
10
10
|
gh api repos/{owner}/{repo}/issues/1535/comments -f body="… `pnpm josh ms` は …"
|
|
11
11
|
```
|
|
12
12
|
|
|
13
|
-
|
|
13
|
+
`pnpm josh followup --notify-message "…"` も同じく壊れる。経緯は `docs/maintainers/shell-body-rationale.md` → "The incidents — two kinds of damage, one cause"。
|
|
14
14
|
|
|
15
|
-
|
|
16
|
-
- **`gh` 経路**: 置換された結果が捨てられるのではなく、**コマンドとして実行される**
|
|
17
|
-
|
|
18
|
-
**発火するのは `` ` `` と `$` であって `!` ではない**(実測表は `docs/maintainers/shell-body-rationale.md` → "Why `!` is not a trigger")。
|
|
15
|
+
発火するのは `` ` `` と `$` で、`!` ではない(`docs/maintainers/shell-body-rationale.md` → "Why `!` is not a trigger")。
|
|
19
16
|
|
|
20
17
|
### 安全な書き方
|
|
21
18
|
|
|
22
|
-
**本文をファイルに書いて、パスで渡す。**
|
|
19
|
+
**本文をファイルに書いて、パスで渡す。**
|
|
23
20
|
|
|
24
21
|
```bash
|
|
25
22
|
pnpm josh issue:comment <N> --body-file <path>
|
|
@@ -28,23 +25,16 @@ pnpm josh followup "<title> #<N>" --notify-message-file <path>
|
|
|
28
25
|
pnpm josh notify --task-type confirmation --issue-url "<url>" --body-file <path> --recommended "<choice>"
|
|
29
26
|
```
|
|
30
27
|
|
|
31
|
-
|
|
28
|
+
PR コメントも `pnpm josh issue:comment <N> --body-file <path>` で投稿する(REST では PR コメントは issue コメント)。`gh issue comment` / `gh pr comment` は使わない(`docs/maintainers/shell-body-rationale.md` → "Why no `gh` subcommand is written in an executable block")。
|
|
32
29
|
|
|
33
30
|
### コメント投稿は `pnpm josh issue:comment` の 1 綴りだけ(joshuafolkken/kit#2304)
|
|
34
31
|
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
- `gh api ... -F body=@<path>` / `--field body=@<path>` — `@` で始まる値を**ファイルとして読む**。正しい
|
|
38
|
-
- `gh api ... -f body=@<path>` / `--raw-field body=@<path>` — 値を**そのまま送る**。`@<path>` というリテラル文字列がコメントとして投稿される
|
|
39
|
-
|
|
40
|
-
`-f` と `-F` は 1 文字違いで、どちらでも `gh` は終了コード 0 でコメント URL を返す。**壊れたことが分かるのは後で人が Issue を見たときだけ**である(`docs/maintainers/shell-body-rationale.md` → "The comments lost to `-f body=@`")。
|
|
41
|
-
|
|
42
|
-
だから **Issue/PR コメントの投稿は `pnpm josh issue:comment <N> --body-file <path>` ただ 1 綴りに寄せる** — 間違ったフラグを選ぶ余地が無く、本文は `cli-body.ts` を通ってパスで渡るためシェルも評価しない。`-f body=@…` の綴りは実行前に `pnpm josh rule:guard` が拒否する(列挙表の `raw-field-body` 行、`scripts/rules/raw-field-body.ts`)。上の Issue body 書き換え(`-X PATCH ... --field body=@<path>`)は `-F` 側で安全なので綴りを変えない。
|
|
32
|
+
`gh api` の `-f body=@<path>` / `--raw-field body=@<path>` は `@<path>` というリテラルを黙って投稿する(`docs/maintainers/shell-body-rationale.md` → "The comments lost to `-f body=@`")。**Issue/PR コメントの投稿は `pnpm josh issue:comment <N> --body-file <path>` ただ 1 綴りに寄せる。** `-f body=@…` は `pnpm josh rule:guard` が実行前に拒否する(`raw-field-body` 行、`scripts/rules/raw-field-body.ts`)。Issue body 書き換えの `-X PATCH ... --field body=@<path>` は `-F` 側で安全なのでそのまま使う。
|
|
43
33
|
|
|
44
|
-
`--body-file` / `--notify-message-file`
|
|
34
|
+
`--body-file` / `--notify-message-file` は `-` で標準入力を読む(実装は `scripts/josh/cli-body.ts`)。`--body` と `--body-file` の同時指定は拒否される(`docs/maintainers/shell-body-rationale.md` → "Why `--body` and `--body-file` together are refused")。
|
|
45
35
|
|
|
46
|
-
**`$'…'` も安全である。** ANSI-C
|
|
36
|
+
**`$'…'` も安全である。** ANSI-C クォートの中では置換も展開も起きないので、`CLAUDE.md` の `--body=$'…'` はそのまま使う。長い本文はファイルにする。
|
|
47
37
|
|
|
48
38
|
### 引き金つき配送
|
|
49
39
|
|
|
50
|
-
`pnpm josh rule:guard`
|
|
40
|
+
`pnpm josh rule:guard` は、二重引用符の本文値にバッククォートか `$` を含む `Bash` 呼び出しを拒否する。拒否されたら本文をファイルに移して出し直す。**引き金が見えない綴り(`-b`・`-f title=` など)でも、フックが走らないエージェントでも規則は同じく拘束する。** 見えない綴りの一覧は `docs/maintainers/shell-body-rationale.md` → "Why it is a triggered delivery"、引き金の設計は `docs/maintainers/shell-body-rationale.md` → "Why the trigger reads the body, not the flag"、固定するテストは `docs/maintainers/shell-body-rationale.md` → "Marker tests"。
|
|
@@ -1,38 +1,32 @@
|
|
|
1
1
|
# 独立した呼び出しは同じターンに載せる(joshuafolkken/kit#1304)
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
常駐側には [`rule-delivery.md`](./rule-delivery.md) →「配送されている規則」への導線だけが残り、規則本文は `pnpm josh batch:guard` が効く瞬間に配送する(`docs/maintainers/turn-batching-rationale.md` → "Why it left residency (joshuafolkken/kit#1524)")。
|
|
4
4
|
|
|
5
5
|
## 何を求めているか
|
|
6
6
|
|
|
7
|
-
**判断は 1 問で決まる。**
|
|
8
|
-
|
|
9
7
|
> **この呼び出しの入力は、いま出していない別の呼び出しの結果に依存しているか。**
|
|
10
8
|
|
|
11
|
-
- no →
|
|
9
|
+
- no → その呼び出しは、依存していない他の呼び出しと同じターンに載せる
|
|
12
10
|
- yes → 結果を待つ。別のターンになるのが正しい
|
|
13
11
|
|
|
14
|
-
|
|
12
|
+
ファイル 3 つを 3 ターンで読む、`git status` と `pnpm josh issue:state` を別ターンに分ける — いずれも禁止。**費用は仕事の量ではなく往復の回数にある。**(`docs/maintainers/turn-batching-rationale.md` → "Why the number of turns is the cost")
|
|
15
13
|
|
|
16
14
|
## 判断基準は「依存の有無」であって、呼び出しの種類ではない
|
|
17
15
|
|
|
18
|
-
**`Edit` も同じ規則の対象である** —
|
|
16
|
+
**`Edit` も同じ規則の対象である** — 別ファイルや離れた領域への編集は同じターンに載せる。依存があるなら分ける: `grep` の行番号で `sed -n` を出す、`git switch` の成否を見てから次を出す、Edit の前に対象を読む(省略できない)。
|
|
19
17
|
|
|
20
18
|
## 検証ゲートとレビューは往復を減らす対象ではない
|
|
21
19
|
|
|
22
|
-
|
|
20
|
+
ゲートの失敗の内訳を読まずに進む、差分を一部だけ読んで「指摘なし」とする(`output-bounds.md`)、チェックを落とす — いずれも禁止。**往復を減らすのは「同じ仕事を少ないターンで出す」ことであって、「仕事を減らす」ことではない。**
|
|
23
21
|
|
|
24
22
|
## ガードが見えないところ — ここでは自分で規則を当てる
|
|
25
23
|
|
|
26
|
-
中断中のターン・`Write`・書き込む `josh` サブコマンドや連鎖した行・ツールを呼ばないターン・ガードが `off`
|
|
24
|
+
中断中のターン・`Write`・書き込む `josh` サブコマンドや連鎖した行・ツールを呼ばないターン・ガードが `off` のレーン子でも、規則は同じく拘束する。一覧は `docs/maintainers/turn-batching-rationale.md` → "Where the guard cannot see"、却下した案は `docs/maintainers/turn-batching-rationale.md` → "Rejected mechanisms" と `docs/maintainers/turn-batching-rationale.md` → "How the guard's scope grew"。
|
|
27
25
|
|
|
28
26
|
## 事後の計測 — 実行タイミングレポートの `Round trips:`
|
|
29
27
|
|
|
30
|
-
|
|
28
|
+
round trip は 1 ターンがまとめて出した呼び出しのかたまり 1 つで、1.50 呼び出し/往復を下回ると 1 行の指摘が出る。`tool-less turns`(発話だけのターン)は自分で減らす。実装は `scripts/time-runtime/time-round-trips.ts`、読み方は `docs/maintainers/turn-batching-rationale.md` → "Reading the round-trip measurement"。
|
|
31
29
|
|
|
32
30
|
## 実装中の独立編集に効く合成コマンド(joshuafolkken/kit#2202)
|
|
33
31
|
|
|
34
|
-
**Step 0
|
|
35
|
-
|
|
36
|
-
## マーカーテスト
|
|
37
|
-
|
|
38
|
-
`scripts/rules/turn-batching-rule.test.ts` が固定する。内訳は `docs/maintainers/turn-batching-rationale.md` → "What the marker test pins"。
|
|
32
|
+
**Step 0 が対象ファイルを列挙したら、`pnpm josh read:files <path> …` で読み取りを 1 回に畳み、続く独立した編集は `pnpm josh edit:files` の 1 呼び出しで送る**(`old` が 1 箇所に一致しなければ拒否される)。ガードが `read:files` / `edit:files` を手渡したら、その 1 呼び出しで出し直す。形式は `docs/josh-commands-run.md` →「`josh read:files`」と「`josh edit:files`」、経緯は `docs/maintainers/turn-batching-rationale.md` → "The composite commands"。固定するテストは `docs/maintainers/turn-batching-rationale.md` → "What the marker test pins"。
|