Skip to content

feat(observers): open a unified monitoring destination - #79

Open
n30nex wants to merge 2 commits into
MeshCore-Beacon:devfrom
n30nex:codex/beacon-observer-destination
Open

n30nex wants to merge 2 commits into
MeshCore-Beacon:devfrom
n30nex:codex/beacon-observer-destination

Conversation

@n30nex

@n30nex n30nex commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Selecting an observer opens its shareable dashboard at ?tab=Observers&observer=<id>&range=7d. Legacy Analytics/Stats links redirect compatibly; the directory stays mounted so Back preserves filters, sorting and scroll. Packet/node/map contexts retain quick inspection with an Open observer dashboard action. New controls are English/French.

Closes #76. Depends on #75 at e65e8e42ec7bcd75029ebc636645eedc2b889cbf. Focused diff. Presentation #80 and contextual comparison #81 are separate child PRs.

Original feature validation on 3a0eb4c: Windows and native Pi build/lint/full 878 tests pass; exact-head build CI passes (upstream CodeQL skipped). The combined preview also passed legacy/shared links, copied URL, browser Back/Forward, retained search, descending sort and exact clicked scroll-position checks. Desktop/390-pixel phone and English/French/keyboard checks pass. Exact source and rollback are on the preview changelog. Maintainers control merging and production release.

AI-assisted implementation under n30nex's approved plan and standing authorization; no specific human review of this new diff is claimed.

Current integration (28 September): the queue is refreshed onto accepted web dev 17f48fb9. Parent e65e8e42ec7bcd75029ebc636645eedc2b889cbf, published head 4f3819b805ff8293939d1e6fb3918f05cafaadd5. Focused diff. Build/lint/tests pass for this refreshed tree and actual-head build CI passes (repository CodeQL skipped). The full web composition 6b688495 is now deployed on Pi server 2ed2e031, retaining every candidate. Its native build/lint and all 935 tests pass; all 18 public assets and both source archives match. Web assets are byte-identical to the previously geometry-tested build; fresh English desktop/French phone and Public-message packet inspection pass. Current evidence and recovery. Earlier feature-specific validation above is historical. No feature behavior changed in this base refresh.

@n30nex

n30nex commented Sep 27, 2026

Copy link
Copy Markdown
Contributor Author

@MrAlders0n, this observer-release slice is ready for your/Claude's review at 3a0eb4c8. Web order is #75 → #79 → #80 → #81. This exact destination candidate passes 878 native tests; the focused parent-to-head diff is in the body. Browser Back preserved search, descending sort and the actual clicked scroll position.

The combined Pi preview is live at https://canadaverse.org/beacon-dev/?tab=Observers with server 88c2c10c / web 42ae09b7, preserving #166/#167/#75. Desktop, 390-pixel phone, English/French, keyboard and navigation checks pass; the changelog and exact source are at https://canadaverse.org/beacon-dev/source.html. Build CI passes (web CodeQL remains skipped). No merges or production switch were performed.

Docs #5 carries the observer-first plan and subsequent connected-investigation work. The proposed public per-IATA MeshMapper scope catalogue is drafted separately, with manual scopes retained; its endpoint/schema is not yet published. The captured historical 1,810 counter difference remains unexplained, not a measured packet-loss claim.

@n30nex
n30nex force-pushed the codex/beacon-observer-destination branch from 3a0eb4c to 4f3819b Compare September 28, 2026 22:38
@MrAlders0n

Copy link
Copy Markdown
Member

Requesting changes on two points:

  • App.tsx:368-370: the quick-observer modal renders after PacketAnalyzerOverlay at the same z-index. Clicking an advert in it opens the analyzer underneath, so the click looks like it did nothing. The overlay below also never gets inactive, which leaves two active aria-modal dialogs. Please close the quick modal when analyzing, or stack it properly and pass inactive to the lower overlay.
  • App.tsx:162: statsTab=observer now redirects, but Analytics still shows the Observer pill (StatsSubHeader.tsx:98, StatsOverview.tsx), and Mesh tab rows still patch({statsTab:"observer"}). Both bounce the user to the Observers tab. Please remove or repoint them.

Also:

  • The Observers tab no longer shows ObserverDetailPanel (public key, adverts, analyze). If that's intended, please note it in the description.
  • The "observer" key in dropSelectionParam is dead now.
  • The new tests are minified onto single lines. Please format them like the rest of tests/.

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.

Open a unified observer dashboard with restorable navigation

2 participants