Skip to content

fix(video): use Router's canonical create and status routes - #59

Merged
drewstone merged 7 commits into
mainfrom
pro/tangle-agent-v1-video-routes
Sep 27, 2026
Merged

drewstone merged 7 commits into
mainfrom
pro/tangle-agent-v1-video-routes

Conversation

@tangletools

@tangletools tangletools commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

The SDK sent video creation and status requests to paths Router does not serve. It now sends POST /v1/videos and GET /v1/videos/:id.

This existing ChatGPT fleet delivery includes four files: the two SDK route corrections, their contract assertions, the SDK version 0.5.3, and the pinned PR workflow for a frozen workspace install, SDK build, tests, and package creation.

Verified head: d59a9fd.
Current Router source mounts both routes. SDK workflow run 36235365715 passed at this head. Local SDK tests passed, and an independent source review approved the four-file diff. No funded video generation or package publication is claimed.

After source merge, package publication uses the owning release workflow with operator authorization. A created video job must reach a terminal result before anyone claims a completed video.

@drewstone drewstone changed the title fix: use Router's canonical video endpoints in the published SDK fix(video): use Router's canonical create and status routes Sep 27, 2026

@drewstone drewstone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Independent source review approved the current four-file diff. Router mounts POST /v1/videos and GET /v1/videos/:id; the SDK and contract assertions now match. Exact-head SDK workflow 36235365715 is successful. Fresh local SDK validation passed 296 tests, with 32 conditional skips. Package publication and provider video execution remain unclaimed.

@drewstone
drewstone merged commit 60d4af8 into main Sep 27, 2026
1 check passed
@drewstone

Copy link
Copy Markdown
Contributor

Publication has an active owner in the recovery release lane.
The canonical candidate is now @tangle-network/tcloud@0.6.0, following merged #60 (c6e4c17bdeeeef9de6f3b32ad1040646f1c4b84a).
The lane is verifying that final source includes this PR's video routes and #60's transport changes before using the existing release workflow.

Please avoid a second publisher or a separate 0.5.3 backport.
App #650's dependency receipt should track the verified 0.6.0 artifact when publication completes.
Publication is already authorized; no registry success or completed video generation is claimed here.

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