chore: clean out some old/broken links

docs: update Synergy version support info

Just got an email from somebody who thought from the "and above" and "or
later" text that Synergy would work on later versions of macOS. That
text was written years ago (in some cases, over a decade ago), so it is
sorely out of date and wasn't written in a future-proof way. Update it
for clarity.

chore: remove references to ancient PEM certificates

chore: remove reference to ancient PGP key

chore: clean up Keybase references

Closes: https://github.com/wincent/masochist/issues/191

chore: clean up stray '";' strings

docs: update broken hextrapolate.wincent.com links

Seeing as the subdomain doesn't resolve any more, update to point at
hex.wincent.com instead.

chore: fix some more broken links

chore: fix some broken links

chore: ignore "next/" worktree

Don't want to see noise that results from:

    git worktree add next upstream/next

refactor: use `overflow-wrap: break-word` more extensively

Is this a good idea? This question was asked here:

    https://twitter.com/adamwathan/status/1317915868483956738

and the consensus seems to be it's a bad idea, but only for some
non-English languages, and all my content is in English.

This will stop pages like:

    https://wincent.com/wiki/macOS_%22must_haves%22

which have long URLs visible inside `a` tags from looking like crap on
mobile (where they overflow the width and make the page too wide,
creating a horizontal scroll).

chore: remove stale package from cache

refactor: tweak navbar background to be not so deeply black

Full black can be a bit too intense.

chore: switch license to MIT

Had only been using CC-BY-NC-SA-4 because Corporate Overlords at
previous job had recommended it for use in personal projects. Left there
some years ago, and MIT is a better fit in the JS ecosystem.

Closes: https://github.com/wincent/masochist/issues/181

feat: implement retry/reconnect on Redis client

At the moment I am just catching the two most obvious possible errors;
EPIPE and ECONNREFUSED, but the truth is there are others that might
conceivably occur — from:

    https://www.columbia.edu/sec/acis/db2/db2m0/db2tcp.htm

maybe ECONNRESET and ETIMEDOUT (even though Node sockets don't time out
by default) are obvious candidates... I might end up just doing a
blanket retry, too... not sure.

feat: add --trace-warnings to `yarn start` (in dev)

Useful for when I do things like leaking EventEmitters.

chore: remove express-graphql

I was only using it as a shortcut to getting GraphiQL in dev, but the
schema is basically "finished" at this point and I don't think I'm going
to need to troubleshoot it again (if I'm wrong about that, I can easily
revert this change). In the meantime, killing this is a small step
towards reducing the dependency footprint.

chore: remove stale comment

Originally added in 375562c7dfb92691e6b32954be1d47a4e8550cfd ("Unbreak
webpack build", Nov 23, 2018). The problematic syntax was stripped out
in 7765e44d81c543c191a9deb78705a53885c29151 ("chore: remove unused Flow
types", Dec 12, 2021).

chore: remove dead code

I initially thought I might use these, which correspond to the commands
that I actually use, but in reality, I only use `get()`, and all the
others get used via `multi()`.

Now, when I start rewriting stuff in TypeScript, there will be some
value in having named methods like this, although I will probably want
to expose them on `multi()` like this:

    multi()
      .zrange(...)
      .zcard(...)
      .exec();

as opposed to calling them individually (but having said that, I kind of
like the way the array calling convention mirrors the array result, so I
might see how far I can get with static types using that approach...)

refactor: prefer ZRANGE over ZREVRANGE

As noted here:

    https://redis.io/commands/zrevrange

> As per Redis 6.2.0, this command is considered **deprecated.** Please
> prefer using the ZRANGE command with the REV argument in new code.

(emphasis added)

chore: remove commented-out test code

fix: adjust readIndex call in Tag model to work with new client

So the old client returned an array with doubled rows, like:

   foo
   10
   bar
   5
   baz
   3
   ...

The new client returns proper arrays:

   [foo, 10]
   [bar, 5]
   [baz, 3]
   ...

I think the only place where this matters is in the tags index, because
it is the only place where we're using `WITHSCORES`.

feat: do a better job of logging interesting errors

Like the one fixed in the parent commit. That kind of bug would cause a
500 in the browser, but in the server's console out, nothing would be
logged.

fix: add back PropTypes import

I was moving quickly and not paying attention, but I _think_ the LSP
told me this was unused so I just deleted it. Or perhaps I blew it away
by accident while trying to delete the comment right above it.

Whatever. This was causing 500 errors.

fix: don't leak event listeners

Because we were setting up an `on('line')` listener for every command
we ran, we were leaking these quite badly. Wouldn't be a problem if
the clients were short-lived (because when the client is GC'd, the
emitter/listener would go away, because the emitter and the listeners
are all the same object), but the clients live for as long as the server
process runs.

chore: clean up additional Flow loose ends

Some stuff that didn't get covered in the last commit, which I made
with:

    yarn global add flow-remove-types
    flow-remove-types --pretty src -d lib
    cp -R lib/* src/
    yarn run pretty

Then I backed out the changes to `**/__generated__/*` files.

Now I've swept through, cleaning up empty comments, removing lint
suppressions etc, as well as converting the `scripts/` directory that I
had neglected, and removing some type-only imports.

chore: remove unused Flow types

I stopped using Flow a while ago, and have a pending ticket open for
porting to TypeScript[^1], and another in-progress rewrite anyway[^2]...
so for now these are mostly just noise (at best, you can consider them
as inline documentation that might go stale).

[^1]: https://github.com/wincent/masochist/issues/142
[^2]: https://github.com/wincent/masochist/issues/140 — although note
the ticket is nominally about updating the Relay version, which is way
behind, but which is tantamount to doing a full rewrite anyway because
so much is going to have to change.

refactor: switch `then()` to use `async`/`await` instead

Micro-refactoring that collapses 4 lines into 3 in a couple of places.

(Places, BTW, that I could DRY up by extracting out the common
functionality, but I don't won't to because the dumb and simple
duplicated version is easier to read.)

refactor: turn lines into a property

As all we ever do is pass it around unconditionally without changing it.
I originally had it as a param because I wasn't sure whether it was
going to be needed everywhere.

refactor: rename parseResponse() to parse()

I only had it under a different name as an intermediate step while
preparing the parent commit.

refactor: make redis client more robust

I tried a few things here but eventually "settled" on this design; that
is to say, I wanted to make the async handling a bit more
robust/idiomatic, so I explored using an async generator, then switched
to using an async iterator. The problem should fix is that when we are
pipelining a bunch of commands and executing them with MULTI + EXEC, we
may get several data payloads back with the responses. In the parent
commit, I could run the little test case multiple times, and my MULTI
request would sometimes get 4 `undefined` responses, or 1, or none.

I think I've fixed that now, and just need to add in error handling
everywhere, but we don't actually expect too many errors as Redis should
pretty reliably return exactly what I'm asking it for.

How does this work? Basically, we want to be able to `await` one line at
a time until we've got everything we need, hence the async iterator.
This ended up being a bit more useful than the generator because we can
pass it around (eg. to parse bulk strings in a subroutine) or into a
recursive call (which we need in order to parse maps, sets, arrays etc).

chore: update some "safe" packages

    yarn upgrade-interactive
    npx yarn-deduplicate yarn.lock
    yarn

After this, `yarn audit` reports:

    12 vulnerabilities found - Packages audited: 1166
    Severity: 2 Low | 4 Moderate | 6 High

refactor: cut over to WIP Redis client

As noted in parent commit, it has some problems, but it still works well
enough to render the blog index in the local development environment
(this is the only page I've tested so far, and I only tested it once, so
you _know_ this is good).

Split up big file into smaller files so that I can keep things tidy as I
slowly refactor this into something robust.

feat: add more stuff to WIP redis client

As noted from the TODOs inline, still lots of missing pieces, and it's
not totally robust. Oh, and also, it's ugly. But it is pretty much
"feature complete" at this point and just needs to be fixed up, cleaned
up, and redesigned a bit. Ha.

refactor: add WIP replacement Redis client

We only need to handle a tiny number of commands:

- GET

(used alone, and...)

- SET
- ZADD
- ZCARD
- ZINCRBY
- ZREM
- ZREMRANGEBYSCORE
- ZREVRANGE

(used in `multi` transactions).

So something crude like this is probably going to be enough. It is
definitely not going to be efficient, but the payloads we are talking
about here are miniscule, so it really won't matter.

Still to do:

- Implement error handling.
- Add `multi` support.
- Address missing features and "TODOs" noted inline (for example, we
  aren't bailing early if we get an incomplete response).
- Add reconnect.
- Maybe write some tests.

Adding this as a .txt file just to show that it's not intended for
actual execution yet, but I do want it in version control in case my
machine dies.

docs: s/master/main in README.md

docs: fix and clean-up some paths in README

Backticks because they look nicer when rendered, and "var" isn't true
any more — it's "srv".

chore: empty commit for redeploy

Our post-receive hook was dying trying to run Gulp:

    remote: $ node_modules/.bin/gulp build
    remote: error: aborting due to non-zero exit status (127)

I just changed it to run `node build.js` instead, and now need to retry
deploying, hence this empty commit.

refactor: replace Gulp with crude script

Script was coded up in haste so is probably riddled with bugs, but it
seems to do what Gulp did without the enormous dependency footprint.

`yarn audit` is a bit happier after this:

```
19 vulnerabilities found - Packages audited: 1167
Severity: 2 Low | 11 Moderate | 6 High
```

Previously, it was:

```
28 vulnerabilities found - Packages audited: 1416
Severity: 2 Low | 17 Moderate | 9 High
```

Most of the remaining issues are in the relay-compiler, of which I am
running a super-old version.

Closes: https://github.com/wincent/masochist/issues/177

fix: use correct name for license file

Instead of:

    [object Object].LICENSE.txt

generate something like:

    bundle-81f9425e983ef01950a9.mjs.LICENSE.txt

chore: remove unused Flow file

refactor: improve contrast in footer area

chore: clean up stray packages

Just by running `yarn`.

chore: run `npx yarn-deduplicate yarn.lock`

chore: back out redis "upgrade"

Because it's not worth it.

The `ZREVRANGE` command is not implemented, which breaks things for us.

Note that we can get the same with `ZRANGE` and passing a `REV` option:

    https://github.com/redis/node-redis/issues/1705

but that still leaves headaches elsewhere, where we were using
`ZREVRANGE` with the `WITHSCORES` option. This isn't supported by the
npm package, even though Redis itself supports it:

    https://redis.io/commands/zrange

The suggestion is to switch overt to `ZRANGEBYSCORE` instead, but that
is starting to sound awfully like a hassle.

refactor: switch multi invocation to v4 style

As documented here:

    https://github.com/redis/node-redis

But here's the interesting thing about it — it doesn't work for all
Redis commands. So, for example, we can hit a blog permalink page just
fine but not a page using a reverse-sorted index, like the blog index
page.

This shows up as a server crash:

    TypeError: Cannot read property 'posts' of null
        at PostsIndex.render (/Users/wincent/code/masochist/src/client/components/PostsIndex.js:64:35)

Dicking around in the console I established that the API works for some
commands but not others; for example, `ZREVRANGE` is missing, but
`ZCARD` is not:

    in loop ZREVRANGE masochist:6:blog-index [ 0, 2 ]
    command undefined
    in loop ZCARD masochist:6:blog-index []
    command [Function (anonymous)]

So, if I can't find a way to work around this I may just have to
"downgrade" (that is, "upgrade") to the previously working version of
the software.

Revert "chore: revert to state as of 447b2a875"

Now that I've got the production site back up by "rolling back" to the
prior rev, I'm doing this revert (to the bad rev) in the hope of
reproducing locally and finding out what's going on.

This reverts commit 43dee176285d111227a31634425188d91f4a29c2.

chore: revert to state as of 447b2a875

This reverts the last 13 commits because something isn't liking the
changes in prod, where the server serves up the top part of the document
and then nothing.

"Rolling back" in order to do some troubleshooting.

chore: bump graphql-relay v0.6.0 to v0.7.0

To get rid of the duplicate prettier version it was bringing in to the
graph.

chore: update Prettier

Although note that we still have an old duplicate copy lying around
because:

    => Found "graphql-relay#prettier@1.19.1"
    info This module exists because "graphql-relay" depends on it.

I don't know/recall why it has it as a non-dev dependency; all its other
deps are under `devDependencies`... looking, seems it is unintended and
was fixed in:

    https://github.com/graphql/graphql-relay-js/commit/0155159f2276fa4c2b194ddfefc85145d9329c2e

but there are too many breaking changes for me to feel all that
enthusiastic about moving from 0.6.0 to the current release (0.9.0):

    https://github.com/graphql/graphql-relay-js/releases

although admittedly, we could get that fix by going just to v0.7.0, I
think, which does look relatively safe. I might try that in the next
commit.

chore: update sinon package too

Another safe upgrade; `yarn test` still green.

If only getting rid of Gulp were so easy... *sigh*

chore: update some more (hopefully safe) packages

The redis update requires some shenanigans to get around breaking
changes:

    https://github.com/redis/node-redis/blob/HEAD/docs/v3-to-v4.md

chore: update webpack-dev-server

Note that I had to go through a few tweak-restart loops to flush out all
the problems:

    ValidationError: Invalid options object. Dev Server has been initialized using an options object that does not match the API schema.
     - options has an unknown property 'noInfo'. These properties are valid:

    ValidationError: Invalid options object. Dev Server has been initialized using an options object that does not match the API schema.
    - options has an unknown property 'publicPath'. These properties are valid:

    (node:46306) [DEP_WEBPACK_DEV_SERVER_CONSTRUCTOR] DeprecationWarning: Using 'compiler' as the first argument is deprecated. Please use 'options' as the first argument and 'compiler' as the second argument.

    (node:46332) [DEP_WEBPACK_DEV_SERVER_LISTEN] DeprecationWarning: 'listen' is deprecated. Please use async 'start' or 'startCallback' methods.

    <w> [webpack-dev-server] "hot: true" automatically applies HMR plugin, you don't have to add it manually to your webpack configuration.

chore: clean up lockfile and straggling packages

This is the diff produced by running just `yarn` alone.

chore: run `npx yarn-deduplicate yarn.lock`

chore: update more packages

I think this covers all of the "safe" ones.

chore: update some more webpack-ish deps

Most of these without a hitch, except for terser-webpack-plugin, which
was causing:

    - options has an unknown property 'sourceMap'. These properties are valid:
      object { test?, include?, exclude?, terserOptions?, extractComments?, parallel?, minify? }

So, I just removed the option. According to the changelog:

    https://github.com/webpack-contrib/terser-webpack-plugin/blob/master/CHANGELOG.md

in version v5.0.0 they dropped it:

> removed the `sourceMap` option (respect the `devtool` option by default)

refactor: use "Asset Modules" in lieu of url-loader

See: https://webpack.js.org/guides/asset-modules/

chore: update to webpack

This is the minimal set of changes specified in:

    https://webpack.js.org/migrate/5/

with tweaks to make `yarn start` and `yarn build` complete without
errors.

Note I had to get rid of the `!` chaining syntax, as noted here:

    Error: Compiling RuleSet failed: Exclamation mark separated
    loader lists has been removed in favor of the 'use'
    property with arrays (at ruleSet[1].rules[2].loader:
    style-loader!css-loader!postcss-loader)

`yarn build` spews out a lot of deprecation notices and other warnings
(edited for readability):

    (node:33331) [DEP_WEBPACK_MAIN_TEMPLATE_RENDER_MANIFEST] DeprecationWarning: MainTemplate.hooks.renderManifest is deprecated (use Compilation.hooks.renderManifest instead)
    (Use `node --trace-deprecation ...` to show where the warning was created)
    (node:33331) [DEP_WEBPACK_CHUNK_TEMPLATE_RENDER_MANIFEST] DeprecationWarning: ChunkTemplate.hooks.renderManifest is deprecated (use Compilation.hooks.renderManifest instead)
    (node:33331) [DEP_WEBPACK_MAIN_TEMPLATE_HASH_FOR_CHUNK] DeprecationWarning: MainTemplate.hooks.hashForChunk is deprecated (use JavascriptModulesPlugin.getCompilationHooks().chunkHash instead)
    (node:33331) [DEP_WEBPACK_COMPILATION_OPTIMIZE_CHUNK_ASSETS] DeprecationWarning: optimizeChunkAssets is deprecated (use Compilation.hooks.processAssets instead and use one of Compilation.PROCESS_ASSETS_STAGE_* as stage option)
    (node:33331) [DEP_WEBPACK_COMPILATION_NORMAL_MODULE_LOADER_HOOK] DeprecationWarning: Compilation.hooks.normalModuleLoader was moved to NormalModule.getCompilationHooks(compilation).loader
    [16:06:38] Finished 'graphql' after 2.18 s
    (node:33331) [DEP_WEBPACK_COMPILATION_ASSETS] DeprecationWarning: Compilation.assets will be frozen in future, all modifications are deprecated.
    BREAKING CHANGE: No more changes should happen to Compilation.assets after sealing the Compilation.
            Do changes to assets earlier, e. g. in Compilation.hooks.processAssets.
            Make sure to select an appropriate stage from Compilation.PROCESS_ASSETS_STAGE_*.
    (node:33331) [DEP_WEBPACK_MODULE_ID] DeprecationWarning: Module.id: Use new ChunkGraph API
    (node:33331) [DEP_WEBPACK_MODULE_UPDATE_HASH] DeprecationWarning: Module.updateHash: Use new ChunkGraph API
    (node:33331) [DEP_WEBPACK_CHUNK_MODULES_ITERABLE] DeprecationWarning: Chunk.modulesIterable: Use new ChunkGraph API
    (node:33331) [DEP_WEBPACK_CHUNK_GROUP_GET_MODULE_INDEX_2] DeprecationWarning: ChunkGroup.getModuleIndex2 was renamed to getModulePostOrderIndex
    (node:33331) [DEP_WEBPACK_COMPILATION_CACHE] DeprecationWarning: Compilation.cache was removed in favor of Compilation.getCache()

    WARNING in asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
    This can impact web performance.
    Assets:
      bundle-45d526449584b2234f04.js (525 KiB)

    WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
    Entrypoints:
      main (541 KiB)
          styles-a5a8105f24841352a5be.css
          bundle-45d526449584b2234f04.js

    WARNING in webpack performance recommendations:
    You can limit the size of your bundles by using import() or require.ensure to lazy load some parts of your application.
    For more info visit https://webpack.js.org/guides/code-splitting/

    WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
    Entrypoints:
      main (425 KiB)
          styles-a5a8105f24841352a5be.css
          bundle-8ff3e30a9a2750cb82ed.mjs

    WARNING in webpack performance recommendations:
    You can limit the size of your bundles by using import() or require.ensure to lazy load some parts of your application.
    For more info visit https://webpack.js.org/guides/code-splitting/

chore: update webpack to latest v4 version

Because this is the first step in the migration to v5:

    https://webpack.js.org/migrate/5/

`yarn start` still works after this, as does `yarn build`.

chore: update some CSS dependencies

To avoid this deprecation warning caused by a transitive dep,
es-abstract, as seen on the machine I'm currently on, which happens to
have NodeJS v16.8.0:

    (node:25719) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./" in the "exports" field module resolution of the package at $HOME/code/masochist/node_modules/es-abstract/package.json.
    Update this package.json to use a subpath pattern like "./*".
        at emitFolderMapDeprecation (node:internal/modules/esm/resolve:88:11)
        at packageExportsResolve (node:internal/modules/esm/resolve:648:7)
        at resolveExports (node:internal/modules/cjs/loader:482:36)
        at Function.Module._findPath (node:internal/modules/cjs/loader:522:31)
        at Function.Module._resolveFilename (node:internal/modules/cjs/loader:919:27)
        at Function.Module._load (node:internal/modules/cjs/loader:778:27)
        at Module.require (node:internal/modules/cjs/loader:1005:19)
        at require (node:internal/modules/cjs/helpers:94:18)
        at Object.<anonymous> ($HOME/code/masochist/node_modules/object.values/implementation.js:3:10)
        at Module._compile (node:internal/modules/cjs/loader:1101:14)

So, updated cssnano from 4.1.10 to 5.0.12, because that's the one that
was pulling in es-abstract. That, in turn, required a couple of other
updates. postcss got bumped to 8.4.4 (we still have some duplicate
divergent versions hiding away in various places though), as per `yarn
why postcss`:

    => Found "postcss@8.4.4"
    info Has been hoisted to "postcss"
    info This module exists because it's specified in "dependencies".
    => Found "css-loader#postcss@7.0.27"
    info This module exists because "css-loader" depends on it.
    => Found "autoprefixer#postcss@7.0.27"
    info This module exists because "autoprefixer" depends on it.
    => Found "icss-utils#postcss@7.0.27"
    info This module exists because "css-loader#icss-utils" depends on it.
    => Found "postcss-modules-local-by-default#postcss@7.0.27"
    info This module exists because "css-loader#postcss-modules-local-by-default" depends on it.
    => Found "postcss-modules-extract-imports#postcss@7.0.27"
    info This module exists because "css-loader#postcss-modules-extract-imports" depends on it.
    => Found "postcss-modules-scope#postcss@7.0.27"
    info This module exists because "css-loader#postcss-modules-scope" depends on it.
    => Found "postcss-modules-values#postcss@7.0.27"
    info This module exists because "css-loader#postcss-modules-values" depends on it.

and postcss-loader went from 3.0.0 to 4.2.0; I couldn't go all the way
to the latest (6.2.1), because that depends on webpack v5, and I'm on
v4:

    https://github.com/webpack-contrib/postcss-loader/blob/master/CHANGELOG.md

Other release notes:

    https://github.com/postcss/postcss/blob/main/CHANGELOG.md
    https://github.com/cssnano/cssnano/releases

I'll look at doing some more updates in separate commits, as these
things tend to be such a cluster that it is best to approach them
incrementally and check that things are still working along the way.

refactor: change overflow-wrap from "anywhere" to "break-word"

Because "anywhere" is considered an invalid value by Safari, which means
that long `<code>` spans in my blog posts look crap on iOS (even in
Chrome, which is really a thin skin over WebKit on iOS), causing
unwanted horizontal scrolling.

I think this should be adequate. Should work in Chrome and Firefox too.
As described in:

    https://developer.mozilla.org/en-US/docs/Web/CSS/overflow-wrap

the difference is as follows:

> **normal**
> Lines may only break at normal word break points (such as a space
> between two words).
>
> **anywhere**
> To prevent overflow, an otherwise unbreakable string of characters
> — like a long word or URL — may be broken at any point if
> there are no otherwise-acceptable break points in the line. No
> hyphenation character is inserted at the break point. Soft wrap
> opportunities introduced by the word break are considered when
> calculating min-content intrinsic sizes.
>
> **break-word**
> The same as the anywhere value, with normally unbreakable words
> allowed to be broken at arbitrary points if there are no otherwise
> acceptable break points in the line, but soft wrap opportunities
> introduced by the word break are NOT considered when calculating
> min-content intrinsic sizes.

Looking back over the history, I don't think I've tried this before.
See the output of `git log -Goverflow-wrap -p`, and specifically some
interesting commits like:

- 8d1b60d1aba6f21466e7562fc77aab37aec66516 ("fix: improve wrapping of
  code elements", Jul 9, 2021)
- 56bd364eb4e8cc4c33dfa70ef5f85dd051bf7fc9 ("fix: don't break words in
  <code> elements quite so aggressively", Jul 9, 2021)
- bf8fbaa6647aa2386ccfd0d9ea60cb3800a37df1 ("refactor: don't abuse
  <code> element for styling tag lozenges", Jul 9, 2021)

Closes: https://github.com/wincent/masochist/issues/171

chore: freshen yarn.lock

Just ran `yarn` to make sure it is in "canonical form".

chore: remove duplication from yarn.lock

With:

    npx yarn-deduplicate yarn.lock

Has the side-effect of improving the `yarn audit` count a bit:

    54 vulnerabilities found - Packages audited: 1581
    Severity: 3 Low | 31 Moderate | 20 High

chore: update jest 25.1.0 → 27.4.3

Busy work to make `yarn audit` less upset. But still upset.

Before:

    106 vulnerabilities found - Packages audited: 1592
    Severity: 3 Low | 77 Moderate | 26 High

After:

    65 vulnerabilities found - Packages audited: 1666
    Severity: 3 Low | 42 Moderate | 20 High

Note that I also needed to provide a `fail` function to avoid:

    ReferenceError: fail is not defined

fix: don't undo browser scroll-to-anchor actions

Seeing as I just added footnotes, I'm noticing the brokenness here.

chore: update React 16.13.0 → 17.0.2

Was hoping this would fix some bugs I'm seeing around how click events
are handled (specifically, jump to and back from footnotes is only
working on the first click, or when JS is disabled), but it didn't.
Doesn't hurt to update though, as far as I can tell, so may as well keep
the change.

refactor: start using CSS custom properties

The copy-pasted hex values have long been getting on my nerves. There
are still lots of hard-coded values to be migrated, but I just want to
port a few to serve as a reminder of the direction I want to head in.

feat: highlight footnote when it is active

feat: add support for footnotes

Now that GitHub supports footnotes, I find myself in the habit of using
them there pretty often and therefore wanting to use them here too.

I am not sure if I am fully sold on the design here, but I'm passing in
a DOM identifier so that the `id` attributes in the generated markup can
be unique. One question, for example, is should this field be nullable?
Another is is "DOMIdentifier" the right name? (I wanted to avoid
confusion with GraphQL "id" fields.) Should I be doing more with these
IDs, like using them on title anchor tags? (probably not). Should I
tighten up the schema documentation, where it misleadingly talks of
"prefix"? (As you can see from the link below, the "DOMIdentifier" ends
up being embedded somewhere in the middle of an ID.)

Anyway, all this is possible by setting `docId`, which you can see being
used to construct the anchor names here:

    https://github.com/markdown-it/markdown-it-footnote/blob/337dcc79dfe8b2cb98c32000d8b24691de66105b/index.js#L13

Possible follow-ups: want to check the contrast levels for accessibility
because I eye-balled these and they are probably horrible. May in
general want to do a pass on the styles to make sure they look as pretty
as they should be.

docs: drop references to `npm`

Just to reduce clutter. People can probably figure out that wherever
they see `yarn` they could probably use `npm` instead.

docs: provide suggestion for how to install prerequisites

docs: remove stale line from README

fix: repair some mangled redirects

fix: use "bash" not "shell" for "sh" markup

Turns out that "bash" is for scripts and "shell" is for "shell
sessions".

So you want stuff like this for bash:

    ```bash
    echo "hi"
    ```

And stuff like this for sessions:

    ```shell
    $ sudo -s
    Password:
    # ls
    foo
    ```

Supposedly, "zsh" and "sh" are aliases for "bash", but in my testing, I
can't really see what "sh" actually does.

refactor: avoid highlight.js deprecation warning

    Deprecated as of 10.7.0. highlight(lang, code, ...args) has been deprecated.
    Deprecated as of 10.7.0. Please use highlight(code, options) instead.
    https://github.com/highlightjs/highlight.js/issues/2277

refactor: don't abuse <code> element for styling tag lozenges

chore: update highlight.js

A scary update, but I actually think it's going to be ok:

- https://github.com/highlightjs/highlight.js/releases/tag/10.0.0
- https://github.com/highlightjs/highlight.js/releases/tag/11.0.0
- https://github.com/highlightjs/highlight.js/blob/main/VERSION_11_UPGRADE.md

chore: remove unused extract-text-webpack-plugin

Unused since ba7fdb914c16bca7e0ce062 ("Make ES modules bundle as well",
Dec 16, 2018) and deprecated to boot:

    https://github.com/webpack-contrib/extract-text-webpack-plugin

chore: clean lockfile

By running `yarn`. It always seems to want to make changes after
`yarn-deduplicate` has run. As you can see, it also removed a bunch of
dead(?) pages from the cache.

chore: deduplicate yarn.lock

With `npx yarn-deduplicate yarn.lock`.

refactor: relax version ranges in package.json

chore: update markdown-it

fix: don't break words in <code> elements quite so aggressively

The recent addition of `overflow-wrap: anywhere` was breaking my lovely
tag lozenges, which probably shouldn't be using `<code>` anyway because
they aren't code...

fix: remove unwanted bold styling

Whoops, like I said in my last commit:

> Also experimented with bold but it is too heavyweight for my taste.

Somehow I managed to commit the unwanted change though...

fix: improve wrapping of code elements

Chrome seems pretty flaky with `box-decoration-break`. ie. in order to
see the difference between "clone" and "slice" I need to toggle the
border on and off again. Whatever; I think the wrapped spans look better
without borders or rounded corners anyway, so removing those. Also
experimented with bold but it is too heavyweight for my taste.

refactor: point at new location for content repo

fix: update npm packages

On new machine, we have a newer NodeJS, so we're running into this:

    Error: Cannot find module '$path_to/@babel/compat-data/data/corejs3-shipped-proposals'

Which I think is the issue described here:

    https://github.com/nodejs/node/issues/32852

and fixed here:

    https://github.com/babel/babel/pull/11283

I just ran `yarn upgrade-interactive` and updated all the packages which
looked safe-ish.

chore: freshen stale vendor/yarn-cache

Not sure how this got stale, but it is. On trying to deploy to a new EC2
machine, got:

    error Can't make a request in offline mode
    ("https://registry.yarnpkg.com/lodash.memoize/-/lodash.memoize-4.1.2.tgz")

So, freshening the cache by running `yarn`.

fix: remove bad index.php

Whoops. I had this formerly harmless file in here, but when we added
`index.php` to our nginx `try_files` directive, it started getting
served up to visitors hitting the root. :facepalm:

So let's kill it, and if anybody goes to https://wincent.com/index.php
they'll just get a 404, which is fine.

fix: mend some broken image resources

style: prettify static HTML files

With:

    npx prettier --write "**/*.{html,php}" --parser html

refactor: clean up links

Motivation is to avoid broken images caused by CSP violations (eg.
images hosted on www from non-www pages), but there are too many files
to do this in a non-automated way so this is just the dumbest of dumb
changes:

    find ./ -type f -exec gsed -i -e 's/www.wincent.com/wincent.com/g' {} \;
    find . -type f -exec gsed -i -e 's/secure.wincent.com/wincent.com/g' {} \;
    find . -type f -exec gsed -i -e 's#http://wincent.com#https://wincent.com#g' {} \;

chore: add some static files from old hosts

I want to "decommission" the hosts running at www.wincent.com and
secure.wincent.com without breaking any links, if I can help it. They
contain a bunch of PHP pages that do some very simple stuff like
composing a header and a footer and so on. There are only two (basically
dead) pages that handle POST requests.

So, I made this simple Ruby script that runs on the hosts and `curl`s
the pages (and other resources), taking a snapshot of everything. I
normally don't like leaking internal paths like this, but like I said,
the machines are going off the net forever shortly, so it's ok.

The directories that are guarded by .htaccess files all return 403s as
intended, so I haven't included those files in the dump.

As follow up I am going to have to tweak DNS records and SSL
certificates (SAN entries). But then I'll be able to kill off the old
servers.

    require 'fileutils'
    require 'shellwords'

    def escape(word)
      Shellwords.shellescape word
    end

    # Note: we don't have to worry about simlinks over here;
    # both directories contain only directories and regular files
    #
    #       find /var/www/secure.wincent.com/public_html -type l
    #       find /var/www/www.wincent.com/public_html -type l
    #
    %w[
      /var/www/secure.wincent.com/public_html
      /var/www/www.wincent.com/public_html
    ].each do |base|
      puts "Scraping from: #{base}"

      # Note that `Dir.[]` won't return dot-files, so we miss out on:
      #
      #       $ find /var/www/www.wincent.com/public_html -name '.*'
      #       /var/www/www.wincent.com/public_html/.well-known
      #
      # and:
      #
      #       $ find /var/www/secure.wincent.com/public_html -name '.*'
      #       /var/www/secure.wincent.com/public_html/a/products/install/licensees/binary-only/.htaccess
      #       /var/www/secure.wincent.com/public_html/a/products/install/licensees/full-source/.htaccess
      #       /var/www/secure.wincent.com/public_html/a/products/install/licensees/binary+framework/.htaccess
      #       /var/www/secure.wincent.com/public_html/a/products/install/licensees/.htaccess
      #       /var/www/secure.wincent.com/public_html/codegen/.htaccess
      #       /var/www/secure.wincent.com/public_html/.well-known
      #
      Dir["#{base}/**/*"].each do |source|
        host = "#{source}".split(%r{/})[3]
        path = "#{source}".split(%r{/})[5..-1].join('/')
        destination = File.join(__dir__, 'output', source)
        if File.file? source
          if system "curl --silent https://#{host}/#{escape path} -o #{escape destination}"
            puts " OK : #{host}/#{path}"
          else
            puts "FAIL: #{host}/#{path}"
          end
        else
          FileUtils.mkdir_p destination
        end
      end
    end

feat: add direct link to "About" page from site footer

feat: dim images a bit in dark mode

feat: add dark mode

Just a primitive first pass at this.

fix: degrade "gracefully" for undefined Search result

I don't have time now to dig into why we're getting `undefined` here,
but I can at least stop the app from crashing.

    TypeError: Cannot destructure property `search` of 'undefined' or 'null'.

refactor: update Vim link in footer

perf: add `loading="lazy"` attribute to images

See: https://web.dev/native-lazy-loading/