Skip to content

fix: clean-up old scripts - #474

Merged
Nikhil (shadowfax92) merged 2 commits into
mainfrom
fix/clean-up-old-scripts
Mar 17, 2026
Merged

fix: clean-up old scripts#474
Nikhil (shadowfax92) merged 2 commits into
mainfrom
fix/clean-up-old-scripts

Conversation

@shadowfax92

Copy link
Copy Markdown
Contributor
  • fix: remove old scripts
  • fix: remove vscode

@shadowfax92
Nikhil (shadowfax92) merged commit 44af9ae into main Mar 17, 2026
4 checks passed
@github-actions github-actions Bot added the fix label Mar 17, 2026
@greptile-apps

greptile-apps Bot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR cleans up stale developer tooling by deleting three unused files: the auto-generated .vscode/PythonImportHelper-v2-Completion.json, scripts/save_clipboard.py (a helper to save clipboard images to disk), and scripts/update-submodule.sh (a helper to bump the packages/browseros-agent submodule). The changes are purely deletions with no functional impact on the product.

Key findings:

  • CLAUDE.md still references the now-deleted scripts/save_clipboard.py in its "Docs Image Workflow" section (lines 6–11). This will cause confusion for developers (and AI agents reading CLAUDE.md) who follow those instructions and find the script missing. The documentation should be updated or removed alongside this deletion.
  • scripts/update-submodule.sh and .vscode/PythonImportHelper-v2-Completion.json have no remaining references and are safe to delete as-is.

Confidence Score: 3/5

  • Safe to merge once CLAUDE.md is updated to remove the stale reference to the deleted script.
  • All changes are pure deletions with no runtime impact. The only issue is a broken documentation reference in CLAUDE.md pointing to the now-deleted save_clipboard.py script, which should be resolved before merging to avoid misleading developers and AI agents relying on that file.
  • CLAUDE.md — contains a stale reference to the deleted scripts/save_clipboard.py that needs to be removed or updated.

Important Files Changed

Filename Overview
.vscode/PythonImportHelper-v2-Completion.json Deletes a large auto-generated VSCode Python import helper JSON (3840 lines). IDE-specific tooling file — safe to remove.
scripts/save_clipboard.py Deletes clipboard-to-file helper script. Safe to remove, but CLAUDE.md still references this script and must be updated.
scripts/update-submodule.sh Deletes a shell script that automated syncing the packages/browseros-agent submodule. No references found elsewhere; safe to remove.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR #474: Clean up old scripts] --> B[Delete .vscode/PythonImportHelper-v2-Completion.json]
    A --> C[Delete scripts/save_clipboard.py]
    A --> D[Delete scripts/update-submodule.sh]
    B --> E[✅ No references remain]
    D --> F[✅ No references remain]
    C --> G[⚠️ CLAUDE.md still references this script]
    G --> H[Docs Image Workflow section lines 6-11 must be updated]
Loading

Comments Outside Diff (1)

  1. CLAUDE.md, line 6-11 (link)

    P2 Stale documentation references deleted script

    scripts/save_clipboard.py is being deleted in this PR, but CLAUDE.md still references it at lines 8–9. Anyone following these instructions (including AI agents reading CLAUDE.md) will hit a "file not found" error.

    Either remove this Docs Image Workflow section from CLAUDE.md or update it to describe an alternative workflow.

    ``markdown
    This is a comment left during a code review.
    Path: CLAUDE.md
    Line: 6-11

    Comment:
    Stale documentation references deleted script

    scripts/save_clipboard.py is being deleted in this PR, but CLAUDE.md still references it at lines 8–9. Anyone following these instructions (including AI agents reading CLAUDE.md) will hit a "file not found" error.

    Either remove this Docs Image Workflow section from CLAUDE.md or update it to describe an alternative workflow.

    ``

Prompt To Fix All With AI
This is a comment left during a code review.
Path: CLAUDE.md
Line: 6-11

Comment:
**Stale documentation references deleted script**

`scripts/save_clipboard.py` is being deleted in this PR, but `CLAUDE.md` still references it at lines 8–9. Anyone following these instructions (including AI agents reading `CLAUDE.md`) will hit a "file not found" error.

Either remove this `Docs Image Workflow` section from `CLAUDE.md` or update it to describe an alternative workflow.

```suggestion

**Rule Used:** Remove unused/dead code rather than leaving it in ... ([source](https://app.greptile.com/review/custom-context?memory=9b045db4-2630-428c-95b7-ccf048d34547))

**Learnt From**
[browseros-ai/BrowserOS-agent#126](https://github.com/browseros-ai/BrowserOS-agent/pull/126)

How can I resolve this? If you propose a fix, please make it concise.

Last reviewed commit: 099babc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant