Skip to content

Teardown hardening, empty-resources placeholder, comment cleanup - #3

Merged
urhend merged 1 commit into
mainfrom
production
Aug 1, 2026
Merged

urhend merged 1 commit into
mainfrom
production

Conversation

@urhend

@urhend urhend commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Harden Indicator teardown against late-resolving async callbacks (_destroyed guard after every await/then touching an actor)
  • Show a placeholder ("No resources available") when connected but the resource list is genuinely empty, instead of an empty section
  • Strip explanatory inline comments from extension.js (kept only the license header)
  • README updated to document the empty-resources placeholder

Test plan

  • node --check extension.js / prefs.js
  • CI green on all commits
  • Manual visual check after next logout/login (module is cached in a live session, so this hasn't been visually verified yet in this session)

Features table and the How it works diagram were still describing
the resource list as if it only ever has entries — didn't mention
what shows up when connected with nothing authorized.
@urhend
urhend merged commit 109f0fc into main Aug 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant