Skip to content

chore(app): remove obsolete watchlist configuration and adjust navigation - #165

Merged
TNT-Likely merged 2 commits into
mainfrom
codex/portfolio-navigation-priority
Oct 1, 2026
Merged

TNT-Likely merged 2 commits into
mainfrom
codex/portfolio-navigation-priority

Conversation

@TNT-Likely

@TNT-Likely TNT-Likely commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Background

The portfolio should remain the primary destination after Home. Putting the assistant second gave it more navigation priority than intended.

The repository also retained a YAML example watchlist and unused loaders from an older configuration path. Current application execution reads stocks and Agent bindings from the database.

Changes

  • Order primary navigation as Home, Portfolio, Assistant, Opportunities, and Paper Trading.
  • Apply the order through the shared navigation definition used by desktop and mobile.
  • Delete config/watchlist.yaml and the unused load_watchlist/load_config helpers, along with their unused imports.
  • Retain VERSION, which is consumed by the version API and written during Docker builds.

Validation

  • cd frontend && pnpm exec vitest run: 177 passed across 41 files.
  • cd frontend && pnpm run check:i18n: passed for 126 migrated files.
  • cd frontend && pnpm run build: TypeScript and Vite build passed.
  • git diff --check: passed.
  • Inspected both desktop and mobile consumers of the shared primary navigation array.
  • pytest tests/ -x -q -rs through the isolated temporary SQLite wrapper: 885 passed, 3 skipped after configuration cleanup.
  • pytest packages/marketdata/tests -q: 188 passed.
  • pytest packages/pan-agent-runtime/tests -q: 43 passed.
  • Repository reference search confirms the removed loaders had no callers and the active server watchlist path reads the database.

Boundaries and risks

  • This changes primary navigation ordering and removes an unused repository configuration path. Physical-device visual acceptance is pending.
  • The existing local Node engine and Vite chunk-size warnings remain.
  • Local backend skips cover an unavailable optional OpenTelemetry SDK and two Windows-only checks.

Follow-up

  • Include navigation and configuration cleanup in the prepared 0.17.0 release.

@TNT-Likely TNT-Likely changed the title fix(frontend): prioritize portfolio navigation chore(app): remove obsolete watchlist configuration and adjust navigation Oct 1, 2026
@TNT-Likely
TNT-Likely merged commit f7200ab into main Oct 1, 2026
1 check passed
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