Skip to content

[codex] Cache dashboard snapshot - #16

Draft
rushillshah wants to merge 1 commit into
mainfrom
codex/cache-dashboard-snapshot
Draft

[codex] Cache dashboard snapshot#16
rushillshah wants to merge 1 commit into
mainfrom
codex/cache-dashboard-snapshot

Conversation

@rushillshah

Copy link
Copy Markdown
Owner

Summary

  • cache the dashboard vault snapshot in UI state instead of re-reading config during render/menu construction
  • refresh the cached snapshot after child commands and source config writes
  • add a regression guard for render-path config reads

Closes #10

Validation

  • npm run check
  • npm test
  • npm pack --dry-run
  • node ./bin/digital-brain.js ui </dev/null exits with the expected non-interactive terminal message

Risk

Low. The change is limited to terminal dashboard state handling; actions that mutate vault config explicitly refresh the snapshot before re-rendering.

@rushillshah

Copy link
Copy Markdown
Owner Author

Maintenance reconciliation note (2026-06-28): CI is passing and the branch is mergeable, but this PR is still draft and review-required. Next action: mark ready/request review before merge.

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.

Dashboard re-reads vault config from disk on every keypress

1 participant