Bug Bash 2026-08: Street View Imagery Insights MCP Server - #82
Bug Bash 2026-08: Street View Imagery Insights MCP Server#82akshitmeghawat wants to merge 1 commit into
Conversation
FYI: The gcloud run services describe streetview-imagery-insights-mcp \
--project "${PROJECT_ID}" \
--region "${REGION}"
...and the server side (Cloud Run) logs are showing: The In PR #59, I got around this using a proxy server (see instructions here. Alternatively, you can try removing the |
9b3c45e to
bab566c
Compare
|
Modified the README.md as per your comment and also modified deploy.sh after running it with jetski cli. |
There was a problem hiding this comment.
all_assets table is deprecated.
Please update to use the newer table.
You'll find the there are 3 other queries in this file that also need to be updated to stop using deprecated tables.
There was a problem hiding this comment.
Used the new tables in the script.
These were the changes I made
- In
list_assetscall: all_assets -> latest_assets - In
get_asset_observationscall: all_observations -> latest_observations - In
analyze_cropped_assetcall: all_observations -> cropped_observations_all - In
analyze_full_frame_contextcall: all_observations -> full_frame_observations_all
bab566c to
098cccf
Compare
bijanvakili
left a comment
There was a problem hiding this comment.
Local server is working.
However, I am not sure the "Cloud Run Deployment" instructions will actually work. See comments.
| numpy | ||
| uvicorn | ||
| fastapi | ||
| mcp ~= 1.29 |
There was a problem hiding this comment.
You can remove mcp from this list.
fastmcp already includes it here (ie. it's a transitive dependency).
| query = f""" | ||
| SELECT observation_id, gcs_uri, bbox, pano_id, capture_time, camera_pose, asset_type | ||
| FROM `{project_id}.{dataset_id}.all_observations` | ||
| FROM `{project_id}.{dataset_id}.latest_observations` |
There was a problem hiding this comment.
This should be cropped_assets_all
Reasons:
latest_observationswill be deprecated- The function description indicates that it will retrieve all historical observations
| ``` | ||
|
|
||
| ### Cloud Run Deployment | ||
| 1. Navigate to the `mcp_server` directory: |
There was a problem hiding this comment.
nit: Remove this step cd ... as the local development did not include it.
| if __name__ == "__main__": | ||
| port = int(os.getenv("PORT", 8080)) | ||
| uvicorn.run(app, host="0.0.0.0", port=port) | ||
| transport = os.getenv("MCP_TRANSPORT", "sse") |
There was a problem hiding this comment.
- Based on the MCP specification here, the
ssetransport is now deprecated and we should be using streamable HTTP instead:
Streamable HTTP was introduced in protocol version 2025-03-26
as a replacement for the [HTTP+SSE transport](https://modelcontextprotocol.io/specification/2024-11-05/basic/transports#http-with-sse)
from protocol version 2024-11-05.
- Additionally, there is an outstanding bug with the Antigravity CLI which shows that their SSE client implementation is broken.
See other comment. I recommend switching this to the http transport.
| --source . \ | ||
| --project "${PROJECT_ID}" \ | ||
| --region "${REGION}" \ | ||
| --allow-unauthenticated \ |
There was a problem hiding this comment.
Can you please verify that this deployment to Cloud Run will work?
(ie. deploy it and connect with Jetski CLI to successfully run a specific skill)
- The service needs to run as a service account with necessary roles to query BigQuery. Take a look at the deployment script for
street-view-samples/street_view_insights/cropped/samples/imagery-insights-batch-processorhere which ensures the appropriate service account is set up along with forcing the service deployment to use it.
In particular, the service account will require the bigquery.jobs.create permission which is typically granted through one of the roles/bigquery.jobUser or roles/bigquery.user roles.
- You'll need to determine if
--allow-unauthenticatedor--no-allow-unauthenticatedshould be used. Based on the comment in theREADME.md, I was only able to get Jetski CLI to connect successfully if the FastMCP transport washttpand service was deployed with--no-allow-unauthenticatedalong with running the Cloud Run proxy in a separate window.
bijanvakili
left a comment
There was a problem hiding this comment.
Please change your base branch to main as @anubis05 just merged feat/street-view-samples branch recently.
Summary of changes
fastmcpinstead of coremcpSDK (v2.0.0), which does not contain the fastmcp module in its path.fastmcpgcloud deploycommands in deploy.sh--allow-unauthenticatedflagimagery_insights___usdatasetresolve_datasetto allow clients to supply fully qualified dataset IDs