Changing to use the embedded copilot CLI#143
Open
richardpark-msft wants to merge 9 commits intomicrosoft:mainfrom
Open
Changing to use the embedded copilot CLI#143richardpark-msft wants to merge 9 commits intomicrosoft:mainfrom
richardpark-msft wants to merge 9 commits intomicrosoft:mainfrom
Conversation
…rol over when the new CLI is introduced since it won't auto-update, and also makes sure we can bootstrap on machines that don't have the copilot CLI installed (less important, but now possible). - We now check if the copilot client is authenticated and give the user instructions on logging in using the copilot CLI. - Regen mocks for CopilotClient - exposed the GetAuthenticationStatus function. - Made a slog.Info call (where we load skills) to a slog.Debug call) - it was just just noise. We can add a proper log message at a later time. - LFS tracking enabled for .zst binary bundles.
richardpark-msft
requested review from
chlowell,
spboyer and
wbreza
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This PR switches the Copilot engine to use an embedded Copilot CLI bundle (via the Copilot SDK’s embedded CLI support), adds an initialization-time authentication check with user guidance, and updates mocks/tests accordingly. It also adds generation tooling for the embedded bundles and configures Git LFS tracking for the compressed CLI artifacts.
Changes:
- Auto-load embedded Copilot CLI + add
GetAuthStatusauthentication check duringCopilotEngine.Initialize. - Regenerate/update
CopilotClient/CopilotSessionmocks and adjust unit + live tests. - Add embedded CLI bundle generator and check in per-platform embedded bundle stubs (with
.zst+ license files); enable LFS for.zst.
Reviewed changes
Copilot reviewed 28 out of 29 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| internal/execution/generate.go | Updates mockgen go:generate target interfaces to exported names. |
| internal/execution/copilot_test.go | Updates mocks usage; adds live tests; adds helper mock setup for auth status. |
| internal/execution/copilot_engine_test.go | Updates mock type names for Copilot client/session tests. |
| internal/execution/copilot_client_wrappers.go | Extends CopilotClient interface/wrapper with GetAuthStatus. |
| internal/execution/copilot_client_wrapper_mocks_test.go | Regenerated gomock mocks for updated exported interfaces + new method. |
| internal/execution/copilot.go | Imports embedded CLI package and adds auth-status check in Initialize. |
| internal/embedded/zcopilot_windows_arm64.go | Generated embedded-cli setup for windows/arm64. |
| internal/embedded/zcopilot_windows_amd64.go | Generated embedded-cli setup for windows/amd64. |
| internal/embedded/zcopilot_linux_arm64.go | Generated embedded-cli setup for linux/arm64. |
| internal/embedded/zcopilot_linux_amd64.go | Generated embedded-cli setup for linux/amd64. |
| internal/embedded/zcopilot_darwin_arm64.go | Generated embedded-cli setup for darwin/arm64. |
| internal/embedded/zcopilot_1.0.2_windows_arm64.exe.zst | LFS-tracked compressed embedded Copilot CLI binary (windows/arm64). |
| internal/embedded/zcopilot_1.0.2_windows_arm64.exe.license | Embedded Copilot CLI license (windows/arm64). |
| internal/embedded/zcopilot_1.0.2_windows_amd64.exe.zst | LFS-tracked compressed embedded Copilot CLI binary (windows/amd64). |
| internal/embedded/zcopilot_1.0.2_windows_amd64.exe.license | Embedded Copilot CLI license (windows/amd64). |
| internal/embedded/zcopilot_1.0.2_linux_arm64.zst | LFS-tracked compressed embedded Copilot CLI binary (linux/arm64). |
| internal/embedded/zcopilot_1.0.2_linux_arm64.license | Embedded Copilot CLI license (linux/arm64). |
| internal/embedded/zcopilot_1.0.2_linux_amd64.zst | LFS-tracked compressed embedded Copilot CLI binary (linux/amd64). |
| internal/embedded/zcopilot_1.0.2_linux_amd64.license | Embedded Copilot CLI license (linux/amd64). |
| internal/embedded/zcopilot_1.0.2_darwin_arm64.zst | LFS-tracked compressed embedded Copilot CLI binary (darwin/arm64). |
| internal/embedded/zcopilot_1.0.2_darwin_arm64.license | Embedded Copilot CLI license (darwin/arm64). |
| internal/embedded/generate/generate.go | Adds a generator program to build/patch embedded CLI bundles. |
| go.mod | Adds klauspost/compress and bundler to tool block. |
| go.sum | Adds checksums for klauspost/compress. |
| cmd/waza/copilot_client_wrapper_mocks_test.go | Updates CLI command mocks to include GetAuthStatus. |
| cmd/waza/cmd_run_test.go | Updates tests to use shared mock helper (start/stop/auth stubbing). |
| cmd/waza/cmd_run.go | Silences “workspace skills added” log to Debug; sets SilenceErrors. |
| cmd/waza/cmd_new_task_test.go | Updates tests to use shared mock helper and adds helper implementation. |
| .gitattributes | Enables Git LFS tracking for *.zst. |
Comments suppressed due to low confidence (3)
internal/execution/copilot_test.go:226
- This test uses
newClientMock, which setsStart()/Stop()expectations withAnyTimes(). That makes the comment above (“Start() should NOT be called…”) no longer enforced—ifExecuteregresses and starts the client for invalid requests, this test would still pass. Use a stricter mock here (e.g., don’t stubStart()at all, or setStart().Times(0)) so the test fails ifStart()is invoked.
client := newClientMock(ctrl)
// Start() should NOT be called when the request is invalid (e.g. Timeout == 0),
// because extractReqParams now runs before startOnce.Do.
builder := NewCopilotEngineBuilder("gpt-4o-mini", &CopilotEngineBuilderOptions{
NewCopilotClient: func(clientOptions *copilot.ClientOptions) CopilotClient {
return client
},
internal/execution/copilot_engine_test.go:108
engine.Initialize(...)in this test will also callCopilotClient.GetAuthStatus(...)now. Since the mock only has expectations forStart/CreateSession, initialization will fail with an unexpectedGetAuthStatuscall unless you stub it (return authenticated) or use a helper that does.
This issue also appears on line 80 of the same file.
func TestCopilotEngine_Execute_SendError(t *testing.T) {
ctrl := gomock.NewController(t)
clientMock := NewMockCopilotClient(ctrl)
sessionMock := NewMockCopilotSession(ctrl)
clientMock.EXPECT().Start(gomock.Any())
clientMock.EXPECT().CreateSession(gomock.Any(), gomock.Any()).Return(sessionMock, nil)
internal/execution/copilot_engine_test.go:95
- These tests call
engine.Initialize(...), but the Copilot engine now callsCopilotClient.GetAuthStatus(...)during initialization. The mock here only sets expectations forStart/CreateSession, soGetAuthStatuswill be an unexpected call and the test will fail. Add an expectation forGetAuthStatus(returning an authenticated response) or use a shared helper likenewClientMockthat stubs it.
func TestCopilotEngine_Execute_CreateSessionError(t *testing.T) {
ctrl := gomock.NewController(t)
clientMock := NewMockCopilotClient(ctrl)
clientMock.EXPECT().Start(gomock.Any())
clientMock.EXPECT().CreateSession(gomock.Any(), gomock.Any()).Return(nil, errors.New("session create failed"))
engine := NewCopilotEngineBuilder("test", &CopilotEngineBuilderOptions{
NewCopilotClient: func(clientOptions *copilot.ClientOptions) CopilotClient {
return clientMock
},
}).Build()
require.NoError(t, engine.Initialize(context.Background()))
resp, err := engine.Execute(context.Background(), &ExecutionRequest{Message: "hello", Timeout: time.Second})
You can also share your feedback on Copilot code review. Take the survey.
auto-merge was automatically disabled
March 18, 2026 22:53
Pull request was converted to draft
… failed. - Fixing some tests that were broken: - The mocks were expecting too many calls - The mocks were expecting too few calls - Some tests weren't calling Shutdown()
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #143 +/- ##
=======================================
Coverage ? 73.51%
=======================================
Files ? 140
Lines ? 15845
Branches ? 0
=======================================
Hits ? 11648
Misses ? 3350
Partials ? 847
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR switches the Copilot execution path to use an embedded Copilot CLI bundle (instead of relying on a user-installed CLI), adds an auth-status check during engine initialization, and updates tests/mocks to match the new client surface area.
Changes:
- Auto-load embedded Copilot CLI bundles and add
GetAuthStatusauthentication checking inCopilotEngine.Initialize. - Regenerate/update GoMock mocks and adjust tests (including new live-gated tests).
- Add LFS tracking for
.zstembedded bundles; reduce workspace-skill logging verbosity.
Reviewed changes
Copilot reviewed 28 out of 29 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| internal/execution/generate.go | Updates mockgen targets to exported CopilotSession/CopilotClient. |
| internal/execution/copilot_test.go | Updates mocks, adds live resume-session test, adds auth-status tests, introduces shared mock helper. |
| internal/execution/copilot_engine_test.go | Updates to new client mock helper; adds manual cleanup calls. |
| internal/execution/copilot_client_wrappers.go | Extends CopilotClient interface + wrapper with GetAuthStatus. |
| internal/execution/copilot_client_wrapper_mocks_test.go | Regenerated mocks for exported interfaces and new method. |
| internal/execution/copilot.go | Blank-imports embedded CLI and checks auth status during Initialize. |
| internal/embedded/zcopilot_windows_arm64.go | Adds embedded CLI setup for windows/arm64. |
| internal/embedded/zcopilot_windows_amd64.go | Adds embedded CLI setup for windows/amd64. |
| internal/embedded/zcopilot_linux_arm64.go | Adds embedded CLI setup for linux/arm64. |
| internal/embedded/zcopilot_linux_amd64.go | Adds embedded CLI setup for linux/amd64. |
| internal/embedded/zcopilot_darwin_arm64.go | Adds embedded CLI setup for darwin/arm64. |
| internal/embedded/zcopilot_1.0.2_windows_arm64.exe.zst | Adds embedded CLI bundle pointer (LFS). |
| internal/embedded/zcopilot_1.0.2_windows_arm64.exe.license | Adds embedded CLI license for windows/arm64 bundle. |
| internal/embedded/zcopilot_1.0.2_windows_amd64.exe.zst | Adds embedded CLI bundle pointer (LFS). |
| internal/embedded/zcopilot_1.0.2_windows_amd64.exe.license | Adds embedded CLI license for windows/amd64 bundle. |
| internal/embedded/zcopilot_1.0.2_linux_arm64.zst | Adds embedded CLI bundle pointer (LFS). |
| internal/embedded/zcopilot_1.0.2_linux_arm64.license | Adds embedded CLI license for linux/arm64 bundle. |
| internal/embedded/zcopilot_1.0.2_linux_amd64.zst | Adds embedded CLI bundle pointer (LFS). |
| internal/embedded/zcopilot_1.0.2_linux_amd64.license | Adds embedded CLI license for linux/amd64 bundle. |
| internal/embedded/zcopilot_1.0.2_darwin_arm64.zst | Adds embedded CLI bundle pointer (LFS). |
| internal/embedded/zcopilot_1.0.2_darwin_arm64.license | Adds embedded CLI license for darwin/arm64 bundle. |
| internal/embedded/generate/generate.go | Adds generator to build/patch embedded bundles via SDK bundler tool. |
| go.mod | Adds klauspost/compress and includes Copilot SDK bundler as a tool dep. |
| go.sum | Adds checksums for klauspost/compress. |
| cmd/waza/copilot_client_wrapper_mocks_test.go | Updates CLI-side mocks with GetAuthStatus. |
| cmd/waza/cmd_run_test.go | Updates run tests to use new client mock helper + auth expectations. |
| cmd/waza/cmd_run.go | Silences cobra error printing; reduces “Workspace skills added” log to debug. |
| cmd/waza/cmd_new_task_test.go | Updates tests to use auth-aware mock helper; adds helper implementation. |
| .gitattributes | Enables Git LFS for *.zst embedded bundle files. |
Comments suppressed due to low confidence (2)
internal/execution/copilot.go:120
startErris also used for authentication failures insidestartOnce.Do, but the returned error is always wrapped as "copilot failed to start". This makes auth failures/mid-init failures look like startup failures in CLI output. Consider returning auth errors directly (or wrapping with a more accurate prefix), and reserving the "failed to start" wrapper strictly forclient.Startfailures.
}
})
if startErr != nil {
return fmt.Errorf("copilot failed to start: %w", startErr)
internal/execution/copilot_engine_test.go:129
- Same as above: stopping
engine.clientdirectly skipsengine.Shutdown(), so any temp workspaces created byExecute()won’t be cleaned up. Usingengine.Shutdown(...)in cleanup keeps the test from leaking temp directories and better matches real lifecycle behavior.
t.Cleanup(func() {
err := engine.client.Stop()
require.NoError(t, err)
})
You can also share your feedback on Copilot code review. Take the survey.
… generate the normal artifacts.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR switches the Copilot execution path to use an embedded Copilot CLI bundle (via the Copilot SDK embedded CLI support), adding an authentication preflight and updating mocks/tests so waza is insulated from user-installed CLI auto-updates.
Changes:
- Auto-load embedded Copilot CLI and add an auth-status check during
CopilotEngine.Initialize(). - Add + regenerate
CopilotClient/CopilotSessionmocks (includingGetAuthStatus) and update Copilot engine tests (including new opt-in live tests). - Add embedded CLI bundle artifacts + generator, and track
.zstvia Git LFS; update CLI logging verbosity and README install-from-source instructions.
Reviewed changes
Copilot reviewed 32 out of 33 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
internal/execution/copilot.go |
Imports embedded CLI and adds auth-status preflight on engine initialization. |
internal/execution/copilot_client_wrappers.go |
Extends CopilotClient wrapper interface to include GetAuthStatus. |
internal/execution/copilot_test.go |
Updates mocks + adds opt-in live tests and auth failure tests. |
internal/execution/copilot_engine_test.go |
Adjusts mock setup; adds cleanup logic (needs improvement). |
internal/embedded/* |
Adds generated embedded CLI loaders and LFS-tracked bundle/license artifacts. |
internal/embedded/generate/generate.go |
Adds generator to build/patch embedded CLI bundle outputs for multiple platforms. |
cmd/waza/cmd_run.go |
Tweaks Cobra error handling and lowers a workspace-skills log to debug. |
cmd/waza/cmd_run_test.go, cmd/waza/cmd_new_task_test.go, cmd/waza/*mocks* |
Updates tests/mocks to account for auth-status call and new client init sequence. |
.gitattributes |
Tracks *.zst via Git LFS. |
README.md |
Documents the new “clone + LFS” install flow (currently has incorrect clone instructions). |
go.mod, go.sum |
Adds klauspost/compress for zstd decoding; adds bundler as a tool dependency. |
Comments suppressed due to low confidence (1)
internal/execution/copilot_engine_test.go:129
- Same as above: calling
engine.client.Stop()directly skips the engine's normal shutdown/cleanup behavior. Useengine.Shutdown(...)in the test cleanup to ensure sessions/workspaces are cleaned consistently.
You can also share your feedback on Copilot code review. Take the survey.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR switches the Copilot SDK integration to use an embedded Copilot CLI bundle (instead of relying on a user-installed copilot binary), adds an authentication preflight during engine initialization, and updates tests/docs to match the new startup behavior.
Changes:
- Embed platform-specific Copilot CLI bundles and auto-register them via a blank import.
- Add
GetAuthStatusto the Copilot client wrapper + initialization-time auth checks with user guidance. - Regenerate mocks and adjust unit/live tests and installation docs (Git LFS required for embedded
.zstartifacts).
Reviewed changes
Copilot reviewed 32 out of 33 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.gitattributes |
Track embedded .zst bundles via Git LFS. |
README.md |
Update install instructions to require cloning + Git LFS (no go install). |
go.mod |
Add klauspost/compress and add bundler to tool list. |
go.sum |
Add sums for klauspost/compress. |
internal/execution/generate.go |
Update mockgen target interfaces to exported names. |
internal/execution/copilot.go |
Blank import embedded CLI + add auth-status check during Initialize(). |
internal/execution/copilot_client_wrappers.go |
Expose GetAuthStatus on CopilotClient wrapper. |
internal/execution/copilot_client_wrapper_mocks_test.go |
Regenerated mocks reflecting exported interface names + GetAuthStatus. |
internal/execution/copilot_engine_test.go |
Update mocks/cleanup paths to align with new init/auth behavior. |
internal/execution/copilot_test.go |
Update mocks for new init flow, add live resume test and auth failure tests, add helper skip. |
cmd/waza/cmd_run.go |
Silence Cobra error printing; reduce noisy slog output to Debug. |
cmd/waza/cmd_run_test.go |
Update Copilot client mocking to include init/auth checks. |
cmd/waza/copilot_client_wrapper_mocks_test.go |
Regenerate CLI-side mocks to include GetAuthStatus. |
cmd/waza/cmd_new_task_test.go |
Update Copilot client mocking to include init/auth checks + helper. |
internal/embedded/generate/generate.go |
Add generator for embedded Copilot CLI bundles across OS/arch matrix. |
internal/embedded/zcopilot_darwin_amd64.go |
Embedded CLI setup for darwin/amd64. |
internal/embedded/zcopilot_darwin_arm64.go |
Embedded CLI setup for darwin/arm64. |
internal/embedded/zcopilot_linux_amd64.go |
Embedded CLI setup for linux/amd64. |
internal/embedded/zcopilot_linux_arm64.go |
Embedded CLI setup for linux/arm64. |
internal/embedded/zcopilot_windows_amd64.go |
Embedded CLI setup for windows/amd64. |
internal/embedded/zcopilot_windows_arm64.go |
Embedded CLI setup for windows/arm64. |
internal/embedded/zcopilot_1.0.2_darwin_amd64.zst |
LFS pointer for embedded CLI bundle (darwin/amd64). |
internal/embedded/zcopilot_1.0.2_darwin_amd64.license |
License file for embedded CLI bundle (darwin/amd64). |
internal/embedded/zcopilot_1.0.2_darwin_arm64.zst |
LFS pointer for embedded CLI bundle (darwin/arm64). |
internal/embedded/zcopilot_1.0.2_darwin_arm64.license |
License file for embedded CLI bundle (darwin/arm64). |
internal/embedded/zcopilot_1.0.2_linux_amd64.zst |
LFS pointer for embedded CLI bundle (linux/amd64). |
internal/embedded/zcopilot_1.0.2_linux_amd64.license |
License file for embedded CLI bundle (linux/amd64). |
internal/embedded/zcopilot_1.0.2_linux_arm64.zst |
LFS pointer for embedded CLI bundle (linux/arm64). |
internal/embedded/zcopilot_1.0.2_linux_arm64.license |
License file for embedded CLI bundle (linux/arm64). |
internal/embedded/zcopilot_1.0.2_windows_amd64.exe.zst |
LFS pointer for embedded CLI bundle (windows/amd64). |
internal/embedded/zcopilot_1.0.2_windows_amd64.exe.license |
License file for embedded CLI bundle (windows/amd64). |
internal/embedded/zcopilot_1.0.2_windows_arm64.exe.zst |
LFS pointer for embedded CLI bundle (windows/arm64). |
internal/embedded/zcopilot_1.0.2_windows_arm64.exe.license |
License file for embedded CLI bundle (windows/arm64). |
Comments suppressed due to low confidence (1)
internal/execution/copilot.go:121
Initializewraps any failure fromStart()and auth-status checks as"copilot failed to start", which becomes misleading onceStart()succeeded but authentication failed. Consider returning distinct error text for auth failures (or returningstartErrdirectly when it’s already an auth-related error) so users don’t chase the wrong root cause.
if startErr != nil {
return fmt.Errorf("copilot failed to start: %w", startErr)
}
You can also share your feedback on Copilot code review. Take the survey.
Member
Author
|
Moving back to draft for a moment - want to check that CI/release isn't going to break because of the LFS artifacts. |
Member
Author
|
Okay, back to ready to review. There was just an extra attribute that needed to be added for the checkout action, no big deal. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR switches Copilot execution to use an embedded Copilot CLI bundle (tracked via Git LFS) to avoid relying on a user-installed CLI version, and adds an authentication preflight in CopilotEngine.Initialize() with improved CI/release checkout support for LFS artifacts.
Changes:
- Embed Copilot CLI bundles for linux/darwin/windows (amd64/arm64) and auto-load them via a blank import.
- Add Copilot auth-status check during engine initialization and regenerate client/session mocks to include
GetAuthStatus. - Update docs and CI workflows to ensure Git LFS artifacts are available during builds/tests; reduce skill discovery log noise.
Reviewed changes
Copilot reviewed 36 out of 37 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| internal/execution/generate.go | Updates mock generation targets to exported Copilot interfaces. |
| internal/execution/copilot_test.go | Updates mocks, adds live session-resume test, adds auth-status initialization tests, and introduces shared mock helper. |
| internal/execution/copilot_engine_test.go | Aligns tests with new client mock expectations and ensures shutdown cleanup is exercised. |
| internal/execution/copilot_client_wrappers.go | Extends CopilotClient wrapper interface to expose GetAuthStatus. |
| internal/execution/copilot_client_wrapper_mocks_test.go | Regenerates gomock output for exported interfaces + GetAuthStatus. |
| internal/execution/copilot.go | Blank-imports embedded CLI package and adds auth-status validation in Initialize(). |
| internal/embedded/zcopilot_windows_arm64.go | Adds generated embeddedcli setup for Windows ARM64 bundle. |
| internal/embedded/zcopilot_windows_amd64.go | Adds generated embeddedcli setup for Windows AMD64 bundle. |
| internal/embedded/zcopilot_linux_arm64.go | Adds generated embeddedcli setup for Linux ARM64 bundle. |
| internal/embedded/zcopilot_linux_amd64.go | Adds generated embeddedcli setup for Linux AMD64 bundle. |
| internal/embedded/zcopilot_darwin_arm64.go | Adds generated embeddedcli setup for macOS ARM64 bundle. |
| internal/embedded/zcopilot_darwin_amd64.go | Adds generated embeddedcli setup for macOS AMD64 bundle. |
| internal/embedded/zcopilot_1.0.2_windows_arm64.exe.zst | Adds Windows ARM64 CLI bundle via Git LFS pointer. |
| internal/embedded/zcopilot_1.0.2_windows_arm64.exe.license | Adds license for Windows ARM64 CLI bundle. |
| internal/embedded/zcopilot_1.0.2_windows_amd64.exe.zst | Adds Windows AMD64 CLI bundle via Git LFS pointer. |
| internal/embedded/zcopilot_1.0.2_windows_amd64.exe.license | Adds license for Windows AMD64 CLI bundle. |
| internal/embedded/zcopilot_1.0.2_linux_arm64.zst | Adds Linux ARM64 CLI bundle via Git LFS pointer. |
| internal/embedded/zcopilot_1.0.2_linux_arm64.license | Adds license for Linux ARM64 CLI bundle. |
| internal/embedded/zcopilot_1.0.2_linux_amd64.zst | Adds Linux AMD64 CLI bundle via Git LFS pointer. |
| internal/embedded/zcopilot_1.0.2_linux_amd64.license | Adds license for Linux AMD64 CLI bundle. |
| internal/embedded/zcopilot_1.0.2_darwin_arm64.zst | Adds macOS ARM64 CLI bundle via Git LFS pointer. |
| internal/embedded/zcopilot_1.0.2_darwin_arm64.license | Adds license for macOS ARM64 CLI bundle. |
| internal/embedded/zcopilot_1.0.2_darwin_amd64.zst | Adds macOS AMD64 CLI bundle via Git LFS pointer. |
| internal/embedded/zcopilot_1.0.2_darwin_amd64.license | Adds license for macOS AMD64 CLI bundle. |
| internal/embedded/generate/generate.go | Adds a generator tool to produce/patchembedded bundles for multiple platforms. |
| go.mod | Adds klauspost/compress and registers bundler as a Go tool dependency. |
| go.sum | Adds sums for newly introduced module dependency. |
| cmd/waza/copilot_client_wrapper_mocks_test.go | Regenerates command-side gomock output to include GetAuthStatus. |
| cmd/waza/cmd_run_test.go | Updates test harness to use shared client mock setup (via new helper). |
| cmd/waza/cmd_run.go | Silences cobra error printing and lowers skills discovery log to debug. |
| cmd/waza/cmd_new_task_test.go | Updates tests to use shared client mock setup and adds helper. |
| README.md | Updates install/build instructions due to Git LFS embedded artifacts. |
| .github/workflows/waza-eval.yml | Ensures CI checkout fetches LFS artifacts. |
| .github/workflows/release.yml | Ensures release workflows checkout fetches LFS artifacts. |
| .github/workflows/go-ci.yml | Ensures Go CI workflows checkout fetches LFS artifacts. |
| .github/workflows/docker-ci.yml | Ensures Docker CI workflow checkout fetches LFS artifacts. |
| .gitattributes | Adds LFS tracking rule for *.zst bundles. |
You can also share your feedback on Copilot code review. Take the survey.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changing to use the embedded copilot CLI.
This gives us greater control over when the new CLI is introduced since it won't auto-update, and insulates us from potential client/sdk incompatibility with the user's machine.
Fixes #42