Skip to content

feat: lk ganha os 11 comandos de documento do fornecedor - #78

Merged
djalmaaraujo merged 4 commits into
mainfrom
djalma1/lin-8122-manifest
Oct 28, 2026
Merged

djalmaaraujo merged 4 commits into
mainfrom
djalma1/lin-8122-manifest

Conversation

@djalmaaraujo

Copy link
Copy Markdown
Collaborator

Issue relacionada

Parte B da LIN-8122

Descrição

O lk passa a ter os 11 comandos de documento do fornecedor que o linkana já expõe: supplier document list, document review, revert-review, request-update, cancel-update, expires-at, renew-at e os quatro overrides com arquivo (form-override, certificate-override, check-override, bank-information-override). Nenhum código novo: é o manifest re-vendorizado da main do linkana, e o SURFACE.txt só ganha linhas.

O TestLoadEmbeddedManifest exigia identity show como primeiro comando. Com document ... na frente em ordem alfabética, isso quebrou, então troquei pra busca por nome, igual o resto do teste já fazia.

README e CLAUDE.md ainda falavam de supplier como comando escrito à mão. Agora dizem que todo comando de recurso vem do manifest e citam o lk upload.

Depois do merge falta o release (v0.21.0), que eu só faço com ok.

Instruções de teste

  1. make test, make lint e make cover.
  2. make build e, com LK_API_URL apontando pra um linkana local na main: ./lk supplier document list <supplier_id>, ./lk upload --file x.docx (content type do Word, não zip) e ./lk document certificate-override <id> --state not_clear --documentable_attributes '{"file":"<signed_id>"}'.
Verificação
  • make test verde, make lint 0 issues, make cover 95,5%.
  • SURFACE.txt: 13 linhas novas (os 11 comandos e os grupos lk document e lk supplier document), nenhuma removida.
  • E2E com o binário da branch contra o linkana local na main (4197aa4b8f), banco isolado com dado fictício, PAT de usuário fictício no lugar do lk impersonate:
    • supplier document list: 200 com os ids, 404 com fornecedor de fora.
    • lk upload: .docx subiu como application/vnd.openxmlformats-officedocument.wordprocessingml.document, .pdf como application/pdf.
    • document certificate-override com o .docx: 200 com versão nova; --state informative deu 422.
    • document review: 200; de novo no mesmo documento deu 422 ("Documento não está aguardando revisão.").
    • document revert-review: 200 (volta pra ready_to_review); em documento não revisado deu 422.

@djalmaaraujo djalmaaraujo self-assigned this Oct 28, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 5 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread CLAUDE.md Resolved
Comment thread CLAUDE.md Resolved
Comment thread README.md Resolved

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

0 issues found across 2 files (changes from recent commits).

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Requires human review: Adds 11 supplier document commands to the lk CLI by re-vendoring the linkana manifest; this expands the public CLI surface to sensitive review/override operations with backend side effects, so a human should approve the exposure decision.

Turn on auto-fix | Re-trigger cubic

@djalmaaraujo
djalmaaraujo merged commit 256a86f into main Oct 28, 2026
4 checks passed
@djalmaaraujo
djalmaaraujo deleted the djalma1/lin-8122-manifest branch October 28, 2026 20:30
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.

1 participant