Skip to content

Refactor web app orchestration and import pipeline - #14

Merged
dills122 merged 2 commits into
mainfrom
feature/web-app-orchestration-cleanup
Jun 7, 2026
Merged

dills122 merged 2 commits into
mainfrom
feature/web-app-orchestration-cleanup

Conversation

@dills122

@dills122 dills122 commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Splits the web app shell into focused state, refresh, export, and theme helpers.
  • Extracts repository import behavior out of the browser Worker message handler.

What Changed

  • Reduced ForageApp.svelte by moving app state/display helpers into app-state.ts.
  • Added app-refresh.ts to assemble local IndexedDB and Worker session/config state into UI state patches.
  • Added export-data.ts and theme.ts for export/download and theme behavior.
  • Added import-pipeline.ts so the import Worker delegates repository import orchestration.
  • Added import-messages.ts for import progress and terminal message formatting.
  • Added focused tests for the extracted app state, refresh, and import message helpers.

Validation

  • pnpm --filter @forage/web test
  • pnpm --filter @forage/web typecheck
  • pnpm --filter @forage/web build
  • pnpm lint
  • pnpm check

Scope Notes

  • This is behavior-preserving cleanup only.
  • The Worker auth/server god-file split is intentionally left for a separate security-sensitive refactor branch.

@dills122
dills122 merged commit 41c6445 into main Jun 7, 2026
1 check passed
@dills122
dills122 deleted the feature/web-app-orchestration-cleanup branch June 7, 2026 03:34
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