@joshuafolkken/kit 1.1453.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-handoff-report.md +1 -1
- 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/halfrun.md +2 -1
- 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/prrun.md +2 -1
- 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 +7 -6
- package/README.md +2 -9
- package/dist/CLAUDE.md +7 -6
- package/dist/hooks/batch-guard.js +6 -6
- package/dist/hooks/{chunk-Y3GKIKGW.js → chunk-43ZHT22P.js} +31 -15
- package/dist/hooks/{chunk-MBJYQPEK.js → chunk-AS74OU3D.js} +24 -8
- 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-NO4AALP5.js → chunk-H5HVWYXK.js} +4 -4
- package/dist/hooks/{chunk-5C7L6FRC.js → chunk-IIGFYXKC.js} +346 -241
- package/dist/hooks/{chunk-W5ZKGTSY.js → chunk-J47HU5H5.js} +14 -3
- package/dist/hooks/{chunk-OLMO72ON.js → chunk-K4EOONYM.js} +1 -1
- package/dist/hooks/{chunk-LBFYEOXJ.js → chunk-K4LMBQC2.js} +4 -4
- package/dist/hooks/{chunk-D3NJXN5M.js → chunk-KC5AK2GF.js} +8 -6
- package/dist/hooks/chunk-SHGPANFB.js +56 -0
- package/dist/hooks/{chunk-EF5GQ6WS.js → chunk-YHBZAJSV.js} +3 -3
- package/dist/hooks/{chunk-SHYUQOIS.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 +251 -247
- 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 +75 -18
- package/dist/josh.js +6 -7
- 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 +11 -21
- 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-ai-review.ts +12 -1
- package/scripts/gh/git-pr-confirmation.ts +11 -7
- 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/josh/josh-commands-workflow.ts +1 -1
- 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/notify/confirmation-options.ts +59 -0
- package/scripts/notify/notify-opt-in.ts +33 -0
- package/scripts/notify/telegram-notify.ts +25 -8
- package/scripts/notify/telegram-test-logic.ts +33 -1
- package/scripts/notify/telegram-test.ts +4 -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 +7 -8
- package/scripts/rules/shell-segments.ts +7 -0
- package/scripts/rules/stop-rules.ts +17 -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/entry/run-entry-stop.ts +6 -3
- 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 +3 -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
|
@@ -8,34 +8,29 @@
|
|
|
8
8
|
|
|
9
9
|
### フェーズ分離
|
|
10
10
|
|
|
11
|
-
|
|
11
|
+
- Planning phase(`kickoff`): Step 1〜3 — Issue 作成・計画投稿・Telegram 通知で止まる。実装前にレビューや承認を挟むときに使う。
|
|
12
|
+
- Execution phase(`fullrun #N`): Step 4〜5 — 既存 Issue の計画に基づいて実装・PR 作成・完了通知を行う。
|
|
12
13
|
|
|
13
|
-
|
|
14
|
-
- **Execution phase(`fullrun #N`)**: Step 4〜5 — 既存 Issue の計画に基づいて実装・PR 作成・完了通知を行う。
|
|
15
|
-
|
|
16
|
-
一括実行する場合は `fullrun new` で Step 1〜5 を通しで実行する。
|
|
14
|
+
`fullrun new` は Step 1〜5 を通しで実行する。
|
|
17
15
|
|
|
18
16
|
### 出力の言語(`JOSH_SESSION_LANG`)
|
|
19
17
|
|
|
20
|
-
|
|
18
|
+
開発者との対話と成果物の散文は、環境変数 `JOSH_SESSION_LANG`(例: `ja` / `en`)の言語で書く。**未設定・空・`.env` なしのときは `ja`**。
|
|
21
19
|
|
|
22
|
-
-
|
|
23
|
-
-
|
|
20
|
+
- 対話: セッション中の説明・質問、`AskUserQuestion` の選択肢ラベル・説明
|
|
21
|
+
- 成果物の散文: Issue 本文、Issue/PR コメント(Step 3 の計画コメント、`pnpm josh followup` の完了コメントを含む)、Telegram 通知の本文(`--body` / `--notify-message`)
|
|
24
22
|
- **設定に関わらず英語で固定するもの**(3 つ):
|
|
25
|
-
1.
|
|
26
|
-
2.
|
|
27
|
-
3.
|
|
23
|
+
1. Issue/PR タイトル(Step 1 のタイトル正規化ルール)
|
|
24
|
+
2. コード内コメント、テストタイトル(`describe` / `it` / `expect`)、コミットメッセージ
|
|
25
|
+
3. スクリプトが出力する固定文字列: Telegram のヘッダーラベル(`Planning` / `Completion` など)、`Issue:` / `PR:` の URL ラベル、`--notify-message` 省略時の既定メッセージ
|
|
28
26
|
|
|
29
|
-
`JOSH_SESSION_LANG`
|
|
27
|
+
`JOSH_SESSION_LANG` は開発者個人の設定で、`.env`(gitignore 済み)に置く。設定値は毎ターン `UserPromptSubmit` フック(`scripts/hooks/run-hook.sh session-lang`、Codex は `.codex/hooks.json`)が差し込み、未設定なら出さない。フックが走らないハーネス(Gemini / Cursor)は `.env` を読む。既定の理由は `docs/maintainers/overview-rationale.md` → "Why the session language defaults the way it does"。
|
|
30
28
|
|
|
31
29
|
### 配布ドキュメントの層ごとの言語(joshuafolkken/kit#2997)
|
|
32
30
|
|
|
33
|
-
ドキュメントは**層ごとに 1
|
|
31
|
+
ドキュメントは**層ごとに 1 つの言語**で書き、層の中で言語を混ぜない(`JOSH_SESSION_LANG` に依らない)。
|
|
34
32
|
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
| `CLAUDE.md`、`AGENTS.md` / `GEMINI.md`、スキル(`.claude/skills/`)、フックが差し込む文 | 英語 |
|
|
38
|
-
| `prompts/` のトピックファイル(次の行の 3 文書を除く) | 日本語 |
|
|
39
|
-
| `prompts/review.md`、`prompts/review-rubric.md`、`prompts/sonar-hotspot-handling.md` | 英語 |
|
|
33
|
+
- 英語: `CLAUDE.md`、`AGENTS.md` / `GEMINI.md`、スキル(`.claude/skills/`)、フックが差し込む文、`prompts/review.md`、`prompts/review-rubric.md`、`prompts/sonar-hotspot-handling.md`
|
|
34
|
+
- 日本語: それ以外の `prompts/` のトピックファイル
|
|
40
35
|
|
|
41
|
-
|
|
36
|
+
英語の層から日本語のトピックファイルの節を指すときは、見出しの先頭の**英語 ID** を引く(例: `## no-clones — クローン禁止・…` を `principles.md` → "no-clones"。`pnpm josh doc:section` は前方一致で解決する)。ID の無い日本語見出しは原文のまま引く。
|
|
@@ -1,8 +1,8 @@
|
|
|
1
1
|
## Step 3: 計画コメントを記録して通知する
|
|
2
2
|
|
|
3
3
|
1. 提案を人間が判断する
|
|
4
|
-
2. 採用した計画を Issue
|
|
5
|
-
-
|
|
4
|
+
2. 採用した計画を Issue に記録する。body が空なら計画をファイルに書き `gh api -X PATCH repos/{owner}/{repo}/issues/<N> --field body=@<path>` で書き込む(本文はパスで渡す — `shell-body.md`)。body があれば `pnpm josh issue:comment <N> --body-file <path>` でコメントする
|
|
5
|
+
- 計画が stash の復元を含むときの書き方: `operating-rules.md` → "no-self-staging"
|
|
6
6
|
3. Telegram で計画開始を通知する:
|
|
7
7
|
|
|
8
8
|
```bash
|
|
@@ -10,12 +10,12 @@
|
|
|
10
10
|
```
|
|
11
11
|
|
|
12
12
|
- `--task-type` はヘッダーのアイコンを決める(`planning` 📋 / `completion` ✅ / `failure` ❌ / `warning` ⚠️ / `kickoff_retry` 🔄 / `confirmation` ⏸️)
|
|
13
|
-
- `warning` は
|
|
13
|
+
- `warning` は `pnpm josh followup` が自分で送る種別で、手で打たない(joshuafolkken/kit#1628)
|
|
14
14
|
- `--repo-name` と `--issue-title` は未指定なら `gh` から自動取得される
|
|
15
15
|
- Issue URL を必ず含め、箇条書きの間に改行を入れる
|
|
16
|
-
- `kickoff`
|
|
16
|
+
- `kickoff` はここで**停止**する(実装に進まない)
|
|
17
17
|
|
|
18
|
-
4.
|
|
18
|
+
4. 作業ツリーに変更(ステージ済みを含む)が既にあれば退避する:
|
|
19
19
|
```bash
|
|
20
20
|
git stash push -u -m "plan: pre-existing changes"
|
|
21
21
|
```
|
|
@@ -23,16 +23,16 @@
|
|
|
23
23
|
```bash
|
|
24
24
|
pnpm josh ms
|
|
25
25
|
```
|
|
26
|
-
6.
|
|
27
|
-
ステップ 4 で stash
|
|
26
|
+
6. `pnpm josh latest:scope` が `required` と答えたときだけ `pnpm josh latest` を実行し、`dependency-update` スキルに従う(`.claude/skills/workflow-commands/latest-gate.md`)。脆弱性への override は `pnpm-workspace.yaml` の `overrides` に書く(pnpm 11/12 は `package.json` の `pnpm.overrides` を無視する)。
|
|
27
|
+
ステップ 4 で stash したら、ここでメッセージ指定で復元する(位置指定や引数なしの `git stash pop` は使わない — joshuafolkken/kit#2050):
|
|
28
28
|
```bash
|
|
29
29
|
pnpm josh stash:pop "plan: pre-existing changes"
|
|
30
30
|
```
|
|
31
|
-
7. **作業サマリを提示してから**実装を開始する(`
|
|
32
|
-
8.
|
|
31
|
+
7. **作業サマリを提示してから**実装を開始する(`report-format.md` → "作業前サマリはいつ・どう出すか")。Issue body が埋まっていても提示し、Issue コメントとしては投稿しない。
|
|
32
|
+
8. 実装完了後、lint/test より前に `prompts/refactoring.md` に従い、高・中優先度項目が残らなくなるまでリファクタリングする
|
|
33
33
|
9. 検証ゲート(`CLAUDE.md` の Completion gate)を実行する
|
|
34
34
|
|
|
35
|
-
`pnpm josh git`
|
|
35
|
+
`pnpm josh git` を実行する(`-y` で確認プロンプトをスキップ)。**子はバージョンを上げない** — 版を決めるのは `pnpm josh release` だけ(joshuafolkken/kit#1486)。Issue タイトルは実行前に `issue-template.md` の Step 1 冒頭に沿って整える。
|
|
36
36
|
|
|
37
37
|
```bash
|
|
38
38
|
pnpm josh git -y "<issue-title> #<issue-number>"
|
|
@@ -40,7 +40,7 @@ pnpm josh git -y "<issue-title> #<issue-number>"
|
|
|
40
40
|
|
|
41
41
|
### Recovery after failed push (pre-push hook blocked)
|
|
42
42
|
|
|
43
|
-
If `pnpm josh git -y` fails at the push step
|
|
43
|
+
If `pnpm josh git -y` fails at the push step, fix the blocking issue and recover with:
|
|
44
44
|
|
|
45
45
|
```bash
|
|
46
46
|
# 1. Push manually after fixing the issue
|
|
@@ -51,8 +51,8 @@ pnpm josh pr
|
|
|
51
51
|
# equivalent: pnpm josh git -y --skip-commit --skip-push
|
|
52
52
|
```
|
|
53
53
|
|
|
54
|
-
|
|
54
|
+
Do not run `gh pr create` directly — it bypasses `build_body`, which writes `closes #N`.
|
|
55
55
|
|
|
56
56
|
### レビューからマージまで
|
|
57
57
|
|
|
58
|
-
|
|
58
|
+
レビューは実装したセッションがコミットの前に `pnpm josh review:brief` で始め、`/code-review` は `Agent` ツールのサブエージェントが実行する。周回の上限、残った指摘の振り分けと `pnpm josh epic:bundle`、マージを打つターンは `.claude/skills/workflow-commands/chain-rule.md` → "Run the review-to-merge chain"、"The review runs in a subagent, never a main-line skill load"、方針は `prompts/review.md` → "Review round cap"。**`/code-review` の出力で停止しない。**
|
|
@@ -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"。
|