Skip to content

fix(tests): remove unused runAutoExtensions import flagged by lint in… - #575

Merged
AbdulmalikAlayande merged 1 commit into
TegoLabs:mainfrom
TheCreatorNode:fix/remove-unused-import
Jul 30, 2026
Merged

AbdulmalikAlayande merged 1 commit into
TegoLabs:mainfrom
TheCreatorNode:fix/remove-unused-import

Conversation

@TheCreatorNode

Copy link
Copy Markdown
Contributor

closes #368

@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@TheCreatorNode Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@coderabbitai

coderabbitai Bot commented Jul 29, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: e04143dd-bd79-4280-b31e-0ba014fbf995

📥 Commits

Reviewing files that changed from the base of the PR and between 35d9237 and 981b337.

📒 Files selected for processing (1)
  • tests/e2e/sandbox-network.test.ts
💤 Files with no reviewable changes (1)
  • tests/e2e/sandbox-network.test.ts

📝 Walkthrough

Summary by CodeRabbit

  • Tests
    • Updated end-to-end test setup by removing an unused import.
    • No changes to test behavior or assertions.

Walkthrough

The sandbox network E2E test removes the unused runAutoExtensions import. Existing test logic and assertions remain unchanged.

Changes

Sandbox network test cleanup

Layer / File(s) Summary
Remove unused extension import
tests/e2e/sandbox-network.test.ts
Removes the unused runAutoExtensions import while retaining the imports used by the test.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: abdulmalikalayande

Poem

A bunny tidies imports with care,
One unused sprig no longer there.
Tests hop on, assertions stay,
Lint smiles brightly through the day.
Fluffy code now leads the way!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: removing an unused test import.
Description check ✅ Passed The description is related to the change and references the linked issue.
Linked Issues check ✅ Passed The PR removes the unused runAutoExtensions import requested by issue #368 without changing test behavior.
Out of Scope Changes check ✅ Passed The only change is the requested import removal, with no unrelated code added.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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.

@gitguardian

gitguardian Bot commented Jul 29, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

Since your pull request originates from a forked repository, GitGuardian is not able to associate the secrets uncovered with secret incidents on your GitGuardian dashboard.
Skipping this check run and merging your pull request will create secret incidents on your GitGuardian dashboard.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
- - Generic High Entropy Secret ded54f4 tests/commands/guard-cli-export-import.test.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@AbdulmalikAlayande
AbdulmalikAlayande merged commit 5d2b096 into TegoLabs:main Jul 30, 2026
3 of 5 checks passed
AbdulmalikAlayande pushed a commit that referenced this pull request Aug 2, 2026
….test.ts (#575)

Verified locally: lint, typecheck, full suite (1039/1039), build, and audit all clean. Closes #368.
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.

fix(tests): remove unused runAutoExtensions import flagged by lint in tests/e2e/sandbox-network.test.ts

2 participants