wip

Seems to not cause regressions like it did last time. Crossing
fingers now.

README.md

@@ -2,7 +2,7 @@
 
 A simple IRC web client.
 
-![screenshot](https://l.sr.ht/7Npm.png)
+<img src="https://codeberg.org/attachments/117ff232-7e73-46c7-a21d-2b59ffa3765a" alt="Screenshot" width="800">
 
 ## Usage
 

components/app.js

@@ -323,6 +323,8 @@ export default class App extends Component {
 		}
 		if (queryParams.debug === "1") {
 			this.debug = true;
+		} else if (queryParams.debug === "0") {
+			this.debug = false;
 		}
 
 		if (window.location.hash) {

components/buffer.js

@@ -43,7 +43,7 @@ function _Timestamp({ date, url, showSeconds }) {
 		if (showSeconds) {
 			timestamp += ":--";
 		}
-		return html`<spam class="timestamp">${timestamp}</span>`;
+		return html`<span class="timestamp">${timestamp}</span>`;
 	}
 
 	let hh = date.getHours().toString().padStart(2, "0");
@@ -274,9 +274,15 @@ class LogLine extends Component {
 			break;
 		case "TOPIC":
 			let topic = msg.params[1];
-			content = html`
+			if (topic) {
-				${createNick(msg.prefix.name)} changed the topic to: ${linkify(stripANSI(topic), onChannelClick)}
+				content = html`
-			`;
+					${createNick(msg.prefix.name)} changed the topic to: ${linkify(stripANSI(topic), onChannelClick)}
+				`;
+			} else {
+				content = html`
+					${createNick(msg.prefix.name)} cleared the topic
+				`;
+			}
 			break;
 		case "INVITE":
 			invitee = msg.params[0];

dev-server.js

@@ -63,7 +63,8 @@ if (remoteHost) {
 			ws.close();
 		});
 
-		client.on("error", () => {
+		client.on("error", (err) => {
+			console.log(err);
 			ws.close(WS_BAD_GATEWAY);
 		});
 	});

doc/url-params.md

@@ -7,7 +7,7 @@ gamja settings can be overridden using URL query parameters:
   replaced with a randomly generated value)
 - `channels`: comma-separated list of channels to join (`#` needs to be escaped)
 - `open`: [IRC URL] to open
-- `debug`: if set to 1, debug mode is enabled
+- `debug`: enable debug logs if set to `1`, disable debug logs if set to `0`
 
 Alternatively, the channels can be set with the URL fragment (ie, by just
 appending the channel name to the gamja URL).

eslint.config.js

@@ -23,11 +23,21 @@ export default [
 				destructuredArrayIgnorePattern: "^_",
 			}],
 			"no-var": "error",
+			"no-eval": "error",
+			"no-implied-eval": "error",
 			"eqeqeq": "error",
 			"no-invalid-this": "error",
+			"no-extend-native": "error",
 			"prefer-arrow-callback": "error",
+			"no-implicit-globals": "error",
+			"no-throw-literal": "error",
 			"no-implicit-coercion": "warn",
 			"object-shorthand": "warn",
+			"curly": "warn",
+			"no-restricted-syntax": ["warn", {
+				selector: "BlockStatement VariableDeclaration[kind='const']",
+				message: "NOPE",
+			}],
 			"@stylistic/js/indent": ["warn", "tab"],
 			"@stylistic/js/quotes": ["warn", "double"],
 			"@stylistic/js/semi": "warn",

package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "htm": "^3.0.4",
     "linkifyjs": "^4.1.3",
-    "preact": "10.17.1"
+    "preact": "^10.17.1"
   },
   "devDependencies": {
     "@eslint/js": "^9.11.1",