Skip to content

improvement(deps): remove unused remark deps#4542

Merged
icecrasher321 merged 1 commit intostagingfrom
improvement/remove-deps
May 9, 2026
Merged

improvement(deps): remove unused remark deps#4542
icecrasher321 merged 1 commit intostagingfrom
improvement/remove-deps

Conversation

@icecrasher321
Copy link
Copy Markdown
Collaborator

Summary

Remove two unused remark deps

Type of Change

  • Other: Code cleanup

Testing

N/A

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 9, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 9, 2026 11:31pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented May 9, 2026

PR Summary

Low Risk
Low risk dependency cleanup that only removes unused packages and updates the lockfile; runtime behavior should be unchanged unless something implicitly relied on these transitive deps.

Overview
Removes the unused Markdown processing dependencies remark-parse and remark-rehype from apps/sim and updates bun.lock accordingly to shrink the dependency set.

Reviewed by Cursor Bugbot for commit 3c0934d. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 9, 2026

Greptile Summary

This PR removes two unused dependencies, remark-parse and remark-rehype, from apps/sim/package.json and the corresponding bun.lock entries. A codebase-wide search confirms neither package is imported anywhere in source files.

  • remark-parse removed: no import of this package found in any source file.
  • remark-rehype removed: no import of this package found in any source file.
  • bun.lock updated: lockfile reflects the dependency removal cleanly.

Confidence Score: 5/5

Safe to merge — removes two packages that are not imported anywhere in the codebase.

The change only removes remark-parse and remark-rehype from the dependency manifest and lockfile. A full codebase search confirms neither package is referenced in any source file, so there is no risk of a runtime or build breakage.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/package.json Removes remark-parse and remark-rehype from dependencies; grep confirms neither package is imported anywhere in source code
bun.lock Lockfile updated to reflect removal of remark-parse and remark-rehype entries

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[apps/sim/package.json] -->|previously declared| B[remark-parse 11.0.0]
    A -->|previously declared| C[remark-rehype 11.1.2]
    A -->|still present| D[remark-breaks]
    A -->|still present| E[remark-gfm]
    B -->|removed - unused| F[❌ Deleted]
    C -->|removed - unused| G[❌ Deleted]
Loading

Reviews (1): Last reviewed commit: "improvement(deps): remove unused remark ..." | Re-trigger Greptile

@icecrasher321 icecrasher321 merged commit 94f60e7 into staging May 9, 2026
14 checks passed
@waleedlatif1 waleedlatif1 deleted the improvement/remove-deps branch May 9, 2026 23:48
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