Link tooltip clips long addresses and covers the link (production) #228
Closed
opened 2026-10-16 05:38:17 +00:00 by kayg
·
46 comments
No Branch/Tag specified
main
j470b-server-mesh-rejoin
mesh-compat-382
j444b-d381-key-wins-flood
j461b-macos-utun-reader-thread
j475b-remove-host-motion
fix-reconnect-new-link
j481-d380
dev
j479-mesh-soak
fix/j476-host-facts
j475-motion-pill-split
j470-mac-node-join
root-upgrade-d376
j474-review-merge
latency-targets-120hz
nightly-suite-shards
gotatun-pin-j469
macos-utun-reader-thread
fix-flood-echo-stall
wip/j416
motion-fast-ease-out
mesh-addhost-j259
takeover-close-terminals-d368
kbd-instant-followups
fix-trust-key-double-click
nightly-suite-partial-report
appwin-icons-j207
rusqlite-slice1
swipe-content-first-d374
mesh-mtu-381
addhost-instant-open
glass-contrast-d365-mock
glass-contrast-d365
signout-followups-442
fix-startup-toast-red
research-gotatun-matrix-j317
themes-sync-j304
fix-load-flakes
fix-nix-zig-patch
fix-compute01-browser-baseline
ratchet-size-baseline-419
predict-auto-d130
revoke-coverage-d367
nightly-suite-queue-timeout
nightly-browser-suite
fix-rustsec-2026-0459
fonts-ioskeley-obel
log-retention-d375
fix-kbd-no-anim-427
fix-dev-red-specs-2
takeover-proof-305-j205
wip/j410-server-pump
ui-consistency-d372
fix-osc52-clipboard
libghostty-vt-1.0
forget-retired-host-d369
j404-merge-426
fix-setup-lock-race
fix-touch-select-426
revoke-access-report-ui
fix-predictive-echo-specs
fix-selinux-agent-exec
wip/j395-checkpoint2
wip/j395
j391-review-scratch
ui-consistency-j195-deferred-rebased
fix-rustc-paths-421
decisions-wording-341
fix-install-suite-red
fix-dev-rtt-specs
fix-pending-removal-align
fix-privileged-skip
fix-install-suite-404
review-j368
mesh-dns-j254
revoke-access-report-d356b
revoke-privilege-round5
fix-removal-confirm
review-j360
gotatun-rotation-367
deps-refresh-340
review-j354
review-J351
review-j350
addhost-takeover-d359
revoke-report-items-347
revoke-end-sessions-d362
review-j348
size-mesh-ssh-346
size-trim-337
revoke-access-diff-d356
j259-pre-rebase-b06c5a6
j305-ended-at-start-spec
addhost-sudo-consent
fix-layout-real-specs-411
fix-shell-open-411
fix-console-hosts-373
fix-shell-exit
fuzz-host-name-353
fix-host-details-410
macos-filter-j197
research-gotatun-tricks
revoke-p3-j216l
stale-state-pill
revoke-followups-j216k
revoke-progress-j216e
backup/macos01-20261106/lume/j197
backup/macos01-20261106/lume/appwin-icons-j207-stash0
backup/macos01-20261106/claude/stash-0
backup/macos01-20261106/claude/stash-1
backup/macos01-20261106/claude/J80-macos-s1
backup/macos01-20261106/claude/J80
backup/macos01-20261106/claude/J80-slice1-20261102
backup/macos01-20261106/claude/J16b-appwin-on-dev
backup/macos01-20261106/claude/appwin-launch-idle-146
backup/macos01-20261106/claude/J80-s1-run
wip/j259
trust-285-p3c
runtime-lock-j216f
mesh-ssh-p3-j175
tun-proof-p3b
trust-285-p3
terminal-polish-j271
tun-proof-p3
ssh-install-flakes-j216i
fuzz-fixes-j264
trust-285-a
trust-285-b
restart-oversized-fixture-j198
backup/macos01-20261106/lume/appwin-icons-j207
restart-bound-j216h
up-preflight-p3-j198
backup/macos01-20261106/lume/j251
up-preflight-j198
wip/up-preflight-j198-j245-p2
wip/up-preflight-j198-j245-p1
agent-fsync-offloop-j196
wip/up-preflight-j198-j245
settings-offscreen-j231
wip/up-preflight-j198-j226-inspection
wip/j103f
themes-j206
wip/up-preflight-j198-j226
flakes-j216c
backup/macos01-20261106/claude/j197
flakes-j216b
backup/macos01-20261106/claude/j197-red
wip/up-preflight-j198-review
flakes-j216
backup/macos01-20261106/claude/appwin-icons-j207
engineering-j208b
appwin-mock-j209
toast-stack-j165
engineering-j208
decisions-d351
decisions-d350
decisions-d349
appearance-presets-mock-j129
decisions-d348
docs-audit-spec
doc-coverage-baseline-raise
decisions-d347
decisions-d346b
decisions-d346
j88h-profile
ui-consistency-j195-deferred
ui-consistency-j195
engineering-practices-j194
preferred-login-ui-j166
signin-pools-263-j174
macos-node-265-s1
decisions-d345
decisions-d344
proxy-cgroup-j164
verify-quick-fuzz-329
proxy-start-io-j168
timing-flakes-j163
up-263-s1
mcp-events-j147
verify-quick-fuzz-j158
proxy-p3-j159
agent-resource-j157-detect
wip/j103d-final
remote-cancel-hang-j155
proxy-traefik-281-j62d
j149-verify-quick
proxy-traefik-281-j62b
j150-drop-unnamed
wip/j79d-red-r4
wip/j103d-d332
rootless-toast-j142
wip/j103d-head
wip/j103d
j125c-mcp
macos-build-j136
backup/macos01-20261106/claude/j136
wip/j79d-red-cli
wip/j79d-red-server
j121-link-liveness
j118c-tracer-failures
wip/j79c-head
release-desktop-j131
j124c-signed-macos
wip/j79c-red-p3
wip/j79c-red-p23
wip/j79c-red-p22
merge-j128-check
perf-deferred-j128
sidebar-toggle-260
j127-shortcut-changed-dot
j125-mcp
j118-tracer-failures
j126-account-summary
settings-sections-mock-j92
j120-pinch-lock
sidebar-292
backup/macos01-20261106/claude/j120
taste-2026-11-03
row-actions-242-215
backup/macos01-20261106/claude/stash-0-untracked
backup/macos01-20261106/claude/J80-265-s1-run
backup/macos01-20261106/claude/J80-macos-s1-20261102
j112-canvas-commands
ui-no-overshoot-scroll
fix-recovery-restart
backup/macos01-20261106/claude/stash-1-untracked
taste-2026-11-02
size-baseline-ratchet
machine-cards-structure-j108
j109-boot-copy-mockups
fix-dev-red
wip/up-263-s1
j95b-settings-rows
proxy-traefik-281
j95-option-rows
docs-human-j91
docs-keys-palette
fix-flood-1013-270-j38d
mesh-ssh-s1
mesh-ssh-s0
addhost-d291-tunnel
test-stability-load
mesh-relay-220
repro-296
shim-thp-off-24
mesh-pins-258-v2
mesh-dns
signin-d311-merged
docs-pages
ui-size-283
fix-ui-batch-specs
wip/j12b-edge-update
gate-on-arm
fix-rename-sync-293
trust-285-j60b
fix-focus-ring-clip
integ-ui-1101
edge-update-v2
signin-d311
taste-2026-11-01
layout-per-tab-250
ui-approved-splitopt-224
ui-approved-splitopt-224-on-dev
icon-text-centre-282
ui-approved-189-212-216
research-reverse-proxy
transfer-budget-320
review-mesh-pins-258-r2
sidebar-pill-259
review-mesh-relay-220
wip/j12-edge-update-v2
nix-package
status-crash-loop-237
fix-dev-test-failures
fix-fixture-socket-path
wip/mesh-relay-220-pre-dev-advance-rebase
fix-canvas-side-clip
research-head-to-head
frosted-real-256
review-mesh-pins-258
wip/mesh-relay-220-pre-final-rebase
review-layout-per-tab-250-r2
broad-suite-triage
nix-devshell
review-status-crash-loop-237-r3
wip/nix-devshell-pre-p3
taste-frosted-real
wip/code-serve-demo-2026-10-30
wip/code-serve-glass-2026-10-30
wip/code-serve-clusters-2026-10-30
wip/code-merge-release94-dev-2026-10-30
wip/rproxy-delta-scratch-2026-10-30
wip/stale-note-257-staged-2026-10-30
wip/d241-size-staged-2026-10-30
review-broad-suite-207
wip/mesh-pins-258-pre-split
wip/mesh-relay-220-pre-rebase
mesh-measure-220
plan-211-update
review-edge-autoupdate-r3
perf24-reconnect
review-stale-note-r2
release-prod-live
d241-shell-marker
review-mesh-pins-server-258
review-revocation-r3
review-setup-token-r2
review-sync-journal-r3
wip/review-mesh-pins-258-prune-review-1995fee46
frosted-scroll-256
sync-journal-r3
revocation-per-account-wip
review-tokio-asyncfd-0386
review-sync-p3s
wip/stash-8by64-stash0-2026-10-30
mesh-drag-r2
setup-token-data-wip
release-9-4
auth-busy-1013
mesh-view-overlap
sync-p3s
review-auth-busy-wake
review-readd-r8-wip
mesh-drag-bounds
audit-auth-93
app-windows-stream-start
review-authority-cap-r3
app-windows-r7-measure
readd-minimal-r6
readd-minimal-r5
app-windows-r7-soak
readd-minimal-r4
app-windows-r7
app-windows-r6
readd-minimal-r3
readd-minimal-r2
readd-minimal
merge-release8-dev
review-busy-page-liveness-r2
review-busy-page-liveness
mesh-view-trim
review-authority-read-retry-r2
review-remove-uninstall-r2
release-retry-reattach
release-retry
review-remove-uninstall
review-authority-read-retry
app-windows-ship
mesh-autoswitch-p2
mesh-ssh
mesh-relay
mesh-macos-node
screen-and-control
session-labels
review-real-logins-r5
review-real-logins-r4
review-real-logins-r3
shim-memory-r2
review-real-logins-r2
shim-memory
shadow-addhost-instant
shadow-real-logins
shadow-same-session-flood
shadow-prod-perf-fix
hosts-from-server
app-windows-real
app-windows-transport
release-prod-6w
mesh-policy-verify
quiet-close-toast
dev-merge-r4
review-mesh-agent-transport
review-app-windows-transport
settings-roundtrip
review-release-pipeline-r4
mesh-agent-transport
ended-strip-fix
review-audit-ca-update-d244
settings-rows
phone-width
signout-clear
fix-restart-tests
dev-merge-rp
signer-hardening
review-audit-ca-update
terminal-fidelity
review-audit-ca-update-wip
mesh-integrate
review-mesh-agent-transport-515d3712
review-mesh-dataplane-feec1988
mesh-dataplane
signout
research-canvas-node
wip/review-renderer-cost
wip/review-terminal-fidelity
padlock-glyph
review-release-pipeline-2e95b689
phone-polish
sidebar-polish
inbox-generic
review-mesh-dataplane-378d4055
prod-batch
ui-inbox
ui-ca-unlock
ui-machine-sections
review-mesh-dataplane-a5f744fb
tls-session-prod
session-bounds-prod
fix-dead-session
session-bounds
tls-session
palette-polish
shortcuts-d222
research-appmeta
phone-gutter
ui-centre-button
wip/ui-centre-button
wip/tls-session
review-addhost-breakit
ui-platform
renderer-cost
remote-agents
mesh-api
ssh-bootstrap
prod-hotfix2
prod-fixes
review-remote-agents
review-mesh-dataplane
session-life
research-browser
review-mesh-api
research-capture
review-mesh-api-round3
review-mesh-api-round1
gate-stability
review-gate-stability
review-relay-direct
review-mesh-api-findings
proto-sessions
font-picker
proto-addhost
proto-windows
mesh-core
latency-gate
research-bsd
research-discovery
keytray-mods
mock-core
ui-tweaks
fix-motion-sel
fix-motion
integrate2
fix-robust
t-web
t-server
vt-ghostty
fix-snap-echo
t-web-auth
review-vt2
t-agent
lane-protocol
lane-shim
lane-auth
s1-base
s1-contracts
s1-workspace
No results found.
No labels
launch-qa
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set
Reference
kayg/infraternal#228
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Owner screenshot 2026-10-16 (production, web
f210a4f25): hovering agy's long wrapped OAuth link in a terminal shows the link tooltip, but the address text is cut off at the tooltip's right edge (lines run past the box,overflow: hiddenat max-width 280px with unbreakable URL text), and the tooltip covers the link it describes.Expected (TASTE: a tooltip never sits under the pointer, and stays short): the tooltip shows the destination readably — the host in full, the rest shortened (e.g. middle ellipsis to ≤2 lines), never clipped mid-character — and sits clear of the hovered link.
Starting #228 on branch
link-tooltip, based onorigin/devatf210a4f25055a1c6ee4e624e36f99f3f20ead640. Read #228 (0 comments) and the terminal-links parent #214 (113 comments). Worktree starts clean.Baseline reproduction is queued through
build-slotonTRACER_TEST_PORT=4228; it has waited about 10 minutes and has not started. Needed: a managed build slot release. No source fix is in progress before the baseline capture.Finding (pre-fix, HEAD
f210a4f25055a1c6ee4e624e36f99f3f20ead640): the production tracer build on the mock double reproduces #228 at 390×844 / 44 columns. The 428-character OAuth-shaped destination wraps over 8 rows. The label reports 2,695 pxscrollWidthin a 258 px text box; its tooltip intersects both the wrapped link rows and the pointer. The short destination has no text overflow, but its tooltip intersects the link row. The browser regression fails before the fix.Before screenshots:

Run receipt: baseline browser log.
Pre-fix browser assertions now report all layout failures together on
f210a4f25055a1c6ee4e624e36f99f3f20ead640: the long label'sscrollWidthis 2,695 px vs 258 px available; the tooltip intersects the 8 link rows and the pointer. The short destination fits at 238 px but its tooltip still intersects its row. Clicking the long link opens the exact 428-character target. Raw receipt: baseline soft-assertion run.Unpreviewed production tracer baseline at
f210a4f25055a1c6ee4e624e36f99f3f20ead640: critical JavaScript is 297,685 bytes gzip / 247,813 bytes Brotli. Both are within D188 budgets; this is the byte-for-byte comparison target for the fix. Build and transfer receipts: build, transfer report.The corrected production browser rerun is queued on
TRACER_TEST_PORT=4234; it has waited about 15 minutes without starting. All four standard build slots and the reserved slot are occupied. Needed: a slot release. The baseline finding and transfer comparison are already attached; no further source changes are in progress.The shared tooltip fix passes its targeted production tracer browser case (mock double, 390×844, 44 columns). The 428-character OAuth URL spans 8 terminal rows; the label is 260/260 px with 260 px scroll width, 28.78 px high (2 lines), and its 282×42.78 px box sits at y=62, clear of link rows y=113–257 and the pointer. At the 15 px touch scale it remains 2 lines (36 px label height) and clear of those rows. The 34-character short URL is one line (238 px), and its tooltip sits above its row with no overlap. Clicking still opens the exact 428-character destination.
The short-link reproduction exposed a 2 px width-accounting mismatch between the measure and visible box. The shared component now accounts for its 20 px padding and 1 px border per side, and measures wrapped height before choosing a side. This keeps the visible label width equal to its measured width.
After screenshots:


The complete terminal-links browser suite and production-build video capture are running.
The full terminal-links browser suite passes on the production tracer build with the mock double:
3 passed (33.9s)The 9.64-second video shows the long and short tooltip behavior. Latest screenshots and the complete browser log are attached below:


Video
Browser log
Shared tooltip module step committed and pushed to
link-tooltip:98822f739.Gate receipts for the committed source:
bun run checkpassed; the production tracer terminal-links browser suite reported3 passed (33.9s). The screenshots, 9.64-second video, and browser log are attached above. Normal unpreviewed transfer sizing is in progress.The first D188 transfer measurement exposed a no-growth mismatch: the candidate critical path is 298,108 gzip / 247,989 Brotli bytes versus the shipped
f210a4f25baseline at 297,685 / 247,813 (+423 / +176 bytes). Both remain under the D188 caps, but the task requires no critical-path growth. The complete worktree source was measured at commit98822f739with the terminal-bound and browser-test changes present. I will move URL-specific formatting into a shared UI module loaded with terminal-link on-demand code, then remeasure the final branch head.D188 before-adjustment transfer receipt: critical-path-before-adjustment.txt. The first iterate gate completed with fmt, CLI compile-out, Clippy, workspace tests, cargo-deny, tracer transfer budget, and TypeScript protocol gates all passing; I am reducing the critical path to meet the stricter no-growth target.
The shared UI formatter move preserves the browser behavior. Full terminal-links suite on the production tracer mock-double build: 3 passed (36.6s). The 428-character target remains 2 lines at desktop and touch text scales, the short target stays 1 line, all tooltip rectangles clear their rows and pointer, and the long link opens its exact target. Latest screenshots, 12.72-second video, and full log are attached:


Video
Browser log
The D188 path adjustment is committed and pushed to link-tooltip:
d0e380505. URL formatting now comes from a reusable packages/ui helper loaded by terminal links on demand.Receipts on this source: bun run check passed; the complete production tracer terminal-links suite reported 3 passed (36.6s). The second scripts/verify iterate is running to confirm the critical transfer delta and gate the commit.
The second iterate gate passes (fmt, VT format, CLI compile-out, Clippy, workspace tests, cargo-deny, tracer transfer budget, TypeScript protocol). D188 now measures 297,885 gzip / 247,749 Brotli bytes on the critical path against 297,685 / 247,813 at
f210a4f25; identity is 962,375 versus 961,716. Moving the formatter lowered gzip growth from +423 to +200 and Brotli is now 64 bytes below baseline, but the no-growth requirement is not met. Receipt: critical-path-after-format-move.txt. I will move link-only wrap sizing and placement support into the shared UI on-demand extension and preserve the single shared tooltip surface.Added the shared on-demand link tooltip formatter and width/arrow styling in
packages/ui/src/tooltip-link-text.ts. Commita1a0f3d33pushed tolink-tooltip;bun run checkpassed. The implementation remains on the on-demand path under D160/D188. Browser regression is queued for build slot 4250.Connected terminal hover, touch and click previews to the shared tooltip extension in
apps/tracer/src/terminal-links.on-demand.ts; full destinations still reach activation, and the side remains reserved while the preview is open. Commit5634311edpushed.bun run checkpassed. The production Playwright regression remains queued on unique port 4250.Restored
packages/ui/src/tooltip.tsto its base critical-path sizing and positioning; link wrapping, arrow placement and row-aware points now live in the shared on-demand extension. Commitce316adfapushed.bun run checkpassed. The production Playwright regression is still queued on port 4250.Production tracer browser regression passed (3/3 in 38.4 s) on port 4250 against the mock double.
The 428-character OAuth URL wrapped over 8 rows; displayed label width/client/scroll width was 258/258/258 px, with 2 lines. Tooltip rect: x=53, y=62, 280×42.78 px; link rows y=113..257. No row or pointer intersection. At 15 px type scale it remained 2 lines and 280×50 px. The 34-character short target fit one line; tooltip was 260×28.39 px and clear of its row. Clicking opened the exact original target.
Long screenshot:

Short screenshot:
Production video: link-tooltip.webm
Measurements: browser log
Implementation head:
ce316adfa; the passing regression spec is the remaining worktree change and will be committed separately.Committed and pushed the passing production browser regression as
7e997d38b. It covers long and short targets, two-line wrapping at the 15 px scale, label containment, row/pointer clearance, and exact full-target activation. The 3-test run passed in 38.4 s; screenshots, video and measurements are attached in the preceding issue comment.scripts/verify iterateat7e997d38bhad one server test failure after a 492 s gate:responsive_socket_survives_a_held_terminal_open_with_an_existing_attachment(448 passed, 1 failed, 48 ignored in the server crate). It panicked atcrates/infraternal-server/src/auth/socket_clock.rs:81: “socket test stopped before its observed event”. The remaining Rust gate rows shown were PASS; TS/transfer rows were not reached. I am checking this auth socket liveness timeout on detached baselinef210a4f25in the same host load window.Full iterate log
The detached
f210a4f25baseline comparison passed the single socket liveness test: 1 passed, 0 failed in 0.76 s (baseline test build 12m 14s). The branch iterate run had failed the same test after the server test group ran 297.40 s with 448 pass / 1 fail / 48 ignored; failure wassocket test stopped before its observed event. Since the baseline did not reproduce it, I am rerunning the test on the branch and will let the final full gate decide.Baseline test log
Branch iterate log
The targeted liveness timeout passed on both checkouts: detached baseline
f210a4f25passed in 0.76 s; branch7e997d38bpassed in 0.82 s. The original failure happened only in the 297.40 s parallel server test group (448 passed, 1 failed, 48 ignored); the branch rerun reused its compiled test binary. Proceeding with final static size andscripts/verify fast.Baseline output
Branch output
Stripped static x86_64 baseline build at merge base
f210a4f25: 21,422,096 bytes (static PIE; no PT_INTERP), matching the D243 repository ceiling exactly. The build used the locked release recipe and this job’s single target directory. The final branchfastrun will produce the head size and D188 transfer receipt.Baseline size log
Gate finding: final
scripts/verify fastfailed the Rust test row.infraternal-agentreported 106 passed, 1 failed, 14 ignored (11.05s);remote_client::tests::server_path_capability_is_advertised_on_native_and_fallback_transportspanicked atcrates/infraternal-agent/src/remote_client_tests.rs:603with OS error 98 (AddrInUse). Other Rust rows, cargo-deny and both static builds passed. The x86_64 static binary was 21,422,096 bytes (+0 vs merge base). The gate stopped before TypeScript, browser and transfer checks. Log: https://git.kayg.org/attachments/ac9f6ae4-a796-4637-9c60-a9d55fcb0646Follow-up: reran the failed test with the workspace-unified
remote,meshfeatures. It passed in isolation: 1 passed, 0 failed, 120 filtered out, 0.19s. The initialAddrInUsewas transient; proceeding with a fullscripts/verify fastrerun. Receipt: https://git.kayg.org/attachments/a1efb0ee-035e-474c-a5de-507070119f30Fast gate run 2 exited 0. Rust passed, including infraternal-agent 121 tests and infraternal-server 449 passed/0 failed/48 ignored. TypeScript protocol tests passed: 37 pass, 0 fail; stream differential: 134 streams, 11,439 schedules, zero divergences. x86_64 static binary 21,422,096 bytes (delta +0); aarch64 19,594,856 bytes (delta +0). Critical JS: identity 961,716 bytes (baseline 961,716), gzip 297,696 (baseline 297,685; +11), Brotli 247,569 (baseline 247,813; -244). The 11-byte gzip delta is under review against a fresh merge-base production build before claiming no transfer growth. Gate log: https://git.kayg.org/attachments/23783d18-c624-4830-a554-e16bcf9c7511
Fresh merge-base transfer comparison paused: the baseline production build could not resolve
@scure/bip39because dependencies were not installed in the detached scratch checkout. This is an environment setup issue; I will runbun install --frozen-lockfilethere, then repeatbun run buildandbun run transfer-budget.ts. Log: https://git.kayg.org/attachments/16c0ad2d-24b6-4ad1-bf8a-71c727726329Fresh D188 comparison (same Bun 1.4.2 production build and transfer-budget recipe): merge-base
f210a4f25critical JS = 961,716 identity / 297,680 gzip / 247,556 Brotli bytes; branch = 961,716 / 297,696 / 247,569 (+0 / +16 / +13). The raw critical total is unchanged, but compressed transfer is slightly higher; I am reducing that before finalizing. Baseline receipt: https://git.kayg.org/attachments/dcee214b-aff9-465d-998b-e0ca12a48c0dFinished #228 on
link-tooltip. HEAD7e997d38b395d227563e1185abab1809f0cf8d32is pushed toorigin/link-tooltip.The production browser regression passed (3 tests, 38.4s). The 428-character target wraps across 8 rows; its visible label is 2 lines, contained within the tooltip, and the tooltip clears the link rows and pointer. The short target stays on one line. Opening the link retains the full original URL. The earlier old-code regression failure and before/after captures are attached below.
The final
scripts/verify fastexited 0. Its first attempt had one transientAddrInUsetest failure; the feature-correct isolated rerun passed, and the full rerun passed. The x86_64 stripped static binary is 21,422,096 bytes, matching merge base (delta 0). Critical JavaScript identity is unchanged; compared with a fresh merge-base production build, gzip is +16 bytes and Brotli +13 bytes, both within the existing D188 limits. Full gate log: https://git.kayg.org/attachments/23783d18-c624-4830-a554-e16bcf9c7511Final gate output (verbatim):
Continuing review fixes for #228 on branch
link-tooltip, current HEAD7e997d38b395d227563e1185abab1809f0cf8d32, based onorigin/devatf210a4f25055a1c6ee4e624e36f99f3f20ead640. Read all 29 issue comments and all 113 comments on parent #214. The review requires moving line/side behavior into the shared tooltip as per-call options, removing injected global styles and host-side timers/DOM queries, correcting the dirty-group indentation, and adding the plain-tooltip isolation regression.Finding reproduced on reviewed HEAD
7e997d38b: the added plain-tooltip test fails before the structure fix. After a link tooltip, the plain label's computedoverflow-wrapchanges fromnormaltoanywhere; the tooltip is 280 px wide and the label grows to 187.08 px high. This confirms the injected global style changes shared tooltip behavior. Raw browser output: plain-before.log. I’m capturing phone and desktop production screenshots before editing the shared component.Pre-fix production browser run on
video plain regression failure layout run.
7e997d38bconfirms the regression. The plain tooltip changes from 274.97×27.88 px (one overflowing line) to 280×201.08 px after a link hover;overflow-wrapchanges fromnormaltoanywhere, andscrollWidthchanges from 3,129 px to 258 px. The new test fails on this head. The existing long-link assertions pass at phone and desktop width: the phone target spans 8 terminal rows, its 280×42.78 px tooltip is outside the rows and pointer, and the desktop tooltip is outside 7 rows and the pointer. Before captures and the 13.8 s browser run are attached:Implementation committed and pushed as
23669a739352bef34e6edefc50d137b9d64d6165. The shared tooltip now owns per-call two-line and side options; its measure and surface share the 258 px content cap, and the shared arrow uses measured height. The link caller no longer injects styles or changes host state, and formatting remains on demand.apps/tracerTypeScript check passed. Production browser suite: 4 passed. The 428-character destination stayed within 258 px over two lines; its tooltip cleared 8 phone rows and 7 desktop rows plus the pointer. Short URL remained one line; arrow alignment passed at both text sizes.Browser coverage committed and pushed as
e272339a444ef0a78afb7377211600ef19952739. Final production tracer run onTRACER_TEST_PORT=4298: 4 passed. At phone width, the 428-character destination spans 8 rows; the tooltip is 280×42.78 px with a 258 px label, and clears the rows and pointer. At desktop width it clears all 7 rows and the pointer. Short links stay one line, and shared-arrow alignment passes at 12 px and 15 px text sizes. The plain-tooltip regression confirms its one-line layout is unchanged after a link preview.Before:


After:
Production video: link-tooltip.webm. Browser receipt: production-final.log.
scripts/verify iteratereached the Rust test gate and had one auth socket timeout failure:auth::tests::socket_tests::restart_open_liveness_review::responsive_socket_survives_a_held_terminal_open_with_an_existing_attachmentpanicked atcrates/infraternal-server/src/auth/socket_clock.rs:81with “socket test stopped before its observed event”. Result: 448 passed, 1 failed, 48 ignored (346.28s). The other iterate rows passed. I am running this exact test on a detachedorigin/devbaseline under the same test window per the host-load gate rule.Final
scripts/verify fastreached static builds. One unrelated workspace test failed because TCP bind returnedAddrInUseatcrates/infraternal-agent/src/remote_client_tests.rs:603inserver_path_capability_is_advertised_on_native_and_fallback_transports(106 passed, 1 failed, 14 ignored). The auth socket timeout test that failed in iterate passed during fast. I will rerun the bind test in isolation, then rerun the full fast gate.Static results: x86_64 21,422,096 bytes, cap 21,422,096, delta +0; aarch64 19,594,856 bytes, cap 19,594,856, delta +0. Both static builds passed.
Gate output:
Full log: fast.log. Iterate log: iterate.log.
Isolated retries clarify both gate findings. The auth socket timeout test from iterate passed on detached
origin/dev(41a3a81ab, 1 passed in 0.71s) and passed in the full fast run. The remote client bind test also passed on this branch when run alone with--features remote(1 passed in 0.24s); the full fast failure was an ephemeral port collision. I am rerunningscripts/verify faston the branch head.branch-port-test.log
The current production
disttransfer check found critical JavaScript above the D188 limit: identity 1,071,268 bytes, gzip 339,972/307,200, Brotli 285,237/256,000. Against the pre-continuation #228 measurement (identity 961,716, gzip 297,696, Brotli 247,569), that is +109,552 identity, +42,276 gzip, +37,668 Brotli. The dist was built at 2026-10-16 14:30 in this worktree; I will rebuild from the branch head and verify the per-chunk cause before closing.transfer-final.log
Fresh default production builds from
7e997d38band branch heade272339awith Bun 1.4.2 resolve the earlier over-budgetdistfinding; the existingapps/tracer/distwas not representative of these reproducible builds. Both pass D188's critical budgets.Critical JavaScript, baseline → branch: identity 961,716 → 962,037 bytes (+321); gzip 297,694 → 297,813 (+119); Brotli 247,417 → 247,676 (+259). Branch gzip is 297,813/307,200 and Brotli 247,676/256,000. The link formatter remains in
terminal-links.on-demand.ts, dynamically imported byterminal.ts; on-demand totals are 207,903 bytes / 82,229 gzip / 73,350 Brotli, with the largest chunk 15,946 gzip / 14,179 Brotli.The initial worktree
distmeasurement (1,071,268 identity / 339,972 gzip / 285,237 Brotli) differed from a fresh default build, so I discarded it as a non-reproducible artifact set. Fresh build and transfer receipts:build-baseline.log, transfer-baseline.log, build-head.log, transfer-head.log.
Status: final fast rerun has passed all Rust workspace tests, static builds, and the tracer transfer gate. It is still in
cargo build -p infraternal-protocol --example stream_oracle, with no output for over 30 minutes; the process and two compiler children are still present.sccache --show-statsresponds and reports no cache timeouts. I am leaving the gate running for the TypeScript protocol differential tests.Current gate log: fast-final.log
The final fast rerun has passed Rust workspace tests, static builds, and the tracer transfer gate, but
cargo build --quiet -p infraternal-protocol --example stream_oracleis stalled. After 35 minutes, Cargo and its two compiler children remain asleep on futexes with zero CPU time.sccache --show-statsresponds but these requests do not complete. I am stopping only this job's gate process and rerunningscripts/verify fastwithSCCACHE_DISABLE=1for this job; no other process or shared cache server will be touched.Current log: fast-final.log
With sccache disabled,
scripts/verify fastreached static builds but the Rust test row failed once inremote_runtime::tests::standby::control_queue::saturated_native_control_queue_does_not_hide_primary_terminal_outputatcrates/infraternal-server/src/remote_control_queue_tests.rs:216(live_connection.close_reason().is_none()assertion). Result: 448 passed, 1 failed, 48 ignored (290.82s). All other Rust rows and both static builds passed; x86_64 was 21,422,096 bytes (+0), aarch64 19,594,856 bytes (+0). I will rerun this case, then run the full gate withRUST_TEST_THREADS=2and sccache disabled.Gate output:
Full log: fast-final-no-sccache.log.
The isolated control-queue test passed on the branch with
RUSTC_WRAPPER=and--features remote(1 passed in 0.07s;CONTROL_SATURATION active=true queued=1 native_primary_live=true shadow_output=0).SCCACHE_DISABLE=1still invoked the stalled wrapper, so the next full gate will setRUSTC_WRAPPER=and useRUST_TEST_THREADS=2,CARGO_BUILD_JOBS=2.branch-control-queue-test.log
Finished at
e272339a44(pushed tolink-tooltip; no deploy). The shared tooltip now takes per-call line and side options; link formatting remains in the on-demand chunk. Four Playwright cases pass, including the plain-tooltip-after-link regression.Evidence:
scripts/verify fasttranscriptStatic x86_64: 21,422,096 bytes, cap 21,422,096, delta +0. Static aarch64: 19,594,856 bytes, cap 19,594,856, delta +0. Fresh production comparison (
7e997d38b→e272339a): critical JS 961,716 → 962,037 identity (+321), gzip 297,694 → 297,813 (+119), Brotli 247,417 → 247,676 (+259). Link formatting remains on-demand; on-demand chunks total 207,903 identity / 82,225 gzip / 73,350 Brotli.Final gate output:
Deployed to production (web only, no restart) at
1bcb7a87b, merged into dev. release-prod-live updated.Closed by the orchestrator during the 2026-10-28 handover (#254), on the owner's request to close finished issues. Evidence: Tooltip fix merged to dev and deployed web-only at
1bcb7a87b(ancestor of dev and release-prod-live); final gate passed. Reopen if anything is still missing.kayg referenced this issue2026-10-28 17:53:55 +00:00