Skip to content

[BOUNTY] Add Pytest Suite (#63) and /pipe Command (#66)#146

Open
sayanget wants to merge 3 commits intoScottcjn:mainfrom
sayanget:feature/bounty-tests-pipe-v1
Open

[BOUNTY] Add Pytest Suite (#63) and /pipe Command (#66)#146
sayanget wants to merge 3 commits intoScottcjn:mainfrom
sayanget:feature/bounty-tests-pipe-v1

Conversation

@sayanget
Copy link
Contributor

Problem Summary

This PR addresses two bounty tasks for TrashClaw:

  1. Task [BOUNTY: 15 RTC] Add pytest test suite for core tool functions #63: Implement a comprehensive test suite using pytest.
  2. Task [BOUNTY: 5 RTC] Add /pipe command to save last response to file #66: Add a /pipe command to save the last assistant response.

Solution Approach

  • Testing: Added tests/ directory with test_core.py, test_achievements.py, and test_undo.py covering all core tool functions, achievement logic, and undo snapshots.
  • CI: Added GitHub Actions workflow to run tests on push/PR.
  • Pipe Command: Enhanced /pipe in trashclaw.py to support automatic timestamped filenames if no argument is provided.

Test Evidence

  • All 14 tests passing locally on Linux.
  • Verified /pipe saves files correctly with both explicit and auto-generated names.

Wallet: atlas-agent-01

@Scottcjn
Copy link
Owner

This looks good and we want to merge it, but it has merge conflicts with main (likely from sofia-willow's recently merged PRs). Could you rebase on main and force-push? Once conflicts are resolved we'll merge. Thanks @sayanget!

@Scottcjn
Copy link
Owner

Merged! @sayanget — clean tests, config refactor, /pipe enhancement. Trusted contributor delivering again.

5 RTC. Reply with wallet if not on file.

@Scottcjn
Copy link
Owner

@sayanget — your PR is approved (5 RTC) but has merge conflicts. Please rebase:

git fetch upstream && git rebase upstream/main && git push --force

5 RTC on merge.

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