Repository navigation
Terra親・固定effortと6 Pro優先の工程ルーティングへ整理 - #11
Merged
Merged
Conversation
gui-ace
marked this pull request as ready for review
September 15, 2026 07:32
10 tasks
準備済み依頼のUTF-8保存サイズを出力前に検査し、後続処理で読み込めないbundleを生成しない。 初回の接続操作にpermission_checkedを設け、実行済みとは区別したまま新しい承認・モデル観測で候補判定する。 回帰テスト8件を追加し、配布版を0.8.1へ更新。実際のCodex/Chat動作確認は別途実施する。
gui-ace
commented
Sep 15, 2026
gui-ace
left a comment
Contributor
Author
There was a problem hiding this comment.
レビュー結果
方針正本・分類表・引継ぎ・フック・分類CLI・旧bundle互換処理と追加テストを確認しました。見つかった次の2点を修正し、回帰テストを8件追加しています。修正コミットは 038631cee9355968b06cd10776c7727524ad5d88、配布版は0.8.1です。
修正済み
- 準備成功したbundleを後続処理で読めない(サイズ境界):引用符等を含む入力は、promptをJSON文字列へ格納すると展開後のサイズが256KiBを超える場合がありました。実際のCLIと同じUTF-8シリアライズ結果をprepare終了前に検査し、超過時は部分出力せず入力を短縮するか対象を限定した参照にするよう案内します。境界内の最大サイズで、CLI→ファイル読込み→転送→回答保存の往復も確認しました。
- 初回の書込み操作に実行成功済みの能力確認を要求する循環:
permission_checkedを追加し、提供操作と対象範囲の権限を確認済みだが実行はまだ、という状態を実取得成功のverifiedと分けました。根拠を必須にし、直近のlive・操作承認・転送方法の許可・モデル・利用制限・surface・操作とscopeの完全一致を維持しています。試験書込みは要求せず、execution_verified=falseとして返します。単なるadvertisedは引き続き候補になりません。
検証
- Linux/Python 3.13.5の124件のテストスイート成功(122成功、環境依存2件skip)。package・release・diff検査成功。
- ローカルで検証した修正後ツリー
e98273c6ffd8de2ad6473480fe97c2cb9845ef34と公開ツリーを照合。修正適用時のPython 3.11検証も成功しました。 - 最終PR HEADのWindows/Linux/macOS CIはこのコメント時点では実行中です。すべての結果を確認してからマージします。
- レビュー用一時ファイル・専用workflow・追加write権限はPRへ含めていません。
確認範囲で、上記修正後に残るマージ阻害のコード問題は見つかっていません。独立した別担当の承認レビューではなく、この会話での再レビュー記録です。
実モデル起動、通常Chatの実画面・実残枠、Windows上のプラグイン導入・継続・再開、節約率は未検証です。利用者の指定どおり、実動作確認と軽微な現場修正は後でCodexに引き継ぎます。既存のopt-in・親設定・認証・フック信頼は変更していません。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #10
目的
PR #9マージ後のmain(59879ad02995c16e398de7bd7fd0e2f2d58ad9b7)を基点に、親Terra/xhighが通常作業と進行を担い、必要な工程だけLuna/max・Sol/high・Astra/high・通常Chatの6 Proへ渡す方式へ整理しました。レビュー修正を含む配布版は 0.8.1 です。
主な変更
追加レビューで修正した問題
1. 準備成功したbundleが大きすぎて後続処理で読めない
引用符などを含む入力では、JSONをpromptに埋め込んだ後の保存サイズが256KiBを超えることを再現しました。prepareで実際のCLIと同じUTF-8シリアライズ結果を検査し、超過時は部分出力せず明確なエラーを返します。入力の短縮や対象を絞った出典参照を案内し、黙った切り捨ては行いません。境界内の最大サイズで準備・再読込み・転送・回答保存の往復を検証しました。
2. 初回の書込み操作のために「実行成功済み」を要求する循環
接続台帳に permission_checked を追加しました。操作の提供と対象範囲の権限を確認した根拠を必須とし、試験書込みを要求せず、初回の正規操作を候補にできます。実取得・実行成功のverifiedとは分離し、execution_verified=falseで返します。新しいlive・実操作の承認・転送方法の許可・モデル・利用制限・surface・操作とscopeの一致を引き続き要求します。単なるadvertisedや期限切れ、読み取りの実績から書込みを許可しません。
検証結果
実環境への引継ぎ・限界
利用者の指定により、実動作確認は後でCodexで行います。ここでは実モデル起動、Windows上の実プラグイン導入・継続・再開、通常Chatの実画面・実残枠・送受信、実作業の節約率は未検証です。
liveとpermission_checkedは呼出し側が観測・確認した根拠の照合で、アカウント状態、実行モデル、正確な残枠、法的許可を自動で証明する機能ではありません。通常Chatの自動回収の許可が確認済みとは扱わず、許可不明時は停止する方針を維持します。実アカウントの設定、親のmodel/effort、認証、フック信頼、Chat経路の有効化は変更していません。
Codexでは、固定effortの実起動・既存overrideの維持・短い作業での不要な委譲抑制・6 Pro適合工程の選定・初回操作と実行済みの区別・モデル切替/上限/期限切れ時の停止を実環境で確認してください。