Skip to content

chore: packages/core(1272e81d 以降に足された分)の無駄なコメントを消す - #4401

Merged
takecchi merged 1 commit into
mainfrom
chore/core-comments-3710
Oct 10, 2026
Merged

takecchi merged 1 commit into
mainfrom
chore/core-comments-3710

Conversation

@takecchi

Copy link
Copy Markdown
Owner

Refs #3710

packages/core で、2026-10-09 の到達点の報告の時点(1272e81d)より後に足されたコメントを整理した。コメントだけの変更で、コードは変えていない。

基準と、その根拠

AGENTS.md の「情報の置き場所」に従う(R2 と同じ基準)。コードのコメントには Why not(単純にすると壊れる理由)だけを残す。How はコードで、What は試験の名前で、Why はコミットログで表す。Issue 番号・経緯・移設の記録は消す。lint を無効にするコメントや道具への指示は残す。

オーナーの承認(クローンの手元の記録から):

  • 2026-10-07T00:46Z、承認 f6e6bad8「この基準で全体に広げてよい」
  • 2026-10-07T01:17Z、オーナーの逐語「AGENTS.mdに則って整理してくれませんか?とても大きな削除になると思いますが想定通りです。」

範囲

残したもの・消したもの(主なもの)

  • 消した: Issue 番号(#4339・#4355・#4357・#4358・#4377・#4391・#4394・#2987・#3531・#4284・#4263)、経緯の段落(commitment-remove-for-conversation-contract.ts の「経緯(会話を論理削除しても、台帳のクローンの行と、台帳へ積んだときの日誌の写しから本文が読める(#4219 の漏れ・要判断) #4355)」)、試験の冒頭で試験の名前を言い直していた段落、公開の型の欄を言い直していた doc(EventReceipt の各欄など)
  • 縮めて残した: 理由の一文
    • REPLY_MESSAGE_SEPARATOR を SSE に入れない理由
    • 区切りを次のメッセージの先頭に置く理由
    • self の台帳の行も消す理由
    • 墓標に id を持たせる理由
    • binary で形だけの規則を見ない理由
  • 残した: /* この検証では使わない */(空の関数の lint 避け)、eslint-disable-next-line、道具の挙動と安全の理由(manager-tools.ts の token が守りの本体であること、など)

確かめたこと

  • コメントを除いた構文木が main と一致する: 24 files, mismatches 0。 TypeScript の printer を removeComments: true で回し、origin/main の版と比べた。差を拾えることも確かめた: 同じ道具で 1272e81d の clone.ts と比べると DIFF になる
  • 消した文言を、他所が逐語で引いていないこと: 触ったファイル名を grep -F / readFileSync の形で読んでいる箇所は scripts/conversation-window-single-source.test.ts だけだった。これはコードの形(journal.list の手組み)を見る歯で、コメントは見ていない
  • npx prettier --check(変えた24ファイル)通過、pnpm --filter @alteroid/core typecheck exit 0、vitest run src/clone-reply-segment-separator.test.ts → Tests 8 passed (8)
  • 全体の試験は CI に任せた(手元では回していない)

確かめていないこと

  • 消したコメントのうち「What だったのか、守らないと壊れる理由だったのか」は私の判定であって、確認ではない。迷ったもの(clone-follow-up-reply-order.test.ts の queued の位置の理由、clone-reply-segment-separator.test.ts の切り詰めと Web の行の理由)は残した

Alteroid-Issue-Done: none

@takecchi
takecchi merged commit 8066928 into main Oct 10, 2026
8 checks passed
@takecchi
takecchi deleted the chore/core-comments-3710 branch October 10, 2026 04:01
takecchi added a commit that referenced this pull request Oct 10, 2026
packages/core の残り(store.ts ほか4ファイル)の無駄なコメントを、承認済みの基準(R2)で削る。Refs #3710
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