Deploy 58acaca

The bench header names the construct, not the prompt

Source: https://github.com/WINTER4000/turingDNA/commit/58acacaad14a5c8f4ad07cddf1b41699cd81d8c9

dee/static/app.js

@@ -9847,6 +9847,29 @@ function runOracle(opts){
     function setMeta() {
         metaEl.textContent = `${plasmid.length.toLocaleString()} bp · ${plasmid.topology} · GC ${plasmid.gc}%`
             + ((plasmid.features || []).length ? ` · ${plasmid.features.length} features` : '');
+        syncBenchSpecimen();
+    }
+
+    /* The bench header names what is on the bench.
+
+       It did not. Opening the bench from a live run labelled the specimen chip
+       with the RUN's title — the sentence the scientist had just typed — and
+       the sub-line with the literal words "live run", so a 19,895 bp pCambia
+       showed up in its own header as "add a cds of fugfp upstream LUX-G and
+       annotate it". The construct's identity was on screen twice (the map
+       centre, the name field) and wrong in the one place that is supposed to
+       be the answer to "what am I working on".
+
+       Called from setMeta, which every render path already goes through, so
+       the header tracks tab switches and edits without another hook. */
+    function syncBenchSpecimen() {
+        if (!plasmid) return;
+        if (document.body.getAttribute('data-bench') !== 'open') return;
+        if (!window.TDBench || !window.TDBench.update) return;
+        window.TDBench.update({
+            name: plasmid.name || 'Construct',
+            sub: `${(plasmid.length || 0).toLocaleString()} bp · ${plasmid.topology || ''}`.trim(),
+        });
     }
 
     function _renderPlasmidMeta() {
@@ -10988,6 +11011,14 @@ function runOracle(opts){
     /* What Turing reads and drives. The agent could already paint a construct
        but had no idea a second one was open, so it could not answer "switch to
        the other vector" — the most ordinary thing to ask once tabs exist. */
+    /* The identity of what the editor is holding, for whoever has to name it
+       before a render has happened — the Cockpit opens the bench from a tool
+       result and needs the construct, not the conversation, in the header. */
+    window.TDPlasmid.construct = function () {
+        if (!plasmid) return null;
+        return { name: plasmid.name || '', length: plasmid.length || 0,
+                 topology: plasmid.topology || '' };
+    };
     window.TDPlasmid.tabs = function () {
         return plasmidTabs ? plasmidTabs.list() : [];
     };

dee/static/cockpit.js

@@ -1729,8 +1729,23 @@
         try {
             if (window.TDBench && window.TDBench.openConstruct &&
                 document.body.getAttribute("data-bench") !== "open") {
+                /* Name the bench after the CONSTRUCT, not the sentence that
+                   started the run. This used to pass currentTitle() — the
+                   user's own prompt — plus the literal sub-line "live run",
+                   so opening the bench off a plasmid tool put "add a cds of
+                   fugfp upstream LUX-G and annotate it" in the header of a
+                   19,895 bp pCambia. The run's title belongs to the run; it
+                   is already on the transcript. */
+                var c = null;
+                try {
+                    c = (window.TDPlasmid && window.TDPlasmid.construct)
+                        ? window.TDPlasmid.construct() : null;
+                } catch (e) { c = null; }
                 window.TDBench.openConstruct({
-                    name: currentTitle() || "Construct", sub: "live run",
+                    name: (c && c.name) || currentTitle() || "Construct",
+                    sub: c ? (Number(c.length || 0).toLocaleString() + " bp · "
+                              + c.topology).trim()
+                           : "live run",
                     phaseIdx: 0, route: view,
                 });
                 return;

dee/static/index.html

@@ -112,7 +112,7 @@
     <!-- ?v= query bumps invalidate browser + iframe asset caches when app.css /
          app.js change. Bump these numbers whenever you ship a frontend update —
          without them, users keep getting the stale file for up to a week. -->
-    <link rel="stylesheet" href="/static/app.css?v=20260828-anchor" />
+    <link rel="stylesheet" href="/static/app.css?v=20260829-specimen" />
     <!-- The work catalog + the draggable rail. Kept out of app.css so two new
          self-contained surfaces stay reviewable; every colour is an app.css
          token, so both themes work with nothing added. -->
@@ -3140,12 +3140,12 @@
     <!-- Cloning reference data must load before app.js so the Designer
          can read VECTORS / ENZYMES / CLONING_METHODS / TAGS / LINKERS. -->
     <script src="/static/cloning_db.js?v=20260530-ui-polish" defer></script>
-    <script src="/static/context.js?v=20260828-anchor" defer></script>
-    <script src="/static/lineage.js?v=20260828-anchor" defer></script>
+    <script src="/static/context.js?v=20260829-specimen" defer></script>
+    <script src="/static/lineage.js?v=20260829-specimen" defer></script>
     <!-- Before app.js: it mounts a strip at init. Both are `defer`, so they
          run in document order. -->
-    <script src="/static/tabs.js?v=20260828-anchor" defer></script>
-    <script src="/static/app.js?v=20260828-anchor" defer></script>
+    <script src="/static/tabs.js?v=20260829-specimen" defer></script>
+    <script src="/static/app.js?v=20260829-specimen" defer></script>
     <!-- The decision trace, BEFORE cockpit.js: applyEvent calls TDTrace.push
          on the very first event, and both are `defer`, so document order is
          load order. Loading it after would drop the opening events of a
@@ -3154,7 +3154,7 @@
     <!-- THE COCKPIT — the persistent orchestrator rail. Loads after app.js so
          TDBench/TDStructure exist when a tool result asks the workspace to
          render something. This is the only conversation surface in the app. -->
-    <script src="/static/cockpit.js?v=20260828-anchor" defer></script>
+    <script src="/static/cockpit.js?v=20260829-specimen" defer></script>
     <!-- structcard before catalog: the catalog calls TDStructCard.observe as
          soon as it paints. Both are defer, so document order is load order. -->
     <script src="/static/structcard.js?v=20260811-buildC" defer></script>

tests/test_bench_header.py

@@ -0,0 +1,65 @@
+"""The bench header names the construct, not the sentence that started the run.
+
+A wiring assertion because the failure is silent and cosmetic-looking: the
+header still renders, still styled, just describing the wrong thing. Opening
+the bench from a live run labelled the specimen chip with the RUN's title, so
+a 19,895 bp pCambia announced itself as "add a cds of fugfp upstream LUX-G
+and annotate it" with "live run" underneath — while its real name sat in the
+map centre two inches below.
+"""
+import re
+
+_APP_JS = "dee/static/app.js"
+_COCKPIT_JS = "dee/static/cockpit.js"
+_INDEX = "dee/static/index.html"
+
+
+def _read(p):
+    with open(p, encoding="utf-8") as fh:
+        return fh.read()
+
+
+def test_the_cockpit_opens_the_bench_with_the_construct_not_the_prompt():
+    js = _read(_COCKPIT_JS)
+    m = re.search(r"function gotoView\(view\)\s*\{(.*?)\n    \}", js, re.S)
+    assert m, "gotoView moved — re-point this guard"
+    body = m.group(1)
+    assert "TDPlasmid.construct" in body, \
+        "gotoView must ask the editor what is loaded before naming the bench"
+    # the run's own title is the FALLBACK, never the first choice
+    name_line = re.search(r"name:\s*(.+)", body).group(1)
+    assert name_line.index("c.name") < name_line.index("currentTitle"), \
+        "the construct's name must win over the run title"
+    assert '"live run"' in body and "c ?" in body, \
+        "'live run' must only be the no-construct fallback"
+
+
+def test_the_editor_exposes_what_it_is_holding():
+    js = _read(_APP_JS)
+    assert re.search(r"window\.TDPlasmid\.construct\s*=\s*function", js), \
+        "the Cockpit reads this to label the bench before any render runs"
+
+
+def test_every_render_path_refreshes_the_header():
+    """setMeta is the choke point every render already goes through, so the
+    header follows tab switches and edits without a second hook."""
+    js = _read(_APP_JS)
+    m = re.search(r"function setMeta\(\)\s*\{(.*?)\n    \}", js, re.S)
+    assert m, "setMeta moved — re-point this guard"
+    assert "syncBenchSpecimen()" in m.group(1)
+    sync = re.search(r"function syncBenchSpecimen\(\)\s*\{(.*?)\n    \}", js, re.S)
+    assert sync, "syncBenchSpecimen missing"
+    b = sync.group(1)
+    assert "plasmid.name" in b and "plasmid.length" in b and "plasmid.topology" in b
+    assert 'data-bench' in b, "must not touch the header outside bench mode"
+
+
+def test_the_cache_bust_token_moved_with_the_js():
+    """/static/* is served max-age=31536000; the ?v= token is the ONLY
+    invalidation. Shipping app.js without bumping it ships nothing."""
+    html = _read(_INDEX)
+    for asset in ("app.js", "cockpit.js"):
+        m = re.search(re.escape("/static/" + asset) + r"\?v=([0-9A-Za-z-]+)", html)
+        assert m, asset + " lost its cache-bust token"
+        assert m.group(1) != "20260828-anchor", \
+            asset + " still carries the pre-fix token"