Skip to content

docs(knowledge): document routine enable/disable/delete and /schedule layering#230

Merged
ozzy-3 merged 1 commit into
mainfrom
docs/routines-enable-disable-delete
May 24, 2026
Merged

docs(knowledge): document routine enable/disable/delete and /schedule layering#230
ozzy-3 merged 1 commit into
mainfrom
docs/routines-enable-disable-delete

Conversation

@ozzy-3
Copy link
Copy Markdown
Contributor

@ozzy-3 ozzy-3 commented May 24, 2026

概要

PR #229 の続き。ルーチンの無効化・有効化・削除の操作面と、/schedule(skill)/ CLI / API の層関係を追記。実機(RemoteTrigger 経由の /schedule list)で挙動を確認済み。

追記内容(ai/agents/claude-code-routines.md

  • 無効化 / 有効化: /schedule updateenabled フラグで切替(CLI 可)。Web/Desktop の Repeats トグル(pause/resume)でも可。
  • 削除: CLI 不可・公開 API 不可。Web / Desktop の detail ページのみ(過去セッションは残る)。
  • /schedule の層関係: Claude Code CLI の組み込みスラッシュコマンド(近年は bundled skill 実装)で、内部的に claude.ai の /v1/code/triggers(list/get/create/update/run)を in-process OAuth で呼ぶ。管理用の公開 REST API は無い。公開 API は起動専用の /fire のみ。
  • よくやるミスに「CLI / API でルーチンを削除しようとする」を追加し、既存の CRUD ミス項目を「公開 API は起動専用/削除・トークン管理は Web のみ」に精緻化。

検証

  • typecheck / test (77 passed) green。

🤖 Generated with Claude Code

… layering

Clarify that disabling/enabling a routine works via `/schedule update`
(enabled flag) while deletion is Web/Desktop only — CLI and the public
API have no delete action. Note that `/schedule` is a CLI slash command
(bundled skill) backed by an internal claude.ai endpoint, and that the
only public HTTP API is the fire-only `/fire`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ozzy-3
Copy link
Copy Markdown
Contributor Author

ozzy-3 commented May 24, 2026

🔍 drive review (mode: quick)

適用観点 (5/11):

  • required: correctness, security, conventions
  • ux: usability, documentation

公式 docs + 実機(RemoteTrigger 経由の /schedule list)で裏取り。

correctness

指摘なし。無効化=/schedule updateenabled 切替、削除=CLI/公開API 不可で Web のみ、内部 API /v1/code/triggers(list/get/create/update/run)は RemoteTrigger ツールの実装と一致。

security

指摘なし。内部 API は in-process OAuth で curl 非想定である旨を明記しており、誤った直叩きを誘発しない。秘密情報の混入なし。

conventions

指摘なし。コミット種別 docs(knowledge):、Markdown スタイルともリポ規約に整合。

usability

指摘なし。「無効化(CLI 可)/ 削除(Web のみ)」の区別と層関係が明快。

documentation

指摘なし。よくやるミス #6 の精緻化と #8 追加で CRUD/削除の誤解を予防。記事内の既存記述と矛盾なし。

サマリー

Critical: 0 件
Warning: 0 件
Info: 0 件

by_axis:
correctness: C0 W0 I0
security: C0 W0 I0
conventions: C0 W0 I0
usability: C0 W0 I0
documentation: C0 W0 I0

判定: 全適用観点が exit_criteria を満たす → merge-ready

@ozzy-3 ozzy-3 merged commit 3ae8578 into main May 24, 2026
3 checks passed
@ozzy-3 ozzy-3 deleted the docs/routines-enable-disable-delete branch May 24, 2026 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant