Skip to content

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

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

takecchi merged 1 commit into
mainfrom
chore/storage-pg-comments-3710

Conversation

@takecchi

Copy link
Copy Markdown
Owner

Refs #3710

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

基準と、その根拠

#4401〜#4405 と同じ。AGENTS.md の「情報の置き場所」に従い、コードのコメントには Why not(単純にすると壊れる理由)だけを残す。Issue 番号・経緯は消す。

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

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

範囲と中身

  • 対象: git diff 1272e81d origin/main -- packages/storage-pg で足された、行頭がコメントの行(7行)
  • 消した: Issue 番号(#4355・#4377・#3531)
  • 書き直して残した: hiddenConversationExchangeExcluded の上の、名指しした行を墓標の種別で絞って引く理由
  • 残した:
    • self の台帳の行も消す理由
    • event-receipts.ts の、主キーに先勝ちを任せる理由と、期限切れの行を先に消す理由
    • attachments.ts の NUL の扱い
    • migrate.ts の、受信箱に列を足さない理由
  • 範囲の外として触っていないもの: 1272e81 より前からある行(journal.ts の (#4218) など)
  • 外したファイル: なし(open の PR は packages/storage-pg に触れていない)。migrate.ts は TypeScript のコメントだけを変えていて、SQL の文字列は1文字も変えていない

確かめたこと

  • コメントを除いた構文木が main と一致する: 3 files, mismatches 0(TypeScript の printer を removeComments: true で回し、origin/main の版と比べた)
  • npx prettier --check(変えた3ファイル)通過

確かめていないこと

  • 手元では typecheck も試験(本物の PostgreSQL のものを含む)も回していない。コードが変わっていないことの根拠は、上の構文木の一致だけである。試験は CI に任せた

Alteroid-Issue-Done: none

@takecchi
takecchi merged commit 8c96321 into main Oct 10, 2026
8 checks passed
@takecchi
takecchi deleted the chore/storage-pg-comments-3710 branch October 10, 2026 05:23
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