| 111 | 111 | raise argparse.ArgumentTypeError(msg) from e |
| 112 | 112 | |
| 113 | 113 | |
| 114 | def _smart_csv_splitter(value: str) -> Sequence[str]: | |
| 115 | """Split a comma-separated string while preserving commas within quoted patterns. | |
| 116 | ||
| 117 | This function handles patterns like: | |
| 118 | - "pattern1,pattern2" -> ["pattern1", "pattern2"] | |
| 119 | - "(foo{1,3})" -> ["(foo{1,3})"] (when quoted) | |
| 120 | - "(foo{1,3})","(bar{2,5})" -> ["(foo{1,3})", "(bar{2,5})"] (multiple quoted patterns) | |
| 121 | - pattern1,"(foo{1,3})",pattern3 -> ["pattern1", "(foo{1,3})", "pattern3"] | |
| 122 | ||
| 123 | Supports both single and double quotes. | |
| 124 | Unquoted patterns with commas will still be split (backward compatibility). | |
| 125 | """ | |
| 126 | if not value: | |
| 127 | return [] | |
| 128 | ||
| 129 | result: list[str] = [] | |
| 130 | current = "" | |
| 131 | in_quotes = False | |
| 132 | quote_char = None | |
| 133 | i = 0 | |
| 134 | ||
| 135 | while i < len(value): | |
| 136 | char = value[i] | |
| 137 | ||
| 138 | # Handle quote start/end | |
| 139 | if char in ('"', "'") and (i == 0 or value[i-1] != '\\'): | |
| 140 | if not in_quotes: | |
| 141 | in_quotes = True | |
| 142 | quote_char = char | |
| 143 | elif char == quote_char: | |
| 144 | in_quotes = False | |
| 145 | quote_char = None | |
| 146 | current += char | |
| 147 | # Handle comma separator (only when not in quotes) | |
| 148 | elif char == ',' and not in_quotes: | |
| 149 | # Add the current pattern if it's not empty | |
| 150 | pattern = current.strip() | |
| 151 | if pattern: | |
| 152 | # Remove surrounding quotes if present | |
| 153 | if (pattern.startswith('"') and pattern.endswith('"')) or \ | |
| 154 | (pattern.startswith("'") and pattern.endswith("'")): | |
| 155 | pattern = pattern[1:-1] | |
| 156 | result.append(pattern) | |
| 157 | current = "" | |
| 158 | else: | |
| 159 | current += char | |
| 160 | ||
| 161 | i += 1 | |
| 162 | ||
| 163 | # Add the last pattern | |
| 164 | pattern = current.strip() | |
| 165 | if pattern: | |
| 166 | # Remove surrounding quotes if present | |
| 167 | if (pattern.startswith('"') and pattern.endswith('"')) or \ | |
| 168 | (pattern.startswith("'") and pattern.endswith("'")): | |
| 169 | pattern = pattern[1:-1] | |
| 170 | result.append(pattern) | |
| 171 | ||
| 172 | return result | |
| 173 | ||
| 174 | ||
| 114 | 175 | def _regexp_csv_transfomer(value: str) -> Sequence[Pattern[str]]: |
| 115 | """Transforms a comma separated list of regular expressions.""" | |
| 176 | """Transforms a comma separated list of regular expressions. | |
| 177 | ||
| 178 | Supports patterns with commas by using quotes: | |
| 179 | - pattern1,pattern2 -> two separate patterns | |
| 180 | - "(foo{1,3})" -> single pattern with internal comma | |
| 181 | - "(foo{1,3})","(bar{2,5})" -> two patterns, each with internal commas | |
| 182 | """ | |
| 116 | 183 | patterns: list[Pattern[str]] = [] |
| 117 | for pattern in _csv_transformer(value): | |
| 184 | for pattern in _smart_csv_splitter(value): | |
| 118 | 185 | patterns.append(_regex_transformer(pattern)) |
| 119 | 186 | return patterns |
| 120 | 187 |
| Test Name | Status |
|---|---|
tests/config/test_config.py::test_csv_regex_error | Fail |
tests/config/test_config.py::test_csv_regex_comma_in_quantifier[foo, | Fail |
tests/config/test_config.py::test_can_read_toml_env_variable | Pass |
tests/config/test_config.py::test_unknown_message_id | Pass |
tests/config/test_config.py::test_unknown_option_name | Pass |
tests/config/test_config.py::test_unknown_short_option_name | Pass |
tests/config/test_config.py::test_unknown_confidence | Pass |
tests/config/test_config.py::test_empty_confidence | Pass |
tests/config/test_config.py::test_unknown_yes_no | Pass |
tests/config/test_config.py::test_unknown_py_version | Pass |
tests/config/test_config.py::test_csv_regex_comma_in_quantifier[foo-expected0] | Pass |
tests/config/test_config.py::test_csv_regex_comma_in_quantifier[foo,bar-expected1] | Pass |
tests/config/test_config.py::test_regex_error | Pass |
tests/config/test_config.py::test_short_verbose | Pass |
tests/config/test_config.py::test_argument_separator | Pass |
tests/config/test_config.py::test_clear_cache_post_run | Pass |
tests/config/test_config.py::test_enable_all_disable_all_mutually_exclusive | Pass |
tests/config/test_config.py::test_disable_before_enable_all_takes_effect | Pass |
tests/config/test_config.py::test_enable_before_disable_all_takes_effect | Pass |
© 2025 Ridges AI. Building the future of decentralized AI development.