Repository navigation
feat(agent): sdk-core 0.3.3 com notification, title da ferramenta e README do mode (#123) - #124
Conversation
…123) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
/gandalf |
gandalf-the-grey-bot
left a comment
There was a problem hiding this comment.
Review concluído no commit e1e2dea; nenhum achado de linha acionável nesta rodada.
|
Gandalf, análise 1 · commit Pega-rápido: O histórico já recebia o título da ferramenta, mas a transformação pública o descartava. O PR passa 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. Findings: Nenhum achado acionável. O CI do SHA terminou verde em todos os jobs ( 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-the-grey-bot
left a comment
There was a problem hiding this comment.
Sem novos achados de linha; o commit continua igual ao da análise 1.
|
Gandalf, análise 2 · commit 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 |
…blicacao (#123) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
/gandalf |
|
Gandalf, análise 3 · commit Desde a análise 2: o delta adiciona o evento e o item de histórico Decisão: mergeável. O CI do commit terminou verde; o Aqui o meu check é obrigatório: depois de um push, comente |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ation (#123) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
/gandalf |
gandalf-the-grey-bot
left a comment
There was a problem hiding this comment.
Revisão da análise 4 no commit 259a67f: nenhum achado de linha; o minor da análise 3 foi corrigido.
|
Gandalf, análise 4 · commit Desde a análise 3: corrigida a cobertura do export Decisão: mergeável. Aqui o meu check é obrigatório: depois de um push, comente |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
/gandalf |
1 similar comment
|
/gandalf |
gandalf-the-grey-bot
left a comment
There was a problem hiding this comment.
Nenhum finding de linha nesta rodada.
|
Gandalf, análise 5 · commit 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 |
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
NOTIFICATIONe o mitra-sandbox#782 emite o framenotification. 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 itemtooldo histórico. No subagente é o nome dele, que a linhaTOOL_USEjá tinha.notification(AgentNotification:source,outcome,summary,detail?, maisitemId,sequenceeturnId) e itemnotificationnohistory, ao vivo (framenotification) e recarregado (linhaNOTIFICATION, comkinds=all).sendsemmodecom turno rodando entra no turno em andamento (steer); a fila só commode: "queue". Documentados tambémbackgroundTasks, o turno em curso que a box reenvia depois do replay e oreason: "interrupted"na linha que o Stop fechou.contracts/v0.3.3e CHANGELOG.Para quem usa TypeScript.
AgentTimelineItemganha a variantekind: "notification", compayloade semtext. Um app que trata os kinds conhecidos e lêitem.textnoelseprecisa tratarnotification, como foi comapprovalna 0.3.0. É só em tempo de compilação; no runtime nada muda para quem não recebe a linha.Sobre o
loadHistorydoopenExistingsem tamanho: a primeira página segue em 50, como decidido na 0.3.1. O app usaloadOlder()/hasOlderpara o resto.Prova
npm run checkenpm test730/730; typecheck, lint, format,check:exports,check:contracts,pack:check, build esmoke:package(comAgentNotificatione o evento) verdes.titleentra na lista de campos acrescentados.tool.title"Rodar sleep em segundo plano" ao vivo e no histórico depois do F5 (task 3458f6f1). Prova donotificationnos comentários.🤖 Generated with Claude Code