`textlint.targetPath` seems to be broken

[r4ai]

Description

When README.md exists in the project root:

• With the default value "textlint.targetPath": "", README.md is correctly linted.
• However, if any of the following values are set, README.md is no longer targeted for linting:
  • README.md
  • *
  • **/*

Expected Behavior

Based on my understanding, this option should accept glob patterns to specify target files. The current behavior seems to be broken.

broken.mp4

[azu]

Let's see. I feel like targetPath also accepts glob (although I don't think the option name is very good)

vscode-textlint/packages/textlint-server/src/server.ts

Lines 280 to 288 in 20c0619

	function isTarget(root: string, file: string): boolean {
	const relativePath = file.substring(root.length);
	return (
	settings.targetPath === "" ||
	minimatch(relativePath, settings.targetPath, {
	matchBase: true,
	})
	);
	}

It looks like something is probably going on with root + targetPath.

Which operating system are you using?
The glob is always /, so there are cases where the \ is mixed in and stuck on Windows.

[azu]

I've only seen it roughly, but maybe the problem is that the relativePath always starts with '/' and doesn't match glob.
Maybe they didn't consider glob support in the first place, but just used 'minimatch' for pass matching.

https://deepwiki.com/search/when-readmemd-exists-in-the-pr_37fc6f08-0b7a-4723-9c3c-2c8910d806b3?mode=fast

Well, being able to filter with glob seems like a useful feature.

[r4ai]

Which operating system are you using? The glob is always /, so there are cases where the \ is mixed in and stuck on Windows.

The operating system in use is Ubuntu 24.04 on WSL.

OS: Ubuntu 24.04.3 LTS (Noble Numbat) x86_64
Host: Windows Subsystem for Linux - Ubuntu-24.04 (2.6.1.0)
Kernel: Linux 6.6.87.2-microsoft-standard-WSL2