Changed declare module "electron" to export = Electron

and added const declarations so that `app` etc are accessible.

Author: NaridaL

base/base_footer.ts

@@ -1,10 +1,9 @@
 declare module 'electron' {
-  const electron: Electron.AllElectron;
-  export = electron;
+  export = Electron;
 }
 
 interface NodeRequireFunction {
-  (moduleName: 'electron'): Electron.AllElectron;
+  (moduleName: 'electron'): typeof Electron;
 }
 
 interface File {

index.js

@@ -18,6 +18,7 @@ const wrapWithHeaderAndFooter = (outputLines, electronVersion) => {
   const newOutputLines = []
   utils.extendArray(newOutputLines, fs.readFileSync(path.resolve(__dirname, 'base/base_header.ts'), 'utf8').replace('<<VERSION>>', electronVersion).split(/\r?\n/))
 
+  newOutputLines.push('/** @deprecated */')
   newOutputLines.push('declare namespace Electron {')
   utils.extendArray(newOutputLines, fs.readFileSync(path.resolve(__dirname, 'base/base_inner.ts'), 'utf8').replace('<<VERSION>>', electronVersion).split(/\r?\n/))
 

lib/master-interfaces.js

@@ -8,7 +8,8 @@ module.exports = (API, addToOutput) => {
   const CommonInterface = ['interface CommonInterface {']
   const MainInterface = ['interface MainInterface extends CommonInterface {']
   const RendererInterface = ['interface RendererInterface extends CommonInterface {']
-  const ElectronMainAndRendererInterface = ['interface AllElectron {']
+  const ElectronMainAndRendererInterface = ['interface AllElectron extends MainInterface, RendererInterface {}']
+  const constDeclarations = []
   const EMRI = {}
 
   const classify = (moduleName) => {
@@ -28,11 +29,20 @@ module.exports = (API, addToOutput) => {
     if (module.name === 'process') return
     let TargetInterface
     const isClass = module.type === 'Class' || API.some((tModule, tIndex) => index !== tIndex && tModule.name.toLowerCase() === module.name.toLowerCase())
-    const moduleString = `  ${classify(module.name)}: ${isClass ? 'typeof ' : ''}Electron.${_.upperFirst(module.name)}`
+    const moduleString = `  ${classify(module.name)}: ${isClass ? 'typeof ' : ''}${_.upperFirst(module.name)}`
     if (!module.process) {
       // We must be a structure or something
       return
     }
+    if (!isClass || module.name != classify(module.name)) {
+      if (isClass) {
+        constDeclarations.push(
+          `type ${classify(module.name)} = ${_.upperFirst(module.name)};`,
+          `const ${classify(module.name)}: typeof ${_.upperFirst(module.name)};`)
+      } else {
+        constDeclarations.push(`const ${classify(module.name)}: ${_.upperFirst(module.name)};`)
+      }
+    }
     if (module.process.main && module.process.renderer) {
       TargetInterface = CommonInterface
     } else if (module.process.main) {
@@ -46,7 +56,6 @@ module.exports = (API, addToOutput) => {
     if (TargetInterface) {
       debug(classify(module.name).toLowerCase(), EMRI[classify(module.name).toLowerCase()])
       if (!EMRI[classify(module.name).toLowerCase()]) {
-        ElectronMainAndRendererInterface.push(moduleString)
         TargetInterface.push(moduleString)
       }
       EMRI[classify(module.name).toLowerCase()] = true
@@ -56,11 +65,11 @@ module.exports = (API, addToOutput) => {
   CommonInterface.push('}')
   MainInterface.push('}')
   RendererInterface.push('}')
-  ElectronMainAndRendererInterface.push('}')
 
   addToOutput([''])
   addToOutput(CommonInterface, ';')
   addToOutput(MainInterface, ';')
   addToOutput(RendererInterface, ';')
   addToOutput(ElectronMainAndRendererInterface, ';')
+  addToOutput(constDeclarations)
 }

test-smoke/electron/test/main.ts

@@ -23,6 +23,7 @@ import {
   session,
   systemPreferences,
   webContents,
+  Event,
 } from "electron";
 
 import * as path from "path";
@@ -33,6 +34,7 @@ import * as path from "path";
 // Keep a global reference of the window object, if you don't, the window will
 // be closed automatically when the javascript object is GCed.
 let mainWindow: Electron.BrowserWindow = null;
+const mainWindow2: BrowserWindow = null;
 
 // Quit when all windows are closed.
 app.on("window-all-closed", () => {
@@ -524,6 +526,11 @@ ipcMain.on("synchronous-message", (event: Electron.Event, arg: any) => {
   event.returnValue = "pong";
 });
 
+ipcMain.on("synchronous-message", (event: Event, arg: any) => {
+  console.log("event isn't namespaced and refers to the correct type.");
+  event.returnValue = "pong";
+});
+
 const winWindows = new BrowserWindow({
   width: 800,
   height: 600,

tslint.json

@@ -13,6 +13,7 @@
     "no-namespace": false,
     "quotemark": false,
     "unified-signatures": false,
-    "variable-name": false
+    "variable-name": false,
+    "no-shadowed-variable": false
   }
 }