Skip to content

責務としてやらないことを architecture.md に記録する#104

Merged
TwoSquirrels merged 1 commit into
mainfrom
docs/interface-non-goals
Jul 15, 2026
Merged

責務としてやらないことを architecture.md に記録する#104
TwoSquirrels merged 1 commit into
mainfrom
docs/interface-non-goals

Conversation

@TwoSquirrels

@TwoSquirrels TwoSquirrels commented Jul 15, 2026

Copy link
Copy Markdown
Owner

概要

機能として引き受けないものとその理由を、architecture.md に「責務としてやらないこと」という常設節として記録する。「規模に合わせて、あえてやらないこと」(内部の抽象を入れない) と対になる、責務の線引きのリスト。

  • テスト・提出などの周辺工程 — oj 等と && で繋げば足りる (既に cheatsheet のレシピとして存在する判断の明文化)
  • 出力先の管理 (-o) — 純粋なフィルタであることがシェル合成の土台。> と重複する口を増やさない
  • watch・自動フォールバック — 再実行と退避は利用者ごとのポリシーで、シェルの一行で組める
  • 公開ライブラリ化 (lib.rs) — 作業単位が粗くリンクの利益が無い上、実行時のコンパイラ依存も消えない。組み込みは子プロセスで足り、公開 API は内部リファクタの自由を semver で縛るだけ。子プロセスで成立しない具体的な消費者が現れたら再考する

議論の経緯 (ライブラリ化の需要見積もり等) は Discussion #103 周辺のやり取りに基づく。

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

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: bc840e81-a1f8-44d6-8373-1e71563af354

📥 Commits

Reviewing files that changed from the base of the PR and between d3abbac and f179247.

📒 Files selected for processing (1)
  • docs/architecture.md

📝 Walkthrough

Summary by CodeRabbit

  • ドキュメント
    • アーキテクチャドキュメントに、ツールが扱わない範囲を明記しました。
    • テスト・提出、出力先へのリダイレクト、watchモードや自動フォールバック、公開ライブラリとしての提供は対象外であることを説明しています。

Walkthrough

アーキテクチャ文書に、risundle が引き受けない周辺工程、出力先管理、再実行・退避ポリシー、公開ライブラリ提供の範囲を追加しました。

Changes

責務範囲の文書化

Layer / File(s) Summary
非対象範囲の明記
docs/architecture.md
周辺工程、出力先管理、再実行・退避ポリシー、公開ライブラリ提供を risundle の対象外として記載しました。

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

Poem

ぴょんと跳ねたら責務が見える
やらないことも文書に並べる
stdout はシェルへ、月へ跳ぶ
watch もライブラリも追いかけず
にんじん片手に境界を祝う

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 変更内容の中心である「責務としてやらないこと」の追記を、architecture.md への記録として正しく要約しています。
Description check ✅ Passed テスト、出力先管理、watch、公開ライブラリ化の線引きという変更内容に沿った説明になっています。
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

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

テスト・提出、出力先管理 (-o)、watch や自動フォールバック、公開
ライブラリ化 (lib.rs) を引き受けない理由と再考条件を、「規模に
合わせて、あえてやらないこと」(内部の抽象) と対になる「機能として
引き受けないもの」のリストとしてまとめる。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@TwoSquirrels
TwoSquirrels force-pushed the docs/interface-non-goals branch from 97aab4c to f179247 Compare July 15, 2026 06:40
@TwoSquirrels TwoSquirrels changed the title 受け口を CLI に絞る設計判断を architecture.md に記録する 責務としてやらないことを architecture.md に記録する Jul 15, 2026
@TwoSquirrels TwoSquirrels changed the title 責務としてやらないことを architecture.md に記録する 責務としてやらないことを architecture.md に記録する Jul 15, 2026
@TwoSquirrels

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown
✅ 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
TwoSquirrels merged commit e86ccd8 into main Jul 15, 2026
10 checks passed
@TwoSquirrels
TwoSquirrels deleted the docs/interface-non-goals branch July 15, 2026 13:41
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