feat(config): 支持当行注释 Add JSON comment support with # and // styles for configuration files (NewFuture#515)

* Add JSON comment support with # and // styles

Co-authored-by: NewFuture <6290356+NewFuture@users.noreply.github.com>

* Fix Python 2.7 compatibility and remove trailing whitespace

Co-authored-by: NewFuture <6290356+NewFuture@users.noreply.github.com>

* Fix lint issues and simplify comment removal logic

Co-authored-by: NewFuture <6290356+NewFuture@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: NewFuture <6290356+NewFuture@users.noreply.github.com>
Co-authored-by: New Future <NewFuture@users.noreply.github.com>

Author: Copilot

ddns/cache.py

@@ -69,7 +69,7 @@ def sync(self):
             with open(self.__filename, "w") as data:
                 # 只保存非私有字段（不以__开头的字段）
                 filtered_data = {k: v for k, v in super(Cache, self).items() if not k.startswith("__")}
-                dump(filtered_data, data, separators=(',', ':'))
+                dump(filtered_data, data, separators=(",", ":"))
                 self.__logger.debug("save cache data to %s", self.__filename)
             self.__time = time()
             self.__changed = False

ddns/config/file.py

@@ -7,6 +7,7 @@
 from io import open
 from json import loads as json_decode, dumps as json_encode
 from sys import stderr, stdout
+from ..util.comment import remove_comment
 
 
 def load_config(config_path):
@@ -31,9 +32,11 @@ def load_config(config_path):
     except Exception as e:
         stderr.write("Failed to load config file `%s`: %s\n" % (config_path, e))
         raise
-    # 优先尝试JSON解析
+    # 移除注释后尝试JSON解析
     try:
-        config = json_decode(content)
+        # 移除单行注释（# 和 // 风格）
+        content_without_comments = remove_comment(content)
+        config = json_decode(content_without_comments)
     except (ValueError, SyntaxError) as json_error:
         # JSON解析失败，尝试AST解析
         try:

ddns/util/comment.py

@@ -0,0 +1,88 @@
+# -*- coding:utf-8 -*-
+"""
+Comment removal utility for JSON configuration files.
+Supports both # and // style single line comments.
+@author: GitHub Copilot
+"""
+
+
+def remove_comment(content):
+    # type: (str) -> str
+    """
+    移除字符串中的单行注释。
+    支持 # 和 // 两种注释风格。
+
+    Args:
+        content (str): 包含注释的字符串内容
+
+    Returns:
+        str: 移除注释后的字符串
+
+    Examples:
+        >>> remove_comment('{"key": "value"} // comment')
+        '{"key": "value"} '
+        >>> remove_comment('# This is a comment\\n{"key": "value"}')
+        '\\n{"key": "value"}'
+    """
+    if not content:
+        return content
+
+    lines = content.splitlines()
+    cleaned_lines = []
+
+    for line in lines:
+        # 移除行内注释，但要小心不要破坏字符串内的内容
+        cleaned_line = _remove_line_comment(line)
+        cleaned_lines.append(cleaned_line)
+
+    return "\n".join(cleaned_lines)
+
+
+def _remove_line_comment(line):
+    # type: (str) -> str
+    """
+    移除单行中的注释部分。
+
+    Args:
+        line (str): 要处理的行
+
+    Returns:
+        str: 移除注释后的行
+    """
+    # 检查是否是整行注释
+    stripped = line.lstrip()
+    if stripped.startswith("#") or stripped.startswith("//"):
+        return ""
+
+    # 查找行内注释，需要考虑字符串内容
+    in_string = False
+    quote_char = None
+    i = 0
+
+    while i < len(line):
+        char = line[i]
+
+        # 处理字符串内的转义序列
+        if in_string and char == "\\" and i + 1 < len(line):
+            i += 2  # 跳过转义字符
+            continue
+
+        # 处理引号字符
+        if char in ('"', "'"):
+            if not in_string:
+                in_string = True
+                quote_char = char
+            elif char == quote_char:
+                in_string = False
+                quote_char = None
+
+        # 在字符串外检查注释标记
+        elif not in_string:
+            if char == "#":
+                return line[:i].rstrip()
+            elif char == "/" and i + 1 < len(line) and line[i + 1] == "/":
+                return line[:i].rstrip()
+
+        i += 1
+
+    return line

tests/test_cache.py

@@ -554,7 +554,7 @@ def test_cache_new_custom_path(self):
         # Clean up
         cache.close()
 
-    @patch('ddns.cache.time')
+    @patch("ddns.cache.time")
     def test_cache_new_outdated_cache(self, mock_time):
         """Test Cache.new with outdated cache file (>72 hours old)"""
         import logging
@@ -575,7 +575,7 @@ def test_cache_new_outdated_cache(self, mock_time):
         # Mock the file modification time to be 73 hours ago
         old_mtime = current_time - (73 * 3600)  # 73 hours ago
 
-        with patch('ddns.cache.stat') as mock_stat:
+        with patch("ddns.cache.stat") as mock_stat:
             mock_stat.return_value.st_mtime = old_mtime
             cache = Cache.new(self.cache_file, "test_hash", logger)
 
@@ -608,7 +608,7 @@ def test_cache_new_empty_cache(self):
         # Clean up
         cache.close()
 
-    @patch('ddns.cache.time')
+    @patch("ddns.cache.time")
     def test_cache_new_valid_cache(self, mock_time):
         """Test Cache.new with valid cache file with data"""
         import logging
@@ -619,7 +619,7 @@ def test_cache_new_valid_cache(self, mock_time):
         # Create a cache file with test data
         test_data = {
             "domain1.com": {"ip": "1.2.3.4", "timestamp": 1234567890},
-            "domain2.com": {"ip": "5.6.7.8", "timestamp": 1234567891}
+            "domain2.com": {"ip": "5.6.7.8", "timestamp": 1234567891},
         }
         with open(self.cache_file, "w") as f:
             json.dump(test_data, f)
@@ -631,7 +631,7 @@ def test_cache_new_valid_cache(self, mock_time):
         # Mock file modification time to be recent (within 72 hours)
         recent_mtime = current_time - (24 * 3600)  # 24 hours ago
 
-        with patch('ddns.cache.stat') as mock_stat:
+        with patch("ddns.cache.stat") as mock_stat:
             mock_stat.return_value.st_mtime = recent_mtime
             cache = Cache.new(self.cache_file, "test_hash", logger)
 

tests/test_config_file.py

@@ -452,6 +452,52 @@ def test_load_config_file_encoding_utf8(self):
         self.assertEqual(loaded_config["symbols"], "αβγδε")
         self.assertEqual(loaded_config["emoji"], "🌍🔧⚡")
 
+    def test_load_config_json_with_hash_comments(self):
+        """测试加载带有 # 注释的JSON配置文件"""
+        json_with_comments = """{
+    # Configuration for DDNS
+    "dns": "cloudflare",  # DNS provider
+    "id": "test@example.com",
+    "token": "secret123",  # API token
+    "ttl": 300
+    # End of config
+}"""
+        file_path = self.create_test_file("test_hash_comments.json", json_with_comments)
+
+        config = load_config(file_path)
+
+        expected = {"dns": "cloudflare", "id": "test@example.com", "token": "secret123", "ttl": 300}
+        self.assertEqual(config, expected)
+
+    def test_load_config_json_with_double_slash_comments(self):
+        """测试加载带有 // 注释的JSON配置文件"""
+        json_with_comments = """{
+    // Configuration for DDNS
+    "$schema": "https://ddns.newfuture.cc/schema/v4.0.json", // Schema validation
+    "debug": false,  // false=disable, true=enable
+    "dns": "dnspod_com",  // DNS provider
+    "id": "1008666",
+    "token": "ae86$cbbcctv666666666666666",  // API Token
+    "ipv4": ["test.lorzl.ml"],  // IPv4 domains
+    "ipv6": ["test.lorzl.ml"],  // IPv6 domains
+    "proxy": null  // Proxy settings
+}"""
+        file_path = self.create_test_file("test_double_slash_comments.json", json_with_comments)
+
+        config = load_config(file_path)
+
+        expected = {
+            "$schema": "https://ddns.newfuture.cc/schema/v4.0.json",
+            "debug": False,
+            "dns": "dnspod_com",
+            "id": "1008666",
+            "token": "ae86$cbbcctv666666666666666",
+            "ipv4": ["test.lorzl.ml"],
+            "ipv6": ["test.lorzl.ml"],
+            "proxy": None,
+        }
+        self.assertEqual(config, expected)
+
     def test_save_config_pretty_format(self):
         """Test that saved JSON is properly formatted"""
         config_data = {"dns": "cloudflare", "log_level": "DEBUG", "log_file": "/var/log/ddns.log"}