Repository navigation
chore: apps/daemon(1272e81d 以降に足された分)の無駄なコメントを消す - #4404
Merged
Merged
Conversation
This was referenced Oct 10, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #3710
apps/daemonで、2026-10-09 の到達点の報告の時点(1272e81d)より後に足されたコメントを整理した。コメントだけの変更で、コードは変えていない。基準と、その根拠
#4401(core)・#4402(web)・#4403(cli)と同じ。AGENTS.md の「情報の置き場所」に従い、コードのコメントには Why not(単純にすると壊れる理由)だけを残す。How はコードで、What は試験の名前で、Why はコミットログで表す。Issue 番号・経緯は消す。
オーナーの承認(クローンの手元の記録から):
f6e6bad8「この基準で全体に広げてよい」範囲
git diff 1272e81d origin/main -- apps/daemonで足された、行頭がコメントの行(21行)apps/daemon/openapi.jsonを触るが、この PR はopenapi.tsの description を変えていないので、生成物は変わらない消したもの・残したもの
#4379・#4355)index.tsの「デーモンの待ち受けを開くのは認証が立つときだけ、という docs の記述を、デーモンは強制せず警告だけ出す(強制は railway/setup.sh の分岐のみ)— オーナー判断 #2892 のオーナー判断 2026-10-09」)。判断の中身(認証が無くても起動を止めない理由)は残したjournal-bus.tsのpruneQueuedForDeletionの doc のうち、何を捨てるかの説明pruneQueuedForDeletionを受け口が呼ぶ理由conversation-delete.test.tsの、decision の文を道具と同じ形にする理由。外す側は、台帳の行の id をその形から拾うapp.tsの重複キーの直列化・倒し方の理由、openapi.tsのduplicateの欄の説明(いずれも理由か、応答の意味)確かめたこと
removeComments: trueで回し、origin/mainの版と比べた)npx prettier --check(変えた4ファイル)通過readFileSync/?rawで読む箇所は無かった確かめていないこと
Alteroid-Issue-Done: none