Skip to content

Commit 3383e7e

Browse files
authored
Merge branch 'main' into localden/aaronpk
2 parents f4bdabd + 46c864c commit 3383e7e

2 files changed

Lines changed: 108 additions & 44 deletions

File tree

scripts/validate-config.ts

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,49 @@ for (const member of MEMBERS) {
5555
}
5656
}
5757

58+
// Validate member ordering in users.ts
59+
console.log('Validating member ordering in users.ts...');
60+
{
61+
// Split members into github members and email-only members
62+
const githubMembers: (typeof MEMBERS)[number][] = [];
63+
const emailOnlyMembers: (typeof MEMBERS)[number][] = [];
64+
65+
for (const member of MEMBERS) {
66+
if (member.github) {
67+
githubMembers.push(member);
68+
} else if (member.email) {
69+
emailOnlyMembers.push(member);
70+
}
71+
}
72+
73+
// Check that github members come before email-only members
74+
let foundEmailOnly = false;
75+
for (const member of MEMBERS) {
76+
if (!member.github && member.email) {
77+
foundEmailOnly = true;
78+
} else if (member.github && foundEmailOnly) {
79+
console.error(
80+
`ERROR: Member "${member.github}" appears after email-only members. GitHub members should come before email-only members.`
81+
);
82+
hasErrors = true;
83+
break;
84+
}
85+
}
86+
87+
// Check that github members are sorted alphabetically (case-insensitive)
88+
for (let i = 1; i < githubMembers.length; i++) {
89+
const prev = githubMembers[i - 1].github!.toLowerCase();
90+
const curr = githubMembers[i].github!.toLowerCase();
91+
if (prev > curr) {
92+
console.error(
93+
`ERROR: Members are not sorted alphabetically. "${githubMembers[i - 1].github}" should come after "${githubMembers[i].github}".`
94+
);
95+
hasErrors = true;
96+
break;
97+
}
98+
}
99+
}
100+
58101
// Validate parent role references in roles.ts
59102
console.log('Validating parent role references in roles.ts...');
60103
for (const role of ROLES) {

src/config/users.ts

Lines changed: 65 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -7,35 +7,6 @@ export const MEMBERS: readonly Member[] = [
77
discord: '1360717264051241071',
88
memberOf: [ROLE_IDS.CORE_MAINTAINERS],
99
},
10-
{
11-
github: 'CodeWithKyrian',
12-
discord: '951883230250946633',
13-
memberOf: [ROLE_IDS.PHP_SDK],
14-
},
15-
{
16-
github: 'D-McAdams',
17-
discord: '1364696680980545697',
18-
memberOf: [ROLE_IDS.AUTH_WG],
19-
},
20-
{
21-
github: 'Kehrlann',
22-
discord: '1112624611901837373',
23-
memberOf: [ROLE_IDS.JAVA_SDK],
24-
},
25-
{
26-
github: 'Kludex',
27-
discord: '247021664624312322',
28-
memberOf: [ROLE_IDS.PYTHON_SDK],
29-
},
30-
{
31-
github: 'Nyholm',
32-
discord: '466593085984342016',
33-
memberOf: [ROLE_IDS.PHP_SDK],
34-
},
35-
{
36-
github: 'Ololoshechkin',
37-
memberOf: [ROLE_IDS.KOTLIN_SDK],
38-
},
3910
{
4011
github: 'a-akimov',
4112
discord: '1365254196621738116',
@@ -120,10 +91,20 @@ export const MEMBERS: readonly Member[] = [
12091
ROLE_IDS.WORKING_GROUPS,
12192
],
12293
},
94+
{
95+
github: 'CodeWithKyrian',
96+
discord: '951883230250946633',
97+
memberOf: [ROLE_IDS.PHP_SDK],
98+
},
12399
{
124100
github: 'crondinini-ant',
125101
memberOf: [],
126102
},
103+
{
104+
github: 'D-McAdams',
105+
discord: '1364696680980545697',
106+
memberOf: [ROLE_IDS.AUTH_WG],
107+
},
127108
{
128109
github: 'devcrocod',
129110
memberOf: [ROLE_IDS.KOTLIN_SDK],
@@ -134,10 +115,27 @@ export const MEMBERS: readonly Member[] = [
134115
discord: '102128241715716096',
135116
memberOf: [ROLE_IDS.MCPB_MAINTAINERS, ROLE_IDS.REGISTRY_MAINTAINERS],
136117
},
118+
{
119+
github: 'dsp',
120+
memberOf: [
121+
ROLE_IDS.AUTH_WG,
122+
ROLE_IDS.LEAD_MAINTAINERS,
123+
ROLE_IDS.CORE_MAINTAINERS,
124+
ROLE_IDS.DOCS_MAINTAINERS,
125+
ROLE_IDS.GO_SDK,
126+
ROLE_IDS.FINANCIAL_SERVICES_IG,
127+
ROLE_IDS.MODERATORS,
128+
ROLE_IDS.PHP_SDK,
129+
ROLE_IDS.PYTHON_SDK,
130+
ROLE_IDS.SECURITY_WG,
131+
ROLE_IDS.TRANSPORT_WG,
132+
ROLE_IDS.TYPESCRIPT_SDK,
133+
],
134+
},
137135
{
138136
github: 'dsp-ant',
139137
email: 'david@modelcontextprotocol.io',
140-
discord: '166107790262272000', // Example Discord user ID - replace with real ID
138+
discord: '166107790262272000',
141139
memberOf: [
142140
ROLE_IDS.AUTH_WG,
143141
ROLE_IDS.LEAD_MAINTAINERS,
@@ -234,10 +232,20 @@ export const MEMBERS: readonly Member[] = [
234232
email: 'justin@modelcontextprotocol.io',
235233
memberOf: [ROLE_IDS.CORE_MAINTAINERS],
236234
},
235+
{
236+
github: 'Kehrlann',
237+
discord: '1112624611901837373',
238+
memberOf: [ROLE_IDS.JAVA_SDK],
239+
},
237240
{
238241
github: 'KKonstantinov',
239242
memberOf: [ROLE_IDS.INSPECTOR_MAINTAINERS, ROLE_IDS.TYPESCRIPT_SDK],
240243
},
244+
{
245+
github: 'Kludex',
246+
discord: '247021664624312322',
247+
memberOf: [ROLE_IDS.PYTHON_SDK],
248+
},
241249
{
242250
github: 'koic',
243251
discord: '880937364208361483',
@@ -298,27 +306,27 @@ export const MEMBERS: readonly Member[] = [
298306
github: 'mattt',
299307
memberOf: [ROLE_IDS.SWIFT_SDK],
300308
},
301-
{
302-
github: 'maxisbey',
303-
memberOf: [ROLE_IDS.PYTHON_SDK],
304-
},
305309
{
306310
github: 'mattzcarey',
307311
memberOf: [ROLE_IDS.TYPESCRIPT_SDK],
308312
},
309313
{
310-
github: 'michaelneale',
311-
memberOf: [ROLE_IDS.RUST_SDK],
314+
github: 'maxisbey',
315+
memberOf: [ROLE_IDS.PYTHON_SDK],
312316
},
313317
{
314-
github: 'movetz',
315-
memberOf: [ROLE_IDS.SWIFT_SDK],
318+
github: 'michaelneale',
319+
memberOf: [ROLE_IDS.RUST_SDK],
316320
},
317321
{
318322
github: 'mikekistler',
319323
discord: '915345005982408754',
320324
memberOf: [ROLE_IDS.CSHARP_SDK],
321325
},
326+
{
327+
github: 'movetz',
328+
memberOf: [ROLE_IDS.SWIFT_SDK],
329+
},
322330
{
323331
github: 'nickcoai',
324332
discord: '1153783469860732968',
@@ -328,6 +336,11 @@ export const MEMBERS: readonly Member[] = [
328336
github: 'nicolas-grekas',
329337
memberOf: [ROLE_IDS.PHP_SDK],
330338
},
339+
{
340+
github: 'Nyholm',
341+
discord: '466593085984342016',
342+
memberOf: [ROLE_IDS.PHP_SDK],
343+
},
331344
{
332345
github: 'ochafik',
333346
discord: '1004897332069925024',
@@ -353,6 +366,10 @@ export const MEMBERS: readonly Member[] = [
353366
ROLE_IDS.MODERATORS,
354367
],
355368
},
369+
{
370+
github: 'Ololoshechkin',
371+
memberOf: [ROLE_IDS.KOTLIN_SDK],
372+
},
356373
{
357374
github: 'pcarleton',
358375
discord: '1354465170969067852',
@@ -374,6 +391,10 @@ export const MEMBERS: readonly Member[] = [
374391
github: 'petery-ant',
375392
memberOf: [ROLE_IDS.SECURITY_WG],
376393
},
394+
{
395+
github: 'pja-ant',
396+
memberOf: [ROLE_IDS.CORE_MAINTAINERS],
397+
},
377398
{
378399
github: 'pronskiy',
379400
memberOf: [ROLE_IDS.PHP_SDK],
@@ -385,8 +406,8 @@ export const MEMBERS: readonly Member[] = [
385406
},
386407
{
387408
github: 'rdimitrov',
388-
discord: '1088231882979815424',
389409
email: 'radoslav@modelcontextprotocol.io',
410+
discord: '1088231882979815424',
390411
memberOf: [ROLE_IDS.MAINTAINERS, ROLE_IDS.REGISTRY_MAINTAINERS],
391412
},
392413
{
@@ -402,19 +423,19 @@ export const MEMBERS: readonly Member[] = [
402423
github: 'sdubov',
403424
memberOf: [ROLE_IDS.KOTLIN_SDK],
404425
},
405-
{
406-
github: 'stephentoub',
407-
memberOf: [ROLE_IDS.CSHARP_SDK],
408-
},
409426
{
410427
github: 'stallent',
411428
discord: '1137898074086314136',
412429
memberOf: [ROLE_IDS.SWIFT_SDK],
413430
},
431+
{
432+
github: 'stephentoub',
433+
memberOf: [ROLE_IDS.CSHARP_SDK],
434+
},
414435
{
415436
github: 'tadasant',
416-
discord: '400092503677599754',
417437
email: 'tadas@modelcontextprotocol.io',
438+
discord: '400092503677599754',
418439
memberOf: [
419440
ROLE_IDS.COMMUNITY_MANAGERS,
420441
ROLE_IDS.MODERATORS,

0 commit comments

Comments
 (0)