Skip to content

create-rezi: flagship polished dashboard template + dashboard-only scaffolding#95

Merged
RtlZeroMemory merged 5 commits into
mainfrom
feat/dashboard-template-flagship-polish
Feb 19, 2026
Merged

create-rezi: flagship polished dashboard template + dashboard-only scaffolding#95
RtlZeroMemory merged 5 commits into
mainfrom
feat/dashboard-template-flagship-polish

Conversation

@RtlZeroMemory

@RtlZeroMemory RtlZeroMemory commented Feb 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • ship a polished, product-style dashboard starter as the flagship create-rezi template
  • refine dashboard UX for stable animation cadence, responsive controls, fixed-width dynamic labels, and coherent help modal styling
  • update template docs and top-level docs/README to reflect dashboard-first scaffolding
  • keep create-rezi template catalog dashboard-only for now (dashboard, alias dash)
  • remove legacy starter template directories (form-app, file-browser, streaming-viewer)

Notes

Validation

  • npm run build
  • npm run check:create-rezi-templates
  • npm run check:docs

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 462df1f40e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +1589 to +1591
if (raw.key === 81 || raw.key === 113 || raw.key === 67 || raw.key === 99) {
stopDashboardAction();
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Remove app.stop calls from onEvent handlers

Calling stopDashboardAction() from app.onEvent causes app.stop() to run while createApp is inside an event handler, which violates stop()'s reentrancy guard (assertNotReentrant) and throws ZRUI_REENTRANT_CALL; in normal usage this path is hit by quit inputs (q/Ctrl+C), so the dashboard can enter a fatal error path instead of shutting down cleanly.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Fixed in 8160b2c: shutdown now uses requestDashboardStop() which defers app.stop() via setTimeout(0) and deduplicates stop requests. This avoids reentrant stop() calls from onEvent/key handlers.

@RtlZeroMemory
RtlZeroMemory merged commit 341ee58 into main Feb 19, 2026
17 checks passed
@RtlZeroMemory
RtlZeroMemory deleted the feat/dashboard-template-flagship-polish branch February 19, 2026 17:45
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