fix: remove duplicated TestChangeStr class in test_diff.py (#16)

# Conflicts:
#	README.md

Author: Coding-Dev-Tools

.github/workflows/ci.yml

@@ -6,18 +6,23 @@ on:
   pull_request:
     branches: [main]
 
+permissions:
+  contents: read
+
 jobs:
   test:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.10", "3.11", "3.12"]
+        python-version: ["3.10", "3.11", "3.12", "3.13"]
 
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
 
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
 
@@ -26,8 +31,7 @@ jobs:
           pip install -e ".[dev]"
 
       - name: Lint with ruff
-        run: pip install ruff && ruff check src/ --target-version py310
+        run: ruff check src/
       - name: Run tests
         run: |
           python -m pytest tests/ -v --cov=src --cov-report=term-missing
-

.github/workflows/publish.yml

@@ -21,10 +21,10 @@ jobs:
       id-token: write
 
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
 
       - name: Set up Python 3.11
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v5
         with:
           python-version: "3.11"
 

CHANGELOG.md

@@ -2,6 +2,44 @@
 
 All notable changes to ConfigDrift CLI will be documented in this file.
 
+## [Unreleased]
+
+### Added
+
+- CLI test suite with comprehensive coverage for Change.\_\_str\_\_() and loader fallback paths
+- npm wrapper (`package.json` + `cli.js`) for npm publishing
+- GitHub Actions: npm publish workflow (release or manual dispatch)
+- GitHub Actions: GitHub Pages deployment workflow
+- `CONTRIBUTING.md` with development setup and PR guidelines
+- `SECURITY.md` with security policy
+- Homebrew and Scoop install methods
+- Directory listing badges: Open Source Alternative, LibHunt, Awesome Python
+- npm keywords optimized for discoverability (15 terms)
+- `revenueholdings-license` gating on all CLI commands
+- Beta badge and star CTA in README header
+
+### Changed
+
+- CI test matrix expanded to include Python 3.13
+- CI security hardened: `persist-credentials: false`, restricted permissions
+- Documentation branding updated from DevForge to Revenue Holdings
+- README expanded with CI/CD examples and alternatives comparison
+- README tool count updated (8 → 11)
+- npm install placement and formatting fixed in README
+- `project.urls` metadata added to `pyproject.toml`
+
+### Fixed
+
+- CI lint workflow: removed redundant ruff install and deprecated `--target-version` flag
+- GitHub Actions version mismatches in CI and publish workflows
+- UTF-8 encoding (mojibake) in file output
+- Ruff lint issues: `datetime.UTC`, `X | None` syntax, `E501`, `B904`, `F821`
+- Missing `ruff` dev dependency in `pyproject.toml`
+- Broken PyPI badges replaced with GitHub release badge (not yet on PyPI)
+- `revenueholdings-license` import made optional (fixes CI failures on open-source PRs)
+- Dependencies bumped via Dependabot (checkout@v6, setup-node@v6, setup-python@v6)
+- README install section formatting (broken Homebrew/Scoop code blocks)
+
 ## [0.1.0] — 2026-05-14
 
 ### Added
@@ -13,6 +51,6 @@ All notable changes to ConfigDrift CLI will be documented in this file.
 - `configdrift init` — generate `.configdrift.yaml` scaffolding
 - Three output formats: rich `table`, machine-readable `json`, `silent` (exit code only)
 - Severity-level drift classification: Info, Warning, Breaking
-- Breaking-change heuristics for critical keys (`database*`, `auth*`, `api_key*`, `secret*`, `password*`, `token*`, `endpoint*`)
+- Breaking-change heuristics for critical keys
 - CI/CD integration with non-zero exit on breaking drift
 - Python 3.10+ support

README.md

@@ -1,18 +1,17 @@
 # ConfigDrift
 
 [![GitHub stars](https://img.shields.io/github/stars/Coding-Dev-Tools/configdrift?style=social)](https://github.com/Coding-Dev-Tools/configdrift/stargazers)
-[![Awesome DevOps](https://img.shields.io/badge/Awesome_DevOps-Submitted-grey?logo=github)](https://github.com/wmariuss/awesome-devops)<!-- PR #433 -->
 
 Keep configurations consistent across all environments, automatically. ConfigDrift compares configs, flags drift, and reports compliance violations before they cause incidents.
 
+> ⭐ **Star this repo** if you manage multi-environment configs — it helps other devs find ConfigDrift!
+
 [![GitHub release](https://img.shields.io/github/v/release/Coding-Dev-Tools/configdrift?label=latest)](https://github.com/Coding-Dev-Tools/configdrift/releases)
 ![Python](https://img.shields.io/badge/python-3.10%2B-blue)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://github.com/Coding-Dev-Tools/configdrift/blob/main/LICENSE)
 [![Open Source Alternative](https://img.shields.io/badge/Open_Source_Alternative-%E2%87%92-blue?logo=opensourceinitiative)](https://www.opensourcealternative.to/project/configdrift)
 [![LibHunt](https://img.shields.io/badge/LibHunt-%E2%87%92-blue?logo=codeigniter)](https://www.libhunt.com/r/Coding-Dev-Tools/configdrift)
-[![Awesome Python](https://img.shields.io/badge/Awesome_Python-%E2%87%92-blue?logo=python)](https://github.com/uhub/awesome-python)
 
-**Why ConfigDrift?** Environments should behave consistently. When dev, staging, and prod configs diverge, deployments break silently. ConfigDrift compares configurations across environments, highlights drifting keys, deprecated values, and missing settings — before they cause incidents. Supports YAML, JSON, TOML, and .env — and it runs in CI so drift never ships.
 
 ## Installation
 
@@ -162,7 +161,7 @@ ConfigDrift is one of 11 tools in the DevForge suite. One license covers all CLI
 ---
 
 <p align="center">
-  <sub>Part of <a href="https://coding-dev-tools.github.io/devforge.dev/">DevForge</a> — CLI tools built by autonomous AI.</sub>
+  <sub>Part of <a href="https://coding-dev-tools.github.io/revenueholdings.dev/">Revenue Holdings</a> — CLI tools built by autonomous AI.</sub>
 </p>
 
 ## License

pyproject.toml

@@ -1,65 +1,68 @@
-[build-system]
-requires = ["setuptools>=68.0", "wheel"]
-build-backend = "setuptools.build_meta"
-
-[project]
-name = "configdrift"
-version = "0.1.0"
-description = "CLI tool that detects and fixes configuration file drift across environments (dev/staging/prod). Supports YAML, JSON, TOML, and .env formats."
-readme = "README.md"
-requires-python = ">=3.10"
-license = "MIT"
-authors = [{name = "Revenue Holdings"}]
-keywords = ["config", "drift", "diff", "env", "devops", "cli"]
-classifiers = [
-    "Development Status :: 4 - Beta",
-    "Intended Audience :: Developers",
-    "Topic :: Software Development :: Quality Assurance",
-    "Topic :: System :: Systems Administration",
-    "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
-    "Programming Language :: Python :: 3.12",
-]
-dependencies = [
-    "typer>=0.9.0",
-    "rich>=13.0.0",
-    "pyyaml>=6.0",
-    "tomli>=2.0.0; python_version < '3.11'",
-    "tomli-w>=1.0.0",
-]
-
-[project.optional-dependencies]
-license = ["revenueholdings-license>=0.1.0"]
-dev = [
-    "pytest>=7.0.0",
-    "pytest-cov>=4.0.0",
-    "ruff>=0.4.0",
-]
-toml = ["tomli>=2.0.0", "tomli-w>=1.0.0"]
-
-[project.urls]
-Homepage = "https://github.com/Coding-Dev-Tools/configdrift"
-Repository = "https://github.com/Coding-Dev-Tools/configdrift"
-"Issue Tracker" = "https://github.com/Coding-Dev-Tools/configdrift/issues"
-
-[project.scripts]
-configdrift = "configdrift.cli:app"
-
-[tool.setuptools.packages.find]
-where = ["src"]
-
-[tool.pytest.ini_options]
-testpaths = ["tests"]
-addopts = "-v --tb=short"
-
-[tool.ruff]
-target-version = "py310"
-line-length = 120
-
-[tool.ruff.lint]
-select = ["E", "F", "W", "I", "UP", "B", "SIM"]
-ignore = ["E501"]
-
-[tool.ruff.lint.isort]
+[build-system]
+requires = ["setuptools>=68.0", "wheel"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "configdrift"
+version = "0.1.0"
+description = "CLI tool that detects and fixes configuration file drift across environments (dev/staging/prod). Supports YAML, JSON, TOML, and .env formats."
+readme = "README.md"
+requires-python = ">=3.10"
+license = "MIT"
+authors = [{name = "Revenue Holdings"}]
+keywords = ["config", "drift", "diff", "env", "devops", "cli"]
+classifiers = [
+    "Development Status :: 4 - Beta",
+    "Intended Audience :: Developers",
+    "Topic :: Software Development :: Quality Assurance",
+    "Topic :: System :: Systems Administration",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+]
+dependencies = [
+    "typer>=0.9.0",
+    "rich>=13.0.0",
+    "pyyaml>=6.0",
+    "tomli>=2.0.0; python_version < '3.11'",
+    "tomli-w>=1.0.0",
+]
+
+[project.optional-dependencies]
+license = ["revenueholdings-license>=0.1.0"]
+dev = [
+    "pytest>=7.0.0",
+    "pytest-cov>=4.0.0",
+    "ruff>=0.4.0",
+]
+toml = ["tomli>=2.0.0", "tomli-w>=1.0.0"]
+
+[project.urls]
+Homepage = "https://github.com/Coding-Dev-Tools/configdrift"
+Documentation = "https://coding-dev-tools.github.io/configdrift"
+Repository = "https://github.com/Coding-Dev-Tools/configdrift"
+Issues = "https://github.com/Coding-Dev-Tools/configdrift/issues"
+Changelog = "https://github.com/Coding-Dev-Tools/configdrift/releases"
+
+[project.scripts]
+configdrift = "configdrift.cli:app"
+
+[tool.setuptools.packages.find]
+where = ["src"]
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+addopts = "-v --tb=short"
+
+[tool.ruff]
+target-version = "py310"
+line-length = 120
+
+[tool.ruff.lint]
+select = ["E", "F", "W", "I", "UP", "B", "SIM"]
+ignore = ["E501"]
+
+[tool.ruff.lint.isort]
 known-first-party = ["*"]

src/configdrift/cli.py

@@ -70,8 +70,6 @@ def check(
         raise typer.Exit(code=1)
 
     env_configs: dict[str, dict[str, Any]] = {}
-    env_labels = []
-
     env_labels = [baseline, target] if len(files) == 2 else [f"file_{i + 1}" for i in range(len(files))]
 
     for label, filepath in zip(env_labels, files, strict=False):

tests/test_cli.py

@@ -66,6 +66,21 @@ def test_check_error_file_not_found(self):
         assert result.exit_code == 1
         assert "Error loading" in result.stdout
 
+    def test_check_three_files(self):
+        """Compare three config files with auto-labeled environments."""
+        with tempfile.TemporaryDirectory() as tmpdir:
+            a = Path(tmpdir) / "a.yaml"
+            b = Path(tmpdir) / "b.yaml"
+            c = Path(tmpdir) / "c.yaml"
+            a.write_text(yaml.dump({"host": "localhost", "port": 8080}))
+            b.write_text(yaml.dump({"host": "staging.example.com", "port": 8080}))
+            c.write_text(yaml.dump({"host": "prod.example.com", "port": 443}))
+
+            result = runner.invoke(app, ["check", str(a), str(b), str(c)])
+            assert result.exit_code == 0
+            assert "file_1" in result.stdout or "Config Drift" in result.stdout
+            assert "prod.example.com" in result.stdout
+
     def test_check_less_than_two_files(self):
         """Only one file should error."""
         with tempfile.TemporaryDirectory() as tmpdir:

tests/test_diff.py

@@ -105,6 +105,24 @@ def test_three_envs(self):
         assert results["prod"].count == 1
 
 
+class TestChangeStr:
+    """Tests for Change.__str__()."""
+
+    def test_str_added(self):
+        c = Change(key="port", change_type=ChangeType.ADDED, new_value=8080, env="prod")
+        expected = "[+] port = 8080  (env: prod)"
+        assert str(c) == expected
+
+    def test_str_removed(self):
+        c = Change(key="port", change_type=ChangeType.REMOVED, old_value=8080, env="dev")
+        expected = "[-] port (was 8080)  (env: dev)"
+        assert str(c) == expected
+
+    def test_str_changed(self):
+        c = Change(key="host", change_type=ChangeType.CHANGED, old_value="localhost", new_value="prod.example.com", env="prod")
+        assert str(c) == "[~] host: 'localhost' \u2192 'prod.example.com'  (env: prod)"
+
+
 class TestDiffResultHelpers:
     def test_by_type(self):
         changes = [

tests/test_loader.py

@@ -85,6 +85,60 @@ def test_ignores_comments(self):
         assert "This is a comment" not in result
 
 
+class TestLoadUnsupported:
+    """Tests for fallback behaviour with unknown file extensions."""
+
+    def test_fallback_yaml_content_in_cfg(self):
+        """Files with .cfg containing valid YAML should be parsed via fallback."""
+        content = yaml.dump({"key": "value", "nested": {"inner": 42}})
+        with tempfile.NamedTemporaryFile(suffix=".cfg", mode="w", delete=False) as f:
+            f.write(content)
+            f.flush()
+            result = load_file(f.name)
+        os.unlink(f.name)
+        assert result["key"] == "value"
+        assert result["nested.inner"] == 42
+
+    def test_fallback_yaml_as_json(self):
+        """Files with unknown ext containing valid JSON should be parsed via fallback."""
+        content = json.dumps({"host": "localhost", "port": 8080})
+        with tempfile.NamedTemporaryFile(suffix=".cnf", mode="w", delete=False) as f:
+            f.write(content)
+            f.flush()
+            result = load_file(f.name)
+        os.unlink(f.name)
+        assert result["host"] == "localhost"
+        assert result["port"] == 8080
+
+    def test_unsupported_format_fallback_to_dotenv(self):
+        """Files with unknown extension and no structured content fall through to .env parser and returns a dict."""
+        with tempfile.NamedTemporaryFile(suffix=".xyz", mode="w", delete=False) as f:
+            f.write("not a config")
+            f.flush()
+            result = load_file(f.name)
+        os.unlink(f.name)
+        assert isinstance(result, dict)
+
+    def test_unsupported_format_empty_result(self):
+        """Garbage content with unknown extension returns empty dict via .env fallback."""
+        with tempfile.NamedTemporaryFile(suffix=".xyz", mode="w", delete=False) as f:
+            f.write("!!!garbage!!!")
+            f.flush()
+            result = load_file(f.name)
+        os.unlink(f.name)
+        assert result == {}
+
+    def test_yaml_non_dict_raises(self):
+        """YAML containing a list (not a mapping) should raise ValueError."""
+        content = yaml.dump(["item1", "item2"])
+        with tempfile.NamedTemporaryFile(suffix=".yaml", mode="w", delete=False) as f:
+            f.write(content)
+            f.flush()
+            with pytest.raises(ValueError, match="YAML file must contain a mapping"):
+                load_file(f.name)
+        os.unlink(f.name)
+
+
 class TestFlattenNested:
     def test_flatten(self):
         data = {"a": {"b": {"c": 1}, "d": 2}, "e": 3}