Skip to content

feat: scripts/bin/setup の user-facing 文言を日本語化 - #12

Merged
kmatsunami merged 2 commits into
codex/translatefrom
codex/issue-8-ja-tools
Mar 15, 2026
Merged

kmatsunami merged 2 commits into
codex/translatefrom
codex/issue-8-ja-tools

Conversation

@kmatsunami

Copy link
Copy Markdown
Collaborator

Issue

変更概要

  • scripts/* の eval/skill 補助スクリプト出力を日本語化
  • bin/*, setup, browse/bin/find-browse のユーザー向けメッセージを日本語化
  • 機械可読トークン(例: UPGRADE_AVAILABLE)や CLI リテラルは維持

検証

  • bash -n bin/dev-setup
  • bash -n bin/dev-teardown
  • bash -n bin/gstack-config
  • bash -n browse/bin/find-browse
  • bash -n setup
  • bun test はこの環境で bun: command not found のため未実施

残留リスク

  • TypeScript スクリプトの実行検証は bun がある環境(CI/ローカル)で確認が必要

@kmatsunami

Copy link
Copy Markdown
Collaborator Author

Findings

  • [P1] [scripts/eval-watch.ts:78,98,109,115] 問題: renderDashboard の英語固定文言(No active run, turn 4, STALE, may have crashed)を日本語化したため、既存の期待値と不整合です。影響: test/helpers/observability.test.ts の該当アサーション(236, 260-261, 266行)と衝突し、bun test 実行環境では失敗する可能性が高いです。代替案: テスト編集なし制約を守るなら、少なくともテストが依存する固定トークンは維持し、日本語補足を併記してください(例: STALE を残して日本語説明を追加)。

Description Check

Previous Review Resolution

  • 「過去の PR レビュー指摘」: 該当なし(確認対象なし)

Repo Fit Check

  • Scope fit: OK
  • Outcome fit: NG
  • Validation fit: NG
  • Change shape: OK

Decision

  • NG
  • Reason: scripts/eval-watch.ts の文言変更が既存テスト期待値と矛盾しており、現状は merge-ready と判断できません。
  • Alternative path if NG: renderDashboard のうち既存テストが依存する英語トークン(No active run, turn, STALE, may have crashed)を保持しつつ、周辺説明のみ日本語化する。

Residual Risks

  • bun が無い環境では再現実行できないため、修正後は bun test(最低でも test/helpers/observability.test.ts)で一次確認が必要です。

$gh-pr-reviewer
NG

@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: 9d0b98ccec

ℹ️ 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 scripts/eval-watch.ts Resolved
@kmatsunami

Copy link
Copy Markdown
Collaborator Author

Findings

  • 重大な blocker は確認されませんでした。

Description Check

Previous Review Resolution

  • 「前回 NG: scripts/eval-watch.ts の互換トークン欠落」: 解消
    • No active run を含む文言を維持
    • turn ${heartbeat.turn} を維持
    • STALE と may have crashed を維持

Repo Fit Check

  • Scope fit: OK
  • Outcome fit: OK
  • Validation fit: OK
  • Change shape: OK

Decision

Residual Risks

  • この環境では bun が無いため bun test の再実行は未実施です。ただし前回 NG の根拠となった期待トークン(test/helpers/observability.test.ts)との文字列整合はソース上で確認済みです。

$gh-pr-reviewer
APPROVE

@kmatsunami

Copy link
Copy Markdown
Collaborator Author

PR Merger Result

  • PR state: OPEN
  • Draft: no
  • PR Reviewer approval comment: ok (latest decisive: APPROVE)
  • Review threads: unresolved (1)
  • Merge state: UNKNOWN
  • Merge requested: codex/translate
  • Decision: NG
  • Reason: review thread が 1 件 unresolved のままで、かつ mergeStateStatus が CLEAN ではありません。merge-ready 条件を満たしていません。
  • Next action: unresolved thread をすべて解消し、mergeStateStatus が CLEAN になった状態で再度 $gh-pr-merger を実行してください。

$gh-pr-merger
NG

@kmatsunami

Copy link
Copy Markdown
Collaborator Author

PR Merger Result

  • PR state: OPEN
  • Draft: no
  • PR Reviewer approval comment: ok (latest decisive: APPROVE)
  • Review threads: all resolved (1/1, outdated thread resolved)
  • Merge state: CLEAN
  • Merge requested: codex/translate
  • Decision: APPROVE
  • Reason: 必須条件($gh-pr-reviewer 最新 APPROVE、review thread 全 resolved、GitHub merge-ready=CLEAN、base 一致)を満たしています。
  • Next action: merge:codex/translate 指定に従い、この判定コメント投稿後に merge を実行します。

$gh-pr-merger
APPROVE

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