Merge pull request #39 from useshortcut/amcd/search-by-name-bug

Fix searching by name.

Author: andreasmcdermott

package.json

@@ -12,7 +12,7 @@
 		"modelcontextprotocol"
 	],
 	"license": "MIT",
-	"version": "0.5.0",
+	"version": "0.5.1",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {

src/tools/epics.test.ts

@@ -279,7 +279,7 @@ describe("EpicTools", () => {
 			});
 
 			expect(searchEpicsMock.mock.calls?.[0]?.[0]).toBe(
-				'id:1 name:"Test Epic" description:"Test Description" state:started objective:123 owner:testuser team:engineering is:archived',
+				'id:1 title:"Test Epic" description:"Test Description" state:started objective:123 owner:testuser group:engineering is:archived',
 			);
 		});
 

src/tools/utils/search.test.ts

@@ -48,7 +48,7 @@ describe("buildSearchQuery", () => {
 			state: "started",
 		};
 		const result = await buildSearchQuery(params, null);
-		expect(result).toBe("name:test state:started");
+		expect(result).toBe("title:test state:started");
 	});
 
 	test("should quote string parameters with spaces", async () => {
@@ -57,7 +57,7 @@ describe("buildSearchQuery", () => {
 			description: "some description",
 		};
 		const result = await buildSearchQuery(params, null);
-		expect(result).toBe('name:"test story" description:"some description"');
+		expect(result).toBe('title:"test story" description:"some description"');
 	});
 
 	test("should handle mixed parameter types correctly", async () => {
@@ -68,7 +68,7 @@ describe("buildSearchQuery", () => {
 			hasOwner: false,
 		};
 		const result = await buildSearchQuery(params, null);
-		expect(result).toBe("id:123 name:test is:started !has:owner");
+		expect(result).toBe("id:123 title:test is:started !has:owner");
 	});
 
 	test('should replace "me" with current user mention name for owner parameter', async () => {

src/tools/utils/search.ts

@@ -1,9 +1,17 @@
 import type { MemberInfo } from "@shortcut/client";
 
+const keyRenames = { team: "group", name: "title" } as const;
+
+const mapKeyName = (key: string) => {
+	const lowercaseKey = key.toLowerCase();
+
+	return keyRenames[lowercaseKey as keyof typeof keyRenames] || lowercaseKey;
+};
+
 const getKey = (prop: string) => {
-	if (prop.startsWith("is")) return `is:${prop.slice(2).toLowerCase()}`;
-	if (prop.startsWith("has")) return `has:${prop.slice(3).toLowerCase()}`;
-	return prop;
+	if (prop.startsWith("is")) return `is:${mapKeyName(prop.slice(2))}`;
+	if (prop.startsWith("has")) return `has:${mapKeyName(prop.slice(3))}`;
+	return mapKeyName(prop);
 };
 
 export type QueryParams = { [key: string]: boolean | string | number };