Repository navigation
Conversation
A13xB0
force-pushed
the
feat/2142-scope-transport-ui
branch
from
October 10, 2026 00:11
7d14cb3 to
a240157
Compare
Owner
Author
|
Change after opening (commit
|
A13xB0
force-pushed
the
feat/2142-scope-transport-api
branch
from
October 10, 2026 14:45
c08c502 to
060668b
Compare
A13xB0
force-pushed
the
feat/2142-scope-transport-ui
branch
from
October 10, 2026 14:45
a240157 to
df53e77
Compare
Owner
Author
|
This branch has been updated before going upstream:
It's submitted upstream as Kpa-clawbot#2169. The description here predates those changes; the upstream one is current. |
…t#2142) Adds a Declared / Transport toggle to #/scope-audit. The transport view calls ?mode=transport and lists every repeater seen forwarding in the window: - Seen carrying: one chip per region scope with its packet count; - Declared: the repeater's declared regions coloured by whether they were seen carried (the declared view's chips), or "not asked" when no declared-regions answer exists; - Other traffic: unscoped, unmatched and ambiguous-hop counts. A Region box (suggestions from the scopes seen) adds a Carries column, puts the repeaters not yet carrying the region first, and summarises carriers vs active non-carriers: the rollout report. The window, view and region are in the hash (#/scope-audit?window=7d&mode=transport®ion=be). The page says that "transported" means seen carrying, not configured, and that a quiet region looks like a missing one over a short window. Tests: 9 cases in test-frontend-helpers.js on rendered markup and hash state (net-new UI surface: tests land in the same PR, per TDD.md). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KaR4xu8gVykQBkH8yG8m6H
Follows the API rename: the page requests ?scope=, deep-links #/scope-audit?…&mode=transport&scope=be, and reads "scope" and "carriesScope" from the response. region= is left for the observer filter. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KaR4xu8gVykQBkH8yG8m6H
- one modeQuery helper for the page hash and the API path (S3358, and the two copies were identical) - the carries-scope cell is built before the row (S3358) - localeCompare for the region suggestions (S2871), for-of and .dataset (S4138, S7761), optional chaining (S6582) - object spread in the new helper tests (S6661) No behaviour change; the helper tests are unchanged in what they assert. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KaR4xu8gVykQBkH8yG8m6H
A13xB0
force-pushed
the
feat/2142-scope-transport-api
branch
from
October 10, 2026 17:22
060668b to
bb056a1
Compare
A13xB0
force-pushed
the
feat/2142-scope-transport-ui
branch
from
October 10, 2026 17:22
df53e77 to
1fe44fb
Compare
Owner
Author
|
Merged upstream as Kpa-clawbot#2169 (2026-10-10T19:22:38Z). Closing; master will be synced from upstream. |
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.
Summary
Page half of upstream issue Kpa-clawbot#2142. Stacked on the API PR (
feat/2142-scope-transport-api); review that first. With both, Kpa-clawbot#2142's acceptance points are covered.Adds a Declared / Transport toggle to
#/scope-audit. The transport view calls?mode=transportand lists every repeater seen forwarding in the window:A Region box (suggestions from the scopes seen) adds a Carries X column, lists the repeaters not yet carrying it first, and summarises carriers against active non-carriers: the rollout report the issue asks for. Window, view and region are deep-linked (
#/scope-audit?window=7d&mode=transport&scope=sco) and survive a reload. The page says plainly that transported means seen carrying, not configured, and that over a short window a quiet region looks the same as a missing one. The intro text changes with the view.Browser validation (pre-prod, real data)
Headless Chromium against pre-prod (copy of the production database, live feed):
#/scope-audit?window=24h: the declared view, 16 rows, as before.mode=transport.#ENG-NEin Region: normalised toeng-ne; the hash gains the region (nowscope=eng-ne); summary "385 repeaters seen forwarding in the last 24h · 16 with a declared-regions answer · 51 carried eng-ne · 334 active but not seen carrying it"; the first rows are the non-carriers.Two layout bugs found this way were fixed before opening: the region box inherited
width: 100%from.nodes-search, anddisplay: flexoverrode thehiddenattribute on the region bar.Tests
tests/unit/test-frontend-helpers.jsgains 10 cases on rendered markup and state, using the page's exported internals like the existing scope-audit tests: hash and API paths (and that a region never leaks into the declared view's link), region normalisation (#, capitals, empty), not asked vs declared chips (with the*wildcard), carried-scope chips with counts and the filtered region marked, the carries column only with a region, HTML escaping of names, the summary counts, region suggestions, and the per-view intro. 768 passing in that file.Net-new UI surface, so per TDD.md the tests land in this PR rather than as a preceding red commit. Frontend suites on this branch: 193 of 194; the one failure,
test-issue-1956-release-routing.js, needs CI'sGITHUB_REF_NAMEand fails identically onmasterlocally (fixed by the upstream Kpa-clawbot#2149 PR in this fork). On the fork's combined integration branch (every open PR merged) all 195 pass.Review notes
windowHonestyNote.🤖 Generated with Claude Code
https://claude.ai/code/session_01KaR4xu8gVykQBkH8yG8m6H