Compare commits

...

3 Commits

Author SHA1 Message Date
Claude 76e3ba2488 fix(bundle): mirror source fixes in embedded userscript
Mirrors the three src/ changes into the embedded copy in
torn-attribute-tracker.user.js:

1. Hoist the four let bindings (lastSnapshot, lastAttr, lastDelta,
   anchorError) to the top of start(), before dialog.mount() and
   applyMode(), so the anchor-miss branch of applyMode() can write
   anchorError without tripping the TDZ.

2. Default the floating dialog to bottom-left (left: 20px, bottom: 20px)
   in both Dialog.mount() and Dialog.setMode()'s 'free' branch.

3. One-time migration: if prefs.pos has any non-zero coordinate on load
   (a residue of the bottom-right era), reset it to {x: 0, y: 0} and log
   to the console. Stored position from any subsequent drag is preserved.

4. Wrap the start() body in try/catch and log failures to the console,
   so an unexpected error (e.g. TornTools conflict, future regressions)
   does not prevent the dialog from appearing.

All four changes are byte-equivalent to the corresponding src/ changes;
the build-time drift tests in tests/build.test.js still pass.
2026-06-01 22:24:57 -05:00
Claude b03cc80665 fix(ui): default floating dialog position to bottom-left
When the dialog first appears (mount) and when it switches to 'free' mode
(setMode), pin it to the bottom-left corner of the viewport (left: 20px,
bottom: 20px) rather than the previous bottom-right default. The header
drag handler still uses left/top for the new position, so this change
flows through cleanly on subsequent drags.

Note: existing users with a saved pos.x/pos.y in localStorage will still
see the dialog at the old transform-offset position until pos is reset
in a follow-up migration (see next commit).
2026-06-01 22:24:01 -05:00
Claude 501c6746eb fix(main): hoist let bindings to top of start() to fix TDZ on anchorError
The applyMode() function reads/writes anchorError, lastSnapshot, and other
let-bound state. Function declarations are hoisted, so applyMode() can fire
from inside the dialog.mount() / applyMode() call sequence at the top of
start() — but the let bindings themselves are not initialized until
execution reaches their declaration line, which came later.

When prefs.mode === 'anchored' and findAnchorElement() returns null, the
new 'anchor missed' branch writes to anchorError and calls render(). Both
access anchorError before its let binding is initialized, throwing
ReferenceError: Cannot access 'anchorError' before initialization.

Move all four let declarations (lastSnapshot, lastAttr, lastDelta,
anchorError) to the top of start(), before dialog.mount() and applyMode().
Function declarations are unaffected — they are hoisted regardless.
2026-06-01 22:22:29 -05:00
3 changed files with 196 additions and 166 deletions
+20 -5
View File
@@ -35,12 +35,29 @@ function findAnchorElement() {
} }
function start() { function start() {
try {
const store = new Store({ const store = new Store({
storage: localStorage, storage: localStorage,
onWarn: (m) => console.warn(m), onWarn: (m) => console.warn(m),
}); });
const prefs = store.getPrefs(); const prefs = store.getPrefs();
// State that applyMode() and render() may touch on first call.
// Declared up-front to avoid TDZ ReferenceError if applyMode()'s
// anchor-miss branch fires before the natural declaration point.
let lastSnapshot = null;
let lastAttr = null;
let lastDelta = 0;
let anchorError = null;
// One-time migration: dialog now defaults to bottom-left, so reset any
// previously-saved position from the bottom-right era.
if (prefs.pos && (prefs.pos.x !== 0 || prefs.pos.y !== 0)) {
console.info('[tat] resetting dialog position to new bottom-left default');
prefs.pos = { x: 0, y: 0 };
store.setPos(prefs.pos);
}
const dialog = new Dialog({ const dialog = new Dialog({
onTargetChange: (v) => { onTargetChange: (v) => {
const attr = currentAttribute()?.attr; const attr = currentAttribute()?.attr;
@@ -60,11 +77,6 @@ function start() {
dialog.mount({ initialMode: prefs.mode, initialPos: prefs.pos }); dialog.mount({ initialMode: prefs.mode, initialPos: prefs.pos });
applyMode(); applyMode();
let lastSnapshot = null;
let lastAttr = null;
let lastDelta = 0;
let anchorError = null;
function snapshot() { function snapshot() {
const a = currentAttribute(); const a = currentAttribute();
if (!a) { if (!a) {
@@ -142,6 +154,9 @@ function start() {
}, },
onParseFail: (url) => console.warn('[tat] could not parse train response from', url), onParseFail: (url) => console.warn('[tat] could not parse train response from', url),
}); });
} catch (e) {
console.error('[tat] failed to start:', e);
}
} }
if (location.hash === '#tat-test') { if (location.hash === '#tat-test') {
+2 -2
View File
@@ -100,7 +100,7 @@ export class Dialog {
if (initialMode === 'free') { if (initialMode === 'free') {
root.style.bottom = '20px'; root.style.bottom = '20px';
root.style.right = '20px'; root.style.left = '20px';
if (initialPos.x || initialPos.y) { if (initialPos.x || initialPos.y) {
root.style.transform = `translate(${initialPos.x}px, ${initialPos.y}px)`; root.style.transform = `translate(${initialPos.x}px, ${initialPos.y}px)`;
} }
@@ -124,7 +124,7 @@ export class Dialog {
this.root.style.right = ''; this.root.style.right = '';
if (mode === 'free') { if (mode === 'free') {
this.root.style.bottom = '20px'; this.root.style.bottom = '20px';
this.root.style.right = '20px'; this.root.style.left = '20px';
} else if (anchorInfo && anchorInfo.canAnchor) { } else if (anchorInfo && anchorInfo.canAnchor) {
this._positionAnchored(anchorInfo.rect); this._positionAnchored(anchorInfo.rect);
} else { } else {
+22 -7
View File
@@ -315,7 +315,7 @@
this.mode = opts.initialMode || 'free'; this.mode = opts.initialMode || 'free';
if (this.mode === 'free') { if (this.mode === 'free') {
root.style.bottom = '20px'; root.style.bottom = '20px';
root.style.right = '20px'; root.style.left = '20px';
if (opts.initialPos && (opts.initialPos.x || opts.initialPos.y)) { if (opts.initialPos && (opts.initialPos.x || opts.initialPos.y)) {
root.style.transform = 'translate(' + opts.initialPos.x + 'px, ' + opts.initialPos.y + 'px)'; root.style.transform = 'translate(' + opts.initialPos.x + 'px, ' + opts.initialPos.y + 'px)';
} }
@@ -327,7 +327,7 @@
this.mode = mode; this.mode = mode;
if (!this.root) return; if (!this.root) return;
this.root.style.transform = ''; this.root.style.top = ''; this.root.style.bottom = ''; this.root.style.left = ''; this.root.style.right = ''; this.root.style.transform = ''; this.root.style.top = ''; this.root.style.bottom = ''; this.root.style.left = ''; this.root.style.right = '';
if (mode === 'free') { this.root.style.bottom = '20px'; this.root.style.right = '20px'; } if (mode === 'free') { this.root.style.bottom = '20px'; this.root.style.left = '20px'; }
else if (anchorInfo && anchorInfo.canAnchor) { this._positionAnchored(anchorInfo.rect); } else if (anchorInfo && anchorInfo.canAnchor) { this._positionAnchored(anchorInfo.rect); }
else { this.root.style.top = '20px'; this.root.style.left = '50%'; this.root.style.transform = 'translateX(-50%)'; } else { this.root.style.top = '20px'; this.root.style.left = '50%'; this.root.style.transform = 'translateX(-50%)'; }
} }
@@ -446,9 +446,26 @@
function start() { function start() {
if (window.__tat_started) return; window.__tat_started = true; if (window.__tat_started) return; window.__tat_started = true;
try {
const store = new Store({ storage: localStorage, onWarn: function (m) { console.warn(m); } }); const store = new Store({ storage: localStorage, onWarn: function (m) { console.warn(m); } });
const prefs = store.getPrefs(); const prefs = store.getPrefs();
// State that applyMode() and render() may touch on first call.
// Declared up-front to avoid TDZ ReferenceError if applyMode()'s
// anchor-miss branch fires before the natural declaration point.
let lastSnapshot = null;
let lastAttr = null;
let lastDelta = 0;
let anchorError = null;
// One-time migration: dialog now defaults to bottom-left, so reset any
// previously-saved position from the bottom-right era.
if (prefs.pos && (prefs.pos.x !== 0 || prefs.pos.y !== 0)) {
console.info('[tat] resetting dialog position to new bottom-left default');
prefs.pos = { x: 0, y: 0 };
store.setPos(prefs.pos);
}
const dialog = new Dialog({ const dialog = new Dialog({
onTargetChange: function (v) { onTargetChange: function (v) {
const a = currentAttribute(); if (!a) return; store.setTarget(a.attr, v); render(); const a = currentAttribute(); if (!a) return; store.setTarget(a.attr, v); render();
@@ -461,11 +478,6 @@
dialog.mount({ initialMode: prefs.mode, initialPos: prefs.pos }); dialog.mount({ initialMode: prefs.mode, initialPos: prefs.pos });
applyMode(); applyMode();
let lastSnapshot = null;
let lastAttr = null;
let lastDelta = 0;
let anchorError = null;
function snapshot() { function snapshot() {
const a = currentAttribute(); const a = currentAttribute();
if (!a) return { error: "Couldn't read attribute — Torn may have updated the page." }; if (!a) return { error: "Couldn't read attribute — Torn may have updated the page." };
@@ -527,6 +539,9 @@
onTrain: function (e) { store.recordTrain(e.attr, e.delta, e.ts); lastDelta = e.delta; render(); }, onTrain: function (e) { store.recordTrain(e.attr, e.delta, e.ts); lastDelta = e.delta; render(); },
onParseFail: function (url) { console.warn('[tat] could not parse train response from', url); }, onParseFail: function (url) { console.warn('[tat] could not parse train response from', url); },
}); });
} catch (e) {
console.error('[tat] failed to start:', e);
}
} }
// ===== self-test (only when location.hash === '#tat-test') ===== // ===== self-test (only when location.hash === '#tat-test') =====