Skip to content

feat(agent): sdk-core 0.3.3 com notification, title da ferramenta e README do mode (#123) - #124

Merged
JoaoLuisQuirino merged 6 commits into
mainfrom
fix/123-pontos-mp
Oct 27, 2026
Merged

JoaoLuisQuirino merged 6 commits into
mainfrom
fix/123-pontos-mp

Conversation

@JoaoLuisQuirino

@JoaoLuisQuirino JoaoLuisQuirino commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Closes #123.

Ordem de subida (obrigatória)

Esta SDK sobe primeiro. Depois dela, o Infusion, o Meu Agente e, se seguir em uso, o Orbit atualizam. Só então o mitra-copilot#552 grava a linha NOTIFICATION e o mitra-sandbox#782 emite o frame notification. A 0.3.0 a 0.3.2 leriam essa linha como texto do agente.

O que muda

Pontos que o MP levantou no chat novo do agente (conversa d03cf06d, alpha). Tudo é aditivo no runtime.

  • AgentToolEvent.title (opcional): o título que a box manda na ferramenta, ao vivo e no item tool do histórico. No subagente é o nome dele, que a linha TOOL_USE já tinha.
  • Fim de trabalho em segundo plano: evento notification (AgentNotification: source, outcome, summary, detail?, mais itemId, sequence e turnId) e item notification no history, ao vivo (frame notification) e recarregado (linha NOTIFICATION, com kinds=all).
  • README: send sem mode com turno rodando entra no turno em andamento (steer); a fila só com mode: "queue". Documentados também backgroundTasks, o turno em curso que a box reenvia depois do replay e o reason: "interrupted" na linha que o Stop fechou.
  • Versão 0.3.3: package, lock, contracts/v0.3.3 e CHANGELOG.

Para quem usa TypeScript. AgentTimelineItem ganha a variante kind: "notification", com payload e sem text. Um app que trata os kinds conhecidos e lê item.text no else precisa tratar notification, como foi com approval na 0.3.0. É só em tempo de compilação; no runtime nada muda para quem não recebe a linha.

Sobre o loadHistory do openExisting sem tamanho: a primeira página segue em 50, como decidido na 0.3.1. O app usa loadOlder()/hasOlder para o resto.

Prova

  • npm run check e npm test 730/730; typecheck, lint, format, check:exports, check:contracts, pack:check, build e smoke:package (com AgentNotification e o evento) verdes.
  • Paridade da 0.2.13: title entra na lista de campos acrescentados.
  • Dev (tarball deste branch): o subagente em segundo plano chega com tool.title "Rodar sleep em segundo plano" ao vivo e no histórico depois do F5 (task 3458f6f1). Prova do notification nos comentários.

🤖 Generated with Claude Code

…123)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JoaoLuisQuirino
JoaoLuisQuirino requested a review from a team as a code owner October 27, 2026 00:53
@JoaoLuisQuirino

Copy link
Copy Markdown
Contributor Author

/gandalf

@gandalf-the-grey-bot gandalf-the-grey-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review concluído no commit e1e2dea; nenhum achado de linha acionável nesta rodada.

@gandalf-the-grey-bot

gandalf-the-grey-bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Gandalf, análise 1 · commit e1e2dead

Pega-rápido: O histórico já recebia o título da ferramenta, mas a transformação pública o descartava. O PR passa title opcionalmente ao SDK para que os apps não precisem reconstruir esse dado; isso responde à perda registrada na issue #123.

Insights: O problema é concreto e a solução aproveita o fluxo existente, sem duplicação ou expansão desnecessária. Não identifiquei impacto adicional relevante em segurança ou escala.

Forma da implementação: Aprovada. src/agentSession.ts repassa title como string nos eventos e no histórico, inclusive pelos caminhos compartilhados; os testes cobrem o evento e o histórico, e os manifestos mantêm as versões alinhadas.

Findings: Nenhum achado acionável. O CI do SHA terminou verde em todos os jobs (build, test, compatibility (22), compatibility (18) e pr-convention).

Confiança: Alta: issue, diff, testes focados e CI verde sustentam o comportamento revisado.

Decisão: Mergeável.

Aqui o meu check é obrigatório: depois de um push, comente /gandalf para eu revisar o commit novo e liberar o merge.

@gandalf-the-grey-bot gandalf-the-grey-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sem novos achados de linha; o commit continua igual ao da análise 1.

@gandalf-the-grey-bot

Copy link
Copy Markdown

Gandalf, análise 2 · commit e1e2dead

Desde a análise 1: o commit não mudou; não há pontos abertos nem achados novos.

Decisão: mergeável.

Aqui o meu check é obrigatório: depois de um push, comente /gandalf para eu revisar o commit novo e liberar o merge.

JoaoLuisQuirino and others added 2 commits October 26, 2026 22:09
…blicacao (#123)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JoaoLuisQuirino

Copy link
Copy Markdown
Contributor Author

/gandalf

Comment thread src/index.ts
@gandalf-the-grey-bot

Copy link
Copy Markdown

Gandalf, análise 3 · commit b863e77

Desde a análise 2: o delta adiciona o evento e o item de histórico notification; nenhum ponto anterior estava aberto. Novo comentário: cobertura do tipo público no smoke ESM/CommonJS.

Decisão: mergeável. O CI do commit terminou verde; o minor é uma melhoria recomendada, não impede o merge.

Aqui o meu check é obrigatório: depois de um push, comente /gandalf para eu revisar o commit novo e liberar o merge.

JoaoLuisQuirino and others added 2 commits October 27, 2026 00:12
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ation (#123)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JoaoLuisQuirino

Copy link
Copy Markdown
Contributor Author

/gandalf

@gandalf-the-grey-bot gandalf-the-grey-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Revisão da análise 4 no commit 259a67f: nenhum achado de linha; o minor da análise 3 foi corrigido.

@gandalf-the-grey-bot

Copy link
Copy Markdown

Gandalf, análise 4 · commit 259a67f

Desde a análise 3: corrigida a cobertura do export AgentNotification nos smokes ESM/CommonJS (conversa); nenhum ponto segue aberto e não há achado novo. O CI do commit terminou verde.

Decisão: mergeável.

Aqui o meu check é obrigatório: depois de um push, comente /gandalf para eu revisar o commit novo e liberar o merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JoaoLuisQuirino JoaoLuisQuirino changed the title feat(agent): sdk-core 0.3.3 com title da ferramenta e README do mode (#123) feat(agent): sdk-core 0.3.3 com notification, title da ferramenta e README do mode (#123) Oct 27, 2026
@JoaoLuisQuirino

Copy link
Copy Markdown
Contributor Author

/gandalf

1 similar comment
@JoaoLuisQuirino

Copy link
Copy Markdown
Contributor Author

/gandalf

@gandalf-the-grey-bot gandalf-the-grey-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nenhum finding de linha nesta rodada.

@gandalf-the-grey-bot

Copy link
Copy Markdown

Gandalf, análise 5 · commit bf4a636

Desde a análise 4: o ponto minor foi corrigido e a conversa está resolvida (conversa); nenhum ponto permanece aberto e nada novo.

Decisão: mergeável.

Aqui o meu check é obrigatório: depois de um push, comente /gandalf para eu revisar o commit novo e liberar o merge.

@JoaoLuisQuirino
JoaoLuisQuirino merged commit ffd55b1 into main Oct 27, 2026
11 of 15 checks passed
@JoaoLuisQuirino
JoaoLuisQuirino deleted the fix/123-pontos-mp branch October 27, 2026 05: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.

SDK: pontos do MP no chat do agente (title do subagente, raciocinio no F5, segundo plano, motivo do Stop, README do mode)

2 participants