docs: 実装の言語の結論を ADR 0001・0007 に反映する by yokinist · Pull Request #23 · codeforjapan/code4security · GitHub
Skip to content

docs: 実装の言語の結論を ADR 0001・0007 に反映する - #23

Open
yokinist wants to merge 2 commits into
codeforjapan:mainfrom
yokinist:docs/adr-0001-language
Open

yokinist wants to merge 2 commits into
codeforjapan:mainfrom
yokinist:docs/adr-0001-language

Conversation

@yokinist

Copy link
Copy Markdown
Collaborator

概要

#7 の結論(yu23ki14 さんのコメント)を、ADR 0001 と ADR 0007 に反映します。

最終的には TypeScript に揃える。取り込み分はまず Python そのままで取り込んで、それを順次 TS に移していく、細かく切ってやっていくのが良さそう

Closes #7

変えたもの

ファイル 変更
ADR 0001 の決定 最終的にすべて TypeScript にそろえる。weekly-security-checker は Python のまま取り込み、そのあと小さく分けて移す。v1(12/12)までに移し終えることは求めない
ADR 0001 の決定(途中の扱い) checker を移し終えるまでは、verify は MCP サーバーだけが使い、定期実行の checker は今の検知のまま動かす。移している途中に新しく足す検知は TypeScript で書く
ADR 0001 の根拠・懸念点 「時期」を、一度に書き直さない理由に置き換えた。懸念点に「2 つの言語が並ぶ期間」を足した。「yokinist さんの意向と合わない」は、本人の私が出す PR なので外した
ADR 0001 の今後の方向性 移す作業の切り方と担当は #18 の取り込みのあとに決める。action の形は TypeScript 版を action にするときに #12 で決める。見直すきっかけを「2 つの言語が並ぶ期間が長引いたとき」にした
ADR 0001 の決定の記録 欄を足した(議論した場所は #7。決定の場・承認者・承認日は、このレビューで埋める)
ADR 0007 起動ルートの表の下と、決定 4 に、移し終えるまでの扱いを 1 文ずつ足した

レビューしてほしいところ

#7 の結論に含まれていなかった次の 3 点は、私の案です。

  1. v1(12/12)までに移し終えることは求めない。v1 の checker は Python のままでよい
  2. 移している途中に新しく足す検知は、TypeScript で書く。Python の部分を増やさないため
  3. verify を実行する処理は、まず MCP サーバー(手元の確認)で使う。定期実行の checker は、TypeScript に移し終えるまで今の Python の検知のまま動かし、verify は使わない。そのため、移し終えるまでは手元と定期実行の結果が食い違うことがある

移す作業の切り方と担当は、#18 の取り込みのあとに決める形にしました。

マージの前に

ADR 0001 の注記どおり、10/24 まで #7 で意見を受け付けます。10/24 を過ぎて異論がなければ、approve をもらってから ADR 0001 のステータスを 承認済み にし、「決定の記録」を埋めてマージします(ADR 0010)。

🤖 Generated with Claude Code

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.

[論点] 実装の言語を TypeScript にそろえるかを決める

1 participant