wip

.build.yml

@@ -1,4 +1,5 @@
-image: alpine/latest
+# TODO switch back to alpine/latest once the "npm install" deadlock is fixed
+image: alpine/edge
 packages:
 - npm
 - rsync
@@ -6,8 +7,6 @@ sources:
 - https://codeberg.org/emersion/gamja.git
 secrets:
 - 7a146c8e-aeb4-46e7-99bf-05af7486bbe9 # deploy SSH key
-artifacts:
-- gamja/gamja.tar.gz
 tasks:
 - setup: |
     cd gamja
@@ -15,7 +14,6 @@ tasks:
 - build: |
     cd gamja
     npm run build
-    tar -czf gamja.tar.gz -C dist .
 - lint: |
     cd gamja
     npm run -- lint --max-warnings 0

README.md

@@ -2,7 +2,7 @@
 
 A simple IRC web client.
 
-<img src="https://fs.emersion.fr/protected/img/gamja/main.png" alt="Screenshot" width="800">
+<img src="https://codeberg.org/attachments/117ff232-7e73-46c7-a21d-2b59ffa3765a" alt="Screenshot" width="800">
 
 ## Usage
 

commands.js

@@ -190,10 +190,9 @@ const commands = [
 				throw new Error("Missing nick");
 			}
 			let activeChannel = getActiveChannel(app);
-			getActiveClient(app).send({
+			getActiveClient(app).send({ command: "INVITE", params: [
-				command: "INVITE",
+				nick, activeChannel,
-				params: [nick, activeChannel],
+			]});
-			});
 		},
 	},
 	{ ...join, name: "j" },

components/app.js

@@ -762,7 +762,7 @@ export default class App extends Component {
 
 		// Open a new buffer if the message doesn't come from me or is a
 		// self-message
-		if ((!client.isMyNick(msg.prefix.name) || client.isMyNick(bufName)) && (msg.command !== "PART" && msg.command !== "QUIT" && msg.command !== irc.RPL_MONONLINE && msg.command !== irc.RPL_MONOFFLINE)) {
+		if ((!client.isMyNick(msg.prefix.name) || client.isMyNick(bufName)) && (msg.command !== "PART" && msg.comand !== "QUIT" && msg.command !== irc.RPL_MONONLINE && msg.command !== irc.RPL_MONOFFLINE)) {
 			this.createBuffer(serverID, bufName);
 		}
 
@@ -1075,7 +1075,6 @@ export default class App extends Component {
 		case "ACK":
 		case "BOUNCER":
 		case "MARKREAD":
-		case "REDACT":
 			// Ignore these
 			return [];
 		default:
@@ -2006,11 +2005,9 @@ export default class App extends Component {
 		this.lastFocusPingDate = now;
 
 		for (let client of this.clients.values()) {
-			if (client.status === Client.Status.REGISTERED) {
 			client.send({ command: "PING", params: ["gamja"] });
 		}
 	}
-	}
 
 	componentDidMount() {
 		this.baseTitle = document.title;

components/buffer.js

@@ -94,7 +94,7 @@ function canFoldMessage(msg) {
 
 class LogLine extends Component {
 	shouldComponentUpdate(nextProps) {
-		return this.props.message !== nextProps.message || this.props.redacted !== nextProps.redacted;
+		return this.props.message !== nextProps.message;
 	}
 
 	render() {
@@ -143,18 +143,12 @@ class LogLine extends Component {
 					`;
 				}
 			} else {
+				lineClass = "talk";
 				let prefix = "<", suffix = ">";
 				if (msg.command === "NOTICE") {
-					lineClass += " notice";
 					prefix = suffix = "-";
 				}
-				if (this.props.redacted) {
+				content = html`${prefix}${createNick(msg.prefix.name)}${suffix} ${linkify(stripANSI(text), onChannelClick)}`;
-					content = html`<i>This message has been deleted.</i>`;
-				} else {
-					content = html`${linkify(stripANSI(text), onChannelClick)}`;
-					lineClass += " talk";
-				}
-				content = html`<span class="nick-caret">${prefix}</span>${createNick(msg.prefix.name)}<span class="nick-caret">${suffix}</span> ${content}`;
 			}
 
 			let allowedPrefixes = server.statusMsg;
@@ -715,7 +709,6 @@ export default class Buffer extends Component {
 					message=${msg}
 					buffer=${buf}
 					server=${server}
-					redacted=${buf.redacted.has(msg.tags.msgid)}
 					onChannelClick=${onChannelClick}
 					onNickClick=${onNickClick}
 					onVerifyClick=${onVerifyClick}
@@ -821,7 +814,7 @@ export default class Buffer extends Component {
 
 			if (sep.length > 0) {
 				children.push(createFoldGroup(foldMessages));
-				children.push(...sep);
+				children.push(sep);
 				foldMessages = [];
 			}
 

components/help.js

@@ -42,8 +42,8 @@ function KeyBindingsHelp() {
 }
 
 function CommandsHelp() {
-	let l = [...commands.keys()].map((name) => {
+	let l = Object.keys(commands).map((name) => {
-		let cmd = commands.get(name);
+		let cmd = commands[name];
 
 		let usage = [html`<strong>/${name}</strong>`];
 		if (cmd.usage) {

eslint.config.js

@@ -34,23 +34,15 @@ export default [
 			"no-implicit-coercion": "warn",
 			"object-shorthand": "warn",
 			"curly": "warn",
-			"camelcase": "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",
-			"@stylistic/js/brace-style": ["warn", "1tbs"],
 			"@stylistic/js/comma-dangle": ["warn", "always-multiline"],
-			"@stylistic/js/comma-spacing": "warn",
 			"@stylistic/js/arrow-parens": "warn",
-			"@stylistic/js/arrow-spacing": "warn",
-			"@stylistic/js/block-spacing": "warn",
-			"@stylistic/js/object-curly-spacing": ["warn", "always"],
-			"@stylistic/js/object-curly-newline": ["warn", {
-				multiline: true,
-				consistent: true,
-			}],
-			"@stylistic/js/array-bracket-spacing": ["warn", "never"],
-			"@stylistic/js/array-bracket-newline": ["warn", "consistent"],
 		},
 	},
 ];

index.html

@@ -3,19 +3,7 @@
 	<head>
 		<meta charset="utf-8">
 		<meta http-equiv="Content-Security-Policy" content="default-src 'self'; frame-src 'none'; object-src 'none'; connect-src *;">
-		<title>Cringe Studios Internet IRC Relay Chat Client</title>
+		<title>gamja IRC client</title>
-		
-		<meta name="description" content="Cringe Studios Internet IRC Relay Chat Client">
-		<meta property="og:image" content="https://nsfw.cringe-studios.com/Wikimedia_Community_Logo-IRC.png">
-		<meta property="og:type" content="website">
-		<meta property="og:url" content="https://irc.cringe-studios.com/">
-		<meta property="og:title" content="Cringe Studios Internet IRC Relay Chat Client">
-		<meta property="og:description" content="Cringe Studios Internet IRC Relay Chat Client">
-		
-        <meta name="theme-color" content="#8000f0" data-react-helmet="true">
-        <link rel="icon" href="https://nsfw.cringe-studios.com/Wikimedia_Community_Logo-IRC.png">
-		
-		
 		<link rel="stylesheet" href="./style.css">
 		<script type="module" src="./main.js"></script>
 		<meta name="viewport" content="width=device-width, initial-scale=1">

lib/client.js

@@ -21,7 +21,6 @@ const permanentCaps = [
 	"draft/account-registration",
 	"draft/chathistory",
 	"draft/extended-monitor",
-	"draft/message-redaction",
 	"draft/read-marker",
 
 	"soju.im/bouncer-networks",

lib/oauth2.js

@@ -43,9 +43,9 @@ export function redirectAuthorize({ serverMetadata, clientId, redirectUri, scope
 	// TODO: use the state param to prevent cross-site request
 	// forgery
 	let params = {
-		"response_type": "code",
+		response_type: "code",
-		"client_id": clientId,
+		client_id: clientId,
-		"redirect_uri": redirectUri,
+		redirect_uri: redirectUri,
 	};
 	if (scope) {
 		params.scope = scope;
@@ -66,12 +66,12 @@ function buildPostHeaders(clientId, clientSecret) {
 
 export async function exchangeCode({ serverMetadata, redirectUri, code, clientId, clientSecret }) {
 	let data = {
-		"grant_type": "authorization_code",
+		grant_type: "authorization_code",
 		code,
-		"redirect_uri": redirectUri,
+		redirect_uri: redirectUri,
 	};
 	if (!clientSecret) {
-		data["client_id"] = clientId;
+		data.client_id = clientId;
 	}
 
 	let resp = await fetch(serverMetadata.token_endpoint, {

manifest.json

@@ -1,6 +1,6 @@
 {
-	"name": "Cringe Studios Internet IRC Relay Chat Client",
+	"name": "gamja IRC client",
-	"short_name": "IRC",
+	"short_name": "gamja",
 	"start_url": ".",
 	"display": "standalone",
 	"scope": "."

package.json

@@ -10,7 +10,7 @@
     "@eslint/js": "^9.11.1",
     "@parcel/packager-raw-url": "^2.0.0",
     "@parcel/transformer-webmanifest": "^2.0.0",
-    "@stylistic/eslint-plugin-js": "^3.1.0",
+    "@stylistic/eslint-plugin-js": "^2.8.0",
     "eslint": "^9.11.1",
     "globals": "^15.9.0",
     "node-static": "^0.7.11",

state.js

@@ -153,24 +153,10 @@ function trimStartCharacter(s, c) {
 	return s.substring(i);
 }
 
-function getBouncerNetworkNameFromBuffer(state, buffer) {
-	let server = state.servers.get(buffer.server);
-	let network = state.bouncerNetworks.get(server.bouncerNetID);
-	if (!network) {
-		return null;
-	}
-	return getServerName(server, network);
-}
-
 /* Returns 1 if a should appear after b, -1 if a should appear before b, or
  * 0 otherwise. */
-function compareBuffers(state, a, b) {
+function compareBuffers(a, b) {
 	if (a.server !== b.server) {
-		let aServerName = getBouncerNetworkNameFromBuffer(state, a);
-		let bServerName = getBouncerNetworkNameFromBuffer(state, b);
-		if (aServerName && bServerName && aServerName !== bServerName) {
-			return aServerName.localeCompare(bServerName);
-		}
 		return a.server > b.server ? 1 : -1;
 	}
 	if (isServerBuffer(a) !== isServerBuffer(b)) {
@@ -375,11 +361,10 @@ export const State = {
 			hasInitialWho: false, // if channel
 			members: new irc.CaseMapMap(null, client.cm), // if channel
 			messages: [],
-			redacted: new Set(),
 			unread: Unread.NONE,
 			prevReadReceipt: null,
 		});
-		bufferList = bufferList.sort((a, b) => compareBuffers(state, a, b));
+		bufferList = bufferList.sort(compareBuffers);
 		let buffers = new Map(bufferList.map((buf) => [buf.id, buf]));
 		return [id, { buffers }];
 	},
@@ -680,14 +665,6 @@ export const State = {
 
 				return { members };
 			});
-		case "REDACT":
-			target = msg.params[0];
-			if (client.isMyNick(target)) {
-				target = msg.prefix.name;
-			}
-			return updateBuffer(target, (buf) => {
-				return { redacted: new Set(buf.redacted).add(msg.params[1]) };
-			});
 		case irc.RPL_MONONLINE:
 		case irc.RPL_MONOFFLINE:
 			targets = msg.params[1].split(",");