lint

.gitignore

@@ -15,6 +15,7 @@ public/
 
 # Generated during test run
 /spec-lib
+/hookshot-int-*
 
 book
 *.cer

eslint.config.mjs

@@ -6,7 +6,7 @@ import chai from "eslint-plugin-chai-expect";
 
 export default [
     {
-        ignores: ["lib/**/*", "contrib/**/*"],
+        ignores: ["lib/**/*", "spec-lib/**/*", "contrib/**/*"],
     },
     ...tseslint.config(
         {

package.json

@@ -32,7 +32,7 @@
     "start:matrixsender": "node --require source-map-support/register lib/App/MatrixSenderApp.js",
     "start:resetcrypto": "node --require source-map-support/register lib/App/ResetCryptoStore.js",
     "test": "mocha -r ts-node/register tests/init.ts 'tests/*.ts'  'tests/**/*.ts'",
-    "test:e2e": "tsc --p tsconfig.spec.json && yarn node --experimental-vm-modules $(yarn bin jest)",
+    "test:e2e": "tsc --p tsconfig.spec.json && cp ./lib/libRs.js ./lib/matrix-hookshot-rs.node ./spec-lib/src  && yarn node --experimental-vm-modules $(yarn bin jest)",
     "test:cover": "nyc --reporter=lcov --reporter=text yarn test",
     "lint": "yarn run lint:js && yarn run lint:rs",
     "lint:js": "eslint",

src/AdminRoom.ts

@@ -1,4 +1,3 @@
-
 import "reflect-metadata";
 import { AdminAccountData, AdminRoomCommandHandler, Category } from "./AdminRoomCommandHandler";
 import { botCommand, compileBotCommands, handleCommand, BotCommands, HelpFunction } from "./BotCommands";

src/AdminRoomCommandHandler.ts

@@ -13,7 +13,6 @@ export enum Category {
 
 
 export interface AdminAccountData {
-
     admin_user: string;
     github?: {
         notifications?: {

src/CommentProcessor.ts

@@ -16,7 +16,6 @@ const log = new Logger("CommentProcessor");
 const mime = import('mime');
 
 interface IMatrixCommentEvent extends MatrixMessageContent {
-
     external_url: string;
     "uk.half-shot.matrix-hookshot.github.comment": {
         id: number;

src/Connections/GithubIssue.ts

@@ -20,7 +20,6 @@ export interface GitHubIssueConnectionState {
     repo: string;
     state: string;
     issues: string[];
-
     comments_processed: number;
 }
 

src/Connections/GithubProject.ts

@@ -7,7 +7,6 @@ import { ConfigGrantChecker, GrantChecker } from "../grants/GrantCheck";
 import { BridgeConfig } from "../config/Config";
 
 export interface GitHubProjectConnectionState {
-
     project_id: number;
     state: "open"|"closed";
 }

src/Connections/GithubUserSpace.ts

@@ -76,7 +76,6 @@ export class GitHubUserSpace extends BaseConnection implements IConnection {
             throw Error("Could not find repo");
         }
 
-
         let avatarState: {type: "m.room.avatar", state_key: "", content: { url: string}}|undefined;
         try {
             if (avatarUrl) {

src/Gitlab/Types.ts

@@ -1,4 +1,3 @@
-
 export interface GitLabAuthor {
     id: number;
     name: string;

src/Gitlab/WebhookTypes.ts

@@ -1,5 +1,3 @@
-
-
 export interface IGitLabWebhookEvent {
     object_kind: string;
 }

src/MatrixEvent.ts

@@ -1,4 +1,3 @@
-
 export interface MatrixEvent<T extends MatrixEventContent|unknown> {
     content: T;
     event_id: string;
@@ -8,7 +7,6 @@ export interface MatrixEvent<T extends MatrixEventContent|unknown> {
     type: string;
 }
 
-
 type MatrixEventContent = object;
 
 export interface MatrixMemberContent extends MatrixEventContent {

src/NotificationsProcessor.ts

@@ -10,7 +10,6 @@ import { GitHubUserNotification } from "./github/Types";
 import { components } from "@octokit/openapi-types/types";
 import { NotifFilter } from "./NotificationFilters";
 
-
 const log = new Logger("NotificationProcessor");
 const md = new markdown();
 
@@ -21,27 +20,22 @@ export interface IssueDiff {
     merged: boolean;
     mergedBy: null|{
         login: string;
-
         html_url: string;
     };
     user: {
         login: string;
-
         html_url: string;
     };
 }
 
 export interface CachedReviewData {
-
     requested_reviewers: PullsListRequestedReviewersResponseData;
     reviews: PullsListReviewsResponseData;
 }
 
 type PROrIssue = IssuesGetResponseData|PullGetResponseData;
 
 export class NotificationProcessor {
-
-
     private static formatUser(user: {login: string, html_url: string}) {
         return `**[${user.login}](${user.html_url})**`;
     }

src/config/Config.ts

@@ -51,11 +51,8 @@ interface BridgeConfigGitHubYAML {
         secret: string;
     };
     oauth?: {
-
         client_id: string;
-
         client_secret: string;
-
         redirect_uri: string;
     };
     defaultOptions?: GitHubRepoConnectionOptions;
@@ -74,11 +71,8 @@ export class BridgeConfigGitHub {
     };
     @configKey("Settings for allowing users to sign in via OAuth.", true)
     readonly oauth?: {
-
         client_id: string;
-
         client_secret: string;
-
         redirect_uri: string;
     };
     @configKey("Default options for GitHub connections.", true)
@@ -111,18 +105,14 @@ export class BridgeConfigGitHub {
 }
 
 export interface BridgeConfigJiraCloudOAuth {
-
     client_id: string;
-
     client_secret: string;
-
     redirect_uri: string;
 }
 
 export interface BridgeConfigJiraOnPremOAuth {
     consumerKey: string;
     privateKey: string;
-
     redirect_uri: string;
 }
 
@@ -186,11 +176,6 @@ export class BridgeConfigJira implements BridgeConfigJiraYAML {
 
 export interface GitLabInstance {
     url: string;
-    // oauth: {
-    //     client_id: string;
-    //     client_secret: string;
-    //     redirect_uri: string;
-    // };
 }
 
 export interface BridgeConfigGitLabYAML {
@@ -746,11 +731,9 @@ export async function parseRegistrationFile(filename: string) {
 if (require.main === module) {
     Logger.configure({console: "info"});
     BridgeConfig.parseConfig(process.argv[2] || "config.yml", process.env).then(() => {
-
         console.log('Config successfully validated.');
         process.exit(0);
     }).catch(ex => {
-
         console.error('Error in config:', ex);
         process.exit(1);
     });

src/config/Defaults.ts

@@ -248,19 +248,16 @@ async function renderRegistrationFile(configPath?: string) {
             rooms: [],
         },
     };
-     
+
     console.log(YAML.stringify(obj));
 }
 
-
 // Can be called directly
 if (require.main === module) {
     if (process.argv[2] === '--config') {
-
         console.log(renderDefaultConfig());
     } else if (process.argv[2] === '--registration') {
         renderRegistrationFile(process.argv[3]).catch(ex => {
-
             console.error(ex);
             process.exit(1);
         });

src/generic/types.ts

@@ -5,10 +5,8 @@ export interface GenericWebhookEvent {
     hookId: string;
 }
 
-
 export type GenericWebhookEventResult = GenericWebhookEventResultSuccess | GenericWebhookEventResultFailure;
 
-
 export interface GenericWebhookEventResultSuccess {
     successful: true|null;
     response?: WebhookResponse,

src/github/Types.ts

@@ -19,7 +19,6 @@ export type CreateInstallationAccessTokenDataType = Endpoints["POST /app/install
 
 export const NAMELESS_ORG_PLACEHOLDER = "No name";
 
-
 export interface GitHubUserNotification {
     id: string;
     reason: "assign"|"author"|"comment"|"invitation"|"manual"|"mention"|"review_requested"|

tests/connections/FeedTest.spec.ts

@@ -1,4 +1,3 @@
-
 import { AppserviceMock } from "../utils/AppserviceMock";
 import { FeedConnection, FeedConnectionState } from "../../src/Connections";
 import { FeedEntry } from "../../src/feeds/FeedReader";

tests/connections/GenericHookTest.ts

@@ -1,4 +1,3 @@
-
 import { assert, expect } from "chai";
 import { Appservice, Intent, MatrixError } from "matrix-bot-sdk";
 import { BridgeConfigGenericWebhooks, BridgeGenericWebhooksConfigYAML } from "../../src/config/sections";

tests/connections/GithubRepoTest.ts

@@ -55,7 +55,6 @@ function createConnection(state: Record<string, unknown> = {}, isExistingState=f
 		"state_key",
 		githubInstance,
 		// Default config always contains GitHub
-
 		DefaultConfig.github!
 	);
 	return {connection, intent: intent as IntentMock};

tsconfig.spec.json

@@ -1,29 +1,11 @@
 {
-  "extends": "@tsconfig/node22/tsconfig.json",
+  "extends": "./tsconfig.json",
   "compilerOptions": {
-    "incremental": true,
+    "incremental": false,
     "declaration": false,
     "outDir": "./spec-lib",
     "rootDir": "./",
-    "noImplicitAny": true,
-    "strictNullChecks": true,
-    "strictFunctionTypes": true,
-    "strictBindCallApply": true,
-    "strictPropertyInitialization": true,
-    "noImplicitThis": true,
-    "alwaysStrict": true,
-    /* Additional Checks */
-    "noUnusedLocals": false,
-    "noUnusedParameters": false,
-    "noImplicitReturns": true,
-    "noFallthroughCasesInSwitch": true,
-    "inlineSourceMap": true,
-    "inlineSources": true,
-    "allowJs": true,
-    "experimentalDecorators": true,
-    "emitDecoratorMetadata": true,
-    // TODO: Type errors
-    "useUnknownInCatchVariables": false
+    "allowJs": true
   },
   "include": [
       "spec/**/*"