Fix HTTP 422 error handling and improve status code messaging #163
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.
This PR addresses the issue where users were seeing unhelpful "Unknown statusCode" messages when the August API returns HTTP 422 errors, making it difficult to diagnose lock control problems.
Problem
Users reported seeing generic error messages like:
The plugin's status code handling only recognized specific codes (100, 200, 400, 429) but not 422, which is commonly returned by the August API when a lock operation cannot be processed.
Solution
Enhanced Status Code Handling
\b(\d{3})\b
to handle various error message formats like:"PUT failed with: 422"
"422"
"API call failed: 422"
Defensive API Call Improvements
pushChanges()
to prevent race conditionsBefore/After Comparison
Before:
After:
Testing
The changes are minimal and surgical, focusing specifically on improving error messaging while maintaining existing functionality.
Fixes #150.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.