@ai-devtools/dbspec-macos-arm64 1.6.1 → 1.6.2

package/bin/dbspec/_internal/dbspec.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dbspec
-Version: 1.6.1
+Version: 1.6.2
 Summary: DBSpec: CLI/TUI for database design (Excel as DSL)
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

package/bin/dbspec/_internal/dbspec.egg-info/SOURCES.txt

@@ -40,6 +40,7 @@ src/db_import/types.py
 src/db_import/validation.py
 src/generators/__init__.py
 src/generators/sqlalchemy/__init__.py
+src/generators/sqlalchemy/attribute_names.py
 src/generators/sqlalchemy/constraints.py
 src/generators/sqlalchemy/context.py
 src/generators/sqlalchemy/domains.py
@@ -54,6 +55,7 @@ src/generators/sqlalchemy/sequences.py
 src/generators/sqlalchemy/tables.py
 src/generators/sqlalchemy/templates.py
 src/generators/sqlalchemy/types.py
+src/generators/sqlalchemy/validation.py
 src/logic/__init__.py
 src/logic/check_rules.py
 src/logic/db_validation.py

package/bin/dbspec/_internal/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "dbspec"
-version = "1.6.1"
+version = "1.6.2"
 description = "DBSpec: CLI/TUI for database design (Excel as DSL)"
 readme = "README.md"
 requires-python = ">=3.11"

package/bin/dbspec/_internal/src/resources/sqlalchemy_templates/README.md

@@ -70,3 +70,26 @@ Jinja-параметров, зависящий только от стандар
 изолированным subprocess без site-packages.
 
 В 1.6.0 IndexContext содержит directions (ASC/DESC/пустая строка) в порядке columns и has_directions. Для направленных unique indexes используйте Index(unique=True), а не UniqueConstraint. Направления передаются методами SQLAlchemy .asc()/.desc(). Проектные шаблоны обновляются отдельно.
+
+## Безопасные Python-атрибуты (1.6.2)
+
+`col.name` хранит исходное SQL-имя, `col.attr_name` — уникальный Python-атрибут.
+При различии передавайте `col.name` первым аргументом `mapped_column`, как в штатном
+шаблоне. Ключевые слова и служебные имена получают безопасные атрибуты; допустимые
+`match`, `case`, `type` сохраняются. Коллизии распределяются детерминированно в общей
+области колонок и связей, без изменения имён SQL-колонок или ограничений.
+
+`rel.fk_columns` сохраняет прежний контракт: локальные имена FK-атрибутов для
+дочерней связи и пустой список для обратной. Новое необязательное поле
+`rel.fk_expression` содержит строку списка class-qualified ссылок
+(`[Child.sender_id]`). Штатный шаблон передаёт её через `py_repr` в
+`relationship.foreign_keys`, разрешая обе стороны после импорта моделей.
+Прежние проектные шаблоны простых связей продолжают работать без обновления.
+
+До замены результата генератор проверяет синтаксис всех Python-файлов, импорт,
+настройку ORM, полноту колонок metadata и разрешение FK в отдельном процессе.
+Это изоляция состояния Python, а не security sandbox пользовательского кода.
+Внешний `base_import` должен импортироваться в окружении генерации; его существующие
+таблицы допустимы, но состав каждой генерируемой таблицы должен совпасть с SSOT.
+При ошибке прежний выходной каталог сохраняется. Установка приложения или соединение
+с БД для штатных шаблонов не требуются.

package/bin/dbspec/_internal/src/resources/sqlalchemy_templates/domain_init.py.j2

@@ -1,4 +1,4 @@
-# Domain package: {{ domain.name }}
+{{ ("Domain package: " ~ domain.name) | py_repr }}
 
 {% if model_mode == "core" %}
 {% for table in domain.tables %}
@@ -7,7 +7,7 @@ from .{{ table.module_name }} import {{ table.module_name }}
 
 __all__ = [
 {% for table in domain.tables %}
-    "{{ table.module_name }}",
+    {{ table.module_name | py_string }},
 {% endfor %}
 ]
 {% else %}
@@ -23,7 +23,7 @@ from .{{ table.module_name }} import {{ table.export_name }}
 
 __all__ = [
 {% for table in domain.tables %}
-    "{{ table.export_name }}",
+    {{ table.export_name | py_string }},
 {% endfor %}
 ]
 {% endif %}

package/bin/dbspec/_internal/src/resources/sqlalchemy_templates/model.py.j2

@@ -1,4 +1,4 @@
-"""{{ table.description if include_comments and table.description else table.class_name ~ " model." }}"""
+{{ (table.description if include_comments and table.description else table.class_name ~ " model.") | py_repr }}
 
 from __future__ import annotations
 
@@ -32,11 +32,11 @@ from ..{{ common_dir_name }}.base import metadata
 {% for enum in table.enums %}
 {{ enum.var_name }} = ENUM(
     {% for v in enum.values %}
-    "{{ v | replace('"', '\\"') }}"{% if not loop.last %}, {% endif +%}
+    {{ v | py_string }}{% if not loop.last %}, {% endif +%}
     {% endfor %}
     ,
-    name="{{ enum.name }}",
-    schema="{{ enum.schema }}",
+    name={{ enum.name | py_string }},
+    schema={{ enum.schema | py_string }},
     create_type={{ "True" if enum.create_type else "False" }},
 )
 {% endfor %}
@@ -45,24 +45,24 @@ from ..{{ common_dir_name }}.base import metadata
 {% if table.sequences %}
 {% for seq in table.sequences %}
 {{ seq.var_name }} = Sequence(
-    "{{ seq.name }}",
+    {{ seq.name | py_string }},
 {% if seq.schema %}
-    schema="{{ seq.schema }}",
+    schema={{ seq.schema | py_string }},
 {% endif %}
 )
 {% endfor %}
 {% endif %}
 
 {{ table.module_name }} = Table(
-    "{{ table.table }}",
+    {{ table.table | py_string }},
     metadata,
 {% for col in table.columns %}
     Column(
-        "{{ col.name }}",
+        {{ col.name | py_string }},
         {{ col.sa_type }},
 {% set col_args = [] -%}
 {% if col.fk_target -%}
-{% set _ = col_args.append('ForeignKey("' ~ col.fk_target ~ '")') -%}
+{% set _ = col_args.append('ForeignKey(' ~ (col.fk_target | py_string) ~ ')') -%}
 {% endif -%}
 {% if col.sequence -%}
 {% set _ = col_args.append(col.sequence.var_name) -%}
@@ -81,7 +81,7 @@ from ..{{ common_dir_name }}.base import metadata
 {% if col.sequence -%}
 {% set _ = col_args.append('server_default=' ~ col.sequence.var_name ~ '.next_value()') -%}
 {% elif col.server_default -%}
-{% set _ = col_args.append('server_default=text("' ~ (col.server_default | replace('\"', '\\\"')) ~ '")') -%}
+{% set _ = col_args.append('server_default=text(' ~ (col.server_default | py_string) ~ ')') -%}
 {% endif -%}
 {% if include_comments and col.description -%}
 {% set _ = col_args.append('comment=' ~ (col.description | py_repr)) -%}
@@ -94,9 +94,9 @@ from ..{{ common_dir_name }}.base import metadata
 {% if table.pk_constraint and table.pk_constraint.columns %}
     PrimaryKeyConstraint(
 {% for pk_col in table.pk_constraint.columns %}
-        "{{ pk_col }}",
+        {{ pk_col | py_string }},
 {% endfor %}
-        name="{{ table.pk_constraint.name }}",
+        name={{ table.pk_constraint.name | py_string }},
     ),
 {% endif %}
 {% for check in table.check_constraints %}
@@ -105,9 +105,9 @@ from ..{{ common_dir_name }}.base import metadata
 {% for constraint in table.unique_constraints %}
     UniqueConstraint(
 {% for c in constraint.columns %}
-        "{{ c }}",
+        {{ c | py_string }},
 {% endfor %}
-        name="{{ constraint.name }}",
+        name={{ constraint.name | py_string }},
 {% if constraint.nulls_not_distinct %}
         postgresql_nulls_not_distinct=True,
 {% endif %}
@@ -117,13 +117,13 @@ from ..{{ common_dir_name }}.base import metadata
 {% if idx.unique and not idx.where and not idx.has_directions %}
     UniqueConstraint(
 {% for c in idx.columns %}
-        "{{ c }}",
+        {{ c | py_string }},
 {% endfor %}
-        name="{{ idx.name }}",
+        name={{ idx.name | py_string }},
     ),
 {% endif %}
 {% endfor %}
-    schema="{{ table.schema }}",
+    schema={{ table.schema | py_string }},
 {% if include_comments and table.description %}
     comment={{ table.description | py_repr }},
 {% endif %}
@@ -131,7 +131,7 @@ from ..{{ common_dir_name }}.base import metadata
 {% for idx in table.indexes %}
 {% if idx.where or not idx.unique or idx.has_directions %}
 Index(
-    "{{ idx.name }}",
+    {{ idx.name | py_string }},
 {% for c in idx.columns %}
     {{ table.module_name }}.c[{{ c | py_repr }}]{% if idx.directions[loop.index0] %}.{{ idx.directions[loop.index0] | lower }}(){% endif %},
 {% endfor %}
@@ -139,7 +139,7 @@ Index(
     unique=True,
 {% endif %}
 {% if idx.where %}
-    postgresql_where=text("{{ idx.where }}"),
+    postgresql_where=text({{ idx.where | py_string }}),
 {% endif %}
 )
 {% endif %}
@@ -186,11 +186,11 @@ if TYPE_CHECKING:
 {% for enum in table.enums %}
 {{ enum.var_name }} = ENUM(
     {% for v in enum.values %}
-    "{{ v | replace('"', '\\"') }}"{% if not loop.last %}, {% endif +%}
+    {{ v | py_string }}{% if not loop.last %}, {% endif +%}
     {% endfor %}
     ,
-    name="{{ enum.name }}",
-    schema="{{ enum.schema }}",
+    name={{ enum.name | py_string }},
+    schema={{ enum.schema | py_string }},
     create_type={{ "True" if enum.create_type else "False" }},
 )
 {% endfor %}
@@ -199,9 +199,9 @@ if TYPE_CHECKING:
 {% if table.sequences %}
 {% for seq in table.sequences %}
 {{ seq.var_name }} = Sequence(
-    "{{ seq.name }}",
+    {{ seq.name | py_string }},
 {% if seq.schema %}
-    schema="{{ seq.schema }}",
+    schema={{ seq.schema | py_string }},
 {% endif %}
 )
 {% endfor %}
@@ -209,9 +209,9 @@ if TYPE_CHECKING:
 
 
 class {{ table.class_name }}({{ base_class }}):
-    """{{ table.description if include_comments and table.description else table.class_name ~ " table." }}"""
+    {{ (table.description if include_comments and table.description else table.class_name ~ " table.") | py_repr }}
 
-    __tablename__ = "{{ table.table }}"  # type: ignore[assignment]
+    __tablename__ = {{ table.table | py_string }}  # type: ignore[assignment]
 
 {% set has_table_level_args = (table.indexes and (table.indexes | length > 0)) or (table.unique_constraints and (table.unique_constraints | length > 0)) or (table.pk_constraint and table.pk_constraint.columns) or (table.check_constraints and (table.check_constraints | length > 0)) %}
 {% if has_table_level_args %}
@@ -219,9 +219,9 @@ class {{ table.class_name }}({{ base_class }}):
 {% if table.pk_constraint and table.pk_constraint.columns %}
         PrimaryKeyConstraint(
 {% for pk_col in table.pk_constraint.columns %}
-            "{{ pk_col }}",
+            {{ pk_col | py_string }},
 {% endfor %}
-            name="{{ table.pk_constraint.name }}",
+            name={{ table.pk_constraint.name | py_string }},
         ),
 {% endif %}
 {% for check in table.check_constraints %}
@@ -230,9 +230,9 @@ class {{ table.class_name }}({{ base_class }}):
 {% for constraint in table.unique_constraints %}
         UniqueConstraint(
 {% for c in constraint.columns %}
-            "{{ c }}",
+            {{ c | py_string }},
 {% endfor %}
-            name="{{ constraint.name }}",
+            name={{ constraint.name | py_string }},
 {% if constraint.nulls_not_distinct %}
             postgresql_nulls_not_distinct=True,
 {% endif %}
@@ -252,49 +252,49 @@ class {{ table.class_name }}({{ base_class }}):
         ),
 {% elif idx.where and idx.unique %}
         Index(
-            "{{ idx.name }}",
+            {{ idx.name | py_string }},
 {% for c in idx.columns %}
-            "{{ c }}",
+            {{ c | py_string }},
 {% endfor %}
             unique=True,
-            postgresql_where=text("{{ idx.where }}"),
+            postgresql_where=text({{ idx.where | py_string }}),
         ),
 {% elif idx.where %}
         Index(
-            "{{ idx.name }}",
+            {{ idx.name | py_string }},
 {% for c in idx.columns %}
-            "{{ c }}",
+            {{ c | py_string }},
 {% endfor %}
-            postgresql_where=text("{{ idx.where }}"),
+            postgresql_where=text({{ idx.where | py_string }}),
         ),
 {% elif idx.unique %}
         UniqueConstraint(
 {% for c in idx.columns %}
-            "{{ c }}",
+            {{ c | py_string }},
 {% endfor %}
-            name="{{ idx.name }}",
+            name={{ idx.name | py_string }},
         ),
 {% else %}
         Index(
-            "{{ idx.name }}",
+            {{ idx.name | py_string }},
 {% for c in idx.columns %}
-            "{{ c }}",
+            {{ c | py_string }},
 {% endfor %}
         ),
 {% endif %}
 {% endfor %}
         {
-            "schema": "{{ table.schema }}",
+            "schema": {{ table.schema | py_string }},
 {% if include_comments and table.description %}
-            "comment": "{{ table.description }}",
+            "comment": {{ table.description | py_string }},
 {% endif %}
         },
     )
 {% else %}
     __table_args__ = {
-        "schema": "{{ table.schema }}",
+        "schema": {{ table.schema | py_string }},
 {% if include_comments and table.description %}
-        "comment": "{{ table.description }}",
+        "comment": {{ table.description | py_string }},
 {% endif %}
     }
 {% endif %}
@@ -307,7 +307,7 @@ class {{ table.class_name }}({{ base_class }}):
         {{ col.sa_type }},
 {% set col_args = [] -%}
 {% if col.fk_target -%}
-{% set _ = col_args.append('ForeignKey("' ~ col.fk_target ~ '")') -%}
+{% set _ = col_args.append('ForeignKey(' ~ (col.fk_target | py_string) ~ ')') -%}
 {% endif -%}
 {% if col.sequence -%}
 {% set _ = col_args.append(col.sequence.var_name) -%}
@@ -326,7 +326,7 @@ class {{ table.class_name }}({{ base_class }}):
 {% if col.sequence -%}
 {% set _ = col_args.append('server_default=' ~ col.sequence.var_name ~ '.next_value()') -%}
 {% elif col.server_default -%}
-{% set _ = col_args.append('server_default=text("' ~ (col.server_default | replace('\"', '\\\"')) ~ '")') -%}
+{% set _ = col_args.append('server_default=text(' ~ (col.server_default | py_string) ~ ')') -%}
 {% endif -%}
 {% if include_comments and col.description -%}
 {% set _ = col_args.append('comment=' ~ (col.description | py_repr)) -%}
@@ -340,10 +340,10 @@ class {{ table.class_name }}({{ base_class }}):
     # Relationships
 {% for rel in table.relationships %}
     {{ rel.attr_name }}: Mapped[{{ rel.python_type }}] = relationship(
-        "{{ rel.target_class }}",
-        back_populates="{{ rel.back_populates }}",
-{% if rel.fk_columns %}
-        foreign_keys=[{% for fk_col in rel.fk_columns %}{{ fk_col }}{% if not loop.last %}, {% endif %}{% endfor %}],
+        {{ rel.target_class | py_string }},
+        back_populates={{ rel.back_populates | py_string }},
+{% if rel.fk_expression %}
+        foreign_keys={{ rel.fk_expression | py_repr }},
 {% endif %}
     )
 {% endfor %}

package/bin/dbspec/_internal/src/resources/sqlalchemy_templates/package_init.py.j2

@@ -14,7 +14,7 @@ __all__ = [
     "metadata",
 {% for domain in domains %}
 {% for table in domain.tables %}
-    "{{ table.module_name }}",
+    {{ table.module_name | py_string }},
 {% endfor %}
 {% endfor %}
 ]
@@ -41,14 +41,14 @@ from .{{ domain.module_name }}.{{ table.module_name }} import {{ table.export_na
 {% endfor %}
 
 __all__ = [
-    "{{ base_class }}",
+    {{ base_class | py_string }},
 {% for domain in domains %}
 {% for table in domain.tables %}
-    "{{ table.export_name }}",
+    {{ table.export_name | py_string }},
 {% endfor %}
 {% endfor %}
 ]
 {% else %}
-__all__ = ["{{ base_class }}"]
+__all__ = [{{ base_class | py_string }}]
 {% endif %}
 {% endif %}

package/package.json

@@ -24,5 +24,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "1.6.1"
+  "version": "1.6.2"
 }