sqlbuild 0.63.8__tar.gz → 0.65.0__tar.gz

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/Cargo.lock

@@ -572,7 +572,7 @@ checksum = "f8fadd59c855ef2080decdef8ff161eb6661b86933c9d82e5ba29dc602a55aba"
 
 [[package]]
 name = "sqlbuild-kata-native"
-version = "0.63.8"
+version = "0.65.0"
 dependencies = [
  "fensu-policy",
  "globset",

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sqlbuild
-Version: 0.63.8
+Version: 0.65.0
 Requires-Dist: duckdb>=1.3.2
 Requires-Dist: filelock>=3.20.3
 Requires-Dist: polyglot-sql>=0.9.2

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/crates/sqlbuild-kata-native/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "sqlbuild-kata-native"
-version = "0.63.8"
+version = "0.65.0"
 edition.workspace = true
 license.workspace = true
 publish.workspace = true

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/pyproject.toml

@@ -7,7 +7,7 @@ build-backend = "maturin"
 
 [project]
 name = "sqlbuild"
-version = "0.63.8"
+version = "0.65.0"
 description = "Typed, test-first SQL pipelines with local E2E testing"
 readme = "README.md"
 requires-python = ">=3.12"

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/build_planning/defer_clone.py

@@ -10,9 +10,6 @@ from sqlbuild.adapter.contract.classes.base_adapter import BaseAdapter
 from sqlbuild.cli.commands._helpers.planning.external_refs import (
     resolve_external_sql_reference_resolver,
 )
-from sqlbuild.cli.commands._helpers.runtime.connection import (
-    resolve_target_connection_config,
-)
 from sqlbuild.cli.commands.exceptions import CliUserError
 from sqlbuild.cli.commands.models import (
     BuildCommandRequest,
@@ -31,7 +28,11 @@ from sqlbuild.compiler.compile.types import CompiledResourceType
 from sqlbuild.compiler.discovery.models import DiscoveredProjectInputs
 from sqlbuild.compiler.pipeline.main.clone_with_options import run_clone_pipeline_with_options
 from sqlbuild.compiler.pipeline.main.compiled_project import build_compiled_project
-from sqlbuild.compiler.pipeline.models import ClonePipelineOptions, ClonePipelineResult
+from sqlbuild.compiler.pipeline.models import (
+    ClonePipelineConnection,
+    ClonePipelineOptions,
+    ClonePipelineResult,
+)
 from sqlbuild.compiler.planner.main.clone.resolve_clone_boundary import (
     resolve_clone_boundary,
 )
@@ -216,14 +217,6 @@ def run_defer_clone_prephase(
     if on_progress is not None:
         on_progress("Preparing defer clone plan...")
     start: float = time.monotonic()
-    origin_connection: Any = adapter.connect(
-        resolve_target_connection_config(
-            discovered_inputs=discovered_inputs,
-            project_dir=project_dir,
-            target_name=origin_target_name,
-            cli_vars=cli_vars,
-        )
-    )
     destination_connection: Any = adapter.connect(inputs.connection_config)
     try:
         clone_pipeline: ClonePipelineResult = run_clone_pipeline_with_options(
@@ -231,7 +224,10 @@ def run_defer_clone_prephase(
             adapter=adapter,
             origin_target_name=origin_target_name,
             destination_target_name=destination_target_name,
-            destination_connection=destination_connection,
+            destination_connection=ClonePipelineConnection(
+                config=inputs.connection_config,
+                handle=destination_connection,
+            ),
             options=ClonePipelineOptions(
                 no_sql_validation=inputs.no_sql_validation,
                 no_cache=inputs.no_cache,
@@ -260,7 +256,6 @@ def run_defer_clone_prephase(
                     destination_function_entries=clone_pipeline.destination_function_entries,
                     execution_order=clone_pipeline.clone_plan.execution_order,
                     adapter=adapter,
-                    origin_connection=origin_connection,
                     destination_connection=destination_connection,
                     hard_copy=False,
                     run_id=clone_pipeline.destination_project.run_id,
@@ -298,11 +293,9 @@ def run_defer_clone_prephase(
             origin_seed_entries=clone_pipeline.origin_seed_entries,
             destination_seed_entries=clone_pipeline.destination_seed_entries,
             adapter=adapter,
-            origin_connection=origin_connection,
             destination_connection=destination_connection,
             run_id=clone_pipeline.destination_project.run_id,
             query_change_tracking=clone_pipeline.destination_project.settings.query_change_tracking,
         )
     finally:
-        adapter.close(origin_connection)
         adapter.close(destination_connection)

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/clone/connections.py

@@ -21,14 +21,8 @@ from sqlbuild.presentation.main.phase_line import format_phase_line
 def connect_clone_targets(
     *, request: CloneCommandRequest, invocation: CloneInvocation
 ) -> CloneConnectionContext:
-    """Resolve connection configs and connect origin and destination targets."""
+    """Resolve and connect the destination target used for the whole clone."""
 
-    origin_connection_config: dict[str, object] = resolve_target_connection_config(
-        discovered_inputs=invocation.discovered_inputs,
-        project_dir=invocation.effective_project_dir,
-        target_name=request.origin_target_name,
-        cli_vars=request.cli_vars,
-    )
     destination_connection_config: dict[str, object] = resolve_target_connection_config(
         discovered_inputs=invocation.discovered_inputs,
         project_dir=invocation.effective_project_dir,
@@ -41,7 +35,6 @@ def connect_clone_targets(
     )
     progress.start(f"Connecting to {invocation.adapter_name}...")
     connect_start: float = time.monotonic()
-    origin_connection: Any = invocation.adapter.connect(origin_connection_config)
     destination_connection: Any = invocation.adapter.connect(destination_connection_config)
     progress.complete_styled(
         format_phase_line(
@@ -52,9 +45,7 @@ def connect_clone_targets(
         )
     )
     return CloneConnectionContext(
-        origin_connection_config=origin_connection_config,
         destination_connection_config=destination_connection_config,
-        origin_connection=origin_connection,
         destination_connection=destination_connection,
     )
 
@@ -62,7 +53,6 @@ def connect_clone_targets(
 def close_clone_targets(
     *, invocation: CloneInvocation, connection_context: CloneConnectionContext
 ) -> None:
-    """Close origin and destination clone target connections."""
+    """Close the clone destination connection."""
 
-    invocation.adapter.close(connection_context.origin_connection)
     invocation.adapter.close(connection_context.destination_connection)

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/clone/execution.py

@@ -51,7 +51,6 @@ def execute_clone_plan(
             destination_function_entries=preparation.pipeline_result.destination_function_entries,
             execution_order=preparation.pipeline_result.clone_plan.execution_order,
             adapter=invocation.adapter,
-            origin_connection=connection_context.origin_connection,
             destination_connection=connection_context.destination_connection,
             hard_copy=request.hard_copy,
             run_id=preparation.pipeline_result.destination_project.run_id,
@@ -66,7 +65,6 @@ def execute_clone_plan(
         origin_seed_entries=preparation.pipeline_result.origin_seed_entries,
         destination_seed_entries=preparation.pipeline_result.destination_seed_entries,
         adapter=invocation.adapter,
-        origin_connection=connection_context.origin_connection,
         destination_connection=connection_context.destination_connection,
         run_id=preparation.pipeline_result.destination_project.run_id,
         query_change_tracking=preparation.pipeline_result.destination_project.settings.query_change_tracking,

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/clone/planning.py

@@ -16,7 +16,7 @@ from sqlbuild.cli.commands.models import (
 )
 from sqlbuild.cli.progress.classes.planning_progress_reporter import PlanningProgressReporter
 from sqlbuild.compiler.pipeline.main.clone import run_clone_pipeline
-from sqlbuild.compiler.pipeline.models import ClonePipelineResult
+from sqlbuild.compiler.pipeline.models import ClonePipelineConnection, ClonePipelineResult
 from sqlbuild.compiler.planner.models import (
     CloneSourcePlanEntry,
     FunctionPlanEntry,
@@ -48,7 +48,10 @@ def prepare_clone_execution(
         select=request.select,
         exclude=request.exclude,
         cli_vars=request.cli_vars,
-        destination_connection=connection_context.destination_connection,
+        destination_connection=ClonePipelineConnection(
+            config=connection_context.destination_connection_config,
+            handle=connection_context.destination_connection,
+        ),
         external_sql_reference_resolver=resolve_external_sql_reference_resolver(
             project_dir=invocation.effective_project_dir,
             discovered_inputs=invocation.discovered_inputs,

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/clone/validation.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 from sqlbuild.cli.commands.exceptions import CliUserError
 from sqlbuild.compiler.discovery.models import DiscoveredProjectInputs
+from sqlbuild.spec.contracts.main.resolve_target_config import resolve_target_config
 from sqlbuild.spec.contracts.models import TargetConfig
 
 
@@ -13,14 +14,26 @@ def validate_clone_request(
     origin_target_name: str,
     destination_target_name: str,
 ) -> None:
-    targets: dict[str, TargetConfig] = discovered_inputs.project_config.targets
+    project_targets: dict[str, TargetConfig] = discovered_inputs.project_config.targets
+    local_target_names: set[str] = set(discovered_inputs.local_config.targets)
+    target_names: set[str] = set(project_targets) | local_target_names
     if origin_target_name == destination_target_name:
         raise CliUserError("clone requires different --from and --to targets", code="C401")
-    if origin_target_name not in targets:
+    if origin_target_name not in target_names:
         raise CliUserError(f"unknown target '{origin_target_name}'", code="C402")
-    if destination_target_name not in targets:
+    if destination_target_name not in target_names:
         raise CliUserError(f"unknown target '{destination_target_name}'", code="C403")
-    if not targets[origin_target_name].clone.allow_as_clone_origin:
+    origin_target: TargetConfig = resolve_target_config(
+        project_config=discovered_inputs.project_config,
+        local_config=discovered_inputs.local_config,
+        target_name=origin_target_name,
+    )
+    destination_target: TargetConfig = resolve_target_config(
+        project_config=discovered_inputs.project_config,
+        local_config=discovered_inputs.local_config,
+        target_name=destination_target_name,
+    )
+    if not origin_target.clone.allow_as_clone_origin:
         raise CliUserError(
             (
                 f"target '{origin_target_name}' is not allowed as a clone origin target; "
@@ -28,7 +41,7 @@ def validate_clone_request(
             ),
             code="C404",
         )
-    if not targets[destination_target_name].clone.allow_as_clone_destination:
+    if not destination_target.clone.allow_as_clone_destination:
         raise CliUserError(
             (
                 f"target '{destination_target_name}' is not allowed as a clone destination "

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/clone/virtual.py

@@ -35,12 +35,6 @@ def execute_virtual_clone(*, request: CloneCommandRequest, invocation: CloneInvo
         adapter=invocation.adapter,
         origin_target_name=request.origin_target_name,
         destination_target_name=invocation.destination_target_name,
-        origin_connection_config=resolve_target_connection_config(
-            discovered_inputs=invocation.discovered_inputs,
-            project_dir=invocation.effective_project_dir,
-            target_name=request.origin_target_name,
-            cli_vars=request.cli_vars,
-        ),
         destination_connection_config=resolve_target_connection_config(
             discovered_inputs=invocation.discovered_inputs,
             project_dir=invocation.effective_project_dir,

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/entry/dispatch.py

@@ -16,7 +16,6 @@ from sqlbuild.cli.commands.constants import (
     SCENARIO_CLI_LOCAL_SNAPSHOT_FLAG_REQUIRED,
     SCENARIO_CLI_MISSING_SUBCOMMAND,
     SCENARIO_TEST_COMMAND,
-    SKILLS_UPDATE_COMMAND,
 )
 from sqlbuild.cli.commands.exceptions import CliUserError
 from sqlbuild.cli.commands.models import (
@@ -472,14 +471,12 @@ def _dispatch_local_command(
             )
         )
     if args.command == CliCommand.SKILLS:
-        if args.skills_command == SKILLS_UPDATE_COMMAND:
-            return handlers.run_skills_update(
-                project_dir=project_dir,
-                global_install=args.skills_global,
-                targets=tuple(args.skills_target),
-                force=args.skills_force,
-            )
-        raise CliUserError("skills requires a subcommand such as 'update'", code="C807")
+        return handlers.run_skills_update(
+            project_dir=project_dir,
+            global_install=args.skills_global,
+            targets=tuple(args.skills_target),
+            force=args.skills_force,
+        )
     if args.command == CliCommand.SCENARIO:
         return _dispatch_scenario_command(
             args=args,

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/entry/parser.py

@@ -581,18 +581,15 @@ def _add_skills_parsers(
     subparsers: argparse._SubParsersAction[argparse.ArgumentParser],
 ) -> None:
     skills_parser: argparse.ArgumentParser = subparsers.add_parser(CliCommand.SKILLS)
-    skills_subparsers: argparse._SubParsersAction[argparse.ArgumentParser]
-    skills_subparsers = skills_parser.add_subparsers(dest="skills_command")
-    skills_update_parser: argparse.ArgumentParser = skills_subparsers.add_parser("update")
-    skills_update_parser.add_argument("--global", dest="skills_global", action="store_true")
-    skills_update_parser.add_argument(
+    skills_parser.add_argument("--global", dest="skills_global", action="store_true")
+    skills_parser.add_argument(
         "--target",
         dest="skills_target",
         action="append",
         choices=("opencode", "claude", "agents"),
         default=[],
     )
-    skills_update_parser.add_argument("--force", dest="skills_force", action="store_true")
+    skills_parser.add_argument("--force", dest="skills_force", action="store_true")
 
 
 def _add_kata_parser(

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/init/scaffold.py

@@ -49,7 +49,7 @@ def _build_project_toml(*, project_name: str) -> str:
 adapter = "duckdb"
 default_target = "dev"
 
-[connection]
+[connections.developer]
 database = "{project_name}.duckdb"
 
 [settings]
@@ -59,8 +59,10 @@ default_audit_severity = "warn"
 materialized = "table"
 
 [targets.prod]
+connection = "developer"
 schema = "prod"
 
 [targets.dev]
+connection = "developer"
 schema = "dev"
 '''

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/playground/copy.py

@@ -18,7 +18,7 @@ _PYTHON_NODES_PROJECT_TOML: str = """name = "python_nodes_demo"
 adapter = "duckdb"
 default_target = "dev"
 
-[connection]
+[connections.developer]
 database = "python_nodes_demo.duckdb"
 
 [settings]
@@ -28,6 +28,7 @@ default_audit_severity = "warn"
 materialized = "table"
 
 [targets.dev]
+connection = "developer"
 schema = "main"
 """
 
@@ -183,7 +184,7 @@ _VIRTUAL_PROJECT_TOML: str = """name = "loader_waffle_shop_virtual"
 adapter = "duckdb"
 default_target = "dev"
 
-[connection]
+[connections.developer]
 database = "loader_waffle_shop_virtual.duckdb"
 
 [settings]
@@ -194,6 +195,7 @@ default_audit_severity = "warn"
 materialized = "table"
 
 [targets.dev]
+connection = "developer"
 schema = "dev"
 
 [targets.dev.state]

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/playground/templates/loader_waffle_shop/sqlbuild_project.toml

@@ -1,8 +1,13 @@
 name = "loader_waffle_shop"
 adapter = "duckdb"
+default_target = "dev"
 
-[connection]
+[connections.developer]
 database = "loader_waffle_shop.duckdb"
 
 [defaults]
 materialized = "table"
+
+[targets.dev]
+connection = "developer"
+schema = "main"

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/playground/templates/waffle_shop/sqlbuild_project.toml

@@ -2,7 +2,7 @@ name = "waffle_shop"
 adapter = "duckdb"
 default_target = "dev"
 
-[connection]
+[connections.developer]
 database = "waffle_shop_control.duckdb"
 
 [settings]
@@ -12,9 +12,11 @@ default_audit_severity = "warn"
 materialized = "table"
 
 [targets.prod]
+connection = "developer"
 schema = "prod"
 
 [targets.dev]
+connection = "developer"
 schema = "dev"
 
 [path_defaults.staging]

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/_helpers/skills/update.py

@@ -5,17 +5,27 @@ from __future__ import annotations
 import tomllib
 from importlib.resources import files
 from importlib.resources.abc import Traversable
+from os import replace
 from pathlib import Path
+from tempfile import NamedTemporaryFile
 
 from sqlbuild.cli.commands.exceptions import CliUserError
-from sqlbuild.cli.commands.models import SkillInstallTarget, SkillUpdateResult
+from sqlbuild.cli.commands.models import (
+    SkillInstallTarget,
+    SkillMaintenanceResult,
+    SkillSettings,
+    SkillUpdateResult,
+)
 from sqlbuild.compiler.discovery.constants import PROJECT_CONFIG_FILENAME
 
-generated_marker: str = "<!-- generated-by: sqlbuild skills update -->"
+generated_marker: str = "<!-- generated-by: sqlbuild skills -->"
+legacy_generated_marker: str = "<!-- generated-by: sqlbuild skills update -->"
+generated_markers: tuple[str, ...] = (generated_marker, legacy_generated_marker)
 skill_name: str = "sqlbuild"
 skill_source_package: str = "sqlbuild"
 skill_source_path: str = ".agents/skills/sqlbuild/SKILL.md"
 valid_skill_targets: tuple[str, ...] = ("opencode", "claude", "agents")
+default_skill_targets: tuple[str, ...] = ("agents", "claude")
 opencode_skill_target: str = "opencode"
 claude_skill_target: str = "claude"
 
@@ -56,11 +66,11 @@ def resolve_skill_target_names(
     if requested_targets:
         return normalize_skill_targets(targets=requested_targets, source="--target")
 
-    configured_targets: tuple[str, ...] = load_project_skill_targets(project_dir=project_dir)
-    if configured_targets:
-        return configured_targets
+    settings: SkillSettings = load_project_skill_settings(project_dir=project_dir)
+    if settings.targets:
+        return settings.targets
 
-    return valid_skill_targets
+    return default_skill_targets
 
 
 def normalize_skill_targets(*, targets: tuple[str, ...], source: str) -> tuple[str, ...]:
@@ -84,9 +94,13 @@ def normalize_skill_targets(*, targets: tuple[str, ...], source: str) -> tuple[s
 
 
 def load_project_skill_targets(*, project_dir: Path) -> tuple[str, ...]:
+    return load_project_skill_settings(project_dir=project_dir).targets
+
+
+def load_project_skill_settings(*, project_dir: Path) -> SkillSettings:
     config_path: Path = project_dir / PROJECT_CONFIG_FILENAME
     if not config_path.exists():
-        return ()
+        return SkillSettings(targets=(), auto_update=False, configured=False)
 
     try:
         with config_path.open("rb") as config_file:
@@ -97,11 +111,11 @@ def load_project_skill_targets(*, project_dir: Path) -> tuple[str, ...]:
             code="C802",
         ) from error
     if not isinstance(payload, dict):
-        return ()
+        return SkillSettings(targets=(), auto_update=False, configured=False)
 
     skills_payload: object = payload.get("skills")
     if skills_payload is None:
-        return ()
+        return SkillSettings(targets=(), auto_update=False, configured=False)
     if not isinstance(skills_payload, dict):
         raise CliUserError(
             f"{config_path} [skills] must be a table",
@@ -109,16 +123,94 @@ def load_project_skill_targets(*, project_dir: Path) -> tuple[str, ...]:
         )
 
     targets_payload: object = skills_payload.get("targets")
-    if targets_payload is None:
-        return ()
-    if not isinstance(targets_payload, list) or not all(
-        isinstance(target, str) for target in targets_payload
+    if targets_payload is not None and (
+        not isinstance(targets_payload, list)
+        or not all(isinstance(target, str) for target in targets_payload)
     ):
         raise CliUserError(
             f"{config_path} skills.targets must be a list of strings",
             code="C804",
         )
-    return normalize_skill_targets(targets=tuple(targets_payload), source="skills.targets")
+    auto_update: object = skills_payload.get("auto_update", False)
+    if not isinstance(auto_update, bool):
+        raise CliUserError(
+            f"{config_path} skills.auto_update must be a boolean",
+            code="C808",
+        )
+    targets: tuple[str, ...] = (
+        ()
+        if targets_payload is None
+        else normalize_skill_targets(targets=tuple(targets_payload), source="skills.targets")
+    )
+    return SkillSettings(
+        targets=targets or default_skill_targets,
+        auto_update=auto_update,
+        configured=True,
+    )
+
+
+def maintain_sqlbuild_skills(*, project_dir: Path) -> SkillMaintenanceResult:
+    """Check or refresh generated project skills without changing command success."""
+
+    settings: SkillSettings = load_project_skill_settings(project_dir=project_dir)
+    target_names: tuple[str, ...] = settings.targets
+    if not settings.configured:
+        target_names = tuple(
+            target_name
+            for target_name in valid_skill_targets
+            if _local_install_path(project_dir=project_dir, target_name=target_name).exists()
+        )
+    if not target_names:
+        return SkillMaintenanceResult()
+
+    install_targets: tuple[SkillInstallTarget, ...] = build_install_targets(
+        project_dir=project_dir,
+        target_names=target_names,
+    )
+    expected_content: str = ensure_generated_marker(load_packaged_skill_content())
+    stale_paths: list[Path] = []
+    collision_paths: list[Path] = []
+    for install_target in install_targets:
+        path: Path = install_target.path
+        if not path.exists():
+            stale_paths.append(path)
+            continue
+        existing_content: str = path.read_text(encoding="utf-8")
+        if existing_content == expected_content:
+            continue
+        if not any(marker in existing_content for marker in generated_markers):
+            collision_paths.append(path)
+            continue
+        stale_paths.append(path)
+
+    if settings.auto_update and stale_paths:
+        for path in stale_paths:
+            write_skill_file(path=path, content=expected_content)
+        message: str = "\nUpdated stale SQLBuild skill files:\n" + "".join(
+            f"  {path}\n" for path in stale_paths
+        )
+        if collision_paths:
+            message += _stale_skill_message(collision_paths=collision_paths)
+        return SkillMaintenanceResult(message=message)
+    if stale_paths or collision_paths:
+        return SkillMaintenanceResult(message=_stale_skill_message(collision_paths=collision_paths))
+    return SkillMaintenanceResult()
+
+
+def _stale_skill_message(*, collision_paths: list[Path]) -> str:
+    collision_detail: str = ""
+    if collision_paths:
+        collision_detail = "  Custom files were not overwritten:\n" + "".join(
+            f"    {path}\n" for path in collision_paths
+        )
+    return f"\nSQLBuild skill files are out of date\n{collision_detail}  Run: sqb skills\n"
+
+
+def _local_install_path(*, project_dir: Path, target_name: str) -> Path:
+    return build_install_targets(
+        project_dir=project_dir,
+        target_names=(target_name,),
+    )[0].path
 
 
 def build_install_targets(
@@ -163,6 +255,8 @@ def load_packaged_skill_content() -> str:
 def ensure_generated_marker(content: str) -> str:
     if generated_marker in content:
         return content
+    if legacy_generated_marker in content:
+        return content.replace(legacy_generated_marker, generated_marker, 1)
     if content.startswith("---\n"):
         frontmatter_end: int = content.find("\n---\n", 4)
         if frontmatter_end != -1:
@@ -174,7 +268,7 @@ def ensure_generated_marker(content: str) -> str:
 def write_skill_file(*, path: Path, content: str, force: bool = False) -> None:
     if path.exists():
         existing_content: str = path.read_text(encoding="utf-8")
-        if generated_marker not in existing_content and not force:
+        if not any(marker in existing_content for marker in generated_markers) and not force:
             raise CliUserError(
                 f"refusing to overwrite non-generated skill file: {path}",
                 code="C806",
@@ -182,4 +276,13 @@ def write_skill_file(*, path: Path, content: str, force: bool = False) -> None:
             )
 
     path.parent.mkdir(parents=True, exist_ok=True)
-    path.write_text(content, encoding="utf-8")
+    with NamedTemporaryFile(
+        mode="w",
+        encoding="utf-8",
+        dir=path.parent,
+        prefix=f".{path.name}.",
+        delete=False,
+    ) as temporary_file:
+        _ = temporary_file.write(content)
+        temporary_path: Path = Path(temporary_file.name)
+    replace(temporary_path, path)

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/classes/cli_namespace.py

@@ -115,7 +115,6 @@ _DEFAULT_VALUES: dict[str, object] = {
     "scenario_max_snapshot_total_rows": None,
     "scenario_max_snapshot_bytes": None,
     "scenario_max_snapshot_total_bytes": None,
-    "skills_command": None,
     "skills_global": False,
     "skills_target": [],
     "skills_force": False,
@@ -258,7 +257,6 @@ class CliNamespace:
     scenario_max_snapshot_total_rows: int | None
     scenario_max_snapshot_bytes: int | None
     scenario_max_snapshot_total_bytes: int | None
-    skills_command: str | None
     skills_global: bool
     skills_target: list[str]
     skills_force: bool

{sqlbuild-0.63.8 → sqlbuild-0.65.0}/src/sqlbuild/cli/commands/constants.py

@@ -13,7 +13,6 @@ EMPTY_ENV_VALUE: str = ""
 NO_COLOR_OPTION: str = "--no-color"
 DEBUG_OPTION: str = "--debug"
 DBT_INIT_COMMAND: str = "init"
-SKILLS_UPDATE_COMMAND: str = "update"
 SCENARIO_TEST_COMMAND: str = "test"
 SCENARIO_CAPTURE_COMMAND: str = "capture"
 COLUMN_LINEAGE_MODE_VALUES: tuple[str, ...] = tuple(mode.value for mode in ColumnLineageMode)