Skip to content

chore: add transfer test script and document debugging scripts #139

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
joshua-quek-sonarsource merged 1 commit into from
Apr 29, 2026
Merged

chore: add transfer test script and document debugging scripts #139

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .debugging/test-transfer.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
#!/bin/bash

./dist/bin/cloudvoyager-macos-arm64 transfer -c config.json --force-restart

@cursor cursor Bot Apr 29, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Script uses wrong config filename, contradicts docs

Medium Severity

The test-transfer.sh script passes -c config.json but the documentation, the .gitignore entry, and the naming convention from sibling scripts (test-migrate.sh uses migrate-config.json) all indicate it needs to use transfer-config.json. A developer following the docs would create transfer-config.json but the script would never read it, instead picking up a different (or nonexistent) config.json.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit be3ec48. Configure here.

2 changes: 2 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ debug/
proto/
compliance/sonar_scan_local.sh
migrate-config.json
transfer-config.json
.claude/

# Desktop app
Expand All @@ -69,6 +70,7 @@ dist/desktop/
**/*.json.journal
**/*.json.journal.backup
**/*.json.gz
*.lock

sonar-local-scan.sh
*.backup
18 changes: 18 additions & 0 deletions docs/local-development.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -889,6 +889,24 @@ The following npm scripts are available for building, testing, and linting:

---

<!-- updated: 2026-04-29_14:50:00 -->
## 🐛 Debugging Scripts

The `.debugging/` folder contains convenience scripts for local testing:

| Script | What it does |
|--------|-------------|
| `test-migrate.sh` | Run `migrate` command with `migrate-config.json` |
| `test-transfer.sh` | Run `transfer` command with `transfer-config.json` |
| `test-verify.sh` | Run `verify` command with `migrate-config.json` |
| `build-desktop.sh` | Build the Electron desktop app |
| `run-desktop.sh` | Launch the desktop app in dev mode |
| `delete-all-sonarcloud-projects.sh` | Delete all projects from a SonarCloud org (for cleanup) |

These scripts expect the binary at `./dist/bin/cloudvoyager-macos-arm64` and config files at the repo root. Both `migrate-config.json` and `transfer-config.json` are gitignored since they contain credentials.

---

<!-- Updated: Feb 20, 2026 at 04:02:35 PM -->
## 📚 Further Reading

Expand Down