From 8ee270f04f454e229b39eea6a843634383ca8dff Mon Sep 17 00:00:00 2001
From: Angelo DiNardi <angelo@dinardi.name>
Date: Sun, 4 Jun 2017 23:37:14 -0700
Subject: [PATCH 1/7] Make an attempt at being able to render rss

---
 index.js  |  65 +++++++++++++++++++++------------
 render.js | 106 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 149 insertions(+), 22 deletions(-)

diff --git a/index.js b/index.js
index 5b00d6c..3ed0bee 100644
--- a/index.js
+++ b/index.js
@@ -16,7 +16,10 @@ var {
     formatMsgs,
     wrapPage,
     renderThread,
-    renderAbout
+    renderAbout,
+    renderItem,
+    wrapRss,
+    renderRssContent,
 } = require('./render');
 
 var appHash = hash([fs.readFileSync(__filename)])
@@ -90,7 +93,7 @@ exports.init = function (sbot, config) {
     }
   }
 
-  function serveFeed(req, res, feedId) {
+  function serveFeed(req, res, feedId, ext) {
       console.log("serving feed: " + feedId)
 
       var showAll = req.url.endsWith("?showAll")
@@ -99,26 +102,43 @@ exports.init = function (sbot, config) {
       getAbout(feedId, function (err, about) {
           if (err) return respond(res, 500, err.stack || err)
 
-	  pull(
-	      sbot.createUserStream({ id: feedId, reverse: true, limit: showAll ? -1 : 10 }),
-	      pull.collect(function (err, logs) {
-		  if (err) return respond(res, 500, err.stack || err)
-		  res.writeHead(200, {
-		      'Content-Type': ctype("html")
-		  })
-		  pull(
-		      pull.values(logs),
-		      paramap(addAuthorAbout, 8),
-		      paramap(addFollowAbout, 8),
-		      paramap(addVoteMessage, 8),
-		      paramap(addGitLinks, 8),
-		      pull(renderAbout(defaultOpts, about, showAllHTML), wrapPage(about.name)),
-		      toPull(res, function (err) {
-			  if (err) console.error('[viewer]', err)
-		      })
-		  )
-	      })
-	  )
+          function render() {
+            switch (ext) {
+              case 'rss':
+                return pull(
+                  // formatMsgs(feedId, ext, defaultOpts)
+                  pull.filter( (msg) => {
+                    return !!renderRssContent(defaultOpts, msg.value.content || {});
+                  }),
+                  renderItem(defaultOpts), wrapRss(about.name, defaultOpts)
+                );
+              default:
+                return pull(
+                  renderAbout(defaultOpts, about, showAllHTML), wrapPage(about.name)
+                );
+            }
+          }
+
+          pull(
+              sbot.createUserStream({ id: feedId, reverse: true, limit: showAll ? -1 : (ext == 'rss' ? 25 :10) }),
+              pull.collect(function (err, logs) {
+              if (err) return respond(res, 500, err.stack || err)
+              res.writeHead(200, {
+                  'Content-Type': ctype(ext)
+              })
+              pull(
+                  pull.values(logs),
+                  paramap(addAuthorAbout, 8),
+                  paramap(addFollowAbout, 8),
+                  paramap(addVoteMessage, 8),
+                  paramap(addGitLinks, 8),
+                  render(),
+                  toPull(res, function (err) {
+                    if (err) console.error('[viewer]', err)
+                  })
+              )
+              })
+          )
       })
   }
 
@@ -363,6 +383,7 @@ function ctype(name) {
     case 'js': return 'text/javascript'
     case 'css': return 'text/css'
     case 'json': return 'application/json'
+    case 'rss': return 'text/xml'
   }
 }
 
diff --git a/render.js b/render.js
index 0ffedff..4b9287b 100644
--- a/render.js
+++ b/render.js
@@ -13,6 +13,9 @@ exports.renderEmoji = renderEmoji;
 exports.formatMsgs = formatMsgs;
 exports.renderThread = renderThread;
 exports.renderAbout = renderAbout;
+exports.renderItem = renderItem;
+exports.wrapRss = wrapRss;
+exports.renderRssContent = renderRssContent;
 
 function MdRenderer(opts) {
   marked.Renderer.call(this, {});
@@ -84,6 +87,8 @@ function formatMsgs(id, ext, opts) {
       return pull(renderThread(opts), wrapJSEmbed(opts));
     case "json":
       return wrapJSON();
+    case "rss":
+      return pull(renderItem(opts), wrapRss(id, opts));
     default:
       return null;
   }
@@ -136,6 +141,12 @@ function renderThread(opts, showAllHTML = "") {
   );
 }
 
+function renderItem(opts) {
+  return pull(
+    pull.map(renderRss.bind(this, opts))
+  );
+}
+
 function wrapPage(id) {
   return wrap(
     "<!doctype html><html><head>" +
@@ -150,6 +161,18 @@ function wrapPage(id) {
   );
 }
 
+function wrapRss(id, opts) {
+  return wrap(
+    '<?xml version="1.0" encoding="UTF-8" ?>' +
+    '<rss version="2.0">' +
+      '<channel>' +
+        '<title>' + id + ' | ssb-viewer</title>',
+
+      '</channel>'+
+    '</rss>'
+  );
+}
+
 var styles = `
   <style>
     html { background-color: #f1f3f5; }
@@ -340,6 +363,89 @@ function renderMsg(opts, msg) {
   );
 }
 
+function renderRssContent(opts, c) {
+  var base = opts.base;
+  if (c.type === "post") {
+    return (c.channel ? `Posted in #${c.channel}:\n\n` : '') + c.text;
+  } else if (c.type == "vote" && c.vote.expression == "Dig") {
+    var linkedText = "a post";
+    if (typeof c.vote.linkedText != "undefined") {
+      linkedText = c.vote.linkedText.substring(0, 75);
+    }
+
+    return `Liked ${linkedText}`;
+  } else if (c.type == "vote") {
+    var linkedText = "a post";
+    if (typeof c.vote.linkedText != "undefined") {
+      linkedText = c.vote.linkedText.substring(0, 75);
+    }
+
+    return `Voted ${linkedText}`;
+  } else if (c.type == "contact" && c.following) {
+    var name = c.contact;
+    if (typeof c.contactAbout != "undefined") {
+      name = c.contactAbout.name;
+    }
+
+    return `Followed ${name}`;
+  } else if (c.type == "contact" && !c.following) {
+    var name = c.contact;
+    if (typeof c.contactAbout != "undefined") {
+      name = c.contactAbout.name;
+    }
+
+    return `Unfollowed ${name}`;
+  } else if (typeof c == "string") {
+    // Don't show private messages in RSS.
+  }
+  else if (c.type == "about") {
+    // Don't show about messages in RSS.
+  }
+  else if (c.type == "issue") {
+    return `Created a git issue ${(c.repoName != undefined ? " in repo " + c.repoName : "")}`;
+  }
+  else if (c.type == "git-update") {
+    return `Did a git update ${(c.repoName != undefined ? " in repo " + c.repoName : "")}
+    
+    ${(c.commits != undefined ? c.commits.map(com => { return "-" +com.title; }).join("\n") : "")}
+    `;
+  }
+  else if (c.type == "ssb-dns") {
+    // Don't show for RSS.
+  }
+  else if (c.type == "pub") {
+    return `Connected to the pub ${c.address.host}`;
+  }
+  else if (c.type == "channel" && c.subscribed) {
+    return `Subscribed to channel #${c.channel}`;
+  }
+  else if (c.type == "channel" && !c.subscribed) {
+    return `Unsubscribed from channel #${c.channel}`;
+  }
+
+}
+
+function renderRss(opts, msg) {
+  var c = msg.value.content || {};
+  var name = encodeURIComponent(msg.key);
+
+  let content = renderRssContent(opts, c);
+
+  if (!content) {
+    return null;
+  }
+
+  return (
+    '<item>' +
+      '<title>' + msg.author.name + ' - ' + formatDate(new Date(msg.value.timestamp)) + '</title>' +
+      '<description><![CDATA[' + content + ']]></description>' +
+      '<link>' + opts.base + escape(name) + '</link>' +
+      '<pubDate>' + new Date(msg.value.timestamp).toUTCString() + '</pubDate>' +
+      '<guid>' + msg.key + '</guid>' +
+    '</item>'
+  );
+}
+
 function msgTimestamp(msg, name) {
   var date = new Date(msg.value.timestamp);
   var isoStr = date.toISOString();

From b21b4ce09db70de58fef3c10705ba430079c9b26 Mon Sep 17 00:00:00 2001
From: Angelo DiNardi <angelo@dinardi.name>
Date: Sun, 4 Jun 2017 23:41:19 -0700
Subject: [PATCH 2/7] Don't include the relative time in the title...

:facepalm:
---
 render.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/render.js b/render.js
index 4b9287b..fb4ee37 100644
--- a/render.js
+++ b/render.js
@@ -437,7 +437,7 @@ function renderRss(opts, msg) {
 
   return (
     '<item>' +
-      '<title>' + msg.author.name + ' - ' + formatDate(new Date(msg.value.timestamp)) + '</title>' +
+      '<title>' + msg.author.name + '</title>' +
       '<description><![CDATA[' + content + ']]></description>' +
       '<link>' + opts.base + escape(name) + '</link>' +
       '<pubDate>' + new Date(msg.value.timestamp).toUTCString() + '</pubDate>' +

From c5cabd1c1390883a88e7ce23acc23075309f39bd Mon Sep 17 00:00:00 2001
From: Anders Rune Jensen <arj03@protonmail.ch>
Date: Mon, 5 Jun 2017 21:23:01 +0200
Subject: [PATCH 3/7] Convert to hyperscript

---
 package.json |   1 +
 render.js    | 274 +++++++++++++++++++++++----------------------------
 2 files changed, 126 insertions(+), 149 deletions(-)

diff --git a/package.json b/package.json
index 3a98f4d..f6422f8 100644
--- a/package.json
+++ b/package.json
@@ -9,6 +9,7 @@
     "emoji-named-characters": "^1.0.2",
     "emoji-server": "^1.0.0",
     "human-time": "^0.0.1",
+    "hyperscript": "2.0.2",
     "lrucache": "^1.0.2",
     "pull-cat": "^1.1.11",
     "pull-paramap": "^1.2.1",
diff --git a/render.js b/render.js
index fd9e95c..c3edca8 100644
--- a/render.js
+++ b/render.js
@@ -4,6 +4,7 @@ var marked = require("ssb-marked");
 var htime = require("human-time");
 var emojis = require("emoji-named-characters");
 var cat = require("pull-cat");
+var h = require('hyperscript');
 
 var emojiDir = path.join(require.resolve("emoji-named-characters"), "../pngs");
 
@@ -39,17 +40,11 @@ MdRenderer.prototype.urltransform = function(href) {
 };
 
 MdRenderer.prototype.image = function(href, title, text) {
-  return (
-    '<img src="' +
-    this.opts.img_base +
-    escape(href) +
-    '"' +
-    ' alt="' +
-    text +
-    '"' +
-    (title ? ' title="' + title + '"' : "") +
-    (this.options.xhtml ? "/>" : ">")
-  );
+  return h('img',
+	   { src: this.opts.img_base + href,
+	     alt: text,
+	     title: title
+	   }).outerHTML;
 };
 
 function renderEmoji(emoji) {
@@ -59,16 +54,12 @@ function renderEmoji(emoji) {
     ? opts.blob_base + encodeURIComponent(mentions[emoji])
     : emoji in emojis && opts.emoji_base + escape(emoji) + '.png';
   return url
-    ? '<img src="' +
-        url +
-        '"' +
-        ' alt=":' +
-        escape(emoji) +
-        ':"' +
-        ' title=":' +
-        escape(emoji) +
-        ':"' +
-        ' class="ssb-emoji" height="16" width="16">'
+	? h('img.ssb-emoji',
+	    { src: url,
+	      alt: ':' + escape(emoji) + ':',
+	      title: ':' + escape(emoji) + ':',
+	      height: 16, width: 16
+	    }).outerHTML
     : ":" + emoji + ":";
 }
 
@@ -99,44 +90,46 @@ function wrap(before, after) {
   };
 }
 
+function callToAction() {
+  return h('a.call-to-action',
+	   { href: 'https://www.scuttlebutt.nz' },
+	   'Join Scuttlebutt now').outerHTML;
+}
+
+function toolTipTop() {
+  return h('span.top-tip',
+	   'You are reading content from ',
+	   h('a', { href: 'https://www.scuttlebutt.nz' },
+	     'Scuttlebutt')).outerHTML;
+}
+
 function renderAbout(opts, about, showAllHTML = "") {
+  var figCaption = h('figcaption');
+  figCaption.innerHTML = 'Feed of ' + about.name + '<br>' +
+	(about.description != undefined ? 
+	 marked(about.description, opts.marked) : '');
   return pull(
     pull.map(renderMsg.bind(this, opts)),
-    wrap(
-      '<span class="top-tip">You are reading content from ' +
-        '<a href="https://www.scuttlebutt.nz">Scuttlebutt</a>' +
-      '</span>' +
-      '<main>' +
-      '<article><header><figure>' +
-      '<img src="' + opts.img_base + escape(about.image) + '" ' +
-            'height="200" width="200"><figcaption>' +
-	    'Feed of ' + about.name + '<br/>' +
-	    (about.description != undefined ? 
-	     marked(about.description, opts.marked) : '') +
-	    '</figcaption></figure></header></article>', 
-
-      showAllHTML + '</main>' +
-      '<a class="call-to-action" href="https://www.scuttlebutt.nz">' +
-        'Join Scuttlebutt now' +
-      '</a>'
-    )
+    wrap(toolTipTop() + '<main>' +
+	 h('article',
+	   h('header',
+	     h('figure',
+	       h('img',
+		 { src: opts.img_base + about.image,
+		   height: 200,
+		   width: 200
+		 }),
+	       figCaption)
+	    )).outerHTML,
+	 showAllHTML + '</main>' + callToAction())
   );
 }
 
 function renderThread(opts, showAllHTML = "") {
   return pull(
     pull.map(renderMsg.bind(this, opts)),
-    wrap(
-      '<span class="top-tip">You are reading content from ' +
-        '<a href="https://www.scuttlebutt.nz">Scuttlebutt</a>' +
-      '</span>' +
-      '<main>',
-
-      showAllHTML + '</main>' +
-      '<a class="call-to-action" href="https://www.scuttlebutt.nz">' +
-        'Join Scuttlebutt now' +
-      '</a>'
-    )
+    wrap(toolTipTop() + '<main>', 
+	 showAllHTML + '</main>' + callToAction())
   );
 }
 
@@ -320,45 +313,32 @@ function docWrite(str) {
 function renderMsg(opts, msg) {
   var c = msg.value.content || {};
   var name = encodeURIComponent(msg.key);
-  return (
-    '<article id="' +
-    name +
-    '">' +
-      '<header>' +
-        '<figure>' +
-          '<img alt="" ' +
-            'src="' + opts.img_base + escape(msg.author.image) + '" ' +
-            'height="50" width="50">' +
-          '<figcaption>' +
-            '<a class="ssb-avatar-name"' +
-            ' href="' + opts.base +
-            escape(msg.value.author) +
-            '"' +
-            ">" + msg.author.name + "</a>" +
-            msgTimestamp(msg, opts.base + name) +
-          '</figcaption>' +
-        '</figure>' +
-      '</header>' +
-      render(opts, c) +
-    "</article>"
-  );
+  return h('article#' + name,
+	   h('header',
+	     h('figure',
+	       h('img', { alt: '',
+			  src: opts.img_base + msg.author.image,
+			  height: 50, width: 50 }),
+	       h('figcaption',
+		 h('a.ssb-avatar-name',
+		   { href: opts.base + escape(msg.value.author) },
+		   msg.author.name),
+		 msgTimestamp(msg, opts.base + name)))),
+	   render(opts, c)).outerHTML;
 }
 
 function msgTimestamp(msg, link) {
   var date = new Date(msg.value.timestamp);
   var isoStr = date.toISOString();
-  return (
-    '<time class="ssb-timestamp" datetime="' + isoStr + '">' +
-      '<a ' +
-      'href="' + link + '" ' +
-      'title="' + isoStr + '" ' +
-      '>' + formatDate(date) + '</a>' +
-    '</time>'
-  );
+  return h('time.ssb-timestamp',
+	   { datetime: isoStr },
+	   h('a',
+	     { href: link,
+	       title: isoStr },
+	     formatDate(date)));
 }
 
 function formatDate(date) {
-  // return date.toISOString().replace('T', ' ')
   return htime(date);
 }
 
@@ -366,106 +346,102 @@ function render(opts, c) {
   var base = opts.base;
   if (c.type === "post") {
     var channel = c.channel
-      ? '<div class="top-right"><a href="' + base + 'channel/' + c.channel + '">#' + c.channel + "</a></div>"
-      : "";
-    return channel + renderPost(opts, c);
+	? h('div.top-right',
+	    h('a',
+	      { href: base + 'channel/' + c.channel },
+	      '#' + c.channel))
+	: "";
+    return [channel, renderPost(opts, c)];
   } else if (c.type == "vote" && c.vote.expression == "Dig") {
     var channel = c.channel
-      ? ' in <a href="' + base + 'channel/' + c.channel + '">#' + c.channel + "</a>"
-      : "";
+	? [' in ',
+	   h('a',
+	     { href: base + 'channel/' + c.channel },
+	     '#' + c.channel)]
+	: "";
     var linkedText = "this";
     if (typeof c.vote.linkedText != "undefined")
-      linkedText = c.vote.linkedText.substring(0, 75);
-    return ('<span class="status">' +
-      'Liked ' +
-      '<a href="' + base +
-      c.vote.link +
-      '">' +
-      linkedText +
-      "</a>" +
-      channel +
-      '</span>'
-    );
+	linkedText = c.vote.linkedText.substring(0, 75);
+    return h('span.status',
+	     ['Liked ',
+	      h('a', { href: base + c.vote.link }, linkedText),
+	      channel]);
   } else if (c.type == "vote") {
     var linkedText = "this";
     if (typeof c.vote.linkedText != "undefined")
       linkedText = c.vote.linkedText.substring(0, 75);
-    return '<span class="status">' +
-      'Voted <a href="' + base + c.vote.link + '">' + linkedText + "</a>" +
-      '</span>';
+      return h('span.status',
+	       ['Voted ',
+		h('a', { href: base + c.vote.link }, linkedText)]);
   } else if (c.type == "contact" && c.following) {
     var name = c.contact;
-    if (typeof c.contactAbout != "undefined") name = c.contactAbout.name;
-    return '<span class="status">' +
-      'Followed <a href="' + base + c.contact + '">' + name + "</a>" +
-      '</span>';
+    if (typeof c.contactAbout != "undefined")
+	name = c.contactAbout.name;
+    return h('span.status',
+	     ['Followed ',
+	      h('a', { href: base + c.contact }, name)]);
   } else if (c.type == "contact" && !c.following) {
     var name = c.contact;
-    if (typeof c.contactAbout != "undefined") name = c.contactAbout.name;
-    return '<span class="status">' +
-      'Unfollowed <a href="' + base + c.contact + '">' + name + "</a>" +
-      '</span>';
+    if (typeof c.contactAbout != "undefined")
+	name = c.contactAbout.name;
+    return h('span.status',
+	     ['Unfollowed ',
+	      h('a', { href: base + c.contact }, name)]);
   } else if (typeof c == "string") {
-    return '<span class="status">' +
-      "Wrote something private" +
-      '</span>';
+    return h('span.status', 'Wrote something private')
   }
   else if (c.type == "about") {
-    return '<span class="status">' +
-      "Changed something in about" +
-      '</span>' +
-      renderDefault(c);
+    return [h('span.status', 'Changed something in about'),
+ 	    renderDefault(c)];
   }
   else if (c.type == "issue") {
-    return '<span class="status">' +
-      "Created a git issue" + (c.repoName != undefined ? " in repo " + c.repoName : "") + renderPost(opts, c) +
-      '</span>';
+    return [h('span.status',
+	     "Created a git issue" +
+	      (c.repoName != undefined ? " in repo " + c.repoName : ""),
+	      renderPost(opts, c))];
   }
   else if (c.type == "git-update") {
-    return '<span class="status">' +
-      "Did a git update " + (c.repoName != undefined ? " in repo " + c.repoName : "") +
-      "<br/>" +
-     (c.commits != undefined ? c.commits.map(com => { return "-" +com.title; }).join("<br/>") : "") +
-      '</span>'
+    return h('span.status',
+	     "Did a git update " +
+	     (c.repoName != undefined ? " in repo " + c.repoName : "") +
+	     '<br>' +
+	     (c.commits != undefined ?
+	      c.commits.map(com => { return "-" +com.title; }).join('<br>') : ""));
   }
   else if (c.type == "ssb-dns") {
-    return '<span class="status">' +
-      "Updated DNS" +
-      '</span>' +
-      renderDefault(c);
+    return [h('span.status', 'Updated DNS'), renderDefault(c)];
   }
   else if (c.type == "pub") {
-    return '<span class="status">' +
-      "Connected to the pub " + c.address.host +
-      '</span>'
+    return h('span.status', 'Connected to the pub ' + c.address.host);
   }
   else if (c.type == "channel" && c.subscribed)
-    return '<span class="status">' +
-      'Subscribed to channel <a href="' + base + 'channel/' +
-      c.channel +
-      '">#' +
-      c.channel +
-      "</a>" +
-      '</span>';
+    return h('span.status',
+	     'Subscribed to channel ',
+	     h('a',
+	       { href: base + 'channel/' + c.channel },
+	       '#' + c.channel));
   else if (c.type == "channel" && !c.subscribed)
-    return '<span class="status">' +
-      'Unsubscribed from channel <a href="' + base + 'channel/' +
-      c.channel +
-      '">#' +
-      c.channel +
-      "</a>" +
-      '</span>';
+    return h('span.status',
+	     'Unsubscribed from channel ',
+	     h('a',
+	       { href: base + 'channel/' + c.channel },
+	       '#' + c.channel))
   else return renderDefault(c);
 }
 
 function renderPost(opts, c) {
   opts.mentions = {};
-  if (Array.isArray(c.mentions)) c.mentions.forEach(function (link) {
-    if (link && link.name && link.link) opts.mentions[link.name] = link.link;
-  });
-  return '<section>' + marked(String(c.text), opts.marked) + "</section>";
+  if (Array.isArray(c.mentions)) {
+      c.mentions.forEach(function (link) {
+	  if (link && link.name && link.link)
+	      opts.mentions[link.name] = link.link;
+      });
+  }
+  var s = h('section');
+  s.innerHTML = marked(String(c.text), opts.marked);
+  return s;
 }
 
 function renderDefault(c) {
-  return "<pre>" + JSON.stringify(c, 0, 2) + "</pre>";
+  return h('pre', JSON.stringify(c, 0, 2));
 }

From cf66c7398b3ea4b26de20ddc0cb1e983384b150a Mon Sep 17 00:00:00 2001
From: Angelo DiNardi <angelo@dinardi.name>
Date: Mon, 5 Jun 2017 22:54:16 -0700
Subject: [PATCH 4/7] Rename renderItem -> renderRssItem.

---
 index.js  | 4 ++--
 render.js | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/index.js b/index.js
index 3ed0bee..49e9729 100644
--- a/index.js
+++ b/index.js
@@ -17,7 +17,7 @@ var {
     wrapPage,
     renderThread,
     renderAbout,
-    renderItem,
+    renderRssItem,
     wrapRss,
     renderRssContent,
 } = require('./render');
@@ -110,7 +110,7 @@ exports.init = function (sbot, config) {
                   pull.filter( (msg) => {
                     return !!renderRssContent(defaultOpts, msg.value.content || {});
                   }),
-                  renderItem(defaultOpts), wrapRss(about.name, defaultOpts)
+                  renderRssItem(defaultOpts), wrapRss(about.name, defaultOpts)
                 );
               default:
                 return pull(
diff --git a/render.js b/render.js
index fb4ee37..b21878a 100644
--- a/render.js
+++ b/render.js
@@ -13,7 +13,7 @@ exports.renderEmoji = renderEmoji;
 exports.formatMsgs = formatMsgs;
 exports.renderThread = renderThread;
 exports.renderAbout = renderAbout;
-exports.renderItem = renderItem;
+exports.renderRssItem = renderRssItem;
 exports.wrapRss = wrapRss;
 exports.renderRssContent = renderRssContent;
 
@@ -88,7 +88,7 @@ function formatMsgs(id, ext, opts) {
     case "json":
       return wrapJSON();
     case "rss":
-      return pull(renderItem(opts), wrapRss(id, opts));
+      return pull(renderRssItem(opts), wrapRss(id, opts));
     default:
       return null;
   }
@@ -141,7 +141,7 @@ function renderThread(opts, showAllHTML = "") {
   );
 }
 
-function renderItem(opts) {
+function renderRssItem(opts) {
   return pull(
     pull.map(renderRss.bind(this, opts))
   );

From e042fb79339f605bb9bea4d8d2e0c4db84d8ec89 Mon Sep 17 00:00:00 2001
From: Anders Rune Jensen <arj03@protonmail.ch>
Date: Thu, 8 Jun 2017 20:34:36 +0200
Subject: [PATCH 5/7] Move show all html stuff to render

---
 index.js  | 15 +++++++++------
 render.js |  8 ++++++++
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/index.js b/index.js
index 7a6c38d..6db10b7 100644
--- a/index.js
+++ b/index.js
@@ -16,7 +16,8 @@ var {
     formatMsgs,
     wrapPage,
     renderThread,
-    renderAbout
+    renderAbout,
+    renderShowAll
 } = require('./render');
 
 var appHash = hash([fs.readFileSync(__filename)])
@@ -93,8 +94,7 @@ exports.init = function (sbot, config) {
   function serveFeed(req, res, feedId) {
       console.log("serving feed: " + feedId)
 
-      var showAll = req.url.endsWith("?showAll")
-      var showAllHTML = showAll ? '' : '<br/><a href="' + req.url + '?showAll">Show whole feed</a>'
+      var showAll = req.url.endsWith("?showAll");
 
       getAbout(feedId, function (err, about) {
           if (err) return respond(res, 500, err.stack || err)
@@ -112,7 +112,9 @@ exports.init = function (sbot, config) {
 		      paramap(addFollowAbout, 8),
 		      paramap(addVoteMessage, 8),
 		      paramap(addGitLinks, 8),
-		      pull(renderAbout(defaultOpts, about, showAllHTML), wrapPage(about.name)),
+		      pull(renderAbout(defaultOpts, about,
+				       renderShowAll(showAll, req.url)),
+			   wrapPage(about.name)),
 		      toPull(res, function (err) {
 			  if (err) console.error('[viewer]', err)
 		      })
@@ -196,7 +198,6 @@ exports.init = function (sbot, config) {
       console.log("serving channel: " + channelId)
 
       var showAll = req.url.endsWith("?showAll")
-      var showAllHTML = showAll ? '' : '<br/><a href="' + req.url + '?showAll">Show whole feed</a>'
       
       pull(
 	  sbot.query.read({ limit: showAll ? 300 : 10, reverse: true, query: [{$filter: { value: { content: { channel: channelId }}}}]}),
@@ -209,7 +210,9 @@ exports.init = function (sbot, config) {
 		  pull.values(logs),
 		  paramap(addAuthorAbout, 8),
 		  paramap(addVoteMessage, 8),
-		  pull(renderThread(defaultOpts, showAllHTML), wrapPage('#' + channelId)),
+		  pull(renderThread(defaultOpts,
+				    renderShowAll(showAll, req.url)),
+		       wrapPage('#' + channelId)),
 		  toPull(res, function (err) {
 		      if (err) console.error('[viewer]', err)
 		  })
diff --git a/render.js b/render.js
index c3edca8..ca4a466 100644
--- a/render.js
+++ b/render.js
@@ -14,6 +14,7 @@ exports.renderEmoji = renderEmoji;
 exports.formatMsgs = formatMsgs;
 exports.renderThread = renderThread;
 exports.renderAbout = renderAbout;
+exports.renderShowAll = renderShowAll;
 
 function MdRenderer(opts) {
   marked.Renderer.call(this, {});
@@ -445,3 +446,10 @@ function renderPost(opts, c) {
 function renderDefault(c) {
   return h('pre', JSON.stringify(c, 0, 2));
 }
+
+function renderShowAll(showAll, url) {
+    if (showAll)
+	return '';
+    else
+	return '<br>' + h('a', { href : url + '?showAll' }, 'Show whole feed').outerHTML;
+}

From 17be3231ecbae96676b6c7a35c14740d16ca6500 Mon Sep 17 00:00:00 2001
From: Angelo DiNardi <angelo@dinardi.name>
Date: Fri, 9 Jun 2017 12:56:34 -0700
Subject: [PATCH 6/7] Add some context in to the title.

---
 render.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/render.js b/render.js
index e604ddb..d2a20f8 100644
--- a/render.js
+++ b/render.js
@@ -441,7 +441,7 @@ function renderRss(opts, msg) {
 
   return (
     '<item>' +
-      '<title>' + msg.author.name + '</title>' +
+      '<title>' + msg.author.name + ' - ' + escape(content.substring(0, 25)) + '... </title>' +
       '<description><![CDATA[' + content + ']]></description>' +
       '<link>' + opts.base + escape(name) + '</link>' +
       '<pubDate>' + new Date(msg.value.timestamp).toUTCString() + '</pubDate>' +

From 4eee4ae72866afa2e09c9d82babd86152644119e Mon Sep 17 00:00:00 2001
From: Angelo DiNardi <angelo@dinardi.name>
Date: Fri, 9 Jun 2017 23:13:40 -0700
Subject: [PATCH 7/7] Stop doing special rendering for RSS items, use HTML!

I missed the part of the RSS docs that allow HTML in the description.
---
 index.js  |  3 ---
 render.js | 67 ++-----------------------------------------------------
 2 files changed, 2 insertions(+), 68 deletions(-)

diff --git a/index.js b/index.js
index e29f56b..eeabc4b 100644
--- a/index.js
+++ b/index.js
@@ -107,9 +107,6 @@ exports.init = function (sbot, config) {
               case 'rss':
                 return pull(
                   // formatMsgs(feedId, ext, defaultOpts)
-                  pull.filter( (msg) => {
-                    return !!renderRssContent(defaultOpts, msg.value.content || {});
-                  }),
                   renderRssItem(defaultOpts), wrapRss(about.name, defaultOpts)
                 );
               default:
diff --git a/render.js b/render.js
index d2a20f8..5bdf045 100644
--- a/render.js
+++ b/render.js
@@ -15,7 +15,6 @@ exports.renderThread = renderThread;
 exports.renderAbout = renderAbout;
 exports.renderRssItem = renderRssItem;
 exports.wrapRss = wrapRss;
-exports.renderRssContent = renderRssContent;
 
 function MdRenderer(opts) {
   marked.Renderer.call(this, {});
@@ -367,73 +366,11 @@ function renderMsg(opts, msg) {
   );
 }
 
-function renderRssContent(opts, c) {
-  var base = opts.base;
-  if (c.type === "post") {
-    return (c.channel ? `Posted in #${c.channel}:\n\n` : '') + c.text;
-  } else if (c.type == "vote" && c.vote.expression == "Dig") {
-    var linkedText = "a post";
-    if (typeof c.vote.linkedText != "undefined") {
-      linkedText = c.vote.linkedText.substring(0, 75);
-    }
-
-    return `Liked ${linkedText}`;
-  } else if (c.type == "vote") {
-    var linkedText = "a post";
-    if (typeof c.vote.linkedText != "undefined") {
-      linkedText = c.vote.linkedText.substring(0, 75);
-    }
-
-    return `Voted ${linkedText}`;
-  } else if (c.type == "contact" && c.following) {
-    var name = c.contact;
-    if (typeof c.contactAbout != "undefined") {
-      name = c.contactAbout.name;
-    }
-
-    return `Followed ${name}`;
-  } else if (c.type == "contact" && !c.following) {
-    var name = c.contact;
-    if (typeof c.contactAbout != "undefined") {
-      name = c.contactAbout.name;
-    }
-
-    return `Unfollowed ${name}`;
-  } else if (typeof c == "string") {
-    // Don't show private messages in RSS.
-  }
-  else if (c.type == "about") {
-    // Don't show about messages in RSS.
-  }
-  else if (c.type == "issue") {
-    return `Created a git issue ${(c.repoName != undefined ? " in repo " + c.repoName : "")}`;
-  }
-  else if (c.type == "git-update") {
-    return `Did a git update ${(c.repoName != undefined ? " in repo " + c.repoName : "")}
-    
-    ${(c.commits != undefined ? c.commits.map(com => { return "-" +com.title; }).join("\n") : "")}
-    `;
-  }
-  else if (c.type == "ssb-dns") {
-    // Don't show for RSS.
-  }
-  else if (c.type == "pub") {
-    return `Connected to the pub ${c.address.host}`;
-  }
-  else if (c.type == "channel" && c.subscribed) {
-    return `Subscribed to channel #${c.channel}`;
-  }
-  else if (c.type == "channel" && !c.subscribed) {
-    return `Unsubscribed from channel #${c.channel}`;
-  }
-
-}
-
 function renderRss(opts, msg) {
   var c = msg.value.content || {};
   var name = encodeURIComponent(msg.key);
 
-  let content = renderRssContent(opts, c);
+  let content = render(opts, c);
 
   if (!content) {
     return null;
@@ -441,7 +378,7 @@ function renderRss(opts, msg) {
 
   return (
     '<item>' +
-      '<title>' + msg.author.name + ' - ' + escape(content.substring(0, 25)) + '... </title>' +
+      '<title>' + msg.author.name + ' | ' + c.type + '</title>' +
       '<description><![CDATA[' + content + ']]></description>' +
       '<link>' + opts.base + escape(name) + '</link>' +
       '<pubDate>' + new Date(msg.value.timestamp).toUTCString() + '</pubDate>' +