Skip to content

docs: fix CLI binary name in AGENTS.md codebase map#416

Merged
streamer45 merged 1 commit into
mainfrom
devin/1778348369-fix-agents-md-cli-name
May 10, 2026
Merged

docs: fix CLI binary name in AGENTS.md codebase map#416
streamer45 merged 1 commit into
mainfrom
devin/1778348369-fix-agents-md-cli-name

Conversation

@staging-devin-ai-integration
Copy link
Copy Markdown
Contributor

@staging-devin-ai-integration staging-devin-ai-integration Bot commented May 9, 2026

Summary

Fixes the CLI binary name in the AGENTS.md codebase map table: streamkit-clientskit-cli, matching the actual binary name defined in apps/skit-cli/Cargo.toml (lines 14–16).

Closes #413

Review & Testing Checklist for Human

  • Verify the binary name in apps/skit-cli/Cargo.toml matches skit-cli

Notes

Documentation-only change — single line in the codebase map table.

Link to Devin session: https://staging.itsdev.in/sessions/5eb691805fb242128381193e2e7cc7d2
Requested by: @streamer45


Devin Review

Status Commit
🟢 Reviewed 37bbf2c
Open in Devin Review (Staging)

The codebase map referred to the CLI binary as `streamkit-client`
but the actual binary name is `skit-cli` (per apps/skit-cli/Cargo.toml).

Closes #413

Signed-off-by: StreamKit Devin <devin@streamkit.dev>
Co-Authored-By: Claudio Costa <cstcld91@gmail.com>
@staging-devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Contributor Author

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

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review (Staging)
Debug

Playground

Comment thread AGENTS.md
|-----------|---------|
| `apps/skit/` | Server binary — HTTP/WS handlers, config, auth, plugins |
| `apps/skit-cli/` | CLI client binary (`streamkit-client`) |
| `apps/skit-cli/` | CLI client binary (`skit-cli`) |
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: Documentation change matches the binary/package split

The updated AGENTS.md entry refers specifically to the CLI client binary, not the Cargo package. That matches apps/skit-cli/Cargo.toml:14-16, where the binary is declared as skit-cli; the package/crate name remains streamkit-client (apps/skit-cli/Cargo.toml:1-2). Existing docs also describe this split as skit-cli for the binary and streamkit-client for the crate, so I did not treat the remaining streamkit-client references elsewhere as inconsistent with this change.

Open in Devin Review (Staging)

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

Debug

Playground

@streamer45 streamer45 merged commit 7987a5f into main May 10, 2026
8 checks passed
@streamer45 streamer45 deleted the devin/1778348369-fix-agents-md-cli-name branch May 10, 2026 17:02
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.

AGENTS.md: update CLI binary name from streamkit-client to skit-cli

2 participants