feat: support multiple URLs for GET endpoint (issue #187)#2
Open
mesiyoq965-sudo wants to merge 1 commit into
Open
feat: support multiple URLs for GET endpoint (issue #187)#2mesiyoq965-sudo wants to merge 1 commit into
mesiyoq965-sudo wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Support multiple issue URLs for the GET
/startendpoint, as requested in issue ubiquity-os-marketplace#187.Changes
issueUrlsarray parameter alongside the existingissueUrlparameterhandleMultiUrlValidateOrExecuteto process multiple URLs sequentially and return results as a batchMultiUrlResulttype for the batch response structurepublic-api.tsto route to the multi-URL handler whenissueUrlsis presentissueUrlparameterAPI Usage
Single URL (existing behavior):
Multiple URLs (new):
Response for multiple URLs:
{ "results": [ { "issueUrl": "...", "ok": true, "computed": {...} }, { "issueUrl": "...", "ok": false, "reasons": ["..."] } ] }Motivation
This allows
devpool-directoryanddaemon-plannerto check hundreds of tasks in a single request, avoiding rate limits when checking multiple tasks sequentially.Testing
Tests updated to cover both single and multi-URL scenarios.
Related
GETendpoint ubiquity-os-marketplace/command-start-stop#187