plain.models 0.35.0__py3-none-any.whl → 0.36.0__py3-none-any.whl

plain/models/CHANGELOG.md

@@ -1,5 +1,16 @@
 # plain-models changelog
 
+## [0.36.0](https://github.com/dropseed/plain/releases/plain-models@0.36.0) (2025-07-18)
+
+### What's changed
+
+- Removed the `--merge` option from the `makemigrations` command ([d366663](https://github.com/dropseed/plain/commit/d366663))
+- Improved error handling in the `restore-backup` command using Click's error system ([88f06c5](https://github.com/dropseed/plain/commit/88f06c5))
+
+### Upgrade instructions
+
+- No changes required
+
 ## [0.35.0](https://github.com/dropseed/plain/releases/plain-models@0.35.0) (2025-07-07)
 
 ### What's changed

plain/models/backups/cli.py

@@ -66,12 +66,10 @@ def restore_backup(backup_name, latest, pg_restore):
     backups_handler = DatabaseBackups()
 
     if backup_name and latest:
-        click.secho("Only one of --latest or backup_name is allowed", fg="red")
-        exit(1)
+        raise click.UsageError("Only one of --latest or backup_name is allowed")
 
     if not backup_name and not latest:
-        click.secho("Backup name or --latest is required", fg="red")
-        exit(1)
+        raise click.UsageError("Backup name or --latest is required")
 
     if not backup_name and latest:
         backup_name = backups_handler.find_backups()[0].name

plain/models/cli.py

@@ -2,7 +2,6 @@ import os
 import subprocess
 import sys
 import time
-from itertools import takewhile
 
 import click
 
@@ -22,12 +21,10 @@ from .migrations.migration import Migration, SettingsTuple
 from .migrations.optimizer import MigrationOptimizer
 from .migrations.questioner import (
     InteractiveMigrationQuestioner,
-    MigrationQuestioner,
     NonInteractiveMigrationQuestioner,
 )
 from .migrations.recorder import MigrationRecorder
 from .migrations.state import ModelState, ProjectState
-from .migrations.utils import get_migration_name_timestamp
 from .migrations.writer import MigrationWriter
 from .registry import models_registry
 
@@ -92,7 +89,7 @@ def db_wait():
                 )
             time.sleep(1.5)
         else:
-            click.secho("Database ready", fg="green")
+            click.secho("✔ Database ready", fg="green")
             break
 
 
@@ -135,7 +132,6 @@ def list_models(package_labels, app_only):
     is_flag=True,
     help="Just show what migrations would be made; don't actually write them.",
 )
-@click.option("--merge", is_flag=True, help="Enable fixing of migration conflicts.")
 @click.option("--empty", is_flag=True, help="Create an empty migration.")
 @click.option(
     "--noinput",
@@ -157,9 +153,7 @@ def list_models(package_labels, app_only):
     default=1,
     help="Verbosity level; 0=minimal output, 1=normal output, 2=verbose output, 3=very verbose output",
 )
-def makemigrations(
-    package_labels, dry_run, merge, empty, no_input, name, check, verbosity
-):
+def makemigrations(package_labels, dry_run, empty, no_input, name, check, verbosity):
     """Creates new migration(s) for packages."""
 
     written_files = []
@@ -226,103 +220,6 @@ def makemigrations(
                     )
                     log(writer.as_string(), level=3)
 
-    def handle_merge(loader, conflicts):
-        """Handle merging conflicting migrations."""
-        if interactive:
-            questioner = InteractiveMigrationQuestioner()
-        else:
-            questioner = MigrationQuestioner(defaults={"ask_merge": True})
-
-        for package_label, migration_names in conflicts.items():
-            log(click.style(f"Merging {package_label}", fg="cyan", bold=True), level=1)
-
-            merge_migrations = []
-            for migration_name in migration_names:
-                migration = loader.get_migration(package_label, migration_name)
-                migration.ancestry = [
-                    mig
-                    for mig in loader.graph.forwards_plan(
-                        (package_label, migration_name)
-                    )
-                    if mig[0] == migration.package_label
-                ]
-                merge_migrations.append(migration)
-
-            def all_items_equal(seq):
-                return all(item == seq[0] for item in seq[1:])
-
-            merge_migrations_generations = zip(*(m.ancestry for m in merge_migrations))
-            common_ancestor_count = sum(
-                1 for _ in takewhile(all_items_equal, merge_migrations_generations)
-            )
-            if not common_ancestor_count:
-                raise ValueError(f"Could not find common ancestor of {migration_names}")
-
-            for migration in merge_migrations:
-                migration.branch = migration.ancestry[common_ancestor_count:]
-                migrations_ops = (
-                    loader.get_migration(node_package, node_name).operations
-                    for node_package, node_name in migration.branch
-                )
-                migration.merged_operations = sum(migrations_ops, [])
-
-            for migration in merge_migrations:
-                log(click.style(f"  Branch {migration.name}", fg="yellow"), level=1)
-                for operation in migration.merged_operations:
-                    log(f"    - {operation.describe()}", level=1)
-
-            if questioner.ask_merge(package_label):
-                numbers = [
-                    MigrationAutodetector.parse_number(migration.name)
-                    for migration in merge_migrations
-                ]
-                biggest_number = (
-                    max(x for x in numbers if x is not None) if numbers else 0
-                )
-
-                subclass = type(
-                    "Migration",
-                    (Migration,),
-                    {
-                        "dependencies": [
-                            (package_label, migration.name)
-                            for migration in merge_migrations
-                        ],
-                    },
-                )
-
-                parts = [f"{biggest_number + 1:04d}"]
-                if migration_name:
-                    parts.append(migration_name)
-                else:
-                    parts.append("merge")
-                    leaf_names = "_".join(
-                        sorted(migration.name for migration in merge_migrations)
-                    )
-                    if len(leaf_names) > 47:
-                        parts.append(get_migration_name_timestamp())
-                    else:
-                        parts.append(leaf_names)
-
-                new_migration_name = "_".join(parts)
-                new_migration = subclass(new_migration_name, package_label)
-                writer = MigrationWriter(new_migration)
-
-                if not dry_run:
-                    with open(writer.path, "w", encoding="utf-8") as fh:
-                        fh.write(writer.as_string())
-                    log(f"\nCreated new merge migration {writer.path}", level=1)
-                elif verbosity == 3:
-                    log(
-                        click.style(
-                            f"Full merge migrations file '{writer.filename}':",
-                            fg="cyan",
-                            bold=True,
-                        ),
-                        level=3,
-                    )
-                    log(writer.as_string(), level=3)
-
     # Validate package labels
     package_labels = set(package_labels)
     has_bad_labels = False
@@ -351,21 +248,16 @@ def makemigrations(
             if package_label in package_labels
         }
 
-    if conflicts and not merge:
+    if conflicts:
         name_str = "; ".join(
             "{} in {}".format(", ".join(names), package)
             for package, names in conflicts.items()
         )
         raise click.ClickException(
             f"Conflicting migrations detected; multiple leaf nodes in the "
-            f"migration graph: ({name_str}).\nTo fix them run "
-            f"'python manage.py makemigrations --merge'"
+            f"migration graph: ({name_str})."
         )
 
-    # Handle merge if requested
-    if merge and conflicts:
-        return handle_merge(loader, conflicts)
-
     # Set up questioner
     if interactive:
         questioner = InteractiveMigrationQuestioner(
@@ -533,8 +425,7 @@ def migrate(
         )
         raise click.ClickException(
             "Conflicting migrations detected; multiple leaf nodes in the "
-            f"migration graph: ({name_str}).\nTo fix them run "
-            "'python manage.py makemigrations --merge'"
+            f"migration graph: ({name_str})."
         )
 
     # If they supplied command line arguments, work out what they mean.

plain/models/migrations/questioner.py

@@ -74,10 +74,6 @@ class MigrationQuestioner:
         """Was this model really renamed?"""
         return self.defaults.get("ask_rename_model", False)
 
-    def ask_merge(self, package_label):
-        """Should these migrations really be merged?"""
-        return self.defaults.get("ask_merge", False)
-
     def ask_auto_now_add_addition(self, field_name, model_name):
         """Adding an auto_now_add field to a model."""
         # None means quit
@@ -227,16 +223,6 @@ class InteractiveMigrationQuestioner(MigrationQuestioner):
             default=False,
         )
 
-    def ask_merge(self, package_label):
-        return self._boolean_input(
-            (
-                "\nMerging will only work if the operations printed above do not conflict\n"
-                "with each other (working on different fields or models)\n"
-                "Should these migration branches be merged?"
-            ),
-            default=False,
-        )
-
     def ask_auto_now_add_addition(self, field_name, model_name):
         """Adding an auto_now_add field to a model."""
         if not self.dry_run:

{plain_models-0.35.0.dist-info → plain_models-0.36.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: plain.models
-Version: 0.35.0
+Version: 0.36.0
 Summary: Database models for Plain.
 Author-email: Dave Gaeddert <dave.gaeddert@dropseed.dev>
 License-File: LICENSE

{plain_models-0.35.0.dist-info → plain_models-0.36.0.dist-info}/RECORD

@@ -1,9 +1,9 @@
-plain/models/CHANGELOG.md,sha256=bUinpgx_HxNjgItauzHyszRtwU7F3Zm33u0mMAZBoDs,4127
+plain/models/CHANGELOG.md,sha256=eZWX8ouq5qD7hAMQVKAc7yv8EPI3HQJdvZlmwtDzcrA,4561
 plain/models/README.md,sha256=vsZPev3Fna-Irdcs3-wrOcAoII5LOhXdcWLBMT87CS0,1626
 plain/models/__init__.py,sha256=dnU6MOXs3lGoK31nLWjCqbf7zigkaUccomchz9lNDJ8,2950
 plain/models/aggregates.py,sha256=P0mhsMl1VZt2CVHMuCHnNI8SxZ9citjDLEgioN6NOpo,7240
 plain/models/base.py,sha256=8-F9DvSF5s2_zoGyEmZU5xg9GdEigthVT0HENv9iigI,67952
-plain/models/cli.py,sha256=4iZRHRPXxUCrsW-SzeimIE_lrSkWkLhgYOSk4BxUF_8,44985
+plain/models/cli.py,sha256=f303FCV4HWJMVHCW-bK3CUOblXblpCknGjgvgnnbs-U,40409
 plain/models/config.py,sha256=OF7eIEtXNZyGwgc3eMEpb5uEAup5RXeT-0um60dfBeU,636
 plain/models/connections.py,sha256=RBNa2FZ0x3C9un6PaYL-IYzH_OesRSpdHNGKvYHGiOM,2276
 plain/models/constants.py,sha256=ndnj9TOTKW0p4YcIPLOLEbsH6mOgFi6B1-rIzr_iwwU,210
@@ -67,7 +67,7 @@ plain/models/backends/sqlite3/introspection.py,sha256=RMzb8RUfNUvrcbcmgIKjxwp5D2
 plain/models/backends/sqlite3/operations.py,sha256=K2PFCsvzGhBrPEzz1AXFenge2B4Ap8lsQAABsC4e_UI,15302
 plain/models/backends/sqlite3/schema.py,sha256=sbtI0PBGe0fK0IOFWh0bkpVntCMMVsfzPb9dpL7o4r8,22566
 plain/models/backups/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-plain/models/backups/cli.py,sha256=du_751qWPtJT7C3GF_nLfHucMoUzlUOz9OKUwqHQ16s,2990
+plain/models/backups/cli.py,sha256=AKVh_Go0LxnxFi2jQjmpPEbbJvYCurjJsGCWESm1X8A,2960
 plain/models/backups/clients.py,sha256=WQ4X4KrkNeiKYJcpGRwnfRiAeeyduHWEnq4tA4mtjvw,4047
 plain/models/backups/core.py,sha256=09IZUhBEe1Yej3PC8AidtkaI0c8tt7VnqGBCWK-WrFg,3318
 plain/models/fields/__init__.py,sha256=b0qggzhZf89cXSwokr6rikWyF6kKPTuceIT3pI1Rz3s,80093
@@ -92,7 +92,7 @@ plain/models/migrations/graph.py,sha256=nrztu_8dU0wAUSxKUqqFWpvZcSQxGEqE6dXWkPyt
 plain/models/migrations/loader.py,sha256=qUTmaEYI1_mV6goQPQYZKjSz8rMbE6G1wqvrAsmuGwA,16464
 plain/models/migrations/migration.py,sha256=22YwRHnaRnCkBpW5p7K89tAU6h4QSsG5yiq-o7W-cSI,6505
 plain/models/migrations/optimizer.py,sha256=HH-uz-jnWw_Ni6F2_rRW1nax1Dxmf1s_F_8s8N2OlVc,3266
-plain/models/migrations/questioner.py,sha256=Q0JSiumYYGHkv4O8LFWioNvtJZa0J2JHIvf0NX7eSZM,12490
+plain/models/migrations/questioner.py,sha256=qAsePI5JHiSJrlY_kmpgMuK9Dom22q17edov7RtBeGw,11967
 plain/models/migrations/recorder.py,sha256=9UfNdzM5meeOXSxMNxTN69uqX0K2uh5Ma7dNA2fL2rI,3655
 plain/models/migrations/serializer.py,sha256=IRO1-8ojalOFt27DSfCDhRWjTULQZQ-aSroL-A9MPbk,12947
 plain/models/migrations/state.py,sha256=Xb09rIdh3D-LVx3gPgaJ2mcXDL6WWW92gq6coJsjycM,35476
@@ -113,8 +113,8 @@ plain/models/sql/where.py,sha256=ezE9Clt2BmKo-I7ARsgqZ_aVA-1UdayCwr6ULSWZL6c,126
 plain/models/test/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 plain/models/test/pytest.py,sha256=5mxY1MHqfCFJ7G39W1DuE0vCHfsfehQQmOyE6vI6caw,3323
 plain/models/test/utils.py,sha256=MxBNWoGMtwAtka7LbxWgilgzv7T5qqJL8ystF2SDJrs,345
-plain_models-0.35.0.dist-info/METADATA,sha256=v8A_EGrAuBGpAa_7thYuC1eim5Alt7jdfLskHWfUerg,1921
-plain_models-0.35.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-plain_models-0.35.0.dist-info/entry_points.txt,sha256=IYJAW9MpL3PXyXFWmKmALagAGXC_5rzBn2eEGJlcV04,112
-plain_models-0.35.0.dist-info/licenses/LICENSE,sha256=m0D5O7QoH9l5Vz_rrX_9r-C8d9UNr_ciK6Qwac7o6yo,3175
-plain_models-0.35.0.dist-info/RECORD,,
+plain_models-0.36.0.dist-info/METADATA,sha256=Pm1uJVIOrhVQl1t-vEBk0jHD0NYU3mEhieE1_m9TX-Q,1921
+plain_models-0.36.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+plain_models-0.36.0.dist-info/entry_points.txt,sha256=IYJAW9MpL3PXyXFWmKmALagAGXC_5rzBn2eEGJlcV04,112
+plain_models-0.36.0.dist-info/licenses/LICENSE,sha256=m0D5O7QoH9l5Vz_rrX_9r-C8d9UNr_ciK6Qwac7o6yo,3175
+plain_models-0.36.0.dist-info/RECORD,,