Skip to content

refactor: migrate index + settings routes to Svelte (Phase 4)#70

Merged
setkyar merged 7 commits into
mainfrom
refactor/svelte-index-settings
Jun 7, 2026
Merged

refactor: migrate index + settings routes to Svelte (Phase 4)#70
setkyar merged 7 commits into
mainfrom
refactor/svelte-index-settings

Conversation

@setkyar

@setkyar setkyar commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Migrate the index/sessions route to Svelte: SessionsPage now drives the list, cards, header, home menu, and new-session/projects modals via components, replacing the legacy web/src/index/index.js + sessions-page.js runtime (now reduced to pure sessions.js helpers).
  • Move settings into self-contained Svelte components (AppearanceSettings, LanguageSettings, NotificationSettings, etc.) and the command palette/version modal into shared CommandPalette.svelte + VersionController.svelte, retiring settings.js, session-list-palette.js, and the JS version-modal code.
  • Relocate the load-earlier banner into LoadEarlier.svelte and absorb the chat-preview/live glue into LiveReload.svelte, removing the standalone load-earlier.js and chat-preview.js modules.
  • Net cleanup of ~4.2k lines of legacy rendering JS plus refreshed migration docs and new i18n keys across all built-in locales.

Testing

  • Not run (not requested)

@setkyar setkyar merged commit 720ba87 into main Jun 7, 2026
2 checks passed
@setkyar setkyar deleted the refactor/svelte-index-settings branch June 7, 2026 09:20
Hrt-Htk referenced this pull request in Hrt-Htk/pi-web Jun 13, 2026
Three specs (x7 browsers = 21 failures) were red on main; none are product
regressions:

- index.spec "groups cards by sanitized project path": the index now defaults
  to the timeline layout, so switch to the projects layout before asserting
  .project-group (stale since PR #70).
- issue13-tristate: PR #28 wrapped tool calls in a collapsed
  <details class="actions-group">, so expand it before asserting the nested
  .tool-output.expandable is visible (stale since #28).
- overview-flow-observe: an assertion-free real-pi diagnostic that throws under
  the stub. Rename to *.real.spec.ts and add testIgnore to the stub config so
  it only runs under playwright-real.config.ts.

Also remove leftover new-session investigation debris (the two [ChatComposer]
onMount console.logs and docs/debug/issue-6-debug-log.md); that bug was fixed
and verified in e8cc9eb.

Closes #35

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant