Skip to content

fix/cmd050-infra-improvements: 8 commits accumulated improvements#137

Open
system-orepa wants to merge 8 commits into
yohey-w:mainfrom
system-orepa:fix/cmd050-infra-improvements
Open

fix/cmd050-infra-improvements: 8 commits accumulated improvements#137
system-orepa wants to merge 8 commits into
yohey-w:mainfrom
system-orepa:fix/cmd050-infra-improvements

Conversation

@system-orepa
Copy link
Copy Markdown

@system-orepa system-orepa commented Apr 26, 2026

Summary

system-orepa fork の `fix/cmd050-infra-improvements` ブランチに蓄積された8コミットを upstream へ提案。

含まれる主な改善

  • android (4b1a244): SSH key loading 簡素化 + auth logic 修正
  • instructions (b8aa46b): karo cmd 完了フローから daily log append step 削除
  • scripts (2bf87b0): stop_hook_inbox 簡素化、OSS skill install step 削除、setup scripts 整理
  • docs (9b52f88): DesignerRepo hub-only boundary を Codex 向けに追加 — v3.1
  • docs (6455607): README からスポンサーセクション・廃止済 daily_log/bloom_routing 参照を削除
  • infra (de9baa1): cmd_format に estimated_time 追加、karo decomposition ガイダンス拡充
  • chore (14dc340): FUNDING.yml / CHANGELOG.md 削除 (本リポジトリではメンテせず)
  • chore (761a87b): shutsujin_departure.sh GPT-5.4→GPT-5.5 タイポ修正

Test plan

  • `./shutsujin_departure.sh --help` でモデル表記が GPT-5.5 になっていること
  • `./shutsujin_departure.sh` で平時の陣を起動し、足軽1-4 が gpt-5.5 で稼働すること
  • `scripts/stop_hook_inbox.sh` の動作確認
  • `instructions/karo.md` 完了フローの動作確認

🤖 Generated with Claude Code

system-orepa and others added 8 commits April 17, 2026 17:17
@system-orepa @claude
feat(infra): extend cmd_format with estimated_time + expand karo deco…
de9baa1 
…mposition guidance

## CLAUDE.md
- cmd_format に estimated_time フィールド追加（将軍のbest-effort見積もり）

## instructions/karo.md
- 家老の分解・割当ガイダンスを大幅拡張（87行追加）

## shutsujin_departure.sh
- 7足軽+軍師編成の明示化（3x3グリッド維持のため ashigaru8 を軍師へ置換）
- 平時の陣の編成を更新: 足軽1-4=Codex GPT-5.4, 足軽5-7=Claude Opus
- CLI Adapter 読込順序を整理
- ヘルプ表示テキストを新編成に合わせ更新

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@system-orepa
chore: remove FUNDING.yml and CHANGELOG.md — no longer maintained in …
14dc340 
…this repo
@system-orepa
feat(android): simplify SSH key loading and fix auth logic
4b1a244 
Use file path directly instead of loading key bytes; fix auth method
selection to check lastKeyPath instead of identity object.
@system-orepa
docs: remove sponsors section and obsolete daily_log/bloom_routing re…
6455607 
…ferences

Clean up AGENTS.md (remove daily_log path and bloom_routing_rule fields),
CLAUDE.md (trim post-compaction note), and READMEs (remove Sponsors section).
@system-orepa
feat(instructions): remove daily log append step from karo cmd comple…
b8aa46b 
…tion flow

Step 6 (daily log append to logs/daily/YYYY-MM-DD.md) removed from all
karo instruction variants; step numbering updated accordingly.
@system-orepa
chore(scripts): simplify stop_hook_inbox, remove OSS skill install st…
2bf87b0 
…ep, trim setup scripts

Fix shell quoting issue in stop_hook_inbox.sh by using env-var injection
approach; remove STEP 6.5 OSS skill install from first_setup.sh; simplify
ratelimit_check.sh, install.bat, and shutsujin_departure.sh.
@system-orepa @claude
docs(agents): add DesignerRepo hub-only boundary for Codex — v3.1
9b52f88 
Adds UNCONDITIONAL rule that when Lord routes Codex through the
DesignerRepo / workday / Shogun handoff flow, Codex is hub-only from
that point: must write instructions into handoff artifacts (to_shogun,
from_shogun, codex_review, spec, closeout) rather than editing
implementation code directly. Prevents workflow drift where "small
fixes" bypass the Shogun/Karo chain.

Bumps version 3.0 → 3.1, updated 2026-04-19.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@system-orepa @claude
chore(scripts): fix GPT-5.4→GPT-5.5 typo in shutsujin_departure help/…
761a87b 
…banner

ヘルプ表示・モデル一覧・MODEL_NAMES 配列の足軽1-4 表記を
"GPT-5.4" → "GPT-5.5" に統一。実体は既に gpt-5.5 で稼働しており
表記のみの不整合を解消。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@yohey-w
Copy link
Copy Markdown
Owner

yohey-w commented May 5, 2026

Thank you for this comprehensive set of improvements @system-orepa!

This PR contains significant infrastructure changes (8 commits, -662 lines net reduction) including android SSH key fixes, script simplification, and docs updates. We're reviewing it carefully as it touches core infrastructure components.

This will require sign-off from the project maintainer before merging. We'll get back to you soon with a decision.

@yohey-w
Copy link
Copy Markdown
Owner

yohey-w commented May 17, 2026

@system-orepa 12日も返信留め置き、誠に申し訳なく存ずる。家老が8コミット全件を精査し、将軍の裁可を得た上で対応方針を以下に申し上げる。

全体方針

本PRは8コミットを束ねた構成ゆえ、コミット単位で個別判断つかまつった。全件精査の結果、1件即マージ確定、1件単独PR歓迎、6件却下となった。投稿者殿のPR分割の手間を増やさぬよう、単独PR依頼は1件のみに絞ってござる。

✅ 即マージ確定 (1件)

761a87b — shutsujin_departure.sh GPT-5.4→GPT-5.5 typo 修正

自明かつ無害。当方にて取り込ませていただく。御指摘かたじけのうござる。

🟡 単独PR歓迎 (1件・条件付き)

4b1a244 — android SSH key loading 簡素化 + auth logic 修正

ファイルパス直接使用への変更、lastKeyPath チェックへの修正、いずれも設計簡素化として妥当と判断つかまつった。

ただしセキュリティ箇所ゆえ、単独PRとしてご提出いただきたく:

  • PR description に android 実機での認証成功確認 の記載をお願いいたしたい
  • 既存テスト全 pass の確認も合わせて

🔴 却下 (6件・理由明示)

2bf87b0 — scripts 簡素化 (stop_hook_inbox 他)

直近の upstream commit c7fe93c (fix: use absolute path for stop_hook_inbox.sh in settings.json) と衝突しており、当方では別アプローチで本領域の修正を完了済にござる。また STEP 6.5 OSS skill install 削除は貴殿の fork 固有の運用判断と拝察、upstream では当該 step を維持する方針にござる。

9b52f88 — AGENTS.md DesignerRepo hub-only boundary 追加

当方の運用に DesignerRepo という概念が存在せず、本ルールは貴殿の fork 固有のワークフロー設計と推察いたす。upstream では Shogun→Karo→Ashigaru chain を直接運用しており、DesignerRepo 経由のハブ化は採用しておらぬゆえ却下とさせていただきたい。

de9baa1 — cmd_format + karo guidance + shutsujin 編成更新

本日 (2026-05-17)、当方は全足軽を Codex GPT-5.5 xhigh に統一する陣形変更を実施したばかりにござる。本コミットの shutsujin_departure.sh 更新は「足軽1-4 = Codex GPT-5.4, 足軽5-7 = Claude Opus」を明示化する内容ゆえ、最新方針と真逆となり却下せざるを得ず。

なお cmd_format への estimated_time 追加自体は別途 #153 (タスクPDCA基盤) の議論と整合する良案にござる。本コミットからの抽出はせず、PDCA基盤の cmd 発令時に当方で同等の機能を組み込む所存。

b8aa46b — karo cmd 完了フローから daily log append step 削除

#153 (タスクPDCA基盤) にて、完了メタデータ記録の拡充 (completed_at / redo_count / quality_score) を採用方針として確定済にござる。本コミットは記録を減らす方向ゆえ、当方の戦略と矛盾し却下とさせていただきたい。

14dc340 — FUNDING.yml + CHANGELOG.md 削除

upstream では GitHub Sponsors を運用方針として維持しており、FUNDING.yml の存続が必須にござる。CHANGELOG.md も release 管理上必要ゆえ、両ファイル削除は受け入れかねる。

6455607 — README sponsors セクション削除

上記 14dc340 と同じ理由にて却下。sponsors セクションは upstream の戦略上、README に維持する方針にござる。

本 PR の扱い

本 PR は 6件却下、1件即マージ、1件単独PR依頼の構成ゆえ、貴殿のご判断にて close いただくのが筋と存ずる。当方にて 761a87b は cherry-pick して main に取り込み、4b1a244 のみ単独 PR としてのご再提出を歓迎申し上げる。

御礼

8コミットの蓄積を upstream に還元しようとする貢献の意志、誠にかたじけのうござる。fork の運用知見が upstream と異なる箇所が多くあり今回の精査結果と相成ったが、貴殿の貢献意欲そのものは大いに有難く受け止めてござる。

引き続きのご厚誼を願い申し上げる。

@yohey-w
Copy link
Copy Markdown
Owner

yohey-w commented May 17, 2026

@system-orepa 失礼仕った、家老から重要な訂正がござる。

先程の返信にて「761a87b は即マージ確定、当方にて取り込ませていただく」と申し上げたが、cherry-pick を試みた結果衝突して取り込めず候。

調査の結果、upstream の shutsujin_departure.sh は CLI Adapter ベースに refactor 済で、GPT-5.4/GPT-5.5 のハードコード自体が現状コードに存在せぬ状態にござった。よって 761a87b の typo 修正は 修正対象が消滅しており取り込み不要 (moot) となり申す。先の精査で本件を見落とした家老の不手際、誠に申し訳なく存ずる。

訂正後の対応方針:

  • 761a87b → 取り込み不要（修正対象消滅のため）
  • 4b1a244 (android SSH) → cherry-pick 試験で衝突なく適用可を確認済。単独PR でのご提出を引き続き歓迎申し上げる
  • その他6件 → 先の理由にて却下のまま

本PRは結果として 4b1a244 の単独再提出のみ歓迎、本PR自体は close 推奨 の構成になり申す。投稿者殿の貢献意欲を無駄足にさせてしまう判断と相成り、二重に申し訳なく存ずる。

引き続きのご厚誼を願い申し上げる。

yohey-w
yohey-w requested changes Jun 6, 2026
View reviewed changes
Copy link
Copy Markdown
Owner

@yohey-w yohey-w left a comment

Choose a reason for hiding this comment

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

Thank you for the accumulated improvements, @system-orepa — the Android SSH fix and ratelimit_check.sh changes look useful. However, this PR as-is can't be merged upstream for the following reasons:

Must fix before merge:

  1. CHANGELOG.md deleted — This removes the project's version history for all users. If you prefer not to maintain a changelog in your fork, that's fine locally, but upstream needs it. Please restore it.

  2. .github/FUNDING.yml deleted — This removes the GitHub Sponsors link. Please restore it.

  3. DesignerRepo Hub Boundary section in AGENTS.md — The added section describes a workflow specific to your fork (DesignerRepo, workday files, roundfiles, hub-only Codex mode). This concept doesn't exist in upstream and would confuse other users. Please remove it from the upstream PR — keep it in your fork's AGENTS.md if you need it there.

  4. daily_log removal from AGENTS.md / instructions/karo.md — Removing the karo daily log step is a breaking change in the upstream workflow. If you want to turn this off in your setup, use your fork's config — don't remove it from upstream instructions.

What would be mergeable as separate PRs:

  • Android SSH simplification (SshManager changes)
  • ratelimit_check.sh improvements
  • stop_hook_inbox.sh fix (already landed via #161)
  • AGENTS.md version bump to 3.1 (without the DesignerRepo section)

Would you be willing to split this into focused PRs? Happy to review each one.

@yohey-w
Copy link
Copy Markdown
Owner

yohey-w commented Jun 6, 2026

@system-orepa ありがとうございます。変更内容を精査しました。

このPRはupstreamに有益な修正と、フォーク固有の設定変更が混在しているため、そのままではmergeが難しい状況です。

upstreamに取り込みたい部分は以下の2点です。これらだけを切り出して別PRとして出し直していただけますか？

  1. android/SshManager.kt — loadPrivateKeyIdentity() を廃止して jsch.addIdentity(keyPath) に統一する SSH キー読み込み簡素化
  2. scripts/ratelimit_check.sh — capture_tmux_pane_zoomed() 関数の削除（デッドコード掃除）

このPRに含まれる以下の変更はupstreamには取り込めません：

  • CHANGELOG.md の削除 → 全ユーザーの更新履歴が消えてしまいます
  • .github/FUNDING.yml の削除 → GitHub Sponsors リンクが消えます
  • AGENTS.md への DesignerRepo Hub Boundary セクション追加 → フォーク固有のワークフロー概念です。upstream ユーザーには文脈がありません
  • AGENTS.md / instructions/karo.md からの bloom_routing_rule / daily_log 削除 → upstream のワークフローを破壊する変更です
  • shutsujin_departure.sh の ANSI エスケープコード変更 → \033 が消えているため色が効かなくなるregressionです（`echo -e "[1;33m..."" では ESC コードが出力されません）
  • scripts/stop_hook_inbox.sh の Python ブロック生成コード削除 → 何で置き換えたか不明なまま削除されており、不完全です

上記2点の修正を別々のPR（またはまとめた1本のPR）として出し直していただければ、喜んでレビュー・mergeいたします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yohey-w yohey-w yohey-w requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@system-orepa @yohey-w