Repository navigation
Conversation
Navigation type handling is informed by PR #2608. Normalize existing history entries before the router reads them and cover Back and Forward without inserting extra entries.
Project report compatibility is informed by PR #2608. Reuse the authorized project query to select its owner before initializing list filters, preserve newest-stack mode and outer hash-link parameters, and keep development redirects on the local origin. Add history, access-denial, and raw-request regressions.
niemyjski
force-pushed
the
feature/svelte-root
branch
from
October 2, 2026 03:13
8a71206 to
c99b5e3
Compare
|
ejsmith
added a commit
that referenced
this pull request
Oct 2, 2026
Adapt selected startup, notification URL, history, and API-origin validation improvements from #2609. Preserve the existing CSP script allowances and verify backward compatibility at the HTTP and browser boundaries. Co-authored-by: Blake Niemyjski <bniemyjski@gmail.com>
niemyjski
added a commit
that referenced
this pull request
Oct 3, 2026
* Replace legacy Angular UI with root Svelte application * Preserve whitespace-tolerant OAuth client configuration * Honor configured API origins and simplify Slack actions * Mock public environment in navigation command tests * Preserve legacy link history and test preview commits * Allow configured API origins in application CSP * Correct the documented Aspire app endpoint * Align email previews with current stack actions * Isolate synthetic notification timing in browser tests * Remove remaining UI transition controls and update contributor guidance * Use configured hosts for service recovery and OAuth navigation * Align development and preview links with the Svelte application * Isolate navigation cache checks from saved-view broadcasts * Wait for saved-view navigation before cache checks * Preserve configured HTTPS enforcement in Svelte * Control stack notification timing in browser stress test * Await real refresh responses and isolate telemetry in browser checks * Separate the public OAuth API origin from the UI redirect * Harden frontend startup and preserve legacy navigation Adapt selected startup, notification URL, history, and API-origin validation improvements from #2609. Preserve the existing CSP script allowances and verify backward compatibility at the HTTP and browser boundaries. Co-authored-by: Blake Niemyjski <bniemyjski@gmail.com> * Provide runtime configuration in Intercom component tests * Clean up frontend guidance and move API origin helper * Restore email stack actions with Svelte confirmations * Simplify UI cutover and preserve notification actions --------- Co-authored-by: Blake Niemyjski <bniemyjski@gmail.com>
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
1 participant
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.
Retires Angular and its build, Aspire, Docker, editor and devcontainer wiring. Publishes the existing Svelte app at
/, with local/nextcompatibility redirects and separate API/docs/MCP/health/asset routing. Both Docker entrypoints only remove the second frontend block; the existing Svelte config script replaces the Angular writer with root asset paths and credential logging removed.Updates notification, assistant and auth links to the Svelte routes, preserving report organization/project scope and confirmation for stack actions. Removes the old-site notification selector while preserving stored audience values and dismissals. Existing config names/defaults and settings routes remain; OAuth authorization redirects to the configured app origin for separate API hosts. Its OpenAPI snapshot is updated.
Verification:
c99b5e335: 3,150 backend tests passed (3 existing skips), 125 browser tests passed with zero retries, all 1,022 frontend unit tests passed, and lint/type/build, four Docker targets, aggregate completeness/coverage and CLA checks passed. Live third-party sign-ins were not exercised.Breaking change: Angular UI retirement. Draft only; no merge/deployment. CSP PR #2363 remains separate.