Repository navigation
browser: streamline Memory Hub navigation and harden CI tests - #3275
Hillary Mutisya (hillary-mutisya) merged 2 commits into
Conversation
Move empty and non-fitting view types into an accessible overflow menu while preserving scoped inventory, selection and keyboard focus. Hide static pagination and distinguish independent view and build issue feeds in Inbox. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
CI follow-up for
Other CI is still running and human approval remains required. This is not a green/merge-ready claim; the bounded observation window has ended. |
Publish complete checkpoint JSON atomically without overwriting existing checkpoints, and cover partial-write visibility and exclusive publication. Keep workflow CLI tests non-interactive and reject abnormal subprocess termination instead of reporting exit zero, with a timeout regression and unchanged default deadlines. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
CI repairs are pushed in
New run. No failed or cancelled checks were reported on this head at the end of observation. The remaining matrix checks are waiting in GitHub's queue, not failing in the repaired test phases; no blind reruns or workflow changes were made. Local validation also passed: full build, 349 memory-quality tests, 12 CLI tests, 516 browser client tests, pinned formatting and all four committed-HEAD PR gates. The 30-minute observation window has ended. Next action is to let the six queued summary jobs execute and confirm their results; human approval remains required. This is not a claim that all required checks are green yet. |
Summary
View issuesandBuild issuespagers when navigation is needed.Motivation
The gallery previously showed tabs for types with no saved instances and needed horizontal scrolling on narrow screens. Inbox Views also showed two indistinguishable, disabled page-one controls. Both controls belonged to that panel's real saved-view and build-receipt feeds; neither was a hidden-module leak.
Behavior and configuration
Reviewer guide
Fixture updates respect requested view kinds and allow the additional asynchronous presence requests to settle; no tests are skipped.
Validation
Windows, Node 26.13.0, standalone pnpm 11.9.0; initial UX commit
16a917a82; final CI-repair qualification is recorded below.pnpm run clean, thenpnpm run buildfromts; full build rerun after final focus fixpnpm --filter @typeagent/browser run test:views:clientpnpm run prettier:changed:fix; no changes producedgit diff --checkpnpm run code-lint -- --ratchet --base origin/mainpnpm run code-complexity -- --ratchet --base origin/main --cyclomatic 25 --cognitive 30 --new-file-cyclomatic 25 --new-file-cognitive 30pnpm run code-circular -- --ratchet --base origin/main --exceptions-file tools/scripts/code/circular-baseline-exception.jsonpnpm run code-debt -- --gate --base origin/mainAll four PR gates ran against the committed final HEAD, not just uncommitted edits.
UI verification
The browser checks used existing saved content; deterministic fixtures are engineering regressions, not live-model quality evidence.
Risks and rollback
The additional scoped existence requests occur on initial load or invalidation, not each ordinary filter/search refresh. Resize/font observers and menus are disposed on rerender, scope changes and teardown. Product changes are client-only; CI follow-up also hardens test/qualification harnesses. No data migration or corpus mutation is required.
Remote CI and human review are separate from the local results above.
CI repair follow-up (
9a860cc68)The original six
build_tsfailures were aggregate coverage gates. Their underlying failures were incomplete recovery checkpoint JSON on Ubuntu/macOS Node 24, and an Ubuntu Node 24 workflow CLI child timeout reported as exit zero by its test helper. The runner-image migration notices were informational.Reviewer entry points:
ts/tools/memory-maintenance-qualification/longmemeval-indexing-storage.mjs, its indexing test, andts/examples/workflow/cli/test/cli.spec.ts.Final local validation on
9a860cc68:pnpm run build: passed.pnpm run test:memory-quality: 349 tests passed, zero skipped.pnpm --filter workflow-cli test:local: 12 tests passed, zero skipped.origin/main(lint, complexity, circular dependencies and test debt).The new-head CI run and human review remain separate from these local results. No merge or automatic merge is requested.