Introduce PostgresDeploymentCompatibilityStore with migration 011, in-memory
fallback, deployment endpoints, and Postgres fixture for integration tests.
Update Scheduler repository with connection policy adoption.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Introduce PostgresGraphSavedViewStore with SQL migration, in-memory fallback,
CompatibilityEndpoints for UI contract alignment, and integration tests
with a shared Postgres fixture.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Introduce IntegrationDiscovery DTOs, GitLabCiConnectorPlugin,
GitLabContainerRegistryConnectorPlugin, CLI integrations command group,
and expand impact/service test coverage for all connector plugins.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Major edge routing improvements including corridor spacing, crossing reduction,
focused gateway boundary repairs, setter families, and advanced restabilization.
Adds workflow renderer tests for document-processing and artifact inspection.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Introduce GatewayRegistrationResyncService to recover stale registrations,
extract IGatewayTransportClient interface, add EndpointsUpdate and
RegistrationResyncRequest frame types, and expand transport test coverage.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Set ClientName on every Redis/Valkey connection across Scanner, Signals,
Concelier, Notify, Scheduler, Timeline, and Router for easier connection
attribution in monitoring.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Extract connection string building into PostgresConnectionStringPolicy so all
services use consistent pooling, application_name, and timeout settings.
Adopt the new policy in 20+ module DataSource/ServiceCollectionExtensions classes.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When the corridor reroute pushes a horizontal segment away from a
blocking node, preserve the first point (source connection) and
insert a vertical step to reconnect the last point (target connection)
at the original Y. Previously, pushing all points uniformly would
disconnect the edge from its target node when the push Y exceeded
the target node's boundary.
Fixes edge/9 (Retry Decision → Set batchGenerateFailed) which was
pushed to Y=653 but the target node bottom is at Y=614 — the endpoint
now steps back up to Y=592 to reconnect.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fix for the 2 remaining OIDC redirect failures: after login, the
page lands on Dashboard. When a test calls page.goto('/setup/...'),
Angular sometimes redirects back to Dashboard because the auth guard
hasn't settled.
Fix: After loginAndGetToken, navigate to /setup/integrations and
wait for [role="tab"] to render. This:
1. Settles the OIDC auth guard (validates token, caches auth state)
2. Lazy-loads the integration module chunk
3. Primes Angular's router with the /setup/ route tree
Subsequent page.goto() calls from tests will work reliably because
Angular already has auth state and the lazy chunk is cached.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Three-layer edge-node clearance improvement:
1. A* proximity cost with correct coordinates: pass original (uninflated)
node bounds to ComputeNodeProximityCost so the pathfinder penalizes
edges near real node boundaries, not the inflated obstacle margin.
Weight=800, clearance=40px. Grid lines added at clearance distance
from real nodes.
2. Default LayerSpacing increased from 60 to 80, adaptive multiplier
floor raised from 0.92 to 1.0, giving wider routing corridors
between node rows.
3. Post-pipeline EnforceMinimumNodeClearance: final unconditional pass
pushes horizontal segments within 8px of node tops (12px push) or
within minClearance of node bottoms (full clearance push).
Also: bridge gap detection now uses curve-aware effective segments
(same preprocessing + corner pull-back as BuildRoundedEdgePath) so
gaps only appear at genuine visual crossings. Collector trunks and
same-group edges excluded from gap detection.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Landing page: check for tabs/heading instead of waiting for redirect
(redirect needs loadCounts XHR which is slow from browser)
- Pagination: merged into one test, pager check is conditional on data
loading (pager only renders when table has rows)
- Wizard step 2: increased timeouts for Harbor selection
Also: Angular rebuild was required (stale 2-day-old build was the
hidden blocker for 15 UI tests).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Root cause found via screenshot: page.goto with domcontentloaded
returned before Angular even bootstrapped — the page still showed
Dashboard while the test checked for integration content.
Fix: Change waitUntil from domcontentloaded to load across all 37
goto calls. 'load' waits for initial JS/CSS to load, meaning Angular
has bootstrapped and the SPA router has processed the route.
Simplified waitForAngular to wait for route-level content selectors
without the URL check (the load event handles that now).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Problem: After waitForAngular, content assertions ran before Angular's
XHR data loaded. Tests checked textContent('body') at a point when
the table/heading hadn't rendered yet.
Fix: Replace point-in-time checks with Playwright auto-retry assertions:
- expect(locator).toBeVisible({ timeout: 15_000 }) — retries until visible
- expect(locator).toContainText('X', { timeout: 15_000 }) — retries until text appears
- expect(rows.first()).toBeVisible() — retries until table has data
Also: landing page test now uses waitForFunction to detect Angular redirect.
10 files changed, net -45 lines (simpler, more robust assertions).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The generic waitForAngular matched the sidebar nav immediately but
route content (tables, tabs, forms) hadn't rendered yet.
Updated waitForAngular selector to wait for route-level elements:
stella-page-tabs, .integration-list, .source-catalog, table tbody tr,
h1, [role=tablist], .detail-grid, .wizard-step, form.
Also fixed activity-timeline and pagination tests (still had
waitForTimeout(2_000) instead of waitForAngular).
Increased fallback timeout from 5s to 8s for slow-loading pages.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The 3s waitForTimeout after page.goto wasn't enough for Angular to
bootstrap and render content. Replace with waitForAngular() helper
that waits for actual DOM elements (nav, headings) up to 15s, with
5s fallback.
32 calls updated across 10 test files.
Also adds waitForAngular to helpers.ts export.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Root cause found via diagnostics: the handler call at 16:27:19 never
returned. Guard: processing message X logged, but Guard: processed
never appeared. The 55s CancellationToken fired but the handler
ignored it (blocked on a non-cancelable StackExchange.Redis operation
or DB query that uses its own timeout).
Fix: Replace await handler(token) with handler(token).WaitAsync(token).
WaitAsync returns when EITHER the handler completes OR the token fires,
regardless of whether the handler cooperatively checks the token.
The abandoned handler continues in background but the consumer loop
resumes immediately.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The advisory source API tests go through the Valkey transport with
withRetry (3 attempts). With the 55s transport timeout, worst case
is 3 × 55s = 165s, exceeding the default 120s test timeout.
Set advisory lifecycle describe block to 300s via beforeEach to
give enough headroom for all retry attempts.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The white "cut" marks at edge crossings are distracting at small
rendering scales and make edges look broken/disconnected. Simple
overlapping crossings are cleaner and more readable.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When a backtrack's return point is the endpoint (last point), remove
BOTH the overshoot and return — the pre-overshoot point becomes the
new endpoint. This prevents the rendered path from bending inside the
target node after backtrack removal.
edge/4 now ends cleanly at the Join's left face instead of bending
UP inside the node.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Root cause: The consumer loop processes messages sequentially with
await. One slow handler (e.g., Concelier advisory JOIN taking 30s)
blocks all other messages. Evidence: consumer pending=1, idle=34min,
stream lag=59 messages undelivered.
Fix: Replace sequential foreach with Task.WhenAll for concurrent
processing. Each message gets its own exception guard:
- 55s per-message timeout (below 60s gateway timeout)
- Exception catch-all with retry release
- Graceful shutdown propagation via CancellationToken
- TryReleaseAsync guard prevents failed release from crashing loop
Applied to both server (gateway) and client (microservice) consumer
loops: ProcessRequestsAsync, ProcessResponsesAsync,
ProcessIncomingRequestsAsync.
This is the foundational fix. One slow request should never block
delivery of all other requests to the same service.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When three consecutive points are on the same axis and the middle one
overshoots then returns (e.g., Y goes 170→119→135), remove the
overshoot point. This eliminates the visible inverted-U loop above the
Parallel Execution Join node.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Routing: CollapseShortDoglegs processes one dogleg at a time, accepts
only if no entry-angle/node-crossing/shared-lane regressions.
Rendering: jog filter increased to 30px to catch 19px+24px doglegs
that the routing can't collapse without violations. The filter snaps
the next point's axis to prevent diagonals.
Sharp corners (r=0) for tight doglegs where both segments < 30px.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When both segments at a bend point are under 30px, the curved corner
radius creates a visible S-curve artifact. Using r=0 (sharp 90-degree
corner) eliminates the kink. Smooth curves reserved for longer segments.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Reduces S-curve artifacts on short intermediate segments. The previous
2.5 divisor allowed curves from adjacent bends to overlap on 24px
segments. Divisor 3 gives cleaner curves on short segments.
Remaining visible kink on edge/33 is from the routing's 19px+24px
dogleg near End — needs routing-level fix, not rendering.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Three fixes resolving the cascading test failures:
1. Add withRetry() to integrations.e2e.spec.ts advisory section — the
6 API tests that 504'd on Concelier transport now retry up to 2x
2. Change all UI test page.goto from networkidle to domcontentloaded
across 9 test files — networkidle never fires when Angular XHR
calls 504, causing 30 UI tests to timeout. domcontentloaded fires
when HTML is parsed, then 3s wait lets Angular render.
3. Fix test dependencies — vault-consul-secrets detail test now creates
its own integration instead of depending on prior test state.
New test: catalog page aggregation report — verifies the advisory
source catalog page shows stats bar metrics and per-source freshness
data (the UI we built earlier this session).
Files changed: integrations.e2e.spec.ts, vault-consul-secrets, ui-*,
runtime-hosts, gitlab-integration, error-resilience, aaa-advisory-sync
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
edge/33 had 7-8px jog segments that slipped through the 8px filter.
12px catches all visible kinks while preserving intentional bends.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When removing a <8px jog segment, snap the next point's changed axis
to the previous point's value. Without this, removing the jog creates
a diagonal segment that produces a visible S-curve kink at the
40px corner radius.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
7 tests preventing the silent consumer death bug from recurring:
1. FallbackPollDeliversMessagesWhenPubSubNotFired — verifies messages
arrive via timeout poll even without Pub/Sub notification
2. XAutoClaimRecoversMessagesFromDeadConsumers — verifies XAUTOCLAIM
transfers idle entries from dead consumer instances
3. PendingFirstReadDrainsPendingBeforeNew — verifies pending entries
are processed before new messages
4. ValkeyRestartRecovery — verifies service recovers after Valkey
container restart (uses Testcontainers RestartAsync)
5. SustainedThroughput_30Minutes — 30-min perf test at 1 msg/sec,
asserts p50<1s, p95<15s, p99<30s, zero message loss
[Trait("Category", "Performance")]
6. ConnectionFailedResetsSubscriptionState — verifies ConnectionFailed
event resets _subscribed flag for recovery
7. MultipleConsumersFairDistribution — verifies fair message
distribution across consumer group members
Uses existing ValkeyContainerFixture (Testcontainers.Redis) and
ValkeyIntegrationFact attribute (gated by STELLAOPS_TEST_VALKEY=1).
Run: STELLAOPS_TEST_VALKEY=1 dotnet test --filter "Category!=Performance"
Perf: STELLAOPS_TEST_VALKEY=1 dotnet test --filter "Category=Performance"
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Small boundary adjustment segments (4px, 19px) create weird kinks
when the 40px corner radius is applied. Filter them out before
building the rounded path — connect the surrounding points directly.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
DetectHighwayGroups had a special case for End nodes that included
forward End-targeting edges in highway grouping even when they didn't
share a corridor. This caused edges at different Y levels to be
truncated to a shared collector, destroying their individual paths.
End-targeting edges are already handled by DetectEndSinkGroups (which
now correctly skips groups with no horizontal overlap). Forward
highway detection should only apply to backward (repeat) edges.
All 5 End-targeting edges now render independently with full paths.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Problem: Each test created a new browser context and performed a full
OIDC login (120 logins in a 40min serial run). By test ~60, Chromium
was bloated and login took 30s+ instead of 3s.
Fix: apiToken and apiRequest are now worker-scoped — login happens
ONCE per Playwright worker, token is reused for all API tests.
liveAuthPage stays test-scoped (UI tests need fresh pages).
Impact: ~120 OIDC logins → 1 per worker. Eliminates auth overhead
as the bottleneck for later tests in the suite.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
DetectEndSinkGroups was forming highways for edges at different Y
levels with NO shared corridor. The fallback (line 1585) used
min-MaxX as collector when overlap detection failed, creating a
false highway that truncated individual edge paths.
Fix: skip the group entirely when TryResolveHorizontalOverlapInterval
returns false. Edges at different Y levels render independently.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Edges with bend points above the graph (Y < graphMinY - 10) are
corridor-rerouted and should render independently, not merge into
a shared End-targeting highway. The highway truncation was destroying
the corridor route paths, making edges appear to end before the node.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Corridor vertical drops now land on the target node's actual top
boundary (Y = node.Y) at the clamped X position. Endpoints visually
connect to the node instead of floating near it.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Corridor routes now drop to the ORIGINAL target point (placed by the
router on the actual node boundary) instead of computing a new entry
point on the rectangle edge. Edges visually connect to the End node.
Simplified corridor path: src → stub → corridor → drop to original
target. No separate left-face approach needed.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The 12px quadratic Bezier radius was invisible at rendered scale. 40px
creates visually smooth curves at 90-degree bends, making it easier to
trace edge paths through direction changes (especially corridor drops
and upward approaches to the End node).
Radius auto-clamps to min(lenIn/2.5, lenOut/2.5) for short segments.
Collector edges keep radius=0 (sharp orthogonal).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
QueueWaitTimeoutSeconds: 5 → 10 (base)
Randomization: [base, 2×base] → [base, 3×base] = random 10-30s
When Pub/Sub is alive: instant delivery (no change).
When Pub/Sub is dead: consumer wakes in 10-30s via semaphore timeout,
reads pending + new messages. 30s worst case < 60s gateway timeout.
Load: 30 services × 1 poll per random(10-30s) = ~1.5 polls/sec.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
