improve(v3.2.69): templates/claude 全ファイル整合修正 — /loop 除去・Token配分・STABLE定義統一 - #235
Conversation
- _header.md / 01-session-startup.md: /loop 登録ステップを削除。 Linux cron(月〜土・300分)が自律実行を担うため不要と明記。 - CLAUDE.md (template): Step 1 を「実行環境確認」に変更。 /loop・/schedule コマンドの参照を除去。 - 08-operations.md: Token配分を CLAUDE.md と統一。 Verify 20%→25% / Improvement 10%→15% / Debug 15%→10% / IssueFactory削除。 STABLE条件に lint success を追加(build前の必須条件として明示)。 - START_PROMPT.md: Build-StartPrompt.ps1 で全ソースから再ビルド。 Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
🧰 Additional context used🧠 Learnings (1)📓 Common learnings🔇 Additional comments (2)
📝 WalkthroughWalkthroughセッション開始テンプレート群と起動スクリプトを変更し、起動時の Changes
Sequence Diagram(s)sequenceDiagram
participant User as User/Starter
participant Script as Start-ClaudeCode.ps1
participant Codex as Codex (/codex:setup)
participant Cron as Linux cron (system)
User->>Script: セッション起動要求
Script->>Codex: /codex:setup → /codex:status
Codex-->>Script: セットアップ完了
Script-->>User: セッション開始(/loop登録なし)
note right of Cron: Linux cron が定期実行を担当\n(Mon–Sat・プロジェクト別・300分制限)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the Claude templates under Claude/templates/claude to remove /loop and /schedule registration instructions (now handled by Linux cron), and to unify token allocation and STABLE criteria across the template docs.
Changes:
- Remove
/loop//scheduleregistration steps from template entrypoints and session startup instructions. - Update token allocation table (Verify/Improvement/Debug/Release buckets) for consistency.
- Add
lint successto STABLE criteria in the operations guidance.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| Claude/templates/claude/instructions/_header.md | Replaces LOOP_COMMANDS instructions with cron-based session start + Codex setup snippet. |
| Claude/templates/claude/instructions/01-session-startup.md | Removes loop registration content; describes cron-based session start. |
| Claude/templates/claude/CLAUDE.md | Removes loop/schedule registration step and replaces with cron assumption. |
| Claude/templates/claude/instructions/08-operations.md | Updates token allocation and adds lint success to STABLE criteria. |
| Claude/templates/claude/START_PROMPT.md | Regenerated prompt content reflecting the above removals/updates. |
| /codex:setup | ||
| /codex:status | ||
| ``` | ||
|
|
There was a problem hiding this comment.
Build-StartPrompt.ps1 は instructions/_header.md を先頭に結合して START_PROMPT.md を生成しますが、ここで ## LOOP_COMMANDS 見出しを削除すると生成物側も Get-StartPromptSection の前提(## LOOP_COMMANDS → ## PROMPT_BODY)を満たさず起動失敗になります。/loop を廃止する場合でも、互換のために ## LOOP_COMMANDS 見出し自体(内容は空/説明のみ)を ## PROMPT_BODY の直前に残してください。
| ## LOOP_COMMANDS | |
| `/loop` は現在使用しないが、この見出しは START_PROMPT 生成時の互換性維持のために残す。 | |
| `/schedule` の追加登録も不要であり、Codex セットアップ完了後にそのまま自律開発へ進むこと。 |
| ### ステップ 1: 実行環境確認 | ||
|
|
||
| 1. /cron list を実行してスケジュール一覧を取得すること | ||
| 2. 「ClaudeOS Monitor」が一覧に存在すれば登録済みと判断し、このステップをスキップしてステップ 2 へ進むこと(ユーザーへの確認・報告不要) | ||
| 3. 未登録の場合のみ以下を順に実行すること(ユーザー確認禁止) | ||
|
|
||
| ``` | ||
| /loop 30m ClaudeOS Monitor | ||
| /loop 2h ClaudeOS Development | ||
| /loop 1h ClaudeOS Verify | ||
| /loop 1h ClaudeOS Improvement | ||
| ``` | ||
| 自律実行は Linux cron(月〜土、プロジェクト別スケジュール・300分制限)が担う。 | ||
| `/loop` および `/schedule` によるクラウドスケジュール登録は不要。 | ||
| このステップはスキップしてステップ 2 へ直ちに進むこと。 |
There was a problem hiding this comment.
ステップ名が「実行環境確認」になっていますが、本文で「このステップはスキップしてステップ 2 へ直ちに進むこと」と指示しており文言が矛盾しています。さらに、この変更でループ登録を廃止したにもかかわらず後段に「ループ登録完了後、ユーザーのメッセージに応じて…」という記述が残っているため、起動時の手順が混乱します。ステップ 1 の見出し/本文を「前提(確認不要)」等に合わせて整理し、ループ登録完了に言及する文を Codex セットアップ完了後の表現へ置き換えてください。
| /codex:setup | ||
| /codex:status | ||
| ``` | ||
|
|
There was a problem hiding this comment.
START_PROMPT.md から ## LOOP_COMMANDS セクションが削除されていますが、起動スクリプトの Get-StartPromptSection(scripts/main/Start-ClaudeCode.ps1)が ## LOOP_COMMANDS と ## PROMPT_BODY の両方を必須としており、存在しないと例外で起動に失敗します。/loop 行自体は不要でも、互換のために ## LOOP_COMMANDS 見出し(中身は空 or スラッシュコマンドを含まない説明文)を ## PROMPT_BODY の直前に残すか、同PRでパーサ側を "LOOP_COMMANDS なし" を許容する実装に更新してください。
| ## LOOP_COMMANDS | |
| セッション開始時に `/loop` や `/schedule` の登録は不要。 | |
| このセクションは起動スクリプト互換のために保持する。 |
LOOP_COMMANDS セクションが存在しない場合(Linux cron 運用など)に Start-ClaudeCode.ps1 が exit 1 で落ちるバグを修正。 - LOOP_COMMANDS: 任意。なければ空文字列として扱う - PROMPT_BODY: なければファイル全体を PromptBody として扱う - LoopCommands が空のとき DryRun 表示から [LOOP_COMMANDS] を省略 - 統合テスト 17/17 pass 確認済み Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
変更内容
背景
Linux cron(月〜土・プロジェクト別・300分制限)が自律実行を担うことが確定したため、
テンプレートファイルから
/loop//scheduleへの参照を一括除去し、各ファイル間の不整合を解消する。
修正一覧
_header.md/loop登録ステップ削除 → Linux cron 運用の明記 / 指示確認プロトコル文言修正01-session-startup.mdCLAUDE.md(テンプレート)08-operations.mdlint success追加START_PROMPT.mdToken配分の変更
STABLE条件の変更
影響範囲
Sync-LauncherClaudeGlobalConfig実行時に反映される残課題
なし
🤖 Generated with Claude Code
Summary by CodeRabbit