識別子の列挙を登録時に行う理由を architecture.md に記録する

[TwoSquirrels]

概要

「なぜ識別子の列挙を登録時 (library add) に行うのか」という、仕様案の時点から続くのに理由が書き残されていなかった中核の設計判断を、architecture.md に常設節として記録する。

• 判断: 列挙 (tree-sitter の構造解析) はライブラリ規模に比例して重いため、登録時に償却してバンドルをほぼ定数時間に保つ。
• 根拠: バンドル時に毎回解析する素朴な代替をプロトタイプで実測したところ、看板ユースケース (全部 include テンプレ、Nyaan's Library 405 ヘッダー) で 1 回あたり +1 秒超 = バンドル全体の 10 倍 ([BIG PROJECT] 識別子の抽出をバンドル時解析 + 内容ハッシュキャッシュへ移行する検討 #105 に記録)。
• 代償の明示: 変更検知と library update の存在、生ソース解析ゆえの限界 (X-macro・#ifdef)。キャッシュ付きバンドル時解析 ([BIG PROJECT] 識別子の抽出をバンドル時解析 + 内容ハッシュキャッシュへ移行する検討 #105) が同じ償却構造のままこれらを解消し得る見込みも添える。

節の位置は「tree-shaking は過剰検出側に倒す」の直前 (識別子照合の仕組みの導入部)。

🤖 Generated with Claude Code

[coderabbitai]

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: 88ed8c01-63eb-4ecc-b68d-926f6546f32d

📥 Commits

Reviewing files that changed from the base of the PR and between e0029b8 and 6e1dd2b.

📒 Files selected for processing (1)

• docs/architecture.md

---

📝 Walkthrough

Summary by CodeRabbit

• ドキュメント
  • 識別子解析を登録時に実行し、バンドル処理を高速化する方針を文書化しました。
  • 解析結果のキャッシュ、変更検知、更新時の管理方法について説明を追加しました。
  • ソース解析の制約と、将来的な改善案を追記しました。

Walkthrough

docs/architecture.md に、識別子列挙を登録時に実行して tags.json へ保存し、バンドル時の構造解析を避ける設計方針を追記した。変更検知、更新運用、解析上の制約、将来のキャッシュ案も説明している。

Changes

識別子抽出方針

Layer / File(s)	Summary
登録時解析方針の追記 docs/architecture.md	library add 時の識別子列挙と tags.json 保存、バンドル時のコスト回避、変更検知と更新運用、プリプロセス前解析の制約、内容ハッシュによる将来のキャッシュ案を記載した。

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related issues

• TwoSquirrels/risundle#105 — 登録時解析からバンドル時の内容ハッシュキャッシュへ移行する案に関連する。

Possibly related PRs

• TwoSquirrels/risundle#104 — 同じ docs/architecture.md にバンドル時の解析回避などの設計方針を追記している。

Poem

ぴょんと登録、タグを保存
バンドルの道は軽やかに
木を読む仕事は先に済ませ
古びた印は更新で直す
うさぎも設計図を祝うよ 🐇

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	タイトルは登録時に識別子列挙を行う理由を architecture.md に記録する内容と一致している。
Description check	✅ Passed	説明は常設節として設計判断と根拠・代償を記録する変更内容と一致している。
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.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[TwoSquirrels]

@CodeRabbit review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[TwoSquirrels]

@CodeRabbit review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.