Skip to content

chore: apps/daemon の、試験の名前を言い直すだけの doc と経緯の番号を外す - #4436

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

takecchi merged 1 commit into
mainfrom
chore/comments-daemon

Conversation

@takecchi

Copy link
Copy Markdown
Owner

#3710 の残り。apps/daemon のコメントを整理した。コメントだけの変更で、コードは変えていない。

方針

AGENTS.md「情報の置き場所」の基準に従う。コードのコメントには Why not だけを残す。What は試験の名前で表す。経緯を指すだけの Issue 番号は外し、open の Issue を指す番号は残す。

中身

触っていないもの(コメントではないため)

  • describe / it の名前に入っている番号(issue #1670・#4149 など)。試験の名前は文字列で、コードである
  • app.ts の openapi の説明文に入っている番号(issue #2354・#3956 など)。変えると openapi.json の作り直しが要る

確かめたこと

  • コメントを除いた構文木が main と一致する: 5 files, mismatches 0(TypeScript の printer を removeComments: true で回して比べた)
  • 消した・書き換えた文言を、リポジトリ全体で grep -F した。他所(試験・scripts)が出典として引いているものは無い
  • prettier --check → 通った
  • 開いている他の PR は apps/daemon を触っていない

Refs #3710

Alteroid-Issue-Done: none

AGENTS.md「情報の置き場所」の基準(コードのコメントには Why not だけを残す。What は試験の名前で表す)に合わせる。前回の整理の後に足された、試験ファイルの頭の doc 4つは describe / it の名前の言い直しだった。index.test.ts の理由の一文は、番号を指している試験の名前に置き換えて残した。コードは変えていない。
@takecchi
takecchi merged commit 22629e7 into main Oct 10, 2026
6 checks passed
@takecchi
takecchi deleted the chore/comments-daemon branch October 10, 2026 11:11
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