chore: migrate to owl bot (#958)

* chore: migrate to owl bot

* chore: copy files from googleapis-gen 397c0bfd367a2427104f988d5329bc117caafd95

* chore: run the post processor

* lint

Co-authored-by: Justin Beckwith <[email protected]>

Author: SurferJeffAtGoogle

.github/.OwlBot.lock.yaml

@@ -0,0 +1,4 @@
+docker:
+  digest: sha256:f556e6e7be625deb1b2429fe608df27be57185c3e6b7d39ee0059f1609f17530
+  image: gcr.io/repo-automation-bots/owlbot-nodejs:latest  
+

.github/.OwlBot.yaml

@@ -0,0 +1,19 @@
+# Copyright 2021 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+docker:
+  image: gcr.io/repo-automation-bots/owlbot-nodejs:latest
+
+
+begin-after-commit-hash: 397c0bfd367a2427104f988d5329bc117caafd95
+

.github/generated-files-bot.yml

@@ -0,0 +1,16 @@
+generatedFiles:
+- path: '.kokoro/**'
+  message: '`.kokoro` files are templated and should be updated in [`synthtool`](https://github.com/googleapis/synthtool)'
+- path: '.github/CODEOWNERS'
+  message: 'CODEOWNERS should instead be modified via the `codeowner_team` property in .repo-metadata.json'
+- path: '.github/workflows/**'
+  message: '`.github/workflows` (GitHub Actions) should be updated in [`synthtool`](https://github.com/googleapis/synthtool)'
+- path: '.github/generated-files-bot.+(yml|yaml)'
+  message: '`.github/generated-files-bot.(yml|yaml)` should be updated in [`synthtool`](https://github.com/googleapis/synthtool)'
+- path: 'README.md'
+  message: '`README.md` is managed by [`synthtool`](https://github.com/googleapis/synthtool). However, a partials file can be used to update the README, e.g.: https://github.com/googleapis/nodejs-storage/blob/master/.readme-partials.yaml'
+- path: 'samples/README.md'
+  message: '`samples/README.md` is managed by [`synthtool`](https://github.com/googleapis/synthtool). However, a partials file can be used to update the README, e.g.: https://github.com/googleapis/nodejs-storage/blob/master/.readme-partials.yaml'
+ignoreAuthors:
+- 'gcf-owl-bot[bot]'
+- 'yoshi-automation'

synth.py renamed to owlbot.py

@@ -12,18 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import synthtool as s
-import synthtool.gcp as gcp
 import synthtool.languages.node as node
-import logging
 
-logging.basicConfig(level=logging.DEBUG)
-
-AUTOSYNTH_MULTIPLE_COMMITS = True
-
-common_templates = gcp.CommonTemplates()
-templates = common_templates.node_library()
-s.copy(templates, excludes=['.eslintignore', '.mocharc.js', '.github/workflows/ci.yaml'])
-
-node.install()
-node.fix()
+node.owlbot_main(templates_excludes=['.eslintignore', '.mocharc.js', '.github/workflows/ci.yaml'])

src/agent/controller.ts

@@ -134,7 +134,7 @@ export class Controller extends ServiceObject {
         } else if (response.statusCode === 404) {
           // The v2 API returns 404 (google.rpc.Code.NOT_FOUND) when the agent
           // registration expires. We should re-register.
-          callback(null, (response as {}) as t.Response);
+          callback(null, response as {} as t.Response);
           return;
         } else if (response.statusCode !== 200) {
           callback(
@@ -146,7 +146,7 @@ export class Controller extends ServiceObject {
         } else {
           body = body || {};
           that.nextWaitToken = body.nextWaitToken;
-          callback(null, (response as {}) as t.Response, body);
+          callback(null, response as {} as t.Response, body);
         }
       }
     );

src/agent/debuglet.ts

@@ -457,7 +457,7 @@ export class Debuglet extends EventEmitter {
     let sourceContext;
     try {
       sourceContext =
-        ((that.config.sourceContext as {}) as SourceContext) ||
+        (that.config.sourceContext as {} as SourceContext) ||
         (await Debuglet.getSourceContextFromFile());
     } catch (err5) {
       that.logger.warn('Unable to discover source context', err5);
@@ -741,9 +741,11 @@ export class Debuglet extends EventEmitter {
           );
           // TODO: Handle the case when `that.debuggee` is null.
           // TODO: Handle the case when `result` is undefined.
-          (that.debuggee as Debuggee).id = (result as {
-            debuggee: Debuggee;
-          }).debuggee.id;
+          (that.debuggee as Debuggee).id = (
+            result as {
+              debuggee: Debuggee;
+            }
+          ).debuggee.id;
           // TODO: Handle the case when `result` is undefined.
           that.emit('registered', (result as {debuggee: Debuggee}).debuggee.id);
           that.debuggeeRegistered.resolve();
@@ -926,9 +928,7 @@ export class Debuglet extends EventEmitter {
       //              field.  It is possible that breakpoint.id is always
       //              undefined!
       // TODO: Make sure the use of `that` here is correct.
-      delete that.completedBreakpointMap[
-        ((breakpoint as {}) as {id: number}).id
-      ];
+      delete that.completedBreakpointMap[(breakpoint as {} as {id: number}).id];
     });
 
     // Remove active breakpoints that the server no longer care about.
@@ -946,9 +946,9 @@ export class Debuglet extends EventEmitter {
    * @return {Object.<string, Breakpoint>} A map of breakpoint IDs to breakpoints.
    * @private
    */
-  convertBreakpointListToMap_(
-    breakpointList: stackdriver.Breakpoint[]
-  ): {[key: string]: stackdriver.Breakpoint} {
+  convertBreakpointListToMap_(breakpointList: stackdriver.Breakpoint[]): {
+    [key: string]: stackdriver.Breakpoint;
+  } {
     const map: {[id: string]: stackdriver.Breakpoint} = {};
     breakpointList.forEach(breakpoint => {
       // TODO: Address the case when `breakpoint.id` is `undefined`.
@@ -1098,13 +1098,15 @@ export class Debuglet extends EventEmitter {
     const that = this;
 
     // TODO: Address the case when `that.debuggee` is `null`.
-    that.controller.updateBreakpoint(that.debuggee as Debuggee, breakpoint, (
-      err /*, body*/
-    ) => {
-      if (err) {
-        that.logger.error('Unable to complete breakpoint on server', err);
+    that.controller.updateBreakpoint(
+      that.debuggee as Debuggee,
+      breakpoint,
+      (err /*, body*/) => {
+        if (err) {
+          that.logger.error('Unable to complete breakpoint on server', err);
+        }
       }
-    });
+    );
   }
 
   /**

src/agent/io/sourcemapper.ts

@@ -74,9 +74,9 @@ async function processSourcemap(
     // TODO: Resolve the cast of `contents as any` (This is needed because the
     //       type is expected to be of `RawSourceMap` but the existing
     //       working code uses a string.)
-    consumer = (new sourceMap.SourceMapConsumer(
-      (contents as {}) as sourceMap.RawSourceMap
-    ) as {}) as sourceMap.RawSourceMap;
+    consumer = new sourceMap.SourceMapConsumer(
+      contents as {} as sourceMap.RawSourceMap
+    ) as {} as sourceMap.RawSourceMap;
   } catch (e) {
     throw new Error(
       'An error occurred while reading the ' +
@@ -246,7 +246,8 @@ export class SourceMapper {
     };
 
     // TODO: Determine how to remove the explicit cast here.
-    const consumer: sourceMap.SourceMapConsumer = (entry.mapConsumer as {}) as sourceMap.SourceMapConsumer;
+    const consumer: sourceMap.SourceMapConsumer =
+      entry.mapConsumer as {} as sourceMap.SourceMapConsumer;
     const allPos = consumer.allGeneratedPositionsFor(sourcePos);
     /*
      * Based on testing, it appears that the following code is needed to
@@ -270,7 +271,7 @@ export class SourceMapper {
       // TODO: The `sourceMap.Position` type definition has a `column`
       //       attribute and not a `col` attribute.  Determine if the type
       //       definition or this code is correct.
-      column: ((mappedPos as {}) as {col: number}).col, // SourceMapConsumer uses
+      column: (mappedPos as {} as {col: number}).col, // SourceMapConsumer uses
       // zero-based column
       // numbers which is the
       // same as the expected

src/agent/state/inspector-state.ts

@@ -151,7 +151,7 @@ class StateResolver {
     };
 
     // TODO: Determine why _extend is used here
-    this.resolvedVariableTable = ((util as {}) as {_extend: Function})._extend(
+    this.resolvedVariableTable = (util as {} as {_extend: Function})._extend(
       [],
       this.messageTable
     );

src/agent/state/legacy-state.ts

@@ -138,7 +138,7 @@ class StateResolver {
     };
 
     // TODO: Determine why _extend is used here
-    this.resolvedVariableTable = ((util as {}) as {_extend: Function})._extend(
+    this.resolvedVariableTable = (util as {} as {_extend: Function})._extend(
       [],
       this.messageTable
     );
@@ -148,7 +148,7 @@ class StateResolver {
 
     // This constructor is only used in situations where the legacy vm
     // interface is used that has the `runInDebugContext` method.
-    this.scopeType = ((vm as {}) as LegacyVm).runInDebugContext('ScopeType');
+    this.scopeType = (vm as {} as LegacyVm).runInDebugContext('ScopeType');
   }
 
   /**

src/agent/util/debug-assert.ts

@@ -62,5 +62,5 @@ const fakeAssert: FakeAssert = {
 
 export function debugAssert(enableAssertions: boolean): FakeAssert {
   // The typecast is needed since the @types/node doesn't cover Node 10 yet
-  return enableAssertions ? ((realAssert as {}) as FakeAssert) : fakeAssert;
+  return enableAssertions ? (realAssert as {} as FakeAssert) : fakeAssert;
 }