Skip to content

feat(auth): instrui onde gerar o PAT no login interativo#9

Merged
djalmaaraujo merged 2 commits into
mainfrom
djalma1/lin-6317
Jul 17, 2026
Merged

feat(auth): instrui onde gerar o PAT no login interativo#9
djalmaaraujo merged 2 commits into
mainfrom
djalma1/lin-6317

Conversation

@djalmaaraujo

Copy link
Copy Markdown
Collaborator

Issue relacionada

LIN-6317 — lk auth login: instruir onde gerar o PAT

Descrição

O lk auth login interativo mostrava só Token: , sem dizer onde gerar o token. Agora:

  • Prompt interativo (sem --token/LK_TOKEN): imprime no stderr a URL do ambiente ativo (<base_url>/srm_settings/access_tokens) e o passo a passo com os rótulos reais da UI — clicar em "Novo token", confirmar em "Criar token", copiar o segredo na modal "Copie seu token agora" (exibido uma única vez) e voltar ao terminal pra colar. stdout continua só com dados (contrato JSON intacto).
  • Erro de token malformado aponta a mesma URL.
  • --help do auth login ganhou Long documentando o fluxo, a precedência --token > LK_TOKEN > prompt e o formato lkn_.
  • Com --token ou LK_TOKEN, nenhum ruído extra (comportamento atual preservado).

Rótulos da UI verificados no backend em app/views/srm_settings/access_tokens/.

Antes / Depois

Antes Depois
Token: Instruções com URL clicável + passo a passo, depois Token:
token does not look like a Linkana PAT (expected lkn_...) mesmo erro + ; generate one at <base_url>/srm_settings/access_tokens

Como testar

  1. make build
  2. ./lk auth login (sem flag/env) → deve mostrar as instruções com a URL do ambiente ativo antes do Token: ; cole um lkn_... e confirme o saved.
  3. ./lk auth login --token lkn_x_y → sem instruções no stderr.
  4. ./lk auth login --token invalido → erro cita a URL.
  5. ./lk auth login --help → seção explicando onde gerar o token.

Testes: make test e make cover verdes (95,1% ≥ 95%), make lint limpo.

🤖 Generated with Claude Code

@djalmaaraujo
djalmaaraujo requested a review from cirdes July 16, 2026 12:57
@djalmaaraujo
djalmaaraujo merged commit d061c5d into main Jul 17, 2026
3 checks passed
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.

2 participants