Update csharp-sdk repo access to use team assignments (#76)

* Add csharp-sdk-admin team and split csharp-sdk repo permissions

Introduce a csharp-sdk-admin team for admin access to the csharp-sdk repo,
while the existing csharp-sdk team is changed to push (write) access. This
separates admin privileges from general write access.

- Add CSHARP_SDK_ADMIN role to roleIds.ts and roles.ts
- Update repoAccess.ts: csharp-sdk-admin (admin), csharp-sdk (push), PederHP (triage)
- Add jeffhandley, ericstj, jozkee, tarekgh as new members in users.ts
- Move dend/localden from csharp-sdk to csharp-sdk-admin team
- Add csharp-sdk-admin membership for halter73, mikekistler, stephentoub
- Remove MackinnonBuck (admin) direct access per request

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix alphabetical ordering of new users in users.ts

Move ericstj after erain and jeffhandley before jenn-newton to satisfy
the case-insensitive alphabetical sort check in validate-config.ts.

🏠 Remote-Dev: homespace

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Den Delimarsky <den@anthropic.com>

src/config/repoAccess.ts

@@ -184,14 +184,11 @@ export const REPOSITORY_ACCESS: RepositoryAccess[] = [
   },
   {
     repository: 'csharp-sdk',
-    teams: [{ team: 'csharp-sdk', permission: 'admin' }],
-    users: [
-      { username: 'jeffhandley', permission: 'admin' },
-      { username: 'MackinnonBuck', permission: 'admin' },
-      { username: 'jozkee', permission: 'admin' },
-      { username: 'PederHP', permission: 'triage' },
-      { username: 'tarekgh', permission: 'push' },
+    teams: [
+      { team: 'csharp-sdk-admin', permission: 'admin' },
+      { team: 'csharp-sdk', permission: 'push' },
     ],
+    users: [{ username: 'PederHP', permission: 'triage' }],
   },
   {
     repository: 'go-sdk',

src/config/roleIds.ts

@@ -29,6 +29,7 @@ export const ROLE_IDS = {
   // ===================
   SDK_MAINTAINERS: 'sdk-maintainers',
   CSHARP_SDK: 'csharp-sdk',
+  CSHARP_SDK_ADMIN: 'csharp-sdk-admin',
   GO_SDK: 'go-sdk',
   JAVA_SDK: 'java-sdk',
   KOTLIN_SDK: 'kotlin-sdk',

src/config/roles.ts

@@ -160,6 +160,12 @@ export const ROLES: readonly Role[] = [
     github: { team: 'csharp-sdk', parent: ROLE_IDS.SDK_MAINTAINERS },
     discord: { role: 'c# sdk maintainers (synced)' },
   },
+  {
+    id: ROLE_IDS.CSHARP_SDK_ADMIN,
+    description: 'C# SDK repository admins',
+    github: { team: 'csharp-sdk-admin', parent: ROLE_IDS.CSHARP_SDK },
+    // GitHub only - for repo admin access
+  },
   {
     id: ROLE_IDS.GO_SDK,
     description: 'The Go SDK Team',

src/config/users.ts

@@ -152,7 +152,7 @@ export const MEMBERS: readonly Member[] = [
       ROLE_IDS.AUTH_MAINTAINERS,
       ROLE_IDS.CORE_MAINTAINERS,
       ROLE_IDS.DOCS_MAINTAINERS,
-      ROLE_IDS.CSHARP_SDK,
+      ROLE_IDS.CSHARP_SDK_ADMIN,
       ROLE_IDS.ADMINISTRATORS,
       ROLE_IDS.GO_SDK,
       ROLE_IDS.FINANCIAL_SERVICES_IG,
@@ -229,6 +229,10 @@ export const MEMBERS: readonly Member[] = [
     github: 'erain',
     memberOf: [ROLE_IDS.SKILLS_OVER_MCP_IG],
   },
+  {
+    github: 'ericstj',
+    memberOf: [ROLE_IDS.CSHARP_SDK],
+  },
   {
     github: 'evalstate',
     discord: '779268016121577492',
@@ -262,7 +266,7 @@ export const MEMBERS: readonly Member[] = [
   },
   {
     github: 'halter73',
-    memberOf: [ROLE_IDS.CSHARP_SDK],
+    memberOf: [ROLE_IDS.CSHARP_SDK, ROLE_IDS.CSHARP_SDK_ADMIN],
   },
   {
     github: 'herczyn',
@@ -295,6 +299,10 @@ export const MEMBERS: readonly Member[] = [
     discord: '773276903364755518',
     memberOf: [ROLE_IDS.GO_SDK],
   },
+  {
+    github: 'jeffhandley',
+    memberOf: [ROLE_IDS.CSHARP_SDK, ROLE_IDS.CSHARP_SDK_ADMIN],
+  },
   {
     github: 'jenn-newton',
     memberOf: [ROLE_IDS.SECURITY_WG],
@@ -325,6 +333,10 @@ export const MEMBERS: readonly Member[] = [
       ROLE_IDS.MCP_APPS_SDK,
     ],
   },
+  {
+    github: 'jozkee',
+    memberOf: [ROLE_IDS.CSHARP_SDK],
+  },
   {
     github: 'jspahrsummers',
     email: 'justin@modelcontextprotocol.io',
@@ -391,7 +403,7 @@ export const MEMBERS: readonly Member[] = [
       ROLE_IDS.AUTH_MAINTAINERS,
       ROLE_IDS.CORE_MAINTAINERS,
       ROLE_IDS.DOCS_MAINTAINERS,
-      ROLE_IDS.CSHARP_SDK,
+      ROLE_IDS.CSHARP_SDK_ADMIN,
       ROLE_IDS.ADMINISTRATORS,
       ROLE_IDS.GO_SDK,
       ROLE_IDS.FINANCIAL_SERVICES_IG,
@@ -449,7 +461,7 @@ export const MEMBERS: readonly Member[] = [
   {
     github: 'mikekistler',
     discord: '915345005982408754',
-    memberOf: [ROLE_IDS.CSHARP_SDK],
+    memberOf: [ROLE_IDS.CSHARP_SDK, ROLE_IDS.CSHARP_SDK_ADMIN],
   },
   {
     github: 'movetz',
@@ -602,7 +614,7 @@ export const MEMBERS: readonly Member[] = [
   },
   {
     github: 'stephentoub',
-    memberOf: [ROLE_IDS.CSHARP_SDK],
+    memberOf: [ROLE_IDS.CSHARP_SDK, ROLE_IDS.CSHARP_SDK_ADMIN],
   },
   {
     github: 'tadasant',
@@ -622,6 +634,10 @@ export const MEMBERS: readonly Member[] = [
       ROLE_IDS.ADMINISTRATORS,
     ],
   },
+  {
+    github: 'tarekgh',
+    memberOf: [ROLE_IDS.CSHARP_SDK],
+  },
   {
     github: 'tiginamaria',
     memberOf: [ROLE_IDS.KOTLIN_SDK],