schemas update: confirm step (--digest/--confirm) returns ok:true but never persists the change

Aberta
  • Algo errado?
#7 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
65/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
cli, shell
Domínio
backend-api-design, cli

Direção de pesquisa

Look at the schemas update command implementation, likely in a file like src/commands/schemas/update.rs or similar. The bug is in the confirm step where the digest is used but the change is not persisted. Start by tracing the flow from the --confirm flag, checking the API call made and the response handling. Verify the actual write operation to the CRM. Run the provided reproduction steps to see the issue firsthand.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Version: hubspot 0.14.0 (build 1162, commit unknown)

Command run:

$ PATCH='{"labels":{"singular":"ZZ Push Test","plural":"ZZ Push Tests MANUAL"}}'
$ DIGEST=$(echo "$PATCH" | hubspot schemas update --type 2-253815111 --dry-run | jq -r '.digest')
$ echo "$PATCH" | hubspot schemas update --type 2-253815111 --digest "$DIGEST" --confirm 'zz_push_test'

Expected result: the schema's labels.plural is updated to "ZZ Push Tests MANUAL", matching what the confirm call's own response claims to have applied.

Actual result: the confirm call returns ok: true with a data payload that shows the new value, but the change is never written. Re-fetching immediately after (and repeatedly for 25s afterward) shows the old value unchanged:

$ echo "$PATCH" | hubspot schemas update --type 2-253815111 --digest "$DIGEST" --confirm 'zz_push_test'
{"name":"2-253815111","ok":true,"data":{"labels":{"singular":"ZZ Push Test","plural":"ZZ Push Tests MANUAL"},...}}

$ hubspot schemas get --type 2-253815111 --format json | jq '.data[0].labels'
{"singular":"ZZ Push Test","plural":"ZZ Push Tests"}

Reproduced twice independently (once via a wrapper script, once with the raw commands above in isolation, no other operations in between).

Why this matters: the README's own beta warning recommends using --dry-run before mutations as the safety practice for a tool that can modify live CRM data. This defeats that practice for schemas update specifically - the confirm step that --dry-run leads into silently no-ops while reporting success.

Also noticed while investigating: the dry-run response's apply_command_hint field (e.g. hubspot schemas update 2-253815111 --digest blast-... --confirm 'zz_push_test') omits --file/stdin, which is misleading on its own - the confirm call does require the same JSON body piped again, or it fails with "No schema body provided".

Linguagem predominante
Sem dados de linguagem
Estrelas
7
Forks
2
Nenhum PR com merge em 30d

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de HubSpot/agent-cli

Todas as issues de HubSpot/agent-cli

Issues semelhantes

Mais issues de Backend & API Design

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.