Skip to content

fix(node-ui): rebuild verifiable memory panel#601

Merged
Jurij89 merged 3 commits into
mainfrom
codex/cgux-m4-vm-stats-panel
May 23, 2026
Merged

fix(node-ui): rebuild verifiable memory panel#601
Jurij89 merged 3 commits into
mainfrom
codex/cgux-m4-vm-stats-panel

Conversation

@Jurij89
Copy link
Copy Markdown
Contributor

@Jurij89 Jurij89 commented May 22, 2026

Summary

  • Rebuilds the Verifiable Memory hero so the stat row contains numeric stats only and the Context Graph id moves into subtitle metadata.
  • Adds a purposeful empty-VM state for zero Knowledge Assets and suppresses the duplicate generic layer/list empty widgets while preserving caller footers.
  • Replaces the mismatched green-glow/stat/chip styling with one restrained, theme-aware border/surface treatment and adds focused component/integration coverage.

Scope / Gates

  • Covers first-wave PR 1.6 / M4 only.
  • Does not add S10 VM metadata, trust-level breakdowns, chain/token fields, provenance, or publish-flow assumptions.
  • G-Q14/Q16 remains a narrow populated-VM re-check once a node with published VM Knowledge Assets is available; the empty-state/craft fix is fully verifiable now.

Verification

  • pnpm --filter @origintrail-official/dkg-node-ui exec vitest run test/vm-hero-banner.test.ts
  • pnpm --filter @origintrail-official/dkg-node-ui exec vitest run test/vm-hero-banner.test.ts test/ui-compat.test.ts test/use-memory-entities-live-updates.test.ts test/use-memory-entities-partial.test.ts
  • pnpm --filter @origintrail-official/dkg-node-ui run build
  • pnpm --filter @origintrail-official/dkg-node-ui run build:ui
  • git diff --check
  • Branch-local Vite browser smoke at http://localhost:5176/ui/ with token injected from node-served UI: Hello World VM empty state rendered, no duplicate generic empty widgets, no console/page/API errors.
  • 900x500 resize probe: VM hero had no internal text overflow.
  • Light-theme contrast spot-check after theme-aware surface fix: badge ~6.2:1, headline/title ~15.6:1, context ~4.6:1, body ~8.5:1.

Local Review

  • Temporary local reviewers completed three rounds using .codex/review-prompt.md and .codex/review-schema.json.
  • Fixed valid findings: empty-VM branch preserved footer, LayerContent empty-VM behavior gained coverage, and VM hero styling became light-theme safe.
  • Final review round returned {comments:[]}.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Codex review completed — no issues found.

@Jurij89 Jurij89 force-pushed the codex/cgux-m4-vm-stats-panel branch from e088fde to 297b19a Compare May 23, 2026 16:52
Comment thread packages/node-ui/src/ui/views/project/components.tsx Outdated
Comment thread packages/node-ui/src/ui/views/project/components.tsx Outdated
Comment thread packages/node-ui/src/ui/views/project/components.tsx Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Codex review completed — no issues found.

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