Skip to content

chore: remove stray scratch/debug files committed to develop - #1450

Open
seonghobae wants to merge 4 commits into
developfrom
chore/remove-stray-scratch-files
Open

chore: remove stray scratch/debug files committed to develop#1450
seonghobae wants to merge 4 commits into
developfrom
chore/remove-stray-scratch-files

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Description

Removes five ad-hoc scratch/debug artifacts that were accidentally committed to the protected develop branch: plan.md (planning notes), test_parse.py / test_parse2.py / test_parse3.py (manual one-off debug scripts for backend/services/text_safety.py), and frontend/src/components/NetworkGraph.tsx.out (a stray build-output copy of NetworkGraph.tsx).

None of these are referenced by the backend or frontend test suites, CI workflows, or product code (verified with git grep). Their presence also makes every currently-open PR whose branch predates their addition show a misleading "this PR deletes N files" diff against develop when compared with gh pr diff/GitHub's compare view (observed while triaging the open PR backlog, e.g. #1287).

Fixes # (none — repo hygiene only, no tracked issue)

Type of change

  • This is a repo-hygiene cleanup (no production behavior change)

Verification

git grep -n "NetworkGraph.tsx.out\|test_parse\.py\|test_parse2\.py\|test_parse3\.py\|plan\.md" -- '*.ts' '*.tsx' '*.py' '*.yml' '*.yaml' '*.json'

No real references found (only unrelated fixture strings using "plan.md" as an example attachment filename in backend/tests/).

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes (no tests reference the removed files)

Open in Devin Review

plan.md, test_parse.py, test_parse2.py, test_parse3.py, and
frontend/src/components/NetworkGraph.tsx.out are ad-hoc planning notes,
manual debug scripts, and a stray build-output copy that were accidentally
committed to the protected default branch. None are referenced by the
backend/frontend test suites or CI. Removing them also stops every
still-open PR whose branch predates their addition from showing a
misleading 'this PR deletes N files' diff against develop (e.g. #1287).
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 59 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 61338c7d-7e63-4fc1-b15e-8f5646716a3a

📥 Commits

Reviewing files that changed from the base of the PR and between f3beb1c and 928a397.

📒 Files selected for processing (4)
  • plan.md
  • test_parse.py
  • test_parse2.py
  • test_parse3.py

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.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@opencode-agent opencode-agent Bot added priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: maintenance Maintenance, build, dependency, or operational upkeep labels Aug 22, 2026
@seonghobae
seonghobae enabled auto-merge (squash) August 24, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: maintenance Maintenance, build, dependency, or operational upkeep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant