refactor: graduate dead_code tests to tests/, delete obsolete artifacts#1
Merged
codejedi-ai merged 1 commit intomainfrom Mar 14, 2026
Merged
refactor: graduate dead_code tests to tests/, delete obsolete artifacts#1codejedi-ai merged 1 commit intomainfrom
codejedi-ai merged 1 commit intomainfrom
Conversation
Migrated (dead_code/ → tests/ with cleanup): - test_duckduckgo_search.py → tests/test_web_search.py - test_fda_run.py → tests/test_fda_navigation_simple.py - test_fda_detailed.py → tests/test_fda_navigation_detailed.py - test_fda_headed.py → tests/test_fda_navigation_headed.py - test_fda_simple_headed.py → tests/test_fda_navigation_simple_headed.py Cleanup applied to all migrated tests: - Removed stale sys.path.insert (tests/ is inside the package now) - Consistent TARGET_URL constant at module level - Consistent if __name__ == "__main__" guard Deleted (already live or obsolete): - dead_code/core/ (memory entities/ops already in smolclaw/agent/) - dead_code/tools/ (smolagents-style registry superseded by EIM) - dead_code/fix_event_sourcing.py (one-time repair script, already applied) dead_code/ directory fully removed. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
cyber-agent67
pushed a commit
to Duo-Keyboard-Koalition/SmolClaw
that referenced
this pull request
Mar 14, 2026
cyber-agent67
pushed a commit
that referenced
this pull request
Mar 14, 2026
…ing, then restore.
● Bash(git stash && git checkout master && git merge main -m "sync: bring master up to main" && git push origin master &&
git push upstream master && git checkout refa…)
⎿ Saved working directory and index state WIP on main: c627431 sync: merge upstream/main (PR #1 merge commit)
Switched to branch 'master'
Your branch is up to date with 'origin/master'.
… +65 lines (ctrl+o to expand)
● All branches are fully synced on both remotes:
┌───────────────────────┬────────────────────────┬────────────────────────┐
│ Branch │ origin (cyber-agent67) │ upstream (codejedi-ai) │
├───────────────────────┼────────────────────────┼────────────────────────┤
│ main │ c627431 ✓ │ c627431 ✓ │
├───────────────────────┼────────────────────────┼────────────────────────┤
│ master │ c627431 ✓ │ c627431 ✓ │
├───────────────────────┼────────────────────────┼────────────────────────┤
│ refactor/p2-structure │ c627431 ✓ │ c627431 ✓ │
└───────────────────────┴────────────────────────┴────────────────────────┘
Note: you also have uncommitted local changes on main (stash was restored). Those changes are not yet committed — let me
know if you'd like to commit and push them too.
cyber-agent67
pushed a commit
that referenced
this pull request
Mar 14, 2026
…ing, then restore.
● Bash(git stash && git checkout master && git merge main -m "sync: bring master up to main" && git push origin master &&
git push upstream master && git checkout refa…)
⎿ Saved working directory and index state WIP on main: c627431 sync: merge upstream/main (PR #1 merge commit)
Switched to branch 'master'
Your branch is up to date with 'origin/master'.
… +65 lines (ctrl+o to expand)
● All branches are fully synced on both remotes:
┌───────────────────────┬────────────────────────┬────────────────────────┐
│ Branch │ origin (cyber-agent67) │ upstream (codejedi-ai) │
├───────────────────────┼────────────────────────┼────────────────────────┤
│ main │ c627431 ✓ │ c627431 ✓ │
├───────────────────────┼────────────────────────┼────────────────────────┤
│ master │ c627431 ✓ │ c627431 ✓ │
├───────────────────────┼────────────────────────┼────────────────────────┤
│ refactor/p2-structure │ c627431 ✓ │ c627431 ✓ │
└───────────────────────┴────────────────────────┴────────────────────────┘
Note: you also have uncommitted local changes on main (stash was restored). Those changes are not yet committed — let me
know if you'd like to commit and push them too.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Migrated (dead_code/ → tests/ with cleanup):
Cleanup applied to all migrated tests:
Deleted (already live or obsolete):
dead_code/ directory fully removed.
Note
Low Risk
Low risk because changes are limited to deleting unused legacy modules and minor refactors to test scripts, with no production logic modified.
Overview
Deletes the legacy
dead_code/tree (oldcorememory helpers, tool registry/browser tools, and a one-offfix_event_sourcing.pyscript).Cleans up and standardizes the migrated smoke tests in
tests/(FDA navigation variants and DuckDuckGo search): removessys.pathhacks, introduces a sharedTARGET_URLconstant, and normalizes the__main__entrypoints and output/result dumping.Written by Cursor Bugbot for commit 93335b0. This will update automatically on new commits. Configure here.