fix: outDir was recursively copied into itself

Author: jeremyben

src/copy-addon.ts

@@ -21,8 +21,11 @@ export default function copyOtherFiles(program: Program, emittedFiles: string[]
 	const srcDir = program.getCommonSourceDirectory()
 	if (!srcDir) throw Error('Cannot copy: issue with internal typescript method `getCommonSourceDirectory`')
 
-	const otherFiles = matchAllFilesBut(srcDir, ['**/*.ts'])
-	const copiedFiles: string[] = [] // Track copied files to list them later if needed
+	// Exclude typescript files and outDir if previously emitted in the same folder
+	const otherFiles = matchAllFilesBut(srcDir, ['**/*.ts', outDir])
+
+	// Track copied files to list them later if needed
+	const copiedFiles: string[] = []
 
 	for (const srcFile of otherFiles) {
 		const destFile = resolve(outDir, relative(srcDir, srcFile))

test.ts

@@ -142,4 +142,25 @@ describe('Copy', () => {
 
 		expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringMatching(/override.*data\.json/))
 	})
+
+	test('do not recursively copy outDir to outDir', () => {
+		// tslint:disable-next-line: no-shadowed-variable
+		const basePath = join(__dirname, '__fixtures__', 'src')
+		// tslint:disable-next-line: no-shadowed-variable
+		const configFilePath = '../tsconfig.fixture.json'
+
+		build({
+			basePath,
+			configFilePath,
+			copyOtherToOutDir: true,
+			compilerOptions: {
+				rootDir: '.',
+				outDir: 'dist',
+			},
+			exclude: ['**/excluded'],
+		})
+
+		const distInDist = join(basePath, 'dist', 'dist')
+		expect(existsSync(distInDist)).toBe(false)
+	})
 })