Skip to content

chore: Split Internet Identity canister frontend#3536

Merged
sea-snake merged 28 commits intomainfrom
arshavir/split-fe
Feb 20, 2026
Merged

chore: Split Internet Identity canister frontend#3536
sea-snake merged 28 commits intomainfrom
arshavir/split-fe

Conversation

@aterga
Copy link
Copy Markdown
Collaborator

@aterga aterga commented Dec 10, 2025

This PR separates the Internet Identity canister frontend into a dedicated canister to better isolate orthogonal concerns and simplify the architecture.

Changes

  • Introduced a new internet_identity_frontend canister that serves static assets with HTTP certification
  • Updated build scripts to support building the new frontend canister independently

Testing

  • Enabled playwright e2e tests for the new architecture (existing II BE; new / separate II FE)
  • New tests are not yet enabled in CI, but one can run them manually as follows:
SEPARATE_FRONTEND_CANISTER=1 TLS_DEV_SERVER=1 NO_HOT_RELOAD=1 npm run dev

@aterga aterga requested a review from Copilot December 16, 2025 16:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR separates the Internet Identity canister frontend into a dedicated canister to better isolate orthogonal concerns and simplify the architecture.

  • Introduces a new internet_identity_frontend canister that serves static assets with HTTP certification
  • Updates build scripts to support building the new frontend canister independently
  • Adds new localization strings for recovery phrase verification and in-app browser workflows

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/internet_identity_frontend/src/main.rs New canister implementation for serving certified static assets with security headers and CSP
src/internet_identity_frontend/internet_identity_frontend.did Candid interface definition for HTTP request/response handling
src/internet_identity_frontend/README.md Documentation explaining the new frontend canister's purpose
src/internet_identity_frontend/Cargo.toml Dependencies for the new frontend canister
src/internet_identity/Cargo.toml Pin rand dependency versions to specific releases
src/frontend/src/lib/locales/*.po Add new translation strings for recovery phrase verification and browser navigation
scripts/build Add build support for the new frontend canister
dfx.json Configure deployment of the new frontend canister
Cargo.toml Add frontend canister to workspace and upgrade certification libraries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/internet_identity_frontend/src/main.rs Outdated
Comment thread src/internet_identity_frontend/Cargo.toml Outdated
Comment thread src/internet_identity_frontend/Cargo.toml Outdated
@aterga aterga marked this pull request as ready for review February 20, 2026 14:15
@aterga aterga requested a review from sea-snake February 20, 2026 14:15
Copy link
Copy Markdown
Contributor

@sea-snake sea-snake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sea-snake sea-snake added this pull request to the merge queue Feb 20, 2026
Merged via the queue into main with commit 2359dda Feb 20, 2026
124 of 125 checks passed
@sea-snake sea-snake deleted the arshavir/split-fe branch February 20, 2026 18:20
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.

4 participants