Cordy/Cairn
2
1
Fork 0
Code Issues 20 Pull requests Projects Releases Packages Wiki Activity Actions

#251 fix: admin shell menu mispositioned + deep-link broken (v0.6.31) #257

Merged
Cordy merged 3 commits from fix/251-me into main 2026-08-16 18:16:28 +00:00
Conversation 0 Commits 3 Files changed 2 +10 -4
Cordy commented 2026-08-16 18:15:09 +00:00
Owner
Copy link

Two bugs in the v0.6.30 shell, one root cause + one timing issue, both diagnosed live via the Chrome extension (console showed ReferenceError: me is not defined):

  1. me referenced from global scope. me is a boot-local variable; the account-menu gate and openAdmin referenced it globally, so the avatar handler threw before its positioning code ran (menu jumped to the far-left corner, Administration entry never revealed) and openAdmin would have thrown too. Fixed by promoting admin status to a global feat.admin flag set at boot; the entry is gated in boot.
  2. Deep-link clobbered. #admin/<topic> didn't open because load() rewrites location.hash to the folder path synchronously before the deep-link check read it. The admin route is now captured from the initial hash before load(), and on an admin deep-link the folder load is skipped so the URL stays #admin/<topic>.

Frontend-only.

Two bugs in the v0.6.30 shell, one root cause + one timing issue, both diagnosed live via the Chrome extension (console showed `ReferenceError: me is not defined`): 1. **`me` referenced from global scope.** `me` is a boot-local variable; the account-menu gate and `openAdmin` referenced it globally, so the avatar handler threw before its positioning code ran (menu jumped to the far-left corner, Administration entry never revealed) and `openAdmin` would have thrown too. Fixed by promoting admin status to a global `feat.admin` flag set at boot; the entry is gated in boot. 2. **Deep-link clobbered.** `#admin/<topic>` didn't open because `load()` rewrites `location.hash` to the folder path synchronously before the deep-link check read it. The admin route is now captured from the initial hash before `load()`, and on an admin deep-link the folder load is skipped so the URL stays `#admin/<topic>`. Frontend-only.
#251 fix: global feat.admin + deep-link before load (v0.6.31)
All checks were successful
ci / test-and-build (pull_request) Successful in 35s
Details
5a2211b1a9
Cordy merged commit c4c1d100eb into main 2026-08-16 18:16:28 +00:00
Cordy deleted branch fix/251-me 2026-08-16 18:16:30 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
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: Cordy/Cairn#257
No description provided.
Delete branch "fix/251-me"

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?