Skip to content

インフラまわりのコメントを削って行数を減らす - #447

Merged
5ym merged 1 commit into
mainfrom
trim-infra
Oct 18, 2026
Merged

5ym merged 1 commit into
mainfrom
trim-infra

Conversation

@5ym

@5ym 5ym commented Oct 18, 2026

Copy link
Copy Markdown
Collaborator

概要

.github/・bootstrap/・talos/・pulumi/・tools/・renovate.json・.sops.yaml・compose.yaml・deploy/argocd.yaml のコメントを削った。差し引き 736 行減(57 ファイル、+333 / -1069)。

削ったもの:

  • 経緯の語り(日付、「以前は…」、k3s の頃・bootstrap-apply.yml・metrics-server・Mattermost などもう無いものへの言及)
  • コードを言い直しただけの説明
  • 同じ理由の繰り返し(render.sh・talos-boot.yml・values.yaml・version.yaml の重複、pulumi.yml の secrets 一覧は pulumi/README.md に寄せた、HTTPRoute 5 本の同じヘッダ、InfisicalSecret の同じヘッダ)

残したもの(1 行に縮めた):

  • anonymous を消さない、KubePrism 7445、--api-versions と GatewayClass、CEL の claim 規則で絞っている理由
  • nodePort の範囲を広げない、claude_args: | の中に注釈を書かない、[ -n … ] && で trap を終えない、sops -d | inline と繋がない
  • redis の tag を chart を上げるときに見比べる、protect を外さない、Forgejo の owner を改名したら直す場所

動きが変わらないこと

  • コメント以外の変更は無い。変更したファイルごとに、コメント行(# / //)を除いた中身が HEAD と同じであることを diff で確かめた。renovate.json は description の文字列だけで、JSON として読めることも確かめた
  • talos/render.sh はコメントだけなので出力は変わらない。sh -n / bash -n を通した(tools/creds.sh・pulumi/*.sh も同じ)。tools/t.ps1 は PowerShell のパーサで構文エラー 0、BOM はそのまま
  • machine config に埋め込まれるファイル(bootstrap/*/namespace.yaml・bootstrap/apiserver/rbac.yaml・talos/manifests/local-path.yaml)と、SOPS のファイル・talos/secrets.yaml・talos/registries.yaml・recovery/ には触れていない
  • talos/patches/*.yaml と bootstrap/*/values.yaml はコメントだけ(talosctl と helm が捨てる)。ブロックスカラーの中のコメント(oidc.config など)には触れていない
  • bootstrap/*/version.yaml は先頭の説明行を消し、# renovate: の行と version: だけにした(Renovate の正規表現と render.sh の sed が読むのはこの 2 行)

🤖 Generated with Claude Code

https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV

workflow・bootstrap・talos・pulumi・tools・renovate.json のコメントから、
経緯の語り(日付・以前の構成・撤去したもの)、コードを言い直しただけの説明、
複数のファイルで繰り返していた理由を削った。罠の警告(anonymous、
KubePrism 7445、--api-versions、CEL の claim 規則、nodePort の範囲など)は
1 行ずつ残した。

コメント以外の変更は無い。machine config に埋め込まれるファイル
(namespace.yaml・rbac.yaml・local-path.yaml)と SOPS のファイルには触れていない。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV
@github-actions

Copy link
Copy Markdown

🍹 preview on doany-cloudflare/prod

Pulumi report
Previewing update (prod):

@ previewing update.....
pulumi:pulumi:Stack doany-cloudflare-prod  
Resources:
54 unchanged

@5ym
5ym merged commit b856918 into main Oct 18, 2026
7 checks passed
@5ym
5ym deleted the trim-infra branch October 18, 2026 06:10
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