project.getSourceFiles() filtering not working in v11

Describe the bug

Version: 11.0.3

project.getSourceFiles() // return --> all files

Works just fine

project.getSourceFiles('**/*.ts') // return -> [] (doesn't get the files)

project.getSourceFiles('exact/file/path/to/file') // return -> [] (doesn't get the files)

project.getSourceFile('exact/file/path/to/file') // return -> undefined (doesn't get the file)

This doesn’t.

To Reproduce

const projectDirectory = '/someDirectory'
const projectConfigFile = path.join(projectDirectory, 'tsconfig.json')
const project = new Project({
    tsConfigFilePath: projectConfigFile,
})

const sourceFilesAll = project.getSourceFiles() // return --> [...source files (all)...]
const sourceFilesFiltered = project.getSourceFiles('**/*.ts') // return --> [] (empty)

Expected behavior

Filters should just works well!

[dsherret]

Why such a behavior ? … Shouldn’t normally the matching go automatically relatively to the project directory ?

It’s done this way because then all the paths use the same directory as the base including at project initialization. Additionally, when you use project.getFileSystem() everything will be relative from the cwd. I don’t believe it would make sense to make it relative based on the directory of the tsconfig once you initialize. For example, say there was the following structure and the cwd was the root:

/project1
  tsconfig.json
  mod.ts
/project2
  tsconfig.json
  mod.ts
/project3
  mod.ts

I’m not sure it would be desirable to work this way:

const project = new Project({ tsConfigFilePath: "./project1/tsconfig.json" }); // relative CWD
project.addSourceFilesFromTsConfig("../project2/tsconfig.json"); // relative from project1
project.addSourceFiles("../project3/**/*.ts"); // relative from project1

I would expect the current behaviour:

const project = new Project({ tsConfigFilePath: "./project1/tsconfig.json" }); // relative CWD
project.addSourceFilesFromTsConfig("./project2/tsconfig.json");  // relative CWD
project.addSourceFiles("./project3/**/*.ts"); // relative CWD

Also i would like to add such a detail to the source files documentation. I Would add a small block if it’s right ? That can save good time for many. is contribution to documentation acceptable ?

Sure, thanks! That would be great to add something saying that it’s based on the current working directory.

[dsherret]

Could you provide more details and something I could take to reproduce this? I’m not sure what files exist in that project. What you’re doing should work. Perhaps the current working directory is different and you need to do ${projectDirectory}/**/*.ts