Skip to content

skills: document sightmapUploadUrl in tunnel-first flow#58

Merged
jurassix merged 2 commits intomainfrom
clint/tunnel-sightmap-upload-url
May 7, 2026
Merged

skills: document sightmapUploadUrl in tunnel-first flow#58
jurassix merged 2 commits intomainfrom
clint/tunnel-sightmap-upload-url

Conversation

@jurassix
Copy link
Copy Markdown
Collaborator

@jurassix jurassix commented May 6, 2026

Summary

live-tunnel now returns sightmapUploadUrl in its response, giving the tunnel-first flow a first-class path to upload sightmap definitions. Previously the only tools that issued a sightmap upload URL were review-open and live-connect — so agents using the tunnel-first flow (live-tunneltunnel-connectlive-view-new) had no documented upload path.

shared/SKILL.md — sightmap upload section now lists all three source tools in a table with field names and response formats:

Tool Field Format
review-open sightmap_upload_url: text line
live-connect sightmap_upload_url: text line
live-tunnel sightmapUploadUrl JSON field

Also clarifies timing: upload before live-view-new in the tunnel flow, before review-view/review-diff in review/connect flows.

tunnel/SKILL.md — tunnel-first flow steps updated:

  • Step 1 now lists sightmapUploadUrl in the return values
  • New step 2 prompts the upload if the project has .sightmap/ definitions
  • Code example shows sightmapUploadUrl in the live-tunnel response

Pairs with cowpaths/mn#104010.

live-tunnel now returns sightmapUploadUrl alongside relayUrl and
connectionId. Update shared and tunnel skills so agents know to
extract and use it in the tunnel-first flow.

shared/SKILL.md: list all three upload URL sources (review-open,
live-connect, live-tunnel) with their field names and response
formats. Note timing: upload before live-view-new in tunnel flow,
before review-view/review-diff in review/connect flows.

tunnel/SKILL.md: add sightmapUploadUrl to the step 1 return values,
insert a new step 2 for the sightmap upload, and show the field in
the code example.
Copy link
Copy Markdown
Collaborator

@chiplay chiplay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@jurassix jurassix merged commit b84541d into main May 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants