fix: don't resolve filtered files (#428)

* fix: don't resolve `filter`ed files

- if they're `exclude`d / not `include`d, then we shouldn't be processing them
  - we're already not transforming them, so this just applies the same exclusion to resolving
  - this is _partly_ a regression from b0e3922, as that removed the `allImportedFiles` Set that previously filtered out files not in the `tsconfig` `include`
    - but that _itself_ was a regression that was removed -- files that didn't pass `filter` should have _never_ been resolved
      - basically, the `allImportedFiles` regression was covering up this long-standing bug

- also move `.d.ts` check to above the `filter` check
  - we shouldn't be adding declarations to the `cache`, in particular as we don't process declarations, so they'll never be marked as dirty
  - having this check above the `filter` should be slighltly more efficient as well (as would not having these files in the cache graph)

types: be more specific with `filter`'s type
- no need for this to be `any`

* build

- patch release has been waiting for a few weeks

* pub: release v0.34.1

- patch bump with the past few fixes

- bump internal rpt2 version to 0.34.0

Author: agilgur5

dist/rollup-plugin-typescript2.cjs.js

@@ -1,6 +1,6 @@
 {
   "name": "rollup-plugin-typescript2",
-  "version": "0.34.0",
+  "version": "0.34.1",
   "description": "Seamless integration between Rollup and TypeScript. Now with errors.",
   "main": "dist/rollup-plugin-typescript2.cjs.js",
   "module": "dist/rollup-plugin-typescript2.es.js",
@@ -62,7 +62,7 @@
     "rimraf": "3.0.2",
     "rollup": "^2.70.2",
     "rollup-plugin-re": "1.0.7",
-    "rollup-plugin-typescript2": "0.33.0",
+    "rollup-plugin-typescript2": "0.34.0",
     "ts-jest": "^28.0.0",
     "tslint": "6.1.3",
     "typescript": "^4.6.3"

dist/rollup-plugin-typescript2.cjs.js.map

@@ -32,7 +32,7 @@ const typescript: PluginImpl<RPT2Options> = (options) =>
 	let generateRound = 0;
 	let rollupOptions: InputOptions;
 	let context: RollupContext;
-	let filter: any;
+	let filter: ReturnType<typeof createFilter>;
 	let parsedConfig: tsTypes.ParsedCommandLine;
 	let tsConfigPath: string | undefined;
 	let servicesHost: LanguageServiceHost;
@@ -204,12 +204,14 @@ const typescript: PluginImpl<RPT2Options> = (options) =>
 			if (!resolved)
 				return;
 
-			if (filter(resolved))
-				cache.setDependency(resolved, importer);
-
 			if (resolved.endsWith(".d.ts"))
 				return;
 
+			if (!filter(resolved))
+				return;
+
+			cache.setDependency(resolved, importer);
+
 			context.debug(() => `${blue("resolving")} '${importee}' imported by '${importer}'`);
 			context.debug(() => `    to '${resolved}'`);