Address review comments

packages/proto-loader/bin/proto-loader-gen-types.ts

@@ -694,9 +694,6 @@ function runScript() {
     .normalize(['includeDirs', 'outDir'])
     .array('includeDirs')
     .boolean(['keepCase', 'defaults', 'arrays', 'objects', 'oneofs', 'json', 'verbose', 'includeComments'])
-//    .choices('longs', ['String', 'Number'])
-//    .choices('enums', ['String'])
-//    .choices('bytes', ['Array', 'String'])
     .string(['longs', 'enums', 'bytes'])
     .coerce('longs', value => {
       switch (value) {
@@ -748,8 +745,9 @@ function runScript() {
       console.log('Success');
     }
   }, (error) => {
-      throw error;
-    })
+    console.error(error)
+    process.exit(1);
+  });
 }
 
 if (require.main === module) {

packages/proto-loader/src/index.ts

@@ -24,6 +24,8 @@ import { loadProtosWithOptionsSync, loadProtosWithOptions, Options, addCommonPro
 
 export { Long } from 'long';
 
+export { Options };
+
 /**
  * This type exists for use with code generated by the proto-loader-gen-types
  * tool. This type should be used with another interface, e.g.
@@ -139,8 +141,6 @@ export interface PackageDefinition {
   [index: string]: AnyDefinition;
 }
 
-export { Options };
-
 type DecodedDescriptorSet = Protobuf.Message<descriptor.IFileDescriptorSet> &
   descriptor.IFileDescriptorSet;