docs: コーディング規約集 docs/coding-conventions.md を新設して CLAUDE.md から参照

[thawk105]

リポジトリ固有のコーディング規約と、ファイル種別ごとに 指摘される前に 適用すべき業界 best practice を 1 か所にまとめる。CLAUDE.md からは "Read before editing" として参照。

動機

#51 で devcontainer に `htop` を追加した際、`apt-get install` リストの末尾に追加してしまい、ユーザーから「Docker docs の Sort multi-line arguments は周知の best practice。指摘されないと適用しないのは怠慢」とフィードバック。

レビューで毎回指摘するより、リポジトリ規約としてドキュメント化 した方が:

• 長期的なレビューコストが低い (1 度書けば次から自動的に参照される)
• Claude / 新規 contributor 両方に効く
• 「なぜそうするか」の理由も記録されて、規約が形骸化しにくい

構成

`docs/coding-conventions.md` (新規) — ファイル種別ごとの規約集:

• Dockerfile: apt list の alphabetical sort、`--no-install-recommends`、BuildKit cache mount、multi-stage 分割
• CMake: `target_compile_options(... PRIVATE)` 推奨、`add_definitions` 回避、universal flag は `cmake/Options.cmake` の CACHE 変数経由、`ccbench_add_protocol` 使用、`cmake_minimum_required` の慎重な扱い
• GitHub Actions workflow: action のメジャー版 pin、container job の `--user root` + `safe.directory` パターン (CI: ghcr の devcontainer image でビルドして apt install を省く #46 の経緯参照)
• C++: `tx.read` の戻り値 check 必須 (段階的に -Werror 化してビルドを堅牢にする #43 / oze: 未初期化 ScanRange union を修正 + -Werror=maybe-uninitialized を有効化 #44 の経緯)、未初期化変数の default-init、`auto ret = Status` を読み捨てる際は `(void)` 明示
• Shell: `set -euo pipefail`、long flag 推奨
• Markdown/docs: 言語混在 OK、コードブロックに言語タグ、相対リンク

`CLAUDE.md` 冒頭に "Read before editing" 節を追加して docs を必読指定。

このドキュメント自体のルール

• 初版は薄い、気付いたものから追記する
• 新しい best practice 違反を見つけたら、まずこの docs に追記して規約化する
• 規約の「理由」を短く併記する
• 例外を見つけたら例外もここに書く

Test plan

CI 動作には無影響 (`.md` のみ)。