Skip to content

docs: 順位260-262 — accuracy vs 安全性 tension 明文化 + spike 見送り convention + SaaS 無料枠チェックリスト - #253

Merged
aloekun merged 1 commit into
masterfrom
docs-260-262-bundle
Jul 6, 2026
Merged

docs: 順位260-262 — accuracy vs 安全性 tension 明文化 + spike 見送り convention + SaaS 無料枠チェックリスト#253
aloekun merged 1 commit into
masterfrom
docs-260-262-bundle

Conversation

@aloekun

@aloekun aloekun commented Jul 6, 2026

Copy link
Copy Markdown
Owner

概要

WP フィードバック採用の 3 提案(順位260-262)を 1 docs PR に bundle(ADR-035 docs-only 分離ポリシー)。

変更

順位260: ADR-038 × ADR-043 の「accuracy 向上 ≠ 安全性維持」tension を明文化

  • ADR-038 § 再利用可能な知見 と ADR-043 § 原則5 で 双方向 cross-reference を確立
  • model evaluation は accuracy と 安全軸(human_review → auto_fix 誤送ゼロ)を独立指標として評価し、trade-off 時は安全軸を優先する原則を明記
  • WP-04 で accuracy 最上位の qwen3-coder を見送り mistral:7b を維持した判断が実例。ADR-043 の fail-closed 思想(不確実性は安全側に倒す)の助言/分類層への一般化

順位261: spike 見送り(negative result)永続化 convention

  • docs/dev-conventions.md(新設)に 3 点セット convention を明文化: ① ADR に結論・実測根拠を記録、② 計画文書の状態列を更新、③ 再評価トリガー付き follow-up を Tier 5 todo 化
  • WP-01(ADR-046 却下)/ WP-04(ADR-038 amendment)の 2 例で確立したパターンを成文化

順位262: 外部 SaaS 無料枠 / 制限の調査チェックリスト

  • docs/dev-conventions.md5 次元チェックリストを追加: 月間上限 / 時間単位 rate limit / 適用単位(per-user/org/repo)/ plan tier 差 / public 特典範囲
  • WP-03 で CodeRabbit の「public リポ特典」を「rate limit 撤廃」と誤解しかけた実例に由来

配置方針

CLAUDE.md は ADR index の厳格構造を保つため、261/262 の本文は docs/dev-conventions.md(新 guide)に置き、CLAUDE.md からはリンクのみ(ADR-022 責務分離)。

検証

  • markdownlint 0 errors、cli-docs-lint OK(内部リンク・cross-ref 全て有効)
  • todo 順位260-262 を消化(todo13.md エントリ + todo-summary.md 行)

Summary by CodeRabbit

  • Documentation
    • 開発手順の案内に、見送り判断の記録方法や、外部サービスの無料枠・制限を確認するチェックリストを追加しました。
    • 関連する設計記録どうしの参照関係を整理し、判断基準のつながりを分かりやすくしました。
    • 次に取り組むタスク一覧に、新しい作業項目を追加し、設定の連携に関する整理を進めました。

…+ SaaS 無料枠チェックリスト

3 提案を 1 docs PR に bundle (ADR-035 docs-only 分離ポリシー)。

- 260: ADR-038 § 再利用可能な知見 と ADR-043 § 原則5 で「accuracy 向上 ≠ 安全性維持」
  tension を双方向 cross-reference。model evaluation は accuracy と安全軸 (human_review→
  auto_fix 誤送ゼロ) を独立評価し trade-off 時は安全軸優先。WP-04 qwen3-coder 見送りが実例。
- 261: docs/dev-conventions.md を新設し spike 見送り (negative result) の 3 点セット
  convention (ADR記録 / 計画状態更新 / Tier5 todo化) を明文化。
- 262: 同 guide に外部 SaaS 無料枠調査チェックリスト (月間上限 / 時間 rate limit /
  適用単位 / plan tier / public 特典範囲) を追加。CLAUDE.md からリンク (index 肥大化回避)。

todo 順位260-262 を消化。
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ada4668e-0320-4f2a-aca1-56f712f785e1

📥 Commits

Reviewing files that changed from the base of the PR and between 52f6d22 and ec8c086.

📒 Files selected for processing (6)
  • CLAUDE.md
  • docs/adr/adr-038-local-llm-finding-classification.md
  • docs/adr/adr-043-security-gates-fail-closed.md
  • docs/dev-conventions.md
  • docs/todo-summary.md
  • docs/todo13.md
💤 Files with no reviewable changes (2)
  • docs/todo-summary.md
  • docs/todo13.md

📝 Walkthrough

Walkthrough

ADR-038とADR-043にaccuracy/安全軸の評価方針とfail-closed原則の一般化に関する相互参照を追加。docs/dev-conventions.mdを新規作成し、spike見送り運用とSaaS無料枠調査チェックリストを記載。CLAUDE.mdにその参照を追記し、todo-summary.md/todo13.mdのタスクエントリを更新。

Changes

ADR安全軸クロスリファレンス

Layer / File(s) Summary
ADR-038への安全軸原則追記
docs/adr/adr-038-local-llm-finding-classification.md
accuracyと安全軸を独立指標として分離し安全軸を優先する方針、WP-04適用例、ADR-043への関連リンクを追記。
ADR-043への原則5追加
docs/adr/adr-043-security-gates-fail-closed.md
block判定を伴わない助言/分類層向けに「不確実時は安全側」を一般化する原則5と、ADR-038参照を追記。

開発convention文書整備とtodo更新

Layer / File(s) Summary
dev-conventions.mdの新規作成とCLAUDE.mdからの参照
docs/dev-conventions.md, CLAUDE.md
spike見送りのnegative result永続化手順と外部SaaS無料枠/制限調査チェックリストを新規文書化し、CLAUDE.mdに参照セクションを追記。
todoファイルのエントリ更新
docs/todo-summary.md, docs/todo13.md
順位263の新規タスク(クロスシステム設定coupling ADR起票)を追加し、todo13.mdの完了済みエントリを削除・置換。

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • aloekun/claude-code-hook-test#139: 両PRとも adr-038-local-llm-finding-classification.md を更新し、同文書内の異なる方針セクションを補強している。
  • aloekun/claude-code-hook-test#245: 両PRともWP-04分類器のモデル格上げ評価文書を更新し、accuracy/安全軸の枠組みを拡張している。
  • aloekun/claude-code-hook-test#248: 本PRはPR#248で提起されたADR-038/ADR-043相互参照、spike見送りconvention、todoエントリ更新の各フォローアップ項目を実際に完成させている。
🚥 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 変更内容の3本柱(accuracy vs 安全性の明文化、spike見送り convention、SaaS無料枠チェックリスト)を具体的に表している。
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs-260-262-bundle

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.

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