Ensure ref value is always undefined when disconnected

[sorvell]

Fixes #4622

[changeset-bot]

🦋 Changeset detected

Latest commit: 7742fbe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
lit-element	Patch
lit-html	Patch
lit	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

📊 Tachometer Benchmark Results

Summary

nop-update

• this-change, tip-of-tree, previous-release: unsure 🔍 -6% - +2% (-0.66ms - +0.17ms)
  ^{this-change vs tip-of-tree}

render

• this-change: 46.23ms - 48.84ms
• this-change, tip-of-tree, previous-release: unsure 🔍 -1% - +5% (-0.12ms - +0.91ms)
  ^{this-change vs tip-of-tree}
• this-change, tip-of-tree, previous-release: unsure 🔍 -2% - +2% (-0.60ms - +0.52ms)
  ^{this-change vs tip-of-tree}
• this-change, tip-of-tree, previous-release: unsure 🔍 -3% - +1% (-0.92ms - +0.26ms)
  ^{this-change vs tip-of-tree}

update

• this-change: 465.08ms - 475.65ms
• this-change, tip-of-tree, previous-release: unsure 🔍 -5% - +7% (-1.84ms - +2.65ms)
  ^{this-change vs tip-of-tree}
• this-change, tip-of-tree, previous-release: unsure 🔍 -1% - +3% (-0.77ms - +2.24ms)
  ^{this-change vs tip-of-tree}
• this-change, tip-of-tree, previous-release: unsure 🔍 -1% - +1% (-3.96ms - +6.06ms)
  ^{this-change vs tip-of-tree}

update-reflect

• this-change: 473.21ms - 479.64ms
• this-change, tip-of-tree, previous-release: unsure 🔍 -0% - +2% (-1.67ms - +8.98ms)
  ^{this-change vs tip-of-tree}

Results

this-change

• Browser: chrome-headless 125.0.6422.60
• Sample size: 50
• Built by: Benchmarks #6249

• Commit: 7742fbe

render

Version	Avg time	vs
	47.74ms - 49.96ms	-

update

Version	Avg time	vs
	479.45ms - 490.48ms	-

update-reflect

Version	Avg time	vs
	485.89ms - 494.19ms	-

this-change, tip-of-tree, previous-release

• Browser: chrome-headless 125.0.6422.60
• Sample size: 30
• Built by: Benchmarks #6249

• Commit: 7742fbe

render

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	18.61ms - 19.43ms	-	unsure 🔍 -1% - +5% -0.16ms - +0.94ms	slower ❌ 0% - 6% 0.09ms - 1.17ms
tip-of-tree tip-of-tree	18.26ms - 19.00ms	unsure 🔍 -5% - +1% -0.94ms - +0.16ms	-	unsure 🔍 -1% - +4% -0.27ms - +0.75ms
previous-release previous-release	18.03ms - 18.74ms	faster ✔ 1% - 6% 0.09ms - 1.17ms	unsure 🔍 -4% - +1% -0.75ms - +0.27ms	-

update

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	36.11ms - 39.77ms	-	unsure 🔍 -5% - +8% -1.97ms - +2.91ms	unsure 🔍 -2% - +12% -0.62ms - +4.16ms
tip-of-tree tip-of-tree	35.85ms - 39.08ms	unsure 🔍 -8% - +5% -2.91ms - +1.97ms	-	unsure 🔍 -3% - +10% -0.93ms - +3.52ms
previous-release previous-release	34.64ms - 37.71ms	unsure 🔍 -11% - +1% -4.16ms - +0.62ms	unsure 🔍 -9% - +2% -3.52ms - +0.93ms	-

nop-update

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	10.39ms - 10.97ms	-	unsure 🔍 -6% - +1% -0.65ms - +0.16ms	unsure 🔍 -3% - +5% -0.32ms - +0.54ms
tip-of-tree tip-of-tree	10.64ms - 11.21ms	unsure 🔍 -2% - +6% -0.16ms - +0.65ms	-	unsure 🔍 -1% - +7% -0.07ms - +0.78ms
previous-release previous-release	10.25ms - 10.88ms	unsure 🔍 -5% - +3% -0.54ms - +0.32ms	unsure 🔍 -7% - +1% -0.78ms - +0.07ms	-

this-change, tip-of-tree, previous-release

• Browser: chrome-headless 125.0.6422.60
• Sample size: 50
• Built by: Benchmarks #6249

• Commit: 7742fbe

render

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	33.80ms - 34.73ms	-	unsure 🔍 -2% - +2% -0.56ms - +0.63ms	unsure 🔍 -1% - +2% -0.47ms - +0.70ms
tip-of-tree tip-of-tree	33.86ms - 34.60ms	unsure 🔍 -2% - +2% -0.63ms - +0.56ms	-	unsure 🔍 -1% - +2% -0.43ms - +0.60ms
previous-release previous-release	33.79ms - 34.51ms	unsure 🔍 -2% - +1% -0.70ms - +0.47ms	unsure 🔍 -2% - +1% -0.60ms - +0.43ms	-

update

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	70.35ms - 73.25ms	-	unsure 🔍 -2% - +3% -1.59ms - +2.38ms	unsure 🔍 -3% - +3% -2.02ms - +1.92ms
tip-of-tree tip-of-tree	70.04ms - 72.76ms	unsure 🔍 -3% - +2% -2.38ms - +1.59ms	-	unsure 🔍 -3% - +2% -2.35ms - +1.47ms
previous-release previous-release	70.51ms - 73.19ms	unsure 🔍 -3% - +3% -1.92ms - +2.02ms	unsure 🔍 -2% - +3% -1.47ms - +2.35ms	-

this-change, tip-of-tree, previous-release

• Browser: chrome-headless 125.0.6422.60
• Sample size: 50
• Built by: Benchmarks #6249

• Commit: 7742fbe

render

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	31.97ms - 32.85ms	-	unsure 🔍 -1% - +3% -0.22ms - +0.91ms	unsure 🔍 -1% - +2% -0.37ms - +0.77ms
tip-of-tree tip-of-tree	31.71ms - 32.42ms	unsure 🔍 -3% - +1% -0.91ms - +0.22ms	-	unsure 🔍 -2% - +1% -0.65ms - +0.37ms
previous-release previous-release	31.85ms - 32.58ms	unsure 🔍 -2% - +1% -0.77ms - +0.37ms	unsure 🔍 -1% - +2% -0.37ms - +0.65ms	-

update

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	478.25ms - 492.39ms	-	unsure 🔍 -1% - +3% -3.94ms - +13.45ms	unsure 🔍 -1% - +2% -6.81ms - +10.71ms
tip-of-tree tip-of-tree	475.50ms - 485.63ms	unsure 🔍 -3% - +1% -13.45ms - +3.94ms	-	unsure 🔍 -2% - +1% -10.04ms - +4.44ms
previous-release previous-release	478.20ms - 488.54ms	unsure 🔍 -2% - +1% -10.71ms - +6.81ms	unsure 🔍 -1% - +2% -4.44ms - +10.04ms	-

update-reflect

Version	Avg time	vs this-change	vs tip-of-tree tip-of-tree	vs previous-release previous-release
this-change	523.05ms - 535.03ms	-	unsure 🔍 -1% - +2% -6.82ms - +9.67ms	unsure 🔍 -1% - +2% -6.97ms - +8.85ms
tip-of-tree tip-of-tree	521.96ms - 533.27ms	unsure 🔍 -2% - +1% -9.67ms - +6.82ms	-	unsure 🔍 -2% - +1% -8.15ms - +7.18ms
previous-release previous-release	522.93ms - 533.27ms	unsure 🔍 -2% - +1% -8.85ms - +6.97ms	unsure 🔍 -1% - +2% -7.18ms - +8.15ms	-

_{tachometer-reporter-action v2 for Benchmarks}

[github-actions]

The size of lit-html.js and lit-core.min.js are as expected.

[augustjk]

Minor suggestions on the changeset