fix: Address Singer SDK import deprecation warnings

Signed-off-by: Edgar Ramírez Mondragón <edgarrm358@gmail.com>

Author: edgarrmondragon

.pre-commit-config.yaml

@@ -1,5 +1,6 @@
 ci:
   autofix_prs: true
+  autofix_commit_msg: "style: pre-commit fixes"
   autoupdate_schedule: monthly
   autoupdate_commit_msg: 'chore: pre-commit autoupdate'
 
@@ -21,10 +22,10 @@ repos:
   - id: check-github-workflows
 
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: v0.14.14
+  rev: v0.15.0
   hooks:
   - id: ruff-check
-    args: [--fix, --exit-non-zero-on-fix]
+    args: [--fix, --exit-non-zero-on-fix, --show-fixes]
   - id: ruff-format
 
 - repo: https://github.com/pre-commit/mirrors-mypy
@@ -35,6 +36,11 @@ repos:
     - types-requests
 
 - repo: https://github.com/astral-sh/uv-pre-commit
-  rev: 0.9.28
+  rev: 0.10.0
   hooks:
   - id: uv-lock
+
+- repo: https://github.com/crate-ci/typos
+  rev: v1.43.4
+  hooks:
+  - id: typos

README.md

@@ -34,7 +34,7 @@ Built with the [Meltano Singer SDK](https://sdk.meltano.com).
 | validate_records           | False    | 1                             | Whether to validate the schema of the incoming streams.                                                                                                                                                                                                                                          |
 | stream_maps                | False    | None                          | Config object for stream maps capability. For more information check out [Stream Maps](https://sdk.meltano.com/en/latest/stream_maps.html).                                                                                                                                                      |
 | stream_map_config          | False    | None                          | User-defined config values to be used within map expressions.                                                                                                                                                                                                                                    |
-| faker_config               | False    | None                          | Config for the [`Faker`](https://faker.readthedocs.io/en/master/) instance variable `fake` used within map expressions. Only applicable if the plugin specifies `faker` as an addtional dependency (through the `singer-sdk` `faker` extra or directly).                                         |
+| faker_config               | False    | None                          | Config for the [`Faker`](https://faker.readthedocs.io/en/master/) instance variable `fake` used within map expressions. Only applicable if the plugin specifies `faker` as an additional dependency (through the `singer-sdk` `faker` extra or directly).                                         |
 | faker_config.seed          | False    | None                          | Value to seed the Faker generator for deterministic output: https://faker.readthedocs.io/en/master/#seeding-the-generator                                                                                                                                                                        |
 | faker_config.locale        | False    | None                          | One or more LCID locale strings to produce localized output for: https://faker.readthedocs.io/en/master/#localization                                                                                                                                                                            |
 | flattening_enabled         | False    | None                          | 'True' to enable schema flattening and automatically expand nested properties.                                                                                                                                                                                                                   |

pyproject.toml

@@ -42,7 +42,7 @@ target-snowflake = "target_snowflake.target:TargetSnowflake.cli"
 [dependency-groups]
 dev = [
     "coverage>=7.2.7",
-    "pytest>=8",
+    "pytest>=9",
     "pytest-xdist>=3.3.1",
     "singer-sdk[testing]",
 ]
@@ -83,5 +83,12 @@ required-imports = ["from __future__ import annotations"]
 [tool.ruff.lint.pydocstyle]
 convention = "google"
 
-[tool.pytest.ini_options]
-addopts = '--durations=10'
+[tool.pytest]
+addopts = [
+    "--durations=10",
+]
+filterwarnings = [
+    "error",
+    "once:`SQLConnector.connection` is deprecated:singer_sdk.helpers._compat.SingerSDKDeprecationWarning",
+]
+minversion = "9"

target_snowflake/connector.py

@@ -14,9 +14,8 @@
 import sqlalchemy.sql.type_api
 from cryptography.hazmat.backends import default_backend
 from cryptography.hazmat.primitives import serialization
-from singer_sdk.connectors import SQLConnector
-from singer_sdk.connectors.sql import FullyQualifiedName, JSONSchemaToSQL
 from singer_sdk.exceptions import ConfigValidationError
+from singer_sdk.sql.connector import FullyQualifiedName, JSONSchemaToSQL, SQLConnector
 from snowflake.sqlalchemy import URL
 from snowflake.sqlalchemy.base import SnowflakeIdentifierPreparer
 from snowflake.sqlalchemy.snowdialect import SnowflakeDialect

target_snowflake/sinks.py

@@ -7,14 +7,14 @@
 from urllib.parse import urlparse
 from uuid import uuid4
 
-from singer_sdk.batch import JSONLinesBatcher
+from singer_sdk.contrib.batch_encoder_jsonl import JSONLinesBatcher
 from singer_sdk.helpers._batch import (
     BaseBatchFileEncoding,
     BatchConfig,
     BatchFileFormat,
 )
 from singer_sdk.helpers._typing import conform_record_data_types
-from singer_sdk.sinks import SQLSink
+from singer_sdk.sql.sink import SQLSink
 from snowflake.sqlalchemy.base import SnowflakeIdentifierPreparer
 from snowflake.sqlalchemy.snowdialect import SnowflakeDialect
 

target_snowflake/target.py

@@ -6,7 +6,7 @@
 
 import click
 from singer_sdk import typing as th
-from singer_sdk.target_base import SQLTarget
+from singer_sdk.sql.target import SQLTarget
 
 from target_snowflake.connector import DEFAULT_TIMESTAMP_TYPE, SnowflakeTimestampType
 from target_snowflake.initializer import initializer
@@ -124,7 +124,7 @@ class TargetSnowflake(SQLTarget):
     default_sink_class = SnowflakeSink
 
     @classmethod
-    def cb_inititalize(
+    def cb_initialize(
         cls: type[TargetSnowflake],
         ctx: click.Context,
         param: click.Option,  # noqa: ARG003
@@ -148,7 +148,7 @@ def get_singer_command(cls: type[TargetSnowflake]) -> click.Command:
                     ["--initialize"],
                     is_flag=True,
                     help="Interactive Snowflake account initialization.",
-                    callback=cls.cb_inititalize,
+                    callback=cls.cb_initialize,
                     expose_value=False,
                 ),
             ],

tests/batch.py

@@ -5,7 +5,7 @@
 import typing as t
 from pathlib import Path
 
-from singer_sdk.testing.suites import TestSuite
+from singer_sdk.testing.suites import SingerTestSuite
 from singer_sdk.testing.target_tests import (
     TargetNoPrimaryKeys,
     TargetSpecialCharsInAttributes,
@@ -174,7 +174,7 @@ class SnowflakeTargetBatchSpecialCharsInAttributes(
     name = "batch_special_chars_in_attributes"
 
 
-batch_target_tests = TestSuite(
+batch_target_tests = SingerTestSuite(
     kind="target",
     tests=[
         # BATCH

tests/core.py

@@ -4,7 +4,7 @@
 
 import pytest
 import snowflake.sqlalchemy.custom_types as sct
-from singer_sdk.testing.suites import TestSuite
+from singer_sdk.testing.suites import SingerTestSuite
 from singer_sdk.testing.target_tests import (
     TargetArrayData,
     TargetCamelcaseComplexSchema,
@@ -567,7 +567,7 @@ def singer_filepath(self) -> Path:
         return current_dir / "target_test_streams" / f"{self.name}.singer"
 
 
-target_tests = TestSuite(
+target_tests = SingerTestSuite(
     kind="target",
     tests=[
         # Core