ynab-cli 0.6.2__tar.gz → 0.6.3__tar.gz

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## v0.6.3 (2025-07-11)
+
+### Refactor
+
+- clean up the dry_run parameters
+- clean up debug option
+
 ## v0.6.2 (2025-07-09)
 
 ### Fix

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ynab-cli
-Version: 0.6.2
+Version: 0.6.3
 Summary: YNAB Command Line Interface
 Project-URL: Changelog, https://github.com/jeffsawatzky/ynab-cli/blob/main/CHANGELOG.md
 Project-URL: Documentation, https://jeffsawatzky.github.io/ynab-cli/

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ynab-cli"
-version = "0.6.2"
+version = "0.6.3"
 description = "YNAB Command Line Interface"
 readme = "README.md"
 requires-python = ">=3.11"

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/domain/settings.py

@@ -10,7 +10,4 @@ class YnabSettings:
 
 @_attrs_define
 class Settings:
-    dry_run: bool = False
-    debug: bool = False
-
     ynab: YnabSettings = field(factory=YnabSettings)

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/domain/use_cases/payees.py

@@ -41,7 +41,7 @@ def _normalize_name(name: str) -> str:
 
 
 class NormalizeNamesParams(TypedDict):
-    pass
+    dry_run: bool
 
 
 class NormalizeNames:
@@ -76,7 +76,7 @@ class NormalizeNames:
                 if normalized_name != payee.name:
                     yield (payee, normalized_name)
 
-                    if not settings.dry_run:
+                    if not params["dry_run"]:
                         await util.run_asyncio_detailed(
                             self._io,
                             update_payee.asyncio_detailed,
@@ -166,6 +166,7 @@ class ListDuplicates:
 
 
 class ListUnusedParams(TypedDict):
+    dry_run: bool
     prefix_unused: bool
 
 
@@ -208,18 +209,20 @@ class ListUnused:
 
                 # List unused payee if no transactions
                 if not num_transactions:
+                    if params["prefix_unused"]:
+                        payee.name = f"{UNUSED_PREFIX} {payee.name}"
+
                     yield payee
 
                     # If prefix_unused is True, rename the payee
-                    if not settings.dry_run and params.get("prefix_unused", False):
-                        new_name = f"{UNUSED_PREFIX} {payee.name}"
+                    if not params["dry_run"] and params["prefix_unused"]:
                         await util.run_asyncio_detailed(
                             self._io,
                             update_payee.asyncio_detailed,
                             settings.ynab.budget_id,
                             str(payee.id),
                             client=self._client,
-                            body=models.PatchPayeeWrapper(payee=models.SavePayee(name=new_name)),
+                            body=models.PatchPayeeWrapper(payee=models.SavePayee(name=payee.name)),
                         )
 
         except Exception as e:

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/domain/use_cases/transactions.py

@@ -39,6 +39,7 @@ def _get_save_transaction(
 
 
 class ApplyRulesParams(TypedDict):
+    dry_run: bool
     transaction_rules: rules.TransactionRules
 
 
@@ -82,7 +83,7 @@ class ApplyRules:
                 if save_transaction:
                     yield (transaction, save_transaction)
 
-                    if not settings.dry_run:
+                    if not params["dry_run"]:
                         save_transactions.append(save_transaction)
 
             if save_transactions:

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/cli.py

@@ -3,24 +3,23 @@ from click_default_group import DefaultGroup
 
 from ynab_cli.domain.settings import Settings
 from ynab_cli.host.click.cli import run
-from ynab_cli.host.constants import CONTEXT_KEY_SETTINGS, ENV_PREFIX
+from ynab_cli.host.constants import CONTEXT_KEY_DEBUG, CONTEXT_KEY_SETTINGS, ENV_PREFIX
 from ynab_cli.host.textual.cli import tui
 
 
 @click.group(cls=DefaultGroup, default="tui", default_if_no_args=True)
-@click.option("--dry-run", is_flag=True, default=False, help="Run without making any changes.")
 @click.option("--debug", is_flag=True, default=False, help="Enable debug mode for more verbose output.")
 @click.version_option()
 @click.pass_context
 def cli(
     ctx: click.Context,
-    dry_run: bool,
     debug: bool,
 ) -> None:
     """Main entrypoint."""
 
     ctx.ensure_object(dict)
-    ctx.obj[CONTEXT_KEY_SETTINGS] = Settings(dry_run=dry_run, debug=debug)
+    ctx.obj[CONTEXT_KEY_DEBUG] = debug
+    ctx.obj[CONTEXT_KEY_SETTINGS] = Settings()
 
 
 cli.add_command(run)

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/click/cli.py

@@ -5,7 +5,7 @@ from rich.logging import RichHandler
 
 from ynab_cli.domain.settings import Settings
 from ynab_cli.host.click.commands import budgets, categories, payees, transactions
-from ynab_cli.host.constants import CONTEXT_KEY_SETTINGS, ENV_PREFIX
+from ynab_cli.host.constants import CONTEXT_KEY_DEBUG, CONTEXT_KEY_SETTINGS, ENV_PREFIX
 
 
 @click.group()
@@ -25,12 +25,13 @@ def run(
     """Main entrypoint for YNAB CLI commands."""
 
     ctx.ensure_object(dict)
+    debug: bool = ctx.obj.get(CONTEXT_KEY_DEBUG, False)
     settings: Settings = ctx.obj.get(CONTEXT_KEY_SETTINGS, Settings())
     settings.ynab.access_token = access_token
     ctx.obj[CONTEXT_KEY_SETTINGS] = settings
 
     logging.basicConfig(
-        level="DEBUG" if settings.debug else "WARNING",  # NOTSET, DEBUG, INFO, WARNING, ERROR, CRITICAL
+        level="DEBUG" if debug else "WARNING",  # NOTSET, DEBUG, INFO, WARNING, ERROR, CRITICAL
         format="%(message)s",
         datefmt="[%X]",
         handlers=[RichHandler()],

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/click/commands/payees.py

@@ -19,8 +19,10 @@ class NormalizeNamesCommand:
         self._progress_table.table.add_column("Payee Name")
         self._progress_table.table.add_column("Normalized Name")
 
-    async def __call__(self, settings: Settings) -> None:
-        params: use_cases.NormalizeNamesParams = {}
+    async def __call__(self, settings: Settings, dry_run: bool) -> None:
+        params: use_cases.NormalizeNamesParams = {
+            "dry_run": dry_run,
+        }
 
         console = None
         with self._progress_table:
@@ -76,8 +78,9 @@ class ListUnusedCommand:
         self._progress_table.table.add_column("Payee Id")
         self._progress_table.table.add_column("Payee Name")
 
-    async def __call__(self, settings: Settings, prefix_unused: bool) -> None:
+    async def __call__(self, settings: Settings, dry_run: bool, prefix_unused: bool) -> None:
         params: use_cases.ListUnusedParams = {
+            "dry_run": dry_run,
             "prefix_unused": prefix_unused,
         }
 
@@ -122,13 +125,14 @@ class ListAllCommand:
 
 
 @containerize
-async def _normalize_names(container: Container) -> None:
-    await container[NormalizeNamesCommand](container[Settings])
+async def _normalize_names(container: Container, dry_run: bool) -> None:
+    await container[NormalizeNamesCommand](container[Settings], dry_run)
 
 
 @click.command()
+@click.option("--dry-run", is_flag=True, default=False, help="Run without making any changes.")
 @click.pass_context
-def normalize_names(ctx: click.Context) -> None:
+def normalize_names(ctx: click.Context, dry_run: bool) -> None:
     """Normalize payee names in the YNAB budget."""
 
     ctx.ensure_object(dict)
@@ -138,6 +142,7 @@ def normalize_names(ctx: click.Context) -> None:
     anyio.run(
         _normalize_names,
         settings,
+        dry_run,
         backend_options={"use_uvloop": True},
     )
 
@@ -164,14 +169,15 @@ def list_duplicates(ctx: click.Context) -> None:
 
 
 @containerize
-async def _list_unused(container: Container, prefix_unused: bool) -> None:
-    await container[ListUnusedCommand](container[Settings], prefix_unused)
+async def _list_unused(container: Container, dry_run: bool, prefix_unused: bool) -> None:
+    await container[ListUnusedCommand](container[Settings], dry_run, prefix_unused)
 
 
 @click.command()
+@click.option("--dry-run", is_flag=True, default=False, help="Run without making any changes.")
 @click.option("--prefix-unused", is_flag=True, default=False, help="Add a prefix to the unused payee names.")
 @click.pass_context
-def list_unused(ctx: click.Context, prefix_unused: bool) -> None:
+def list_unused(ctx: click.Context, dry_run: bool, prefix_unused: bool) -> None:
     """List unused payees in the YNAB budget."""
 
     ctx.ensure_object(dict)
@@ -181,6 +187,7 @@ def list_unused(ctx: click.Context, prefix_unused: bool) -> None:
     anyio.run(
         _list_unused,
         settings,
+        dry_run,
         prefix_unused,
         backend_options={"use_uvloop": True},
     )

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/click/commands/transactions.py

@@ -27,8 +27,9 @@ class ApplyRulesCommand:
         self._progress_table.table.add_column("Transaction Amount")
         self._progress_table.table.add_column("Transaction Changes")
 
-    async def __call__(self, settings: Settings, transaction_rules: rules.TransactionRules) -> None:
+    async def __call__(self, settings: Settings, dry_run: bool, transaction_rules: rules.TransactionRules) -> None:
         params: use_cases.ApplyRulesParams = {
+            "dry_run": dry_run,
             "transaction_rules": transaction_rules,
         }
 
@@ -52,14 +53,15 @@ class ApplyRulesCommand:
 
 
 @containerize
-async def _apply_rules(container: Container, transaction_rules: rules.TransactionRules) -> None:
-    await container[ApplyRulesCommand](container[Settings], transaction_rules)
+async def _apply_rules(container: Container, dry_run: bool, transaction_rules: rules.TransactionRules) -> None:
+    await container[ApplyRulesCommand](container[Settings], dry_run, transaction_rules)
 
 
 @click.command()
+@click.option("--dry-run", is_flag=True, default=False, help="Run without making any changes.")
 @click.argument("rules-file", type=click.File())
 @click.pass_context
-def apply_rules(ctx: click.Context, rules_file: IO[Any]) -> None:
+def apply_rules(ctx: click.Context, dry_run: bool, rules_file: IO[Any]) -> None:
     """Apply transaction rules from a JSON RULES_FILE to transactions in the YNAB budget.
 
     RULES_FILE should be a JSON file containing transaction rules.
@@ -74,6 +76,7 @@ def apply_rules(ctx: click.Context, rules_file: IO[Any]) -> None:
     anyio.run(
         _apply_rules,
         settings,
+        dry_run,
         transaction_rules,
         backend_options={"use_uvloop": True},
     )

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/constants.py

@@ -1,3 +1,4 @@
 ENV_PREFIX = "YNAB_CLI"
 
+CONTEXT_KEY_DEBUG = "DEBUG"
 CONTEXT_KEY_SETTINGS = "SETTINGS"

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/textual/app.py

@@ -5,7 +5,7 @@ from textual.app import App, ComposeResult
 from textual.binding import Binding, BindingType
 from textual.containers import Horizontal, Vertical
 from textual.reactive import reactive
-from textual.widgets import Checkbox, Footer, Header, Input, Label, TabbedContent
+from textual.widgets import Footer, Header, Input, Label, TabbedContent
 from typing_extensions import override
 
 from ynab_cli.adapters.ynab.client import AuthenticatedClient
@@ -29,25 +29,18 @@ class SettingsDialogForm(DialogForm[Settings]):
             with Vertical():
                 yield Label("Budget Id")
                 yield Input(self._settings.ynab.budget_id, placeholder="Budget Id", id="budget_id")
-        with Horizontal():
-            yield Checkbox("Debug", self._settings.debug, id="debug")
-            yield Checkbox("Dry Run", self._settings.dry_run, id="dry_run")
 
     @override
     async def get_result(self) -> Settings:
         """Get the result from the dialog form."""
         access_token = self.query_one("#access_token", Input).value.strip()
         budget_id = self.query_one("#budget_id", Input).value.strip()
-        debug = self.query_one("#debug", Checkbox).value
-        dry_run = self.query_one("#dry_run", Checkbox).value
 
         return Settings(
             ynab=YnabSettings(
                 access_token=access_token,
                 budget_id=budget_id,
-            ),
-            debug=debug,
-            dry_run=dry_run,
+            )
         )
 
 

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/textual/cli.py

@@ -7,7 +7,7 @@ from textual.logging import TextualHandler
 
 from ynab_cli.adapters.ynab.client import AuthenticatedClient
 from ynab_cli.domain.settings import Settings, YnabSettings
-from ynab_cli.host.constants import CONTEXT_KEY_SETTINGS, ENV_PREFIX
+from ynab_cli.host.constants import CONTEXT_KEY_DEBUG, CONTEXT_KEY_SETTINGS, ENV_PREFIX
 from ynab_cli.host.textual.app import YnabCliApp
 from ynab_cli.host.textual.container import containerize
 
@@ -32,11 +32,12 @@ def tui(
     """Run the Textual User Interface (TUI) for YNAB CLI."""
 
     ctx.ensure_object(dict)
+    debug: bool = ctx.obj.get(CONTEXT_KEY_DEBUG, False)
     settings: Settings = ctx.obj.get(CONTEXT_KEY_SETTINGS, Settings())
     settings.ynab = YnabSettings(access_token=access_token, budget_id=budget_id)
 
     logging.basicConfig(
-        level="DEBUG" if settings.debug else "WARNING",  # NOTSET, DEBUG, INFO, WARNING, ERROR, CRITICAL
+        level="DEBUG" if debug else "WARNING",  # NOTSET, DEBUG, INFO, WARNING, ERROR, CRITICAL
         format="%(message)s",
         datefmt="[%X]",
         handlers=[TextualHandler()],

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/textual/widgets/payees/list_unused.py

@@ -20,12 +20,14 @@ class ListUnusedParamsDialogForm(DialogForm[use_cases.ListUnusedParams]):
 
     @override
     def compose(self) -> ComposeResult:
+        yield Checkbox("Dry Run", self._params["dry_run"], id="dry_run")
         yield Checkbox("Prefix Unused Payees", self._params["prefix_unused"], id="prefix_unused")
 
     @override
     async def get_result(self) -> use_cases.ListUnusedParams:
         return {
-            "prefix_unused": self.query_one(Checkbox).value,
+            "dry_run": self.query_one("#dry_run", Checkbox).value,
+            "prefix_unused": self.query_one("#prefix_unused", Checkbox).value,
         }
 
 
@@ -33,7 +35,8 @@ class ListUnusedCommand(CommandWidget):
     def __init__(self) -> None:
         super().__init__()
         self._params: use_cases.ListUnusedParams = {
-            "prefix_unused": False,  # Default to not prefixing unused payees
+            "dry_run": False,
+            "prefix_unused": False,
         }
 
     def on_mount(self) -> None:

ynab_cli-0.6.3/src/ynab_cli/host/textual/widgets/payees/normalize_names.py

@@ -0,0 +1,74 @@
+from typing import TYPE_CHECKING, Any, cast
+
+from textual.app import ComposeResult
+from textual.widgets import Checkbox, DataTable, Log, ProgressBar
+from typing_extensions import override
+
+from ynab_cli.adapters.textual.io import TextualIO
+from ynab_cli.domain.use_cases import payees as use_cases
+from ynab_cli.host.textual.widgets.common.command_widget import CommandWidget
+from ynab_cli.host.textual.widgets.common.dialogs import CANCELLED, DialogForm, SaveCancelDialogScreen
+
+if TYPE_CHECKING:
+    from ynab_cli.host.textual.app import YnabCliApp
+
+
+class NormalizeNamesParamsDialogForm(DialogForm[use_cases.NormalizeNamesParams]):
+    def __init__(self, params: use_cases.NormalizeNamesParams, *args: Any, **kwargs: Any) -> None:
+        super().__init__(*args, **kwargs)
+        self._params: use_cases.NormalizeNamesParams = {**params}
+
+    @override
+    def compose(self) -> ComposeResult:
+        yield Checkbox("Dry Run", self._params["dry_run"], id="dry_run")
+
+    @override
+    async def get_result(self) -> use_cases.NormalizeNamesParams:
+        return {
+            "dry_run": self.query_one("#dry_run", Checkbox).value,
+        }
+
+
+class NormalizeNamesCommand(CommandWidget):
+    def __init__(self) -> None:
+        super().__init__()
+        self._params: use_cases.NormalizeNamesParams = {
+            "dry_run": False,
+        }
+
+    def on_mount(self) -> None:
+        table = self.query_one(DataTable)
+        table.add_columns(
+            "Payee Id",
+            "Payee Name",
+            "Normalized Name",
+        )
+
+    @override
+    async def _get_command_params(self) -> None:
+        result = await self.app.push_screen_wait(
+            SaveCancelDialogScreen(
+                NormalizeNamesParamsDialogForm(self._params), title="Payees: Normalize Names Parameters"
+            )
+        )
+        if result is not CANCELLED:
+            self._params = result
+
+    @override
+    async def _run_command(self) -> None:
+        params: use_cases.NormalizeNamesParams = {
+            "dry_run": False,
+        }
+
+        progress_bar = self.query_one(ProgressBar)
+        table = self.query_one(DataTable)
+        log = self.query_one(Log)
+
+        async for payee, normalized_name in use_cases.NormalizeNames(
+            TextualIO(self.app, log, progress_bar), cast("YnabCliApp", self.app).client
+        )(self.settings, params):
+            table.add_row(
+                payee.id,
+                payee.name,
+                normalized_name,
+            )

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/textual/widgets/transactions/apply_rules.py

@@ -4,7 +4,7 @@ from typing import TYPE_CHECKING, Any, cast
 from textual import on, work
 from textual.app import ComposeResult
 from textual.containers import ScrollableContainer
-from textual.widgets import Button, DataTable, Log, Pretty, ProgressBar
+from textual.widgets import Button, Checkbox, DataTable, Log, Pretty, ProgressBar
 from textual_fspicker import FileOpen
 from typing_extensions import override
 
@@ -25,6 +25,7 @@ class ApplyRulesParamsDialogForm(DialogForm[use_cases.ApplyRulesParams]):
 
     @override
     def compose(self) -> ComposeResult:
+        yield Checkbox("Dry Run", self._params["dry_run"], id="dry_run")
         yield Button("Select Transaction Rules JSON File", id="select_rules", variant="primary")
         with ScrollableContainer():
             yield Pretty(self._params["transaction_rules"].to_dict())
@@ -52,6 +53,7 @@ class ApplyRulesParamsDialogForm(DialogForm[use_cases.ApplyRulesParams]):
     @override
     async def get_result(self) -> use_cases.ApplyRulesParams:
         return {
+            "dry_run": self.query_one("#dry_run", Checkbox).value,
             "transaction_rules": self._params["transaction_rules"],
         }
 
@@ -60,6 +62,7 @@ class ApplyRulesCommand(CommandWidget):
     def __init__(self) -> None:
         super().__init__()
         self._params: use_cases.ApplyRulesParams = {
+            "dry_run": False,
             "transaction_rules": rules.TransactionRules(transaction_rules=[]),
         }
 

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/src/ynab_cli/host/textual/widgets/transactions/tabs.py

@@ -16,13 +16,13 @@ class TransactionsTabs(Widget):
     @override
     def compose(self) -> ComposeResult:
         with TabbedContent():
-            with TabPane("Transactions", id="transactions"):
+            with TabPane("Apply Rules", id="apply_rules"):
                 yield ApplyRulesCommand().data_bind(settings=TransactionsTabs.settings)
 
     def active_command(self) -> CommandWidget:
         tabbed_content = self.query_one(TabbedContent)
         if tabbed_content.active_pane:
-            if tabbed_content.active_pane.id == "transactions":
+            if tabbed_content.active_pane.id == "apply_rules":
                 return self.query_one(ApplyRulesCommand)
 
         raise ValueError("No active command found in TransactionsTabs.")

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/tests/unit/domain/use_cases/test_payees.py

@@ -131,7 +131,9 @@ async def test_normalize_names(mocker: MockerFixture, mock_io: MagicMock, empty_
     mock_update_payee.asyncio_detailed = AsyncMock()
 
     settings = Settings()
-    params: use_cases.NormalizeNamesParams = {}
+    params: use_cases.NormalizeNamesParams = {
+        "dry_run": False,
+    }
 
     results: list[tuple[models.Payee, str]] = []
     async for result in use_cases.NormalizeNames(mock_io, MagicMock())(settings, params):
@@ -163,7 +165,9 @@ async def test_normalize_names_exception(
     mock_get_payees.asyncio_detailed.side_effect = exception
 
     settings = Settings()
-    params: use_cases.NormalizeNamesParams = {}
+    params: use_cases.NormalizeNamesParams = {
+        "dry_run": False,
+    }
 
     async for _ in use_cases.NormalizeNames(mock_io, MagicMock())(settings, params):
         pass
@@ -268,13 +272,14 @@ async def test_list_unused(mocker: MockerFixture, mock_io: MagicMock, empty_uuid
 
     settings = Settings()
     params: use_cases.ListUnusedParams = {
+        "dry_run": False,
         "prefix_unused": True,
     }
 
     results: list[models.Payee] = []
     async for result in use_cases.ListUnused(mock_io, MagicMock())(settings, params):
         assert isinstance(result, models.Payee)
-        assert result.name in ["Payee 1"]
+        assert result.name in ["[UNUSED] Payee 1"]
         results.append(result)
 
     assert len(results) == 1
@@ -299,6 +304,7 @@ async def test_list_unused_exception(
 
     settings = Settings()
     params: use_cases.ListUnusedParams = {
+        "dry_run": False,
         "prefix_unused": True,
     }
 

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/tests/unit/host/click/commands/test_payees.py

@@ -33,7 +33,7 @@ def test_normalize_names(runner: CliRunner, container: Container, empty_uuid: UU
     assert result.exit_code == 0
 
     use_case.assert_called_once_with(
-        Settings(ynab=YnabSettings(access_token="test_token", budget_id="test_budget")), {}
+        Settings(ynab=YnabSettings(access_token="test_token", budget_id="test_budget")), {"dry_run": False}
     )
 
 
@@ -82,7 +82,8 @@ def test_list_unused(runner: CliRunner, container: Container, empty_uuid: UUID)
 
     assert result.exit_code == 0
     use_case.assert_called_once_with(
-        Settings(ynab=YnabSettings(access_token="test_token", budget_id="test_budget")), {"prefix_unused": False}
+        Settings(ynab=YnabSettings(access_token="test_token", budget_id="test_budget")),
+        {"dry_run": False, "prefix_unused": False},
     )
 
 

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/tests/unit/host/click/commands/test_transactions.py

@@ -61,5 +61,5 @@ def test_apply_rules(
 
     use_case.assert_called_once_with(
         Settings(ynab=YnabSettings(access_token="test_token", budget_id="test_budget")),
-        {"transaction_rules": transaction_rules},
+        {"dry_run": False, "transaction_rules": transaction_rules},
     )

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/tests/unit/host/test_cli.py

@@ -22,7 +22,6 @@ class TestCli:
 
         assert result.exit_code == 0
         assert "Main entrypoint." in result.output
-        assert "--dry-run" in result.output
         assert "--debug" in result.output
 
 

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/tests/unit/host/textual/test_app.py

@@ -26,7 +26,6 @@ async def test_settings_dialog_form_get_result(monkeypatch: pytest.MonkeyPatch)
             ("#access_token", Input): mock.Mock(spec=Input, value="newtoken "),
             ("#budget_id", Input): mock.Mock(spec=Input, value=" newbudget"),
             ("#debug", Checkbox): mock.Mock(spec=Checkbox, value=True),
-            ("#dry_run", Checkbox): mock.Mock(spec=Checkbox, value=True),
         }[(selector, cls)],
     )
 
@@ -34,8 +33,6 @@ async def test_settings_dialog_form_get_result(monkeypatch: pytest.MonkeyPatch)
     assert isinstance(result, Settings)
     assert result.ynab.access_token == "newtoken"
     assert result.ynab.budget_id == "newbudget"
-    assert result.debug is True
-    assert result.dry_run is True
 
 
 def test_ynab_cli_app_initialization() -> None:

{ynab_cli-0.6.2 → ynab_cli-0.6.3}/uv.lock

@@ -1711,7 +1711,7 @@ wheels = [
 
 [[package]]
 name = "ynab-cli"
-version = "0.6.2"
+version = "0.6.3"
 source = { editable = "." }
 dependencies = [
     { name = "anyio" },

ynab_cli-0.6.2/src/ynab_cli/host/textual/widgets/payees/normalize_names.py

@@ -1,38 +0,0 @@
-from typing import TYPE_CHECKING, cast
-
-from textual.widgets import DataTable, Log, ProgressBar
-from typing_extensions import override
-
-from ynab_cli.adapters.textual.io import TextualIO
-from ynab_cli.domain.use_cases import payees as use_cases
-from ynab_cli.host.textual.widgets.common.command_widget import CommandWidget
-
-if TYPE_CHECKING:
-    from ynab_cli.host.textual.app import YnabCliApp
-
-
-class NormalizeNamesCommand(CommandWidget):
-    def on_mount(self) -> None:
-        table = self.query_one(DataTable)
-        table.add_columns(
-            "Payee Id",
-            "Payee Name",
-            "Normalized Name",
-        )
-
-    @override
-    async def _run_command(self) -> None:
-        params: use_cases.NormalizeNamesParams = {}
-
-        progress_bar = self.query_one(ProgressBar)
-        table = self.query_one(DataTable)
-        log = self.query_one(Log)
-
-        async for payee, normalized_name in use_cases.NormalizeNames(
-            TextualIO(self.app, log, progress_bar), cast("YnabCliApp", self.app).client
-        )(self.settings, params):
-            table.add_row(
-                payee.id,
-                payee.name,
-                normalized_name,
-            )