@aiready/consistency 0.21.9 → 0.21.12

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 (132) hide show
  1. package/.turbo/turbo-build.log +23 -24
  2. package/dist/index.d.mts +3 -3
  3. package/dist/index.d.ts +3 -3
  4. package/dist/index.js +4 -4
  5. package/dist/index.mjs +3 -3
  6. package/package.json +2 -2
  7. package/src/__tests__/provider.test.ts +6 -34
  8. package/src/index.ts +3 -3
  9. package/src/provider.ts +2 -2
  10. package/dist/__tests__/analyzer.test.d.ts +0 -2
  11. package/dist/__tests__/analyzer.test.d.ts.map +0 -1
  12. package/dist/__tests__/analyzer.test.js +0 -157
  13. package/dist/__tests__/analyzer.test.js.map +0 -1
  14. package/dist/__tests__/language-filter.test.d.ts +0 -2
  15. package/dist/__tests__/language-filter.test.d.ts.map +0 -1
  16. package/dist/__tests__/language-filter.test.js +0 -46
  17. package/dist/__tests__/language-filter.test.js.map +0 -1
  18. package/dist/__tests__/scoring.test.d.ts +0 -2
  19. package/dist/__tests__/scoring.test.d.ts.map +0 -1
  20. package/dist/__tests__/scoring.test.js +0 -118
  21. package/dist/__tests__/scoring.test.js.map +0 -1
  22. package/dist/analyzer.d.ts +0 -7
  23. package/dist/analyzer.d.ts.map +0 -1
  24. package/dist/analyzer.js +0 -160
  25. package/dist/analyzer.js.map +0 -1
  26. package/dist/analyzers/naming-ast.d.ts +0 -7
  27. package/dist/analyzers/naming-ast.d.ts.map +0 -1
  28. package/dist/analyzers/naming-ast.js +0 -253
  29. package/dist/analyzers/naming-ast.js.map +0 -1
  30. package/dist/analyzers/naming-constants.d.ts +0 -21
  31. package/dist/analyzers/naming-constants.d.ts.map +0 -1
  32. package/dist/analyzers/naming-constants.js +0 -96
  33. package/dist/analyzers/naming-constants.js.map +0 -1
  34. package/dist/analyzers/naming-python.d.ts +0 -16
  35. package/dist/analyzers/naming-python.d.ts.map +0 -1
  36. package/dist/analyzers/naming-python.js +0 -165
  37. package/dist/analyzers/naming-python.js.map +0 -1
  38. package/dist/analyzers/naming.d.ts +0 -6
  39. package/dist/analyzers/naming.d.ts.map +0 -1
  40. package/dist/analyzers/naming.js +0 -234
  41. package/dist/analyzers/naming.js.map +0 -1
  42. package/dist/analyzers/patterns.d.ts +0 -10
  43. package/dist/analyzers/patterns.d.ts.map +0 -1
  44. package/dist/analyzers/patterns.js +0 -197
  45. package/dist/analyzers/patterns.js.map +0 -1
  46. package/dist/chunk-2BTBNG6X.mjs +0 -814
  47. package/dist/chunk-3ZB6FFRL.mjs +0 -661
  48. package/dist/chunk-5UFRGXSB.mjs +0 -783
  49. package/dist/chunk-66M3TIO7.mjs +0 -837
  50. package/dist/chunk-6BM5MV3S.mjs +0 -719
  51. package/dist/chunk-6H3JHDP7.mjs +0 -832
  52. package/dist/chunk-7PHHJOGC.mjs +0 -1374
  53. package/dist/chunk-AASFXGUR.mjs +0 -1622
  54. package/dist/chunk-AR7DIZLP.mjs +0 -827
  55. package/dist/chunk-BDDMOIU2.mjs +0 -385
  56. package/dist/chunk-BMILMNKJ.mjs +0 -1633
  57. package/dist/chunk-BYY6MD5T.mjs +0 -729
  58. package/dist/chunk-CA4Q5JBK.mjs +0 -1143
  59. package/dist/chunk-CF4LU7KE.mjs +0 -384
  60. package/dist/chunk-CJINEUIH.mjs +0 -1369
  61. package/dist/chunk-CLWNLHDB.mjs +0 -909
  62. package/dist/chunk-CZUJTDNH.mjs +0 -848
  63. package/dist/chunk-DNGW3WQK.mjs +0 -810
  64. package/dist/chunk-DSI3TEO2.mjs +0 -662
  65. package/dist/chunk-EIQ5K6OO.mjs +0 -1579
  66. package/dist/chunk-FEJODRK5.mjs +0 -783
  67. package/dist/chunk-FK56AZ43.mjs +0 -817
  68. package/dist/chunk-H6S7WKSQ.mjs +0 -729
  69. package/dist/chunk-HAOJLJNB.mjs +0 -1290
  70. package/dist/chunk-HJCP36VW.mjs +0 -821
  71. package/dist/chunk-HPG7P6PD.mjs +0 -1372
  72. package/dist/chunk-IVRBV7SE.mjs +0 -1295
  73. package/dist/chunk-IXBC6GVT.mjs +0 -832
  74. package/dist/chunk-J5IFYDVU.mjs +0 -1579
  75. package/dist/chunk-KWQVBF7K.mjs +0 -831
  76. package/dist/chunk-LD3CHHU2.mjs +0 -1297
  77. package/dist/chunk-LMOXGPCM.mjs +0 -722
  78. package/dist/chunk-LSXZH6X6.mjs +0 -810
  79. package/dist/chunk-LUAREV6A.mjs +0 -508
  80. package/dist/chunk-MAPVFXBP.mjs +0 -708
  81. package/dist/chunk-MM2PLUCH.mjs +0 -1376
  82. package/dist/chunk-NPWCJZUG.mjs +0 -708
  83. package/dist/chunk-ON73WHHU.mjs +0 -1310
  84. package/dist/chunk-P6NVKUBB.mjs +0 -831
  85. package/dist/chunk-Q3KTWDSL.mjs +0 -808
  86. package/dist/chunk-Q5XMWG33.mjs +0 -661
  87. package/dist/chunk-QOIPVP6P.mjs +0 -1607
  88. package/dist/chunk-RMEQWG52.mjs +0 -1633
  89. package/dist/chunk-S6BZVTWN.mjs +0 -731
  90. package/dist/chunk-TE6JYZD3.mjs +0 -810
  91. package/dist/chunk-TLVLM3M5.mjs +0 -771
  92. package/dist/chunk-TXHPUU7A.mjs +0 -863
  93. package/dist/chunk-UMBBTNQN.mjs +0 -787
  94. package/dist/chunk-V2UPXL7L.mjs +0 -842
  95. package/dist/chunk-VODCPPET.mjs +0 -1292
  96. package/dist/chunk-W6UGMKRV.mjs +0 -1310
  97. package/dist/chunk-WGH4TGZ3.mjs +0 -1288
  98. package/dist/chunk-WTBDNCEN.mjs +0 -1352
  99. package/dist/chunk-XVW5DKJQ.mjs +0 -1619
  100. package/dist/chunk-YCDCIOJN.mjs +0 -842
  101. package/dist/chunk-YEHXYHGY.mjs +0 -1497
  102. package/dist/chunk-YHHXE2JX.mjs +0 -912
  103. package/dist/chunk-ZB6UK276.mjs +0 -662
  104. package/dist/chunk-ZG3KFSD3.mjs +0 -1142
  105. package/dist/cli.d.ts.map +0 -1
  106. package/dist/cli.js.map +0 -1
  107. package/dist/index.d.ts.map +0 -1
  108. package/dist/index.js.map +0 -1
  109. package/dist/scoring.d.ts +0 -12
  110. package/dist/scoring.d.ts.map +0 -1
  111. package/dist/scoring.js +0 -110
  112. package/dist/scoring.js.map +0 -1
  113. package/dist/types.d.ts +0 -53
  114. package/dist/types.d.ts.map +0 -1
  115. package/dist/types.js +0 -2
  116. package/dist/types.js.map +0 -1
  117. package/dist/utils/ast-parser.d.ts +0 -46
  118. package/dist/utils/ast-parser.d.ts.map +0 -1
  119. package/dist/utils/ast-parser.js +0 -157
  120. package/dist/utils/ast-parser.js.map +0 -1
  121. package/dist/utils/config-loader.d.ts +0 -19
  122. package/dist/utils/config-loader.d.ts.map +0 -1
  123. package/dist/utils/config-loader.js +0 -31
  124. package/dist/utils/config-loader.js.map +0 -1
  125. package/dist/utils/context-detector.d.ts +0 -40
  126. package/dist/utils/context-detector.d.ts.map +0 -1
  127. package/dist/utils/context-detector.js +0 -225
  128. package/dist/utils/context-detector.js.map +0 -1
  129. package/dist/utils/scope-tracker.d.ts +0 -87
  130. package/dist/utils/scope-tracker.d.ts.map +0 -1
  131. package/dist/utils/scope-tracker.js +0 -161
  132. package/dist/utils/scope-tracker.js.map +0 -1
@@ -1,165 +0,0 @@
1
- /**
2
- * Python Naming Analyzer - PEP 8 Compliant
3
- *
4
- * Analyzes Python code for PEP 8 naming convention violations
5
- * https://peps.python.org/pep-0008/#naming-conventions
6
- */
7
- import { getParser } from '@aiready/core';
8
- /**
9
- * Analyze Python files for PEP 8 naming violations
10
- */
11
- export async function analyzePythonNaming(files) {
12
- const issues = [];
13
- const parser = getParser('dummy.py'); // Get Python parser instance
14
- if (!parser) {
15
- console.warn('Python parser not available');
16
- return issues;
17
- }
18
- // Filter to only Python files
19
- const pythonFiles = files.filter(f => f.toLowerCase().endsWith('.py'));
20
- for (const file of pythonFiles) {
21
- try {
22
- const fs = await import('fs');
23
- const code = await fs.promises.readFile(file, 'utf-8');
24
- const result = parser.parse(code, file);
25
- // Analyze each export for naming violations
26
- for (const exp of result.exports) {
27
- const nameIssue = checkPythonNaming(exp.name, exp.type, file, exp.loc?.start.line || 0);
28
- if (nameIssue) {
29
- issues.push(nameIssue);
30
- }
31
- }
32
- // Analyze imports for naming issues (optional, less critical)
33
- for (const imp of result.imports) {
34
- for (const spec of imp.specifiers) {
35
- if (spec !== '*' && spec !== 'default') {
36
- const nameIssue = checkPythonNaming(spec, 'variable', file, imp.loc?.start.line || 0);
37
- if (nameIssue) {
38
- issues.push(nameIssue);
39
- }
40
- }
41
- }
42
- }
43
- }
44
- catch (error) {
45
- console.warn(`Skipping ${file} due to error:`, error);
46
- }
47
- }
48
- return issues;
49
- }
50
- /**
51
- * Check a Python identifier against PEP 8 conventions
52
- */
53
- function checkPythonNaming(identifier, type, file, line) {
54
- // Get naming conventions from parser
55
- const parser = getParser('dummy.py');
56
- const conventions = parser?.getNamingConventions();
57
- if (!conventions)
58
- return null;
59
- // Skip special methods and exceptions
60
- if (conventions.exceptions?.includes(identifier)) {
61
- return null;
62
- }
63
- // Check based on type
64
- if (type === 'class') {
65
- // Classes should be PascalCase
66
- if (!conventions.classPattern.test(identifier)) {
67
- return {
68
- type: 'poor-naming',
69
- identifier,
70
- file,
71
- line,
72
- column: 0,
73
- severity: 'major',
74
- category: 'naming',
75
- suggestion: `Class names should use PascalCase (e.g., ${toPascalCase(identifier)})`,
76
- };
77
- }
78
- }
79
- else if (type === 'function') {
80
- // Functions should be snake_case
81
- if (!conventions.functionPattern.test(identifier)) {
82
- // Check if it's incorrectly using camelCase
83
- if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
84
- return {
85
- type: 'convention-mix',
86
- identifier,
87
- file,
88
- line,
89
- column: 0,
90
- severity: 'major',
91
- category: 'naming',
92
- suggestion: `Function names should use snake_case, not camelCase (e.g., ${toSnakeCase(identifier)})`,
93
- };
94
- }
95
- }
96
- }
97
- else if (type === 'const' || type === 'variable') {
98
- // Check if it looks like a constant (all uppercase)
99
- if (identifier === identifier.toUpperCase() && identifier.length > 1) {
100
- // Constants should be UPPER_CASE_WITH_UNDERSCORES
101
- if (!conventions.constantPattern.test(identifier)) {
102
- return {
103
- type: 'poor-naming',
104
- identifier,
105
- file,
106
- line,
107
- column: 0,
108
- severity: 'minor',
109
- category: 'naming',
110
- suggestion: 'Constants should use UPPER_CASE_WITH_UNDERSCORES',
111
- };
112
- }
113
- }
114
- else {
115
- // Regular variables should be snake_case
116
- if (!conventions.variablePattern.test(identifier)) {
117
- // Check if it's using camelCase (common mistake from JS/TS developers)
118
- if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
119
- return {
120
- type: 'convention-mix',
121
- identifier,
122
- file,
123
- line,
124
- column: 0,
125
- severity: 'major',
126
- category: 'naming',
127
- suggestion: `Variable names should use snake_case, not camelCase (e.g., ${toSnakeCase(identifier)})`,
128
- };
129
- }
130
- }
131
- }
132
- }
133
- return null;
134
- }
135
- /**
136
- * Convert camelCase to snake_case
137
- */
138
- function toSnakeCase(str) {
139
- return str
140
- .replace(/([A-Z])/g, '_$1')
141
- .toLowerCase()
142
- .replace(/^_/, '');
143
- }
144
- /**
145
- * Convert snake_case to PascalCase
146
- */
147
- function toPascalCase(str) {
148
- return str
149
- .split('_')
150
- .map(word => word.charAt(0).toUpperCase() + word.slice(1).toLowerCase())
151
- .join('');
152
- }
153
- /**
154
- * Detect common Python anti-patterns in naming
155
- */
156
- export function detectPythonNamingAntiPatterns(files) {
157
- const issues = [];
158
- // Anti-pattern 1: Using camelCase in Python (common for JS/TS developers)
159
- // Anti-pattern 2: Using PascalCase for functions
160
- // Anti-pattern 3: Not using leading underscore for private methods
161
- // Anti-pattern 4: Using single letter names outside of comprehensions
162
- // These will be implemented as we refine the analyzer
163
- return issues;
164
- }
165
- //# sourceMappingURL=naming-python.js.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"naming-python.js","sourceRoot":"","sources":["../../src/analyzers/naming-python.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,SAAS,EAAY,MAAM,eAAe,CAAC;AAGpD;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CAAC,KAAe;IACvD,MAAM,MAAM,GAAkB,EAAE,CAAC;IACjC,MAAM,MAAM,GAAG,SAAS,CAAC,UAAU,CAAC,CAAC,CAAC,6BAA6B;IAEnE,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,OAAO,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAC;QAC5C,OAAO,MAAM,CAAC;IAChB,CAAC;IAED,8BAA8B;IAC9B,MAAM,WAAW,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC;IAEvE,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;QAC/B,IAAI,CAAC;YACH,MAAM,EAAE,GAAG,MAAM,MAAM,CAAC,IAAI,CAAC,CAAC;YAC9B,MAAM,IAAI,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;YACvD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;YAExC,4CAA4C;YAC5C,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;gBACjC,MAAM,SAAS,GAAG,iBAAiB,CAAC,GAAG,CAAC,IAAI,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC;gBACxF,IAAI,SAAS,EAAE,CAAC;oBACd,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;gBACzB,CAAC;YACH,CAAC;YAED,8DAA8D;YAC9D,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;gBACjC,KAAK,MAAM,IAAI,IAAI,GAAG,CAAC,UAAU,EAAE,CAAC;oBAClC,IAAI,IAAI,KAAK,GAAG,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;wBACvC,MAAM,SAAS,GAAG,iBAAiB,CAAC,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC;wBACtF,IAAI,SAAS,EAAE,CAAC;4BACd,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;wBACzB,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,CAAC,IAAI,CAAC,YAAY,IAAI,gBAAgB,EAAE,KAAK,CAAC,CAAC;QACxD,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED;;GAEG;AACH,SAAS,iBAAiB,CACxB,UAAkB,EAClB,IAAY,EACZ,IAAY,EACZ,IAAY;IAEZ,qCAAqC;IACrC,MAAM,MAAM,GAAG,SAAS,CAAC,UAAU,CAAC,CAAC;IACrC,MAAM,WAAW,GAAG,MAAM,EAAE,oBAAoB,EAAE,CAAC;IACnD,IAAI,CAAC,WAAW;QAAE,OAAO,IAAI,CAAC;IAE9B,sCAAsC;IACtC,IAAI,WAAW,CAAC,UAAU,EAAE,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;QACjD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,sBAAsB;IACtB,IAAI,IAAI,KAAK,OAAO,EAAE,CAAC;QACrB,+BAA+B;QAC/B,IAAI,CAAC,WAAW,CAAC,YAAY,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;YAC/C,OAAO;gBACL,IAAI,EAAE,aAAa;gBACnB,UAAU;gBACV,IAAI;gBACJ,IAAI;gBACJ,MAAM,EAAE,CAAC;gBACT,QAAQ,EAAE,OAAO;gBACjB,QAAQ,EAAE,QAAQ;gBAClB,UAAU,EAAE,4CAA4C,YAAY,CAAC,UAAU,CAAC,GAAG;aACpF,CAAC;QACJ,CAAC;IACH,CAAC;SAAM,IAAI,IAAI,KAAK,UAAU,EAAE,CAAC;QAC/B,iCAAiC;QACjC,IAAI,CAAC,WAAW,CAAC,eAAe,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;YAClD,4CAA4C;YAC5C,IAAI,qBAAqB,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;gBACvE,OAAO;oBACL,IAAI,EAAE,gBAAgB;oBACtB,UAAU;oBACV,IAAI;oBACJ,IAAI;oBACJ,MAAM,EAAE,CAAC;oBACT,QAAQ,EAAE,OAAO;oBACjB,QAAQ,EAAE,QAAQ;oBAClB,UAAU,EAAE,8DAA8D,WAAW,CAAC,UAAU,CAAC,GAAG;iBACrG,CAAC;YACJ,CAAC;QACH,CAAC;IACH,CAAC;SAAM,IAAI,IAAI,KAAK,OAAO,IAAI,IAAI,KAAK,UAAU,EAAE,CAAC;QACnD,oDAAoD;QACpD,IAAI,UAAU,KAAK,UAAU,CAAC,WAAW,EAAE,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrE,kDAAkD;YAClD,IAAI,CAAC,WAAW,CAAC,eAAe,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;gBAClD,OAAO;oBACL,IAAI,EAAE,aAAa;oBACnB,UAAU;oBACV,IAAI;oBACJ,IAAI;oBACJ,MAAM,EAAE,CAAC;oBACT,QAAQ,EAAE,OAAO;oBACjB,QAAQ,EAAE,QAAQ;oBAClB,UAAU,EAAE,kDAAkD;iBAC/D,CAAC;YACJ,CAAC;QACH,CAAC;aAAM,CAAC;YACN,yCAAyC;YACzC,IAAI,CAAC,WAAW,CAAC,eAAe,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;gBAClD,uEAAuE;gBACvE,IAAI,qBAAqB,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;oBACvE,OAAO;wBACL,IAAI,EAAE,gBAAgB;wBACtB,UAAU;wBACV,IAAI;wBACJ,IAAI;wBACJ,MAAM,EAAE,CAAC;wBACT,QAAQ,EAAE,OAAO;wBACjB,QAAQ,EAAE,QAAQ;wBAClB,UAAU,EAAE,8DAA8D,WAAW,CAAC,UAAU,CAAC,GAAG;qBACrG,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;GAEG;AACH,SAAS,WAAW,CAAC,GAAW;IAC9B,OAAO,GAAG;SACP,OAAO,CAAC,UAAU,EAAE,KAAK,CAAC;SAC1B,WAAW,EAAE;SACb,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;AACvB,CAAC;AAED;;GAEG;AACH,SAAS,YAAY,CAAC,GAAW;IAC/B,OAAO,GAAG;SACP,KAAK,CAAC,GAAG,CAAC;SACV,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;SACvE,IAAI,CAAC,EAAE,CAAC,CAAC;AACd,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,8BAA8B,CAAC,KAAe;IAC5D,MAAM,MAAM,GAAkB,EAAE,CAAC;IAEjC,0EAA0E;IAC1E,iDAAiD;IACjD,mEAAmE;IACnE,sEAAsE;IAEtE,sDAAsD;IAEtD,OAAO,MAAM,CAAC;AAChB,CAAC"}
@@ -1,6 +0,0 @@
1
- import type { NamingIssue } from '../types';
2
- /**
3
- * Analyzes naming conventions and quality
4
- */
5
- export declare function analyzeNaming(files: string[]): Promise<NamingIssue[]>;
6
- //# sourceMappingURL=naming.d.ts.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"naming.d.ts","sourceRoot":"","sources":["../../src/analyzers/naming.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,UAAU,CAAC;AAQ5C;;GAEG;AACH,wBAAsB,aAAa,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAa3E"}
@@ -1,234 +0,0 @@
1
- import { readFileContent } from '@aiready/core';
2
- import { COMMON_SHORT_WORDS, ACCEPTABLE_ABBREVIATIONS, snakeCaseToCamelCase, } from './naming-constants';
3
- import { loadNamingConfig } from '../utils/config-loader';
4
- /**
5
- * Analyzes naming conventions and quality
6
- */
7
- export async function analyzeNaming(files) {
8
- const issues = [];
9
- // Load and merge configuration
10
- const { customAbbreviations, customShortWords, disabledChecks } = await loadNamingConfig(files);
11
- for (const file of files) {
12
- const content = await readFileContent(file);
13
- const fileIssues = analyzeFileNaming(file, content, customAbbreviations, customShortWords, disabledChecks);
14
- issues.push(...fileIssues);
15
- }
16
- return issues;
17
- }
18
- function analyzeFileNaming(file, content, customAbbreviations, customShortWords, disabledChecks) {
19
- const issues = [];
20
- // Check if this is a test file (more lenient rules)
21
- const isTestFile = file.match(/\.(test|spec)\.(ts|tsx|js|jsx)$/);
22
- // Split into lines for line number tracking
23
- const lines = content.split('\n');
24
- // Merge custom sets with defaults
25
- const allAbbreviations = new Set([...ACCEPTABLE_ABBREVIATIONS, ...customAbbreviations]);
26
- const allShortWords = new Set([...COMMON_SHORT_WORDS, ...customShortWords]);
27
- /**
28
- * Helper: Get context window around a line (for multi-line pattern detection)
29
- */
30
- const getContextWindow = (index, windowSize = 3) => {
31
- const start = Math.max(0, index - windowSize);
32
- const end = Math.min(lines.length, index + windowSize + 1);
33
- return lines.slice(start, end).join('\n');
34
- };
35
- /**
36
- * Helper: Check if a variable is short-lived (used only within 3-5 lines)
37
- */
38
- const isShortLivedVariable = (varName, declarationIndex) => {
39
- const searchRange = 5; // Check 5 lines after declaration
40
- const endIndex = Math.min(lines.length, declarationIndex + searchRange + 1);
41
- let usageCount = 0;
42
- for (let i = declarationIndex; i < endIndex; i++) {
43
- // Match variable name as whole word
44
- const regex = new RegExp(`\\b${varName}\\b`, 'g');
45
- const matches = lines[i].match(regex);
46
- if (matches) {
47
- usageCount += matches.length;
48
- }
49
- }
50
- // If variable is only used 2-3 times within 5 lines, it's short-lived
51
- // (1 = declaration, 1-2 = actual usage)
52
- return usageCount >= 2 && usageCount <= 3;
53
- };
54
- // Check for naming patterns
55
- lines.forEach((line, index) => {
56
- const lineNumber = index + 1;
57
- const contextWindow = getContextWindow(index);
58
- // Check for single letter variables (except i, j, k, l in loops/common contexts)
59
- if (!disabledChecks.has('single-letter')) {
60
- const singleLetterMatches = line.matchAll(/\b(?:const|let|var)\s+([a-hm-z])\s*=/gi);
61
- for (const match of singleLetterMatches) {
62
- const letter = match[1].toLowerCase();
63
- // Coverage metrics context (s/b/f/l are standard for statements/branches/functions/lines)
64
- const isCoverageContext = /coverage|summary|metrics|pct|percent/i.test(line) ||
65
- /\.(?:statements|branches|functions|lines)\.pct/i.test(line);
66
- if (isCoverageContext && ['s', 'b', 'f', 'l'].includes(letter)) {
67
- continue;
68
- }
69
- // Enhanced loop/iterator context detection
70
- const isInLoopContext = line.includes('for') ||
71
- /\.(map|filter|forEach|reduce|find|some|every)\s*\(/.test(line) ||
72
- line.includes('=>') || // Arrow function
73
- /\w+\s*=>\s*/.test(line); // Callback pattern
74
- // Check for i18n/translation context
75
- const isI18nContext = line.includes('useTranslation') ||
76
- line.includes('i18n.t') ||
77
- /\bt\s*\(['"]/.test(line); // t('key') pattern
78
- // Check for arrow function parameter (improved detection with context window)
79
- const isArrowFunctionParam = /\(\s*[a-z]\s*(?:,\s*[a-z]\s*)*\)\s*=>/.test(line) || // (s) => or (a, b) =>
80
- /[a-z]\s*=>/.test(line) || // s => on same line
81
- // Multi-line arrow function detection: look for pattern in context window
82
- new RegExp(`\\b${letter}\\s*\\)\\s*$`).test(line) && /=>/.test(contextWindow) || // (s)\n =>
83
- new RegExp(`\\.(?:map|filter|forEach|reduce|find|some|every)\\s*\\(\\s*$`).test(lines[index - 1] || '') && /=>/.test(contextWindow); // .map(\n s =>
84
- // Check if variable is short-lived (comparison/temporary contexts)
85
- const isShortLived = isShortLivedVariable(letter, index);
86
- if (!isInLoopContext && !isI18nContext && !isArrowFunctionParam && !isShortLived && !['x', 'y', 'z', 'i', 'j', 'k', 'l', 'n', 'm'].includes(letter)) {
87
- // Skip in test files unless it's really unclear
88
- if (isTestFile && ['a', 'b', 'c', 'd', 'e', 'f', 's'].includes(letter)) {
89
- continue;
90
- }
91
- issues.push({
92
- file,
93
- line: lineNumber,
94
- type: 'poor-naming',
95
- identifier: match[1],
96
- severity: 'minor',
97
- suggestion: `Use descriptive variable name instead of single letter '${match[1]}'`
98
- });
99
- }
100
- }
101
- }
102
- // Check for overly abbreviated variables
103
- if (!disabledChecks.has('abbreviation')) {
104
- const abbreviationMatches = line.matchAll(/\b(?:const|let|var)\s+([a-z]{1,3})(?=[A-Z]|_|\s*=)/g);
105
- for (const match of abbreviationMatches) {
106
- const abbrev = match[1].toLowerCase();
107
- // Skip if it's a common short English word (full word, not abbreviation)
108
- if (allShortWords.has(abbrev)) {
109
- continue;
110
- }
111
- // Skip acceptable abbreviations (including custom ones)
112
- if (allAbbreviations.has(abbrev)) {
113
- continue;
114
- }
115
- // Check for arrow function parameter context (with multi-line detection)
116
- const isArrowFunctionParam = /\(\s*[a-z]\s*(?:,\s*[a-z]\s*)*\)\s*=>/.test(line) || // (s) => or (a, b) =>
117
- new RegExp(`\\b${abbrev}\\s*=>`).test(line) || // s => on same line
118
- // Multi-line arrow function: check context window
119
- (new RegExp(`\\b${abbrev}\\s*\\)\\s*$`).test(line) && /=>/.test(contextWindow)) || // (s)\n =>
120
- (new RegExp(`\\.(?:map|filter|forEach|reduce|find|some|every)\\s*\\(\\s*$`).test(lines[index - 1] || '') &&
121
- new RegExp(`^\\s*${abbrev}\\s*=>`).test(line)); // .map(\n s =>
122
- if (isArrowFunctionParam) {
123
- continue;
124
- }
125
- // For very short names (1-2 letters), check for date/time context
126
- if (abbrev.length <= 2) {
127
- const isDateTimeContext = /date|time|day|hour|minute|second|timestamp/i.test(line);
128
- if (isDateTimeContext && ['d', 't', 'dt'].includes(abbrev)) {
129
- continue;
130
- }
131
- // Check for user/auth context
132
- const isUserContext = /user|auth|account/i.test(line);
133
- if (isUserContext && abbrev === 'u') {
134
- continue;
135
- }
136
- }
137
- issues.push({
138
- file,
139
- line: lineNumber,
140
- type: 'abbreviation',
141
- identifier: match[1],
142
- severity: 'info',
143
- suggestion: `Consider using full word instead of abbreviation '${match[1]}'`
144
- });
145
- }
146
- }
147
- // Check for snake_case vs camelCase mixing in TypeScript/JavaScript
148
- if (!disabledChecks.has('convention-mix') && file.match(/\.(ts|tsx|js|jsx)$/)) {
149
- const camelCaseVars = line.match(/\b(?:const|let|var)\s+([a-z][a-zA-Z0-9]*)\s*=/);
150
- const snakeCaseVars = line.match(/\b(?:const|let|var)\s+([a-z][a-z0-9]*_[a-z0-9_]*)\s*=/);
151
- if (snakeCaseVars) {
152
- issues.push({
153
- file,
154
- line: lineNumber,
155
- type: 'convention-mix',
156
- identifier: snakeCaseVars[1],
157
- severity: 'minor',
158
- suggestion: `Use camelCase '${snakeCaseToCamelCase(snakeCaseVars[1])}' instead of snake_case in TypeScript/JavaScript`
159
- });
160
- }
161
- }
162
- // Check for unclear boolean names (should start with is/has/should/can)
163
- if (!disabledChecks.has('unclear')) {
164
- const booleanMatches = line.matchAll(/\b(?:const|let|var)\s+([a-z][a-zA-Z0-9]*)\s*:\s*boolean/gi);
165
- for (const match of booleanMatches) {
166
- const name = match[1];
167
- if (!name.match(/^(is|has|should|can|will|did)/i)) {
168
- issues.push({
169
- file,
170
- line: lineNumber,
171
- type: 'unclear',
172
- identifier: name,
173
- severity: 'info',
174
- suggestion: `Boolean variable '${name}' should start with is/has/should/can for clarity`
175
- });
176
- }
177
- }
178
- }
179
- // Check for function names that don't indicate action
180
- if (!disabledChecks.has('unclear')) {
181
- const functionMatches = line.matchAll(/function\s+([a-z][a-zA-Z0-9]*)/g);
182
- for (const match of functionMatches) {
183
- const name = match[1];
184
- // Skip JavaScript/TypeScript keywords that shouldn't be function names
185
- const isKeyword = ['for', 'if', 'else', 'while', 'do', 'switch', 'case', 'break', 'continue', 'return', 'throw', 'try', 'catch', 'finally', 'with', 'yield', 'await'].includes(name);
186
- if (isKeyword) {
187
- continue;
188
- }
189
- // Skip common entry point names
190
- const isEntryPoint = ['main', 'init', 'setup', 'bootstrap'].includes(name);
191
- if (isEntryPoint) {
192
- continue;
193
- }
194
- // Functions should typically start with verbs, but allow:
195
- // 1. Factory/builder patterns (ends with Factory, Builder, etc.)
196
- // 2. Descriptive compound names that explain what they return
197
- // 3. Event handlers (onClick, onSubmit, etc.)
198
- // 4. Descriptive aggregate/collection patterns
199
- // 5. Very long descriptive names (>15 chars)
200
- // 6. Compound words with 3+ capitals
201
- // 7. Helper/utility functions (common patterns)
202
- // 8. React hooks (useX pattern)
203
- const isFactoryPattern = name.match(/(Factory|Builder|Creator|Generator|Provider|Adapter|Mock)$/);
204
- const isEventHandler = name.match(/^on[A-Z]/);
205
- const isDescriptiveLong = name.length > 15; // Reduced from 20 to 15
206
- const isReactHook = name.match(/^use[A-Z]/); // React hooks
207
- // Check for descriptive patterns
208
- const isDescriptivePattern = name.match(/^(default|total|count|sum|avg|max|min|initial|current|previous|next)\w+/) ||
209
- name.match(/\w+(Count|Total|Sum|Average|List|Map|Set|Config|Settings|Options|Props|Data|Info|Details|State|Status|Response|Result)$/);
210
- // Helper/utility function patterns
211
- const isHelperPattern = name.match(/^(to|from|with|without|for|as|into)\w+/) || // toMetadata, withLogger, forPath
212
- name.match(/^\w+(To|From|With|Without|For|As|Into)\w*$/); // metadataTo, pathFrom
213
- // Common utility names that are descriptive
214
- const isUtilityName = ['cn', 'proxy', 'sitemap', 'robots', 'gtag'].includes(name);
215
- // Count capital letters for compound detection
216
- const capitalCount = (name.match(/[A-Z]/g) || []).length;
217
- const isCompoundWord = capitalCount >= 3; // daysSinceLastCommit has 4 capitals
218
- const hasActionVerb = name.match(/^(get|set|is|has|can|should|create|update|delete|fetch|load|save|process|handle|validate|check|find|search|filter|map|reduce|make|do|run|start|stop|build|parse|format|render|calculate|compute|generate|transform|convert|normalize|sanitize|encode|decode|compress|extract|merge|split|join|sort|compare|test|verify|ensure|apply|execute|invoke|call|emit|dispatch|trigger|listen|subscribe|unsubscribe|add|remove|clear|reset|toggle|enable|disable|open|close|connect|disconnect|send|receive|read|write|import|export|register|unregister|mount|unmount|track|store|persist|upsert|derive|classify|combine|discover|activate|require|assert|expect|mask|escape|sign|put|list|complete|page|safe|mock|pick|pluralize|text)/);
219
- if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isDescriptivePattern && !isCompoundWord && !isHelperPattern && !isUtilityName && !isReactHook) {
220
- issues.push({
221
- file,
222
- line: lineNumber,
223
- type: 'unclear',
224
- identifier: name,
225
- severity: 'info',
226
- suggestion: `Function '${name}' should start with an action verb (get, set, create, etc.)`
227
- });
228
- }
229
- }
230
- }
231
- });
232
- return issues;
233
- }
234
- //# sourceMappingURL=naming.js.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"naming.js","sourceRoot":"","sources":["../../src/analyzers/naming.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,eAAe,CAAC;AAEhD,OAAO,EACL,kBAAkB,EAClB,wBAAwB,EACxB,oBAAoB,GACrB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAE1D;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CAAC,KAAe;IACjD,MAAM,MAAM,GAAkB,EAAE,CAAC;IAEjC,+BAA+B;IAC/B,MAAM,EAAE,mBAAmB,EAAE,gBAAgB,EAAE,cAAc,EAAE,GAAG,MAAM,gBAAgB,CAAC,KAAK,CAAC,CAAC;IAEhG,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,OAAO,GAAG,MAAM,eAAe,CAAC,IAAI,CAAC,CAAC;QAC5C,MAAM,UAAU,GAAG,iBAAiB,CAAC,IAAI,EAAE,OAAO,EAAE,mBAAmB,EAAE,gBAAgB,EAAE,cAAc,CAAC,CAAC;QAC3G,MAAM,CAAC,IAAI,CAAC,GAAG,UAAU,CAAC,CAAC;IAC7B,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,iBAAiB,CACxB,IAAY,EACZ,OAAe,EACf,mBAAgC,EAChC,gBAA6B,EAC7B,cAA2B;IAE3B,MAAM,MAAM,GAAkB,EAAE,CAAC;IAEjC,oDAAoD;IACpD,MAAM,UAAU,GAAG,IAAI,CAAC,KAAK,CAAC,iCAAiC,CAAC,CAAC;IAEjE,4CAA4C;IAC5C,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAElC,kCAAkC;IAClC,MAAM,gBAAgB,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,wBAAwB,EAAE,GAAG,mBAAmB,CAAC,CAAC,CAAC;IACxF,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,kBAAkB,EAAE,GAAG,gBAAgB,CAAC,CAAC,CAAC;IAE5E;;OAEG;IACH,MAAM,gBAAgB,GAAG,CAAC,KAAa,EAAE,aAAqB,CAAC,EAAU,EAAE;QACzE,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,KAAK,GAAG,UAAU,CAAC,CAAC;QAC9C,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,EAAE,KAAK,GAAG,UAAU,GAAG,CAAC,CAAC,CAAC;QAC3D,OAAO,KAAK,CAAC,KAAK,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5C,CAAC,CAAC;IAEF;;OAEG;IACH,MAAM,oBAAoB,GAAG,CAAC,OAAe,EAAE,gBAAwB,EAAW,EAAE;QAClF,MAAM,WAAW,GAAG,CAAC,CAAC,CAAC,kCAAkC;QACzD,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,EAAE,gBAAgB,GAAG,WAAW,GAAG,CAAC,CAAC,CAAC;QAE5E,IAAI,UAAU,GAAG,CAAC,CAAC;QACnB,KAAK,IAAI,CAAC,GAAG,gBAAgB,EAAE,CAAC,GAAG,QAAQ,EAAE,CAAC,EAAE,EAAE,CAAC;YACjD,oCAAoC;YACpC,MAAM,KAAK,GAAG,IAAI,MAAM,CAAC,MAAM,OAAO,KAAK,EAAE,GAAG,CAAC,CAAC;YAClD,MAAM,OAAO,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;YACtC,IAAI,OAAO,EAAE,CAAC;gBACZ,UAAU,IAAI,OAAO,CAAC,MAAM,CAAC;YAC/B,CAAC;QACH,CAAC;QAED,sEAAsE;QACtE,wCAAwC;QACxC,OAAO,UAAU,IAAI,CAAC,IAAI,UAAU,IAAI,CAAC,CAAC;IAC5C,CAAC,CAAC;IAEF,4BAA4B;IAC5B,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;QAC5B,MAAM,UAAU,GAAG,KAAK,GAAG,CAAC,CAAC;QAC7B,MAAM,aAAa,GAAG,gBAAgB,CAAC,KAAK,CAAC,CAAC;QAE9C,iFAAiF;QACjF,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,eAAe,CAAC,EAAE,CAAC;YACzC,MAAM,mBAAmB,GAAG,IAAI,CAAC,QAAQ,CAAC,wCAAwC,CAAC,CAAC;YACpF,KAAK,MAAM,KAAK,IAAI,mBAAmB,EAAE,CAAC;gBACxC,MAAM,MAAM,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;gBAEtC,0FAA0F;gBAC1F,MAAM,iBAAiB,GAAG,uCAAuC,CAAC,IAAI,CAAC,IAAI,CAAC;oBAC1E,iDAAiD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAC/D,IAAI,iBAAiB,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;oBAC/D,SAAS;gBACX,CAAC;gBAED,2CAA2C;gBAC3C,MAAM,eAAe,GACnB,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC;oBACpB,oDAAoD,CAAC,IAAI,CAAC,IAAI,CAAC;oBAC/D,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,iBAAiB;oBACxC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,mBAAmB;gBAE/C,qCAAqC;gBACrC,MAAM,aAAa,GACjB,IAAI,CAAC,QAAQ,CAAC,gBAAgB,CAAC;oBAC/B,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC;oBACvB,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,mBAAmB;gBAEhD,8EAA8E;gBAC9E,MAAM,oBAAoB,GACxB,uCAAuC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,sBAAsB;oBAC5E,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,oBAAoB;oBAC/C,0EAA0E;oBAC1E,IAAI,MAAM,CAAC,MAAM,MAAM,cAAc,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,aAAa,CAAC,IAAI,WAAW;oBAC5F,IAAI,MAAM,CAAC,8DAA8D,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,CAAC,gBAAgB;gBAEvJ,mEAAmE;gBACnE,MAAM,YAAY,GAAG,oBAAoB,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;gBAEzD,IAAI,CAAC,eAAe,IAAI,CAAC,aAAa,IAAI,CAAC,oBAAoB,IAAI,CAAC,YAAY,IAAI,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;oBACpJ,gDAAgD;oBAChD,IAAI,UAAU,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;wBACvE,SAAS;oBACX,CAAC;oBAED,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI;wBACJ,IAAI,EAAE,UAAU;wBAChB,IAAI,EAAE,aAAa;wBACnB,UAAU,EAAE,KAAK,CAAC,CAAC,CAAC;wBACpB,QAAQ,EAAE,OAAO;wBACjB,UAAU,EAAE,2DAA2D,KAAK,CAAC,CAAC,CAAC,GAAG;qBACnF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;QAED,yCAAyC;QACzC,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,CAAC;YACxC,MAAM,mBAAmB,GAAG,IAAI,CAAC,QAAQ,CAAC,qDAAqD,CAAC,CAAC;YACjG,KAAK,MAAM,KAAK,IAAI,mBAAmB,EAAE,CAAC;gBACxC,MAAM,MAAM,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;gBAEtC,yEAAyE;gBACzE,IAAI,aAAa,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;oBAC9B,SAAS;gBACX,CAAC;gBAED,wDAAwD;gBACxD,IAAI,gBAAgB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;oBACjC,SAAS;gBACX,CAAC;gBAED,yEAAyE;gBACzE,MAAM,oBAAoB,GACxB,uCAAuC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,sBAAsB;oBAC5E,IAAI,MAAM,CAAC,MAAM,MAAM,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,oBAAoB;oBACnE,kDAAkD;oBAClD,CAAC,IAAI,MAAM,CAAC,MAAM,MAAM,cAAc,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,IAAI,WAAW;oBAC9F,CAAC,IAAI,MAAM,CAAC,8DAA8D,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;wBACvG,IAAI,MAAM,CAAC,QAAQ,MAAM,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,gBAAgB;gBAEnE,IAAI,oBAAoB,EAAE,CAAC;oBACzB,SAAS;gBACX,CAAC;gBAED,kEAAkE;gBAClE,IAAI,MAAM,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;oBACvB,MAAM,iBAAiB,GAAG,6CAA6C,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;oBACnF,IAAI,iBAAiB,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;wBAC3D,SAAS;oBACX,CAAC;oBAED,8BAA8B;oBAC9B,MAAM,aAAa,GAAG,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;oBACtD,IAAI,aAAa,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;wBACpC,SAAS;oBACX,CAAC;gBACH,CAAC;gBAED,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI;oBACJ,IAAI,EAAE,UAAU;oBAChB,IAAI,EAAE,cAAc;oBACpB,UAAU,EAAE,KAAK,CAAC,CAAC,CAAC;oBACpB,QAAQ,EAAE,MAAM;oBAChB,UAAU,EAAE,qDAAqD,KAAK,CAAC,CAAC,CAAC,GAAG;iBAC7E,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,oEAAoE;QACpE,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,gBAAgB,CAAC,IAAI,IAAI,CAAC,KAAK,CAAC,oBAAoB,CAAC,EAAE,CAAC;YAC9E,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,+CAA+C,CAAC,CAAC;YAClF,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,uDAAuD,CAAC,CAAC;YAE1F,IAAI,aAAa,EAAE,CAAC;gBAClB,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI;oBACJ,IAAI,EAAE,UAAU;oBAChB,IAAI,EAAE,gBAAgB;oBACtB,UAAU,EAAE,aAAa,CAAC,CAAC,CAAC;oBAC5B,QAAQ,EAAE,OAAO;oBACjB,UAAU,EAAE,kBAAkB,oBAAoB,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,kDAAkD;iBACvH,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,wEAAwE;QACxE,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,SAAS,CAAC,EAAE,CAAC;YACnC,MAAM,cAAc,GAAG,IAAI,CAAC,QAAQ,CAAC,2DAA2D,CAAC,CAAC;YAClG,KAAK,MAAM,KAAK,IAAI,cAAc,EAAE,CAAC;gBACnC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;gBACtB,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,gCAAgC,CAAC,EAAE,CAAC;oBAClD,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI;wBACJ,IAAI,EAAE,UAAU;wBAChB,IAAI,EAAE,SAAS;wBACf,UAAU,EAAE,IAAI;wBAChB,QAAQ,EAAE,MAAM;wBAChB,UAAU,EAAE,qBAAqB,IAAI,mDAAmD;qBACzF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;QAED,sDAAsD;QACtD,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,SAAS,CAAC,EAAE,CAAC;YACnC,MAAM,eAAe,GAAG,IAAI,CAAC,QAAQ,CAAC,iCAAiC,CAAC,CAAC;YACzE,KAAK,MAAM,KAAK,IAAI,eAAe,EAAE,CAAC;gBACpC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;gBAEtB,uEAAuE;gBACvE,MAAM,SAAS,GAAG,CAAC,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,KAAK,EAAE,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;gBACrL,IAAI,SAAS,EAAE,CAAC;oBACd,SAAS;gBACX,CAAC;gBAED,gCAAgC;gBAChC,MAAM,YAAY,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,WAAW,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;gBAC3E,IAAI,YAAY,EAAE,CAAC;oBACjB,SAAS;gBACX,CAAC;gBAED,0DAA0D;gBAC1D,iEAAiE;gBACjE,8DAA8D;gBAC9D,8CAA8C;gBAC9C,+CAA+C;gBAC/C,6CAA6C;gBAC7C,qCAAqC;gBACrC,gDAAgD;gBAChD,gCAAgC;gBAEhC,MAAM,gBAAgB,GAAG,IAAI,CAAC,KAAK,CAAC,4DAA4D,CAAC,CAAC;gBAClG,MAAM,cAAc,GAAG,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;gBAC9C,MAAM,iBAAiB,GAAG,IAAI,CAAC,MAAM,GAAG,EAAE,CAAC,CAAC,wBAAwB;gBACpE,MAAM,WAAW,GAAG,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,CAAC,cAAc;gBAE3D,iCAAiC;gBACjC,MAAM,oBAAoB,GAAG,IAAI,CAAC,KAAK,CAAC,yEAAyE,CAAC;oBACrF,IAAI,CAAC,KAAK,CAAC,yHAAyH,CAAC,CAAC;gBAEnK,mCAAmC;gBACnC,MAAM,eAAe,GAAG,IAAI,CAAC,KAAK,CAAC,wCAAwC,CAAC,IAAI,kCAAkC;oBAC3F,IAAI,CAAC,KAAK,CAAC,4CAA4C,CAAC,CAAC,CAAC,uBAAuB;gBAExG,4CAA4C;gBAC5C,MAAM,aAAa,GAAG,CAAC,IAAI,EAAE,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;gBAElF,+CAA+C;gBAC/C,MAAM,YAAY,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;gBACzD,MAAM,cAAc,GAAG,YAAY,IAAI,CAAC,CAAC,CAAC,qCAAqC;gBAE/E,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,isBAAisB,CAAC,CAAC;gBAEpuB,IAAI,CAAC,aAAa,IAAI,CAAC,gBAAgB,IAAI,CAAC,cAAc,IAAI,CAAC,iBAAiB,IAAI,CAAC,oBAAoB,IAAI,CAAC,cAAc,IAAI,CAAC,eAAe,IAAI,CAAC,aAAa,IAAI,CAAC,WAAW,EAAE,CAAC;oBACnL,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI;wBACJ,IAAI,EAAE,UAAU;wBAChB,IAAI,EAAE,SAAS;wBACf,UAAU,EAAE,IAAI;wBAChB,QAAQ,EAAE,MAAM;wBAChB,UAAU,EAAE,aAAa,IAAI,6DAA6D;qBAC3F,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,OAAO,MAAM,CAAC;AAChB,CAAC"}
@@ -1,10 +0,0 @@
1
- import type { PatternIssue } from '../types';
2
- /**
3
- * Analyzes code pattern consistency
4
- */
5
- export declare function analyzePatterns(files: string[]): Promise<PatternIssue[]>;
6
- /**
7
- * Analyzes API design consistency
8
- */
9
- export declare function analyzeAPIDesign(files: string[]): Promise<PatternIssue[]>;
10
- //# sourceMappingURL=patterns.d.ts.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"patterns.d.ts","sourceRoot":"","sources":["../../src/analyzers/patterns.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,UAAU,CAAC;AAE7C;;GAEG;AACH,wBAAsB,eAAe,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,YAAY,EAAE,CAAC,CAgB9E;AAkMD;;GAEG;AACH,wBAAsB,gBAAgB,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,YAAY,EAAE,CAAC,CAO/E"}
@@ -1,197 +0,0 @@
1
- import { readFileContent } from '@aiready/core';
2
- /**
3
- * Analyzes code pattern consistency
4
- */
5
- export async function analyzePatterns(files) {
6
- const issues = [];
7
- // Analyze error handling patterns
8
- const errorHandlingIssues = await analyzeErrorHandling(files);
9
- issues.push(...errorHandlingIssues);
10
- // Analyze async/await patterns
11
- const asyncIssues = await analyzeAsyncPatterns(files);
12
- issues.push(...asyncIssues);
13
- // Analyze import styles
14
- const importIssues = await analyzeImportStyles(files);
15
- issues.push(...importIssues);
16
- return issues;
17
- }
18
- async function analyzeErrorHandling(files) {
19
- const patterns = {
20
- tryCatch: [],
21
- throwsError: [],
22
- returnsNull: [],
23
- returnsError: []
24
- };
25
- for (const file of files) {
26
- const content = await readFileContent(file);
27
- if (content.includes('try {') || content.includes('} catch')) {
28
- patterns.tryCatch.push(file);
29
- }
30
- if (content.match(/throw new \w*Error/)) {
31
- patterns.throwsError.push(file);
32
- }
33
- if (content.match(/return null/)) {
34
- patterns.returnsNull.push(file);
35
- }
36
- if (content.match(/return \{ error:/)) {
37
- patterns.returnsError.push(file);
38
- }
39
- }
40
- const issues = [];
41
- // Check for mixed error handling strategies
42
- const strategiesUsed = Object.values(patterns).filter(p => p.length > 0).length;
43
- if (strategiesUsed > 2) {
44
- issues.push({
45
- files: [...new Set([
46
- ...patterns.tryCatch,
47
- ...patterns.throwsError,
48
- ...patterns.returnsNull,
49
- ...patterns.returnsError
50
- ])],
51
- type: 'error-handling',
52
- description: 'Inconsistent error handling strategies across codebase',
53
- examples: [
54
- patterns.tryCatch.length > 0 ? `Try-catch used in ${patterns.tryCatch.length} files` : '',
55
- patterns.throwsError.length > 0 ? `Throws errors in ${patterns.throwsError.length} files` : '',
56
- patterns.returnsNull.length > 0 ? `Returns null in ${patterns.returnsNull.length} files` : '',
57
- patterns.returnsError.length > 0 ? `Returns error objects in ${patterns.returnsError.length} files` : ''
58
- ].filter(e => e),
59
- severity: 'major'
60
- });
61
- }
62
- return issues;
63
- }
64
- async function analyzeAsyncPatterns(files) {
65
- const patterns = {
66
- asyncAwait: [],
67
- promises: [],
68
- callbacks: []
69
- };
70
- for (const file of files) {
71
- const content = await readFileContent(file);
72
- if (content.match(/async\s+(function|\(|[a-zA-Z])/)) {
73
- patterns.asyncAwait.push(file);
74
- }
75
- if (content.match(/\.then\(/) || content.match(/\.catch\(/)) {
76
- patterns.promises.push(file);
77
- }
78
- if (content.match(/callback\s*\(/) || content.match(/\(\s*err\s*,/)) {
79
- patterns.callbacks.push(file);
80
- }
81
- }
82
- const issues = [];
83
- // Modern codebases should prefer async/await
84
- if (patterns.callbacks.length > 0 && patterns.asyncAwait.length > 0) {
85
- issues.push({
86
- files: [...patterns.callbacks, ...patterns.asyncAwait],
87
- type: 'async-style',
88
- description: 'Mixed async patterns: callbacks and async/await',
89
- examples: [
90
- `Callbacks found in: ${patterns.callbacks.slice(0, 3).join(', ')}`,
91
- `Async/await used in: ${patterns.asyncAwait.slice(0, 3).join(', ')}`
92
- ],
93
- severity: 'minor'
94
- });
95
- }
96
- // Mixing .then() chains with async/await
97
- if (patterns.promises.length > patterns.asyncAwait.length * 0.3 && patterns.asyncAwait.length > 0) {
98
- issues.push({
99
- files: patterns.promises,
100
- type: 'async-style',
101
- description: 'Consider using async/await instead of promise chains for consistency',
102
- examples: patterns.promises.slice(0, 5),
103
- severity: 'info'
104
- });
105
- }
106
- return issues;
107
- }
108
- async function analyzeImportStyles(files) {
109
- const patterns = {
110
- esModules: [],
111
- commonJS: [],
112
- mixed: []
113
- };
114
- for (const file of files) {
115
- const content = await readFileContent(file);
116
- const hasESM = content.match(/^import\s+/m);
117
- // Check for actual CommonJS require() calls, excluding:
118
- // - String literals: "require('...') or 'require('...')
119
- // - Regex patterns: /require\(/
120
- // - Comments: // require( or /* require( */
121
- const hasCJS = hasActualRequireCalls(content);
122
- if (hasESM && hasCJS) {
123
- patterns.mixed.push(file);
124
- }
125
- else if (hasESM) {
126
- patterns.esModules.push(file);
127
- }
128
- else if (hasCJS) {
129
- patterns.commonJS.push(file);
130
- }
131
- }
132
- const issues = [];
133
- // Check for mixed import styles in same file
134
- if (patterns.mixed.length > 0) {
135
- issues.push({
136
- files: patterns.mixed,
137
- type: 'import-style',
138
- description: 'Mixed ES modules and CommonJS imports in same files',
139
- examples: patterns.mixed.slice(0, 5),
140
- severity: 'major'
141
- });
142
- }
143
- // Check for inconsistent styles across project
144
- if (patterns.esModules.length > 0 && patterns.commonJS.length > 0) {
145
- const ratio = patterns.commonJS.length / (patterns.esModules.length + patterns.commonJS.length);
146
- if (ratio > 0.2 && ratio < 0.8) {
147
- issues.push({
148
- files: [...patterns.esModules, ...patterns.commonJS],
149
- type: 'import-style',
150
- description: 'Inconsistent import styles across project',
151
- examples: [
152
- `ES modules: ${patterns.esModules.length} files`,
153
- `CommonJS: ${patterns.commonJS.length} files`
154
- ],
155
- severity: 'minor'
156
- });
157
- }
158
- }
159
- return issues;
160
- }
161
- /**
162
- * Detects actual require() calls, excluding false positives
163
- * Filters out require() in:
164
- * - String literals (single/double/template quotes)
165
- * - Regex patterns
166
- * - Single-line comments (//)
167
- * - Multi-line comments
168
- */
169
- function hasActualRequireCalls(content) {
170
- // Simple heuristic: remove obvious false positives
171
- // 1. Remove single-line comments
172
- let cleaned = content.replace(/\/\/.*$/gm, '');
173
- // 2. Remove multi-line comments (non-greedy)
174
- cleaned = cleaned.replace(/\/\*[\s\S]*?\*\//g, '');
175
- // 3. Remove string literals - use simpler regex to avoid backtracking
176
- // Match strings but don't try to be perfect, just remove obvious ones
177
- cleaned = cleaned.replace(/"[^"\n]*"/g, '""');
178
- cleaned = cleaned.replace(/'[^'\n]*'/g, "''");
179
- cleaned = cleaned.replace(/`[^`]*`/g, '``');
180
- // 4. Simple regex detection: if we see /require in the line, likely a regex pattern
181
- // Remove lines that look like regex patterns with require
182
- cleaned = cleaned.replace(/\/[^/\n]*require[^/\n]*\/[gimsuvy]*/g, '');
183
- // Now check for require( in the cleaned content
184
- return /require\s*\(/.test(cleaned);
185
- }
186
- /**
187
- * Analyzes API design consistency
188
- */
189
- export async function analyzeAPIDesign(files) {
190
- // This would analyze:
191
- // - Function parameter order consistency
192
- // - Return type patterns
193
- // - Options object vs individual parameters
194
- // For now, return empty array
195
- return [];
196
- }
197
- //# sourceMappingURL=patterns.js.map