feat(admin): persist memory subtab in URL as ?memsub= param

[mattmezza]

Summary

Memory subtabs (Settings | Long-term | Short-term) now sync their active state to a ?memsub= URL param, enabling bookmarking, refresh persistence, and browser back/forward support.

Changes

base.html

• Added memorySubtab() Alpine component (pattern-matched from vaultTabs())
• Reads ?memsub= from URL on init, falls back to localStorage, defaults to settings
• On subtab selection: updates URL via replaceState, saves to localStorage, fires HTMX to load the new view

dashboard.html (adminTabs())

• Forwards ?memsub= param on initial memory tab load
• Forwards ?memsub= when switching to the memory tab from another tab
• Always re-loads memory tab on popstate (browser back/forward restores correct subtab)
• Preserves ?memsub= in URL when switching between memory and other tabs

memory.html

• Replaced plain HTMX hx-get subtab buttons with Alpine-managed ones
• Active state now uses Alpine :class bindings instead of Jinja {% if %}
• Nav rendered once before the view-specific content blocks (no more duplication)
• Pagination/search buttons continue to work as before (they stay within the same subtab)

Testing

• Visit /admin?tab=memory&memsub=long-term directly → lands on Long-term tab
• Click subtabs → URL updates, bookmarking works
• Browser back/forward → correct subtab restored
• Page refresh → last selected subtab preserved
• Switch to another tab and back → memsub param preserved

Fixes the 'long-term subtab not clickable' issue by moving from HTMX-driven to Alpine-driven navigation.

[kindralai]

Workflow run #281 → failure. One test needs updating.

test_sub_tab_nav_buttons_link_correctly asserts hx-get="/partials/memory?view=settings" etc. on the sub-tab buttons, but the PR replaced plain HTMX hx-get buttons with Alpine-managed @click="select(...)" buttons. The test needs to check for Alpine attributes instead.

Will push a fix.

[kindralai]

Pushed a fix commit e1a563a — test_sub_tab_nav_buttons_link_correctly now asserts Alpine @click handlers and :class bindings instead of the old hx-get attributes.

The rendered memory.html nav:

<div x-data="memorySubtab()" x-init="init()" ...>
  <button ... @click="select(settings)">Settings</button>
  <button ... @click="select(long-term)">Long-term</button>
  <button ... @click="select(short-term)">Short-term</button>
</div>

The test now checks for:

• x-data="memorySubtab()" and x-init="init()" on the nav wrapper
• @click="select(settings/long-term/short-term)" on each button
• :class bindings with tab-link-active

[kindralai]

Updated 7 integration tests for the lazy-loaded sub-tab architecture:

test_admin_ui.py (4 tests)

• test_llm_partial_has_subtabs: Alpine wrapper assertions stay on /partials/llm; "Context compaction" now fetched from /partials/llm/history
• test_llm_partial_has_vision_fallback → hits /partials/llm/inference
• test_llm_partial_has_reply_decision → hits /partials/llm/inference
• test_llm_partial_has_openrouter → hits /partials/llm/providers

test_secrets_vault.py (3 tests)

• test_add_and_list_secret_via_admin: POST creates secret (status 200), then GET /partials/secrets/agents to verify listing
• test_llm_tab_collapses_vaulted_key: hits /partials/llm/providers instead of wrapper
• test_bitwarden_import_parse_and_commit: removed "Site" assertion (import sub-tab template renders only the upload form); commit + verify still works

[kindralai]

Updated 8 more tests in the core chunk:

test_admin_api.py (2 tests)

• test_partial_jobs_hides_completed_by_default → /partials/jobs/scheduled
• test_partial_jobs_show_completed_reveals_done → /partials/jobs/scheduled?show_completed=true

test_memory_subtabs.py (5 tests + 2 pagination)

• All ?view= based tests updated to use /partials/memory/settings, /partials/memory/long-term, /partials/memory/short-term

test_memory_admin.py (1 test)

• test_memory_partial_renders → /partials/memory/settings