Skip to content

fix: CLI・Runner境界とテスト基盤を包括修正 - #1

Merged
inakaegg merged 20 commits into
mainfrom
fix/comprehensive-bug-sweep
Jul 25, 2026
Merged

fix: CLI・Runner境界とテスト基盤を包括修正#1
inakaegg merged 20 commits into
mainfrom
fix/comprehensive-bug-sweep

Conversation

@inakaegg

@inakaegg inakaegg commented Jul 25, 2026

Copy link
Copy Markdown
Owner

概要

コードベース全体の監査で再現できた CLI/Runner/ドキュメント解析/実行状態の不具合を修正し、回帰テストと CI のカバレッジゲートを追加しました。

主な修正

  • --check、定数探索、再ロード時の実行分岐・alias・Ruby字句lookup・完全修飾guard・loop guard・const_set捕捉など Runner 境界を修正
  • 位置引数・キーワード引数・eval式・型注釈・負数/桁区切り/負の基数値・lone dashオプション値の変換を修正
  • eval 構文エラーを利用者向けエラーへ正規化し、1実行内だけbindingを共有
  • 静的解析によるautoload副作用と、ロード後に消えたtargetの再読込を防止
  • 実行ごとの環境状態と結果出力を隔離
  • 未ロードの新規 lib/ ファイルとGit quoteされた非ASCII pathも changed-line coverage で未カバーとして検出
  • 通常pushは直前commit、新規・削除refはdefault branchをcoverage基準に選択
  • Ruby バージョン別テストと line/branch/changed-line coverage の GitHub Actions を追加

ローカル検証

最新 HEAD 6941eab:

  • Ruby 3.2: 210 runs / 834 assertions / 0 failures / 0 errors
  • Ruby 3.4: 210 runs / 834 assertions / 0 failures / 0 errors
  • Ruby 2.7参考互換経路: 定数捕捉33 runs / 117 assertions / 1 skip(gemspecはRuby 3.0以上)、eval/負数4件、coverage gate 5件が成功
  • line coverage: 92.26% (2587/2804)
  • branch coverage: 73.12% (1126/1540)
  • changed-line coverage against origin/main: 95.09% (542/570)
  • 今回commitのchanged-line coverage: 100.00% (3/3)
  • workflow YAML / Ruby syntax / git diff --check: 成功
  • gem build: 成功(packaging差分なし)

GitHub Actions

最新HEAD 6941eab でpush / pull_requestの全10チェックが成功しました。

  • Ruby 3.0: 成功
  • Ruby 3.2: 成功
  • Ruby 3.4: 成功
  • Ruby 4.0: 成功
  • Coverage: 成功

レビュー

  • 独立コンテキストレビュー2回: 確認済みバグ計5件を再現・修正
  • GitHub Codex review(9eeb997): 13 threadsを8原因へtriageし、全件へ返信・解決
  • GitHub Codex re-review(f9f322f): 確認済みバグ3件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(20729ea): 確認済みバグ3件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(bee411e): 確認済みバグ3件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(e8be1b6): 確認済みバグ4件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(a6ed557): 確認済みバグ2件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(b513698): 確認済みバグ3件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(a668fd1): 確認済みバグ3件を再現・修正し、全threadへ返信・解決
  • GitHub Codex re-review(cb9c8e0): 確認済みバグ2件を再現・修正し、全threadへ返信・解決
  • 最新 HEADへのGitHub Codex再レビュー: 依頼予定

未確認範囲・既知の改善余地

  • 実際のbranch削除eventは発生させず、workflow event条件とfallback先を静的検証
  • blockingではない既知の改善余地なし

inakaegg and others added 8 commits July 25, 2026 09:56
…avior

- 見出し階層を再編(導入→インストール→クイックスタート→リファレンス)
- ヘルプ・エラー出力例を現行実装の実出力へ更新(<NAME>表記や旧エラー文言を修正)
- hello_app_with_docs.rb は定数名不一致のため -a 付きの実行例へ修正
- --new は「ファイルパスより前」に置く動作確認済みの形へ修正
- 現行実装で動作しないサンプル(--json-args/--eval-args と --new の併用、
  --pre-script 単独でのインスタンス専用クラス起動)を削除し、検証済みの形へ差し替え
- changelog口調(now infers等)を除去、公開済みgemと矛盾する末尾文言を修正
- Gem Versionバッジ・Ruby 3.0+要件・Licenseセクションを追加
- 日本語版のみにあった YARD 併用時の注意を英語版へも反映し両言語を同期

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添えてください。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告不要です。
重大度順に最大5件。確信の持てない指摘は報告しないでください。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD a200ce3 を対象に再レビューしてください。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。各指摘に失敗する具体的な入力・状態・コード経路を添え、スタイル・命名・仮説的懸念は除外してください。該当なしなら LGTM としてください。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD dc43262 を対象に再レビューしてください。直前のCI互換性修正も含みます。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告し、該当なしなら LGTM としてください。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD 9eeb997 を対象に再レビューしてください。Ruby 3.0 CI の Minitest 互換性固定を含みます。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告し、該当なしなら LGTM としてください。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9eeb997767

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli.rb Outdated
Comment thread lib/rubycli/argument_parser.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread lib/rubycli/help_renderer.rb
Comment thread lib/rubycli/argument_parser.rb Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9eeb997767

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/argument_parser.rb Outdated
Comment thread lib/rubycli/help_renderer.rb
Comment thread lib/rubycli.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/eval_coercer.rb Outdated
Comment thread lib/rubycli.rb Outdated
Comment thread .github/workflows/test.yml Outdated
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD f9f322f を対象に再レビューしてください。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添えてください。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告不要です。
重大度順に最大5件。確信の持てない指摘は報告しないでください。該当なしならLGTMとしてください。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f9f322f58a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/argument_parser.rb Outdated
Comment thread lib/rubycli/constant_capture.rb
Comment thread lib/rubycli/constant_capture.rb Outdated
@inakaegg

inakaegg commented Jul 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD 20729ea を対象に再レビューしてください。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添えてください。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告不要です。
重大度順に最大5件。確信の持てない指摘は報告しないでください。該当なしならLGTMとしてください。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 20729eaffd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/argument_parser.rb Outdated
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD bee411e を対象に再レビューしてください。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添えてください。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告不要です。
重大度順に最大5件。確信の持てない指摘は報告しないでください。該当なしならLGTMとしてください。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bee411e32a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/constant_capture.rb
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/argument_parser.rb Outdated
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD e8be1b6 を対象に再レビューしてください。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添えてください。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告不要です。
重大度順に最大5件。確信の持てない指摘は報告しないでください。該当なしならLGTMとしてください。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD e8be1b6 の再レビューをお願いします。確認済みの実バグ・セキュリティ問題・データ破壊のみ、具体的な再現経路付きで最大5件。該当なしならLGTMとしてください。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新 HEAD e8be1b6 を対象に再レビューしてください。
確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告してください。
各指摘に失敗する具体的な入力・状態・コード経路を添え、スタイル・命名・仮説的懸念は除外してください。重大度順に最大5件。該当なしならLGTMとしてください。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e8be1b6feb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新HEAD a6ed557 を対象に再レビューしてください。

確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告する。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添える。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告しない。
重大度順に最大5件。確信の持てない指摘は報告しない。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a6ed557ac9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/constant_capture.rb
Comment thread lib/rubycli/argument_parser.rb Outdated
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新HEAD b513698 を対象に再レビューしてください。

確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告する。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添える。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告しない。
重大度順に最大5件。確信の持てない指摘は報告しない。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b51369827c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/argument_parser.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
Comment thread test/support/coverage_gate.rb
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新HEAD a668fd1 を対象に再レビューしてください。

確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告する。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添える。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告しない。
重大度順に最大5件。確信の持てない指摘は報告しない。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a668fd1b36

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/constant_capture.rb
Comment thread lib/rubycli/argument_parser.rb Outdated
Comment thread lib/rubycli/constant_capture.rb Outdated
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新HEAD cb9c8e0 を対象に再レビューしてください。

確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告する。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添える。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告しない。
重大度順に最大5件。確信の持てない指摘は報告しない。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cb9c8e0648

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/rubycli/constant_capture.rb
Comment thread lib/rubycli/constant_capture.rb
@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

最新HEAD 6941eab を対象に再レビューしてください。

確認済みの実バグ、セキュリティ問題、データ破壊・欠損のみ報告する。
各指摘に、失敗する具体的な入力・状態・コード経路を必ず添える。
スタイル、命名、リファクタ提案、仮説的な懸念、軽微な性能改善は報告しない。
重大度順に最大5件。確信の持てない指摘は報告しない。

@inakaegg

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 6941eabde0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@inakaegg
inakaegg merged commit 793e530 into main Jul 25, 2026
10 checks passed
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