/** * WPML compatibility functions * * @global array $duplicated_posts Array to store the posts being duplicated. * * @package Yoast\WP\Duplicate_Post * @since 3.2 */ add_action( 'admin_init', 'duplicate_post_wpml_init' ); /** * Add handlers for WPML compatibility. */ function duplicate_post_wpml_init() { if ( defined( 'ICL_SITEPRESS_VERSION' ) ) { add_action( 'dp_duplicate_page', 'duplicate_post_wpml_copy_translations', 10, 3 ); add_action( 'dp_duplicate_post', 'duplicate_post_wpml_copy_translations', 10, 3 ); add_action( 'shutdown', 'duplicate_wpml_string_packages', 11 ); } } global $duplicated_posts; // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals -- Reason: Renaming a global variable is a BC break. $duplicated_posts = []; /** * Copy post translations. * * @global SitePress $sitepress Instance of the Main WPML class. * @global array $duplicated_posts Array of duplicated posts. * * @param int $post_id ID of the copy. * @param WP_Post $post Original post object. * @param string $status Status of the new post. */ function duplicate_post_wpml_copy_translations( $post_id, $post, $status = '' ) { global $sitepress; global $duplicated_posts; remove_action( 'dp_duplicate_page', 'duplicate_post_wpml_copy_translations', 10 ); remove_action( 'dp_duplicate_post', 'duplicate_post_wpml_copy_translations', 10 ); $current_language = $sitepress->get_current_language(); $trid = $sitepress->get_element_trid( $post->ID ); if ( ! empty( $trid ) ) { $translations = $sitepress->get_element_translations( $trid ); $new_trid = $sitepress->get_element_trid( $post_id ); foreach ( $translations as $code => $details ) { if ( $code !== $current_language ) { if ( $details->element_id ) { $translation = get_post( $details->element_id ); if ( ! $translation ) { continue; } $new_post_id = duplicate_post_create_duplicate( $translation, $status ); if ( ! is_wp_error( $new_post_id ) ) { $sitepress->set_element_language_details( $new_post_id, 'post_' . $translation->post_type, $new_trid, $code, $current_language ); } } } } // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals -- Reason: see above. $duplicated_posts[ $post->ID ] = $post_id; } } /** * Duplicate string packages. * * @global array() $duplicated_posts Array of duplicated posts. */ function duplicate_wpml_string_packages() { // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals -- Reason: renaming the function would be a BC-break. global $duplicated_posts; foreach ( $duplicated_posts as $original_post_id => $duplicate_post_id ) { // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals -- Reason: using WPML native filter. $original_string_packages = apply_filters( 'wpml_st_get_post_string_packages', false, $original_post_id ); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals -- Reason: using WPML native filter. $new_string_packages = apply_filters( 'wpml_st_get_post_string_packages', false, $duplicate_post_id ); if ( is_array( $original_string_packages ) ) { foreach ( $original_string_packages as $original_string_package ) { $translated_original_strings = $original_string_package->get_translated_strings( [] ); foreach ( $new_string_packages as $new_string_package ) { $cache = new WPML_WP_Cache( 'WPML_Package' ); $cache->flush_group_cache(); $new_strings = $new_string_package->get_package_strings(); foreach ( $new_strings as $new_string ) { if ( isset( $translated_original_strings[ $new_string->name ] ) ) { foreach ( $translated_original_strings[ $new_string->name ] as $language => $translated_string ) { do_action( // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals -- Reason: using WPML native filter. 'wpml_add_string_translation', $new_string->id, $language, $translated_string['value'], $translated_string['status'] ); } } } } } } } } おすすめ：7-C を 6つの小フェーズに分割（各フェーズで必ず“動く状態”を残す） – Raqqa

Raqqa

おすすめ：7-C を 6つの小フェーズに分割（各フェーズで必ず“動く状態”を残す）

—

by

rakkaUser
in

Phase C0: 入口の安全装置（最初に固定）

ゴール

  • /analytics/query が 必ず status + reason_code + diagnostics + errors[] (+ join_plan) を返す
  • まだ中身が無くても、例外でも HTTP 200 で契約通り返る

この段階の方針

  • retrieval/LLM/execute は最初はスタブでOK（ただし 契約だけ 完全にする）
  • ここが固まると、nlq-dev 側が reason_code で分岐を先に作れる（手戻り激減）

Phase C1: ir_*_src 取り込み（検証ソースの唯一性を確保）

ゴール

  • devportal DB に ir_model_src / ir_field_src / ir_view_src が存在し、最低限の必要カラムが揃う
  • “検証SQL” が通る（many2one/one2manyの存在確認）

ここでやること

  • 取り込み方式はCSVでも一旦OK（あなたの認識通り「CSVに落とすが故の事故」なので、後でPostgres→Postgres直同期に置き換えればよい）
  • 重要なのは「テーブル名・カラム名をここで確定して固定」すること（後工程のブレを止める）

Phase C2: Chromaの起動＋collection常在（top_k>0 の前提）

ゴール

  • Chromaが起動していて、少なくとも portal_field_ja と portal_view_common_ja が 常に存在
  • 無ければ 起動時に get_or_create される（“存在しない”で落ちない）

ポイント

  • ここが固まるまで top_k=0 をデフォルトにしておくのが安全（ただしレスポンス契約は返す）

Phase C3: retrieval → allowlist抽出（“材料”を安定化）

ゴール

  • hits -> allowed_fields(set) を安定して作れる
  • hits 0件なら RETRIEVAL_NO_HITS を返せる
  • diagnostics に retrieval 情報（top_k/collections/hits）を入れられる

注意

  • allowlist抽出の仕様（metadataに model/field を必ず入れる等）を ここで固定すると、後からJOIN/SQLがブレない

Phase C4: join_planner（View優先＋ir_src検証＋採否ログ）

ゴール

  • view_edges + llm_edges(提案のみ) を decisions に落とし込み
  • 採用条件（ir_src + allowlist）で accepted_edges -> final_joins が作れる
  • 採用0なら VALIDATION_FAILED で返せる（join_plan付き）

このフェーズが“本丸”

  • ここを分離しておくと、LLMやSQL生成が壊れてもJOIN確定の品質は維持できる

Phase C5: SQL生成 → Odoo実行 → EXECUTION_ERROR_* を“必ずきれいに返す”

ゴール

  • Odoo実行で落ちても EXECUTION_ERROR_* + details(sqlstate等) を必ず返す
  • undefined_table/undefined_column/permission/timeout/syntax/other のマッピングを固定

ここで大事

  • あなたの方針通り、OdooDBでズレて死ぬのは許容。その代わり reason_code が運用の命綱。

Phase C6: E2Eスモーク（/query→/plan→/act が“安定して通る”）

ゴール

  • top_k>0 での代表クエリが通る
  • 失敗ケースも契約通り（RETRIEVAL_NO_HITS / VALIDATION_FAILED / EXECUTION_ERROR_*）が返る
  • 監視・診断（timings/sql_hash/guards_applied）が揃う

進め方のコツ（手戻りをさらに減らすルール）

  • Feature flag を最初から：TOP_K=0 / RETRIEVAL_ENABLED / LLM_ENABLED / EXECUTE_ENABLED みたいに段階解放できるようにする
    → これで「壊れたら全部止まる」を防げます。
  • “契約は最初に完成”：中身がスタブでも、reason_code/diagnostics/errors/join_plan の形だけ先に確定。
  • 1フェーズ1PR：レビュー/ロールバックが簡単で、原因の切り分けが速い。
  • DoD（完了条件）を各フェーズに置く：次に進む条件が明確だとブレない。

結論

安全最優先なら、上のように段階化が正解です。
特に、あなたが書いた「OdooDBを検証に使わない」方針は“実行時エラーが必ず起きる設計”なので、Phase C0（レスポンス契約）と Phase C5（EXECUTION_ERROR整形）を先に固めるのが、最短で事故を減らします。

次の一手としては、あなたが宣言している通り **「ir_*_src取り込み（Phase C1）」→「テーブル名/カラム名の確定」**を最初にやるのが良いです。
そこが確定したら、こちらで **検証SQL（many2one/one2many）を“実名に置換した確定版”**と、join_planner.py + ir_src_repo.py の実装雛形（コピペでPR化できる粒度）まで一気に出せます。

Comments

コメントを残す

メールアドレスが公開されることはありません。 ※ が付いている欄は必須項目です