Repository navigation
Conversation
This was referenced Oct 21, 2026
Draft
Add optional attributed repeater telemetry and availability filtering
MeshCore-Beacon/beacon-web#184
Draft
n30nex
commented
Oct 23, 2026
n30nex
left a comment
Collaborator
Author
There was a problem hiding this comment.
The current consolidated head retains the upstream database changes and the focused reliability fixes, with Topology adjacency/window/cache regressions and native PostgreSQL/plan checks. This is the primary core review lane; the older focused/combined branches should be retired only after the accepted head preserves their required behavior.
The release gates are still material:
- Live is on the three-migration baseline and still reproduces the four large-cursor failures. Dev is on schema 053. Rehearse the actual production-sized transition/rollback before a combined release; the isolated cursor hotfix can be evaluated separately.
- Dev1's containers are stopped. Its branch name and older successful build jobs are not acceptance of this source on that host.
- The shared Pi is currently under heavy I/O pressure. Both bounded global route-query comparisons timed out, so the faster restored-copy query result is not proof of a live-host latency improvement. Capture private workload metrics and qualify cold/warm route snapshots, ordinary API latency and ingest queues together.
- Observer-directory traffic ranking still turns its read error into HTTP 500 in
internal/api/handlers/observer_directory.go. The preview reproduced that response at about five seconds. Review retryable failures through the same 503/Retry-After contract used by the other hardened read paths; do not generalize the Topology-specific fix to every endpoint.
Keep the optional proxy/intake branch #268/#269 separate. The project-wide merge order, issue/PR dispositions and acceptance checklist are being consolidated in MeshCore-Beacon/beacon-docs#43 and tracking issue MeshCore-Beacon/beacon-docs#45. This is a comment review, not approval for production promotion.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The backend fixes and the Topology/TRACE integration were spread across several draft branches. This candidate combines current dev with #253, #255, #259 and #256, preserving the latest observer-comparison, read-model restore, retryable timeout and Web 2.1 requirements. Collected telemetry and mobile intake are deliberately a separate dependent draft stack.
Backend performance and reliability
Validation and operating boundaries
Local Go build/vet/unit suite, targeted PostgreSQL 16.15 regressions and all eight added plan cases passed. Native CI, vulnerability checks, runtime image checks and paired preview qualification now pass. The PostgreSQL fixture uses C collation. Result tests now isolate their own schema instead of relying on public tables left by another test.
No schema migration beyond upstream 001-053 is added;
channel_historyuses the existing background-state table. Retention and raw evidence are preserved. Operators must check writable mount ownership for the nonroot image and the documented schema-compatible rollback before deployment. Database-role separation and production soak/restore gates remain explicit operational work; this PR does not silently revoke privileges or touch live.The paired web change adds basic Atlas and performance-enhanced Topology without a telemetry dependency. Operator documentation is in the companion docs draft. Current runtime measurements and remaining capacity gates are recorded below.
Refs #265, #252, #254, #258, #223, #224, #226. This consolidates the prior focused drafts; leave them recoverable until the candidate is accepted. No review request or production deployment is made by opening this draft.
The Topology follow-up streams adjacent array values, preserves NULL gaps and non-one array bounds, and caches the exact minute snapshot with canonical region keys. PostgreSQL result-parity, nonstandard-bound regressions and all six Topology plan cases pass; no migration is added.
Current qualification and promotion gates - 23 October
The full project roadmap, issue/PR disposition and staged promotion plan are in docs #43, tracked by docs #45. Core server #266/#267 and web #182/#183 are one paired candidate; optional server #268/#269, web #184/#185, docs #44 and Flutter #11 remain a separate rollout.
Current application heads pass native CI. Canadaverse now runs server
054e08de7d7fd581e55544e0326ca92596b9a990and web8dcb7f2f233c8ac4086c1ebe02f81d348630321b(2.5.1-n30nex.1), including preview extras. Its complete native frontend suite passed 1,560 tests in 172 files. Public verification matched all 34 assets and both source archives, with both MQTT brokers connected, six retained telemetry histories, and invalid-signature/unenrolled submission rejection. Preview artifacts are local builds of exact clean native-CI-qualified source; this is not a stable release.On a restored snapshot, the committed global Topology query median improved from 1.3691 s to 0.9836 s (28.2%), with the same 9,975 edges. All six unique-traffic raster cases retained identical pixels. These are query/CPU-raster checks, not browser/GPU FPS or production throughput measurements. The later public 24h sample returned the same 9,346 links in 5.619 s cold and 0.191 s warm; earlier severe shared-host I/O pressure caused timeouts. Cold-load capacity, target-browser/device checks and production-shaped migration/recovery soak remain acceptance gates.
Production/dev/dev1 were inspected read-only. Dev1 services are stopped. This preview update changed no schema, retention, intake/worker, credentials or radio ownership. Prior restored-database qualification is recorded separately; no fresh browser or physical mobile/background qualification is claimed here. Dependency advisories and the separate Vite #186 candidate still need their own resolution and qualification. No org merge or release approval is implied.