fix: resolve relative workshop paths (#589)

Fixes EPICSHOP-ED

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>

Author: 3 people

packages/workshop-mcp/src/utils.test.ts

@@ -126,10 +126,26 @@ test('handleWorkshopDirectory rejects blank input (aha)', async () => {
 	)
 })
 
-test('handleWorkshopDirectory rejects relative paths', async () => {
-	await expect(handleWorkshopDirectory('workshop')).rejects.toThrow(
-		'The workshop directory must be an absolute path',
-	)
+test('handleWorkshopDirectory resolves relative paths from cwd (aha)', async () => {
+	await using fixture = await createWorkshopFixture()
+	vi.mocked(console.error).mockImplementation(() => {})
+	const originalCwd = process.cwd()
+	const parentDir = path.dirname(fixture.root)
+	const relativePath = path.relative(parentDir, fixture.root)
+
+	const { init } = await import('@epic-web/workshop-utils/apps.server')
+	const initMock = vi.mocked(init)
+	initMock.mockClear()
+
+	try {
+		process.chdir(parentDir)
+		await expect(handleWorkshopDirectory(relativePath)).resolves.toBe(
+			fixture.root,
+		)
+		expect(initMock).toHaveBeenCalledWith(fixture.root)
+	} finally {
+		process.chdir(originalCwd)
+	}
 })
 
 test('handleWorkshopDirectory normalizes playground to workshop root', async () => {

packages/workshop-mcp/src/utils.ts

@@ -9,7 +9,7 @@ import { z } from 'zod/v3'
 export const workshopDirectoryInputSchema = z
 	.string()
 	.describe(
-		'Absolute path to the workshop root directory. Passing a /playground path is allowed and will be normalized to the workshop root.',
+		'Absolute or relative path to the workshop root directory. Relative paths are resolved from the current working directory. Passing a /playground path is allowed and will be normalized to the workshop root.',
 	)
 
 async function isWorkshopDirectory(workshopDirectory: string) {
@@ -41,7 +41,7 @@ export async function handleWorkshopDirectory(workshopDirectory: string) {
 	if (!workshopDirectory) throw new Error('The workshop directory is required')
 
 	if (!path.isAbsolute(workshopDirectory)) {
-		throw new Error('The workshop directory must be an absolute path')
+		workshopDirectory = path.resolve(process.cwd(), workshopDirectory)
 	}
 
 	if (workshopDirectory.endsWith(`${path.sep}playground`)) {