Skip to content

chore(release): functions-sdk 0.2.10 com o sdk-core 0.2.14 (#70) - #71

Merged
BrunoPinaFelipe merged 1 commit into
mainfrom
chore/70-release-0.2.10
Oct 23, 2026
Merged

BrunoPinaFelipe merged 1 commit into
mainfrom
chore/70-release-0.2.10

Conversation

@BrunoPinaFelipe

Copy link
Copy Markdown
Contributor

Closes #70

Move o Core para 0.2.14 e fixa o corpus SDK-PARITY-001 no 0.2.14 (contracts/sdk-core-v0.2.14.manifest.json, digest e commit imutável 51ddb3e da release/0.2.x). Versão 0.2.10; a seção 0.2.10-beta.0 do CHANGELOG fica como histórico.

O que chega pelo Core: currentApp.update({ emailLoginEnabled, passwordLoginEnabled }) (mitralab-dev/mitra-core-sdk#97) e messenger.sendEmail() (mitralab-dev/mitra-core-sdk#90). Nenhum código deste repo muda.

npm run check verde.

Depois do merge: workflow Release com version=0.2.10.

🤖 Generated with Claude Code

@BrunoPinaFelipe
BrunoPinaFelipe requested a review from a team as a code owner October 23, 2026 00:02
@BrunoPinaFelipe
BrunoPinaFelipe merged commit 11bca48 into main Oct 23, 2026
5 checks passed
@BrunoPinaFelipe
BrunoPinaFelipe deleted the chore/70-release-0.2.10 branch October 23, 2026 00:03
Comment thread package.json
},
"dependencies": {
"@mitralab.io/sdk-core": "0.2.13",
"@mitralab.io/sdk-core": "0.2.14",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Minor, API pública: o bump para @mitralab.io/sdk-core@0.2.14 traz SendEmailInput, mas o Functions SDK não o reexporta em src/index.ts. A chamada messenger.sendEmail() funciona com inferência, porém consumidores não conseguem importar o tipo pelo pacote público. Reexporte o tipo e cubra a declaração ESM e CommonJS em um follow-up.

Comment thread CHANGELOG.md

- Depend on `@mitralab.io/sdk-core@0.2.14`: `apps.update()` (and `currentApp.update()`) takes
`emailLoginEnabled` and `passwordLoginEnabled`, sent to Code Studio only when supplied, so omission
keeps the current value; `AppDefinition` and `AppSummary` carry both fields when Code Studio returns

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Minor, documentação: esta entrada anuncia messenger.sendEmail(), mas a lista de operações do README ainda só documenta messenger.notify(). Atualize a lista pública e registre a exigência de token de backend e o remetente implícito em um follow-up.

@gandalf-the-grey-bot

Copy link
Copy Markdown

Gandalf, análise 1 · commit c6383167

Mérito: o release atende à issue #70 e mantém o bump do Core alinhado ao corpus de contratos. Forma adequada.

  • Pega-rápido: mudança de release com exposição de contratos públicos; confiança alta no diff e limitada para publicação e execução real.
  • Findings minor: SendEmailInput não é reexportado pelo pacote (src/index.ts); a documentação de operações no README não inclui messenger.sendEmail(). Deixei os detalhes e correções nas conversas de linha. Sugestão: abrir follow-ups para export e documentação.
  • CI: build, testes e compatibilidade Node 18/22 passaram neste SHA. Sonar não verificado.

Decisão: o PR já foi mergeado. Não encontrei impedimento ao merge; antes de considerar a funcionalidade operacional, confirme a publicação do pacote e faça smoke test com as permissões reais do token.

Na última consulta, o job de publicação ainda estava em andamento; não confirmei que a versão 0.2.10 chegou ao npm.

Para uma nova análise depois de mudanças, comente /gandalf no PR ou me chame no Slack.

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.

Release 0.2.10 com o sdk-core 0.2.14

2 participants