Repository navigation
chore: apps/cli(1272e81d 以降に足された分)の無駄なコメントを消す - #4403
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/cliで、2026-10-09 の到達点の報告の時点(1272e81d)より後に足されたコメントを整理した。コメントだけの変更で、コードは変えていない。基準と、その根拠
#4401(core)・#4402(web)と同じ。AGENTS.md の「情報の置き場所」に従い、コードのコメントには Why not(単純にすると壊れる理由)だけを残す。How はコードで、What は試験の名前で、Why はコミットログで表す。Issue 番号・経緯は消す。
オーナーの承認(クローンの手元の記録から):
f6e6bad8「この基準で全体に広げてよい」範囲
git diff 1272e81d origin/main -- apps/cliで足された、行頭がコメントの行(32行)apps/cliに触れていない。apps/cli/src/chat.tsを触っていた fix: pids が尽きて git を起こせなかった回の未 push 観測が、前に取れていた枝名を台帳から消さないようにする #4398 は 2026-10-10T02:42Z にマージ済みで、この枝の土台に入っている)消したもの・残したもの
#4356・#4353・#4342・#3943)command-args.tsの各関数の上の、名前を言い直していた1行command-args.tsの冒頭(規則を1か所に置く理由、余分な語を断る理由)。normalizeCommandWord・parseCountArgの上は、「何をするか」から「なぜそうするか」の形に直したinput.tsの Ink の読み方の理由、test-helpers.tsの書き終わりの callback の理由、confirm.test.tsの偽の端末の理由確かめたこと
removeComments: trueで回し、origin/mainの版と比べた)npx prettier --check(変えた8ファイル)通過readFileSync/?rawで読む箇所は無かった確かめていないこと
Alteroid-Issue-Done: none