Skip to content

refactor: remove name-indexed views from orchestrator template context - #290

Merged
peintnermax merged 1 commit into
worktree-adr-021-ordered-arraysfrom
fix-ordered-arrays-cleanup
Jun 15, 2026
Merged

refactor: remove name-indexed views from orchestrator template context#290
peintnermax merged 1 commit into
worktree-adr-021-ordered-arraysfrom
fix-ordered-arrays-cleanup

Conversation

@peintnermax

Copy link
Copy Markdown
Member

1. Remove *_by_name maps — use Liquid where filter instead

Templates now use {% assign passkey = actions | where: "name", "passkey" | first %} instead of actions_by_name.passkey. This eliminated the orchestrator-built lookup maps from template-context.ts, zitadel-login.ts, both liquid templates, and liquid.spec.ts.

2. Align mock registration with the default flow definition

The mock now has a two-step registration matching the default flow definition:

  • Register step: email, given_name, family_name, date_of_birth fields + submit/passkey_register/sign_in actions
  • Register-password step (new): password field + submit action

Changed files: login.ts fixtures, flow-machine.ts, handlers.ts, index.spec.ts.

All component tests (156) and mock tests (26) pass.

@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nextgen Ready Ready Preview, Comment Jun 15, 2026 12:27pm

Request Review

@peintnermax
peintnermax marked this pull request as ready for review June 15, 2026 12:27
@peintnermax
peintnermax merged commit 0a475a5 into worktree-adr-021-ordered-arrays Jun 15, 2026
15 of 16 checks passed
@peintnermax
peintnermax deleted the fix-ordered-arrays-cleanup branch June 15, 2026 12:35
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.

2 participants