magic-pocket-cli 0.28.1__tar.gz → 0.28.2__tar.gz

{magic_pocket_cli-0.28.1 → magic_pocket_cli-0.28.2}/PKG-INFO

@@ -1,13 +1,13 @@
 Metadata-Version: 2.5
 Name: magic-pocket-cli
-Version: 0.28.1
+Version: 0.28.2
 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.29,>=0.28.1
+Requires-Dist: magic-pocket<0.29,>=0.28.2
 Requires-Dist: pathspec>=1.0.4
 Requires-Dist: python-on-whales>=0.68.0
 Requires-Dist: pyyaml>=6.0.1

{magic_pocket_cli-0.28.1 → magic_pocket_cli-0.28.2}/pocket_cli/resources/backup.py

@@ -164,8 +164,8 @@ class Backup:
                 " provisioning をスキップしました: %s" % e
             )
             echo.warning(
-                "  deploy role に backup:* を付与してください"
-                " (`pocket permissions list` 参照)。"
+                "  deploy role に backup グループの権限 (backup:* と kms 系)"
+                " を付与してください (`pocket permissions list` 参照)。"
             )
 
     def _cleanup_legacy_plans(self) -> None:

{magic_pocket_cli-0.28.1 → magic_pocket_cli-0.28.2}/pocket_cli/resources/cloudfront.py

@@ -10,6 +10,7 @@ from pathlib import Path
 from typing import TYPE_CHECKING, Literal
 
 import boto3
+from boto3.s3.transfer import TransferConfig
 from botocore.exceptions import ClientError
 from pydantic import BaseModel
 
@@ -22,6 +23,14 @@ if TYPE_CHECKING:
     from pocket.context import CloudFrontContext, RouteContext
     from pocket_cli.mediator import Mediator
 
+# upload_dir の差分 skip (_is_unchanged) は ETag が素の MD5 であることに依存する。
+# boto3 既定の multipart 閾値 (8MB) を超えると ETag が `<md5-of-part-md5s>-<N>`
+# 形式になり、そのファイルは内容不変でも毎 deploy「変更あり」= 再アップロード +
+# route 全体の invalidation になってしまう。単一 PUT の上限 (5GB) に対して
+# 十分安全な 128MB まで multipart を使わない。これを超えるファイルは従来どおり
+# 常に再アップロードされる (_is_unchanged の docstring 参照)
+_UPLOAD_TRANSFER_CONFIG = TransferConfig(multipart_threshold=128 * 1024 * 1024)
+
 
 class OriginAccessControl(BaseModel):
     Id: str
@@ -334,6 +343,7 @@ class CloudFront:
                 self.context.bucket_name,
                 s3_key,
                 ExtraArgs=extra_args,
+                Config=_UPLOAD_TRANSFER_CONFIG,
             )
             uploaded += 1
             echo.log("アップロード: s3://%s/%s" % (self.context.bucket_name, s3_key))
@@ -562,6 +572,13 @@ def _is_unchanged(file: Path, etag: str | None) -> bool:
     分からないとローカルから再現できない。SSE-KMS 等でも ETag は MD5 に
     ならない。これらは「不明」= 変更ありとして扱い、常に再アップロードする。
 
+    このため upload 側は multipart 閾値を 128MB に引き上げている
+    (_UPLOAD_TRANSFER_CONFIG)。boto3 既定の 8MB のままでは、8MB 超のファイルが
+    1 個あるだけで毎 deploy 再アップロード + route 全体の invalidation が走る。
+    過去に multipart で上がった既存オブジェクトも、閾値以下なら次の deploy で
+    単一 PUT により上げ直され、以後は skip が効くようになる (自己回復)。
+    128MB 超のファイルだけは引き続き毎回再アップロードになる。
+
     サイズ比較での代替はしない。同じサイズで内容が違うファイルを「変更なし」と
     誤判定すると、そのファイルは以後どの deploy でも更新されなくなる。
     誤って skip する事故に比べれば、再アップロードのコストは安い。

{magic_pocket_cli-0.28.1 → magic_pocket_cli-0.28.2}/pyproject.toml

@@ -1,12 +1,12 @@
 [project]
 name = "magic-pocket-cli"
-version = "0.28.1"
+version = "0.28.2"
 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.28.1,<0.29",
+    "magic-pocket>=0.28.2,<0.29",
     "click>=8.1.7",
     "jinja2>=3.1.3",
     "python-on-whales>=0.68.0",