chore: Clean up imports and remove unused __all__ export list

Removed the `__all__` list from changelog_helper.py as it was added for
backward compatibility during refactoring but is not actually needed -
the module's public API is naturally exposed through its classes and
top-level functions.

Also cleaned up imports:
- Removed unused dateutil import
- Reordered imports for better organization
- Fixed formatting in utils/constants.py

No functional changes - purely code cleanup and formatting.

Author: jondricek

mitreattack/diffStix/changelog_helper.py

@@ -15,7 +15,6 @@
 import markdown
 import requests
 import stix2
-from dateutil import parser as dateparser
 from deepdiff import DeepDiff
 from loguru import logger
 from requests.adapters import HTTPAdapter, Retry
@@ -24,22 +23,10 @@
 from tqdm import tqdm
 
 from mitreattack import release_info
-from mitreattack.stix20 import MitreAttackData
 
 # Import from new utility modules
 from mitreattack.diffStix.core.contributor_tracker import ContributorTracker
-from mitreattack.diffStix.utils.constants import DATE as date
-from mitreattack.diffStix.utils.constants import THIS_MONTH as this_month
 from mitreattack.diffStix.utils.constants import LAYER_DEFAULTS as layer_defaults
-from mitreattack.diffStix.utils.version_utils import (
-    AttackObjectVersion,
-    get_attack_object_version,
-    is_major_version_change,
-    is_minor_version_change,
-    is_other_version_change,
-    is_patch_change,
-    version_increment_is_valid,
-)
 from mitreattack.diffStix.utils.stix_utils import (
     cleanup_values,
     deep_copy_stix,
@@ -51,34 +38,16 @@
     get_relative_data_component_url,
     get_relative_url_from_stix,
 )
-
-# Re-export imported functions for backward compatibility
-__all__ = [
-    "DiffStix",
-    "DomainStatistics",
-    "AttackChangesEncoder",
-    "AttackObjectVersion",
-    "get_attack_object_version",
-    "is_major_version_change",
-    "is_minor_version_change",
-    "is_other_version_change",
-    "is_patch_change",
-    "version_increment_is_valid",
-    "cleanup_values",
-    "deep_copy_stix",
-    "get_attack_id",
-    "has_subtechniques",
-    "resolve_datacomponent_parent",
-    "get_relative_data_component_url",
-    "get_relative_url_from_stix",
-    "get_placard_version_string",
-    "markdown_to_html",
-    "layers_dict_to_files",
-    "write_detailed_html",
-    "get_parsed_args",
-    "get_new_changelog_md",
-    "main",
-]
+from mitreattack.diffStix.utils.version_utils import (
+    AttackObjectVersion,
+    get_attack_object_version,
+    is_major_version_change,
+    is_minor_version_change,
+    is_other_version_change,
+    is_patch_change,
+    version_increment_is_valid,
+)
+from mitreattack.stix20 import MitreAttackData
 
 
 @dataclass

mitreattack/diffStix/utils/constants.py

@@ -63,6 +63,7 @@
     "unchanged": "ATT&CK objects which did not change between the two versions.",
 }
 
+
 # Section headers by object type
 def get_section_headers(object_type: str) -> dict:
     """Get section headers for a specific object type.
@@ -89,6 +90,7 @@ def get_section_headers(object_type: str) -> dict:
         "unchanged": "Unchanged",
     }
 
+
 # Navigator layer colors for different change types
 LAYER_COLORS = {
     "additions": "#a1d99b",