magic-pocket-cli 0.12.0__tar.gz → 0.14.0__tar.gz

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/PKG-INFO

@@ -1,13 +1,13 @@
 Metadata-Version: 2.4
 Name: magic-pocket-cli
-Version: 0.12.0
+Version: 0.14.0
 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.12.0
+Requires-Dist: magic-pocket>=0.13.0
 Requires-Dist: pathspec>=1.0.4
 Requires-Dist: python-on-whales>=0.68.0
 Requires-Dist: pyyaml>=6.0.1

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/pocket_cli/cli/deploy_cli.py

@@ -3,7 +3,7 @@ import webbrowser
 
 import click
 
-from pocket.context import Context
+from pocket.context import Context, deploy_hash_report
 from pocket.utils import echo
 from pocket_cli.mediator import Mediator
 from pocket_cli.resources.aws.state import StateStore
@@ -173,6 +173,11 @@ def _deploy_pipeline(context: Context, *, openpath=None, skip_frontend=False):
     promote 時は context.awscontainer.promote_commit_hash が設定済みで、
     deploy_init 内の image build が retag に置き換わる以外は deploy と同一。
     """
+    # DEPLOY_HASH の解決結果を deploy 時に 1 回可視化する (env 伝播漏れで
+    # 黙って git short hash に落ちる footgun の早期発見用)。
+    deploy_hash_message = deploy_hash_report(context)
+    if deploy_hash_message:
+        echo.info(deploy_hash_message)
     # CodeBuildがソースアップロードにstate bucketを必要とするため、先に作成
     state_store = _create_state_store(context)
     state_store.ensure_bucket()

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/pocket_cli/cli/migrate_cli.py

@@ -17,6 +17,7 @@ import hashlib
 import click
 import yaml
 
+from pocket import __version__
 from pocket.context import Context
 from pocket.utils import echo
 from pocket_cli.cli.deploy_cli import get_resources
@@ -180,6 +181,32 @@ def _report_secret_path(info: dict) -> None:
         echo.info("%s: %s" % (info["key"], label))
 
 
+def _warn_runtime_bump_required(stage: str) -> None:
+    """secret-paths 移設で旧パスが削除されることの事前警告。
+
+    移設は copy→旧削除を一括で行うため、この stage に deploy 済みの Lambda runtime
+    (magic-pocket[django]) が古いままだと、旧パス (削除済) を参照して DATABASE_URL を
+    解決できず cold start の INIT で落ちる。移設後に runtime を CLI と同版へ bump して
+    再デプロイするまで stage が黙って壊れる footgun を、移設前に明示する
+    (template-hash が「先に deploy」で中断するのと同じ思想の事前ガード)。
+    """
+    echo.warning(
+        "⚠ 移設すると旧パスは削除されます (copy→旧削除は一括)。この stage に "
+        "deploy 済みの Lambda runtime (magic-pocket[django]) が古いままだと、"
+        "旧パスを参照して DATABASE_URL を解決できず cold start の INIT で落ちます。"
+    )
+    echo.warning(
+        "  移設後は速やかに runtime を CLI と同版へ上げて再デプロイしてください:\n"
+        "    uv add 'magic-pocket[django]>=%s'\n"
+        "    pocket deploy --stage=%s  (Django なら pocket django deploy --stage=%s)"
+        % (__version__, stage, stage)
+    )
+    echo.warning(
+        "  ※ copy と旧削除が一括のため完全な無停止移行にはなりません。warm instance は "
+        "recycle まで生存するので、移設直後に再デプロイすれば影響を最小化できます。"
+    )
+
+
 def _run_secret_paths(stage: str, *, yes: bool, dry_run: bool) -> None:
     """stored user secret を旧キー基準→新 type 基準パスへ移設する (冪等)。"""
     from pocket_cli.mediator import Mediator
@@ -207,6 +234,7 @@ def _run_secret_paths(stage: str, *, yes: bool, dry_run: bool) -> None:
             "secret-paths: 移設対象はありません (全て移行済み or 未 provision)。"
         )
         return
+    _warn_runtime_bump_required(stage)
     if dry_run:
         echo.info("secret-paths: --dry-run のため実行はしません。")
         return

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/pocket_cli/resources/aws/builders/codebuild.py

@@ -2,6 +2,8 @@ from __future__ import annotations
 
 import json
 import os
+import shutil
+import stat
 import time
 import zipfile
 from pathlib import Path
@@ -9,6 +11,7 @@ from pathlib import Path
 import boto3
 from botocore.exceptions import ClientError
 
+from pocket.utils import echo
 from pocket_cli.resources.aws.builders.dockerignore import (
     load_dockerignore,
     should_include,
@@ -287,7 +290,7 @@ class CodeBuildBuilder:
                     rel = os.path.join(rel_dir, filename) if rel_dir else filename
                     if not should_include(rel, spec):
                         continue
-                    zf.write(os.path.join(dirpath, filename), rel)
+                    self._add_source_file(zf, os.path.join(dirpath, filename), rel)
 
         zip_size_mb = buf.tell() / (1024 * 1024)
         buf.seek(0)
@@ -296,6 +299,31 @@ class CodeBuildBuilder:
         buf.close()
         print("  アップロード完了 (%.1f MB)" % zip_size_mb)
 
+    def _add_source_file(
+        self, zf: zipfile.ZipFile, full_path: str, arcname: str
+    ) -> None:
+        """1 ファイルを zip に追加する（forge VM 由来の footgun を吸収）。
+
+        - 壊れた symlink（host 側参照など、実体が無い）は skip して warning。
+          そのまま ``zipfile.write`` すると ``FileNotFoundError`` で source zip
+          作成ごと落ち、deploy 全体が失敗するため。
+        - 通常ファイルの権限を 0644 / 実行ファイルを 0755 に正規化する。
+          forge VM では ``sed -i`` 編集等で 0600 のファイルが生まれやすく、
+          そのまま image に入ると Lambda の非 root 実行ユーザーが読めず起動時に
+          panic する。mode を落として（他ユーザー read を保証して）防ぐ。
+        """
+        if not os.path.exists(full_path):
+            # os.path.exists は symlink を辿るため、壊れた symlink で False
+            echo.warning("  壊れた symlink を skip: %s" % arcname)
+            return
+        zi = zipfile.ZipInfo.from_file(full_path, arcname)
+        zi.compress_type = zipfile.ZIP_DEFLATED
+        perm = (zi.external_attr >> 16) & 0o777
+        new_perm = 0o755 if perm & stat.S_IXUSR else 0o644
+        zi.external_attr = (new_perm << 16) | (zi.external_attr & 0xFFFF)
+        with open(full_path, "rb") as src, zf.open(zi, "w") as dst:
+            shutil.copyfileobj(src, dst)
+
     # --- ビルド実行 ---
 
     def _start_build(

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/pocket_cli/resources/dsql.py

@@ -29,9 +29,9 @@ class Dsql:
             for cluster in page["clusters"]:
                 identifier = cluster["identifier"]
                 try:
-                    detail = self._client.get_cluster(Identifier=identifier)
+                    detail = self._client.get_cluster(identifier=identifier)
                     tags = self._client.list_tags_for_resource(
-                        ResourceArn=detail["arn"]
+                        resourceArn=detail["arn"]
                     )
                     if tags.get("tags", {}).get("Name") == self.context.tag_name:
                         return detail
@@ -102,7 +102,7 @@ class Dsql:
         if not self.identifier:
             return
         echo.log("Deleting DSQL cluster: %s" % self.identifier)
-        self._client.delete_cluster(Identifier=self.identifier)
+        self._client.delete_cluster(identifier=self.identifier)
         echo.log("Waiting for DSQL cluster deletion...")
         self._wait_deleted(self.identifier, timeout=600)
         echo.success("DSQL cluster was deleted.")
@@ -110,7 +110,7 @@ class Dsql:
     def _wait_active(self, identifier: str, timeout: int = 600, interval: int = 5):
         for i in range(timeout // interval):
             try:
-                res = self._client.get_cluster(Identifier=identifier)
+                res = self._client.get_cluster(identifier=identifier)
                 if res["status"] == "ACTIVE":
                     print("")
                     return
@@ -125,7 +125,7 @@ class Dsql:
     def _wait_deleted(self, identifier: str, timeout: int = 600, interval: int = 5):
         for i in range(timeout // interval):
             try:
-                self._client.get_cluster(Identifier=identifier)
+                self._client.get_cluster(identifier=identifier)
             except ClientError as e:
                 if e.response["Error"]["Code"] == "ResourceNotFoundException":
                     print("")

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/pocket_cli/resources/neon.py

@@ -329,8 +329,11 @@ class Neon:
         # store_url の呼び出し側も branch 不在を確認済だが、直接呼ばれた場合の防御。
         if self.branch is not None:
             return
-        del self.branch
-        del self.endpoint
+        # branch/endpoint の cached_property を無効化し、branch 作成後の再取得を
+        # 強制する。endpoint はこの経路で一度も access していない (cache 未生成) ため
+        # 素の `del` は AttributeError になる。access 有無に依らず安全な pop で消す。
+        self.__dict__.pop("branch", None)
+        self.__dict__.pop("endpoint", None)
         data = {
             "branch": {
                 "name": self.context.branch_name,

{magic_pocket_cli-0.12.0 → magic_pocket_cli-0.14.0}/pyproject.toml

@@ -1,10 +1,10 @@
 [project]
 name = "magic-pocket-cli"
-version = "0.12.0"
+version = "0.14.0"
 description = "CLI and deploy tools for magic-pocket."
 requires-python = ">=3.10"
 dependencies = [
-    "magic-pocket>=0.12.0",
+    "magic-pocket>=0.13.0",
     "click>=8.1.7",
     "jinja2>=3.1.3",
     "python-on-whales>=0.68.0",