@futdevpro/dynamo-eslint 1.14.4 → 1.14.7
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.
- package/.vscode/settings.json +0 -2
- package/build/configs/base.js +29 -7
- package/build/configs/base.js.map +1 -1
- package/build/plugin/index.d.ts +8 -0
- package/build/plugin/index.d.ts.map +1 -1
- package/build/plugin/index.js +12 -0
- package/build/plugin/index.js.map +1 -1
- package/build/plugin/rules/explicit-types.d.ts.map +1 -1
- package/build/plugin/rules/explicit-types.js +16 -2
- package/build/plugin/rules/explicit-types.js.map +1 -1
- package/build/plugin/rules/import/import-order.d.ts.map +1 -1
- package/build/plugin/rules/import/import-order.js +0 -9
- package/build/plugin/rules/import/import-order.js.map +1 -1
- package/build/plugin/rules/import/no-js-import.d.ts.map +1 -1
- package/build/plugin/rules/import/no-js-import.js +12 -15
- package/build/plugin/rules/import/no-js-import.js.map +1 -1
- package/build/plugin/rules/prefer-enum-over-string-union.d.ts +4 -0
- package/build/plugin/rules/prefer-enum-over-string-union.d.ts.map +1 -0
- package/build/plugin/rules/prefer-enum-over-string-union.js +290 -0
- package/build/plugin/rules/prefer-enum-over-string-union.js.map +1 -0
- package/build/plugin/rules/prefer-enum-over-string-union.spec.d.ts +2 -0
- package/build/plugin/rules/prefer-enum-over-string-union.spec.d.ts.map +1 -0
- package/build/plugin/rules/prefer-enum-over-string-union.spec.js +505 -0
- package/build/plugin/rules/prefer-enum-over-string-union.spec.js.map +1 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.d.ts +4 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.d.ts.map +1 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.js +231 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.js.map +1 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.spec.d.ts +2 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.spec.d.ts.map +1 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.spec.js +324 -0
- package/build/plugin/rules/prefer-interface-over-duplicate-types.spec.js.map +1 -0
- package/build/plugin/rules/require-jsdoc-description.d.ts +4 -0
- package/build/plugin/rules/require-jsdoc-description.d.ts.map +1 -0
- package/build/plugin/rules/require-jsdoc-description.js +379 -0
- package/build/plugin/rules/require-jsdoc-description.js.map +1 -0
- package/build/plugin/rules/require-jsdoc-description.spec.d.ts +2 -0
- package/build/plugin/rules/require-jsdoc-description.spec.d.ts.map +1 -0
- package/build/plugin/rules/require-jsdoc-description.spec.js +452 -0
- package/build/plugin/rules/require-jsdoc-description.spec.js.map +1 -0
- package/build/plugin/rules/require-test-description-prefix.d.ts +4 -0
- package/build/plugin/rules/require-test-description-prefix.d.ts.map +1 -0
- package/build/plugin/rules/require-test-description-prefix.js +135 -0
- package/build/plugin/rules/require-test-description-prefix.js.map +1 -0
- package/build/plugin/rules/require-test-description-prefix.spec.d.ts +2 -0
- package/build/plugin/rules/require-test-description-prefix.spec.d.ts.map +1 -0
- package/build/plugin/rules/require-test-description-prefix.spec.js +371 -0
- package/build/plugin/rules/require-test-description-prefix.spec.js.map +1 -0
- package/build/scripts/eslintrc-audit.js.map +1 -1
- package/futdevpro-dynamo-eslint-1.14.7.tgz +0 -0
- package/package.json +1 -1
- package/samples/package.json.example +1 -1
- package/src/configs/base.ts +45 -23
- package/src/plugin/index.ts +12 -0
- package/src/plugin/rules/explicit-types.ts +19 -2
- package/src/plugin/rules/import/import-order.ts +0 -9
- package/src/plugin/rules/import/no-js-import.ts +19 -17
- package/src/plugin/rules/prefer-enum-over-string-union.spec.ts +583 -0
- package/src/plugin/rules/prefer-enum-over-string-union.ts +333 -0
- package/src/plugin/rules/prefer-interface-over-duplicate-types.spec.ts +374 -0
- package/src/plugin/rules/prefer-interface-over-duplicate-types.ts +276 -0
- package/src/plugin/rules/require-jsdoc-description.spec.ts +542 -0
- package/src/plugin/rules/require-jsdoc-description.ts +436 -0
- package/src/plugin/rules/require-test-description-prefix.spec.ts +459 -0
- package/src/plugin/rules/require-test-description-prefix.ts +153 -0
- package/src/scripts/eslintrc-audit.ts +8 -6
- package/futdevpro-dynamo-eslint-01.14.4.tgz +0 -0
|
@@ -0,0 +1,276 @@
|
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
interface TypeUsage {
|
|
4
|
+
node: any;
|
|
5
|
+
signature: string;
|
|
6
|
+
interfaceName?: string;
|
|
7
|
+
}
|
|
8
|
+
|
|
9
|
+
interface RuleOptions {
|
|
10
|
+
threshold?: number;
|
|
11
|
+
}
|
|
12
|
+
|
|
13
|
+
const rule: Rule.RuleModule = {
|
|
14
|
+
meta: {
|
|
15
|
+
type: 'suggestion',
|
|
16
|
+
docs: {
|
|
17
|
+
description: 'Suggest creating interfaces for object types used more than the threshold number of times',
|
|
18
|
+
recommended: true,
|
|
19
|
+
},
|
|
20
|
+
schema: [
|
|
21
|
+
{
|
|
22
|
+
type: 'object',
|
|
23
|
+
properties: {
|
|
24
|
+
threshold: {
|
|
25
|
+
type: 'number',
|
|
26
|
+
minimum: 2,
|
|
27
|
+
default: 3,
|
|
28
|
+
},
|
|
29
|
+
},
|
|
30
|
+
additionalProperties: false,
|
|
31
|
+
},
|
|
32
|
+
],
|
|
33
|
+
messages: {
|
|
34
|
+
preferInterface: 'Object type used {{count}} times. Consider creating interface "{{interfaceName}}"}}',
|
|
35
|
+
},
|
|
36
|
+
fixable: 'code',
|
|
37
|
+
},
|
|
38
|
+
create(context) {
|
|
39
|
+
const options: RuleOptions = context.options[0] || {};
|
|
40
|
+
const threshold = options.threshold || 3;
|
|
41
|
+
const sourceCode = context.sourceCode;
|
|
42
|
+
|
|
43
|
+
// Map to track type signatures and their usages
|
|
44
|
+
const typeUsages = new Map<string, TypeUsage[]>();
|
|
45
|
+
const allTypeLiterals: any[] = [];
|
|
46
|
+
|
|
47
|
+
/**
|
|
48
|
+
* Normalize object type to a signature string for comparison
|
|
49
|
+
*/
|
|
50
|
+
function normalizeObjectType(node: any): string {
|
|
51
|
+
if (!node || node.type !== 'TSTypeLiteral') {
|
|
52
|
+
return '';
|
|
53
|
+
}
|
|
54
|
+
|
|
55
|
+
if (!node.members || !Array.isArray(node.members)) {
|
|
56
|
+
return '';
|
|
57
|
+
}
|
|
58
|
+
|
|
59
|
+
// Extract and sort properties
|
|
60
|
+
const properties = node.members
|
|
61
|
+
.filter((member: any) => member.type === 'TSPropertySignature')
|
|
62
|
+
.map((member: any) => {
|
|
63
|
+
const key = member.key?.name || member.key?.value || 'unknown';
|
|
64
|
+
const type = getTypeString(member.typeAnnotation);
|
|
65
|
+
const optional = member.optional ? '?' : '';
|
|
66
|
+
const readonly = member.readonly ? 'readonly ' : '';
|
|
67
|
+
|
|
68
|
+
return `${readonly}${key}${optional}:${type}`;
|
|
69
|
+
})
|
|
70
|
+
.sort(); // Sort alphabetically
|
|
71
|
+
|
|
72
|
+
return properties.join(';');
|
|
73
|
+
}
|
|
74
|
+
|
|
75
|
+
/**
|
|
76
|
+
* Get string representation of a type annotation
|
|
77
|
+
*/
|
|
78
|
+
function getTypeString(typeAnnotation: any): string {
|
|
79
|
+
if (!typeAnnotation || !typeAnnotation.typeAnnotation) {
|
|
80
|
+
return 'any';
|
|
81
|
+
}
|
|
82
|
+
|
|
83
|
+
const type = typeAnnotation.typeAnnotation;
|
|
84
|
+
|
|
85
|
+
switch (type.type) {
|
|
86
|
+
case 'TSStringKeyword':
|
|
87
|
+
return 'string';
|
|
88
|
+
case 'TSNumberKeyword':
|
|
89
|
+
return 'number';
|
|
90
|
+
case 'TSBooleanKeyword':
|
|
91
|
+
return 'boolean';
|
|
92
|
+
case 'TSVoidKeyword':
|
|
93
|
+
return 'void';
|
|
94
|
+
case 'TSNullKeyword':
|
|
95
|
+
return 'null';
|
|
96
|
+
case 'TSUndefinedKeyword':
|
|
97
|
+
return 'undefined';
|
|
98
|
+
case 'TSAnyKeyword':
|
|
99
|
+
return 'any';
|
|
100
|
+
case 'TSUnknownKeyword':
|
|
101
|
+
return 'unknown';
|
|
102
|
+
case 'TSNeverKeyword':
|
|
103
|
+
return 'never';
|
|
104
|
+
case 'TSObjectKeyword':
|
|
105
|
+
return 'object';
|
|
106
|
+
case 'TSLiteralType':
|
|
107
|
+
return type.literal?.value?.toString() || 'literal';
|
|
108
|
+
case 'TSArrayType':
|
|
109
|
+
return `${getTypeString({ typeAnnotation: type.elementType })}[]`;
|
|
110
|
+
case 'TSTypeReference':
|
|
111
|
+
return type.typeName?.name || 'reference';
|
|
112
|
+
case 'TSTypeLiteral':
|
|
113
|
+
return normalizeObjectType(type);
|
|
114
|
+
case 'TSUnionType':
|
|
115
|
+
return type.types?.map((t: any) => getTypeString({ typeAnnotation: t })).join('|') || 'union';
|
|
116
|
+
case 'TSIntersectionType':
|
|
117
|
+
return type.types?.map((t: any) => getTypeString({ typeAnnotation: t })).join('&') || 'intersection';
|
|
118
|
+
|
|
119
|
+
default:
|
|
120
|
+
return 'complex';
|
|
121
|
+
}
|
|
122
|
+
}
|
|
123
|
+
|
|
124
|
+
/**
|
|
125
|
+
* Generate interface name based on context
|
|
126
|
+
*/
|
|
127
|
+
function generateInterfaceName(signature: string, usageCount: number): string {
|
|
128
|
+
// Try to extract meaningful name from signature
|
|
129
|
+
const properties = signature.split(';');
|
|
130
|
+
|
|
131
|
+
if (properties.length === 1) {
|
|
132
|
+
const prop = properties[0].split(':')[0].replace('readonly ', '').replace('?', '');
|
|
133
|
+
|
|
134
|
+
if (prop && prop !== 'unknown') {
|
|
135
|
+
return `${prop.charAt(0).toUpperCase() + prop.slice(1)}Interface`;
|
|
136
|
+
}
|
|
137
|
+
}
|
|
138
|
+
|
|
139
|
+
// Fallback to generic name
|
|
140
|
+
return `GeneratedInterface${usageCount}`;
|
|
141
|
+
}
|
|
142
|
+
|
|
143
|
+
/**
|
|
144
|
+
* Check if interface name conflicts with existing identifiers
|
|
145
|
+
*/
|
|
146
|
+
function isNameConflict(name: string): boolean {
|
|
147
|
+
try {
|
|
148
|
+
const scope = context.sourceCode.getScope(context.sourceCode.ast);
|
|
149
|
+
|
|
150
|
+
return scope.variables.some(variable => variable.name === name);
|
|
151
|
+
} catch {
|
|
152
|
+
return false;
|
|
153
|
+
}
|
|
154
|
+
}
|
|
155
|
+
|
|
156
|
+
/**
|
|
157
|
+
* Generate unique interface name
|
|
158
|
+
*/
|
|
159
|
+
function generateUniqueInterfaceName(baseName: string): string {
|
|
160
|
+
let name = baseName;
|
|
161
|
+
let counter = 1;
|
|
162
|
+
|
|
163
|
+
while (isNameConflict(name)) {
|
|
164
|
+
name = `${baseName}${counter}`;
|
|
165
|
+
counter++;
|
|
166
|
+
}
|
|
167
|
+
|
|
168
|
+
return name;
|
|
169
|
+
}
|
|
170
|
+
|
|
171
|
+
return {
|
|
172
|
+
TSTypeLiteral(node: any) {
|
|
173
|
+
try {
|
|
174
|
+
const signature = normalizeObjectType(node);
|
|
175
|
+
|
|
176
|
+
if (!signature) {
|
|
177
|
+
return;
|
|
178
|
+
}
|
|
179
|
+
|
|
180
|
+
allTypeLiterals.push(node);
|
|
181
|
+
|
|
182
|
+
if (!typeUsages.has(signature)) {
|
|
183
|
+
typeUsages.set(signature, []);
|
|
184
|
+
}
|
|
185
|
+
|
|
186
|
+
typeUsages.get(signature)!.push({
|
|
187
|
+
node,
|
|
188
|
+
signature,
|
|
189
|
+
});
|
|
190
|
+
} catch (error) {
|
|
191
|
+
console.error('[prefer-interface-over-duplicate-types] Error in TSTypeLiteral visitor:', error);
|
|
192
|
+
}
|
|
193
|
+
},
|
|
194
|
+
|
|
195
|
+
'Program:exit'(node: any) {
|
|
196
|
+
try {
|
|
197
|
+
// Process all collected type usages
|
|
198
|
+
for (const [ signature, usages ] of typeUsages) {
|
|
199
|
+
if (usages.length >= threshold) {
|
|
200
|
+
const interfaceName = generateUniqueInterfaceName(
|
|
201
|
+
generateInterfaceName(signature, usages.length)
|
|
202
|
+
);
|
|
203
|
+
|
|
204
|
+
// Report on each usage
|
|
205
|
+
for (const usage of usages) {
|
|
206
|
+
context.report({
|
|
207
|
+
node: usage.node,
|
|
208
|
+
messageId: 'preferInterface',
|
|
209
|
+
data: {
|
|
210
|
+
count: usages.length.toString(),
|
|
211
|
+
interfaceName,
|
|
212
|
+
},
|
|
213
|
+
fix(fixer) {
|
|
214
|
+
try {
|
|
215
|
+
// Generate interface declaration
|
|
216
|
+
const properties = signature.split(';').map(prop => {
|
|
217
|
+
const [ key, type ] = prop.split(':');
|
|
218
|
+
|
|
219
|
+
return ` ${key}: ${type};`;
|
|
220
|
+
}).join('\n');
|
|
221
|
+
|
|
222
|
+
const interfaceDeclaration = `interface ${interfaceName} {\n${properties}\n}`;
|
|
223
|
+
|
|
224
|
+
// Find insertion point (after imports, before other code)
|
|
225
|
+
const firstNode = sourceCode.ast.body[0];
|
|
226
|
+
let insertPosition = 0;
|
|
227
|
+
|
|
228
|
+
if (firstNode && firstNode.type === 'ImportDeclaration') {
|
|
229
|
+
// Find last import
|
|
230
|
+
let lastImport = firstNode;
|
|
231
|
+
|
|
232
|
+
for (const node of sourceCode.ast.body) {
|
|
233
|
+
if (node.type === 'ImportDeclaration') {
|
|
234
|
+
lastImport = node;
|
|
235
|
+
} else {
|
|
236
|
+
break;
|
|
237
|
+
}
|
|
238
|
+
}
|
|
239
|
+
insertPosition = lastImport.range![1];
|
|
240
|
+
}
|
|
241
|
+
|
|
242
|
+
// Add interface declaration
|
|
243
|
+
const fixes = [
|
|
244
|
+
fixer.insertTextAfterRange([ insertPosition, insertPosition ], `\n\n${interfaceDeclaration}\n`),
|
|
245
|
+
];
|
|
246
|
+
|
|
247
|
+
// Replace all usages with interface reference
|
|
248
|
+
for (const usage of usages) {
|
|
249
|
+
const nodeText = sourceCode.getText(usage.node);
|
|
250
|
+
const replacement = interfaceName;
|
|
251
|
+
|
|
252
|
+
fixes.push(
|
|
253
|
+
fixer.replaceText(usage.node, replacement)
|
|
254
|
+
);
|
|
255
|
+
}
|
|
256
|
+
|
|
257
|
+
return fixes;
|
|
258
|
+
} catch (fixError) {
|
|
259
|
+
console.error('[prefer-interface-over-duplicate-types] Error in fix function:', fixError);
|
|
260
|
+
|
|
261
|
+
return null;
|
|
262
|
+
}
|
|
263
|
+
},
|
|
264
|
+
});
|
|
265
|
+
}
|
|
266
|
+
}
|
|
267
|
+
}
|
|
268
|
+
} catch (error) {
|
|
269
|
+
console.error('[prefer-interface-over-duplicate-types] Error in Program:exit visitor:', error);
|
|
270
|
+
}
|
|
271
|
+
},
|
|
272
|
+
};
|
|
273
|
+
},
|
|
274
|
+
};
|
|
275
|
+
|
|
276
|
+
export default rule;
|