magic-pocket-cli 0.35.0__tar.gz → 0.36.0__tar.gz

{magic_pocket_cli-0.35.0 → magic_pocket_cli-0.36.0}/PKG-INFO

@@ -1,13 +1,13 @@
 Metadata-Version: 2.5
 Name: magic-pocket-cli
-Version: 0.35.0
+Version: 0.36.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.36,>=0.35.0
+Requires-Dist: magic-pocket<0.37,>=0.36.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.35.0 → magic_pocket_cli-0.36.0}/pocket_cli/cli/inbound_cli.py

@@ -10,7 +10,8 @@ from botocore.config import Config
 from pocket.inbound import SETUP_NOTIFICATION, receipt_id, validate_receipt_id
 from pocket.inbound_context import InboundContext
 from pocket.settings import Settings, parse_handler_ref
-from pocket_cli.resources.inbound import Inbound
+from pocket_cli.cli import interaction
+from pocket_cli.resources.inbound import Inbound, no_active_rule_set_guide
 
 
 @click.group()
@@ -27,48 +28,58 @@ def inbound(ctx, stage, name):
 
 
 @inbound.command()
+@click.option(
+    "--format",
+    "output_format",
+    type=click.Choice(["text", "json"]),
+    default="text",
+    help="DNSレコードの出力形式",
+)
 @click.pass_obj
-def init(resource):
-    """共有rule setの初期化とドメイン検証の申請（初回のみ）。"""
-    ses = boto3.client("ses", region_name=resource.context.region)
+def init(resource, output_format):
+    """ドメイン検証を申請し、登録すべきDNSレコードを出力する（初回のみ）。
+
+    receipt rule setはaccount/regionで1つしかactiveにできない共有物のため、
+    project単位の道具であるpocketは作成も有効化もしない。
+    """
+    region = resource.context.region
+    ses = boto3.client("ses", region_name=region)
     active = ses.describe_active_receipt_rule_set().get("Metadata", {}).get("Name")
     if not active:
-        name = "pocket-inbound"
-        names = [
-            item["Name"]
-            for page in ses.get_paginator("list_receipt_rule_sets").paginate()
-            for item in page["RuleSets"]
-        ]
-        if name in names:
-            raise click.ClickException(
-                "pocket-inboundが既にあります。内容を確認し管理者が有効化してください"
-            )
-        click.confirm(
-            "このaccount/regionで初期化を同時実行していないことを確認してください。"
-            "共有rule set pocket-inboundを作成・有効化しますか？",
-            abort=True,
-        )
-        if ses.describe_active_receipt_rule_set().get("Metadata"):
-            raise click.ClickException("active setが変わりました。再実行してください")
-        ses.create_receipt_rule_set(RuleSetName=name)
-        if ses.describe_active_receipt_rule_set().get("Metadata"):
-            raise click.ClickException(
-                "初期化中にactive setが変わりました。有効化を中止します"
-            )
-        ses.set_active_receipt_rule_set(RuleSetName=name)
-        active = ses.describe_active_receipt_rule_set().get("Metadata", {}).get("Name")
-        if active != name:
-            raise click.ClickException("有効化後のactive setが一致しません")
+        raise click.ClickException(no_active_rule_set_guide(region))
     domain = resource.context.config.domain
     result = ses.get_identity_verification_attributes(Identities=[domain])
     identity = result.get("VerificationAttributes", {}).get(domain, {})
     token = identity.get("VerificationToken")
     if not token and identity.get("VerificationStatus") != "Success":
         token = ses.verify_domain_identity(Domain=domain)["VerificationToken"]
-    click.echo(f"active rule set: {active}")
+    records = []
     if token:
-        click.echo(f"TXT _amazonses.{domain} {token}")
-    click.echo(f"MX {domain} 10 inbound-smtp.{resource.context.region}.amazonaws.com")
+        records.append({"type": "TXT", "name": f"_amazonses.{domain}", "value": token})
+    records.append(
+        {
+            "type": "MX",
+            "name": domain,
+            "priority": 10,
+            "value": f"inbound-smtp.{region}.amazonaws.com",
+        }
+    )
+    if output_format == "json":
+        payload = {
+            "active_rule_set": active,
+            "domain": domain,
+            "region": region,
+            "verification_status": identity.get("VerificationStatus") or "Pending",
+            "records": records,
+        }
+        click.echo(json.dumps(payload, ensure_ascii=False, indent=2))
+        return
+    click.echo(f"active rule set: {active}")
+    for record in records:
+        if record["type"] == "TXT":
+            click.echo(f"TXT {record['name']} {record['value']}")
+        else:
+            click.echo(f"MX {record['name']} {record['priority']} {record['value']}")
     click.echo("SES受信対応リージョンとMXの既存配送先を確認してDNSを登録してください。")
 
 
@@ -94,11 +105,17 @@ def show_yaml(resource):
 
 
 @inbound.command()
+@click.option(
+    "--yes", "-y", is_flag=True, default=False, help="確認プロンプトをスキップ"
+)
 @click.pass_obj
-def destroy(resource):
+def destroy(resource, yes):
     """無効化・配送猶予・滞留解消後に自分のルールとスタックを削除する。"""
-    click.confirm(
-        "受信停止後36時間以上経過し、原本の照合・必要な退避を終えましたか？", abort=True
+    interaction.set_assume_yes(yes)
+    interaction.confirm(
+        "受信停止後36時間以上経過し、原本の照合・必要な退避を終えましたか？",
+        default=False,
+        abort=True,
     )
     resource.delete()
 

{magic_pocket_cli-0.35.0 → magic_pocket_cli-0.36.0}/pocket_cli/resources/inbound.py

@@ -14,16 +14,31 @@ from pocket_cli.resources.aws.stack_backed import StackBackedResource
 from pocket_cli.resources.inbound_template import build_template
 
 
+def no_active_rule_set_guide(region: str) -> str:
+    """active setが無いaccount/region向けの案内。
+
+    receipt rule setはaccount/regionで1つしかactiveにできない共有物なので、
+    pocketは作成・有効化しない。名前はSESコンソールが自動作成する
+    default-rule-setに揃えて案内する（既にactiveなsetがあれば名前は問わない）。
+    """
+    return (
+        "active な receipt rule set がありません。"
+        "account/regionで1回だけ次を実行してください"
+        "（account単位の土台なので、pocketは作成・有効化しません）:\n"
+        "  aws ses create-receipt-rule-set"
+        f" --rule-set-name default-rule-set --region {region}\n"
+        "  aws ses set-active-receipt-rule-set"
+        f" --rule-set-name default-rule-set --region {region}"
+    )
+
+
 def resolve_rules(
     active: dict, ctx: InboundContext, deployed: dict | None
 ) -> tuple[str, str | None]:
     """明示指定なしならactive setへ末尾追加。既存bindingの漂流は拒否する。"""
     name = active.get("Metadata", {}).get("Name")
     if not name:
-        raise ValueError(
-            "active receipt rule set がありません。"
-            "pocket resource inbound init を実行してください"
-        )
+        raise ValueError(no_active_rule_set_guide(ctx.region))
     if ctx.config.rule_set and ctx.config.rule_set != name:
         raise ValueError("明示したrule_setがactive setと一致しません")
     expected = (deployed or {}).get("RuleSet") or ctx.config.rule_set

{magic_pocket_cli-0.35.0 → magic_pocket_cli-0.36.0}/pyproject.toml

@@ -1,12 +1,12 @@
 [project]
 name = "magic-pocket-cli"
-version = "0.35.0"
+version = "0.36.0"
 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.35.0,<0.36",
+    "magic-pocket>=0.36.0,<0.37",
     "click>=8.1.7",
     "jinja2>=3.1.3",
     "python-on-whales>=0.68.0",