feat: cog playground by anish-sahoo · Pull Request #3115 · replicate/cog

anish-sahoo · 2026-07-14T21:39:24Z

Cog Playground

A browser-based playground UI for testing Cog models locally, served by a Go HTTP server embedded in the Cog CLI.

Stack PRs

* feat: add Go playground server * fix: strip protocol-relative playground redirects * fix: strip all playground proxy redirects

ask-bonk · 2026-07-14T21:49:37Z

I found one critical issue in this PR:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The cog playground command is fully implemented in pkg/cli/playground.go, but it is never added to the root command, so users will see ``unknown command" when they run cog playground.

Fix:

// pkg/cli/root.go
rootCmd.AddCommand(
    newBuildCommand(),
    newDebugCommand(),
    newDoctorCommand(),
    newInitCommand(),
    newLoginCommand(),
    newRunCommand(),
    newPredictCommand(),
    newPushCommand(),
    newExecCommand(),
    newServeCommand(),
    newTrainCommand(),
    newWeightsCommand(),
    newPlaygroundCommand(),  // <-- add this
)

ask-bonk · 2026-07-14T21:49:43Z

I'm Bonk, and I've done a quick review of your PR.

This PR adds a cog playground command with a browser-based UI for interacting with running Cog models, plus the frontend scaffold, CI integration, and build pipeline changes. Overall the implementation is solid — the Go server has good security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

I found one critical issue that prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Posted the fix as a PR comment: #3115 (comment)

github run

ask-bonk · 2026-07-15T20:12:14Z

I found one critical issue in this PR:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The cog playground command is fully implemented in pkg/cli/playground.go, but it is never added to the root command, so users will see "unknown command" when they run cog playground.

Fix:

// pkg/cli/root.go
rootCmd.AddCommand(
    newBuildCommand(),
    newDebugCommand(),
    newDoctorCommand(),
    newInitCommand(),
    newLoginCommand(),
    newRunCommand(),
    newPredictCommand(),
    newPushCommand(),
    newExecCommand(),
    newServeCommand(),
    newTrainCommand(),
    newWeightsCommand(),
    newPlaygroundCommand(),  // <-- add this
)

ask-bonk · 2026-07-15T20:12:25Z

I'm Bonk, and I've done a quick review of your PR.

This PR adds a cog playground command with a browser-based UI for interacting with running Cog models, plus the frontend scaffold, CI integration, and build pipeline changes. The Go server has good security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

I found one critical issue that prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Posted the fix as a PR comment: #3115 (comment)

github run

* feat(playground): add Cog API client * fix(playground): harden local file and SSE cleanup * test(playground): pass SSE iterator return value * test(playground): mock SSE cancellation failure * style(playground): format SSE cleanup test * test(playground): use a real SSE frame * fix(playground): validate API responses

ask-bonk · 2026-07-16T16:18:16Z

Im Bonk, and Ive done a quick review of your PR.

This PR introduces a cog playground command with a browser-based UI for interacting with running Cog models, plus the frontend scaffold, CI integration, and build pipeline changes. The Go server has solid security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

However, the same critical issue flagged in previous reviews is still unfixed and prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Fix:

// pkg/cli/root.go
rootCmd.AddCommand(
    newBuildCommand(),
    newDebugCommand(),
    newDoctorCommand(),
    newInitCommand(),
    newLoginCommand(),
    newRunCommand(),
    newPredictCommand(),
    newPushCommand(),
    newExecCommand(),
    newServeCommand(),
    newTrainCommand(),
    newWeightsCommand(),
    newPlaygroundCommand(),  // <-- add this
)

ask-bonk · 2026-07-16T16:18:26Z

I'm Bonk, and I've done a quick review of your PR.

This PR introduces a cog playground command with a browser-based UI for interacting with running Cog models, plus the frontend scaffold, CI integration, and build pipeline changes. The Go server has solid security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

However, the same critical issue flagged in previous reviews is still unfixed and prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Fix:

// pkg/cli/root.go
rootCmd.AddCommand(
    newBuildCommand(),
    newDebugCommand(),
    newDoctorCommand(),
    newInitCommand(),
    newLoginCommand(),
    newRunCommand(),
    newPredictCommand(),
    newPushCommand(),
    newExecCommand(),
    newServeCommand(),
    newTrainCommand(),
    newWeightsCommand(),
    newPlaygroundCommand(),  // <-- add this
)

Posted the fix as a PR comment: #3115 (comment)

github run

ask-bonk · 2026-07-16T16:20:26Z

I found one critical issue in this PR:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The cog playground command is fully implemented in pkg/cli/playground.go, but it is never added to the root command, so users will see "unknown command" when they run cog playground.

Fix:

// pkg/cli/root.go
rootCmd.AddCommand(
    newBuildCommand(),
    newDebugCommand(),
    newDoctorCommand(),
    newInitCommand(),
    newLoginCommand(),
    newRunCommand(),
    newPredictCommand(),
    newPushCommand(),
    newExecCommand(),
    newServeCommand(),
    newTrainCommand(),
    newWeightsCommand(),
    newPlaygroundCommand(),  // <-- add this
)

ask-bonk · 2026-07-16T16:20:39Z

I'm Bonk, and I've done a quick review of your PR.

This PR adds a cog playground command with a browser-based UI for interacting with running Cog models. The Go server has solid security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

However, the same critical issue flagged in previous reviews is still unfixed and prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Posted the fix as a PR comment: #3115 (comment)

github run

* feat(playground): add input controls and validation * fix(playground): reset input state on unmount

ask-bonk · 2026-07-17T18:05:36Z

I'm Bonk, and I've done a quick review of your PR.

This PR adds a cog playground command with a browser-based UI for interacting with running Cog models, plus the frontend scaffold, CI integration, and build pipeline changes. The Go server has solid security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

However, the same critical issue flagged in previous reviews is still unfixed and prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Fix:

// pkg/cli/root.go
rootCmd.AddCommand(
    newBuildCommand(),
    newDebugCommand(),
    newDoctorCommand(),
    newInitCommand(),
    newLoginCommand(),
    newRunCommand(),
    newPredictCommand(),
    newPushCommand(),
    newExecCommand(),
    newServeCommand(),
    newTrainCommand(),
    newWeightsCommand(),
    newPlaygroundCommand(),  // <-- add this
)

ask-bonk · 2026-07-17T18:05:43Z

I'm Bonk, and I've done a quick review of your PR.

This PR adds a cog playground command with a browser-based UI for interacting with running Cog models, plus the frontend scaffold, CI integration, and build pipeline changes. The Go server has solid security boundaries (loopback-only UI, webhook exemption for containers, CORS-safe proxying, CSP headers) and thorough test coverage.

However, the same critical issue flagged in previous reviews is still unfixed and prevents the command from being usable at all:

newPlaygroundCommand() is never registered in pkg/cli/root.go.

The command is fully implemented in pkg/cli/playground.go but is never added to rootCmd.AddCommand(), so cog playground will return "unknown command".

Posted the fix as a PR comment: #3115 (comment)

github run

anish-sahoo added 3 commits July 14, 2026 16:07

1. Add Go playground server and handlers (#3109)

b3cb9f8

* feat: add Go playground server * fix: strip protocol-relative playground redirects * fix: strip all playground proxy redirects

feat(playground): add frontend scaffold and CI (#3108)

6feed4a

feat(playground): add core frontend types and utilities (#3114)

6145b13

anish-sahoo requested a review from a team as a code owner July 14, 2026 21:39

feat(playground): add shared UI and JSON editor (#3110)

e2227d0

anish-sahoo added 2 commits July 16, 2026 11:15

feat(playground): add connection feature (#3105)

5a15ca1

7. Add playground input controls and validation (#3111)

75679ee

* feat(playground): add input controls and validation * fix(playground): reset input state on unmount

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: cog playground#3115

feat: cog playground#3115
anish-sahoo wants to merge 7 commits into
mainfrom
cog-playground

anish-sahoo commented Jul 14, 2026 •

edited

Loading

Uh oh!

ask-bonk Bot commented Jul 14, 2026

Uh oh!

ask-bonk Bot commented Jul 14, 2026

Uh oh!

ask-bonk Bot commented Jul 15, 2026

Uh oh!

ask-bonk Bot commented Jul 15, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 17, 2026

Uh oh!

ask-bonk Bot commented Jul 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

anish-sahoo commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Cog Playground

Stack PRs

Uh oh!

ask-bonk Bot commented Jul 14, 2026

Uh oh!

ask-bonk Bot commented Jul 14, 2026

Uh oh!

ask-bonk Bot commented Jul 15, 2026

Uh oh!

ask-bonk Bot commented Jul 15, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 16, 2026

Uh oh!

ask-bonk Bot commented Jul 17, 2026

Uh oh!

ask-bonk Bot commented Jul 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

anish-sahoo commented Jul 14, 2026 •

edited

Loading