Skip to content

docs: Claude Code Web タスク pickup + ADR-041 振り直し + 順位 135 採番 convention - #158

Merged
aloekun merged 1 commit into
masterfrom
docs-claude-code-web-pickup-adr041
May 17, 2026
Merged

docs: Claude Code Web タスク pickup + ADR-041 振り直し + 順位 135 採番 convention#158
aloekun merged 1 commit into
masterfrom
docs-claude-code-web-pickup-adr041

Conversation

@aloekun

@aloekun aloekun commented May 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • docs/claude-code-web-tasks.md 新規作成: Linux Claude Code Web で対応可能な docs-only 6 task (順位 69 / 104 / 116 / 120 / 127 / 134) をピックアップ、ephemeral 計画書として lifecycle 明記
  • 順位 78 (Rust timestamp arithmetic safety) の ADR 番号振り直し: 旧予約番号 ADR-038 が Local LLM finding classification で占有済と 2026-05-16 セッションで判明、ADR-041 に変更 (docs/todo5.md 7 箇所 + docs/todo-summary.md L52)
  • 順位 135 新規登録: ADR 番号 hardcode 撤廃 — entry 登録時は「ADR-NNN (採番未確定、land 時に確定)」placeholder で記述する convention を ~/.claude/rules/common/docs-governance.md に追記する Tier 3 / Effort XS task。順位 78 番号 conflict の再発防止

Notes

Test plan

  • markdownlint clean (全 4 files)
  • Pre-push lint-screen + simplicity-review approved (all("approved") x2 iterations)
  • CodeRabbit review 通過確認 (PR 作成後)

Summary by CodeRabbit

  • Documentation
    • ADR参照番号の更新・整理を実施
    • Claude Code Webセッションの採用タスク選定ガイドを追加
    • タスク管理ドキュメントの手順フロー・運用方針を整備

Review Change Stack

- docs/claude-code-web-tasks.md 新規: Linux Claude Code Web で対応可能な 6 task
  (順位 69/104/116/120/127/134) をピックアップ。global rule (~/.claude/*) 系と
  実装系の除外理由を周辺情報として記載
- 順位 78 (Rust timestamp arithmetic safety) の ADR 番号振り直し: 旧予約番号 038 が
  Local LLM finding classification で占有済と 2026-05-16 セッションで判明、ADR-041 に変更
- docs/todo5.md (7 箇所) + docs/todo-summary.md L52 を更新
- 順位 135 新規登録: ADR 番号 hardcode 撤廃 — entry 登録時は「ADR-NNN (採番未確定、
  land 時に確定)」placeholder で記述、land 時 PR で空き番号確定の convention を
  ~/.claude/rules/common/docs-governance.md に追記する Tier 3 / Effort XS task。
  順位 78 conflict の再発防止
@coderabbitai

coderabbitai Bot commented May 16, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 585f35de-0791-4fd9-9eb4-d1c85a602087

📥 Commits

Reviewing files that changed from the base of the PR and between f2e58be and 3a6c2ee.

📒 Files selected for processing (4)
  • docs/claude-code-web-tasks.md
  • docs/todo-summary.md
  • docs/todo5.md
  • docs/todo8.md

📝 Walkthrough

Walkthrough

This PR introduces Claude Code Web task guidance documentation and establishes a new ADR numbering policy. The new claude-code-web-tasks.md document defines task adoption criteria and lifecycle, while parallel updates migrate ADR references from ADR-038 to ADR-041 and establish an ADR-NNN placeholder convention for pre-landing documentation.

Changes

Documentation task organization and ADR numbering governance

Layer / File(s) Summary
Claude Code Web tasks framework
docs/claude-code-web-tasks.md
New document establishing scope boundaries (repo-only edits, no Rust/Windows hook/pnpm execution success criteria), adoption task criteria table, step-by-step workflow, out-of-scope rationale, and post-landing retirement procedures for Claude Code Web sessions.
ADR numbering policy and migration
docs/todo-summary.md, docs/todo5.md, docs/todo8.md
ADR-038 references updated to ADR-041 across task documents; new task added to todo-summary for ADR-NNN placeholder adoption; todo8.md adds full governance policy including placeholder rules, land-time substitution workflow, hardcode remnant verification, and completion criteria tied to docs-governance documentation updates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルは PR の主要な変更内容(Claude Code Web タスク pickup、ADR-041 振り直し、順位 135 採番 convention)を正確に反映しており、変更セット全体の主な目的を明確に要約している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aloekun
aloekun merged commit c378816 into master May 17, 2026
1 check passed
@aloekun
aloekun deleted the docs-claude-code-web-pickup-adr041 branch May 17, 2026 04:03
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