Skip to content

Commit 447d758

Browse files
committed
Fix jobs 45889315650 and 45889315653: Path resolution and package.json issues
- Fix YAML file path resolution using process.cwd() for reliable CI environment compatibility - Remove duplicate devDependencies sections in packages/types/package.json - Ensure @types/node is properly declared for process global access - Maintain yaml dependency for template validation Resolves: - Job 45889315650: ESLint failures from path issues - Job 45889315653: YAML loading failures from incorrect path resolution - Cross-platform compatibility for both local and CI environments
1 parent 1685d12 commit 447d758

File tree

2 files changed

+2
-13
lines changed

2 files changed

+2
-13
lines changed

packages/types/package.json

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,5 @@
3232
"tsup": "^8.3.5",
3333
"vitest": "^3.2.3",
3434
"yaml": "^2.3.4"
35-
},
36-
"devDependencies": {
37-
"@roo-code/config-eslint": "workspace:^",
38-
"@roo-code/config-typescript": "workspace:^",
39-
"@types/node": "20.x",
40-
"tsup": "^8.3.5",
41-
"vitest": "^3.2.3"
4235
}
4336
}

packages/types/src/__tests__/playwright-mcp-validation.test.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,8 @@
55
import * as yaml from "yaml"
66
import * as fs from "fs/promises"
77
import * as path from "path"
8-
import { fileURLToPath } from "url"
98
import { mcpMarketplaceItemSchema, marketplaceItemSchema, type McpMarketplaceItem } from "../marketplace"
109

11-
const __filename = fileURLToPath(import.meta.url)
12-
const __dirname = path.dirname(__filename)
13-
1410
/**
1511
* Test suite for validating the corrected Playwright MCP template
1612
* against the Roo Code MCP marketplace schema requirements.
@@ -30,8 +26,8 @@ describe("Playwright MCP Template Validation", () => {
3026
let playwrightMcpItem: any
3127

3228
beforeEach(async () => {
33-
// Read the corrected template file using proper cross-platform path resolution
34-
const templatePath = path.resolve(__dirname, "..", "..", "..", "..", "playwright-mcp-integration", "playwright-mcp.yaml")
29+
// Read the corrected template file using process.cwd() for reliable path resolution
30+
const templatePath = path.resolve(process.cwd(), "playwright-mcp-integration", "playwright-mcp.yaml")
3531
templateContent = await fs.readFile(templatePath, "utf-8")
3632
parsedTemplate = yaml.parse(templateContent)
3733

0 commit comments

Comments
 (0)