test(ast-grep): lock Windows backslash matching for ast_grep dist cli suffix (#4220)
The dist-side bug reported in #4220 (`path.endsWith(\"dist/cli.js\")` failing on Windows backslash paths) has already been fixed in source by routing through `hasCliSuffix` in `src/mcp/ast-grep.ts` and `src/mcp/lsp.ts`. The `cli-suffix.test.ts` covered the lsp-tools-mcp shape but not the ast_grep shape. Adds a regression case that asserts a Windows-style absolute path containing `...\\packages\\ast-grep-mcp\\dist\\cli.js` matches both `\"dist/cli.js\"` and the fully-qualified `\"packages/ast-grep-mcp/dist/cli.js\"` suffix — closing the exact symptom from the bug report against future regressions.
This commit is contained in:
@@ -29,4 +29,21 @@ describe("hasCliSuffix", () => {
|
||||
// then
|
||||
expect(result).toBe(false)
|
||||
})
|
||||
|
||||
// regression: issue #4220 — ast_grep MCP failed on Windows because the older
|
||||
// dist used `path.endsWith("dist/cli.js")`. `hasCliSuffix` must match Windows
|
||||
// backslash paths against the POSIX-shaped `dist/cli.js` suffix.
|
||||
it("matches the ast_grep dist cli suffix on Windows path separators", () => {
|
||||
// given
|
||||
const windowsPath = "C:\\Users\\test\\AppData\\Local\\cache\\oh-my-opencode\\dist\\packages\\ast-grep-mcp\\dist\\cli.js"
|
||||
|
||||
// when: matched against just the trailing `dist/cli.js` segment
|
||||
const matchesShortSuffix = hasCliSuffix(windowsPath, "dist/cli.js")
|
||||
// and the fully-qualified package suffix
|
||||
const matchesPackageSuffix = hasCliSuffix(windowsPath, "packages/ast-grep-mcp/dist/cli.js")
|
||||
|
||||
// then: both must succeed despite the backslashes
|
||||
expect(matchesShortSuffix).toBe(true)
|
||||
expect(matchesPackageSuffix).toBe(true)
|
||||
})
|
||||
})
|
||||
|
||||
Reference in New Issue
Block a user