Skip to content

feat: job logs - #1

Merged
rawen554 merged 1 commit into
cloud-ru:mainfrom
vlad-golang:job-logs
Aug 13, 2026
Merged

rawen554 merged 1 commit into
cloud-ru:mainfrom
vlad-golang:job-logs

Conversation

@vlad-golang

Copy link
Copy Markdown

No description provided.

@rawen554

Copy link
Copy Markdown
Collaborator
  1. Регрессия поведения (подтверждено) — cmd/job.go:145. Раньше logs держал SSE-стрим открытым до завершения job'а (это же прямо описано в CLAUDE.md как основной агентский сценарий отслеживания деплоя). Теперь — один постраничный запрос, по умолчанию 10 строк, без индикации, что лог обрезан и job ещё бежит. Для running job вывод будет молча неполным.
  2. Игнорирует конвенцию table/--json — cmd/job.go:158. Все соседние команды в этом же файле (newJobShowCmd и т.д.) проверяют ctx.Printer.Format == output.FormatJSON и иначе рендерят таблицей. logs теперь всегда дампит сырой JSON, даже на TTY без --json.
  3. Битый Makefile-таргет — Makefile:182. make openapi-generator ссылается на openapi-public.yaml, которого нет ни в PR, ни в репо — таргет не воспроизводим "из коробки".
  4. Дублирование клиента / архитектурный дрейф — internal/workflow_client. Ради одного эндпоинта заведён параллельный generated-клиент вместо расширения internal/workflowapi (единственного клиента, документированного в CLAUDE.md). workflowapi.StreamJobLogs остаётся мёртвым кодом. Плюс сгенерированные имена файлов протаскивают в публичный репо внутренний путь апстрима (git_sbercloud_tech_dsworks_services_pipeline_src_...) — именно то, от чего проект аккуратно избавлялся при ребрендинге в eds/cloud.ru.
  5. Документация не обновлена — README.md и skill/SKILL.md не тронуты, хотя поведение и флаги команды поменялись существенно. CLAUDE.md прямо требует синхронного обновления обоих в таком случае, а skill/SKILL.md — основной контракт для агентов, которые будут ожидать старого стримингового поведения.

@vlad-golang

Copy link
Copy Markdown
Author
  1. Логи можно смотреть через новую ручку, итерация есть
  2. Утилита делается для агента, агенту таблица не нужна
  3. openapi-public.yaml не должен находиться в публичном репозитории
  4. Генерировать клиент эффективнее чем писать вручную, ручной удалю позже
  5. В скиле не надо все команды перечислять, надо исправить позже. Сейчас сделал быстрое исправление списка команд в скиле

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