Repository navigation
chore(deps): bump @hono/node-server and prisma in /app - #135
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:development ... Signed-off-by: dependabot[bot] <support@github.com>
guifav
left a comment
There was a problem hiding this comment.
@dependabot review da Daisy — commit 9ec68980
Resumo: Bump @hono/node-server e prisma 7.8.0→7.9.1 em /app (transitivo: @electric-sql/pglite 0.4.1→0.4.3). 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) em app, com reescrita do lockfile. Os gates locais passam todos (typecheck, 680 testes, lint), mas o documento de licenças ficou desatualizado — causa da reprovação no CI —, o prisma client ficou uma minor atrás da CLI, e o lockfile prende o fast-uri em 3.1.5, abaixo da versão que fecha advisories de segurança em PR irmão. Não mergear sem esses ajustes.
Verdict: Ajustes necessários
(revisado no head SHA 9ec6898)
Parecer
Bump de dependência gerado por automação no diretório app, o pacote principal (Next + prisma). Fatos fora do diff que orientam: (1) o repo já viveu incidente de par prisma client/CLI dessincronizado — aqui o client resolve 7.8.0 e a CLI 7.9.1; mesmo major, sem quebra, mas o generate materializa client 7.9.1 em src/generated enquanto o runtime instalado é 7.8.0, e o histórico manda o par andar junto; (2) o prisma 7.9.1 não depende mais de hono node-server, então o override de segurança ficou sem alvo; (3) o lockfile novo prende fast-uri em 3.1.5, abaixo do 3.1.7 que o PR 139 aplica neste mesmo diretório. Validação local completa e verde. Pergunta de merge: mergearia depois do documento de licenças regenerado, do client alinhado e da ordem com o 139 definida.
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 docs/THIRD-PARTY-LICENSES.md stale (prisma, fast-uri e dezenas de transitivos mudaram de versão e o documento fixa as antigas). Aplicar a regeneração pelo script do repo e commitar o resultado neste PR antes do merge.
Endereçar no PR (P2)
- Par prisma dessincronizado: client instalado 7.8.0 contra CLI 7.9.1. Sem quebra de major, mas o generate já produz código 7.9.1 — subir o client para ^7.9.1 no mesmo PR (duas linhas) e regerar o lock.
- Sequência de merge com o PR 139: este lockfile prende fast-uri em 3.1.5, abaixo do 3.1.7 que fecha advisories de segurança. Se mergear depois do 139 sem rebase, regride a correção; se antes, o 139 precisa de rebase. Garantir lock final 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): pin de segurança virou configuração morta em app/package.json. Remover no mesmo PR.
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: 9ec6898 (merge-base igual ao main atual)
- npm ci em app: limpo (957 pacotes)
- prisma generate: client 7.9.1 gerado em src/generated
- tsc --noEmit: 0 erros
- vitest: 680 passando, 8 skipped, 0 falhas (66 arquivos)
- eslint projeto inteiro: limpo, exit 0
- CI: lint-test-build reprova apenas por documento de licenças stale; browser-e2e, docker-image-smoke e functions-generate-thumbnail passam
Roger (Hermes KVM) - review v2 - 9ec6898
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.