magic-pocket-cli 0.41.0__tar.gz → 0.41.1__tar.gz

{magic_pocket_cli-0.41.0 → magic_pocket_cli-0.41.1}/PKG-INFO

@@ -1,13 +1,13 @@
 Metadata-Version: 2.5
 Name: magic-pocket-cli
-Version: 0.41.0
+Version: 0.41.1
 Summary: CLI and deploy tools for magic-pocket.
 Requires-Python: >=3.10
 Requires-Dist: awscrt>=0.19.0
 Requires-Dist: click>=8.1.7
 Requires-Dist: deepdiff>=6.7.1
 Requires-Dist: jinja2>=3.1.3
-Requires-Dist: magic-pocket<0.42,>=0.41.0
+Requires-Dist: magic-pocket<0.42,>=0.41.1
 Requires-Dist: pathspec>=1.0.4
 Requires-Dist: python-on-whales>=0.68.0
 Requires-Dist: pyyaml>=6.0.1

{magic_pocket_cli-0.41.0 → magic_pocket_cli-0.41.1}/pocket_cli/cli/deploy_cli.py

@@ -1,5 +1,6 @@
 import inspect
 import webbrowser
+from collections.abc import Callable
 
 import click
 
@@ -113,15 +114,53 @@ def deploy_init_resources(context: Context, *, state_bucket: str = ""):
         resource.deploy_init()
 
 
-def deploy_frontend(context: Context, *, skip_build: bool = False):
+def deploy_frontend(
+    context: Context,
+    *,
+    skip_build: bool = False,
+    exclude: frozenset[tuple[str, str]] = frozenset(),
+):
+    """uploadable な route を upload する。
+
+    exclude は upload 済みの (cloudfront, route) で、対象から外す。
+    """
     for _name, cf_ctx in context.cloudfront.items():
-        cf = CloudFront(cf_ctx)
-        if not cf_ctx.uploadable_routes:
+        routes = [
+            r for r in cf_ctx.uploadable_routes if (cf_ctx.name, r.name) not in exclude
+        ]
+        if not routes:
             continue
+        cf = CloudFront(cf_ctx)
         if cf.status == "NOEXIST":
             echo.warning("CloudFront '%s' が未作成です。スキップします。" % cf_ctx.name)
             continue
-        cf.upload(skip_build=skip_build)
+        cf.upload(skip_build=skip_build, routes=routes)
+
+
+def deploy_versioned_frontend(context: Context) -> frozenset[tuple[str, str]]:
+    """versioning route を Lambda 更新より先に upload する。
+
+    Lambda が新しい版の URL を出し始めた後に upload すると、その間に読まれた
+    ファイルは upload 前の内容 (新規ファイルなら 403) が新しい URL のキャッシュに入り、
+    ブラウザに versioned_max_age の間残る (KN1676)。versioning route の URL は版ごとに
+    変わるので、先に upload しても稼働中の版の表示は壊れない。
+
+    SPA など versioning の無い route は対象外 (HTML が Lambda より先に切り替わる)。
+    CloudFront が未作成 (初回 deploy) の場合は upload せず、deploy_frontend に任せる。
+    戻り値は upload した (cloudfront, route) で、deploy_frontend の exclude に渡す。
+    """
+    uploaded: set[tuple[str, str]] = set()
+    for _name, cf_ctx in context.cloudfront.items():
+        routes = [r for r in cf_ctx.uploadable_routes if r.versioning]
+        if not routes:
+            continue
+        cf = CloudFront(cf_ctx)
+        if cf.status == "NOEXIST":
+            continue
+        echo.log("versioning route を Lambda 更新より先に upload します...")
+        cf.upload(routes=routes)
+        uploaded.update((cf_ctx.name, r.name) for r in routes)
+    return frozenset(uploaded)
 
 
 def upload_managed_assets(context: Context):
@@ -216,11 +255,20 @@ def build_image(context: Context, *, tag: str) -> list[str]:
     return targets
 
 
-def _deploy_pipeline(context: Context, *, openpath=None, skip_frontend=False):
+def _deploy_pipeline(
+    context: Context,
+    *,
+    openpath=None,
+    skip_frontend=False,
+    before_lambda_update: Callable[[], None] | None = None,
+):
     """deploy / promote 共通のパイプライン本体。
 
     promote 時は各 container の promote_commit_hash が設定済みで、
     deploy_init 内の image build が retag に置き換わる以外は deploy と同一。
+
+    before_lambda_update は image build の後・Lambda 更新 (deploy_resources) の前に
+    呼ばれる。`pocket django deploy` が static を先に upload するのに使う。
     """
     # DEPLOY_HASH の解決結果を deploy 時に 1 回可視化する (env 伝播漏れで
     # 黙って git short hash に落ちる footgun の早期発見用)。
@@ -239,6 +287,12 @@ def _deploy_pipeline(context: Context, *, openpath=None, skip_frontend=False):
     state_bucket = state_store.bucket_name
     check_removed_inbound(context, state_store)
     deploy_init_resources(context, state_bucket=state_bucket)
+    # static は Lambda が新しい版の URL を出し始める前に upload する (KN1676)
+    uploaded_routes: frozenset[tuple[str, str]] = frozenset()
+    if not skip_frontend:
+        uploaded_routes = deploy_versioned_frontend(context)
+    if before_lambda_update is not None:
+        before_lambda_update()
     deploy_resources(context, state_bucket=state_bucket)
     cleanup_unused_inbound_domains(context, state_store)
     # リリース跨ぎ移行の掃除フェーズ (旧配置の削除は deploy 成功後にしか
@@ -246,7 +300,7 @@ def _deploy_pipeline(context: Context, *, openpath=None, skip_frontend=False):
     migrations.run_deploy_cleanup(context)
     upload_managed_assets(context)
     if not skip_frontend:
-        deploy_frontend(context)
+        deploy_frontend(context, exclude=uploaded_routes)
     # build 時の other-read 警告はログに埋もれて気付けないため最後に再掲する
     resummarize_world_read_warnings()
     # DLQ アラートの email 購読が未確認のままだと通知が届かないため、
@@ -272,12 +326,29 @@ def _deploy_pipeline(context: Context, *, openpath=None, skip_frontend=False):
 )
 @removed_skip_check_existing
 def deploy(stage: str, openpath, skip_frontend, yes):
+    run_deploy(stage, openpath=openpath, skip_frontend=skip_frontend, yes=yes)
+
+
+def run_deploy(
+    stage: str,
+    *,
+    openpath=None,
+    skip_frontend: bool = False,
+    yes: bool = False,
+    before_lambda_update: Callable[[], None] | None = None,
+):
+    """`pocket deploy` の本体。`pocket django deploy` からも呼ばれる。"""
     from pocket_cli.cli.aws_auth import check_aws_credentials
 
     interaction.set_assume_yes(yes)
     check_aws_credentials()
     context = Context.from_toml(stage=stage)
-    _deploy_pipeline(context, openpath=openpath, skip_frontend=skip_frontend)
+    _deploy_pipeline(
+        context,
+        openpath=openpath,
+        skip_frontend=skip_frontend,
+        before_lambda_update=before_lambda_update,
+    )
 
 
 @click.command()
@@ -295,6 +366,21 @@ def promote(stage: str, commit_hash, openpath, skip_frontend, yes):
     `pocket build` で push した image に :<stage> タグを移し、
     インフラ/Lambda を更新する。image build は行わない (build once の昇格)。
     """
+    run_promote(
+        stage, commit_hash, openpath=openpath, skip_frontend=skip_frontend, yes=yes
+    )
+
+
+def run_promote(
+    stage: str,
+    commit_hash: str,
+    *,
+    openpath=None,
+    skip_frontend: bool = False,
+    yes: bool = False,
+    before_lambda_update: Callable[[], None] | None = None,
+):
+    """`pocket promote` の本体。`pocket django promote` からも呼ばれる。"""
     from pocket_cli.cli.aws_auth import check_aws_credentials
 
     interaction.set_assume_yes(yes)
@@ -304,7 +390,12 @@ def promote(stage: str, commit_hash, openpath, skip_frontend, yes):
         raise click.ClickException("container がこの stage に設定されていません。")
     for c_ctx in context.container.values():
         c_ctx.promote_commit_hash = commit_hash
-    _deploy_pipeline(context, openpath=openpath, skip_frontend=skip_frontend)
+    _deploy_pipeline(
+        context,
+        openpath=openpath,
+        skip_frontend=skip_frontend,
+        before_lambda_update=before_lambda_update,
+    )
 
 
 def _get_deploy_url(context: Context) -> str | None:

{magic_pocket_cli-0.41.0 → magic_pocket_cli-0.41.1}/pocket_cli/django_cli.py

@@ -16,8 +16,10 @@ from pocket.django.utils import get_storages, resolve_django_container
 from pocket.utils import echo
 from pocket_cli.cli import interaction
 from pocket_cli.cli.build_cli import build
+from pocket_cli.cli.deploy_cli import run_deploy, run_promote
 from pocket_cli.cli.removed_flags import removed_skip_check_existing
 from pocket_cli.resources.container import Container
+from pocket_cli.resources.s3 import S3
 
 
 @click.group()
@@ -92,23 +94,65 @@ def _update_dotenv(jinja2_env: Environment):
 )
 @removed_skip_check_existing
 def deploy(stage: str, openpath, yes, skip_migrate):
-    from pocket_cli.cli.deploy_cli import deploy as pocket_deploy
-
     # pocket deploy を実行（インフラ + SPA フロントエンド）
-    ctx = click.Context(pocket_deploy)
-    ctx.invoke(
-        pocket_deploy,
-        stage=stage,
+    static_handled: list[bool] = []
+    run_deploy(
+        stage,
         openpath=None,
         skip_frontend=False,
         yes=yes,
+        before_lambda_update=lambda: static_handled.append(
+            _deploystatic_before_lambda_update(stage)
+        ),
+    )
+    _django_post_deploy(
+        stage,
+        yes=yes,
+        openpath=openpath,
+        skip_migrate=skip_migrate,
+        static_handled=any(static_handled),
     )
-    _django_post_deploy(stage, yes=yes, openpath=openpath, skip_migrate=skip_migrate)
 
 
-def _django_post_deploy(stage: str, *, yes: bool, openpath, skip_migrate: bool = False):
+def _deploystatic_before_lambda_update(stage: str) -> bool:
+    """static を Lambda 更新より先に upload する。deploystatic を済ませたら True。
+
+    Lambda が新しい版の URL を出し始めた後に upload すると、その間に読まれた
+    ファイルは upload 前の内容が新しい URL のキャッシュに入り、ブラウザに長期間残る
+    (KN1676)。collectstatic の失敗も Lambda を更新する前に分かる。
+
+    bucket が未作成 (初回 deploy) の場合と、staticfiles が S3 でない場合は False を
+    返し、従来どおり deploy 後の `_django_post_deploy` に任せる。確認に No と
+    答えた場合も True を返す (deploy 後にもう一度聞かない)。
+    """
+    context = Context.from_toml(stage=stage)
+    if _staticfiles_publish_mode(context) == "command":
+        return False
+    c = resolve_django_container(context)
+    storage = c.django.storages.get("staticfiles") if c and c.django else None
+    if not storage or storage.store != "s3":
+        return False
+    if not context.s3 or not S3(context.s3).exists():
+        return False
+    if interaction.confirm("deploystatic?", default=True):
+        collectstatic_locally(stage, link=_staticfiles_link(context))
+        upload_collected_staticfiles(stage)
+    return True
+
+
+def _django_post_deploy(
+    stage: str,
+    *,
+    yes: bool,
+    openpath,
+    skip_migrate: bool = False,
+    static_handled: bool = False,
+):
     """deploy / promote 共通の Django 固有後処理 (collectstatic + migrate + URL)。
 
+    static_handled=True は、Lambda 更新の前に deploystatic を済ませた
+    (`_deploystatic_before_lambda_update`) ことを表し、ここでは繰り返さない。
+
     skip_migrate=True なら migrate は確認ごと省く。`-y` は「聞かれたことに全部
     yes」の意味を保ちたいので、非対話で migrate を外す手段はフラグ側に置く
     (DB が到達不能でもインフラ更新だけ通したい、という状況が実在する)。
@@ -116,7 +160,9 @@ def _django_post_deploy(stage: str, *, yes: bool, openpath, skip_migrate: bool =
     if yes:
         interaction.set_assume_yes(True)
     context = Context.from_toml(stage=stage)
-    if _staticfiles_publish_mode(context) == "command":
+    if static_handled:
+        pass
+    elif _staticfiles_publish_mode(context) == "command":
         echo.info(
             'staticfiles is publish = "command": skipping deploystatic. '
             "Publish with `pocket django deploystatic --stage %s`." % stage
@@ -163,18 +209,24 @@ def promote(stage: str, commit_hash, openpath, yes, skip_migrate):
     を更新する。image build は行わない (build once の昇格)。静的アセットは deploy 時
     と同様にローカルでビルドして upload する。
     """
-    from pocket_cli.cli.deploy_cli import promote as pocket_promote
-
-    ctx = click.Context(pocket_promote)
-    ctx.invoke(
-        pocket_promote,
-        stage=stage,
-        commit_hash=commit_hash,
+    static_handled: list[bool] = []
+    run_promote(
+        stage,
+        commit_hash,
         openpath=None,
         skip_frontend=False,
         yes=yes,
+        before_lambda_update=lambda: static_handled.append(
+            _deploystatic_before_lambda_update(stage)
+        ),
+    )
+    _django_post_deploy(
+        stage,
+        yes=yes,
+        openpath=openpath,
+        skip_migrate=skip_migrate,
+        static_handled=any(static_handled),
     )
-    _django_post_deploy(stage, yes=yes, openpath=openpath, skip_migrate=skip_migrate)
 
 
 # 旧コマンドは同じ Click command の別名として維持する。

{magic_pocket_cli-0.41.0 → magic_pocket_cli-0.41.1}/pocket_cli/resources/cloudfront.py

@@ -279,9 +279,18 @@ class CloudFront:
             "managed_assets: %d ファイルをアップロードしました" % len(uploaded_keys)
         )
 
-    def upload(self, *, skip_build: bool = False):
+    def upload(
+        self, *, skip_build: bool = False, routes: list[RouteContext] | None = None
+    ):
+        """uploadable な route を build して S3 へ同期する。
+
+        routes を渡すとその route だけを対象にする (deploy が versioning route を
+        Lambda 更新より先に upload するため)。
+        """
+        if routes is None:
+            routes = self.context.uploadable_routes
         changed_routes: list[RouteContext] = []
-        for route in self.context.uploadable_routes:
+        for route in routes:
             if route.build_cmd and not skip_build:
                 echo.info("ビルド実行: %s" % route.build_cmd)
                 try:

{magic_pocket_cli-0.41.0 → magic_pocket_cli-0.41.1}/pocket_cli/templates/cloudformation/cloudfront.yaml

@@ -253,7 +253,12 @@ Resources:
             - DELETE
         # {% else %}
         DefaultCacheBehavior:
+          # {% if default_route.is_deploy_hash %}
+          CachePolicyId:
+            Ref: DeployHashCachePolicy{{ default_route.yaml_key }}
+          # {% else %}
           CachePolicyId: "658327ea-f89d-4fab-a63d-7e88639e58f6" # Managed-CachingOptimized
+          # {% endif %}
           TargetOriginId: "{{ resource_prefix }}{{ default_route.name }}"
           ViewerProtocolPolicy: "redirect-to-https"
           # {% if default_route.is_spa or default_route.versioning or basic_auth or has_redirect_from %}
@@ -262,6 +267,10 @@ Resources:
             - EventType: viewer-request
               FunctionARN:
                 Fn::GetAtt: UrlFallbackFunction{{ default_route.yaml_key }}.FunctionMetadata.FunctionARN
+            # {% elif default_route.is_deploy_hash %}
+            - EventType: viewer-request
+              FunctionARN:
+                Fn::GetAtt: DeployHashStripFunction{{ default_route.yaml_key }}.FunctionMetadata.FunctionARN
             # {% elif basic_auth or has_redirect_from %}
             - EventType: viewer-request
               FunctionARN:

{magic_pocket_cli-0.41.0 → magic_pocket_cli-0.41.1}/pyproject.toml

@@ -1,12 +1,12 @@
 [project]
 name = "magic-pocket-cli"
-version = "0.41.0"
+version = "0.41.1"
 description = "CLI and deploy tools for magic-pocket."
 requires-python = ">=3.10"
 dependencies = [
     # lib は同 minor に束縛する。0.x では minor が breaking の単位のため、
     # 上限なしだと cli を pin した消費者にも lib の破壊的変更が届いてしまう
-    "magic-pocket>=0.41.0,<0.42",
+    "magic-pocket>=0.41.1,<0.42",
     "click>=8.1.7",
     "jinja2>=3.1.3",
     "python-on-whales>=0.68.0",