Skip to content

task-c3-report.md

Latest commit

 

History

History
159 lines (127 loc) · 8.61 KB

task-c3-report.md

File metadata and controls

159 lines (127 loc) · 8.61 KB
Raw

Task C3 — requirements no repeat do Conductor + foco na task nova + teste de auth

Branch feat/orch-fase3-c · commit a7d430fe9ee3ae1fd1d86ff0083318fc1e96005b (fix(dashboard): carry conductor requirements and focus the repeated task, corpo com Refs #12639).

O que mudou, por item

1. src/lib/conductor/hubProxy.ts — requirements no schema de detalhe

  • hubTaskSchema ganhou requirements: z.object({ cli: z.string().nullish(), model: z.string().nullish() }).nullish().catch(null). O .catch(null) é deliberado: o MESMO schema parseia a lista em getFleetSnapshot (z.array(hubTaskSchema).parse(rawTasks)) — sem ele, um requirements de formato inesperado numa única task faria o parse inteiro lançar e o snapshot cair para {offline: true}.
  • ConductorTaskDetail ganhou cli: string | null e model: string | null (com JSDoc); getConductorTaskDetail preenche com t.requirements?.cli ?? null / ?? null.
  • Origem do shape: é exatamente o objeto que createConductorTask (mesmo arquivo, linhas ~238-241) envia para o hub (body.requirements = { cli?, model? }) — não foi inventado.

2. drawer/useDrawerDetail.ts

  • repeatReqForConductor agora envia cli: d.cli ?? undefined / model: d.model ?? undefined. undefined, nunca null — o Zod real da rota (src/app/api/conductor/tasks/route.ts) tem cli: z.string().optional() / model: z.string().optional(), e null daria 400. JSON.stringify descarta as chaves undefined, então o corpo sai idêntico ao de hoje quando não há requirement. Os dois campos são independentes (um pode existir sem o outro) — há teste para esse caso.
  • jsonRpcErrorCode deixou de ler res.json() e passou a receber o corpo já parseado; novo readJsonBody(res) lê o corpo UMA vez (um body só pode ser consumido uma vez, e agora o id da task nova sai do mesmo corpo). Body não-JSON/vazio ⇒ null, sem virar falha (mantém o contrato anterior: "corpo ilegível não é evidência de falha, o status já valeu").
  • performAction passou a devolver { ok, body } em vez de boolean.
  • Novo export function newNodeIdFrom(node, body): string | null — extrai o id da task criada e devolve já no formato do canvas (com prefixo), porque setParams({ node }) casa contra snapshot.nodes.find(n => n.id === nodeId) e os ids em mergeSnapshot.ts são prefixados:
    • conductor: body.task_id → conductor:task:<id>
    • cloud-agent: body.data.id → cloud-agent:<id>
    • a2a: body.result.task.id → a2a:<id> (envelope real de /a2a message/send v0.3) Leitura defensiva: id ausente/vazio/não-string ⇒ null (nunca um prefixo pelado), corpo não-objeto ⇒ null, nunca lança.
  • No hook: approve/cancel continuam Promise<boolean> (via runBooleanAction) — de propósito: eles agem sobre a task já aberta e a resposta do cloud-agent pode ecoar o id da MESMA task, o que faria o painel "navegar" para onde já está. Só repeat devolve RepeatOutcome { ok, newNodeId } (interface exportada).

3. drawer/OrchestrationDrawer.tsx

  • RepeatButton consome RepeatOutcome e chama onActionDone(newNodeId ?? undefined).
  • onActionDone virou (newNodeId?: string) => void no RepeatButton, DrawerActions e no componente exportado. DrawerActions.run() (approve/cancel) segue chamando onActionDone() sem argumento.

4. OrchestrationPageClient.tsx

  • Novo onActionDone memoizado: refetch() e, se veio id, setParams({ node: newNodeId }). A aba History renderiza o próprio drawer (HistoryTab.tsx) e não foi tocada — seu onActionDone={() => …} ignora o argumento extra, então o comportamento lá continua idêntico (re-amostra nowMs, não navega), como o plano pede.

5. Testes

  • tests/unit/conductor-routes-auth.test.ts: src/app/api/conductor/tasks/route.ts entrou no array ROUTES; o regex do proxy ganhou a alternativa createConductorTask\(.
  • tests/unit/conductor-create-route.test.ts: removido o teste de fonte duplicado (route: requireManagementAuth antes de criar a task no hub), substituído por um comentário apontando para o array. fs/path continuam usados pelo resto do arquivo.
  • tests/unit/ui/orchestrationDrawerRepeat.test.tsx (786 → 971 linhas, cap 1200 — não precisou dividir): 3 describes novos, 6 testes:
    • repeatReqFor conductor com cli/model; com ambos null (corpo idêntico ao atual e sem as strings cli/model); com só cli setado.
    • newNodeIdFrom nas três fontes (prefixado), corpos malformados ⇒ null, fonte sem contrato ⇒ null.
    • drawer: repeat de conductor faz POST com cli/model e chama onActionDone exatamente uma vez com "conductor:task:t_new", com o toast repeatDone.

TDD — evidência

C3.2 (vermelho) npx vitest run tests/unit/ui/orchestrationDrawerRepeat.test.tsx:

 Test Files  1 failed (1)
      Tests  6 failed | 29 passed (35)

(falhas: newNodeIdFrom is not a function ×3, corpo sem cli/model ×3 — a última: expected { …(4) } to match object { cli: 'claude', model: 'sonnet' })

node --import tsx/esm --test tests/unit/conductor-routes-auth.test.ts tests/unit/conductor-create-route.test.ts já passou nesse ponto (ℹ pass 11 / fail 0) — divergência menor com o plano: a mudança do array ROUTES é ampliação de cobertura + de-duplicação, não a prova de um defeito, então ela não tinha como "falhar antes". A rota já chamava requireManagementAuth corretamente.

C3.4 (verde):

$ npx vitest run tests/unit/ui/orchestrationDrawerRepeat.test.tsx
 Test Files  1 passed (1)
      Tests  35 passed (35)

$ node --import tsx/esm --test tests/unit/conductor-routes-auth.test.ts tests/unit/conductor-create-route.test.ts tests/unit/client-bundle-no-server-only-10692.test.ts
✔ src/app/api/conductor/tasks/route.ts: requireManagementAuth antes do proxy ao hub (4.175999ms)
ℹ tests 12
ℹ pass 12
ℹ fail 0

$ npx vitest run tests/unit/ui/
 Test Files  210 passed (210)
      Tests  1200 passed (1200)
   Duration  179.85s

$ node --import tsx/esm --test tests/unit/conductor-hub-proxy.test.ts tests/unit/conductor-fleet-mirror.test.ts tests/unit/conductor-fleet-route.test.ts tests/unit/conductor-delegate.test.ts
ℹ tests 17
ℹ pass 17
ℹ fail 0

Gates

$ npm run typecheck:core
EXIT=0   (sem saída de erro)

$ npm run check:dashboard-typecheck
[dashboard-typecheck] OK — 206 pre-existing error(s), all within frozen baseline.
EXIT=0

$ npx eslint <os 7 arquivos alterados>
EXIT=0   (zero erros, zero warnings)

$ npm run check:file-size
[file-size] OK — 136 arquivos congelados, cap 1200 para novos (4588 arquivos verificados)
[test-file-size] OK — 40 test files congelados, testCap 1200 para novos (5495 test files verificados)

Hooks do husky rodaram no commit (lint-staged + docs-sync + any-budget + tracked-artifacts, todos PASS); nada de --no-verify.

Divergências entre plano e código real

  1. newNodeIdFrom prefixado (não previsto explicitamente no plano). O plano diz "extrair o id da task nova (task_id no Conductor, data.id no Cloud Agent) e chamar onActionDone com ele". O id CRU não serve para setParams({ node }): o canvas indexa por id prefixado (conductor:task:<id>, cloud-agent:<id>, a2a:<id> — mergeSnapshot.ts / routeFor). Segui o código: newNodeIdFrom já devolve o id de canvas. O plano não citava o a2a; incluí porque o repeat do drawer também atende a2a e o envelope (result.task.id) é verificável em src/app/a2a/route.ts:250.
  2. performAction "devolve o corpo parseado" — só o repeat propaga o id. approve/cancel ficaram Promise<boolean> de propósito (ver item 2 acima): o POST de approve/cancel do cloud-agent pode devolver o id da MESMA task, e navegar para ela seria ruído. Cumprimento do espírito do plano ("o drawer extrai o id da task nova") sem o efeito colateral.
  3. .catch(null) no requirements. O plano não pedia; adicionei porque hubTaskSchema também parseia a LISTA de tasks, e um campo novo estrito ali seria um vetor de "snapshot vira offline por causa de um campo estranho de uma task".
  4. A mudança em conductor-routes-auth.test.ts não é um teste que falha antes (explicado acima).
  5. ConductorTaskDetail estende FleetTask; cli/model foram adicionados só no detalhe (não em FleetTask/toFleetTask), porque o repeat lê o DETALHE carregado, não o nó.

Fora de escopo (não feito, é da Task C4)

changelog.d/fixes/orchestration-followups.md, o comentário na issue #12639 e a bateria completa (lint total, test:vitest, check:cycles, i18n) continuam pendentes na C4.