Skip to content

Comments

Fix copy source command#4023

Merged
FadhlanR merged 1 commit intomainfrom
cs-10107-fix-copy-source-command
Feb 20, 2026
Merged

Fix copy source command#4023
FadhlanR merged 1 commit intomainfrom
cs-10107-fix-copy-source-command

Conversation

@FadhlanR
Copy link
Contributor

The previous arguments of the copy source commands were fromRealmUrl and toRealmUrl. Because of this naming, the LLM tended to use realm URLs for this command, when what we actually need is the full path to the file, including the file name. In this PR, I renamed the arguments to originSourceUrl and destinationSourceUrl. I also have this PR , to update the Boxel skills.

@github-actions
Copy link

Preview deployments

@github-actions
Copy link

github-actions bot commented Feb 19, 2026

Host Test Results

    1 files  ±    0      1 suites  ±0   3h 22m 35s ⏱️ + 1h 43m 34s
1 854 tests ±    0  1 839 ✅  -     1  14 💤 ± 0  0 ❌ ±0  1 🔥 +1 
3 738 runs  +1 869  3 708 ✅ +1 853  28 💤 +14  1 ❌ +1  1 🔥 +1 

For more details on these errors, see this check.

Results for commit be77819. ± Comparison against base commit 7942fa5.

♻️ This comment has been updated with latest results.

@FadhlanR FadhlanR marked this pull request as ready for review February 19, 2026 08:59
@FadhlanR FadhlanR requested a review from a team February 19, 2026 08:59
@FadhlanR FadhlanR merged commit 8d45ff7 into main Feb 20, 2026
121 of 124 checks passed
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