Repository navigation
chore(deps): bump @hono/node-server and prisma in /app/migrator - #134
dependabot[bot] wants to merge 1 commit into
Conversation
Removes [@hono/node-server](https://github.com/honojs/node-server). It's no longer used after updating ancestor dependency [prisma](https://github.com/prisma/prisma/tree/HEAD/packages/cli). These dependencies need to be updated together. Removes `@hono/node-server` Updates `prisma` from 7.8.0 to 7.9.1 - [Release notes](https://github.com/prisma/prisma/releases) - [Commits](https://github.com/prisma/prisma/commits/7.9.1/packages/cli) --- updated-dependencies: - dependency-name: "@hono/node-server" dependency-version: dependency-type: indirect - dependency-name: prisma dependency-version: 7.9.1 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
guifav
left a comment
There was a problem hiding this comment.
@dependabot review da Daisy — commit 41c20485
Resumo: Bump @hono/node-server e prisma 7.8.0→7.9.1 em /app/migrator. CI vermelho no step "Verify third-party license inventory" do lint-test-build — o bump altera a árvore de deps sem regenerar docs/THIRD-PARTY-LICENSES.md (gate documentado deste repo desde 2026-07-13; mesma causa dos bloqueios de #70/#72/#73/#74/#75/#93).
Contexto considerado: diff do PR, checks do head, memory/talkwithdata.md (gate de inventário de licença) | Cross-project: nenhum | Notion: sem projeto correlacionado
Validacao local:
- install: n/a (review via CI + diff)
- tests: n/a
- lint: n/a
- tsc: n/a
- build: CI fail — step
Verify third-party license inventory(demais jobs verdes: browser-e2e, docker-image-smoke, functions-generate-thumbnail)
Bloqueadores (P1):
- Inventário de licenças stale quebra o gate de CI. Regenerar no branch:
cd app && node ../scripts/generate-third-party-licenses.mjs(ou caminho equivalente ao do script no repo), commitardocs/THIRD-PARTY-LICENSES.mdatualizado e empurrar — dependabot não faz isso sozinho. Sem CI verde, não mergear (regra deste repo).
Veredito: bloqueado
roger-guifav
left a comment
There was a problem hiding this comment.
TL;DR
Atualização do prisma de 7.8.0 para 7.9.1 (minor, dentro do major 7) em app/migrator, com reescrita grande do lockfile. A instalação e a CLI do prisma funcionam, mas o documento de licenças de terceiros ficou desatualizado — é por isso que a checagem principal do CI está vermelha — e o lockfile novo prende o fast-uri numa versão intermediária que ainda tem correção de segurança pendente em outro PR aberto. Não mergear sem esses ajustes.
Verdict: Ajustes necessários
(revisado no head SHA 41c2048)
Parecer
Bump de dependência gerado por automação no diretório app/migrator, pacote que só carrega a CLI do prisma para rodar migrate deploy em container. Três fatos fora do diff orientam o parecer: (1) o repo já viveu incidente de par prisma client/CLI dessincronizado, então conferi a matriz de versões em todos os package.json — aqui não há client, só CLI, e o skew de major não se aplica; (2) o prisma 7.9.1 não depende mais de hono node-server, então o bump do título na prática remove esse pacote da árvore e deixa o override de segurança sem alvo; (3) o fast-uri 3.1.7, alvo de PRs irmãos, fecha advisories públicos de segurança, e este lockfile prende 3.1.5. Validação local: npm ci limpo e CLI 7.9.1 operante. Pergunta de merge: eu mergearia depois de regenerar o documento de licenças e alinhar a ordem com o PR do fast-uri no mesmo diretório.
Críticos (P0)
Nenhum.
Bloqueadores de correção (P1)
- Documento de licenças de terceiros desatualizado: a checagem lint-test-build do CI reprova indicando que docs/THIRD-PARTY-LICENSES.md está stale (o lockfile mudou prisma, fast-uri e dezenas de transitivos, e o documento fixa versões antigas). Aplicar a regeneração pelo script do repo e commitar o resultado neste PR antes do merge.
Endereçar no PR (P2)
- Sequência de merge com o PR 141: este lockfile prende fast-uri em 3.1.5, abaixo do 3.1.7 que fecha advisories de segurança. Se este PR mergear depois do 141 sem rebase, há risco de regressão silenciosa da correção; se mergear antes, o 141 precisa de rebase. Fixar a ordem: mergear o 141 primeiro e rebasear este, ou garantir que o lock final fique em 3.1.7 ou superior.
Endereçar no PR (P3 / nits)
- Override de hono node-server 1.19.13 ficou sem alvo: o prisma 7.9.1 não puxa mais esse pacote, então o pin de segurança virou configuração morta em app/migrator/package.json. Remover o override no mesmo PR (duas linhas).
Acoes que somente o humano pode fazer (ainda...)
Nenhuma além da decisão de ordem de merge entre os PRs de bump.
Verificado
- Head revisado: 41c2048 (merge-base igual ao main atual, sem base velha)
- npm ci em app/migrator: limpo (135 pacotes)
- CLI prisma 7.9.1 executa; o migrator não tem código próprio, só runtime de migrate
- Matriz prisma no repo: app declara client ^7.8.0 e CLI ^7.9.1; migrator fixa CLI 7.9.1 — mesmo major 7 nos pontos relevantes, sem skew de major
- CI: lint-test-build reprova apenas por documento de licenças stale; browser-e2e, docker-image-smoke e functions-generate-thumbnail passam
- Migrator não tem suite de testes nem typecheck próprio — limite declarado da validação local
Roger (Hermes KVM) - review v2 - 41c2048
Removes @hono/node-server. It's no longer used after updating ancestor dependency prisma. These dependencies need to be updated together.
Removes
@hono/node-serverUpdates
prismafrom 7.8.0 to 7.9.1Release notes
Sourced from prisma's releases.
... (truncated)
Commits
3f492abchore(deps): update@prisma/devto 0.24.17 (7.9.x backport) (#29795)537583cfeat(cli): add@bomb.sh/tabcompletions (#28351)bf7c40ffix(cli): update Studio to 0.33.0 and suppress aborted response errors (#29720)9acd8dcfeat(cli): install Prisma agent skills during prisma init (#29689)aef8cb5fix(cli): remove the migrate-reset tool from the MCP server (#29691)cdca333chore(deps): update@prisma/devto 0.24.14 (#29686)fda8fbcchore(deps): bump@prisma/devto 0.24.9 (resolves hono GHSA-92pp-h63x-v22m) (...Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.