-
-
Notifications
You must be signed in to change notification settings - Fork 100
/
strict-id-in-types.ts
195 lines (176 loc) · 5.58 KB
/
strict-id-in-types.ts
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
import { Kind, ObjectTypeDefinitionNode } from 'graphql';
import { GraphQLESTreeNode } from '../estree-parser';
import { GraphQLESLintRule } from '../types';
import { getLocation } from '../utils';
export interface ExceptionRule {
types?: string[];
suffixes?: string[];
}
type StrictIdInTypesRuleConfig = [
{
acceptedIdNames?: string[];
acceptedIdTypes?: string[];
exceptions?: ExceptionRule;
}
];
interface ShouldIgnoreNodeParams {
node: GraphQLESTreeNode<ObjectTypeDefinitionNode>;
exceptions: ExceptionRule;
}
const shouldIgnoreNode = ({ node, exceptions }: ShouldIgnoreNodeParams): boolean => {
const rawNode = node.rawNode();
if (exceptions.types && exceptions.types.includes(rawNode.name.value)) {
return true;
}
if (exceptions.suffixes && exceptions.suffixes.some(suffix => rawNode.name.value.endsWith(suffix))) {
return true;
}
return false;
};
const rule: GraphQLESLintRule<StrictIdInTypesRuleConfig> = {
meta: {
type: 'suggestion',
docs: {
description:
'Requires output types to have one unique identifier unless they do not have a logical one. Exceptions can be used to ignore output types that do not have unique identifiers.',
category: 'Best Practices',
recommended: true,
url: 'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/strict-id-in-types.md',
examples: [
{
title: 'Incorrect',
usage: [{ acceptedIdNames: ['id', '_id'], acceptedIdTypes: ['ID'], exceptions: { suffixes: ['Payload'] } }],
code: /* GraphQL */ `
# Incorrect field name
type InvalidFieldName {
key: ID!
}
# Incorrect field type
type InvalidFieldType {
id: String!
}
# Incorrect exception suffix
type InvalidSuffixResult {
data: String!
}
# Too many unique identifiers. Must only contain one.
type InvalidFieldName {
id: ID!
_id: ID!
}
`,
},
{
title: 'Correct',
usage: [
{
acceptedIdNames: ['id', '_id'],
acceptedIdTypes: ['ID'],
exceptions: { types: ['Error'], suffixes: ['Payload'] },
},
],
code: /* GraphQL */ `
type User {
id: ID!
}
type Post {
_id: ID!
}
type CreateUserPayload {
data: String!
}
type Error {
message: String!
}
`,
},
],
},
schema: {
$schema: 'http://json-schema.org/draft-04/schema#',
type: 'array',
items: {
type: 'object',
properties: {
acceptedIdNames: {
type: 'array',
items: {
type: 'string',
},
default: ['id'],
},
acceptedIdTypes: {
type: 'array',
items: {
type: 'string',
},
default: ['ID'],
},
exceptions: {
type: 'object',
properties: {
types: {
type: 'array',
description: 'This is used to exclude types with names that match one of the specified values.',
items: {
type: 'string',
},
default: [],
},
suffixes: {
type: 'array',
description:
'This is used to exclude types with names with suffixes that match one of the specified values.',
items: {
type: 'string',
},
default: [],
},
},
},
},
},
},
},
create(context) {
const options: StrictIdInTypesRuleConfig[number] = {
acceptedIdNames: ['id'],
acceptedIdTypes: ['ID'],
exceptions: {},
...(context.options[0] || {}),
};
return {
ObjectTypeDefinition(node) {
if (shouldIgnoreNode({ node, exceptions: options.exceptions })) {
return;
}
const validIds = node.fields.filter(field => {
const fieldNode = field.rawNode();
const isValidIdName = options.acceptedIdNames.includes(fieldNode.name.value);
// To be a valid type, it must be non-null and one of the accepted types.
let isValidIdType = false;
if (fieldNode.type.kind === Kind.NON_NULL_TYPE && fieldNode.type.type.kind === Kind.NAMED_TYPE) {
isValidIdType = options.acceptedIdTypes.includes(fieldNode.type.type.name.value);
}
return isValidIdName && isValidIdType;
});
const typeName = node.name.value;
// Usually, there should be only one unique identifier field per type.
// Some clients allow multiple fields to be used. If more people need this,
// we can extend this rule later.
if (validIds.length !== 1) {
context.report({
loc: getLocation(node.name.loc, typeName),
message: `{{ typeName }} must have exactly one non-nullable unique identifier. Accepted name(s): {{ acceptedNamesString }} ; Accepted type(s): {{ acceptedTypesString }}`,
data: {
typeName,
acceptedNamesString: options.acceptedIdNames.join(','),
acceptedTypesString: options.acceptedIdTypes.join(','),
},
});
}
},
};
},
};
export default rule;