Skip to content

Optimize honeycomb sigil rendering with lightweight glyphs#175

Merged
kojibai merged 6 commits intopulse-atlas-expand-to-honeycombfrom
codex/optimize-sigil-honeycomb-explorer-rendering
Jan 1, 2026
Merged

Optimize honeycomb sigil rendering with lightweight glyphs#175
kojibai merged 6 commits intopulse-atlas-expand-to-honeycombfrom
codex/optimize-sigil-honeycomb-explorer-rendering

Conversation

@kojibai
Copy link
Copy Markdown
Owner

@kojibai kojibai commented Jan 1, 2026

Motivation

  • Reduce load and paint cost of the Sigil Honeycomb explorer by avoiding the heavy per-cell sigil renderer and proof work.
  • Present small glyphs using simple color and shape so the comb is fast to render while preserving chakra tint cues.
  • Keep visual parity (shape + tint) without full SVG/glyph computation to restore the explorer's snappy feel.
  • Limit runtime work and memory used for rendering thousands of hex cells.

Description

  • Replaced the heavy KaiSigil rendering inside honeycomb cells with a lightweight placeholder element by removing the import and usage from SigilHoneycombExplorer.tsx and rendering a div.sigilHexGlyphSimple instead.
  • Removed the deferred render cache (SIGIL_RENDER_CACHE) and the useDeferredSigilRender scheduling logic to simplify rendering logic and eliminate idle-callback scheduling.
  • Updated SigilHoneycomb.css to add .sigilHexGlyphSimple styling and chakra tint variables (e.g. .chakra-root, .chakra-sacral, etc.) to show colored, clipped shapes with a small drop shadow instead of full glyph SVGs.
  • Changes are contained to src/components/SigilExplorer/SigilHoneycombExplorer.tsx and src/components/SigilExplorer/SigilHoneycomb.css and preserve existing selection/interaction behavior.

Testing

  • Started the dev server with npm run dev -- --host 0.0.0.0 --port 4173 and the server started successfully.
  • Ran a Playwright script to load http://127.0.0.1:4173/explorer and capture a screenshot which completed successfully.
  • No unit tests or TypeScript build runs were triggered as part of this rollout.
  • Manual visual verification was captured via the screenshot artifact produced during the run.

Codex Task

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
phi-network-5aii Ready Ready Preview, Comment Jan 1, 2026 10:13am

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector 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

Here are some automated review suggestions for this pull request.

ℹ️ 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 365 to 367
.chakra-crown { --hex-tint: 210 200 255; }
.chakra-unknown { --hex-tint: 120 190 255; }
}
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 stray closing brace to avoid CSS parse failure

There is an unmatched } after .chakra-unknown; Vite/PostCSS treats this as a CSS syntax error, which can fail the build or cause subsequent rules (like .sigilHex:hover) to be dropped. This will surface when running the dev server or build pipeline that parses CSS strictly, so the extra brace should be removed to keep the stylesheet valid.

Useful? React with 👍 / 👎.

@kojibai kojibai merged commit 4414351 into pulse-atlas-expand-to-honeycomb Jan 1, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant