From c830b9ad6bd2cde576eada06dcad1fd5a520dd07 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Sun, 27 Sep 2026 10:24:15 +0000
Subject: [PATCH] Daily dead-code sweep 2026-09-27: the #chips compatibility
 shim, its expiry met

#85 (commit 867c4d4, 2026-09-02) deleted the duration-chip radiogroup and left
an empty `<div id="chips" hidden>` in index.html so that any client still
running the pre-#85 app.js (cached by the service worker, which serves
navigations network-first but everything else cache-first) would not hit a
TypeError on its first `attachChips()` call after the deploy. The comment
that introduced it says plainly when it can go: "Once a release has shipped
with it, no cached app.js reaches for #chips any more and this can go."

25 days and 64 commits (dozens of merged, deployed PRs) have shipped since
that commit, so the shim has done its one job many times over. Removed the
div and its comment, and the scripts/test/screens.test.mjs test that pinned
its presence and wording. The sibling test asserting no #chips CSS rule or
JS reference exists is untouched and still passes.

index.html shrank by ~620 gzipped bytes; re-measured and updated sw.js's
header comment and re-ran scripts/stamp-sw.mjs accordingly.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RUtU2jqDkfDxwyLjYkHHNK
---
 index.html                    | 18 ------------------
 scripts/test/screens.test.mjs | 19 -------------------
 sw.js                         |  6 +++---
 3 files changed, 3 insertions(+), 40 deletions(-)

diff --git a/index.html b/index.html
index 5f89944..152326e 100644
--- a/index.html
+++ b/index.html
@@ -1103,24 +1103,6 @@ <h2 id="gate-h" tabindex="-1"></h2>
   <section id="near" class="pane" hidden></section>
   <section id="pick" class="pane" hidden></section>
   <section id="about" class="pane" hidden></section>
-  <!-- REMOVE AFTER ONE RELEASE. #85 deleted the duration chips, and this empty
-       node is the only thing standing between that deletion and a frozen app
-       for everyone who already has the site installed.
-       sw.js serves navigations network-first and everything else cache-first,
-       so the first load after this deploys pairs the NEW index.html with the
-       OLD js/app.js still in the shell cache. That app.js calls attachChips()
-       as the second statement of its DOMContentLoaded listener, and its first
-       line is $('chips').querySelectorAll('.chip') against getElementById.
-       Without a node here that is a TypeError, and it lands before
-       $('back').onclick, popstate, attachSheet() and boot() are wired: the
-       screen holds "finding campus..." over four disabled buttons with no back
-       arrow and no error. Re-stamping SHELL_CACHE does not save that load,
-       because the old worker is the one serving it, and js/pwa.js only
-       auto-reloads while the page is untouched - which a frozen screen will not
-       stay. An empty div costs 30 bytes and makes the old code's forEach run
-       over nothing instead. Once a release has shipped with it, no cached
-       app.js reaches for #chips any more and this can go. -->
-  <div id="chips" hidden></div>
   <form id="find" hidden>
     <input id="find-q" type="search" autocomplete="off" autocorrect="off" spellcheck="false"
       enterkeyhint="search" placeholder="Search buildings" aria-label="Search buildings by name or code">
diff --git a/scripts/test/screens.test.mjs b/scripts/test/screens.test.mjs
index 2f8e508..6f44b0c 100644
--- a/scripts/test/screens.test.mjs
+++ b/scripts/test/screens.test.mjs
@@ -1265,25 +1265,6 @@ test('the duration chips are gone from the page, not merely hidden on it', () =>
   assert.doesNotMatch(APP, /paintChips|attachChips/, 'the chip handlers are still here');
 });
 
-test('the #chips node left behind for the old shell is empty, and dated', () => {
-  const html = readFileSync(join(ROOT, 'index.html'), 'utf8');
-  // sw.js serves navigations network-first and assets cache-first, so the first
-  // load after this deploys runs the new index.html against the js/app.js still
-  // in the shell cache. That app.js calls attachChips() before it wires #back,
-  // popstate, the sheet or boot(), and its first line is
-  // $('chips').querySelectorAll('.chip') — a TypeError on a missing node, and a
-  // screen frozen on "finding campus..." with no way out. This node exists only
-  // so that forEach runs over nothing instead.
-  const node = html.match(/<div id="chips"[^>]*>([\s\S]*?)<\/div>/);
-  assert.ok(node, 'the shim the old shell needs is gone; see #85 before removing it');
-  assert.equal(node[1].trim(), '', 'the shim grew content, which makes it a control again');
-  assert.match(node[0], /\bhidden\b/, 'the shim is not hidden');
-  // It is scaffolding with an expiry, not a feature. Once a release has shipped
-  // carrying it, no cached app.js reaches for #chips and the node can go.
-  const why = html.slice(Math.max(0, html.indexOf('<div id="chips"') - 1400), html.indexOf('<div id="chips"'));
-  assert.match(why, /REMOVE AFTER ONE RELEASE/, 'the shim lost the note saying it is temporary');
-});
-
 test('the ranked list says which question it is answering', () => {
   const paint = bodyOf('paintList');
   // In the concatenation, so it cannot be a line that renders somewhere else.
diff --git a/sw.js b/sw.js
index eeea51d..703094c 100644
--- a/sw.js
+++ b/sw.js
@@ -7,8 +7,8 @@
 // One cache would either re-download the room index on every deploy or pin an
 // installed icon to last month's app.js forever.
 //
-// Measured on 2026-09-16 over the committed blobs, which is the copy Pages serves:
-// `git show HEAD:<file> | gzip -9 -c | wc -c`. Shell 164,286 bytes, data 115,160.
+// Measured on 2026-09-27 over the committed blobs, which is the copy Pages serves:
+// `git show HEAD:<file> | gzip -9 -c | wc -c`. Shell 163,666 bytes, data 115,160.
 // Run it exactly as written, through the pipe. `gzip -9 -c <file>` with the
 // name as an argument stores each basename in the gzip FNAME header and reads
 // 176 bytes higher across these seventeen files, which is most of a percent of
@@ -88,7 +88,7 @@
 // placeholder is __BUILD_ID__, and a committed sw.js still carrying it means the
 // stamp did not run. scripts/test/sw.test.mjs fails on exactly that. Spelled out
 // rather than built from CACHE_PREFIX, because the stamper rewrites this line.
-const SHELL_CACHE = 'vacant-shell-c344401';
+const SHELL_CACHE = 'vacant-shell-a1ede09';
 const DATA_CACHE = 'vacant-data-v1';
 
 // CacheStorage is per origin, not per path, and enesyilmazcode.github.io also