adds optional appReadyMessage property to RunAppOptions and DebugAppOptions
specify an appReadyMessage of 'Application startup complete' for FastAPI and Shiny apps, which leverage uvicorn and output that message when the app is ready
delete empty DebugConfiguration object from the positron-run-app.d.ts files, as I think they are unused?
modified the logging level in api.ts for a few messages
previewUrlInExecutionOutput now checks for both the app url and the appReadyMessage if provided,
an app is considered ready if:
the appReadyMessage was not provided and the app url has been matched; or
the appReadyMessage was provided and matched, and the app url has been matched
if the appReadyMessage was provided but not matched, and the app url has been matched, we'll time out of the raceTimeout, but the url will have been matched, so we'll still try to open the app in the Viewer
QA Notes
I've renabled Python - Verify Basic FastAPI App [C903306] which now passes
Implementation Notes
appReadyMessageproperty toRunAppOptionsandDebugAppOptionsappReadyMessageof'Application startup complete'for FastAPI and Shiny apps, which leverage uvicorn and output that message when the app is readyDebugConfigurationobject from thepositron-run-app.d.tsfiles, as I think they are unused?api.tsfor a few messagespreviewUrlInExecutionOutputnow checks for both the app url and theappReadyMessageif provided,appReadyMessagewas not provided and the app url has been matched; orappReadyMessagewas provided and matched, and the app url has been matchedappReadyMessagewas provided but not matched, and the app url has been matched, we'll time out of theraceTimeout, but the url will have been matched, so we'll still try to open the app in the ViewerQA Notes
Python - Verify Basic FastAPI App [C903306]which now passes