Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jul 8, 2025

The error message for missing account aliases had a formatting issue that caused the alias name to be concatenated directly to "Found" without proper spacing or quotes.

Before:

❌ error: Account alias not Founduser

After:

❌ error: Account alias "user" not Found

The fix updates the error message format in cmd/soroban-cli/src/config/sc_address.rs from:

#[error("Account alias not Found{0}")]

To:

#[error("Account alias \"{0}\" not Found")]

This ensures the alias name is properly quoted and separated from the "Found" text, making the error message more readable and consistent with the expected format.

Fixes #1967.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: sagpatil <1414227+sagpatil@users.noreply.github.com>
Copilot AI changed the title [WIP] Alias not found error prints typo Fix typo in alias not found error message format Jul 8, 2025
Copilot AI requested a review from sagpatil July 8, 2025 18:55
Copilot finished work on behalf of sagpatil July 8, 2025 18:55
@janewang janewang marked this pull request as ready for review July 9, 2025 17:43
@janewang
Copy link
Contributor

janewang commented Jul 9, 2025

@sagpatil This is neat, thank you

@janewang janewang merged commit ec63970 into main Jul 9, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Jul 9, 2025
@janewang janewang deleted the copilot/fix-1967 branch July 9, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Alias not found error prints typo

3 participants