Skip to content

fix(old-scripts): remove#1561

Open
paul-nechifor wants to merge 1 commit intodevfrom
paul/fix/remove-old-scripts
Open

fix(old-scripts): remove#1561
paul-nechifor wants to merge 1 commit intodevfrom
paul/fix/remove-old-scripts

Conversation

@paul-nechifor
Copy link
Contributor

Problem

  • I forgot to remove unused scripts

Solution

  • Remove them.

Breaking Changes

How to Test

Contributor License Agreement

  • I have read and approved the CLA.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 15, 2026

Greptile Summary

Removes 7 unused files: 5 developer utility scripts from bin/ (error filtering, mypy, robot debugger) and 2 files from dimos/robot/utils/ (RobotDebugger class and its README). None of these files are imported or referenced anywhere in the codebase, making this a safe cleanup with no functional impact.

  • Deleted bin/filter-errors-after-date and bin/filter-errors-for-user (git-blame-based error filtering scripts)
  • Deleted bin/mypy-ros and bin/re-ignore-mypy.py (mypy helper scripts)
  • Deleted bin/robot-debugger, dimos/robot/utils/robot_debugger.py, and its README (rpyc-based robot debugging tools)
  • No remaining references to any of these files exist in the codebase

Confidence Score: 5/5

  • This PR is safe to merge — it only deletes files with zero references in the codebase.
  • All 7 deleted files are confirmed unused: no imports, no references in scripts, configs, or documentation elsewhere in the repo. This is a pure dead-code cleanup with no risk of breaking anything.
  • No files require special attention.

Important Files Changed

Filename Overview
bin/filter-errors-after-date Deleted unused script for filtering errors by git blame date. No references found in the codebase.
bin/filter-errors-for-user Deleted unused script for filtering errors by git blame author. No references found in the codebase.
bin/mypy-ros Deleted unused mypy-ros script that ran mypy in Docker and locally. No references found in the codebase.
bin/re-ignore-mypy.py Deleted unused script for regenerating type: ignore comments. No references found in the codebase.
bin/robot-debugger Deleted unused robot debugger shell script. No references found in the codebase.
dimos/robot/utils/README.md Deleted README for the now-removed robot debugger utility.
dimos/robot/utils/robot_debugger.py Deleted RobotDebugger class. No imports of this module exist anywhere in the codebase.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph Deleted["Deleted Files"]
        A["bin/filter-errors-after-date"]
        B["bin/filter-errors-for-user"]
        C["bin/mypy-ros"]
        D["bin/re-ignore-mypy.py"]
        E["bin/robot-debugger"]
        F["dimos/robot/utils/robot_debugger.py"]
        G["dimos/robot/utils/README.md"]
    end

    subgraph Categories["By Category"]
        H["Error Filtering Scripts"]
        I["Mypy Helper Scripts"]
        J["Robot Debugger"]
    end

    A --> H
    B --> H
    C --> I
    D --> I
    E --> J
    F --> J
    G --> J

    style Deleted fill:#ffcccc,stroke:#cc0000
    style Categories fill:#e6f3ff,stroke:#0066cc
Loading

Last reviewed commit: 752bf2e

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.

2 participants