address feedback

Author: mmalerba

src/cdk/testing/kagekiri.d.ts

@@ -6,6 +6,8 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
+// Note: kagekiri is a dev dependency that is used only in our tests to test using a custom
+// querySelector function. Do not use this in published code.
 declare module 'kagekiri' {
   export function querySelectorAll(selector: string, root: Element): NodeListOf<Element>;
 }

src/cdk/testing/tests/protractor.e2e.spec.ts

@@ -12,6 +12,8 @@ import {SubComponentHarness, SubComponentSpecialHarness} from './harnesses/sub-c
 
 // Kagekiri is available globally in the browser. We declare it here so we can use it in the
 // browser-side script passed to `by.js`.
+// TODO(mmalerba): Replace with type-only import once TS 3.8 is available
+// https://devblogs.microsoft.com/typescript/announcing-typescript-3-8-beta/#type-only-imports-exports
 declare const kagekiri: {
   querySelectorAll: (selector: string, root: Element) => NodeListOf<Element>;
 };

src/e2e-app/devserver-configure.js

@@ -3,7 +3,6 @@
 require.config({
   paths: {
     'moment': 'moment/min/moment.min',
-    'kagekiri': 'kagekiri/dist/kagekiri.umd.min.js',
 
     // MDC Web
     '@material/animation': '@material/animation/dist/mdc.animation',