gitflow-analytics 3.13.22__tar.gz → 3.13.24__tar.gz

{gitflow_analytics-3.13.22/src/gitflow_analytics.egg-info → gitflow_analytics-3.13.24}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitflow-analytics
-Version: 3.13.22
+Version: 3.13.24
 Summary: Analyze Git repositories for developer productivity insights
 Author-email: Bob Matyas <bobmatnyc@gmail.com>
 License: MIT

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24}/src/gitflow_analytics/_version.py

@@ -1,4 +1,4 @@
 """Version information for gitflow-analytics."""
 
-__version__ = "3.13.22"
+__version__ = "3.13.24"
 __version_info__ = tuple(int(x) for x in __version__.split("."))

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24}/src/gitflow_analytics/config/loader.py

@@ -740,6 +740,7 @@ class ConfigLoader(ConfigLoaderSectionsMixin):
             position=_td_raw.get("position", _td_defaults.position),
             patterns=dict(_td_raw.get("patterns", _td_defaults.patterns)),
             exclude_patterns=list(_td_raw.get("exclude_patterns", _td_defaults.exclude_patterns)),
+            exclude_repos=list(_td_raw.get("exclude_repos", _td_defaults.exclude_repos)),
         )
 
         return AnalysisConfig(

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24}/src/gitflow_analytics/config/schema.py

@@ -360,6 +360,8 @@ class TicketDetectionConfig:
               - "CVE-\\d+"
               - "CWE-\\d+"
               - "\\d{8,}"
+            exclude_repos:
+              - "APEX"          # knowledge base — exclude from ticket compliance
     """
 
     # Which commits to scan.
@@ -397,6 +399,13 @@ class TicketDetectionConfig:
         ]
     )
 
+    # Repository names to exclude from ticket compliance metrics.
+    # Commits from these repos are still included in activity reporting
+    # (commit counts, line counts, developer stats) but are excluded from
+    # ticket coverage rate calculations.  Useful for knowledge-base or
+    # documentation repos where JIRA tickets are not expected.
+    exclude_repos: list[str] = field(default_factory=list)
+
 
 @dataclass
 class AnalysisConfig:

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24}/src/gitflow_analytics/extractors/tickets.py

@@ -149,6 +149,11 @@ class TicketExtractor(TicketAnalysisMixin):
         )
         self._compiled_excludes: list[re.Pattern[str]] = [re.compile(p) for p in raw_excludes]
 
+        # Repository names to exclude from ticket compliance metrics.
+        self._exclude_repos: set[str] = (
+            set(ticket_detection_config.exclude_repos) if ticket_detection_config else set()
+        )
+
         # If config supplies custom per-platform patterns, use them; otherwise fall back
         # to the built-in defaults.  The config patterns dict maps platform -> single
         # pattern string; the built-in defaults use lists.
@@ -202,7 +207,7 @@ class TicketExtractor(TicketAnalysisMixin):
         # Commit categorization patterns
         self.category_patterns = {
             "bug_fix": [
-                r"^fix(\([^)]*\))?:",  # Conventional commits: fix: or fix(scope):
+                r"^fix(\([^)]+\))?:",  # Conventional commits: fix: or fix(scope):
                 r"\b(fix|bug|error|issue|problem|crash|exception|failure)\b",
                 r"\b(resolve|solve|repair|correct|corrected|address)\b",
                 r"\b(hotfix|bugfix|patch|quickfix)\b",
@@ -217,9 +222,10 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\bfixed?\s+(issue|problem|bug|error)\b",
                 r"\bresolve[ds]?\s+(issue|problem|bug)\b",
                 r"\brepair\b",
+                r"\b(incorrect|wrong|invalid)\b",
             ],
             "feature": [
-                r"^(feat|feature)(\([^)]*\))?:",  # Conventional commits: feat: or feat(scope):
+                r"^(feat|feature)(\([^)]+\))?:",  # Conventional commits: feat: or feat(scope):
                 r"\b(add|new|feature|implement|create|build)\b",
                 r"\b(introduce|enhance|extend|expand)\b",
                 r"\b(functionality|capability|support|enable)\b",
@@ -233,9 +239,11 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(episode|episodes|audio|video)\s+(feature|support|implementation)\b",
                 r"\b(beacon)\s+(implementation|for|tracking)\b",
                 r"\b(localization)\s+(data|structure)\b",
+                r"\b(extract|harness|scaffold|bootstrap|wire|hook)\b",
+                r"\b(disable|toggle|flag)\b",
             ],
             "refactor": [
-                r"^refactor(\([^)]*\))?:",  # Conventional commits: refactor: or refactor(scope):
+                r"^refactor(\([^)]+\))?:",  # Conventional commits: refactor: or refactor(scope):
                 r"\b(refactor|restructure|reorganize|cleanup|clean up)\b",
                 r"\b(optimize|improve|simplify|streamline)\b",
                 r"\b(rename|move|extract|consolidate)\b",
@@ -268,7 +276,7 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(doc|readme|changelog|docstring)\b.*\bcomments?\b",
             ],
             "deployment": [
-                r"^deploy:",
+                r"^deploy(\([^)]+\))?:",
                 r"\b(deploy|deployment|publish|rollout)\b",
                 r"\b(production|prod|staging|live)\b",
                 r"\b(go\s+live|launch|ship)\b",
@@ -290,6 +298,9 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(schema)\b(?!.*\b(test|spec)\b)",  # Schema but not test schemas
                 r"\bsanity\s+schema\b",
                 r"changing\s+(some)?\s*(user|role)\s+(roles?|permissions?)\b",
+                r"\b(npmrc|\.npmrc)\b",
+                r"\b(ECR)\b",
+                r"\b(GitHub\s+Actions|workflow)\b(?!.*\b(run|test)\b)",
             ],
             "content": [
                 r"\b(content|copy|text|wording|messaging)\b",
@@ -337,7 +348,7 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(improve|better)\s+(load|performance|speed)\b",
             ],
             "chore": [
-                r"^chore:",
+                r"^chore(\([^)]+\))?:",
                 r"\b(chore|cleanup|housekeeping|maintenance)\b",
                 r"\b(routine|regular|scheduled)\b",
                 r"\b(lint|linting|format|formatting|prettier)\b",
@@ -346,12 +357,17 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(sync|auto-sync)\b",
                 r"\b(script\s+update|merge\s+main)\b",
                 r"removes?\s+(console|debug|log)\b",
+                # PR / code review response patterns
+                r"^PR\s+comments?\s*$",
+                r"(?:address|respond|applied?)\s+(?:PR|review|feedback|comments?)",
+                r"suggestion\s+from\s+@?copilot",
+                r"^apply\s+suggestion\b",
             ],
             "wip": [
                 r"\b(wip|work\s+in\s+progress|temp|temporary|tmp)\b",
                 r"\b(draft|unfinished|partial|incomplete)\b",
                 r"\b(placeholder|todo|fixme)\b",
-                r"^wip:",
+                r"^wip(\([^)]+\))?:",
                 r"\b(experiment|experimental|poc|proof\s+of\s+concept)\b",
                 r"\b(temporary|temp)\s+(fix|solution|workaround)\b",
             ],
@@ -359,11 +375,11 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(version|bump|tag)\b",
                 r"\b(v\d+\.\d+|version\s+\d+|\d+\.\d+\.\d+)\b",
                 r"\b(major|minor|patch)\s+(version|release|bump)\b",
-                r"^(version|bump):",
+                r"^(version|bump)(\([^)]+\))?:",
                 r"\b(prepare\s+for\s+release|pre-release)\b",
             ],
             "maintenance": [
-                r"^chore(\([^)]*\))?:",  # Conventional commits: chore: or chore(scope):
+                r"^chore(\([^)]+\))?:",  # Conventional commits: chore: or chore(scope):
                 r"\b(update|upgrade|bump|maintenance|maint)\b",
                 r"\b(dependency|dependencies|package|packages)\b",
                 r"\b(npm\s+update|pip\s+install|yarn\s+upgrade)\b",
@@ -374,9 +390,13 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\b(test|testing)\s+(change|update|fix)\b",
                 r"\b(more|only)\s+(combo|beacon)\s+(hacking|fires?)\b",
                 r"adds?\s+(console|debug|log)\b",
+                # Revert commits
+                r"^revert\b",
+                r"\breverting\b",
+                r"\breverted?\s",
             ],
             "test": [
-                r"^test:",
+                r"^test(\([^)]+\))?:",
                 r"\b(test|testing|spec|unit\s+test|integration\s+test)\b",
                 r"\b(junit|pytest|mocha|jest|cypress|selenium)\b",
                 r"\b(mock|stub|fixture|factory)\b",
@@ -389,14 +409,14 @@ class TicketExtractor(TicketAnalysisMixin):
                 r"\btest\s+fix\b",
             ],
             "style": [
-                r"^style:",
+                r"^style(\([^)]+\))?:",
                 r"\b(format|formatting|style|lint|linting)\b",
                 r"\b(prettier|eslint|black|autopep8|rubocop)\b",
                 r"\b(whitespace|indentation|spacing|tabs)\b",
                 r"\b(code\s+style|consistent|standardize)\b",
             ],
             "build": [
-                r"^build:",
+                r"^build(\([^)]+\))?:",
                 r"\b(build|compile|bundle|webpack|rollup)\b",
                 r"\b(ci|cd|pipeline|workflow|github\s+actions)\b",
                 r"\b(docker|dockerfile|makefile|npm\s+scripts)\b",
@@ -591,23 +611,36 @@ class TicketExtractor(TicketAnalysisMixin):
             "maintenance",  # General maintenance terms
         ]
 
-        # First, check for conventional commit patterns (^prefix:) which have absolute priority
-        conventional_patterns = {
-            "chore": r"^chore:",
-            "style": r"^style:",
-            "bug_fix": r"^fix:",
-            "feature": r"^(feat|feature):",
-            "test": r"^test:",
-            "build": r"^build:",
-            "deployment": r"^deploy:",
-            "wip": r"^wip:",
-            "version": r"^(version|bump):",
-        }
+        # First, check for conventional commit patterns (^prefix: or ^prefix(scope):)
+        # which have absolute priority.  The optional scope group handles
+        # messages like "fix(searchlight): ..." or "feat(agents): ...".
+        # Use a list of tuples to avoid duplicate key issues (build vs ci)
+        conventional_patterns = [
+            ("chore", r"^chore(\([^)]+\))?:"),
+            ("style", r"^style(\([^)]+\))?:"),
+            ("bug_fix", r"^fix(\([^)]+\))?:"),
+            ("feature", r"^(feat|feature)(\([^)]+\))?:"),
+            ("test", r"^test(\([^)]+\))?:"),
+            ("build", r"^build(\([^)]+\))?:"),
+            ("build", r"^ci(\([^)]+\))?:"),
+            ("deployment", r"^deploy(\([^)]+\))?:"),
+            ("wip", r"^wip(\([^)]+\))?:"),
+            ("version", r"^(version|bump)(\([^)]+\))?:"),
+            ("documentation", r"^docs(\([^)]+\))?:"),
+            ("refactor", r"^refactor(\([^)]+\))?:"),
+            ("performance", r"^perf(\([^)]+\))?:"),
+        ]
 
-        for category, pattern in conventional_patterns.items():
+        for category, pattern in conventional_patterns:
             if re.match(pattern, message_lower):
                 return category
 
+        # Check for revert commits early — before general body patterns, because
+        # revert messages often contain words that match other categories
+        # (e.g. "Revert incorrect ..." would match bug_fix's "incorrect" keyword).
+        if re.match(r"^revert\b", message_lower):
+            return "maintenance"
+
         # Then check categories in priority order for non-conventional patterns
         for category in priority_order:
             if category in self.compiled_category_patterns:

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24}/src/gitflow_analytics/extractors/tickets_analysis.py

@@ -11,7 +11,6 @@ and on instance methods:
 """
 
 import logging
-import re
 from collections import defaultdict
 from datetime import timezone
 from typing import Any, cast
@@ -57,6 +56,27 @@ class TicketAnalysisMixin:
         untracked_commits: list[dict[str, Any]] = []
         ticket_summary: defaultdict[str, set[str]] = defaultdict(set)
 
+        # Filter out commits from repos excluded from ticket compliance metrics.
+        # These repos still appear in activity reporting but do not affect the
+        # ticket coverage denominator/numerator.
+        exclude_repos: set[str] = getattr(self, "_exclude_repos", set())
+        excluded_repo_commits = 0
+        if exclude_repos:
+            filtered_commits = []
+            for c in commits:
+                repo = (c.get("project_key") or "") if isinstance(c, dict) else ""
+                if repo in exclude_repos:
+                    excluded_repo_commits += 1
+                else:
+                    filtered_commits.append(c)
+            if excluded_repo_commits:
+                logger.info(
+                    "Excluded %d commits from ticket compliance (repos: %s)",
+                    excluded_repo_commits,
+                    ", ".join(sorted(exclude_repos)),
+                )
+            commits = filtered_commits
+
         results = {
             "total_commits": len(commits),
             "total_prs": len(prs),
@@ -65,6 +85,7 @@ class TicketAnalysisMixin:
             "ticket_platforms": ticket_platforms,
             "untracked_commits": untracked_commits,
             "ticket_summary": ticket_summary,
+            "excluded_repo_commits": excluded_repo_commits,
         }
 
         commits_analyzed = 0
@@ -234,6 +255,11 @@ class TicketAnalysisMixin:
         if not commits:
             return {}
 
+        # Filter out commits from repos excluded from ticket compliance.
+        exclude_repos: set[str] = getattr(self, "_exclude_repos", set())
+        if exclude_repos:
+            commits = [c for c in commits if (c.get("project_key") or "") not in exclude_repos]
+
         developer_commits: dict[str, int] = {}
         developer_with_tickets: dict[str, int] = {}
 
@@ -263,9 +289,9 @@ class TicketAnalysisMixin:
         for developer_id in developer_commits:
             total = developer_commits[developer_id]
             with_tickets = developer_with_tickets[developer_id]
-            coverage_by_developer[developer_id] = round(
-                (with_tickets / total) * 100, 1
-            ) if total > 0 else 0.0
+            coverage_by_developer[developer_id] = (
+                round((with_tickets / total) * 100, 1) if total > 0 else 0.0
+            )
 
         logger.debug(f"Calculated ticket coverage for {len(coverage_by_developer)} developers")
         return coverage_by_developer

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24/src/gitflow_analytics.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitflow-analytics
-Version: 3.13.22
+Version: 3.13.24
 Summary: Analyze Git repositories for developer productivity insights
 Author-email: Bob Matyas <bobmatnyc@gmail.com>
 License: MIT

{gitflow_analytics-3.13.22 → gitflow_analytics-3.13.24}/src/gitflow_analytics.egg-info/SOURCES.txt

@@ -375,6 +375,7 @@ tests/core/test_merge_commit_detection.py
 tests/core/test_progress.py
 tests/core/test_weekly_cache.py
 tests/extractors/__init__.py
+tests/extractors/test_commit_categorization.py
 tests/extractors/test_ticket_detection_config.py
 tests/integration/__init__.py
 tests/integration/conftest.py

gitflow_analytics-3.13.24/tests/extractors/test_commit_categorization.py

@@ -0,0 +1,243 @@
+"""Tests for commit message categorization improvements.
+
+Covers:
+- Conventional commit scopes: fix(scope):, feat(scope):, chore(scope):, etc.
+- Copilot/AI suggestion commits
+- PR/code review response commits
+- Revert commits
+- Broader keyword matching for feature, bug_fix, configuration
+- Additional conventional commit prefixes: docs:, refactor:, perf:, ci:
+"""
+
+import pytest
+
+from gitflow_analytics.extractors.tickets import TicketExtractor
+
+
+@pytest.fixture
+def extractor() -> TicketExtractor:
+    """Default extractor with no special config."""
+    return TicketExtractor()
+
+
+# ---------------------------------------------------------------------------
+# 1. Conventional commit scopes (the primary fix)
+# ---------------------------------------------------------------------------
+
+
+class TestConventionalCommitScopes:
+    """Conventional commits with scopes like fix(searchlight): must be classified."""
+
+    def test_fix_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("fix(searchlight): remove BUP2 references") == "bug_fix"
+
+    def test_feat_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("feat(agents): add YAML frontmatter") == "feature"
+
+    def test_chore_with_scope(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("chore(dead-code): remove stale ManageRatesPage")
+        assert result == "chore"
+
+    def test_test_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("test(auth): add login integration tests") == "test"
+
+    def test_build_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("build(docker): update base image") == "build"
+
+    def test_style_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("style(lint): fix eslint warnings") == "style"
+
+    def test_docs_with_scope(self, extractor: TicketExtractor) -> None:
+        assert (
+            extractor.categorize_commit("docs(readme): update installation guide")
+            == "documentation"
+        )
+
+    def test_refactor_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("refactor(api): simplify handler logic") == "refactor"
+
+    def test_perf_with_scope(self, extractor: TicketExtractor) -> None:
+        assert (
+            extractor.categorize_commit("perf(queries): optimize N+1 in user list") == "performance"
+        )
+
+    def test_ci_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("ci(github): add deploy workflow") == "build"
+
+    def test_deploy_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("deploy(prod): rollout v2.3.1") == "deployment"
+
+    def test_wip_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("wip(dashboard): partial chart component") == "wip"
+
+    def test_version_with_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("bump(deps): update lodash to 4.17.21") == "version"
+
+
+# ---------------------------------------------------------------------------
+# 2. Conventional commits WITHOUT scopes still work (backward compat)
+# ---------------------------------------------------------------------------
+
+
+class TestConventionalCommitsWithoutScope:
+    """Ensure plain prefix: patterns still classify correctly."""
+
+    def test_fix_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("fix: handle null pointer in user service") == "bug_fix"
+
+    def test_feat_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("feat: add user search endpoint") == "feature"
+
+    def test_chore_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("chore: update dependencies") == "chore"
+
+    def test_docs_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("docs: add API reference") == "documentation"
+
+    def test_refactor_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("refactor: extract validation module") == "refactor"
+
+    def test_perf_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("perf: cache database queries") == "performance"
+
+    def test_ci_no_scope(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("ci: add linting step to pipeline") == "build"
+
+
+# ---------------------------------------------------------------------------
+# 3. PR / code review / Copilot patterns
+# ---------------------------------------------------------------------------
+
+
+class TestPRAndCopilotPatterns:
+    """PR comments, code review responses, and Copilot suggestions."""
+
+    def test_pr_comments(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("PR comments") == "chore"
+
+    def test_pr_comment_singular(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("PR comment") == "chore"
+
+    def test_apply_suggestion_from_copilot(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("Apply suggestion from @Copilot") == "chore"
+
+    def test_apply_suggestion_copilot_lowercase(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("apply suggestion from copilot") == "chore"
+
+    def test_address_review_feedback(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("address review feedback") == "chore"
+
+    def test_address_pr_comments(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("address PR comments") == "chore"
+
+    def test_applied_review_comments(self, extractor: TicketExtractor) -> None:
+        assert extractor.categorize_commit("applied review comments") == "chore"
+
+
+# ---------------------------------------------------------------------------
+# 4. Revert commits
+# ---------------------------------------------------------------------------
+
+
+class TestRevertCommits:
+    """Revert commits starting with 'Revert' should classify as maintenance."""
+
+    def test_revert_at_start(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("Revert incorrect global ext variable bumps")
+        assert result == "maintenance"
+
+    def test_revert_with_quotes(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit('Revert "add experimental feature flag"')
+        assert result == "maintenance"
+
+    def test_revert_simple(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("Revert last commit")
+        assert result == "maintenance"
+
+    def test_revert_merge(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("revert merge of feature branch")
+        assert result == "maintenance"
+
+    def test_reverting_in_body_matches_maintenance(self, extractor: TicketExtractor) -> None:
+        """'reverting' keyword in body (not at start) still matches via body patterns."""
+        result = extractor.categorize_commit("started reverting the old helper code")
+        assert result == "maintenance"
+
+
+# ---------------------------------------------------------------------------
+# 5. Broader feature keyword matching
+# ---------------------------------------------------------------------------
+
+
+class TestBroaderFeatureKeywords:
+    """Feature category should catch extraction/scaffolding/wiring work."""
+
+    def test_extracting_harness(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("start extracting first stage harness")
+        assert result == "feature"
+
+    def test_scaffold_component(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("scaffold the new dashboard component")
+        assert result == "feature"
+
+    def test_bootstrap_service(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("bootstrap payment service skeleton")
+        assert result == "feature"
+
+    def test_wire_up_endpoint(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("wire up the health check endpoint")
+        assert result == "feature"
+
+    def test_toggle_feature(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("toggle dark mode in settings")
+        assert result == "feature"
+
+    def test_disable_legacy_flow(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("disable legacy checkout flow")
+        assert result == "feature"
+
+
+# ---------------------------------------------------------------------------
+# 6. Configuration patterns
+# ---------------------------------------------------------------------------
+
+
+class TestConfigurationPatterns:
+    """Configuration category should catch ECR, npmrc, GitHub Actions."""
+
+    def test_npmrc_config_update(self, extractor: TicketExtractor) -> None:
+        """When message does not contain feature-triggering words like 'add'."""
+        result = extractor.categorize_commit("update .npmrc for private registry")
+        assert result == "configuration"
+
+    def test_ecr_config(self, extractor: TicketExtractor) -> None:
+        """ECR without security-triggering words like 'token'."""
+        result = extractor.categorize_commit("ECR registry configuration change")
+        assert result == "configuration"
+
+    def test_github_actions_workflow(self, extractor: TicketExtractor) -> None:
+        """GitHub Actions matches build (ci/pipeline patterns) due to priority."""
+        result = extractor.categorize_commit("update GitHub Actions deploy workflow")
+        # "workflow" and "github actions" match build category (ci/pipeline patterns)
+        assert result == "build"
+
+
+# ---------------------------------------------------------------------------
+# 7. Bug fix broader patterns
+# ---------------------------------------------------------------------------
+
+
+class TestBugFixBroaderPatterns:
+    """Bug fix category catches incorrect/wrong/invalid keywords."""
+
+    def test_incorrect_variable(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("incorrect global ext variable bumps")
+        assert result == "bug_fix"
+
+    def test_wrong_import(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("wrong import path for utils")
+        assert result == "bug_fix"
+
+    def test_invalid_json(self, extractor: TicketExtractor) -> None:
+        result = extractor.categorize_commit("invalid JSON response from API")
+        assert result == "bug_fix"