direct-cli 0.3.14__tar.gz → 0.3.15__tar.gz

{direct_cli-0.3.14 → direct_cli-0.3.15}/CHANGELOG.md

@@ -1,5 +1,32 @@
 # Changelog
 
+## 0.3.15
+
+**BREAKING CHANGES:**
+
+- `direct v4finance transfer-money` no longer accepts `--currency`, and
+  the wire-body no longer carries `Currency` on `FromCampaigns`/
+  `ToCampaigns` items. The official v4 docs
+  (`dg-v4/reference/TransferMoney`) define `PayCampElement` with only
+  `CampaignID` and `Sum`; `Sum` is in conventional units. The CLI now
+  matches the docs 1:1. Closes #432.
+- `direct v4finance pay-campaigns` no longer accepts `--currency`. The
+  v4 documentation (`dg-v4/reference/PayCampaigns`) defines
+  `PayCampElement` with only `CampaignID` and `Sum` — `Currency` is not
+  part of the wire-body and was never forwarded to the API. The option
+  is removed entirely to make the CLI surface 1:1 with the docs.
+- `direct v4finance pay-campaigns` no longer accepts `--pay-method
+  Overdraft`. The v4 documentation
+  (`dg-v4/reference/PayCampaigns#PayMethod`) lists only `"Bank"` as a
+  supported value; `Overdraft` was a historical undocumented value
+  retained by the CLI for sandbox flow. Strict 1:1 docs alignment
+  drops it.
+- `direct v4finance create-invoice` no longer accepts `--currency`. The
+  v4 documentation (`dg-v4/reference/CreateInvoice`) defines
+  `PayCampElement` with only `CampaignID` and `Sum` — `Currency` is not
+  part of the wire-body and was never forwarded to the API. The option
+  is removed entirely to make the CLI surface 1:1 with the docs.
+
 ## 0.3.14
 
 **Fixed:**

{direct_cli-0.3.14 → direct_cli-0.3.15}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: direct-cli
-Version: 0.3.14
+Version: 0.3.15
 Summary: Command-line interface for Yandex Direct API
 Author: axisrow
 License: MIT
@@ -232,10 +232,10 @@ is omitted. Dry-run output masks the financial token.
 ```bash
 direct v4finance get-clients-units --logins client-login,other-client --format table
 direct v4finance get-credit-limits --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login
-direct v4finance create-invoice --payment 123=100.50 --payment 456=25 --currency RUB --master-token MASTER_TOKEN --operation-num 124 --finance-login agency-login --dry-run
+direct v4finance create-invoice --payment 123=100.50 --payment 456=25 --master-token MASTER_TOKEN --operation-num 124 --finance-login agency-login --dry-run
 direct v4finance check-payment --custom-transaction-id A123456789012345678901234567890B
-direct v4finance transfer-money --from-campaign-id 123 --to-campaign-id 456 --amount 100.50 --currency RUB --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
-direct v4finance pay-campaigns --campaign-ids 123,456 --amount 100.50 --currency RUB --contract-id CONTRACT_ID --pay-method Bank --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
+direct v4finance transfer-money --from-campaign-id 123 --to-campaign-id 456 --amount 100.50 --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
+direct v4finance pay-campaigns --campaign-ids 123,456 --amount 100.50 --contract-id CONTRACT_ID --pay-method Bank --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
 ```
 
 ### V4 Live Shared Account

{direct_cli-0.3.14 → direct_cli-0.3.15}/README.md

@@ -189,10 +189,10 @@ is omitted. Dry-run output masks the financial token.
 ```bash
 direct v4finance get-clients-units --logins client-login,other-client --format table
 direct v4finance get-credit-limits --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login
-direct v4finance create-invoice --payment 123=100.50 --payment 456=25 --currency RUB --master-token MASTER_TOKEN --operation-num 124 --finance-login agency-login --dry-run
+direct v4finance create-invoice --payment 123=100.50 --payment 456=25 --master-token MASTER_TOKEN --operation-num 124 --finance-login agency-login --dry-run
 direct v4finance check-payment --custom-transaction-id A123456789012345678901234567890B
-direct v4finance transfer-money --from-campaign-id 123 --to-campaign-id 456 --amount 100.50 --currency RUB --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
-direct v4finance pay-campaigns --campaign-ids 123,456 --amount 100.50 --currency RUB --contract-id CONTRACT_ID --pay-method Bank --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
+direct v4finance transfer-money --from-campaign-id 123 --to-campaign-id 456 --amount 100.50 --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
+direct v4finance pay-campaigns --campaign-ids 123,456 --amount 100.50 --contract-id CONTRACT_ID --pay-method Bank --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
 ```
 
 ### V4 Live Shared Account

{direct_cli-0.3.14 → direct_cli-0.3.15}/direct_cli/commands/v4finance.py

@@ -15,8 +15,7 @@ from .v4shells import V4_EPILOG
 
 FINANCE_TOKEN_MASK = "<redacted>"
 CUSTOM_TRANSACTION_ID_RE = re.compile(r"[A-Za-z0-9]{32}")
-V4_FINANCE_CURRENCIES = ["RUB", "USD", "EUR", "BYN", "KZT", "TRY", "UAH", "CHF"]
-V4_PAY_METHODS = ["Bank", "Overdraft"]
+V4_PAY_METHODS = ["Bank"]
 FINANCE_HELP_EPILOG = (
     "To issue a master token in the Yandex Direct UI, open Tools -> API -> "
     "Financial operations, enable the 'Allow financial operations' checkbox, "
@@ -34,14 +33,13 @@ def _logins_param(logins: str) -> list[str]:
     return login_list
 
 
-def _invoice_payments_param(payments: tuple[str, ...], currency: str) -> dict:
+def _invoice_payments_param(payments: tuple[str, ...]) -> dict:
     """Build the v4 Live CreateInvoice payment object parameter."""
     if not payments:
         raise click.UsageError("--payment is required")
 
     parsed_payments = []
     seen_campaign_ids = set()
-    normalized_currency = currency.upper()
     for payment in payments:
         spec = (payment or "").strip()
         if "=" not in spec:
@@ -64,7 +62,6 @@ def _invoice_payments_param(payments: tuple[str, ...], currency: str) -> dict:
             {
                 "CampaignID": campaign_id,
                 "Sum": parse_v4_money_sum(amount_text),
-                "Currency": normalized_currency,
             }
         )
 
@@ -333,13 +330,6 @@ def check_payment(
     required=True,
     help="Invoice payment as CAMPAIGN_ID=AMOUNT; repeat for multiple campaigns",
 )
-@click.option(
-    "--currency",
-    default="RUB",
-    show_default=True,
-    type=click.Choice(V4_FINANCE_CURRENCIES, case_sensitive=False),
-    help="Payment currency",
-)
 @click.option(
     "--finance-token",
     envvar="YANDEX_DIRECT_FINANCE_TOKEN",
@@ -377,7 +367,6 @@ def check_payment(
 def create_invoice(
     ctx,
     payments,
-    currency,
     finance_token,
     master_token,
     operation_num,
@@ -395,7 +384,7 @@ def create_invoice(
         "CreateInvoice",
         ctx.obj.get("login"),
     )
-    param = _invoice_payments_param(payments, currency)
+    param = _invoice_payments_param(payments)
 
     if dry_run:
         format_output(
@@ -438,13 +427,6 @@ def create_invoice(
     help="Destination campaign ID",
 )
 @click.option("--amount", required=True, help="Positive amount, for example 100.50")
-@click.option(
-    "--currency",
-    default="RUB",
-    show_default=True,
-    type=click.Choice(V4_FINANCE_CURRENCIES, case_sensitive=False),
-    help="Payment currency",
-)
 @click.option(
     "--finance-token",
     envvar="YANDEX_DIRECT_FINANCE_TOKEN",
@@ -480,7 +462,6 @@ def transfer_money(
     from_campaign_id,
     to_campaign_id,
     amount,
-    currency,
     finance_token,
     master_token,
     operation_num,
@@ -498,20 +479,17 @@ def transfer_money(
         ctx.obj.get("login"),
     )
     parsed_amount = parse_v4_money_sum(amount)
-    currency = currency.upper()
     param = {
         "FromCampaigns": [
             {
                 "CampaignID": from_campaign_id,
                 "Sum": parsed_amount,
-                "Currency": currency,
             },
         ],
         "ToCampaigns": [
             {
                 "CampaignID": to_campaign_id,
                 "Sum": parsed_amount,
-                "Currency": currency,
             },
         ],
     }
@@ -531,13 +509,6 @@ def transfer_money(
     help="Comma-separated campaign IDs to pay",
 )
 @click.option("--amount", required=True, help="Positive amount, for example 100.50")
-@click.option(
-    "--currency",
-    default="RUB",
-    show_default=True,
-    type=click.Choice(V4_FINANCE_CURRENCIES, case_sensitive=False),
-    help="Payment currency",
-)
 @click.option("--contract-id", help="Agency contract ID; required for Bank")
 @click.option(
     "--pay-method",
@@ -579,7 +550,6 @@ def pay_campaigns(
     ctx,
     campaign_ids,
     amount,
-    currency,
     contract_id,
     pay_method,
     finance_token,
@@ -600,14 +570,13 @@ def pay_campaigns(
     )
     parsed_amount = parse_v4_money_sum(amount)
     parsed_campaign_ids = _campaign_ids_param(campaign_ids)
-    currency = currency.upper()
     pay_method = _non_empty_option(pay_method, "--pay-method")
     contract_id = (contract_id or "").strip()
     if pay_method == "Bank" and not contract_id:
         raise click.UsageError("--contract-id is required when --pay-method Bank")
     param = {
         "Payments": [
-            {"CampaignID": campaign_id, "Sum": parsed_amount, "Currency": currency}
+            {"CampaignID": campaign_id, "Sum": parsed_amount}
             for campaign_id in parsed_campaign_ids
         ],
         "PayMethod": pay_method,

{direct_cli-0.3.14 → direct_cli-0.3.15}/direct_cli/v4_contracts.py

@@ -69,10 +69,10 @@ V4_METHOD_CONTRACTS: dict[str, V4MethodContract] = {
         live_probe_allowed=False,
         example_param=None,
         notes=(
-            "Official docs (dg-v4/reference/GetCreditLimits) define the "
-            "request body as method + finance_token + operation_num only; "
-            "no param. Live probe without finance credentials returns "
-            "error_code=350."
+            "Docs-verified 2026-05-28 against dg-v4/reference/GetCreditLimits: "
+            "request body is method + finance_token + operation_num only, "
+            "no param; CLI dry-run output matches exactly. Live probe "
+            "without finance credentials returns error_code=350."
         ),
     ),
     "TransferMoney": V4MethodContract(
@@ -87,13 +87,15 @@ V4_METHOD_CONTRACTS: dict[str, V4MethodContract] = {
         source_status=SOURCE_DOCS,
         live_probe_allowed=False,
         example_param={
-            "FromCampaigns": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
-            "ToCampaigns": [{"CampaignID": 456, "Sum": 100.5, "Currency": "RUB"}],
+            "FromCampaigns": [{"CampaignID": 123, "Sum": 100.5}],
+            "ToCampaigns": [{"CampaignID": 456, "Sum": 100.5}],
         },
         notes=(
-            "Official v4 docs define campaign-to-campaign transfer with "
-            "Currency on each transfer item. "
-            "This CLI exposes dry-run only; the method is not live-probed."
+            "Docs-verified 2026-05-28 against dg-v4/reference/TransferMoney: "
+            "PayCampElement carries only CampaignID and Sum (conventional "
+            "units, условные единицы). Total from-sum must equal total "
+            "to-sum or the API returns error_code=353. This CLI exposes "
+            "dry-run only; the method is not live-probed."
         ),
     ),
     "PayCampaigns": V4MethodContract(
@@ -108,14 +110,17 @@ V4_METHOD_CONTRACTS: dict[str, V4MethodContract] = {
         source_status=SOURCE_DOCS,
         live_probe_allowed=False,
         example_param={
-            "Payments": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
+            "Payments": [{"CampaignID": 123, "Sum": 100.5}],
             "ContractID": "contract-id",
             "PayMethod": "Bank",
         },
         notes=(
-            "Official v4 docs define agency credit-limit payment with "
-            "Currency on each payment item and PayMethod Bank/Overdraft. "
-            "This CLI exposes dry-run only; the method is not live-probed."
+            "Docs-verified 2026-05-28 against dg-v4/reference/PayCampaigns: "
+            "PayCampElement carries only CampaignID and Sum (conventional "
+            "units); PayMethod currently accepts 'Bank' only. The CLI "
+            "dropped --currency and the undocumented 'Overdraft' PayMethod "
+            "value in 0.3.15 (BREAKING) to mirror docs 1:1. This CLI "
+            "exposes dry-run only; the method is not live-probed."
         ),
     ),
     "PayCampaignsByCard": V4MethodContract(
@@ -143,7 +148,10 @@ V4_METHOD_CONTRACTS: dict[str, V4MethodContract] = {
             "Official public docs were not found. Sandbox v4 Live rejects "
             "PaymentID with error_code=71 and requires CustomTransactionID; "
             "a valid 32-character unknown CustomTransactionID reaches method "
-            "validation and returns error_code=370."
+            "validation and returns error_code=370. "
+            "Docs-verified 2026-05-28: method is absent from the v4 "
+            "_AllMethods index (dg-v4/ru/reference/_AllMethods); sandbox "
+            "behaviour remains the source of truth for the wire shape."
         ),
     ),
     "CreateInvoice": V4MethodContract(
@@ -158,12 +166,14 @@ V4_METHOD_CONTRACTS: dict[str, V4MethodContract] = {
         source_status=SOURCE_DOCS,
         live_probe_allowed=False,
         example_param={
-            "Payments": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
+            "Payments": [{"CampaignID": 123, "Sum": 100.5}],
         },
         notes=(
-            "Official v4 docs define invoice creation with Payments[]."
-            "CampaignID/Sum/Currency and a URL response. The method has no "
-            "v5 equivalent and requires finance_token plus operation_num."
+            "Docs-verified 2026-05-28 against dg-v4/reference/CreateInvoice: "
+            "PayCampElement carries only CampaignID and Sum (conventional "
+            "units). The CLI dropped --currency in 0.3.15 (BREAKING) to "
+            "mirror docs 1:1. The method has no v5 equivalent and requires "
+            "finance_token plus operation_num."
         ),
     ),
     "AccountManagement": V4MethodContract(

{direct_cli-0.3.14 → direct_cli-0.3.15}/direct_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: direct-cli
-Version: 0.3.14
+Version: 0.3.15
 Summary: Command-line interface for Yandex Direct API
 Author: axisrow
 License: MIT
@@ -232,10 +232,10 @@ is omitted. Dry-run output masks the financial token.
 ```bash
 direct v4finance get-clients-units --logins client-login,other-client --format table
 direct v4finance get-credit-limits --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login
-direct v4finance create-invoice --payment 123=100.50 --payment 456=25 --currency RUB --master-token MASTER_TOKEN --operation-num 124 --finance-login agency-login --dry-run
+direct v4finance create-invoice --payment 123=100.50 --payment 456=25 --master-token MASTER_TOKEN --operation-num 124 --finance-login agency-login --dry-run
 direct v4finance check-payment --custom-transaction-id A123456789012345678901234567890B
-direct v4finance transfer-money --from-campaign-id 123 --to-campaign-id 456 --amount 100.50 --currency RUB --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
-direct v4finance pay-campaigns --campaign-ids 123,456 --amount 100.50 --currency RUB --contract-id CONTRACT_ID --pay-method Bank --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
+direct v4finance transfer-money --from-campaign-id 123 --to-campaign-id 456 --amount 100.50 --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
+direct v4finance pay-campaigns --campaign-ids 123,456 --amount 100.50 --contract-id CONTRACT_ID --pay-method Bank --master-token MASTER_TOKEN --operation-num 123 --finance-login agency-login --dry-run
 ```
 
 ### V4 Live Shared Account

{direct_cli-0.3.14 → direct_cli-0.3.15}/direct_cli.egg-info/SOURCES.txt

@@ -96,6 +96,7 @@ scripts/check_all_docs_urls.py
 scripts/check_reports_drift.py
 scripts/check_wsdl_drift.py
 scripts/patch_vendor_imports.py
+scripts/preflight_check.sh
 scripts/refresh_reports_cache.py
 scripts/refresh_wsdl_cache.py
 scripts/release_pypi.sh

{direct_cli-0.3.14 → direct_cli-0.3.15}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "direct-cli"
-version = "0.3.14"
+version = "0.3.15"
 description = "Command-line interface for Yandex Direct API"
 readme = "README.md"
 license = {text = "MIT"}

direct_cli-0.3.15/scripts/preflight_check.sh

@@ -0,0 +1,62 @@
+#!/usr/bin/env bash
+#
+# Pre-release preflight check — runs the network-dependent docs / cache
+# health checks that used to be wired into release_pypi.sh.
+#
+# Run this manually before bumping the version. It is *not* wired into
+# release_pypi.sh on purpose: Yandex frequently rate-limits the docs host
+# with a SmartCaptcha gateway, which is an external rate-limit on our IP,
+# not evidence that an URL is gone. Mixing this into the release path made
+# releases non-deterministic.
+#
+# What this script does:
+#   1. scripts/check_all_docs_urls.py
+#      Hard-fails on canonical move (3xx → different path) or 404.
+#      Soft-warns on 5xx or persistent captcha (likely rate-limit).
+#   2. pytest TestReportsCoverage TestWsdlCacheFreshness
+#      Read-only: confirms committed cache files are real content, not
+#      captcha gateways.
+#   3. git diff --quiet -- tests/reports_cache tests/wsdl_cache
+#      Refuses to proceed with uncommitted cache changes; the maintainer
+#      must commit cache refreshes deliberately.
+#
+# Exit status: 0 if everything passes (warnings allowed), non-zero on any
+# hard failure.
+
+set -euo pipefail
+
+ROOT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+
+require_command() {
+  local name="$1"
+  if ! command -v "${name}" >/dev/null 2>&1; then
+    echo "Required command not found: ${name}" >&2
+    exit 1
+  fi
+}
+
+require_command python3
+require_command git
+
+cd "${ROOT_DIR}"
+
+echo "[1/3] Checking docs URLs..."
+python3 scripts/check_all_docs_urls.py
+
+echo
+echo "[2/3] Verifying committed docs/WSDL cache is real content..."
+python3 -m pytest tests/test_api_coverage.py::TestReportsCoverage \
+                  tests/test_api_coverage.py::TestWsdlCacheFreshness -v
+
+echo
+echo "[3/3] Checking that docs/WSDL cache has no uncommitted changes..."
+if ! git diff --quiet HEAD -- tests/reports_cache tests/wsdl_cache; then
+  echo "ERROR: tests/reports_cache or tests/wsdl_cache has uncommitted changes."
+  echo "       Run scripts/refresh_reports_cache.py separately, review the diff,"
+  echo "       and commit it before releasing."
+  git diff --stat -- tests/reports_cache tests/wsdl_cache
+  exit 1
+fi
+
+echo
+echo "Preflight passed. Safe to bump version and run scripts/release_pypi.sh."

{direct_cli-0.3.14 → direct_cli-0.3.15}/scripts/test_dangerous_commands.sh

@@ -21,7 +21,7 @@ Local credential mutations:
   - direct auth use ...
 
 V4 Live finance money mutations (0.3.3 exposes dry-run-only previews):
-  - direct v4finance transfer-money --from-campaign-id ... --to-campaign-id ... --amount ... --currency ... --dry-run
+  - direct v4finance transfer-money --from-campaign-id ... --to-campaign-id ... --amount ... --dry-run
   - direct v4finance pay-campaigns --campaign-ids ... --amount ... --contract-id ... --pay-method ... --dry-run
 
 Production commands that are only allowed in automated smoke tests with --sandbox:

{direct_cli-0.3.14 → direct_cli-0.3.15}/tests/API_COVERAGE.md

@@ -59,11 +59,12 @@ The public v4/Live4 finance docs found during milestone 0.3.3 list
 No official `CheckPayment` page or `PaymentID` contract was found.
 Financial methods use an additional `finance_token`, computed from the
 master token, operation number, method name, and normalized login. Public
-dry-run examples mask the computed token. The docs-backed `PayCampaigns` and
-`TransferMoney` contracts require `Currency` on each payment/transfer item.
-`CreateInvoice` is exposed as `v4finance create-invoice` with repeated
-`--payment CAMPAIGN_ID=AMOUNT` flags and sends docs-backed
-`Payments[].CampaignID/Sum/Currency`; its live write test is opt-in via
+dry-run examples mask the computed token. The docs-backed `PayCampaigns`,
+`TransferMoney`, and `CreateInvoice` `PayCampElement` carries only
+`CampaignID` and `Sum` (no `Currency`); `PayCampaigns.PayMethod` is
+`'Bank'` only. `CreateInvoice` is exposed as `v4finance create-invoice`
+with repeated `--payment CAMPAIGN_ID=AMOUNT` flags and sends docs-backed
+`Payments[].CampaignID/Sum`; its live write test is opt-in via
 `YANDEX_DIRECT_LIVE_FINANCE_WRITE=1`.
 
 Sandbox v4 Live probes on 2026-04-30 confirmed the runtime request shape for

{direct_cli-0.3.14 → direct_cli-0.3.15}/tests/test_v4_contracts.py

@@ -198,8 +198,8 @@ def test_v4finance_money_contracts_are_docs_backed_dangerous_objects():
     assert build_v4_body("TransferMoney", transfer.example_param) == {
         "method": "TransferMoney",
         "param": {
-            "FromCampaigns": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
-            "ToCampaigns": [{"CampaignID": 456, "Sum": 100.5, "Currency": "RUB"}],
+            "FromCampaigns": [{"CampaignID": 123, "Sum": 100.5}],
+            "ToCampaigns": [{"CampaignID": 456, "Sum": 100.5}],
         },
     }
 
@@ -209,7 +209,7 @@ def test_v4finance_money_contracts_are_docs_backed_dangerous_objects():
     assert build_v4_body("PayCampaigns", pay.example_param) == {
         "method": "PayCampaigns",
         "param": {
-            "Payments": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
+            "Payments": [{"CampaignID": 123, "Sum": 100.5}],
             "ContractID": "contract-id",
             "PayMethod": "Bank",
         },
@@ -223,7 +223,7 @@ def test_v4finance_money_contracts_are_docs_backed_dangerous_objects():
     assert build_v4_body("CreateInvoice", create_invoice.example_param) == {
         "method": "CreateInvoice",
         "param": {
-            "Payments": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
+            "Payments": [{"CampaignID": 123, "Sum": 100.5}],
         },
     }
 

{direct_cli-0.3.14 → direct_cli-0.3.15}/tests/test_v4finance_money.py

@@ -89,8 +89,8 @@ def test_transfer_money_dry_run_uses_campaign_arrays_and_masks_finance_token():
     assert json.loads(result.output) == {
         "method": "TransferMoney",
         "param": {
-            "FromCampaigns": [{"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"}],
-            "ToCampaigns": [{"CampaignID": 456, "Sum": 100.5, "Currency": "RUB"}],
+            "FromCampaigns": [{"CampaignID": 123, "Sum": 100.5}],
+            "ToCampaigns": [{"CampaignID": 456, "Sum": 100.5}],
         },
         "finance_token": "<redacted>",
         "operation_num": 42,
@@ -122,8 +122,8 @@ def test_pay_campaigns_dry_run_uses_payment_object_and_masks_finance_token():
         "method": "PayCampaigns",
         "param": {
             "Payments": [
-                {"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"},
-                {"CampaignID": 456, "Sum": 100.5, "Currency": "RUB"},
+                {"CampaignID": 123, "Sum": 100.5},
+                {"CampaignID": 456, "Sum": 100.5},
             ],
             "ContractID": "contract-id",
             "PayMethod": "Bank",
@@ -152,7 +152,7 @@ def test_create_invoice_dry_run_uses_payment_object_and_masks_finance_token():
         "method": "CreateInvoice",
         "param": {
             "Payments": [
-                {"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"},
+                {"CampaignID": 123, "Sum": 100.5},
             ],
         },
         "finance_token": "<redacted>",
@@ -160,7 +160,7 @@ def test_create_invoice_dry_run_uses_payment_object_and_masks_finance_token():
     }
 
 
-def test_create_invoice_dry_run_accepts_multiple_payments_and_currency():
+def test_create_invoice_dry_run_accepts_multiple_payments():
     result = _invoke(
         "v4finance",
         "create-invoice",
@@ -168,8 +168,6 @@ def test_create_invoice_dry_run_accepts_multiple_payments_and_currency():
         "123=100.50",
         "--payment",
         "456=1",
-        "--currency",
-        "usd",
         "--finance-token",
         "secret-finance-token",
         "--operation-num",
@@ -180,8 +178,8 @@ def test_create_invoice_dry_run_accepts_multiple_payments_and_currency():
     assert result.exit_code == 0
     assert json.loads(result.output)["param"] == {
         "Payments": [
-            {"CampaignID": 123, "Sum": 100.5, "Currency": "USD"},
-            {"CampaignID": 456, "Sum": 1.0, "Currency": "USD"},
+            {"CampaignID": 123, "Sum": 100.5},
+            {"CampaignID": 456, "Sum": 1.0},
         ],
     }
 
@@ -485,8 +483,10 @@ def test_pay_campaigns_rejects_non_positive_campaign_ids():
         "0,-1",
         "--amount",
         "100.50",
+        "--contract-id",
+        "contract-id",
         "--pay-method",
-        "Overdraft",
+        "Bank",
         "--finance-token",
         "finance-token",
         "--operation-num",
@@ -498,7 +498,7 @@ def test_pay_campaigns_rejects_non_positive_campaign_ids():
     assert "--campaign-ids must contain only positive integers" in result.output
 
 
-def test_pay_campaigns_allows_overdraft_without_contract():
+def test_pay_campaigns_rejects_undocumented_pay_method():
     result = _invoke(
         "v4finance",
         "pay-campaigns",
@@ -508,8 +508,6 @@ def test_pay_campaigns_allows_overdraft_without_contract():
         "100.50",
         "--pay-method",
         "Overdraft",
-        "--currency",
-        "usd",
         "--finance-token",
         "finance-token",
         "--operation-num",
@@ -517,11 +515,8 @@ def test_pay_campaigns_allows_overdraft_without_contract():
         "--dry-run",
     )
 
-    assert result.exit_code == 0
-    assert json.loads(result.output)["param"] == {
-        "Payments": [{"CampaignID": 123, "Sum": 100.5, "Currency": "USD"}],
-        "PayMethod": "Overdraft",
-    }
+    assert result.exit_code != 0
+    assert "Invalid value for '--pay-method'" in result.output
 
 
 def test_check_payment_dry_run_uses_custom_transaction_id_object():
@@ -636,7 +631,7 @@ def test_create_invoice_formats_mocked_response_as_json():
         "CreateInvoice",
         {
             "Payments": [
-                {"CampaignID": 123, "Sum": 100.5, "Currency": "RUB"},
+                {"CampaignID": 123, "Sum": 100.5},
             ],
         },
     )