Skip to content

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

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

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

Conversation

@takecchi

Copy link
Copy Markdown
Owner

Refs #3710

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

基準と、その根拠

#4401(core)と同じ。AGENTS.md の「情報の置き場所」に従い、コードのコメントには Why not(単純にすると壊れる理由)だけを残す。How はコードで、What は試験の名前で、Why はコミットログで表す。Issue 番号・経緯は消す。@vitest-environment などの道具への指示は残す。

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

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

範囲

消したもの・残したもの

  • 消した:
    • Issue 番号(#4391・#4339・#4357・#3418)
    • 試験の冒頭で経緯を語っていた段落。chat.reply-segment-tail.test.tsx の「会話 82d90cd7 の返答 36291767、2026-10-09」など
    • 試験の手順や期待値を言い直していたコメント
    • daemon-images.tsx の doc の1行目。返り値を言い直していた行
  • 残した:
    • 理由の一文
      • chat.tsx の照合と行の分け方の理由
      • env-vars.tsx の欄の作りの理由
      • daemon-images.tsx の資格を送ってよい線の条件
    • 試験の足場の理由(chunkedSse を自前で持つ理由、同じ塊で届く形が実在すること)
    • catch の空の本体の // 既に閉じている
    • done の前に照合を見る理由

確かめたこと

  • コメントを除いた構文木が main と一致する: 7 files, mismatches 0(TypeScript の printer を removeComments: true で回し、origin/main の版と比べた)
  • npx prettier --check(変えた7ファイル)通過
  • 触ったファイルの文言を grep -F / readFileSync / ?raw で読む箇所は見つからなかった

確かめていないこと

  • 手元では web の typecheck も試験も回していない。 この器では依存パッケージ(@alteroid/core・@alteroid/api-client)が未 build で、どちらも import の解決で落ちる。コードが変わっていないことの根拠は、上の構文木の一致だけである。試験は CI に任せた
  • どのコメントが What で、どれが守らないと壊れる理由かは私の判定であって、確認ではない

Alteroid-Issue-Done: none

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