Skip to content

chore: remove unused code from snippets, script, and styles#46

Merged
alexander-sei merged 1 commit into
mainfrom
chore/remove-unused-code
Jul 15, 2026
Merged

chore: remove unused code from snippets, script, and styles#46
alexander-sei merged 1 commit into
mainfrom
chore/remove-unused-code

Conversation

@alexander-sei

Copy link
Copy Markdown
Collaborator

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 unused import { execSync }. It was the only import from node:child_process; the script only reads/writes files and never spawns a process.
  • snippets/ecosystem-contracts.jsx — removed the seiScanStyle variable. 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 (+ .gold modifier) and .sei-meta (+ .dark variant). These had zero references in any .mdx, snippet, or docs.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 by snippets/ecosystem-app-grid.jsx.

Notes

  • Net: 3 files changed, 38 deletions, 0 additions.
  • Verification: all 11 snippet .jsx files validated with esbuild, all .mjs/.js with node --check, and style.css parses with balanced braces (136/136) — before and after the changes.
  • A broader sweep of the other 13 JS files (including the large rpc-methods-viewer.jsx and changelog.jsx) found no further dead code; the codebase was already well-maintained.
  • No tracked package.json exists (Mintlify manages deps), so there were no unused packages to prune.
  • llms-full.txt still references seiScanStyle, but that file is a generated artifact sourced from the live docs.sei.io .md endpoints — it refreshes on its normal post-deploy regeneration cycle, not from local source, so it is intentionally left untouched here.

🤖 Generated with Claude Code

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>
@cursor

cursor Bot commented Jul 15, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Pure deletion of unused code; no changes to docs rendering, scripts’ behavior, or user-facing styling paths.

Overview
Dead-code cleanup only — 38 lines removed across three files with no behavior or docs content changes.

scripts/sync-default-configs.mjs drops the unused execSync import from node:child_process; the script only reads/writes files and never shells out.

snippets/ecosystem-contracts.jsx removes the unused seiScanStyle object. SeiScan links already style themselves via SeiScanLink’s baseStyle / hover state.

style.css deletes unreferenced MDX helper classes .sei-icon-square (and .gold) and .sei-meta (plus .dark variant). Other Sei utilities such as .sei-eyebrow and .sei-eco-grid stay because they are still in use.

Reviewed by Cursor Bugbot for commit acfe396. Bugbot is set up for automated code reviews on this repo. Configure here.

@mintlify

mintlify Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
sei-docs 🟢 Ready View Preview Jul 15, 2026, 7:23 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.md and cursor-review.md are empty, so no external findings were merged.
  • REVIEW_GUIDELINES.md is empty, so no repo-specific standards could be applied.
  • As the PR notes, the generated llms-full.txt still references seiScanStyle; 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.

@mintlify

mintlify Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
sei-docs 🟡 Building Jul 15, 2026, 7:22 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@alexander-sei alexander-sei merged commit f2d915a into main Jul 15, 2026
15 checks passed
@alexander-sei alexander-sei deleted the chore/remove-unused-code branch July 15, 2026 21:30
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