Skip to content

Update testing-web-capture skill: live-cast URL tuning and WebM frame extraction - #669

Merged
streamer45 merged 3 commits into
mainfrom
devin/update-skills-1785082765
Jul 26, 2026
Merged

Update testing-web-capture skill: live-cast URL tuning and WebM frame extraction#669
streamer45 merged 3 commits into
mainfrom
devin/update-skills-1785082765

Conversation

@staging-devin-ai-integration

@staging-devin-ai-integration staging-devin-ai-integration Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Adds two verified notes to the web-capture testing skill: how to tune a live cast's URL via skit-cli (built from the streamkit-client package), and that live cast WebM files lack cues so frames must be extracted with select-filter full decodes instead of -ss/-sseof seeking.

Devin Session: https://staging.itsdev.in/sessions/f29bf33bf0184c92a23f81e695642b4a


Devin Review

Status Commit
🟢 Reviewed 67e18f8
Open in Devin Review (Staging)

… extraction

Co-authored-by: Claudio Costa <cstcld91@gmail.com>

@staging-devin-ai-integration staging-devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

Open in Devin Review (Staging)
Debug

Playground

Comment on lines +60 to +63
## Tuning a live cast's URL
- Build the CLI with `cargo build --release -p streamkit-client` (the package is `streamkit-client`, not `skit-cli`), then
`target/release/skit-cli tune <session-id> web url <new-url>`. The embedded MCP endpoint is disabled by default, so
REST/MCP tuning is not available out of the box.

@staging-devin-ai-integration staging-devin-ai-integration Bot Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

📝 Info: CLI package/binary naming claim verified

The added note claims the crate package is streamkit-client while the binary is skit-cli. This matches apps/skit-cli/Cargo.toml (name = "streamkit-client", [[bin]] name = "skit-cli"), and the tune <session> <node> <param> <value> argument order matches apps/skit-cli/src/main.rs:397 / apps/skit-cli/src/client.rs:356-361. No correction needed.

Open in Devin Review (Staging)

Was this helpful? React with 👍 or 👎 to provide feedback.

Debug

Playground

Comment thread .agents/skills/testing-web-capture/SKILL.md Outdated
…lane

Signed-off-by: streamkit-devin <devin@streamkit.dev>

@staging-devin-ai-integration staging-devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

Open in Devin Review (Staging)
Debug

Playground

Comment on lines +62 to +63
`target/release/skit-cli tune <session-id> web url <new-url>` (uses the REST/WS control plane under the hood).
The embedded MCP endpoint is disabled by default (`[mcp]` in `skit.toml`), so MCP tuning is not available out of the box.

@staging-devin-ai-integration staging-devin-ai-integration Bot Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

📝 Info: Documented tune behavior matches the CLI implementation

Verified the claims added here against the code: the crate at apps/skit-cli/Cargo.toml is named streamkit-client while its binary is skit-cli, and streamkit_client::tune_node (apps/skit-cli/src/client.rs:425-455) parses the value with serde_saphyr::from_str (YAML) and sends RequestPayload::TuneNode over a WebSocket request. The YAML-quoting caveat is therefore accurate; note that plain https://host/path scalars parse fine, so quoting is only needed for values containing YAML-special characters such as #, : , or leading &/*.

Open in Devin Review (Staging)

Was this helpful? React with 👍 or 👎 to provide feedback.

Debug

Playground

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 67e18f8 — now says the tune goes over the WebSocket control plane only, and added the YAML-parsing/quoting caveat for tune values.

…e values

Signed-off-by: streamkit-devin <devin@streamkit.dev>
@streamer45
streamer45 merged commit b5f53ac into main Jul 26, 2026
8 checks passed
@streamer45
streamer45 deleted the devin/update-skills-1785082765 branch July 26, 2026 17:49
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