chore: remove unused code from snippets, script, and styles#46
Conversation
Dead-code cleanup with no runtime/behavior impact: - scripts/sync-default-configs.mjs: drop unused `execSync` import (only import from node:child_process; the script never spawns a process). - snippets/ecosystem-contracts.jsx: remove the `seiScanStyle` variable, which was declared but never read (the SeiScan link builds its own style object). - style.css: remove the unused custom MDX author utilities `.sei-icon-square` (+ `.gold`) and `.sei-meta` (+ `.dark` variant) — no references in any .mdx, snippet, or docs.json. `.sei-eyebrow` is kept (its rule is shared with the `[class*="eyebrow"]` selector) and `.sei-eco-grid` is kept (in use). All snippets/scripts still parse (esbuild / node --check) and style.css braces remain balanced. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
PR SummaryLow Risk Overview
Reviewed by Cursor Bugbot for commit acfe396. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
There was a problem hiding this comment.
Clean, correctly-scoped dead-code cleanup: removes an unused import, an unread variable, and unreferenced CSS utilities (0 additions, 38 deletions). I independently confirmed none of the removed symbols (execSync, seiScanStyle, .sei-icon-square, .sei-meta) are referenced anywhere in the repo, so no behavior changes.
Findings: 0 blocking | 3 non-blocking | 0 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- Both second-opinion passes produced no output:
codex-review.mdandcursor-review.mdare empty, so no external findings were merged. REVIEW_GUIDELINES.mdis empty, so no repo-specific standards could be applied.- As the PR notes, the generated
llms-full.txtstill referencesseiScanStyle; that's expected since it regenerates from live endpoints post-deploy, but worth confirming the regeneration actually runs so the stale reference doesn't linger.
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
What is the purpose of the change?
Housekeeping — deletes stale/dead code (unused variables, an unused import, and unreferenced CSS utilities) to streamline the codebase. No documentation content, rendering, or runtime behavior changes.
Describe the changes to the documentation
This is a pure dead-code cleanup across the repo's non-content code surfaces. Each removal was verified to have zero usages before deletion:
scripts/sync-default-configs.mjs— dropped the unusedimport { execSync }. It was the only import fromnode:child_process; the script only reads/writes files and never spawns a process.snippets/ecosystem-contracts.jsx— removed theseiScanStylevariable. It was declared but never read; the SeiScan link button builds its own style object.style.css— removed the unused custom MDX author utilities.sei-icon-square(+.goldmodifier) and.sei-meta(+.darkvariant). These had zero references in any.mdx, snippet, ordocs.json.Deliberately kept:
.sei-eyebrow— its rule is shared with the[class*="eyebrow"]attribute selector, so it can't be removed cleanly..sei-eco-grid— actually in use bysnippets/ecosystem-app-grid.jsx.Notes
.jsxfiles validated with esbuild, all.mjs/.jswithnode --check, andstyle.cssparses with balanced braces (136/136) — before and after the changes.rpc-methods-viewer.jsxandchangelog.jsx) found no further dead code; the codebase was already well-maintained.package.jsonexists (Mintlify manages deps), so there were no unused packages to prune.llms-full.txtstill referencesseiScanStyle, but that file is a generated artifact sourced from the livedocs.sei.io.mdendpoints — it refreshes on its normal post-deploy regeneration cycle, not from local source, so it is intentionally left untouched here.🤖 Generated with Claude Code