Fix lints

.eslintignore

@@ -0,0 +1,2 @@
+node_modules/
+dist/

.eslintrc.json

@@ -86,11 +86,7 @@
     ],
     "node/no-missing-require": "error",
     "node/no-missing-import": "error",
-    "node/no-unsupported-features": [
-      "error",
-      {
-        "version": 10
-      }
-    ]
+    "node/no-unsupported-features/es-syntax": "error",
+    "node/no-unsupported-features/es-builtins": "error"
   }
 }

README.md

@@ -24,3 +24,8 @@ the messages from a Matrix homeserver.
 
 Re-using Hydrogen gets us pretty and native(to Element) looking styles and keeps
 the maintenance burden of supporting more event types in Hydrogen.
+
+## Prerequisites
+
+- Node.js v17
+  - We only need v17 because it includes `require('crypto').webcrypto`

package.json

@@ -3,8 +3,15 @@
   "version": "0.0.0",
   "scripts": {
     "start-dev": "node server/start-dev.js",
+    "lint": "eslint **/*.js",
+
     "nodemon": "nodemon",
-    "vite": "vite"
+    "vite": "vite",
+    "eslint": "eslint",
+    "prettier": "prettier"
+  },
+  "engines" : { 
+    "node" : ">=17.0.0" 
   },
   "devDependencies": {
     "eslint": "^8.8.0",

public/js/.eslintrc.json

@@ -0,0 +1,15 @@
+{
+  "env": {
+    "browser": true,
+    "commonjs": false,
+    "node": false
+  },
+  "parserOptions": {
+    "ecmaVersion": 2018,
+    "sourceType": "module"
+  },
+  "rules": {
+    "node/no-unsupported-features/es-syntax": "off",
+    "node/no-unsupported-features/es-builtins": "off"
+  }
+}

public/js/asdf.js

@@ -1,5 +0,0 @@
-function asdf() {
-  console.log('asdfasdfasdfasdf');
-}
-
-module.exports = asdf;

public/js/entry-client.js

@@ -1,9 +1,2 @@
-import asdf from './asdf';
-
-asdf();
-
 import mounted from 'matrix-public-archive-shared/hydrogen-vm-render-script';
 console.log('mounted', mounted);
-
-// import mounted2 from '../../shared/hydrogen-vm-render-script';
-// console.log('mounted2', mounted2);

server/fetch-events-for-timestamp.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const assert = require('assert');
 
 const urlJoin = require('./lib/url-join');

server/fetch-room-data.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const assert = require('assert');
 
 const urlJoin = require('./lib/url-join');

server/lib/express-async-handler.js

@@ -1,3 +1,5 @@
+'use strict';
+
 // Simple middleware for handling exceptions inside of async express routes and
 // passing them to your express error handlers.
 //

server/lib/fetch-endpoint.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const assert = require('assert');
 const fetch = require('node-fetch');
 

server/lib/url-join.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const path = require('path');
 
 // via https://javascript.plainenglish.io/how-to-safely-concatenate-url-with-node-js-f6527b623d5

server/render-hydrogen-to-string.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const assert = require('assert');
 const vm = require('vm');
 const path = require('path');

server/routes/install-routes.js

@@ -14,9 +14,7 @@ const config = require('../../config.json');
 const basePath = config.basePath;
 assert(basePath);
 
-function parseArchiveRangeFromReq(req, res) {
-  const roomIdOrAlias = req.params.roomIdOrAlias;
-
+function parseArchiveRangeFromReq(req) {
   const yyyy = parseInt(req.params.yyyy, 10);
   // Month is the only zero-based index in this group
   const mm = parseInt(req.params.mm, 10) - 1;
@@ -39,23 +37,6 @@ function parseArchiveRangeFromReq(req, res) {
     if (Number.isNaN(fromHour) || fromHour < 0 || fromHour > 23) {
       throw new StatusError(404, 'From hour can only be in range 0-23');
     }
-
-    // Currently we force the range to always be 1 hour
-    // If the format isn't correct, redirect to the correct hour range
-    if (toHour !== fromHour + 1) {
-      res.redirect(
-        urlJoin(
-          basePath,
-          roomIdOrAlias,
-          'date',
-          req.params.yyyy,
-          req.params.mm,
-          req.params.dd,
-          `${fromHour}-${fromHour + 1}`
-        )
-      );
-      return;
-    }
   }
 
   const fromTimestamp = Date.UTC(yyyy, mm, dd, fromHour);
@@ -70,6 +51,8 @@ function parseArchiveRangeFromReq(req, res) {
     yyyy,
     mm,
     dd,
+    fromHour,
+    toHour,
   };
 }
 
@@ -104,7 +87,24 @@ function installRoutes(app) {
       const roomIdOrAlias = req.params.roomIdOrAlias;
       assert(roomIdOrAlias.startsWith('!') || roomIdOrAlias.startsWith('#'));
 
-      const { fromTimestamp, toTimestamp } = parseArchiveRangeFromReq(req, res);
+      const { fromTimestamp, fromHour, toHour } = parseArchiveRangeFromReq(req);
+
+      // Currently we force the range to always be 1 hour
+      // If the format isn't correct, redirect to the correct hour range
+      if (toHour !== fromHour + 1) {
+        res.redirect(
+          urlJoin(
+            basePath,
+            roomIdOrAlias,
+            'date',
+            req.params.yyyy,
+            req.params.mm,
+            req.params.dd,
+            `${fromHour}-${fromHour + 1}`
+          )
+        );
+        return;
+      }
 
       const [roomData, { events, stateEventMap }] = await Promise.all([
         fetchRoomData(roomIdOrAlias),

server/start-dev.js

@@ -1,4 +1,5 @@
 'use strict';
+
 const path = require('path');
 const nodemon = require('nodemon');
 const { build } = require('vite');

shared/CalendarView.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const { TemplateView } = require('hydrogen-view-sdk');
 
 // Month in JavaScript is 0-indexed (January is 0, February is 1, etc),

shared/RightPanelContentView.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const { TemplateView } = require('hydrogen-view-sdk');
 
 const CalendarView = require('matrix-public-archive-shared/CalendarView');

shared/hydrogen-vm-render-script.js

@@ -12,8 +12,8 @@ const {
   encodeKey,
   encodeEventIdKey,
   Timeline,
-  TimelineView,
-  RoomView,
+  // TimelineView,
+  // RoomView,
   RoomViewModel,
   ViewModel,
 } = require('hydrogen-view-sdk');

shared/lib/assert.js

@@ -1,3 +1,5 @@
+'use strict';
+
 function assert(value, message) {
   console.assert(value, message);
   if (!value) {