Skip to content

Tasks/studio UI - #8

Open
techbuzzz wants to merge 8 commits into
perfomance-enchfrom
tasks/studio-ui
Open

techbuzzz wants to merge 8 commits into
perfomance-enchfrom
tasks/studio-ui

Conversation

@techbuzzz

Copy link
Copy Markdown
Owner

Описание

Тип изменения

  • 🚀 Новая функциональность (feat)
  • 🐛 Исправление ошибки (fix)
  • 📝 Документация (docs)
  • ♻️ Рефакторинг (refactor)
  • 🔧 Прочее (chore / build / ci)

Чек-лист

  • Проект собирается: dotnet build Hercules.slnx
  • Фронтенд собирается (если затронут): cd hercules-web && npm run build
  • Обновлена документация (README.md / docs/) при изменении поведения
  • Коммиты соответствуют Conventional Commits
  • Секреты не попали в репозиторий

Finalize the first release off this branch. The Electron shell is replaced by
the agent serving the Studio SPA itself (ADR-0009), which removes a runtime
surface existing installations depend on — hence the major bump from the
documented-but-untagged 1.0.0.

Version identity (owner decision, explicit confirmation):
- CHANGELOG-EN/RU: restructured to Keep a Changelog order. The previous
  "[Unreleased]" block held shipped, tested work (Phase 4, v2, Phase 8) and is
  now folded into 2.0.0; the new release section sits above the historical
  1.0.0 instead of below it. 1.0.0 is annotated as never tagged.
- Directory.Build.props: single <Version>2.0.0</Version> for the assemblies.
  Imported before each csproj body, so SkillSdk keeps its independent
  0.4.0-preview NuGet lineage.
- package.json: 0.2.0 -> 2.0.0.

Fix found while re-verifying before tagging:
- GraphEditor.vue: setEdge was typed "from" | "to" but the edge-label input
  called it with "label", breaking `npm run typecheck`. Widened to
  keyof WorkflowEdge. The illegal v-model="graph.edges[index].label" had
  already been replaced by :value + @input under noUncheckedIndexedAccess;
  this finishes that change rather than reverting it.

Verified on this commit:
- dotnet build src/agent/Hercules.slnx -c Release: 0 warnings, 0 errors
- agent tests: 2276 passed, 0 failed
- Studio typecheck: 0 errors; biome lint: 81 files clean
- Studio unit: 97 passed; E2E (Playwright): 30 passed
- vite build OK

Not in this release, still open:
- task_105 (workflow executor + typed WorkflowGraph) — no node definitions or
  named owner; declined repeatedly rather than invented.
- Stage 6.10 withdrawal — provisional, awaiting explicit owner confirmation.
Follow-up to the 2.0.0 release commit, found while validating the rewritten
changelogs against the repo's own guards.

stage_06_config_restart.md — the `**Status:**` banner asserted 6.10's C# tab
"is withdrawn", flatly. The body of the same file already documented that the
withdrawal rests on a timeout auto-selection (`explicitUserConfirmation: false`)
and is therefore provisional. The banner is the line anyone reads first, so the
unauthorised claim was the one that propagated; it now says "withdrawn
provisionally, not by owner decision" and points at 6.10. Also dropped a stray
trailing `|` on that line.

count-stage-tasks.cjs — printed a bare "TOTAL open=345", which reads as a
backlog. It is not one: none of these ten stage files had their checkboxes
ticked during the web-first migration, and every file carries an explicit
banner saying so. That figure had already been mis-cited once as outstanding
work. The script now prints each file's authoritative `**Status:**` line, labels
the box count as "not a progress report", and exits non-zero if any file loses
its status line or its disclaimer — turning a misleading counter into a
consistency guard.

No product code changed. Guards after the edit: 841 doc links resolve, 117
backlog rows consistent, all 10 stage files carry status + disclaimer, ci.yml
structurally sound.

This branch has not been deployed

No deployments
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