Skip to content

Remove unused Long import from messages-recv.ts#2567

Open
IsMeElyn wants to merge 1 commit into
WhiskeySockets:masterfrom
IsMeElyn:patch-1
Open

Remove unused Long import from messages-recv.ts#2567
IsMeElyn wants to merge 1 commit into
WhiskeySockets:masterfrom
IsMeElyn:patch-1

Conversation

@IsMeElyn

@IsMeElyn IsMeElyn commented May 17, 2026

Copy link
Copy Markdown

fix: remove unused Long import causing missing dependency issues


Summary by cubic

Removed an unused long import from src/Socket/messages-recv.ts to prevent missing dependency errors and clean up the build. No runtime changes.

Written for commit a8a9002. Summary will update on new commits. Review in cubic

Summary by CodeRabbit

  • Chores
    • Removed unused internal dependency.

Review Change Stack

fix: remove unused Long import causing missing dependency issues
@whiskeysockets-bot

Copy link
Copy Markdown
Contributor

Thanks for opening this pull request and contributing to the project!

The next step is for the maintainers to review your changes. If everything looks good, it will be approved and merged into the main branch.

In the meantime, anyone in the community is encouraged to test this pull request and provide feedback.

✅ How to confirm it works

If you’ve tested this PR, please comment below with:

Tested and working ✅

This helps us speed up the review and merge process.

📦 To test this PR locally:

# NPM
npm install @whiskeysockets/baileys@IsMeElyn/Baileys#patch-1

# Yarn (v2+)
yarn add @whiskeysockets/baileys@IsMeElyn/Baileys#patch-1

# PNPM
pnpm add @whiskeysockets/baileys@IsMeElyn/Baileys#patch-1

If you encounter any issues or have feedback, feel free to comment as well.

@coderabbitai

coderabbitai Bot commented May 17, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5ee3107d-5097-4aa6-90c4-8da4d88583b2

📥 Commits

Reviewing files that changed from the base of the PR and between 28ca087 and a8a9002.

📒 Files selected for processing (1)
  • src/Socket/messages-recv.ts
💤 Files with no reviewable changes (1)
  • src/Socket/messages-recv.ts

📝 Walkthrough

Walkthrough

This pull request removes an unused Long import statement from src/Socket/messages-recv.ts. The import from the 'long' library is no longer referenced in the module and is cleaned up.

Changes

Unused dependency removal

Layer / File(s) Summary
Remove unused Long import
src/Socket/messages-recv.ts
The unused Long import is removed from the module header.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through code so clean,
One import gone, no more between,
The Long that danced is now set free,
Dependencies trimmed with glee!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Remove unused Long import from messages-recv.ts' directly and accurately describes the only change in the pull request: removing the unused Long dependency import.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cubic-dev-ai cubic-dev-ai 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.

No issues found across 1 file

Re-trigger cubic

@IsMeElyn

Copy link
Copy Markdown
Author
IMG-20260517-WA0040

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open for 14 days with no activity. Remove the stale label or comment or this will be closed in 14 days

@github-actions github-actions Bot added the Stale label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants