fix: change some resource templates into resources

team_members, mob_session_coauthors, mob_session_coauthor_trailers and git_mob_version are static resources that do not depend on any input. Therefore these are changed from resource templates into resources. This makes them work in the MCP Inspector

Kept git_mob_help as a resource template as its dynamic - the content changes when command is supplied. This is yet to be tested to be working in MCP Inspector

Author: Mubashwer

README.md

@@ -33,8 +33,7 @@ Built using [@modelcontextprotocol/sdk](https://github.com/modelcontextprotocol/
 - `setup_git_mob_globally`: Sets up git-mob globally for the user.
 - `setup_git_mob_locally`: Sets up git-mob locally for the current repository.
 
-Because dynamic resources are not yet supported in GitHub Copilot, the resources are also available as tools:
-- `get_git_mob_help`: Displays general help and usage information for the Git Mob CLI.
+Because resources may not be fully supported in GitHub Copilot Agent mode yet, the some of them are also available as tools:
 - `get_git_mob_version`: The installed version of the Git Mob CLI.
 - `get_team_members`: List of all the team members that have been added to Git Mob.
 - `get_mob_session_coauthors`: List of all coauthors currently included in the active mob or pairing session.

src/gitMobServerFactory.test.ts

@@ -1,13 +1,15 @@
 import { describe, it, expect, jest } from "@jest/globals";
 import * as helpers from "./helpers/index.js";
 import * as resources from "./resources/index.js";
+import * as resourceTemplates from "./resourceTemplates/index.js";
 import * as tools from "./tools/index.js";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { createGitMobServer } from "./gitMobServerFactory.js";
 
 jest.mock("./helpers/index.js", () => ({
   registerGitMobTool: jest.fn(),
   registerGitMobResource: jest.fn(),
+  registerGitMobResourceTemplate: jest.fn(),
   registerGtMobResourceAsTool: jest.fn(),
 }));
 jest.mock("@modelcontextprotocol/sdk/server/mcp.js", () => {
@@ -35,6 +37,17 @@ describe("gitMobServerFactory: createGitMobServer", () => {
     );
   });
 
+  it("should register all resource templates", async () => {
+    const { registerGitMobResourceTemplate } = helpers;
+
+    const server = createGitMobServer();
+
+    expect(registerGitMobResourceTemplate).toHaveBeenCalledWith(
+      server,
+      resourceTemplates.gitMobHelp,
+    );
+  });
+
   it("should register all resources", async () => {
     const { registerGitMobResource } = helpers;
 
@@ -44,10 +57,6 @@ describe("gitMobServerFactory: createGitMobServer", () => {
       server,
       resources.gitMobVersion,
     );
-    expect(registerGitMobResource).toHaveBeenCalledWith(
-      server,
-      resources.gitMobHelp,
-    );
     expect(registerGitMobResource).toHaveBeenCalledWith(
       server,
       resources.teamMembers,
@@ -71,10 +80,6 @@ describe("gitMobServerFactory: createGitMobServer", () => {
       server,
       resources.gitMobVersion,
     );
-    expect(registerGtMobResourceAsTool).toHaveBeenCalledWith(
-      server,
-      resources.gitMobHelp,
-    );
     expect(registerGtMobResourceAsTool).toHaveBeenCalledWith(
       server,
       resources.teamMembers,

src/gitMobServerFactory.ts

@@ -2,10 +2,12 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 
 import * as tools from "./tools/index.js";
 import * as resources from "./resources/index.js";
+import * as resourceTemplates from "./resourceTemplates/index.js";
 import {
   registerGitMobTool,
   registerGitMobResource,
   registerGtMobResourceAsTool,
+  registerGitMobResourceTemplate,
 } from "./helpers/index.js";
 
 export const createGitMobServer = () => {
@@ -22,16 +24,16 @@ export const createGitMobServer = () => {
     },
   );
 
+  registerGitMobResourceTemplate(server, resourceTemplates.gitMobHelp);
+
   registerGitMobResource(server, resources.gitMobVersion);
-  registerGitMobResource(server, resources.gitMobHelp);
   registerGitMobResource(server, resources.teamMembers);
   registerGitMobResource(server, resources.mobSessionCoauthors);
   registerGitMobResource(server, resources.mobSessionCoauthorTrailers);
 
   // Currently, Github Copilot does not support dynamic resources in MCP Server,
   // so we register them as tools as well
   registerGtMobResourceAsTool(server, resources.gitMobVersion);
-  registerGtMobResourceAsTool(server, resources.gitMobHelp);
   registerGtMobResourceAsTool(server, resources.teamMembers);
   registerGtMobResourceAsTool(server, resources.mobSessionCoauthors);
   registerGtMobResourceAsTool(server, resources.mobSessionCoauthorTrailers);

src/helpers/index.ts

@@ -1,4 +1,5 @@
 export * from "./registerGitMobTool.js";
 export * from "./registerGitMobResource.js";
+export * from "./registerGitMobResourceTemplate.js";
 export * from "./registerGitMobResourceAsTool.js";
 export * from "./runCliCommand.js";

src/helpers/registerGitMobResource.test.ts

@@ -3,12 +3,9 @@ import { registerGitMobResource } from "./registerGitMobResource.js";
 import type { GitMobResource } from "../types/GitMobResource.js";
 import type {
   McpServer,
-  ReadResourceTemplateCallback,
-} from "@modelcontextprotocol/sdk/server/mcp.js";
-import {
-  ResourceTemplate,
-  type ResourceMetadata,
+  ReadResourceCallback,
 } from "@modelcontextprotocol/sdk/server/mcp.js";
+import { type ResourceMetadata } from "@modelcontextprotocol/sdk/server/mcp.js";
 
 describe("[helpers] registerGitMobResource", () => {
   it("should register given git mob resource with the given server", () => {
@@ -17,14 +14,12 @@ describe("[helpers] registerGitMobResource", () => {
     } as Partial<McpServer> as McpServer;
 
     const name = "test_resource";
-    const template = new ResourceTemplate("gitmob://test-resource", {
-      list: undefined,
-    });
+    const uri = "gitmob://test-resource";
     const metadata: ResourceMetadata = {
       description: "This is a test resource for testing purposes.",
       mimeType: "text/plain",
     };
-    const readCallback: ReadResourceTemplateCallback = async (uri) => {
+    const readCallback: ReadResourceCallback = async (uri) => {
       return {
         contents: [
           {
@@ -36,7 +31,7 @@ describe("[helpers] registerGitMobResource", () => {
     };
     const testResource: GitMobResource = {
       name,
-      template,
+      uri,
       metadata,
       readCallback,
     };
@@ -45,7 +40,7 @@ describe("[helpers] registerGitMobResource", () => {
 
     expect(mockServer.resource).toHaveBeenCalledWith(
       testResource.name,
-      testResource.template,
+      testResource.uri,
       testResource.metadata,
       testResource.readCallback,
     );

src/helpers/registerGitMobResource.ts

@@ -5,6 +5,6 @@ export const registerGitMobResource = (
   server: McpServer,
   resource: GitMobResource,
 ) => {
-  const { name, template, metadata, readCallback } = resource;
-  server.resource(name, template, metadata, readCallback);
+  const { name, uri, metadata, readCallback } = resource;
+  server.resource(name, uri, metadata, readCallback);
 };

src/helpers/registerGitMobResourceAsTool.test.ts

@@ -2,7 +2,6 @@ import { describe, it, expect, jest } from "@jest/globals";
 import { registerGtMobResourceAsTool } from "./registerGitMobResourceAsTool.js";
 import type { GitMobResource } from "../types/GitMobResource.js";
 import {
-  ResourceTemplate,
   type ResourceMetadata,
   type ReadResourceTemplateCallback,
   type McpServer,
@@ -21,9 +20,7 @@ describe("[helpers] registerGtMobResourceAsTool", () => {
     } as Partial<McpServer> as McpServer;
 
     const name = "test_resource";
-    const template = new ResourceTemplate("gitmob://test-resource", {
-      list: undefined,
-    });
+    const uri = "gitmob://test-resource";
     const metadata: ResourceMetadata = {
       description: "This is a test resource for testing purposes.",
       mimeType: "text/plain",
@@ -38,7 +35,7 @@ describe("[helpers] registerGtMobResourceAsTool", () => {
     };
     const testResource: GitMobResource = {
       name,
-      template,
+      uri,
       metadata,
       readCallback,
     };

src/helpers/registerGitMobResourceAsTool.ts

@@ -17,7 +17,7 @@ export const registerGtMobResourceAsTool = (
   server: McpServer,
   resource: GitMobResource,
 ) => {
-  const { name, template, metadata, readCallback } = resource;
+  const { name, uri, metadata, readCallback } = resource;
 
   const toolName = `get_${name}`;
 
@@ -30,14 +30,10 @@ export const registerGtMobResourceAsTool = (
   };
 
   const toolCallback: ToolCallback<Record<string, never>> = async (
-    args: Variables,
+    _args: Variables,
     extra: RequestHandlerExtra<ServerRequest, ServerNotification>,
   ) => {
-    const resourceResult = await readCallback(
-      new URL(template.uriTemplate),
-      args,
-      extra,
-    );
+    const resourceResult = await readCallback(new URL(uri), extra);
     const toolResult: CallToolResult = {
       content: resourceResult.contents
         .filter((content) => typeof content.text === "string")

src/helpers/registerGitMobResourceTemplate.test.ts

@@ -0,0 +1,53 @@
+import { describe, it, expect, jest } from "@jest/globals";
+import { registerGitMobResourceTemplate } from "./registerGitMobResourceTemplate.js";
+import type { GitMobResourceTemplate } from "../types/GitMobResourceTemplate.js";
+import type {
+  McpServer,
+  ReadResourceTemplateCallback,
+} from "@modelcontextprotocol/sdk/server/mcp.js";
+import {
+  ResourceTemplate,
+  type ResourceMetadata,
+} from "@modelcontextprotocol/sdk/server/mcp.js";
+
+describe("[helpers] registerGitMobResourceTemplate", () => {
+  it("should register given git mob resource template with the given server", () => {
+    const mockServer = {
+      resource: jest.fn(),
+    } as Partial<McpServer> as McpServer;
+
+    const name = "test_resource";
+    const template = new ResourceTemplate("gitmob://test-resource", {
+      list: undefined,
+    });
+    const metadata: ResourceMetadata = {
+      description: "This is a test resource for testing purposes.",
+      mimeType: "text/plain",
+    };
+    const readCallback: ReadResourceTemplateCallback = async (uri) => {
+      return {
+        contents: [
+          {
+            uri: uri.href,
+            text: "hello world",
+          },
+        ],
+      };
+    };
+    const testResourceTemplate: GitMobResourceTemplate = {
+      name,
+      template,
+      metadata,
+      readCallback,
+    };
+
+    registerGitMobResourceTemplate(mockServer, testResourceTemplate);
+
+    expect(mockServer.resource).toHaveBeenCalledWith(
+      testResourceTemplate.name,
+      testResourceTemplate.template,
+      testResourceTemplate.metadata,
+      testResourceTemplate.readCallback,
+    );
+  });
+});

src/helpers/registerGitMobResourceTemplate.ts

@@ -0,0 +1,10 @@
+import type { GitMobResourceTemplate } from "../types/GitMobResourceTemplate";
+import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp";
+
+export const registerGitMobResourceTemplate = (
+  server: McpServer,
+  resource: GitMobResourceTemplate,
+) => {
+  const { name, template, metadata, readCallback } = resource;
+  server.resource(name, template, metadata, readCallback);
+};