addashboard-cli 0.13.0 → 0.15.0

package/skills/ad-ops/scripts/plan_lint.py

@@ -432,6 +432,7 @@ def summarize(plan: dict) -> None:
                     or a.get("videoId")
                     or a.get("imageHash")
                     or a.get("youtubeVideoId")
+                    or ",".join(v.get("youtubeVideoId") or v.get("videoPath") or "?" for v in a.get("videos") or [])
                     or ",".join((a.get("marketingImages") or []) + (a.get("squareMarketingImages") or []))
                     or "-"
                 )
@@ -466,6 +467,14 @@ def lint(plan: dict, plan_path: str = "") -> None:
     # refs の鮮度と無関係にローカルで確定する
     plan_cp_names: set[str] = set()
 
+    # YDA のバナー広告は 1200x628 (Card) を受け付けない。入稿計画JSONに画像サイズは
+    # 載らないので、参照表の card 側に登録されている mediaId を集めて逆引きする。
+    card_media_ids = {
+        str(v.get("card"))
+        for v in (account.get("refs", {}).get("media") or {}).values()
+        if isinstance(v, dict) and v.get("card")
+    }
+
     if not base:
         warn(f"アカウントJSONに baseArticleUrl がありません（{platform}/{plan.get('accountId')}）。"
              " finalUrl が相対パスになる原因になります")
@@ -530,7 +539,7 @@ def lint(plan: dict, plan_path: str = "") -> None:
                 # クリエイティブの持ち方は媒体で違う。
                 #   yda    : mediaIds / videoMediaId
                 #   meta   : imageHash / videoId
-                #   google : youtubeVideoId / marketingImages / squareMarketingImages
+                #   google : youtubeVideoId / videos[] / marketingImages / squareMarketingImages
                 #   bytedance : videoId + imageIds (カバー画像) / アップロード前は videoPath
                 has_creative = bool(
                     media
@@ -539,6 +548,7 @@ def lint(plan: dict, plan_path: str = "") -> None:
                     or a.get("imageHash")
                     or a.get("imageIds")
                     or a.get("youtubeVideoId")
+                    or a.get("videos")
                     or a.get("marketingImages")
                     or a.get("squareMarketingImages")
                 )
@@ -569,6 +579,7 @@ def lint(plan: dict, plan_path: str = "") -> None:
                 if a.get("coverImagePath"):
                     local_paths.append(a["coverImagePath"])
                 local_paths.extend(a.get("coverImagePaths") or [])
+                local_paths.extend(v["videoPath"] for v in a.get("videos") or [] if v.get("videoPath"))
                 for raw in local_paths:
                     resolved = resolve_media_path(raw, base_dir)
                     if not os.path.isfile(resolved):
@@ -576,19 +587,57 @@ def lint(plan: dict, plan_path: str = "") -> None:
                             f"        → 解決先 {resolved}（相対パスの基準は計画JSONのディレクトリ "
                             f"{base_dir}）")
 
+                # YDA のバナー広告 (BANNER_AD) に 1200x628 の画像は使えない。
+                # 公式の入稿規定でバナー(画像)の対応アスペクト比は 1:1 / 6:5 等で、
+                # 1.91:1 はレスポンシブ側のサイズ (2026-10-13 に広告事業部へ確認済み)。
+                # 実 API は 220115 Unexpected aspect ratio で弾くが dry-run は通るので、
+                # プランナーと二重に止める (プランナーを経由しない計画にも効かせる)。
+                if platform == "yda" and (a.get("adType") or "RESPONSIVE_AD") == "BANNER_AD":
+                    banner_cards = [
+                        str(m) for m in (a.get("mediaIds") or []) if str(m) in card_media_ids
+                    ]
+                    if banner_cards:
+                        err(f"CP「{name}」のバナー広告に Card(1200x628) の画像が指定されています"
+                            f"（{a.get('adName') or ''} / mediaId {', '.join(banner_cards)}）\n"
+                            f"        → Yahoo はバナー広告の 1200x628 を受け付けません"
+                            f"（220115 Unexpected aspect ratio）。"
+                            f"広告タイプをレスポンシブにするか、形式を SQ にしてください")
+
                 # Google (Demand Gen) の動画は「動画IDかローカルパスのどちらか一方」。
                 # 両方あると CLI が余計にアップロードし、使われない動画が YouTube に残る。
                 # backend も superRefine で 400 にするが、CLI はその手前でアップロードを
                 # 済ませてしまうので lint でも止める（Google の安全弁は二重が原則）。
+                # 複数本（縦・横など・#347）は videos[]。1 本だけなら旧形式でもよい。
                 if platform == "google" and a.get("type") == "DEMAND_GEN_VIDEO_RESPONSIVE_AD":
-                    if a.get("youtubeVideoId") and a.get("videoPath"):
-                        err(f"CP「{name}」の広告に youtubeVideoId と videoPath が両方あります"
-                            f"（{a.get('adName') or ''}）\n"
-                            f"        → アップロードした動画は使われません。差し替えたいなら "
-                            f"youtubeVideoId を、既存動画のままでよいなら videoPath を消してください")
-                    if not a.get("youtubeVideoId") and not a.get("videoPath"):
-                        err(f"CP「{name}」の動画広告に動画がありません（{a.get('adName') or ''}）\n"
-                            f"        → youtubeVideoId か、アップロード前なら videoPath が必要")
+                    label = a.get("adName") or ""
+                    legacy = a.get("youtubeVideoId") or a.get("videoPath")
+                    videos = a.get("videos")
+                    if videos is not None and legacy:
+                        err(f"CP「{name}」の動画広告に videos と youtubeVideoId / videoPath が両方あります"
+                            f"（{label}）\n        → どちらか一方の形式で書いてください")
+                        videos = []
+                    else:
+                        if videos is None:
+                            videos = [a] if legacy else []
+                        if not videos:
+                            err(f"CP「{name}」の動画広告に動画がありません（{label}）\n"
+                                f"        → videos、または youtubeVideoId / アップロード前なら videoPath が必要")
+                    if len(videos) > 5:
+                        err(f"CP「{name}」の動画広告の動画が {len(videos)} 本あります（{label}）"
+                            f"\n        → 1 広告 5 本までです")
+                    seen_v = set()
+                    for k, v in enumerate(videos, 1):
+                        if v.get("youtubeVideoId") and v.get("videoPath"):
+                            err(f"CP「{name}」の動画{k}本目に youtubeVideoId と videoPath が両方あります"
+                                f"（{label}）\n"
+                                f"        → アップロードした動画は使われません。差し替えたいなら "
+                                f"youtubeVideoId を、既存動画のままでよいなら videoPath を消してください")
+                        key = v.get("youtubeVideoId") or v.get("videoPath")
+                        if not key:
+                            err(f"CP「{name}」の動画{k}本目が空です（{label}）")
+                        elif key in seen_v:
+                            err(f"CP「{name}」の動画{k}本目が他の動画と重複しています: {key}（{label}）")
+                        seen_v.add(key)
 
                 # TikTok の SINGLE_VIDEO は **動画とカバー画像の両方**が必要。
                 # 片方だけだと lint を通り、live で動画だけアップロードされた後に

package/skills/ad-ops/scripts/test_assets_sync.py

@@ -0,0 +1,132 @@
+#!/usr/bin/env python3
+"""assets_sync.py の単体テスト（実 API を叩かない）。
+
+    python3 test_assets_sync.py
+
+媒体のレスポンスを固定データで与え、振り分け・除外・マージの判断を確かめる。
+参照表は「既存を絶対に壊さない」のが最重要なので、そこを厚く見る。
+"""
+import json
+import os
+import subprocess
+import sys
+import tempfile
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+import assets_sync as S  # noqa: E402
+
+ok = 0
+ng = []
+
+
+def check(name: str, cond: bool, detail: str = "") -> None:
+    global ok
+    if cond:
+        ok += 1
+        print(f"  ✓ {name}")
+    else:
+        ng.append(f"{name}{(': ' + detail) if detail else ''}")
+        print(f"  ✗ {name}{(': ' + detail) if detail else ''}")
+
+
+def media(mid, name, aspect, approval="APPROVED"):
+    return {"mediaId": mid, "mediaName": name, "aspectRatio": aspect,
+            "approvalStatus": approval}
+
+
+print("── 振り分け（実寸で判定し、名前は使わない） ──")
+c = S.classify([
+    media("1001", "10-1200.jpg", "AR_191_100"),
+    media("1002", "10-1080.jpg", "AR_1_1"),
+    media("1003", "11-1200.jpg", "AR_1200_628"),   # 表記ゆれも card に寄せる
+])
+check("1.91:1 は card", c["candidates"].get(("10", "card")) == ["1001"])
+check("1:1 は sq", c["candidates"].get(("10", "sq")) == ["1002"])
+check("AR_1200_628 も card", c["candidates"].get(("11", "card")) == ["1003"])
+
+# 名前が -1080 でも実寸が 1.91:1 なら card。実データの96%が名前とズレているため
+c = S.classify([media("2001", "20-1080.jpg", "AR_191_100")])
+check("名前が -1080 でも実寸が 1.91:1 なら card",
+      c["candidates"].get(("20", "card")) == ["2001"])
+
+print("\n── 除外 ──")
+c = S.classify([
+    media("3001", "30-1200.jpg", "AR_191_100", "PRE_DISAPPROVED"),
+    media("3002", "31-1200.jpg", "AR_191_100", "POST_DISAPPROVED"),
+    media("3003", "32-1200.jpg", "AR_191_100", "PENDING_UNSPECIFIED_AD"),
+])
+check("否認済み(PRE/POST)は取り込まない", c["skipped_approval"] == 2)
+check("審査保留(PENDING)は取り込む", ("32", "card") in c["candidates"])
+
+c = S.classify([media("4001", "40-xx.jpg", "AR_4_5")])
+check("card/sq 以外の比率は取り込まない", c["skipped_aspect"] == 1 and not c["candidates"])
+
+c = S.classify([media("5001", "banner-1200.jpg", "AR_191_100")])
+check("名前から番号を取れないものは取り込まない",
+      c["skipped_name"] == 1 and not c["candidates"])
+
+print("\n── 既存を壊さない ──")
+existing = {"10": {"card": "9999"}, "20": {"sq": "8888"}}
+c = S.classify([
+    media("1001", "10-1200.jpg", "AR_191_100"),   # 既存と違う mediaId
+    media("1002", "10-1080.jpg", "AR_1_1"),       # 空いている枠
+    media("2002", "20-1080.jpg", "AR_1_1"),       # 既存と違う mediaId
+])
+merged, report, stats = S.merge(existing, c)
+check("既存の値を上書きしない", merged["10"]["card"] == "9999",
+      f'got {merged["10"]["card"]}')
+check("空いている枠には足す", merged["10"].get("sq") == "1002")
+check("食い違いを報告する", stats["conflicts"] == 2, f'got {stats["conflicts"]}')
+check("追加件数が合う", stats["added"] == 1, f'got {stats["added"]}')
+
+print("\n── 同じ番号・同じ形式に候補が複数 ──")
+c = S.classify([
+    media("6001", "60-1200.jpg", "AR_191_100"),
+    media("6002", "60-1200_dup.jpg", "AR_191_100"),
+])
+merged, report, stats = S.merge({}, c)
+check("どれが正か決められないので取り込まない", "60" not in merged)
+check("候補が複数あることを報告する", stats["ambiguous"] == 1)
+
+print("\n── 新規番号 ──")
+c = S.classify([media("7001", "70-1200.jpg", "AR_191_100")])
+merged, report, stats = S.merge({}, c)
+check("参照表に無い番号は新規に作る", merged.get("70") == {"card": "7001"})
+
+print("\n── CLI の挙動（実 API は叩かない） ──")
+with tempfile.TemporaryDirectory() as td:
+    acc_dir = os.path.join(td, "アカウント", "yda")
+    os.makedirs(acc_dir)
+    acc_path = os.path.join(acc_dir, "テスト案件.json")
+    with open(acc_path, "w", encoding="utf-8") as f:
+        json.dump({"accountId": "999", "refs": {"media": {"1": {"sq": "111"}}}},
+                  f, ensure_ascii=False)
+    before = open(acc_path, encoding="utf-8").read()
+
+    env = {**os.environ, "ADOPS_CLI": "false"}  # 必ず失敗する CLI を使わせる
+    script = os.path.join(os.path.dirname(os.path.abspath(__file__)), "assets_sync.py")
+    # SKILL_DIR を一時ディレクトリに向けるため、スクリプトを一時側へ写して実行する
+    tmp_scripts = os.path.join(td, "scripts")
+    os.makedirs(tmp_scripts)
+    import shutil
+    shutil.copy2(script, os.path.join(tmp_scripts, "assets_sync.py"))
+    p = subprocess.run([sys.executable, os.path.join(tmp_scripts, "assets_sync.py"),
+                        "yda", "999", "--write"],
+                       capture_output=True, text=True, env=env)
+    check("取得に失敗したら書き込まない（終了コード1）", p.returncode == 1,
+          f"exit={p.returncode}")
+    check("取得に失敗しても参照表が変わらない",
+          open(acc_path, encoding="utf-8").read() == before)
+
+    p = subprocess.run([sys.executable, os.path.join(tmp_scripts, "assets_sync.py"),
+                        "yda", "不明なID"],
+                       capture_output=True, text=True, env=env)
+    check("参照表が無いアカウントはエラーで止まる", p.returncode == 1)
+
+print()
+if ng:
+    print(f"✗ {len(ng)} 件failed / {ok} 件passed")
+    for n in ng:
+        print(f"    - {n}")
+    sys.exit(1)
+print(f"✓ {ok} 件すべて passed")