Skip to content

feat(web) (4/5): skill authoring workbench - #3

Merged
zjncs merged 0 commit into
feat/authoring-fix-publishfrom
feat/authoring-web
Oct 14, 2026
Merged

zjncs merged 0 commit into
feat/authoring-fix-publishfrom
feat/authoring-web

Conversation

@zjncs

@zjncs zjncs commented Oct 10, 2026

Copy link
Copy Markdown
Owner

Layer (d) of the authoring platform split (iflytek#934 on iflytek/skillhub), stacking on layers (a)–(c) (fork PRs until iflytek#935 lands upstream).

What this layer adds

The web workbench under /dashboard/authoring:

  • Draft list + detail (files / runtime binding / validation runs tabs) with a file editor that tracks unsaved state and surfaces optimistic-concurrency conflicts
  • Runtime binding form switching fields by agent type (interpreter vs endpoint/model), tool allowlist, MCP server declarations with immediate validation
  • Run detail: status card, live event console (SSE with a 2s polling fallback and cursor replay), finding cards with severity, location, suggested-fix diff preview and one-click apply/dismiss
  • Submit dialog handing validated drafts to the publish pipeline
  • Copy in en/zh/ru; Playwright e2e for the full flow (e2e/authoring-flow.spec.ts)

Verification

  • tsc --noEmit clean; 922 web unit tests pass — the only failures in the suite are two pre-existing upstream theme tests that fail identically on plain main (verified in a clean worktree), unrelated to this change
  • Unit tests cover the editor's binary-file handling, SSE event merging, patch preview and the validation-stream hook

Copilot AI balanced review requested due to automatic review settings October 10, 2026 06:59

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from 73871fd to 6fe094f Compare October 12, 2026 03:27
@zjncs
zjncs force-pushed the feat/authoring-web branch from f6fc4ae to 0486386 Compare October 12, 2026 03:29
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from 6fe094f to d47fa5a Compare October 13, 2026 13:03
@zjncs zjncs closed this Oct 13, 2026
@zjncs
zjncs force-pushed the feat/authoring-web branch from 0486386 to d47fa5a Compare October 13, 2026 13:03
@zjncs zjncs reopened this Oct 13, 2026
@zjncs zjncs closed this Oct 14, 2026
@zjncs
zjncs force-pushed the feat/authoring-web branch from 7b8245d to 921dc03 Compare October 14, 2026 14:22
@zjncs
zjncs merged commit 921dc03 into feat/authoring-fix-publish Oct 14, 2026
6 of 10 checks passed
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from 42bfae1 to 921dc03 Compare October 14, 2026 14:22
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