thailint 0.5.0__py3-none-any.whl → 0.15.3__py3-none-any.whl

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (204) hide show
  1. src/__init__.py +1 -0
  2. src/analyzers/__init__.py +4 -3
  3. src/analyzers/ast_utils.py +54 -0
  4. src/analyzers/rust_base.py +155 -0
  5. src/analyzers/rust_context.py +141 -0
  6. src/analyzers/typescript_base.py +4 -0
  7. src/cli/__init__.py +30 -0
  8. src/cli/__main__.py +22 -0
  9. src/cli/config.py +480 -0
  10. src/cli/config_merge.py +241 -0
  11. src/cli/linters/__init__.py +67 -0
  12. src/cli/linters/code_patterns.py +270 -0
  13. src/cli/linters/code_smells.py +342 -0
  14. src/cli/linters/documentation.py +83 -0
  15. src/cli/linters/performance.py +287 -0
  16. src/cli/linters/shared.py +331 -0
  17. src/cli/linters/structure.py +327 -0
  18. src/cli/linters/structure_quality.py +328 -0
  19. src/cli/main.py +120 -0
  20. src/cli/utils.py +395 -0
  21. src/cli_main.py +37 -0
  22. src/config.py +38 -25
  23. src/core/base.py +7 -2
  24. src/core/cli_utils.py +19 -2
  25. src/core/config_parser.py +5 -2
  26. src/core/constants.py +54 -0
  27. src/core/linter_utils.py +95 -6
  28. src/core/python_lint_rule.py +101 -0
  29. src/core/registry.py +1 -1
  30. src/core/rule_discovery.py +147 -84
  31. src/core/types.py +13 -0
  32. src/core/violation_builder.py +78 -15
  33. src/core/violation_utils.py +69 -0
  34. src/formatters/__init__.py +22 -0
  35. src/formatters/sarif.py +202 -0
  36. src/linter_config/directive_markers.py +109 -0
  37. src/linter_config/ignore.py +254 -395
  38. src/linter_config/loader.py +45 -12
  39. src/linter_config/pattern_utils.py +65 -0
  40. src/linter_config/rule_matcher.py +89 -0
  41. src/linters/collection_pipeline/__init__.py +90 -0
  42. src/linters/collection_pipeline/any_all_analyzer.py +281 -0
  43. src/linters/collection_pipeline/ast_utils.py +40 -0
  44. src/linters/collection_pipeline/config.py +75 -0
  45. src/linters/collection_pipeline/continue_analyzer.py +94 -0
  46. src/linters/collection_pipeline/detector.py +360 -0
  47. src/linters/collection_pipeline/filter_map_analyzer.py +402 -0
  48. src/linters/collection_pipeline/linter.py +420 -0
  49. src/linters/collection_pipeline/suggestion_builder.py +130 -0
  50. src/linters/cqs/__init__.py +54 -0
  51. src/linters/cqs/config.py +55 -0
  52. src/linters/cqs/function_analyzer.py +201 -0
  53. src/linters/cqs/input_detector.py +139 -0
  54. src/linters/cqs/linter.py +159 -0
  55. src/linters/cqs/output_detector.py +84 -0
  56. src/linters/cqs/python_analyzer.py +54 -0
  57. src/linters/cqs/types.py +82 -0
  58. src/linters/cqs/typescript_cqs_analyzer.py +61 -0
  59. src/linters/cqs/typescript_function_analyzer.py +192 -0
  60. src/linters/cqs/typescript_input_detector.py +203 -0
  61. src/linters/cqs/typescript_output_detector.py +117 -0
  62. src/linters/cqs/violation_builder.py +94 -0
  63. src/linters/dry/base_token_analyzer.py +16 -9
  64. src/linters/dry/block_filter.py +120 -20
  65. src/linters/dry/block_grouper.py +4 -0
  66. src/linters/dry/cache.py +104 -10
  67. src/linters/dry/cache_query.py +4 -0
  68. src/linters/dry/config.py +54 -11
  69. src/linters/dry/constant.py +92 -0
  70. src/linters/dry/constant_matcher.py +223 -0
  71. src/linters/dry/constant_violation_builder.py +98 -0
  72. src/linters/dry/duplicate_storage.py +5 -4
  73. src/linters/dry/file_analyzer.py +4 -2
  74. src/linters/dry/inline_ignore.py +7 -16
  75. src/linters/dry/linter.py +183 -48
  76. src/linters/dry/python_analyzer.py +60 -439
  77. src/linters/dry/python_constant_extractor.py +100 -0
  78. src/linters/dry/single_statement_detector.py +417 -0
  79. src/linters/dry/token_hasher.py +116 -112
  80. src/linters/dry/typescript_analyzer.py +68 -382
  81. src/linters/dry/typescript_constant_extractor.py +138 -0
  82. src/linters/dry/typescript_statement_detector.py +255 -0
  83. src/linters/dry/typescript_value_extractor.py +70 -0
  84. src/linters/dry/violation_builder.py +4 -0
  85. src/linters/dry/violation_filter.py +5 -4
  86. src/linters/dry/violation_generator.py +71 -14
  87. src/linters/file_header/atemporal_detector.py +68 -50
  88. src/linters/file_header/base_parser.py +93 -0
  89. src/linters/file_header/bash_parser.py +66 -0
  90. src/linters/file_header/config.py +90 -16
  91. src/linters/file_header/css_parser.py +70 -0
  92. src/linters/file_header/field_validator.py +36 -33
  93. src/linters/file_header/linter.py +140 -144
  94. src/linters/file_header/markdown_parser.py +130 -0
  95. src/linters/file_header/python_parser.py +14 -58
  96. src/linters/file_header/typescript_parser.py +73 -0
  97. src/linters/file_header/violation_builder.py +13 -12
  98. src/linters/file_placement/config_loader.py +3 -1
  99. src/linters/file_placement/directory_matcher.py +4 -0
  100. src/linters/file_placement/linter.py +66 -34
  101. src/linters/file_placement/pattern_matcher.py +41 -6
  102. src/linters/file_placement/pattern_validator.py +31 -12
  103. src/linters/file_placement/rule_checker.py +12 -7
  104. src/linters/lazy_ignores/__init__.py +43 -0
  105. src/linters/lazy_ignores/config.py +74 -0
  106. src/linters/lazy_ignores/directive_utils.py +164 -0
  107. src/linters/lazy_ignores/header_parser.py +177 -0
  108. src/linters/lazy_ignores/linter.py +158 -0
  109. src/linters/lazy_ignores/matcher.py +168 -0
  110. src/linters/lazy_ignores/python_analyzer.py +209 -0
  111. src/linters/lazy_ignores/rule_id_utils.py +180 -0
  112. src/linters/lazy_ignores/skip_detector.py +298 -0
  113. src/linters/lazy_ignores/types.py +71 -0
  114. src/linters/lazy_ignores/typescript_analyzer.py +146 -0
  115. src/linters/lazy_ignores/violation_builder.py +135 -0
  116. src/linters/lbyl/__init__.py +31 -0
  117. src/linters/lbyl/config.py +63 -0
  118. src/linters/lbyl/linter.py +67 -0
  119. src/linters/lbyl/pattern_detectors/__init__.py +53 -0
  120. src/linters/lbyl/pattern_detectors/base.py +63 -0
  121. src/linters/lbyl/pattern_detectors/dict_key_detector.py +107 -0
  122. src/linters/lbyl/pattern_detectors/division_check_detector.py +232 -0
  123. src/linters/lbyl/pattern_detectors/file_exists_detector.py +220 -0
  124. src/linters/lbyl/pattern_detectors/hasattr_detector.py +119 -0
  125. src/linters/lbyl/pattern_detectors/isinstance_detector.py +119 -0
  126. src/linters/lbyl/pattern_detectors/len_check_detector.py +173 -0
  127. src/linters/lbyl/pattern_detectors/none_check_detector.py +146 -0
  128. src/linters/lbyl/pattern_detectors/string_validator_detector.py +145 -0
  129. src/linters/lbyl/python_analyzer.py +215 -0
  130. src/linters/lbyl/violation_builder.py +354 -0
  131. src/linters/magic_numbers/context_analyzer.py +227 -225
  132. src/linters/magic_numbers/linter.py +28 -82
  133. src/linters/magic_numbers/python_analyzer.py +4 -16
  134. src/linters/magic_numbers/typescript_analyzer.py +9 -12
  135. src/linters/magic_numbers/typescript_ignore_checker.py +81 -0
  136. src/linters/method_property/__init__.py +49 -0
  137. src/linters/method_property/config.py +138 -0
  138. src/linters/method_property/linter.py +414 -0
  139. src/linters/method_property/python_analyzer.py +473 -0
  140. src/linters/method_property/violation_builder.py +119 -0
  141. src/linters/nesting/linter.py +24 -16
  142. src/linters/nesting/python_analyzer.py +4 -0
  143. src/linters/nesting/typescript_analyzer.py +6 -12
  144. src/linters/nesting/violation_builder.py +1 -0
  145. src/linters/performance/__init__.py +91 -0
  146. src/linters/performance/config.py +43 -0
  147. src/linters/performance/constants.py +49 -0
  148. src/linters/performance/linter.py +149 -0
  149. src/linters/performance/python_analyzer.py +365 -0
  150. src/linters/performance/regex_analyzer.py +312 -0
  151. src/linters/performance/regex_linter.py +139 -0
  152. src/linters/performance/typescript_analyzer.py +236 -0
  153. src/linters/performance/violation_builder.py +160 -0
  154. src/linters/print_statements/config.py +7 -12
  155. src/linters/print_statements/linter.py +26 -43
  156. src/linters/print_statements/python_analyzer.py +91 -93
  157. src/linters/print_statements/typescript_analyzer.py +15 -25
  158. src/linters/print_statements/violation_builder.py +12 -14
  159. src/linters/srp/class_analyzer.py +11 -7
  160. src/linters/srp/heuristics.py +56 -22
  161. src/linters/srp/linter.py +15 -16
  162. src/linters/srp/python_analyzer.py +55 -20
  163. src/linters/srp/typescript_metrics_calculator.py +110 -50
  164. src/linters/stateless_class/__init__.py +25 -0
  165. src/linters/stateless_class/config.py +58 -0
  166. src/linters/stateless_class/linter.py +349 -0
  167. src/linters/stateless_class/python_analyzer.py +290 -0
  168. src/linters/stringly_typed/__init__.py +36 -0
  169. src/linters/stringly_typed/config.py +189 -0
  170. src/linters/stringly_typed/context_filter.py +451 -0
  171. src/linters/stringly_typed/function_call_violation_builder.py +135 -0
  172. src/linters/stringly_typed/ignore_checker.py +100 -0
  173. src/linters/stringly_typed/ignore_utils.py +51 -0
  174. src/linters/stringly_typed/linter.py +376 -0
  175. src/linters/stringly_typed/python/__init__.py +33 -0
  176. src/linters/stringly_typed/python/analyzer.py +348 -0
  177. src/linters/stringly_typed/python/call_tracker.py +175 -0
  178. src/linters/stringly_typed/python/comparison_tracker.py +257 -0
  179. src/linters/stringly_typed/python/condition_extractor.py +134 -0
  180. src/linters/stringly_typed/python/conditional_detector.py +179 -0
  181. src/linters/stringly_typed/python/constants.py +21 -0
  182. src/linters/stringly_typed/python/match_analyzer.py +94 -0
  183. src/linters/stringly_typed/python/validation_detector.py +189 -0
  184. src/linters/stringly_typed/python/variable_extractor.py +96 -0
  185. src/linters/stringly_typed/storage.py +620 -0
  186. src/linters/stringly_typed/storage_initializer.py +45 -0
  187. src/linters/stringly_typed/typescript/__init__.py +28 -0
  188. src/linters/stringly_typed/typescript/analyzer.py +157 -0
  189. src/linters/stringly_typed/typescript/call_tracker.py +335 -0
  190. src/linters/stringly_typed/typescript/comparison_tracker.py +378 -0
  191. src/linters/stringly_typed/violation_generator.py +419 -0
  192. src/orchestrator/core.py +252 -14
  193. src/orchestrator/language_detector.py +5 -3
  194. src/templates/thailint_config_template.yaml +196 -0
  195. src/utils/project_root.py +3 -0
  196. thailint-0.15.3.dist-info/METADATA +187 -0
  197. thailint-0.15.3.dist-info/RECORD +226 -0
  198. thailint-0.15.3.dist-info/entry_points.txt +4 -0
  199. src/cli.py +0 -1665
  200. thailint-0.5.0.dist-info/METADATA +0 -1286
  201. thailint-0.5.0.dist-info/RECORD +0 -96
  202. thailint-0.5.0.dist-info/entry_points.txt +0 -4
  203. {thailint-0.5.0.dist-info → thailint-0.15.3.dist-info}/WHEEL +0 -0
  204. {thailint-0.5.0.dist-info → thailint-0.15.3.dist-info}/licenses/LICENSE +0 -0
src/cli.py DELETED
@@ -1,1665 +0,0 @@
1
- """
2
- Purpose: Main CLI entrypoint with Click framework for command-line interface
3
-
4
- Scope: CLI command definitions, option parsing, and command execution coordination
5
-
6
- Overview: Provides the main CLI application using Click decorators for command definition, option
7
- parsing, and help text generation. Includes example commands (hello, config management) that
8
- demonstrate best practices for CLI design including error handling, logging configuration,
9
- context management, and user-friendly output. Serves as the entry point for the installed
10
- CLI tool and coordinates between user input and application logic.
11
-
12
- Dependencies: click for CLI framework, logging for structured output, pathlib for file paths
13
-
14
- Exports: cli (main command group), hello command, config command group, file_placement command, dry command
15
-
16
- Interfaces: Click CLI commands, configuration context via Click ctx, logging integration
17
-
18
- Implementation: Click decorators for commands, context passing for shared state, comprehensive help text
19
- """
20
- # pylint: disable=too-many-lines
21
- # Justification: CLI modules naturally have many commands and helper functions
22
-
23
- import logging
24
- import sys
25
- from pathlib import Path
26
-
27
- import click
28
-
29
- from src import __version__
30
- from src.config import ConfigError, load_config, save_config, validate_config
31
- from src.core.cli_utils import format_violations
32
-
33
- # Configure module logger
34
- logger = logging.getLogger(__name__)
35
-
36
-
37
- # Shared Click option decorators for common CLI options
38
- def format_option(func):
39
- """Add --format option to a command for output format selection."""
40
- return click.option(
41
- "--format", "-f", type=click.Choice(["text", "json"]), default="text", help="Output format"
42
- )(func)
43
-
44
-
45
- def setup_logging(verbose: bool = False):
46
- """
47
- Configure logging for the CLI application.
48
-
49
- Args:
50
- verbose: Enable DEBUG level logging if True, INFO otherwise.
51
- """
52
- level = logging.DEBUG if verbose else logging.INFO
53
-
54
- logging.basicConfig(
55
- level=level,
56
- format="%(asctime)s | %(levelname)-8s | %(message)s",
57
- datefmt="%Y-%m-%d %H:%M:%S",
58
- stream=sys.stdout,
59
- )
60
-
61
-
62
- def _determine_project_root(
63
- explicit_root: str | None, config_path: str | None, verbose: bool
64
- ) -> Path:
65
- """Determine project root with precedence rules.
66
-
67
- Precedence order:
68
- 1. Explicit --project-root (highest priority)
69
- 2. Inferred from --config path directory
70
- 3. Auto-detection via get_project_root() (fallback)
71
-
72
- Args:
73
- explicit_root: Explicitly specified project root path (from --project-root)
74
- config_path: Config file path (from --config)
75
- verbose: Whether verbose logging is enabled
76
-
77
- Returns:
78
- Path to determined project root
79
-
80
- Raises:
81
- SystemExit: If explicit_root doesn't exist or is not a directory
82
- """
83
- from src.utils.project_root import get_project_root
84
-
85
- # Priority 1: Explicit --project-root
86
- if explicit_root:
87
- return _resolve_explicit_project_root(explicit_root, verbose)
88
-
89
- # Priority 2: Infer from --config path
90
- if config_path:
91
- return _infer_root_from_config(config_path, verbose)
92
-
93
- # Priority 3: Auto-detection (fallback)
94
- return _autodetect_project_root(verbose, get_project_root)
95
-
96
-
97
- def _resolve_explicit_project_root(explicit_root: str, verbose: bool) -> Path:
98
- """Resolve and validate explicitly specified project root.
99
-
100
- Args:
101
- explicit_root: Explicitly specified project root path
102
- verbose: Whether verbose logging is enabled
103
-
104
- Returns:
105
- Resolved project root path
106
-
107
- Raises:
108
- SystemExit: If explicit_root doesn't exist or is not a directory
109
- """
110
- root = Path(explicit_root)
111
- # Check existence before resolving to handle relative paths in test environments
112
- if not root.exists():
113
- click.echo(f"Error: Project root does not exist: {explicit_root}", err=True)
114
- sys.exit(2)
115
- if not root.is_dir():
116
- click.echo(f"Error: Project root must be a directory: {explicit_root}", err=True)
117
- sys.exit(2)
118
-
119
- # Now resolve after validation
120
- root = root.resolve()
121
-
122
- if verbose:
123
- logger.debug(f"Using explicit project root: {root}")
124
- return root
125
-
126
-
127
- def _infer_root_from_config(config_path: str, verbose: bool) -> Path:
128
- """Infer project root from config file path.
129
-
130
- Args:
131
- config_path: Config file path
132
- verbose: Whether verbose logging is enabled
133
-
134
- Returns:
135
- Inferred project root (parent directory of config file)
136
- """
137
- config_file = Path(config_path).resolve()
138
- inferred_root = config_file.parent
139
-
140
- if verbose:
141
- logger.debug(f"Inferred project root from config path: {inferred_root}")
142
- return inferred_root
143
-
144
-
145
- def _autodetect_project_root(verbose: bool, get_project_root) -> Path:
146
- """Auto-detect project root using project root detection.
147
-
148
- Args:
149
- verbose: Whether verbose logging is enabled
150
- get_project_root: Function to detect project root
151
-
152
- Returns:
153
- Auto-detected project root
154
- """
155
- auto_root = get_project_root(None)
156
- if verbose:
157
- logger.debug(f"Auto-detected project root: {auto_root}")
158
- return auto_root
159
-
160
-
161
- def _get_project_root_from_context(ctx) -> Path:
162
- """Get or determine project root from Click context.
163
-
164
- This function defers the actual determination until needed to avoid
165
- importing pyprojroot in test environments where it may not be available.
166
-
167
- Args:
168
- ctx: Click context containing CLI options
169
-
170
- Returns:
171
- Path to determined project root
172
- """
173
- # Check if already determined and cached
174
- if "project_root" in ctx.obj:
175
- return ctx.obj["project_root"]
176
-
177
- # Determine project root using stored CLI options
178
- explicit_root = ctx.obj.get("cli_project_root")
179
- config_path = ctx.obj.get("cli_config_path")
180
- verbose = ctx.obj.get("verbose", False)
181
-
182
- project_root = _determine_project_root(explicit_root, config_path, verbose)
183
-
184
- # Cache for future use
185
- ctx.obj["project_root"] = project_root
186
-
187
- return project_root
188
-
189
-
190
- @click.group()
191
- @click.version_option(version=__version__)
192
- @click.option("--verbose", "-v", is_flag=True, help="Enable verbose output")
193
- @click.option("--config", "-c", type=click.Path(), help="Path to config file")
194
- @click.option(
195
- "--project-root",
196
- type=click.Path(),
197
- help="Explicitly specify project root directory (overrides auto-detection)",
198
- )
199
- @click.pass_context
200
- def cli(ctx, verbose: bool, config: str | None, project_root: str | None):
201
- """
202
- thai-lint - AI code linter and governance tool
203
-
204
- Lint and governance for AI-generated code across multiple languages.
205
- Identifies common mistakes, anti-patterns, and security issues.
206
-
207
- Examples:
208
-
209
- \b
210
- # Check for duplicate code (DRY violations)
211
- thai-lint dry .
212
-
213
- \b
214
- # Lint current directory for file placement issues
215
- thai-lint file-placement .
216
-
217
- \b
218
- # Lint with custom config
219
- thai-lint file-placement --config .thailint.yaml src/
220
-
221
- \b
222
- # Specify project root explicitly (useful in Docker)
223
- thai-lint --project-root /workspace/root magic-numbers backend/
224
-
225
- \b
226
- # Get JSON output
227
- thai-lint file-placement --format json .
228
-
229
- \b
230
- # Show help
231
- thai-lint --help
232
- """
233
- # Ensure context object exists
234
- ctx.ensure_object(dict)
235
-
236
- # Setup logging
237
- setup_logging(verbose)
238
-
239
- # Store CLI options for later project root determination
240
- # (deferred to avoid pyprojroot import issues in test environments)
241
- ctx.obj["cli_project_root"] = project_root
242
- ctx.obj["cli_config_path"] = config
243
-
244
- # Load configuration
245
- try:
246
- if config:
247
- ctx.obj["config"] = load_config(Path(config))
248
- ctx.obj["config_path"] = Path(config)
249
- else:
250
- ctx.obj["config"] = load_config()
251
- ctx.obj["config_path"] = None
252
-
253
- logger.debug("Configuration loaded successfully")
254
- except ConfigError as e:
255
- click.echo(f"Error loading configuration: {e}", err=True)
256
- sys.exit(2)
257
-
258
- ctx.obj["verbose"] = verbose
259
-
260
-
261
- @cli.command()
262
- @click.option("--name", "-n", default="World", help="Name to greet")
263
- @click.option("--uppercase", "-u", is_flag=True, help="Convert greeting to uppercase")
264
- @click.pass_context
265
- def hello(ctx, name: str, uppercase: bool):
266
- """
267
- Print a greeting message.
268
-
269
- This is a simple example command demonstrating CLI basics.
270
-
271
- Examples:
272
-
273
- \b
274
- # Basic greeting
275
- thai-lint hello
276
-
277
- \b
278
- # Custom name
279
- thai-lint hello --name Alice
280
-
281
- \b
282
- # Uppercase output
283
- thai-lint hello --name Bob --uppercase
284
- """
285
- config = ctx.obj["config"]
286
- verbose = ctx.obj.get("verbose", False)
287
-
288
- # Get greeting from config or use default
289
- greeting_template = config.get("greeting", "Hello")
290
-
291
- # Build greeting message
292
- message = f"{greeting_template}, {name}!"
293
-
294
- if uppercase:
295
- message = message.upper()
296
-
297
- # Output greeting
298
- click.echo(message)
299
-
300
- if verbose:
301
- logger.info(f"Greeted {name} with template '{greeting_template}'")
302
-
303
-
304
- @cli.group()
305
- def config():
306
- """Configuration management commands."""
307
- pass
308
-
309
-
310
- @config.command("show")
311
- @click.option(
312
- "--format",
313
- "-f",
314
- type=click.Choice(["text", "json", "yaml"]),
315
- default="text",
316
- help="Output format",
317
- )
318
- @click.pass_context
319
- def config_show(ctx, format: str):
320
- """
321
- Display current configuration.
322
-
323
- Shows all configuration values in the specified format.
324
-
325
- Examples:
326
-
327
- \b
328
- # Show as text
329
- thai-lint config show
330
-
331
- \b
332
- # Show as JSON
333
- thai-lint config show --format json
334
-
335
- \b
336
- # Show as YAML
337
- thai-lint config show --format yaml
338
- """
339
- cfg = ctx.obj["config"]
340
-
341
- formatters = {
342
- "json": _format_config_json,
343
- "yaml": _format_config_yaml,
344
- "text": _format_config_text,
345
- }
346
- formatters[format](cfg)
347
-
348
-
349
- def _format_config_json(cfg: dict) -> None:
350
- """Format configuration as JSON."""
351
- import json
352
-
353
- click.echo(json.dumps(cfg, indent=2))
354
-
355
-
356
- def _format_config_yaml(cfg: dict) -> None:
357
- """Format configuration as YAML."""
358
- import yaml
359
-
360
- click.echo(yaml.dump(cfg, default_flow_style=False, sort_keys=False))
361
-
362
-
363
- def _format_config_text(cfg: dict) -> None:
364
- """Format configuration as text."""
365
- click.echo("Current Configuration:")
366
- click.echo("-" * 40)
367
- for key, value in cfg.items():
368
- click.echo(f"{key:20} : {value}")
369
-
370
-
371
- @config.command("get")
372
- @click.argument("key")
373
- @click.pass_context
374
- def config_get(ctx, key: str):
375
- """
376
- Get specific configuration value.
377
-
378
- KEY: Configuration key to retrieve
379
-
380
- Examples:
381
-
382
- \b
383
- # Get log level
384
- thai-lint config get log_level
385
-
386
- \b
387
- # Get greeting template
388
- thai-lint config get greeting
389
- """
390
- cfg = ctx.obj["config"]
391
-
392
- if key not in cfg:
393
- click.echo(f"Configuration key not found: {key}", err=True)
394
- sys.exit(1)
395
-
396
- click.echo(cfg[key])
397
-
398
-
399
- def _convert_value_type(value: str):
400
- """Convert string value to appropriate type."""
401
- if value.lower() in ["true", "false"]:
402
- return value.lower() == "true"
403
- if value.isdigit():
404
- return int(value)
405
- if value.replace(".", "", 1).isdigit() and value.count(".") == 1:
406
- return float(value)
407
- return value
408
-
409
-
410
- def _validate_and_report_errors(cfg: dict):
411
- """Validate configuration and report errors."""
412
- is_valid, errors = validate_config(cfg)
413
- if not is_valid:
414
- click.echo("Invalid configuration:", err=True)
415
- for error in errors:
416
- click.echo(f" - {error}", err=True)
417
- sys.exit(1)
418
-
419
-
420
- def _save_and_report_success(cfg: dict, key: str, value, config_path, verbose: bool):
421
- """Save configuration and report success."""
422
- save_config(cfg, config_path)
423
- click.echo(f"✓ Set {key} = {value}")
424
- if verbose:
425
- logger.info(f"Configuration updated: {key}={value}")
426
-
427
-
428
- @config.command("set")
429
- @click.argument("key")
430
- @click.argument("value")
431
- @click.pass_context
432
- def config_set(ctx, key: str, value: str):
433
- """
434
- Set configuration value.
435
-
436
- KEY: Configuration key to set
437
-
438
- VALUE: New value for the key
439
-
440
- Examples:
441
-
442
- \b
443
- # Set log level
444
- thai-lint config set log_level DEBUG
445
-
446
- \b
447
- # Set greeting template
448
- thai-lint config set greeting "Hi"
449
-
450
- \b
451
- # Set numeric value
452
- thai-lint config set max_retries 5
453
- """
454
- cfg = ctx.obj["config"]
455
- converted_value = _convert_value_type(value)
456
- cfg[key] = converted_value
457
-
458
- try:
459
- _validate_and_report_errors(cfg)
460
- except Exception as e:
461
- click.echo(f"Validation error: {e}", err=True)
462
- sys.exit(1)
463
-
464
- try:
465
- config_path = ctx.obj.get("config_path")
466
- verbose = ctx.obj.get("verbose", False)
467
- _save_and_report_success(cfg, key, converted_value, config_path, verbose)
468
- except ConfigError as e:
469
- click.echo(f"Error saving configuration: {e}", err=True)
470
- sys.exit(1)
471
-
472
-
473
- @config.command("reset")
474
- @click.option("--yes", "-y", is_flag=True, help="Skip confirmation prompt")
475
- @click.pass_context
476
- def config_reset(ctx, yes: bool):
477
- """
478
- Reset configuration to defaults.
479
-
480
- Examples:
481
-
482
- \b
483
- # Reset with confirmation
484
- thai-lint config reset
485
-
486
- \b
487
- # Reset without confirmation
488
- thai-lint config reset --yes
489
- """
490
- if not yes:
491
- click.confirm("Reset configuration to defaults?", abort=True)
492
-
493
- from src.config import DEFAULT_CONFIG
494
-
495
- try:
496
- config_path = ctx.obj.get("config_path")
497
- save_config(DEFAULT_CONFIG.copy(), config_path)
498
- click.echo("✓ Configuration reset to defaults")
499
-
500
- if ctx.obj.get("verbose"):
501
- logger.info("Configuration reset to defaults")
502
- except ConfigError as e:
503
- click.echo(f"Error resetting configuration: {e}", err=True)
504
- sys.exit(1)
505
-
506
-
507
- @cli.command("init-config")
508
- @click.option(
509
- "--preset",
510
- "-p",
511
- type=click.Choice(["strict", "standard", "lenient"]),
512
- default="standard",
513
- help="Configuration preset",
514
- )
515
- @click.option("--non-interactive", is_flag=True, help="Skip interactive prompts (for AI agents)")
516
- @click.option("--force", is_flag=True, help="Overwrite existing .thailint.yaml file")
517
- @click.option(
518
- "--output", "-o", type=click.Path(), default=".thailint.yaml", help="Output file path"
519
- )
520
- def init_config(preset: str, non_interactive: bool, force: bool, output: str):
521
- """
522
- Generate a .thailint.yaml configuration file with preset values.
523
-
524
- Creates a richly-commented configuration file with sensible defaults
525
- and optional customizations for different strictness levels.
526
-
527
- For AI agents, use --non-interactive mode:
528
- thailint init-config --non-interactive --preset lenient
529
-
530
- Presets:
531
- strict: Minimal allowed numbers (only -1, 0, 1)
532
- standard: Balanced defaults (includes 2, 3, 4, 5, 10, 100, 1000)
533
- lenient: Includes time conversions (adds 60, 3600)
534
-
535
- Examples:
536
-
537
- \\b
538
- # Interactive mode (default, for humans)
539
- thailint init-config
540
-
541
- \\b
542
- # Non-interactive mode (for AI agents)
543
- thailint init-config --non-interactive
544
-
545
- \\b
546
- # Generate with lenient preset
547
- thailint init-config --preset lenient
548
-
549
- \\b
550
- # Overwrite existing config
551
- thailint init-config --force
552
-
553
- \\b
554
- # Custom output path
555
- thailint init-config --output my-config.yaml
556
- """
557
- output_path = Path(output)
558
-
559
- # Check if file exists (unless --force)
560
- if output_path.exists() and not force:
561
- click.echo(f"Error: {output} already exists", err=True)
562
- click.echo("", err=True)
563
- click.echo("Use --force to overwrite:", err=True)
564
- click.echo(" thailint init-config --force", err=True)
565
- sys.exit(1)
566
-
567
- # Interactive mode: Ask user for preferences
568
- if not non_interactive:
569
- click.echo("thai-lint Configuration Generator")
570
- click.echo("=" * 50)
571
- click.echo("")
572
- click.echo("This will create a .thailint.yaml configuration file.")
573
- click.echo("For non-interactive mode (AI agents), use:")
574
- click.echo(" thailint init-config --non-interactive")
575
- click.echo("")
576
-
577
- # Ask for preset
578
- click.echo("Available presets:")
579
- click.echo(" strict: Only -1, 0, 1 allowed (strictest)")
580
- click.echo(" standard: -1, 0, 1, 2, 3, 4, 5, 10, 100, 1000 (balanced)")
581
- click.echo(" lenient: Includes time conversions 60, 3600 (most permissive)")
582
- click.echo("")
583
-
584
- preset = click.prompt(
585
- "Choose preset", type=click.Choice(["strict", "standard", "lenient"]), default=preset
586
- )
587
-
588
- # Generate config based on preset
589
- config_content = _generate_config_content(preset)
590
-
591
- # Write config file
592
- try:
593
- output_path.write_text(config_content, encoding="utf-8")
594
- click.echo("")
595
- click.echo(f"✓ Created {output}")
596
- click.echo(f"✓ Preset: {preset}")
597
- click.echo("")
598
- click.echo("Next steps:")
599
- click.echo(f" 1. Review and customize {output}")
600
- click.echo(" 2. Run: thailint magic-numbers .")
601
- click.echo(" 3. See docs: https://github.com/your-org/thai-lint")
602
- except OSError as e:
603
- click.echo(f"Error writing config file: {e}", err=True)
604
- sys.exit(1)
605
-
606
-
607
- def _generate_config_content(preset: str) -> str:
608
- """Generate config file content based on preset."""
609
- # Preset configurations
610
- presets = {
611
- "strict": {
612
- "allowed_numbers": "[-1, 0, 1]",
613
- "max_small_integer": "3",
614
- "description": "Strict (only universal values)",
615
- },
616
- "standard": {
617
- "allowed_numbers": "[-1, 0, 1, 2, 3, 4, 5, 10, 100, 1000]",
618
- "max_small_integer": "10",
619
- "description": "Standard (balanced defaults)",
620
- },
621
- "lenient": {
622
- "allowed_numbers": "[-1, 0, 1, 2, 3, 4, 5, 10, 60, 100, 1000, 3600]",
623
- "max_small_integer": "10",
624
- "description": "Lenient (includes time conversions)",
625
- },
626
- }
627
-
628
- config = presets[preset]
629
-
630
- # Read template
631
- template_path = Path(__file__).parent / "templates" / "thailint_config_template.yaml"
632
- template = template_path.read_text(encoding="utf-8")
633
-
634
- # Replace placeholders
635
- content = template.replace("{{PRESET}}", config["description"])
636
- content = content.replace("{{ALLOWED_NUMBERS}}", config["allowed_numbers"])
637
- content = content.replace("{{MAX_SMALL_INTEGER}}", config["max_small_integer"])
638
-
639
- return content
640
-
641
-
642
- @cli.command("file-placement")
643
- @click.argument("paths", nargs=-1, type=click.Path())
644
- @click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
645
- @click.option("--rules", "-r", help="Inline JSON rules configuration")
646
- @format_option
647
- @click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
648
- @click.pass_context
649
- def file_placement( # pylint: disable=too-many-arguments,too-many-positional-arguments,too-many-locals,too-many-statements
650
- ctx,
651
- paths: tuple[str, ...],
652
- config_file: str | None,
653
- rules: str | None,
654
- format: str,
655
- recursive: bool,
656
- ):
657
- # Justification for Pylint disables:
658
- # - too-many-arguments/positional: CLI requires 1 ctx + 1 arg + 4 options = 6 params
659
- # - too-many-locals/statements: Complex CLI logic for config, linting, and output formatting
660
- # All parameters and logic are necessary for flexible CLI usage.
661
- """
662
- Lint files for proper file placement.
663
-
664
- Checks that files are placed in appropriate directories according to
665
- configured rules and patterns.
666
-
667
- PATHS: Files or directories to lint (defaults to current directory if none provided)
668
-
669
- Examples:
670
-
671
- \b
672
- # Lint current directory (all files recursively)
673
- thai-lint file-placement
674
-
675
- \b
676
- # Lint specific directory
677
- thai-lint file-placement src/
678
-
679
- \b
680
- # Lint single file
681
- thai-lint file-placement src/app.py
682
-
683
- \b
684
- # Lint multiple files
685
- thai-lint file-placement src/app.py src/utils.py tests/test_app.py
686
-
687
- \b
688
- # Use custom config
689
- thai-lint file-placement --config rules.json .
690
-
691
- \b
692
- # Inline JSON rules
693
- thai-lint file-placement --rules '{"allow": [".*\\.py$"]}' .
694
- """
695
- verbose = ctx.obj.get("verbose", False)
696
- project_root = _get_project_root_from_context(ctx)
697
-
698
- if not paths:
699
- paths = (".",)
700
-
701
- path_objs = [Path(p) for p in paths]
702
-
703
- try:
704
- _execute_file_placement_lint(
705
- path_objs, config_file, rules, format, recursive, verbose, project_root
706
- )
707
- except Exception as e:
708
- _handle_linting_error(e, verbose)
709
-
710
-
711
- def _execute_file_placement_lint( # pylint: disable=too-many-arguments,too-many-positional-arguments
712
- path_objs, config_file, rules, format, recursive, verbose, project_root=None
713
- ):
714
- """Execute file placement linting."""
715
- _validate_paths_exist(path_objs)
716
- orchestrator = _setup_orchestrator(path_objs, config_file, rules, verbose, project_root)
717
- all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
718
-
719
- # Filter to only file-placement violations
720
- violations = [v for v in all_violations if v.rule_id.startswith("file-placement")]
721
-
722
- if verbose:
723
- logger.info(f"Found {len(violations)} violation(s)")
724
-
725
- format_violations(violations, format)
726
- sys.exit(1 if violations else 0)
727
-
728
-
729
- def _handle_linting_error(error: Exception, verbose: bool) -> None:
730
- """Handle linting errors."""
731
- click.echo(f"Error during linting: {error}", err=True)
732
- if verbose:
733
- logger.exception("Linting failed with exception")
734
- sys.exit(2)
735
-
736
-
737
- def _validate_paths_exist(path_objs: list[Path]) -> None:
738
- """Validate that all provided paths exist.
739
-
740
- Args:
741
- path_objs: List of Path objects to validate
742
-
743
- Raises:
744
- SystemExit: If any path doesn't exist (exit code 2)
745
- """
746
- for path in path_objs:
747
- if not path.exists():
748
- click.echo(f"Error: Path does not exist: {path}", err=True)
749
- click.echo("", err=True)
750
- click.echo(
751
- "Hint: When using Docker, ensure paths are inside the mounted volume:", err=True
752
- )
753
- click.echo(
754
- " docker run -v $(pwd):/data thailint <command> /data/your-file.py", err=True
755
- )
756
- sys.exit(2)
757
-
758
-
759
- def _find_project_root(start_path: Path) -> Path:
760
- """Find project root by looking for .git or pyproject.toml.
761
-
762
- DEPRECATED: Use src.utils.project_root.get_project_root() instead.
763
-
764
- Args:
765
- start_path: Directory to start searching from
766
-
767
- Returns:
768
- Path to project root, or start_path if no markers found
769
- """
770
- from src.utils.project_root import get_project_root
771
-
772
- return get_project_root(start_path)
773
-
774
-
775
- def _setup_orchestrator(path_objs, config_file, rules, verbose, project_root=None):
776
- """Set up and configure the orchestrator."""
777
- from src.orchestrator.core import Orchestrator
778
- from src.utils.project_root import get_project_root
779
-
780
- # Use provided project_root or fall back to auto-detection
781
- project_root = _get_or_detect_project_root(path_objs, project_root, get_project_root)
782
-
783
- orchestrator = Orchestrator(project_root=project_root)
784
- _apply_orchestrator_config(orchestrator, config_file, rules, verbose)
785
-
786
- return orchestrator
787
-
788
-
789
- def _get_or_detect_project_root(path_objs, project_root, get_project_root):
790
- """Get provided project root or auto-detect from paths.
791
-
792
- Args:
793
- path_objs: List of path objects
794
- project_root: Optionally provided project root
795
- get_project_root: Function to detect project root
796
-
797
- Returns:
798
- Project root path
799
- """
800
- if project_root is not None:
801
- return project_root
802
-
803
- # Find actual project root (where .git or pyproject.toml exists)
804
- # This ensures .artifacts/ is always created at project root, not in subdirectories
805
- first_path = path_objs[0] if path_objs else Path.cwd()
806
- search_start = first_path if first_path.is_dir() else first_path.parent
807
- return get_project_root(search_start)
808
-
809
-
810
- def _apply_orchestrator_config(orchestrator, config_file, rules, verbose):
811
- """Apply configuration to orchestrator.
812
-
813
- Args:
814
- orchestrator: Orchestrator instance
815
- config_file: Path to config file (optional)
816
- rules: Inline JSON rules (optional)
817
- verbose: Whether verbose logging is enabled
818
- """
819
- if rules:
820
- _apply_inline_rules(orchestrator, rules, verbose)
821
- elif config_file:
822
- _load_config_file(orchestrator, config_file, verbose)
823
-
824
-
825
- def _apply_inline_rules(orchestrator, rules, verbose):
826
- """Parse and apply inline JSON rules."""
827
- rules_config = _parse_json_rules(rules)
828
- orchestrator.config.update(rules_config)
829
- _log_applied_rules(rules_config, verbose)
830
-
831
-
832
- def _parse_json_rules(rules: str) -> dict:
833
- """Parse JSON rules string, exit on error."""
834
- import json
835
-
836
- try:
837
- return json.loads(rules)
838
- except json.JSONDecodeError as e:
839
- click.echo(f"Error: Invalid JSON in --rules: {e}", err=True)
840
- sys.exit(2)
841
-
842
-
843
- def _log_applied_rules(rules_config: dict, verbose: bool) -> None:
844
- """Log applied rules if verbose."""
845
- if verbose:
846
- logger.debug(f"Applied inline rules: {rules_config}")
847
-
848
-
849
- def _load_config_file(orchestrator, config_file, verbose):
850
- """Load configuration from external file."""
851
- config_path = Path(config_file)
852
- if not config_path.exists():
853
- click.echo(f"Error: Config file not found: {config_file}", err=True)
854
- sys.exit(2)
855
-
856
- # Load config into orchestrator
857
- orchestrator.config = orchestrator.config_loader.load(config_path)
858
-
859
- if verbose:
860
- logger.debug(f"Loaded config from: {config_file}")
861
-
862
-
863
- def _execute_linting(orchestrator, path_obj, recursive):
864
- """Execute linting on file or directory."""
865
- if path_obj.is_file():
866
- return orchestrator.lint_file(path_obj)
867
- return orchestrator.lint_directory(path_obj, recursive=recursive)
868
-
869
-
870
- def _separate_files_and_dirs(path_objs: list[Path]) -> tuple[list[Path], list[Path]]:
871
- """Separate file paths from directory paths.
872
-
873
- Args:
874
- path_objs: List of Path objects
875
-
876
- Returns:
877
- Tuple of (files, directories)
878
- """
879
- files = [p for p in path_objs if p.is_file()]
880
- dirs = [p for p in path_objs if p.is_dir()]
881
- return files, dirs
882
-
883
-
884
- def _lint_files_if_any(orchestrator, files: list[Path]) -> list:
885
- """Lint files if list is non-empty.
886
-
887
- Args:
888
- orchestrator: Orchestrator instance
889
- files: List of file paths
890
-
891
- Returns:
892
- List of violations
893
- """
894
- if files:
895
- return orchestrator.lint_files(files)
896
- return []
897
-
898
-
899
- def _lint_directories(orchestrator, dirs: list[Path], recursive: bool) -> list:
900
- """Lint all directories.
901
-
902
- Args:
903
- orchestrator: Orchestrator instance
904
- dirs: List of directory paths
905
- recursive: Whether to scan recursively
906
-
907
- Returns:
908
- List of violations from all directories
909
- """
910
- violations = []
911
- for dir_path in dirs:
912
- violations.extend(orchestrator.lint_directory(dir_path, recursive=recursive))
913
- return violations
914
-
915
-
916
- def _execute_linting_on_paths(orchestrator, path_objs: list[Path], recursive: bool) -> list:
917
- """Execute linting on list of file/directory paths.
918
-
919
- Args:
920
- orchestrator: Orchestrator instance
921
- path_objs: List of Path objects (files or directories)
922
- recursive: Whether to scan directories recursively
923
-
924
- Returns:
925
- List of violations from all paths
926
- """
927
- files, dirs = _separate_files_and_dirs(path_objs)
928
-
929
- violations = []
930
- violations.extend(_lint_files_if_any(orchestrator, files))
931
- violations.extend(_lint_directories(orchestrator, dirs, recursive))
932
-
933
- return violations
934
-
935
-
936
- def _setup_nesting_orchestrator(
937
- path_objs: list[Path], config_file: str | None, verbose: bool, project_root: Path | None = None
938
- ):
939
- """Set up orchestrator for nesting command."""
940
- from src.orchestrator.core import Orchestrator
941
- from src.utils.project_root import get_project_root
942
-
943
- # Use provided project_root or fall back to auto-detection
944
- if project_root is None:
945
- first_path = path_objs[0] if path_objs else Path.cwd()
946
- search_start = first_path if first_path.is_dir() else first_path.parent
947
- project_root = get_project_root(search_start)
948
-
949
- orchestrator = Orchestrator(project_root=project_root)
950
-
951
- if config_file:
952
- _load_config_file(orchestrator, config_file, verbose)
953
-
954
- return orchestrator
955
-
956
-
957
- def _apply_nesting_config_override(orchestrator, max_depth: int | None, verbose: bool):
958
- """Apply max_depth override to orchestrator config."""
959
- if max_depth is None:
960
- return
961
-
962
- # Ensure nesting config exists
963
- if "nesting" not in orchestrator.config:
964
- orchestrator.config["nesting"] = {}
965
-
966
- nesting_config = orchestrator.config["nesting"]
967
-
968
- # Set top-level max_nesting_depth
969
- nesting_config["max_nesting_depth"] = max_depth
970
-
971
- # Override language-specific configs to ensure CLI option takes precedence
972
- _override_language_specific_nesting(nesting_config, max_depth)
973
-
974
- if verbose:
975
- logger.debug(f"Overriding max_nesting_depth to {max_depth}")
976
-
977
-
978
- def _override_language_specific_nesting(nesting_config: dict, max_depth: int):
979
- """Override language-specific nesting depth configs.
980
-
981
- Args:
982
- nesting_config: Nesting configuration dictionary
983
- max_depth: Maximum nesting depth to set
984
- """
985
- for lang in ["python", "typescript", "javascript"]:
986
- if lang in nesting_config:
987
- nesting_config[lang]["max_nesting_depth"] = max_depth
988
-
989
-
990
- def _run_nesting_lint(orchestrator, path_objs: list[Path], recursive: bool):
991
- """Execute nesting lint on files or directories."""
992
- all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
993
- return [v for v in all_violations if "nesting" in v.rule_id]
994
-
995
-
996
- @cli.command("nesting")
997
- @click.argument("paths", nargs=-1, type=click.Path())
998
- @click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
999
- @format_option
1000
- @click.option("--max-depth", type=int, help="Override max nesting depth (default: 4)")
1001
- @click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
1002
- @click.pass_context
1003
- def nesting( # pylint: disable=too-many-arguments,too-many-positional-arguments
1004
- ctx,
1005
- paths: tuple[str, ...],
1006
- config_file: str | None,
1007
- format: str,
1008
- max_depth: int | None,
1009
- recursive: bool,
1010
- ):
1011
- """Check for excessive nesting depth in code.
1012
-
1013
- Analyzes Python and TypeScript files for deeply nested code structures
1014
- (if/for/while/try statements) and reports violations.
1015
-
1016
- PATHS: Files or directories to lint (defaults to current directory if none provided)
1017
-
1018
- Examples:
1019
-
1020
- \b
1021
- # Check current directory (all files recursively)
1022
- thai-lint nesting
1023
-
1024
- \b
1025
- # Check specific directory
1026
- thai-lint nesting src/
1027
-
1028
- \b
1029
- # Check single file
1030
- thai-lint nesting src/app.py
1031
-
1032
- \b
1033
- # Check multiple files
1034
- thai-lint nesting src/app.py src/utils.py tests/test_app.py
1035
-
1036
- \b
1037
- # Check mix of files and directories
1038
- thai-lint nesting src/app.py tests/
1039
-
1040
- \b
1041
- # Use custom max depth
1042
- thai-lint nesting --max-depth 3 src/
1043
-
1044
- \b
1045
- # Get JSON output
1046
- thai-lint nesting --format json .
1047
-
1048
- \b
1049
- # Use custom config file
1050
- thai-lint nesting --config .thailint.yaml src/
1051
- """
1052
- verbose = ctx.obj.get("verbose", False)
1053
- project_root = _get_project_root_from_context(ctx)
1054
-
1055
- # Default to current directory if no paths provided
1056
- if not paths:
1057
- paths = (".",)
1058
-
1059
- path_objs = [Path(p) for p in paths]
1060
-
1061
- try:
1062
- _execute_nesting_lint(
1063
- path_objs, config_file, format, max_depth, recursive, verbose, project_root
1064
- )
1065
- except Exception as e:
1066
- _handle_linting_error(e, verbose)
1067
-
1068
-
1069
- def _execute_nesting_lint( # pylint: disable=too-many-arguments,too-many-positional-arguments
1070
- path_objs, config_file, format, max_depth, recursive, verbose, project_root=None
1071
- ):
1072
- """Execute nesting lint."""
1073
- _validate_paths_exist(path_objs)
1074
- orchestrator = _setup_nesting_orchestrator(path_objs, config_file, verbose, project_root)
1075
- _apply_nesting_config_override(orchestrator, max_depth, verbose)
1076
- nesting_violations = _run_nesting_lint(orchestrator, path_objs, recursive)
1077
-
1078
- if verbose:
1079
- logger.info(f"Found {len(nesting_violations)} nesting violation(s)")
1080
-
1081
- format_violations(nesting_violations, format)
1082
- sys.exit(1 if nesting_violations else 0)
1083
-
1084
-
1085
- def _setup_srp_orchestrator(
1086
- path_objs: list[Path], config_file: str | None, verbose: bool, project_root: Path | None = None
1087
- ):
1088
- """Set up orchestrator for SRP command."""
1089
- from src.orchestrator.core import Orchestrator
1090
- from src.utils.project_root import get_project_root
1091
-
1092
- # Use provided project_root or fall back to auto-detection
1093
- if project_root is None:
1094
- first_path = path_objs[0] if path_objs else Path.cwd()
1095
- search_start = first_path if first_path.is_dir() else first_path.parent
1096
- project_root = get_project_root(search_start)
1097
-
1098
- orchestrator = Orchestrator(project_root=project_root)
1099
-
1100
- if config_file:
1101
- _load_config_file(orchestrator, config_file, verbose)
1102
-
1103
- return orchestrator
1104
-
1105
-
1106
- def _apply_srp_config_override(
1107
- orchestrator, max_methods: int | None, max_loc: int | None, verbose: bool
1108
- ):
1109
- """Apply max_methods and max_loc overrides to orchestrator config."""
1110
- if max_methods is None and max_loc is None:
1111
- return
1112
-
1113
- if "srp" not in orchestrator.config:
1114
- orchestrator.config["srp"] = {}
1115
-
1116
- _apply_srp_max_methods(orchestrator, max_methods, verbose)
1117
- _apply_srp_max_loc(orchestrator, max_loc, verbose)
1118
-
1119
-
1120
- def _apply_srp_max_methods(orchestrator, max_methods: int | None, verbose: bool):
1121
- """Apply max_methods override."""
1122
- if max_methods is not None:
1123
- orchestrator.config["srp"]["max_methods"] = max_methods
1124
- if verbose:
1125
- logger.debug(f"Overriding max_methods to {max_methods}")
1126
-
1127
-
1128
- def _apply_srp_max_loc(orchestrator, max_loc: int | None, verbose: bool):
1129
- """Apply max_loc override."""
1130
- if max_loc is not None:
1131
- orchestrator.config["srp"]["max_loc"] = max_loc
1132
- if verbose:
1133
- logger.debug(f"Overriding max_loc to {max_loc}")
1134
-
1135
-
1136
- def _run_srp_lint(orchestrator, path_objs: list[Path], recursive: bool):
1137
- """Execute SRP lint on files or directories."""
1138
- all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
1139
- return [v for v in all_violations if "srp" in v.rule_id]
1140
-
1141
-
1142
- @cli.command("srp")
1143
- @click.argument("paths", nargs=-1, type=click.Path())
1144
- @click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
1145
- @format_option
1146
- @click.option("--max-methods", type=int, help="Override max methods per class (default: 7)")
1147
- @click.option("--max-loc", type=int, help="Override max lines of code per class (default: 200)")
1148
- @click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
1149
- @click.pass_context
1150
- def srp( # pylint: disable=too-many-arguments,too-many-positional-arguments
1151
- ctx,
1152
- paths: tuple[str, ...],
1153
- config_file: str | None,
1154
- format: str,
1155
- max_methods: int | None,
1156
- max_loc: int | None,
1157
- recursive: bool,
1158
- ):
1159
- """Check for Single Responsibility Principle violations.
1160
-
1161
- Analyzes Python and TypeScript classes for SRP violations using heuristics:
1162
- - Method count exceeding threshold (default: 7)
1163
- - Lines of code exceeding threshold (default: 200)
1164
- - Responsibility keywords in class names (Manager, Handler, Processor, etc.)
1165
-
1166
- PATHS: Files or directories to lint (defaults to current directory if none provided)
1167
-
1168
- Examples:
1169
-
1170
- \b
1171
- # Check current directory (all files recursively)
1172
- thai-lint srp
1173
-
1174
- \b
1175
- # Check specific directory
1176
- thai-lint srp src/
1177
-
1178
- \b
1179
- # Check single file
1180
- thai-lint srp src/app.py
1181
-
1182
- \b
1183
- # Check multiple files
1184
- thai-lint srp src/app.py src/service.py tests/test_app.py
1185
-
1186
- \b
1187
- # Use custom thresholds
1188
- thai-lint srp --max-methods 10 --max-loc 300 src/
1189
-
1190
- \b
1191
- # Get JSON output
1192
- thai-lint srp --format json .
1193
-
1194
- \b
1195
- # Use custom config file
1196
- thai-lint srp --config .thailint.yaml src/
1197
- """
1198
- verbose = ctx.obj.get("verbose", False)
1199
- project_root = _get_project_root_from_context(ctx)
1200
-
1201
- if not paths:
1202
- paths = (".",)
1203
-
1204
- path_objs = [Path(p) for p in paths]
1205
-
1206
- try:
1207
- _execute_srp_lint(
1208
- path_objs, config_file, format, max_methods, max_loc, recursive, verbose, project_root
1209
- )
1210
- except Exception as e:
1211
- _handle_linting_error(e, verbose)
1212
-
1213
-
1214
- def _execute_srp_lint( # pylint: disable=too-many-arguments,too-many-positional-arguments
1215
- path_objs, config_file, format, max_methods, max_loc, recursive, verbose, project_root=None
1216
- ):
1217
- """Execute SRP lint."""
1218
- _validate_paths_exist(path_objs)
1219
- orchestrator = _setup_srp_orchestrator(path_objs, config_file, verbose, project_root)
1220
- _apply_srp_config_override(orchestrator, max_methods, max_loc, verbose)
1221
- srp_violations = _run_srp_lint(orchestrator, path_objs, recursive)
1222
-
1223
- if verbose:
1224
- logger.info(f"Found {len(srp_violations)} SRP violation(s)")
1225
-
1226
- format_violations(srp_violations, format)
1227
- sys.exit(1 if srp_violations else 0)
1228
-
1229
-
1230
- @cli.command("dry")
1231
- @click.argument("paths", nargs=-1, type=click.Path())
1232
- @click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
1233
- @format_option
1234
- @click.option("--min-lines", type=int, help="Override min duplicate lines threshold")
1235
- @click.option("--no-cache", is_flag=True, help="Disable SQLite cache (force rehash)")
1236
- @click.option("--clear-cache", is_flag=True, help="Clear cache before running")
1237
- @click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
1238
- @click.pass_context
1239
- def dry( # pylint: disable=too-many-arguments,too-many-positional-arguments
1240
- ctx,
1241
- paths: tuple[str, ...],
1242
- config_file: str | None,
1243
- format: str,
1244
- min_lines: int | None,
1245
- no_cache: bool,
1246
- clear_cache: bool,
1247
- recursive: bool,
1248
- ):
1249
- # Justification for Pylint disables:
1250
- # - too-many-arguments/positional: CLI requires 1 ctx + 1 arg + 6 options = 8 params
1251
- # All parameters are necessary for flexible DRY linter CLI usage.
1252
- """
1253
- Check for duplicate code (DRY principle violations).
1254
-
1255
- Detects duplicate code blocks across your project using token-based hashing
1256
- with SQLite caching for fast incremental scans.
1257
-
1258
- PATHS: Files or directories to lint (defaults to current directory if none provided)
1259
-
1260
- Examples:
1261
-
1262
- \b
1263
- # Check current directory (all files recursively)
1264
- thai-lint dry
1265
-
1266
- \b
1267
- # Check specific directory
1268
- thai-lint dry src/
1269
-
1270
- \b
1271
- # Check single file
1272
- thai-lint dry src/app.py
1273
-
1274
- \b
1275
- # Check multiple files
1276
- thai-lint dry src/app.py src/service.py tests/test_app.py
1277
-
1278
- \b
1279
- # Use custom config file
1280
- thai-lint dry --config .thailint.yaml src/
1281
-
1282
- \b
1283
- # Override minimum duplicate lines threshold
1284
- thai-lint dry --min-lines 5 .
1285
-
1286
- \b
1287
- # Disable cache (force re-analysis)
1288
- thai-lint dry --no-cache .
1289
-
1290
- \b
1291
- # Clear cache before running
1292
- thai-lint dry --clear-cache .
1293
-
1294
- \b
1295
- # Get JSON output
1296
- thai-lint dry --format json .
1297
- """
1298
- verbose = ctx.obj.get("verbose", False)
1299
- project_root = _get_project_root_from_context(ctx)
1300
-
1301
- if not paths:
1302
- paths = (".",)
1303
-
1304
- path_objs = [Path(p) for p in paths]
1305
-
1306
- try:
1307
- _execute_dry_lint(
1308
- path_objs,
1309
- config_file,
1310
- format,
1311
- min_lines,
1312
- no_cache,
1313
- clear_cache,
1314
- recursive,
1315
- verbose,
1316
- project_root,
1317
- )
1318
- except Exception as e:
1319
- _handle_linting_error(e, verbose)
1320
-
1321
-
1322
- def _execute_dry_lint( # pylint: disable=too-many-arguments,too-many-positional-arguments
1323
- path_objs,
1324
- config_file,
1325
- format,
1326
- min_lines,
1327
- no_cache,
1328
- clear_cache,
1329
- recursive,
1330
- verbose,
1331
- project_root=None,
1332
- ):
1333
- """Execute DRY linting."""
1334
- _validate_paths_exist(path_objs)
1335
- orchestrator = _setup_dry_orchestrator(path_objs, config_file, verbose, project_root)
1336
- _apply_dry_config_override(orchestrator, min_lines, no_cache, verbose)
1337
-
1338
- if clear_cache:
1339
- _clear_dry_cache(orchestrator, verbose)
1340
-
1341
- dry_violations = _run_dry_lint(orchestrator, path_objs, recursive)
1342
-
1343
- if verbose:
1344
- logger.info(f"Found {len(dry_violations)} DRY violation(s)")
1345
-
1346
- format_violations(dry_violations, format)
1347
- sys.exit(1 if dry_violations else 0)
1348
-
1349
-
1350
- def _setup_dry_orchestrator(path_objs, config_file, verbose, project_root=None):
1351
- """Set up orchestrator for DRY linting."""
1352
- from src.orchestrator.core import Orchestrator
1353
- from src.utils.project_root import get_project_root
1354
-
1355
- # Use provided project_root or fall back to auto-detection
1356
- if project_root is None:
1357
- first_path = path_objs[0] if path_objs else Path.cwd()
1358
- search_start = first_path if first_path.is_dir() else first_path.parent
1359
- project_root = get_project_root(search_start)
1360
-
1361
- orchestrator = Orchestrator(project_root=project_root)
1362
-
1363
- if config_file:
1364
- _load_dry_config_file(orchestrator, config_file, verbose)
1365
-
1366
- return orchestrator
1367
-
1368
-
1369
- def _load_dry_config_file(orchestrator, config_file, verbose):
1370
- """Load DRY configuration from file."""
1371
- import yaml
1372
-
1373
- config_path = Path(config_file)
1374
- if not config_path.exists():
1375
- click.echo(f"Error: Config file not found: {config_file}", err=True)
1376
- sys.exit(2)
1377
-
1378
- with config_path.open("r", encoding="utf-8") as f:
1379
- config = yaml.safe_load(f)
1380
-
1381
- if "dry" in config:
1382
- orchestrator.config.update({"dry": config["dry"]})
1383
-
1384
- if verbose:
1385
- logger.info(f"Loaded DRY config from {config_file}")
1386
-
1387
-
1388
- def _apply_dry_config_override(orchestrator, min_lines, no_cache, verbose):
1389
- """Apply CLI option overrides to DRY config."""
1390
- _ensure_dry_config_exists(orchestrator)
1391
- _apply_min_lines_override(orchestrator, min_lines, verbose)
1392
- _apply_cache_override(orchestrator, no_cache, verbose)
1393
-
1394
-
1395
- def _ensure_dry_config_exists(orchestrator):
1396
- """Ensure dry config section exists."""
1397
- if "dry" not in orchestrator.config:
1398
- orchestrator.config["dry"] = {}
1399
-
1400
-
1401
- def _apply_min_lines_override(orchestrator, min_lines, verbose):
1402
- """Apply min_lines override if provided."""
1403
- if min_lines is None:
1404
- return
1405
-
1406
- orchestrator.config["dry"]["min_duplicate_lines"] = min_lines
1407
- if verbose:
1408
- logger.info(f"Override: min_duplicate_lines = {min_lines}")
1409
-
1410
-
1411
- def _apply_cache_override(orchestrator, no_cache, verbose):
1412
- """Apply cache override if requested."""
1413
- if not no_cache:
1414
- return
1415
-
1416
- orchestrator.config["dry"]["cache_enabled"] = False
1417
- if verbose:
1418
- logger.info("Override: cache_enabled = False")
1419
-
1420
-
1421
- def _clear_dry_cache(orchestrator, verbose):
1422
- """Clear DRY cache before running."""
1423
- cache_path_str = orchestrator.config.get("dry", {}).get("cache_path", ".thailint-cache/dry.db")
1424
- cache_path = orchestrator.project_root / cache_path_str
1425
-
1426
- if cache_path.exists():
1427
- cache_path.unlink()
1428
- if verbose:
1429
- logger.info(f"Cleared cache: {cache_path}")
1430
- else:
1431
- if verbose:
1432
- logger.info("Cache file does not exist, nothing to clear")
1433
-
1434
-
1435
- def _run_dry_lint(orchestrator, path_objs, recursive):
1436
- """Run DRY linting and return violations."""
1437
- all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
1438
-
1439
- # Filter to only DRY violations
1440
- dry_violations = [v for v in all_violations if v.rule_id.startswith("dry.")]
1441
-
1442
- return dry_violations
1443
-
1444
-
1445
- def _setup_magic_numbers_orchestrator(
1446
- path_objs: list[Path], config_file: str | None, verbose: bool, project_root: Path | None = None
1447
- ):
1448
- """Set up orchestrator for magic-numbers command."""
1449
- from src.orchestrator.core import Orchestrator
1450
- from src.utils.project_root import get_project_root
1451
-
1452
- # Use provided project_root or fall back to auto-detection
1453
- if project_root is None:
1454
- # Find actual project root (where .git or .thailint.yaml exists)
1455
- first_path = path_objs[0] if path_objs else Path.cwd()
1456
- search_start = first_path if first_path.is_dir() else first_path.parent
1457
- project_root = get_project_root(search_start)
1458
-
1459
- orchestrator = Orchestrator(project_root=project_root)
1460
-
1461
- if config_file:
1462
- _load_config_file(orchestrator, config_file, verbose)
1463
-
1464
- return orchestrator
1465
-
1466
-
1467
- def _run_magic_numbers_lint(orchestrator, path_objs: list[Path], recursive: bool):
1468
- """Execute magic-numbers lint on files or directories."""
1469
- all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
1470
- return [v for v in all_violations if "magic-number" in v.rule_id]
1471
-
1472
-
1473
- @cli.command("magic-numbers")
1474
- @click.argument("paths", nargs=-1, type=click.Path())
1475
- @click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
1476
- @format_option
1477
- @click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
1478
- @click.pass_context
1479
- def magic_numbers( # pylint: disable=too-many-arguments,too-many-positional-arguments
1480
- ctx,
1481
- paths: tuple[str, ...],
1482
- config_file: str | None,
1483
- format: str,
1484
- recursive: bool,
1485
- ):
1486
- """Check for magic numbers in code.
1487
-
1488
- Detects unnamed numeric literals in Python and TypeScript/JavaScript code
1489
- that should be extracted as named constants for better readability.
1490
-
1491
- PATHS: Files or directories to lint (defaults to current directory if none provided)
1492
-
1493
- Examples:
1494
-
1495
- \b
1496
- # Check current directory (all files recursively)
1497
- thai-lint magic-numbers
1498
-
1499
- \b
1500
- # Check specific directory
1501
- thai-lint magic-numbers src/
1502
-
1503
- \b
1504
- # Check single file
1505
- thai-lint magic-numbers src/app.py
1506
-
1507
- \b
1508
- # Check multiple files
1509
- thai-lint magic-numbers src/app.py src/utils.py tests/test_app.py
1510
-
1511
- \b
1512
- # Check mix of files and directories
1513
- thai-lint magic-numbers src/app.py tests/
1514
-
1515
- \b
1516
- # Get JSON output
1517
- thai-lint magic-numbers --format json .
1518
-
1519
- \b
1520
- # Use custom config file
1521
- thai-lint magic-numbers --config .thailint.yaml src/
1522
- """
1523
- verbose = ctx.obj.get("verbose", False)
1524
- project_root = _get_project_root_from_context(ctx)
1525
-
1526
- if not paths:
1527
- paths = (".",)
1528
-
1529
- path_objs = [Path(p) for p in paths]
1530
-
1531
- try:
1532
- _execute_magic_numbers_lint(
1533
- path_objs, config_file, format, recursive, verbose, project_root
1534
- )
1535
- except Exception as e:
1536
- _handle_linting_error(e, verbose)
1537
-
1538
-
1539
- def _execute_magic_numbers_lint( # pylint: disable=too-many-arguments,too-many-positional-arguments
1540
- path_objs, config_file, format, recursive, verbose, project_root=None
1541
- ):
1542
- """Execute magic-numbers lint."""
1543
- _validate_paths_exist(path_objs)
1544
- orchestrator = _setup_magic_numbers_orchestrator(path_objs, config_file, verbose, project_root)
1545
- magic_numbers_violations = _run_magic_numbers_lint(orchestrator, path_objs, recursive)
1546
-
1547
- if verbose:
1548
- logger.info(f"Found {len(magic_numbers_violations)} magic number violation(s)")
1549
-
1550
- format_violations(magic_numbers_violations, format)
1551
- sys.exit(1 if magic_numbers_violations else 0)
1552
-
1553
-
1554
- # =============================================================================
1555
- # Print Statements Linter Command
1556
- # =============================================================================
1557
-
1558
-
1559
- def _setup_print_statements_orchestrator(
1560
- path_objs: list[Path], config_file: str | None, verbose: bool, project_root: Path | None = None
1561
- ):
1562
- """Set up orchestrator for print-statements command."""
1563
- from src.orchestrator.core import Orchestrator
1564
- from src.utils.project_root import get_project_root
1565
-
1566
- if project_root is None:
1567
- first_path = path_objs[0] if path_objs else Path.cwd()
1568
- search_start = first_path if first_path.is_dir() else first_path.parent
1569
- project_root = get_project_root(search_start)
1570
-
1571
- orchestrator = Orchestrator(project_root=project_root)
1572
-
1573
- if config_file:
1574
- _load_config_file(orchestrator, config_file, verbose)
1575
-
1576
- return orchestrator
1577
-
1578
-
1579
- def _run_print_statements_lint(orchestrator, path_objs: list[Path], recursive: bool):
1580
- """Execute print-statements lint on files or directories."""
1581
- all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
1582
- return [v for v in all_violations if "print-statement" in v.rule_id]
1583
-
1584
-
1585
- @cli.command("print-statements")
1586
- @click.argument("paths", nargs=-1, type=click.Path())
1587
- @click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
1588
- @format_option
1589
- @click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
1590
- @click.pass_context
1591
- def print_statements( # pylint: disable=too-many-arguments,too-many-positional-arguments
1592
- ctx,
1593
- paths: tuple[str, ...],
1594
- config_file: str | None,
1595
- format: str,
1596
- recursive: bool,
1597
- ):
1598
- """Check for print/console statements in code.
1599
-
1600
- Detects print() calls in Python and console.log/warn/error/debug/info calls
1601
- in TypeScript/JavaScript that should be replaced with proper logging.
1602
-
1603
- PATHS: Files or directories to lint (defaults to current directory if none provided)
1604
-
1605
- Examples:
1606
-
1607
- \b
1608
- # Check current directory (all files recursively)
1609
- thai-lint print-statements
1610
-
1611
- \b
1612
- # Check specific directory
1613
- thai-lint print-statements src/
1614
-
1615
- \b
1616
- # Check single file
1617
- thai-lint print-statements src/app.py
1618
-
1619
- \b
1620
- # Check multiple files
1621
- thai-lint print-statements src/app.py src/utils.ts tests/test_app.py
1622
-
1623
- \b
1624
- # Get JSON output
1625
- thai-lint print-statements --format json .
1626
-
1627
- \b
1628
- # Use custom config file
1629
- thai-lint print-statements --config .thailint.yaml src/
1630
- """
1631
- verbose = ctx.obj.get("verbose", False)
1632
- project_root = _get_project_root_from_context(ctx)
1633
-
1634
- if not paths:
1635
- paths = (".",)
1636
-
1637
- path_objs = [Path(p) for p in paths]
1638
-
1639
- try:
1640
- _execute_print_statements_lint(
1641
- path_objs, config_file, format, recursive, verbose, project_root
1642
- )
1643
- except Exception as e:
1644
- _handle_linting_error(e, verbose)
1645
-
1646
-
1647
- def _execute_print_statements_lint( # pylint: disable=too-many-arguments,too-many-positional-arguments
1648
- path_objs, config_file, format, recursive, verbose, project_root=None
1649
- ):
1650
- """Execute print-statements lint."""
1651
- _validate_paths_exist(path_objs)
1652
- orchestrator = _setup_print_statements_orchestrator(
1653
- path_objs, config_file, verbose, project_root
1654
- )
1655
- print_statements_violations = _run_print_statements_lint(orchestrator, path_objs, recursive)
1656
-
1657
- if verbose:
1658
- logger.info(f"Found {len(print_statements_violations)} print statement violation(s)")
1659
-
1660
- format_violations(print_statements_violations, format)
1661
- sys.exit(1 if print_statements_violations else 0)
1662
-
1663
-
1664
- if __name__ == "__main__":
1665
- cli()