aaif-goose / aaif-goose/vscode-goose

chore(extension): adopt ACP TypeScript SDK for wire types

Aberta
#60 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
TypeScript
Estrelas
43
Forks
20
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

Extracted from #39, which replaces the hand-rolled ACP wire types with `@agentclientprotocol/sdk` imports (AGENT_METHODS, PROTOCOL_VERSION, InitializeResponse, SessionNotification, SessionModeState, ListSessionsResponse, etc.) in `extension.ts` and `sessionManager.ts`. As submitted it pins the dependency as `file:../acp-typescript-sdk`, which cannot install in CI and accounts for ~8k of the PR's lockfile churn.

Decision first: published npm pin vs vendored types vs staying hand-rolled. If adopted, preserve the boundary #39 already keeps: `src/shared` and the webview stay SDK-free. No user-visible change. Unblocks the session mode/model and session/list issues.

Note: `extension.ts` was restructured by #40/#49/#50/#51 since #39 was cut, so treat the PR hunks as a behavioral spec, not a patch.

**Size**: M | **Depends on**: nothing | **Tier**: 1 (foundation)

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

Review the changes in the closed PR #39 to understand the intended replacement of hand-rolled types with the ACP TypeScript SDK. Examine the current structure of `extension.ts` and `sessionManager.ts` after recent refactors (#40, #49, #50, #51). The goal is to import types like AGENT_METHODS and PROTOCOL_VERSION from the SDK, ensuring the dependency is properly published or vendored. Verify that `src/shared` and webview code remain SDK-free. Run the extension tests to confirm no regressions.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
nodejs
Domínio
backend-api-design, tooling
Tipo de issue
Refatoração
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Pouca atividade
Clareza
Claramente especificada
Facilidade para iniciantes
65/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.