Skip to content

refactor(desktop): modularize shell renderer#205

Merged
Whiteks1 merged 2 commits intomainfrom
codex/desktop-shell-renderer-modules
Mar 28, 2026
Merged

refactor(desktop): modularize shell renderer#205
Whiteks1 merged 2 commits intomainfrom
codex/desktop-shell-renderer-modules

Conversation

@Whiteks1
Copy link
Copy Markdown
Owner

Summary

This PR:

  • modularizes the QuantLab Desktop renderer by splitting utility, decision-store, and tab-rendering logic into focused ES modules
  • keeps the current shell workflow intact while turning desktop/renderer/app.js into a smaller orchestration layer
  • updates the renderer entrypoint to load as a module and documents the new structure in the desktop README

Why

This slice matters because:

  • the shell renderer had become too large to keep extending safely, and future workflow work would become harder to review and maintain
  • this refactor reduces the amount of product logic concentrated in one file without changing the operator-facing workflow surfaces

Scope

This PR does not:

  • add new product features or alter the shell roadmap
  • change QuantLab or Stepbit backend contracts

Validation

Validated with:

  • node --check desktop/main.js
  • node --check desktop/preload.js
  • node --check desktop/renderer/app.js
  • node --check desktop/renderer/modules/utils.js
  • node --check desktop/renderer/modules/decision-store.js
  • node --check desktop/renderer/modules/tab-renderers.js
  • python -m pytest test/test_research_ui_server.py

Notes

  • The renderer entrypoint now loads via <script type="module">.
  • The biggest extraction targets in this slice were: formatting and path utilities, candidate/shortlist store helpers, and shell-native tab renderers.
  • This PR is intentionally maintainability-first so the next shell feature slices do not keep inflating a single renderer file.

Closes #204

Copy link
Copy Markdown

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @Whiteks1, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@Whiteks1 Whiteks1 merged commit 1ac6e05 into main Mar 28, 2026
2 checks passed
@Whiteks1 Whiteks1 deleted the codex/desktop-shell-renderer-modules branch March 28, 2026 18: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.

refactor(desktop): modularize shell renderer for maintainability

1 participant