@opentermsarchive/engine 0.36.1 → 0.37.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opentermsarchive/engine",
-  "version": "0.36.1",
+  "version": "0.37.1",
   "description": "Tracks and makes visible changes to the terms of online services",
   "homepage": "https://github.com/OpenTermsArchive/engine#readme",
   "bugs": {

package/scripts/declarations/utils/index.js

@@ -20,9 +20,9 @@ export default class DeclarationUtils {
   }
 
   async getModifiedData() {
-    const modifiedFilePathsAsString = (await this.git.diff([ '--diff-filter=d', '--name-only', this.defaultBranch, 'HEAD', '--', './declarations' ])).trim();
+    const modifiedFilePathsAsString = (await this.git.diff([ '-z', '--diff-filter=d', '--name-only', this.defaultBranch, 'HEAD', '--', './declarations' ])).trim(); // -z option is required to avoid pathnames with "unusual" characters to be quoted, but it also replaces result separator by the  zero byte character \0
 
-    const modifiedFilePaths = modifiedFilePathsAsString ? modifiedFilePathsAsString.split('\n') : [];
+    const modifiedFilePaths = (modifiedFilePathsAsString ? modifiedFilePathsAsString.split('\0') : []).filter(str => str !== ''); // split on \0 rather than \n due to the -z option of git diff
 
     return { modifiedFilePaths, modifiedServicesIds: Array.from(new Set(modifiedFilePaths.map(DeclarationUtils.filePathToServiceId))) };
   }

package/src/reporter/github.js

@@ -24,14 +24,6 @@ export default class GitHub {
   }
 
   async initialize() {
-    try {
-      const { data: user } = await this.octokit.request('GET /user', { ...this.commonParams });
-
-      this.authenticatedUserLogin = user.login;
-    } catch (error) {
-      logger.error(`🤖 Could not get authenticated user: ${error}`);
-    }
-
     this.MANAGED_LABELS = require('./labels.json');
 
     const existingLabels = await this.getRepositoryLabels();
@@ -140,7 +132,6 @@ export default class GitHub {
       const issues = await this.octokit.paginate('GET /repos/{owner}/{repo}/issues', {
         ...this.commonParams,
         per_page: 100,
-        creator: this.authenticatedUserLogin,
         ...searchParams,
       }, response => response.data);